<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" /><style type="text/css"><!--
#msg dl { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt;  }
#msg dl a { font-weight: bold}
#msg dl a:link    { color:#fc3; }
#msg dl a:active  { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fc0 solid; padding: 6px; }
#msg ul, pre { overflow: auto; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff  {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<title>[14819] trunk: Provide some Before/After strings for Bulk Upgrades,
  Add a spinner whilst a upgrade is in process,
  Remove scrollbars on update iframe.</title>
</head>
<body>

<div id="msg">
<dl>
<dt>Revision</dt> <dd><a href="http://trac.wordpress.org/changeset/14819">14819</a></dd>
<dt>Author</dt> <dd>dd32</dd>
<dt>Date</dt> <dd>2010-05-23 12:29:00 +0000 (Sun, 23 May 2010)</dd>
</dl>

<h3>Log Message</h3>
<pre>Provide some Before/After strings for Bulk Upgrades, Add a spinner whilst a upgrade is in process, Remove scrollbars on update iframe. Props ocean90. See <a href="http://trac.wordpress.org/ticket/11232">#11232</a></pre>

<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkwpadmincsswpadmindevcss">trunk/wp-admin/css/wp-admin.dev.css</a></li>
<li><a href="#trunkwpadminincludesclasswpupgraderphp">trunk/wp-admin/includes/class-wp-upgrader.php</a></li>
<li><a href="#trunkwpadminupdatecorephp">trunk/wp-admin/update-core.php</a></li>
<li><a href="#trunkwpincludesscriptloaderphp">trunk/wp-includes/script-loader.php</a></li>
</ul>

