You are viewing a plain text version of this content. The canonical link for it is here.
Posted to droids-commits@incubator.apache.org by ja...@apache.org on 2011/10/25 19:31:29 UTC
svn commit: r1188876 -
/incubator/droids/trunk/droids-core/src/main/java/org/apache/droids/helper/factories/URLFiltersFactory.java
Author: javier
Date: Tue Oct 25 19:31:29 2011
New Revision: 1188876
URL: http://svn.apache.org/viewvc?rev=1188876&view=rev
Log:
Code refactorization. DROIDS-149
Modified:
incubator/droids/trunk/droids-core/src/main/java/org/apache/droids/helper/factories/URLFiltersFactory.java
Modified: incubator/droids/trunk/droids-core/src/main/java/org/apache/droids/helper/factories/URLFiltersFactory.java
URL: http://svn.apache.org/viewvc/incubator/droids/trunk/droids-core/src/main/java/org/apache/droids/helper/factories/URLFiltersFactory.java?rev=1188876&r1=1188875&r2=1188876&view=diff
==============================================================================
--- incubator/droids/trunk/droids-core/src/main/java/org/apache/droids/helper/factories/URLFiltersFactory.java (original)
+++ incubator/droids/trunk/droids-core/src/main/java/org/apache/droids/helper/factories/URLFiltersFactory.java Tue Oct 25 19:31:29 2011
@@ -35,13 +35,8 @@ public class URLFiltersFactory extends G
*/
public boolean accept(String urlString) {
for (String key : getMap().keySet()) {
- if (urlString == null) {
- return false;
- }
- URLFilter filter = getMap().get(key);
- urlString = filter.filter(urlString);
- if (urlString == null) {
- return false;
+ if(!accept(urlString, key)) {
+ return false;
}
}
return true;
@@ -60,8 +55,7 @@ public class URLFiltersFactory extends G
if (urlString == null) {
return false;
}
- URLFilter filter = getMap().get(filterName);
- urlString = filter.filter(urlString);
+ urlString = getMap().get(filterName).filter(urlString);
if (urlString == null) {
return false;
}