jasig-ssp IRC Logs-2013-02-12
[12:07:57 CST(-0600)] <prabhu> hi
[12:08:28 CST(-0600)] <prabhu> I have a question on SSP portlet installation
[12:08:45 CST(-0600)] <dmccallum54> shoot
[12:09:03 CST(-0600)] <prabhu> How install portlet in SSP
[12:10:12 CST(-0600)] <prabhu> How to install portlet in SSP
[12:11:03 CST(-0600)] <dmccallum54> it should be the same process as for a vanilla uPortal: https://wiki.jasig.org/display/UPM40/Deploy+and+Publish+a+Portlet
[13:10:38 CST(-0600)] <TonyUnicon> org.jasig.mygps.business.StudentIntakeFormManager = 2861 line class
[13:10:43 CST(-0600)] <TonyUnicon>
[13:10:57 CST(-0600)] <TonyUnicon> 2681*
[13:12:30 CST(-0600)] <TonyUnicon> this class is aweful
[13:25:26 CST(-0600)] <TonyUnicon> Jason it looks like all the fields mentioned in 671 have been implemented but are wrapped in dependencies
[13:26:47 CST(-0600)] <TonyUnicon> Do we have any information besides code comments that outline what these actually are?
[13:42:47 CST(-0600)] <cwaymire> any of you built ssp-master lately?
[13:46:56 CST(-0600)] <dmccallum54> what's the error
[13:48:18 CST(-0600)] <cwaymire> compilation fails on the test side with:
[13:48:19 CST(-0600)] <cwaymire> [artifact:mvn] [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:2.3.2:testCompile (default-testCompile) on project platform-war: Compilation failure
[13:48:19 CST(-0600)] <cwaymire> [artifact:mvn] [ERROR] /home/jasig/git/uPortal/uportal-war/src/test/java/org/jasig/portal/rdbm/TransientDatasource.java:[41,7] error: TransientDatasource is not abstract and does not override abstract method getParentLogger() in CommonDataSource
[13:52:17 CST(-0600)] <dmccallum54> what version of java
[13:52:48 CST(-0600)] <dmccallum54> that method was added in 1.7
[13:52:59 CST(-0600)] <dmccallum54> you'll need 1.6
[13:54:04 CST(-0600)] <cwaymire> yeah. I had the system set for 1.6 but my JAVA_HOME was still pointing to 1.7
[13:54:07 CST(-0600)] <cwaymire> re-running it now
[14:14:52 CST(-0600)] <JasonElwood> Tony- I'll look into the comment you put in 671. I don't have any documentation related to the intake in MyGPS.
[14:16:03 CST(-0600)] <TonyUnicon> there is also another issue with Military Affiliations
[14:16:27 CST(-0600)] <TonyUnicon> you probably only want that to show if the person is a veteran
[14:16:41 CST(-0600)] <JasonElwood> Just to be clear. there is veteran's status and military affiliations.
[14:16:52 CST(-0600)] <TonyUnicon> yes im getting to that
[14:16:55 CST(-0600)] <TonyUnicon> the issue is
[14:17:07 CST(-0600)] <TonyUnicon> veteran status also has custom values
[14:17:34 CST(-0600)] <TonyUnicon> from the code you can only have dependencies on hard coded values
[14:18:08 CST(-0600)] <TonyUnicon> meaning there is no way to tell if the person is a veteran because there is no way to tell which statuses actually indicate that they are a veteran
[14:18:19 CST(-0600)] <TonyUnicon> or active military
[14:18:24 CST(-0600)] <TonyUnicon> or DoD
[14:19:15 CST(-0600)] <JasonElwood> the veterans status and military affiliation are two completely separate fields with no connection or dependency between the two. most of the fields/reference items in the student intake a configurable through the admin.
[14:19:15 CST(-0600)] <TonyUnicon> so we can either remove custom values for veteran status
[14:19:26 CST(-0600)] <TonyUnicon> ...
[14:20:01 CST(-0600)] <TonyUnicon> so in other words you want military affiliation to appear as a field even if they say they are no affiliated to the military?
[14:20:15 CST(-0600)] <JasonElwood> That's correct
[14:20:25 CST(-0600)] <JasonElwood> that's how the student intake tool works
[14:20:29 CST(-0600)] <TonyUnicon> it is not
[14:20:45 CST(-0600)] <TonyUnicon> for example
[14:20:53 CST(-0600)] <TonyUnicon> military branch only appears
[14:21:03 CST(-0600)] <TonyUnicon> when the career goal of 'Military' is chosen
[14:21:12 CST(-0600)] <TonyUnicon> thats what I mean by dependencies
[14:21:28 CST(-0600)] <TonyUnicon> there are about a half dozen examples like that
[14:24:13 CST(-0600)] <TonyUnicon> another example
[14:24:14 CST(-0600)] <TonyUnicon> // DEPENDENCY -> childCareArrangementQuestion shown when
[14:24:14 CST(-0600)] <TonyUnicon> // childCareNeededQuestion selection matches "Yes"
[14:24:26 CST(-0600)] <JasonElwood> I see what you are saying. I was only looking at the demographics tab in the intake tool
[14:25:04 CST(-0600)] <JasonElwood> you are looking at intake fields that have additional input if selected
[14:25:26 CST(-0600)] <JasonElwood> this is a third field dealing with military
[14:25:36 CST(-0600)] <TonyUnicon> right
[14:25:44 CST(-0600)] <TonyUnicon> which is only valid if they are in the military
[14:25:56 CST(-0600)] <TonyUnicon> like any other 'additional input' fields
[14:26:02 CST(-0600)] <TonyUnicon> they require context
[14:26:32 CST(-0600)] <JasonElwood> it's only valid if they select the military education/career goal. that is not related to the two fields on the demographics tab
[14:26:41 CST(-0600)] <TonyUnicon> so again
[14:26:45 CST(-0600)] <TonyUnicon> two different things
[14:26:55 CST(-0600)] <TonyUnicon> career goals —> military branch
[14:26:58 CST(-0600)] <TonyUnicon> one dependency
[14:27:10 CST(-0600)] <TonyUnicon> veteran status —> military affiliation
[14:27:14 CST(-0600)] <TonyUnicon> probably should be another
[14:27:35 CST(-0600)] <TonyUnicon> we dont have to do it, it just doens't make much sense
[14:28:06 CST(-0600)] <JasonElwood> that's not how the current tool works though. MyGPS should match what the tool does
[14:28:39 CST(-0600)] <TonyUnicon> ookay
[14:29:02 CST(-0600)] <JasonElwood> the origin of this is that the military affiliation field originally existed in another tool, not student intake. Shawn put it in the intake instead of creating a separate tool.
[14:29:28 CST(-0600)] <TonyUnicon> ok, not sure thats a good reason to confuse the user though
[14:29:30 CST(-0600)] <JasonElwood> so you need from me a full list of fields in the intake that have a dependency
[14:29:47 CST(-0600)] <TonyUnicon> yes that would be great, to verify and validate whats already in there
[14:33:53 CST(-0600)] <TonyUnicon> I'll have to remember the 'lets do what the last tool did' mantra next time we discuss the 'Active' field for the self help guide admin
[14:34:48 CST(-0600)] <JasonElwood> I'm with you man. It's a mixed bag of requirements that don't necessarily make sense.