</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkwpadmincsswpadmindevcss"></a>
<div class="modfile"><h4>Modified: trunk/wp-admin/css/wp-admin.dev.css (14818 => 14819)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/wp-admin/css/wp-admin.dev.css        2010-05-23 12:04:24 UTC (rev 14818)
+++ trunk/wp-admin/css/wp-admin.dev.css        2010-05-23 12:29:00 UTC (rev 14819)
</span><span class="lines">@@ -3752,3 +3752,8 @@
</span><span class="cx">         padding: 4px 20px 6px;
</span><span class="cx">         font: italic normal normal 24px/35px Georgia, &quot;Times New Roman&quot;, &quot;Bitstream Charter&quot;, Times, serif;
</span><span class="cx"> }
</span><ins>+
+/* Scrollbar fix for bulk upgrade iframe */
+body.update-php {
+        height: 98%;
+}
</ins></span></pre></div>
<a id="trunkwpadminincludesclasswpupgraderphp"></a>
<div class="modfile"><h4>Modified: trunk/wp-admin/includes/class-wp-upgrader.php (14818 => 14819)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/wp-admin/includes/class-wp-upgrader.php        2010-05-23 12:04:24 UTC (rev 14818)
+++ trunk/wp-admin/includes/class-wp-upgrader.php        2010-05-23 12:29:00 UTC (rev 14819)
</span><span class="lines">@@ -466,6 +466,7 @@
</span><span class="cx">                         return false;
</span><span class="cx">                 }
</span><span class="cx"> 
</span><ins>+                $this-&gt;skin-&gt;feedback('bulk_upgrade_start');
</ins><span class="cx">                 $this-&gt;maintenance_mode(true);
</span><span class="cx"> 
</span><span class="cx">                 $results = array();
</span><span class="lines">@@ -507,7 +508,10 @@
</span><span class="cx">                         if ( false === $result )
</span><span class="cx">                                 break;
</span><span class="cx">                 } //end foreach $plugins
</span><ins>+
</ins><span class="cx">                 $this-&gt;maintenance_mode(false);
</span><ins>+                $this-&gt;skin-&gt;feedback('bulk_upgrade_end');
+
</ins><span class="cx">                 $this-&gt;skin-&gt;footer();
</span><span class="cx"> 
</span><span class="cx">                 // Cleanup our hooks, incase something else does a upgrade on this connection.
</span><span class="lines">@@ -704,6 +708,7 @@
</span><span class="cx">                         return false;
</span><span class="cx">                 }
</span><span class="cx"> 
</span><ins>+                $this-&gt;skin-&gt;feedback('bulk_upgrade_start');
</ins><span class="cx">                 $this-&gt;maintenance_mode(true);
</span><span class="cx"> 
</span><span class="cx">                 $results = array();
</span><span class="lines">@@ -745,7 +750,9 @@
</span><span class="cx">                         if ( false === $result )
</span><span class="cx">                                 break;
</span><span class="cx">                 } //end foreach $plugins
</span><ins>+
</ins><span class="cx">                 $this-&gt;maintenance_mode(false);
</span><ins>+                $this-&gt;skin-&gt;feedback('bulk_upgrade_end');
</ins><span class="cx">                 $this-&gt;skin-&gt;footer();
</span><span class="cx"> 
</span><span class="cx">                 // Cleanup our hooks, incase something else does a upgrade on this connection.
</span><span class="lines">@@ -1057,9 +1064,11 @@
</span><span class="cx">         }
</span><span class="cx"> 
</span><span class="cx">         function add_strings() {
</span><ins>+                $this-&gt;upgrader-&gt;strings['bulk_upgrade_start'] = __('The update process is starting. This process may take awhile on some hosts, so please be patient.');
</ins><span class="cx">                 $this-&gt;upgrader-&gt;strings['skin_update_failed_error'] = __('An error occured while updating %1$s: &lt;strong&gt;%2$s&lt;/strong&gt;.');
</span><span class="cx">                 $this-&gt;upgrader-&gt;strings['skin_update_failed'] = __('The update of %1$s failed.');
</span><span class="cx">                 $this-&gt;upgrader-&gt;strings['skin_update_successful'] = __('%1$s updated successfully.').' &lt;a onclick=&quot;%2$s&quot; href=&quot;#&quot; class=&quot;hide-if-no-js&quot;&gt;&lt;span&gt;'.__('Show Details').'&lt;/span&gt;&lt;span class=&quot;hidden&quot;&gt;'.__('Hide Details').'&lt;/span&gt;.&lt;/a&gt;';
</span><ins>+                $this-&gt;upgrader-&gt;strings['bulk_upgrade_end'] = __('All upgrades have been completed.');
</ins><span class="cx">         }
</span><span class="cx"> 
</span><span class="cx">         function feedback($string) {
</span><span class="lines">@@ -1100,11 +1109,13 @@
</span><span class="cx">                         }
</span><span class="cx">                         $this-&gt;error = implode(', ', $messages);
</span><span class="cx">                 }
</span><ins>+                echo '&lt;script type=&quot;text/javascript&quot;&gt;jQuery(\'.waiting-' . esc_js($this-&gt;upgrader-&gt;update_current) . '\').hide();&lt;/script&gt;';
</ins><span class="cx">         }
</span><span class="cx"> 
</span><span class="cx">         function before($title = '') {
</span><span class="cx">                 $this-&gt;in_loop = true;
</span><del>-                printf( '&lt;h4&gt;' . $this-&gt;upgrader-&gt;strings['skin_before_update_header'] . '&lt;/h4&gt;',  $title, $this-&gt;upgrader-&gt;update_current, $this-&gt;upgrader-&gt;update_count);
</del><ins>+                printf( '&lt;h4&gt;' . $this-&gt;upgrader-&gt;strings['skin_before_update_header'] . ' &lt;img alt=&quot;&quot; src=&quot;' . admin_url( 'images/wpspin_light.gif' ) . '&quot; class=&quot;hidden waiting-' . $this-&gt;upgrader-&gt;update_current . '&quot; style=&quot;vertical-align:middle;&quot;&gt;&lt;/h4&gt;',  $title, $this-&gt;upgrader-&gt;update_current, $this-&gt;upgrader-&gt;update_count);
+                echo '&lt;script type=&quot;text/javascript&quot;&gt;jQuery(\'.waiting-' . esc_js($this-&gt;upgrader-&gt;update_current) . '\').show();&lt;/script&gt;';
</ins><span class="cx">                 echo '&lt;div class=&quot;update-messages hide-if-js&quot; id=&quot;progress-' . esc_attr($this-&gt;upgrader-&gt;update_current) . '&quot;&gt;&lt;p&gt;';
</span><span class="cx">                 $this-&gt;flush_output();
</span><span class="cx">         }
</span><span class="lines">@@ -1121,6 +1132,7 @@
</span><span class="cx">                 }
</span><span class="cx">                 if ( !empty($this-&gt;result) &amp;&amp; !is_wp_error($this-&gt;result) ) {
</span><span class="cx">                         echo '&lt;div class=&quot;updated&quot;&gt;&lt;p&gt;' . sprintf($this-&gt;upgrader-&gt;strings['skin_update_successful'], $title, 'jQuery(\'#progress-' . esc_js($this-&gt;upgrader-&gt;update_current) . '\').toggle();jQuery(\'span\', this).toggle(); return false;') . '&lt;/p&gt;&lt;/div&gt;';
</span><ins>+                        echo '&lt;script type=&quot;text/javascript&quot;&gt;jQuery(\'.waiting-' . esc_js($this-&gt;upgrader-&gt;update_current) . '\').hide();&lt;/script&gt;';
</ins><span class="cx">                 }
</span><span class="cx">                 $this-&gt;reset();
</span><span class="cx">                 $this-&gt;flush_output();
</span></span></pre></div>
<a id="trunkwpadminupdatecorephp"></a>
<div class="modfile"><h4>Modified: trunk/wp-admin/update-core.php (14818 => 14819)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/wp-admin/update-core.php        2010-05-23 12:04:24 UTC (rev 14818)
+++ trunk/wp-admin/update-core.php        2010-05-23 12:29:00 UTC (rev 14819)
</span><span class="lines">@@ -418,7 +418,7 @@
</span><span class="cx">         echo '&lt;div class=&quot;wrap&quot;&gt;';
</span><span class="cx">         screen_icon('plugins');
</span><span class="cx">         echo '&lt;h2&gt;' . esc_html__('Update Plugins') . '&lt;/h2&gt;';
</span><del>-        echo &quot;&lt;iframe src='$url' style='width: 100%; height:100%; min-height:850px;'&gt;&lt;/iframe&gt;&quot;;
</del><ins>+        echo &quot;&lt;iframe src='$url' style='width: 100%; height: 100%; min-height: 750px;' frameborder='0'&gt;&lt;/iframe&gt;&quot;;
</ins><span class="cx">         echo '&lt;/div&gt;';
</span><span class="cx"> } elseif ( 'do-theme-upgrade' == $action ) {
</span><span class="cx">         check_admin_referer('upgrade-core');
</span><span class="lines">@@ -441,7 +441,7 @@
</span><span class="cx">         echo '&lt;div class=&quot;wrap&quot;&gt;';
</span><span class="cx">         screen_icon('themes');
</span><span class="cx">         echo '&lt;h2&gt;' . esc_html__('Update Themes') . '&lt;/h2&gt;';
</span><del>-        echo &quot;&lt;iframe src='$url' style='width: 100%; height:100%; min-height:850px;'&gt;&lt;/iframe&gt;&quot;;
</del><ins>+        echo &quot;&lt;iframe src='$url' style='width: 100%; height: 100%; min-height: 750px;' frameborder='0'&gt;&lt;/iframe&gt;&quot;;
</ins><span class="cx">         echo '&lt;/div&gt;';
</span><span class="cx"> }
</span><span class="cx"> 
</span></span></pre></div>
<a id="trunkwpincludesscriptloaderphp"></a>
<div class="modfile"><h4>Modified: trunk/wp-includes/script-loader.php (14818 => 14819)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/wp-includes/script-loader.php        2010-05-23 12:04:24 UTC (rev 14818)
+++ trunk/wp-includes/script-loader.php        2010-05-23 12:29:00 UTC (rev 14819)
</span><span class="lines">@@ -435,7 +435,7 @@
</span><span class="cx">         // Any rtl stylesheets that don't have a .dev version for ltr
</span><span class="cx">         $no_suffix = array( 'farbtastic' );
</span><span class="cx"> 
</span><del>-        $styles-&gt;add( 'wp-admin', &quot;/wp-admin/css/wp-admin$suffix.css&quot;, array(), '20100523' );
</del><ins>+        $styles-&gt;add( 'wp-admin', &quot;/wp-admin/css/wp-admin$suffix.css&quot;, array(), '20100523b' );
</ins><span class="cx"> 
</span><span class="cx">         $styles-&gt;add( 'ie', &quot;/wp-admin/css/ie$suffix.css&quot;, array(), '20100219' );
</span><span class="cx">         $styles-&gt;add_data( 'ie', 'conditional', 'lte IE 7' );
</span></span></pre>
</div>
</div>

</body>
</html>