You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by cz...@apache.org on 2009/11/18 15:05:18 UTC
svn commit: r881783 -
/sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/request/SlingRequestPathInfo.java
Author: cziegeler
Date: Wed Nov 18 14:05:18 2009
New Revision: 881783
URL: http://svn.apache.org/viewvc?rev=881783&view=rev
Log:
SLING-633 : RequestDispatcherOptions.setReplaceSelectors("") doesn't work
Modified:
sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/request/SlingRequestPathInfo.java
Modified: sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/request/SlingRequestPathInfo.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/request/SlingRequestPathInfo.java?rev=881783&r1=881782&r2=881783&view=diff
==============================================================================
--- sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/request/SlingRequestPathInfo.java (original)
+++ sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/request/SlingRequestPathInfo.java Wed Nov 18 14:05:18 2009
@@ -126,6 +126,10 @@
// replacement selectors
String selectors = options.getReplaceSelectors();
if (selectors != null) {
+ // if we get the empty string, all original selectors will be removed
+ if ( selectors.length() == 0 ) {
+ selectors = null;
+ }
needCreate = true;
} else {
selectors = getSelectorString();