[#CHAIN-43] ChainListener URL translation does not work as expected

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:

Repository	Revision	Date	User	Message
ASF	#658426	Tue May 20 20:55:38 UTC 2008	niallp	~~CHAIN-43~~ - ChainListener URL translation does not work as expected - thanks to Ales Dolecek
				Files Changed
				MODIFY /commons/proper/chain/trunk/src/java/org/apache/commons/chain/web/ChainListener.java

[ Permlink | « Hide ]

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

Niall Pemberton made changes - 20/May/08 08:58 PM

Field	Original Value	New Value
Fix Version/s		1.2 [ 12311963 ]
Resolution		Fixed [ 1 ]
Status	Open [ 1 ]	Resolved [ 5 ]
Assignee		Niall Pemberton [ niallp ]