Merge pull request #14364 from niteskum/AutoUpdateLocalizationBug

Fix for Localization Bug and Retaining the Installer Log

Author: vickramdhawal

src/extensions/default/AutoUpdate/UpdateStatus.js

@@ -27,7 +27,8 @@ define(function (require, exports, module) {
     var UpdateStatusHtml = require("text!htmlContent/updateStatus.html"),
         ExtensionUtils   = brackets.getModule("utils/ExtensionUtils"),
         Mustache         = brackets.getModule("thirdparty/mustache/mustache"),
-        Strings          = brackets.getModule("strings");
+        Strings          = brackets.getModule("strings"),
+        StringUtils      = brackets.getModule("utils/StringUtils");
 
     ExtensionUtils.loadStyleSheet(module, "styles/styles.css");
 
@@ -47,6 +48,8 @@ define(function (require, exports, module) {
 
         var $updateStatus = $(Mustache.render(UpdateStatusHtml, {"Strings": Strings}));
         $updateStatus.appendTo('#status-bar');
+        var valStr = StringUtils.format(Strings.NUMBER_WITH_PERCENTAGE, 0);
+        $('#update-status #' + id + ' #' + 'percent').text(valStr);
         $('#update-status #' + id).show();
     }
 
@@ -61,7 +64,8 @@ define(function (require, exports, module) {
      */
     function modifyUpdateStatus(statusObj) {
         statusObj.spans.forEach(function (span) {
-            $('#update-status #' + statusObj.target + ' #' + span.id).text(span.val);
+            var valStr = StringUtils.format(Strings.NUMBER_WITH_PERCENTAGE, span.val.split('%')[0]);
+            $('#update-status #' + statusObj.target + ' #' + span.id).text(valStr);
         });
     }
 

src/extensions/default/AutoUpdate/htmlContent/updateStatus.html

@@ -1,5 +1,5 @@
 <div id="update-status">
-    <p id="initial-download">{{Strings.INITIAL_DOWNLOAD}}<span id="percent">0%</span></p>
+    <p id="initial-download">{{Strings.INITIAL_DOWNLOAD}}<span id="percent"></span></p>
     <p id="retry-download">{{Strings.RETRY_DOWNLOAD}}<span id="attempt">1/5</span></p>
     <p id="validating-installer">{{Strings.VALIDATING_INSTALLER}}</p>
 </div>

src/extensions/default/AutoUpdate/main.js

@@ -217,7 +217,7 @@ define(function (require, exports, module) {
      * Checks and handles the update success and failure scenarios
      */
     function checkUpdateStatus() {
-        var filesToCache = null,
+        var filesToCache = ['.logs'],
             downloadCompleted = updateJsonHandler.get("downloadCompleted"),
             updateInitiatedInPrevSession = updateJsonHandler.get("updateInitiatedInPrevSession");
 
@@ -240,7 +240,6 @@ define(function (require, exports, module) {
             } else {
                 // We get here if the update started but failed
                 checkInstallationStatus();
-                filesToCache = ['.logs']; //AUTOUPDATE_PRERELEASE
                 UpdateInfoBar.showUpdateBar({
                     type: "error",
                     title: Strings.UPDATE_FAILED,

src/nls/root/strings.js

@@ -873,6 +873,7 @@ define({
     "AUTOUPDATE_ERROR"                               : "Error!",
     "AUTOUPDATE_IN_PROGRESS"                         : "An update is already in progress.",
 
+    "NUMBER_WITH_PERCENTAGE"                         : "{0}%",
     // Strings for Related Files
     "CMD_FIND_RELATED_FILES"                         : "Find Related Files"
 });