@wordpress/editor 12.2.2 → 12.3.1-next.a55ed9455a.0

package/src/store/utils/notice-builder.js

@@ -46,19 +46,19 @@ export function getNotificationArgumentsForSaveSuccess( data ) {
 		noticeMessage = __( 'Draft saved' );
 		shouldShowLink = false;
 	} else if ( isPublished && ! willPublish ) {
-		// If undoing publish status, show specific notice
+		// If undoing publish status, show specific notice.
 		noticeMessage = postType.labels.item_reverted_to_draft;
 		shouldShowLink = false;
 	} else if ( ! isPublished && willPublish ) {
 		// If publishing or scheduling a post, show the corresponding
-		// publish message
+		// publish message.
 		noticeMessage = {
 			publish: postType.labels.item_published,
 			private: postType.labels.item_published_privately,
 			future: postType.labels.item_scheduled,
 		}[ post.status ];
 	} else {
-		// Generic fallback notice
+		// Generic fallback notice.
 		noticeMessage = postType.labels.item_updated;
 	}
 
@@ -98,7 +98,7 @@ export function getNotificationArgumentsForSaveFail( data ) {
 	const publishStatus = [ 'publish', 'private', 'future' ];
 	const isPublished = publishStatus.indexOf( post.status ) !== -1;
 	// If the post was being published, we show the corresponding publish error message
-	// Unless we publish an "updating failed" message
+	// Unless we publish an "updating failed" message.
 	const messages = {
 		publish: __( 'Publishing failed.' ),
 		private: __( 'Publishing failed.' ),

package/src/utils/url.js

@@ -1,26 +1,8 @@
-/**
- * External dependencies
- */
-import { deburr, trim } from 'lodash';
-
 /**
  * WordPress dependencies
  */
-import { addQueryArgs } from '@wordpress/url';
-
-/**
- * Returns the URL of a WPAdmin Page.
- *
- * TODO: This should be moved to a module less specific to the editor.
- *
- * @param {string} page  Page to navigate to.
- * @param {Object} query Query Args.
- *
- * @return {string} WPAdmin URL.
- */
-export function getWPAdminURL( page, query ) {
-	return addQueryArgs( page, query );
-}
+import { cleanForSlug as urlCleanForSlug } from '@wordpress/url';
+import deprecated from '@wordpress/deprecated';
 
 /**
  * Performs some basic cleanup of a string for use as a post slug
@@ -39,14 +21,10 @@ export function getWPAdminURL( page, query ) {
  * @return {string} Processed string
  */
 export function cleanForSlug( string ) {
-	if ( ! string ) {
-		return '';
-	}
-	return trim(
-		deburr( string )
-			.replace( /[\s\./]+/g, '-' )
-			.replace( /[^\p{L}\p{N}_-]+/gu, '' )
-			.toLowerCase(),
-		'-'
-	);
+	deprecated( 'wp.editor.cleanForSlug', {
+		since: '12.7',
+		plugin: 'Gutenberg',
+		alternative: 'wp.url.cleanForSlug',
+	} );
+	return urlCleanForSlug( string );
 }

package/src/utils/test/url.js

@@ -1,22 +0,0 @@
-/**
- * Internal dependencies
- */
-import { cleanForSlug } from '../url';
-
-describe( 'cleanForSlug()', () => {
-	it( 'Should return string prepared for use as url slug', () => {
-		expect( cleanForSlug( '/Is th@t Déjà_vu? ' ) ).toBe( 'is-tht-deja_vu' );
-	} );
-
-	it( 'Should allow non-latin characters', () => {
-		expect( cleanForSlug( 'Καλημέρα Κόσμε' ) ).toBe( 'καλημέρα-κόσμε' );
-	} );
-
-	it( 'Should return an empty string for missing argument', () => {
-		expect( cleanForSlug() ).toBe( '' );
-	} );
-
-	it( 'Should return an empty string for falsy argument', () => {
-		expect( cleanForSlug( null ) ).toBe( '' );
-	} );
-} );