Merge pull request playframework#253 from hwartig/lighthouse-882-patch

[playframework#882] fixed slugify so that it removes leading/trailing -'s
ransombriggs · Apr 6, 2012 · aff952a · aff952a
2 parents f2b021e + 375fe3d
commit aff952a
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/framework/src/play/templates/JavaExtensions.java b/framework/src/play/templates/JavaExtensions.java
@@ -367,7 +367,7 @@ public static String slugify(String string, Boolean lowercase) {
         string = string.replaceAll("([a-z])'s([^a-z])", "$1s$2");
         string = string.replaceAll("[^\\w]", "-").replaceAll("-{2,}", "-");
         // Get rid of any - at the start and end.
-        string.replaceAll("-+$", "").replaceAll("^-+", "");
+        string = string.replaceAll("-+$", "").replaceAll("^-+", "");
 
         return (lowercase ? string.toLowerCase() : string);
     }