History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: UP-1891
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Minor Minor
Assignee: Eric Dalquist
Reporter: Parker Grimes
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
uPortal

Extra space in tester-class element in PAGSGroupStoreConfig.xml causes ClassNotFoundException

Created: 28/Dec/07 05:08 PM   Updated: 15/Sep/08 06:47 PM
Component/s: Person Attributes
Affects Version/s: 2.6.1, 3.0.0-M5, 3.0.0-RC2, 3.0.0-RC3, 3.0.0
Fix Version/s: 3.0.1, 3.1.0-M1

Original Estimate: Unknown Remaining Estimate: Unknown Time Spent: Unknown
File Attachments: 1. Text File UP-1891-3-0-patch.txt (1 kb)
2. Text File XMLPersonAttributesConfiguration.patch (1 kb)



 Description  « Hide
If you have any extra space in a <tester-class> element in PAGSGroupStoreConfig.xml, XMLPersonAttributesConfiguration.initializeTester will throw a ClassNotFoundException.

I ran into this when I had Eclipse format my XML for me in PAGSGroupStoreConfig.xml. For example:

This works fine:
<tester-class>org.jasig.portal.groups.pags.testers.StringEqualsIgnoreCaseTester</tester-class>

However, this results in a ClassNotFoundException:
<tester-class>
org.jasig.portal.groups.pags.testers.StringEqualsIgnoreCaseTester
</tester-class>

The fix is very simple. You just have to add call .trim() when getting the data for that element. .trim() was already in use in one place in XMLPersonAttributesConfiguration.java, but not when getting the tester-class element.

 All   Comments   Work Log   Change History      Sort Order:
Parker Grimes [28/Dec/07 05:09 PM]
This patch will fix the problem. There are two places in XMLPersonAttributesConfiguration.java that should be calling .trim() but were not.

Parker Grimes [19/May/08 07:02 PM]
This bug still exists in the 3.0 code base.

Parker Grimes [20/May/08 10:06 AM]
This is a patch for XMLPersonAttributesConfiguration.java in uPortal 3.0

Eric Dalquist [10/Jun/08 12:53 PM]
Fixed in trunk and 3.0 patches.

Eric Dalquist [10/Jun/08 12:53 PM]
Also thanks for the patch Parker, makes fixing these much easier.

Eric Dalquist [15/Sep/08 06:47 PM]
Closing issues that have been released