Merge pull request #101

Fixed title normalization

Author: andrey-helldar

src/utils/strings.ts

@@ -1,5 +1,18 @@
 import { reservedWords } from "../libs/words";
 
+const prepareWords = (value: string): string => {
+    value = value.replace(/([^\w\d]|[-/_])+/g, " ");
+
+    for (const word of reservedWords) {
+        value = value.replace(
+            new RegExp(`\\b${word}\\b`, "i"),
+            word.toLowerCase(),
+        );
+    }
+
+    return value;
+};
+
 const normalizeWords = (value: string): string => {
     for (const word of reservedWords) {
         value = value.replace(new RegExp(`\\b${word}\\b`, "i"), word);
@@ -13,11 +26,17 @@ export const titleCase = (title?: string) => {
         return "";
     }
 
-    title = title
-        .replace(/([A-Z])/g, "$1")
+    const upper: string = title.toUpperCase();
+
+    if (upper === title) {
+        title = title.toLowerCase();
+    }
+
+    title = prepareWords(title)
+        .replace(/([A-Z])/g, " $1")
         .toLowerCase()
         .replace(/(^|\s|-|_)\S/g, (match: string) => match.toUpperCase())
-        .replace(/[-_]/g, " ")
+        .replace(/(\s|\u3164|\u1160)+/gu, " ")
         .trim();
 
     const normalized = normalizeWords(title);

tests/unit/strings.test.ts

@@ -24,3 +24,26 @@ test("words", () => {
         expect(items.map((item: string) => titleCase(item))).toMatchSnapshot();
     }
 });
+
+test("case", () => {
+    expect(titleCase("fooBar")).toBe("Foo Bar");
+    expect(titleCase("foo Bar")).toBe("Foo Bar");
+    expect(titleCase("foo bar")).toBe("Foo Bar");
+    expect(titleCase("FOO BAR")).toBe("Foo Bar");
+    expect(titleCase("fOo bAr")).toBe("F Oo B Ar");
+
+    expect(titleCase("foo-Bar")).toBe("Foo Bar");
+    expect(titleCase("foo-bar")).toBe("Foo Bar");
+    expect(titleCase("FOO-BAR")).toBe("Foo Bar");
+    expect(titleCase("fOo-bAr")).toBe("F Oo B Ar");
+
+    expect(titleCase("foo_Bar")).toBe("Foo Bar");
+    expect(titleCase("foo_bar")).toBe("Foo Bar");
+    expect(titleCase("FOO_BAR")).toBe("Foo Bar");
+    expect(titleCase("fOo_bAr")).toBe("F Oo B Ar");
+
+    expect(titleCase("foo/Bar")).toBe("Foo Bar");
+    expect(titleCase("foo/bar")).toBe("Foo Bar");
+    expect(titleCase("FOO/BAR")).toBe("Foo Bar");
+    expect(titleCase("fOo/bAr")).toBe("F Oo B Ar");
+});