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> (sad)

[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 (smile)

[14:34:48 CST(-0600)] <JasonElwood> I'm with you man. It's a mixed bag of requirements that don't necessarily make sense.