fix(Slugify pipe): issue (danrevah#18)

sarahsga · Jan 26, 2017 · 376cff7 · 376cff7
1 parent 509cb47
commit 376cff7
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/src/app/pipes/string/slugify.spec.ts b/src/app/pipes/string/slugify.spec.ts
@@ -16,4 +16,9 @@ describe('SlugifyPipe Tests', () => {
     expect(pipe.transform('Foo Bar Baz')).toEqual('foo-bar-baz');
     expect(pipe.transform('UPPER CASE TEXT')).toEqual('upper-case-text');
   });
+
+  it('Should slugify special strings', () => {
+    expect(pipe.transform('http://example.com/foo')).toEqual('http-example-com-foo');
+    expect(pipe.transform(' http://example.com/foo ')).toEqual('http-example-com-foo');
+  });
 });
diff --git a/src/app/pipes/string/slugify.ts b/src/app/pipes/string/slugify.ts
@@ -6,7 +6,9 @@ export class SlugifyPipe implements PipeTransform {
 
   transform(str: string): string {
     return GeneralHelper.isString(str)
-      ? str.toLowerCase().replace(/\s+/g, '-')
+      ? str.toLowerCase().trim()
+        .replace(/[^\w\-]+/g, ' ')
+        .replace(/\s+/g, '-')
       : str;
   }
 }