Issue Details (XML | Word | Printable)

Key: CHAIN-43
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Trivial Trivial
Assignee: Niall Pemberton
Reporter: Ales Dolecek
Votes: 0
Watchers: 0
Operations

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

ChainListener URL translation does not work as expected

Created: 24/Feb/08 02:43 PM   Updated: 20/May/08 08:58 PM
Return to search
Component/s: None
Affects Version/s: 1.1, Nightly Builds
Fix Version/s: 1.2

Time Tracking:
Not Specified

Resolution Date: 20/May/08 08:58 PM


 Description  « Hide
Method org.apache.commons.chain.web.ChainListener#translate(String) simply discards any spaces in URL instead of replacing them with %20 as stated in it's documentation.

As result the configuration form chain-config.xml files stored in JAR files on path with spaces is not processed.

BTW: Why not use

jarURL.toExternalForm().replaceAll(" ", "%20")

instead of

translate(jarURL.toExternalForm())

AlesD



 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Niall Pemberton added a comment - 20/May/08 08:58 PM
Fixed thanks - http://svn.apache.org/viewvc?view=rev&revision=658426

P.S. The reason for not using replaceAll is that wasn't introduced until JDK 1.4 and Chain is still targetting JDK 1.3 as a minimum