-----------------------------------------------------------------------
uPortal > 3.2 Patches - Site > #323 failed.
-----------------------------------------------------------------------
Code has been updated by Pascal Rigaux <[email protected]>.
No failed tests found, a possible compilation error.

https://developer.jasig.org/bamboo/browse/UP-32PATCHESSITE-323/


--------------
Failing Jobs
--------------
  - Default Job (Default Stage): 330 tests passed.


--------------
Code Changes
--------------
Pascal Rigaux <[email protected]> 
(f4bd982a82179783b2aea908305d49351e0b17b2):

>fix UP-3365: PAGS XML parser small issue
>
>        <selection-test>
>          <test-group>
>            <test>
>              <attribute-name>eduPersonAffiliation</attribute-name>
>              
> <tester-class>org.jasig.portal.groups.pags.testers.StringEqualsIgnoreCaseTester</tester-class>
>              <test-value>employee</test-value>
>            </test>
>          </test-group>
>        </selection-test>
>
>was parsed as if it was
>
>        <selection-test>
>          <test-group>
>            <test>
>              <attribute-name>eduPersonAffiliation</attribute-name>
>              
> <tester-class>org.jasig.portal.groups.pags.testers.StringEqualsIgnoreCaseTester</tester-class>
>              <test-value>employee</test-value>
>            </test>
>            <test>
>              <attribute-name>eduPersonAffiliation</attribute-name>
>              
> <tester-class>org.jasig.portal.groups.pags.testers.StringEqualsIgnoreCaseTester</tester-class>
>              <test-value>employee</test-value>
>            </test>
>          </test-group>
>        </selection-test>
>
>which is equivalent but more costly!
>
>NB: it is equivalent because groupDef.addTestGroup(tg) was done too soon, but 
>tg is not deep copied, so it finally got the good value, but was added many 
>times.
>



--
This message is automatically generated by Atlassian Bamboo

Reply via email to