Grouper Users - Open Discussion List

["Hyzer, Chris" <>]

This is in 2.3.0 ui patch #20.  Thanks Chad!  :)

Ps. if you can apply the patch and make sure everything is ok, that would be 
swell...


Pps. On the demo server I got this error, did you?

Problem applying patch since this patch old file:
  
/opt/grouper/2.3/patchesAuto/tarballs/patches/grouper_v2_3_0_ui_patch_20/old/classes/grouperText/grouper.text.fr.fr.base.properties
  is not the same as what the patch expects:
  
/opt/tomcats/tomcat_f/webapps/grouper_v2_3/WEB-INF/classes/grouperText/grouper.text.fr.fr.base.properties
  Do you want to force install this patch (t|f)? [f]: 
<using default which is blank due to 
grouperInstaller.autorun.useDefaultsAsMuchAsAvailable and 
grouperInstaller.autorun.forceInstallPatch>: 
Cannot apply patch since this patch file:
  
/opt/grouper/2.3/patchesAuto/tarballs/patches/grouper_v2_3_0_ui_patch_20/old/classes/grouperText/grouper.text.fr.fr.base.properties
  is not the same as what the patch expects:
  
/opt/tomcats/tomcat_f/webapps/grouper_v2_3/WEB-INF/classes/grouperText/grouper.text.fr.fr.base.properties
 - added to end of property file: grouper_v2_3_0_ui_patch_20.state = error


Fixed by just forcing the patch, or doing this:

[appadmin@i2midev1 patchesAuto]$ cp 
/opt/grouper/2.3/patchesAuto/tarballs/patches/grouper_v2_3_0_ui_patch_20/old/classes/grouperText/grouper.text.fr.fr.base.properties
 
/opt/tomcats/tomcat_f/webapps/grouper_v2_3/WEB-INF/classes/grouperText/grouper.text.fr.fr.base.properties

IF you get the error too I can fix the patch by removing that file I guess...


-----Original Message-----
From: Hyzer, Chris 
Sent: Friday, March 24, 2017 1:33 PM
To: 

 Mailing List 
<>
Subject: RE: [JIRA] (GRP-1506) New feature - Configurable subject icons 
(ready for pull)

Chad, this looks like a great enhancement!  Thanks!  Can you make a pull 
request for 2.3 and for master (or just master or whatever you want?

Thanks
Chris

-----Original Message-----
From: Chad Redman (JIRA) 
[mailto:]
 
Sent: Friday, March 24, 2017 12:06 PM
To: Hyzer, Chris 
<>
Subject: [JIRA] (GRP-1506) New feature - Configurable subject icons (ready 
for pull)

Chad Redman created GRP-1506:
--------------------------------

             Summary: New feature - Configurable subject icons (ready for 
pull)
                 Key: GRP-1506
                 URL: https://bugs.internet2.edu/jira/browse/GRP-1506
             Project: Grouper
          Issue Type: New Feature
          Components: UI
    Affects Versions: 2.3.0, 2.4.0
            Reporter: Chad Redman
            Assignee: Chris Hyzer
            Priority: Trivial


Our institutional Grouper config uses four distinct sources. To make it 
easier to identify them, we have customized a few UI classes and added new 
properties, so that different icons are used for different subjects. An 
example showing usage (set in grouper-ui.properties) is:

grouperUi.screenSubjectIcon2.screenHtmlEl.default = ${'<i class="fa 
fa-user"></i> '}
  - this is added to grouper-ui.base.properties as the default or fallback 
which can be overrided per subject id. If for some reason a patcher fails to 
upgrade grouper-ui.base.properties, the same value is hardcoded in 
GrouperUiUtils so that the existing person icon is still displayed as before.

grouperUi.screenSubjectIcon2.sourceId.0 = student
grouperUi.screenSubjectIcon2.screenHtmlEl.0 = ${'<i class="fa 
fa-graduation-cap"></i> '}
grouperUi.screenSubjectIcon2.sourceId.1 = nonstudent
grouperUi.screenSubjectIcon2.screenHtmlEl.1 = ${'<i class="fa fa-coffee"></i> 
'}
  - like the other indexed properties, these start at zero and iterate 
through sequential grouperUi.screenSubjectIcon2.sourceId.N indexes. If any 
defined sourceId value matches the source ID for a subject, the corresponding 
grouperUi.screenSubjectIcon2.screenHtmlEl.N value is used for the icon HTML 
instead of the default.

Any valid EL can work here. For example, you can set a different icon based 
on attributes. E.g.,:

grouperUi.screenSubjectIcon2.screenHtmlEl.1 = 
${subject.attributes['businessCategory']==null ? '<i class="fa 
fa-question-circle"></i> ' : 
subject.attributes['businessCategory'].iterator().next() == 'Account Payable' 
? '<i style="color: red" class="fa fa-coffee"></i> ' : '<i class="fa 
fa-coffee"></i> '}

There is also a change in grouperText/grouper.text.*.base.properties, just to 
replace the hardcoded HTML with the dynamic value. If you have customized 
guiSubjectShortLink from the base default, you may want to change your 
version to use the dynamic icon.

This works everywhere in the New UI (at least where I checked) - see links 
for example screenshots:

* Search results 
([example|https://image.ibb.co/eKnBRF/grouper_custom_icons_01_search_results.png])
* Subject detail page ([example 
1|https://image.ibb.co/mzFy6F/grouper_custom_icons_02_student_source.png], 
[example 
2|https://image.ibb.co/cG65mF/grouper_custom_icons_03_nonstudent_source.png]
* Add members combo box ([example 
1|https://image.ibb.co/gm3J6F/grouper_custom_icons_04_add_members.png], 
[example 
2|https://image.ibb.co/hyprRF/grouper_custom_icons_07_add_members_attribute.png])
* Group member listing 
([example|https://image.ibb.co/cBnd6F/grouper_custom_icons_05_group_members.png])
* Subject API diagnostics page 
([example|https://image.ibb.co/jk42Yv/grouper_custom_icons_06_subject_api_diag.png])

This only affects the New UI, not the Admin UI or Lite UI.

As it's a major feature add, I didn't know if this would be better as a 
straight pull request, or as an optional package. The changeset are currently 
in my [Github 
fork|https://github.com/cer28/grouper/tree/cer28-custom-subject-icons].




--
This message was sent by Atlassian JIRA
(v7.2.6#72008)