Do not slugify manual slugs

We have accidentally been slugifying manually defined slugs, so only slugify if slug is already null Signed-off-by: BT (calcastor/mame) <43831917+calcastor@users.noreply.github.com>
PGMDev · Mar 9, 2024 · ff802a3 · ff802a3
1 parent 2708649
commit ff802a3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/core/src/main/java/tc/oc/pgm/map/MapInfoImpl.java b/core/src/main/java/tc/oc/pgm/map/MapInfoImpl.java
@@ -386,7 +386,7 @@ public VariantData(Element root, @Nullable Element variantEl) throws InvalidXMLE
         this.world = variantEl.getAttributeValue("world");
         if (slug != null) slug += "_" + variantId;
       }
-      this.mapId = assertNotNull(StringUtils.slugify(slug != null ? slug : mapName));
+      this.mapId = assertNotNull(slug != null ? slug : StringUtils.slugify(mapName));
     }
 
     @Override

diff --git a/util/src/main/java/tc/oc/pgm/util/StringUtils.java b/util/src/main/java/tc/oc/pgm/util/StringUtils.java
@@ -136,7 +136,7 @@ public static String normalize(String text) {
     return text == null
         ? ""
         : Normalizer.normalize(text, Normalizer.Form.NFD)
-            .replaceAll("[^A-Za-z0-9 ]", "")
+            .replaceAll("[^A-Za-z0-9_ ]", "")
             .toLowerCase(Locale.ROOT);
   }