Changeset 39574

Timestamp:

12/12/2016 02:10:30 AM (8 years ago)

Author:

dd32

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.
Merges [39547] to the 4.7 branch.
Fixes #39175.

Location:

branches/4.7

Files:

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

branches/4.7/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 );