PackageManager: Added more detailed errors messages on load methods

Author: andersevenrud

src/client/javascript/core/package-manager.js

@@ -92,10 +92,9 @@
             var len = Object.keys(packages).length;
             if ( len ) {
               cb();
-              return;
+            } else {
+              callback(false, API._('ERR_PACKAGE_ENUM_FAILED'), PackageManager);
             }
-
-            callback(false, 'No packages found!', PackageManager);
           });
         }
 
@@ -173,7 +172,7 @@
           var uri = API.getConfig('Connection.MetadataURI');
           Utils.preload([uri], function(total, failed) {
             if ( failed.length ) {
-              callback('Failed to load package manifest', failed);
+              callback(API._('ERR_PACKAGE_MANIFEST'), failed);
               return;
             }
 

src/client/javascript/locales/en_EN.js

@@ -304,6 +304,8 @@
     //
 
     'ERR_PACKAGE_EXISTS': 'Package installation directory already exists. Cannot continue!',
+    'ERR_PACKAGE_MANIFEST': 'Failed to load package manifest',
+    'ERR_PACKAGE_ENUM_FAILED': 'Failed to get package list. Make sure you\'re not using "Private Mode" as this might case problems.',
 
     //
     // DefaultApplication