Changeset 39547

Timestamp:

12/09/2016 01:58:18 AM (8 years ago)

Author:

westonruter

Message:

Customize: Allow (optional) url parameter to be omitted in intercepted calls to history.pushState() and history.replaceState() in customize preview.

Fixes issue where calls without the url parameter erroneously end up rewriting the location path to /undefined.

Props Christian1012, westonruter.
Fixes #39175.

File:

• trunk/src/wp-includes/js/customize-preview.js (modified) (2 diffs)

trunk/src/wp-includes/js/customize-preview.js

@@ -50 +50 @@
             return function historyReplaceState( data, title, url ) {
                 currentHistoryState = data;
-                return nativeReplaceState.call( history, data, title, injectUrlWithState( url ) );
+                return nativeReplaceState.call( history, data, title, 'string' === typeof url && url.length > 0 ? injectUrlWithState( url ) : url );
             };
         } )( history.replaceState );
@@ -57 +57 @@
             return function historyPushState( data, title, url ) {
                 currentHistoryState = data;
-                return nativePushState.call( history, data, title, injectUrlWithState( url ) );
+                return nativePushState.call( history, data, title, 'string' === typeof url && url.length > 0 ? injectUrlWithState( url ) : url );
             };
         } )( history.pushState );