Slugify selector: Remove leading & trailing dash (#2202)

server/test/utils/slugify.test.js

@@ -0,0 +1,21 @@
+const { expect } = require('chai');
+const { slugify } = require('../../utils/slugify');
+
+describe('slugify', () => {
+  it('should return new slug', () => {
+    const slug = slugify('This is a test slug, and this is fine');
+    expect(slug).to.equal('this-is-a-test-slug-and-this-is-fine');
+  });
+  it('should return new slug without trailing dash', () => {
+    const slug = slugify('This is a test slug, and this is fine |');
+    expect(slug).to.equal('this-is-a-test-slug-and-this-is-fine');
+  });
+  it('should return new slug without leading dash', () => {
+    const slug = slugify('| This is a test slug, and this is fine |');
+    expect(slug).to.equal('this-is-a-test-slug-and-this-is-fine');
+  });
+  it('should return new slug without emoji', () => {
+    const slug = slugify('😀 Test emoji test 😀');
+    expect(slug).to.equal('test-emoji-test');
+  });
+});

server/utils/slugify.js

@@ -22,6 +22,14 @@ function slugify(str) {
     .replace(/\s+/g, '-') // collapse whitespace and replace by -
     .replace(/-+/g, '-'); // collapse dashes
 
+  // Remove trailing and leading dashes if present
+  if (newString.startsWith('-')) {
+    newString = newString.slice(1);
+  }
+  if (newString.endsWith('-')) {
+    newString = newString.slice(0, -1);
+  }
+
   return newString;
 }