js optimisations and style fixes

diff --git a/core/js/js.js b/core/js/js.js
index 4e35547c0582791490fc50a7358870c486dff458..710ef416b699585ed6a12eccf6b1e0647f47a41d 100644 (file)
--- a/core/js/js.js
+++ b/core/js/js.js
@@ -37,14 +37,14 @@ function t(app,text, vars){
                        t.cache[app] = [];
                }
        }
-       var _build = function(text, vars) {
-               return text.replace(/{([^{}]*)}/g,
-                       function (a, b) {
-                               var r = vars[b];
-                               return typeof r === 'string' || typeof r === 'number' ? r : a;
-                       }
-               );
-       }
+       var _build = function (text, vars) {
+        return text.replace(/{([^{}]*)}/g,
+            function (a, b) {
+                var r = vars[b];
+                return typeof r === 'string' || typeof r === 'number' ? r : a;
+            }
+        );
+    };
        if( typeof( t.cache[app][text] ) !== 'undefined' ){
                if(typeof vars === 'object') {
                        return _build(t.cache[app][text], vars);
@@ -247,7 +247,6 @@ var OC={
                var popup = $('#appsettings_popup');
                if(popup.length == 0) {
                        $('body').prepend('<div class="popup hidden" id="appsettings_popup"></div>');
-                       popup = $('#appsettings_popup');
                        popup.addClass(settings.hasClass('topright') ? 'topright' : 'bottomleft');
                }
                if(popup.is(':visible')) {
@@ -308,14 +307,16 @@ OC.Notification={
         });
     },
     showHtml: function(html) {
-        $('#notification').hide();
-        $('#notification').html(html);
-        $('#notification').fadeIn();
+        var notification = $('#notification');
+        notification.hide();
+        notification.html(html);
+        notification.fadeIn();
     },
     show: function(text) {
-        $('#notification').hide();
-        $('#notification').text(text);
-        $('#notification').fadeIn();
+        var notification = $('#notification');
+        notification.hide();
+        notification.text(text);
+        notification.fadeIn();
     }
 };