Changeset 38205 for trunk/src/wp-admin/includes/update.php

Timestamp:

08/05/2016 07:27:56 PM (8 years ago)

Author:

ocean90

Message:

Update/Install: Replace "error" and "-1" failure messages with a more meaningful one.

• "-1" is an invalid nonce error, show 'An error has occurred. Please reload the page and try again.'.
• "error" means that the connection to the server was lost, show 'Connection lost or the server is busy. Please try again later.'.
• Escape the message in wp-updates-admin-notice because the response may include HTML.
• Remove HTML tags in wp.updates.isValidResponse() to make PHP's error messages more readable.

Props azaozz for review.
Fixes #37583.

File:

• trunk/src/wp-admin/includes/update.php (modified) (1 diff)

trunk/src/wp-admin/includes/update.php

@@ -632 +632 @@
     ?>
     <script id="tmpl-wp-updates-admin-notice" type="text/html">
-        <div <# if ( data.id ) { #>id="{{ data.id }}"<# } #> class="notice {{ data.className }}"><p>{{{ data.message }}}</p></div>
+        <div <# if ( data.id ) { #>id="{{ data.id }}"<# } #> class="notice {{ data.className }}"><p>{{ data.message }}</p></div>
     </script>
     <script id="tmpl-wp-bulk-updates-admin-notice" type="text/html">