cleanForSlug: Replace multiple hyphens with a single one (#44873)

WordPress · Oct 11, 2022 · a3360a8 · a3360a8
1 parent bac2c08
commit a3360a8
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/packages/url/src/clean-for-slug.js b/packages/url/src/clean-for-slug.js
@@ -31,6 +31,8 @@ export function cleanForSlug( string ) {
 			.replace( /[^\p{L}\p{N}_-]+/gu, '' )
 			// Convert to lowercase
 			.toLowerCase()
+			// Replace multiple hyphens with a single one.
+			.replace( /-+/g, '-' )
 			// Remove any remaining leading or trailing hyphens.
 			.replace( /(^-+)|(-+$)/g, '' )
 	);

diff --git a/packages/url/src/test/index.js b/packages/url/src/test/index.js
@@ -1004,6 +1004,11 @@ describe( 'cleanForSlug', () => {
 			'is-tht-deja_vu'
 		);
 	} );
+
+	it( 'Should replace multiple hyphens with a single one', () => {
+		expect( cleanForSlug( 'the long - cat' ) ).toBe( 'the-long-cat' );
+		expect( cleanForSlug( 'the----long---cat' ) ).toBe( 'the-long-cat' );
+	} );
 } );
 
 describe( 'normalizePath', () => {