From a4865adf44c318efb8f894ca36e5e3dd9330f49e Mon Sep 17 00:00:00 2001
From: alecpl <alec@alec.pl>
Date: Sun, 07 Mar 2010 12:42:09 -0500
Subject: [PATCH] - code cleanup

---
 skins/default/functions.js |   14 +++++++-------
 1 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/skins/default/functions.js b/skins/default/functions.js
index 3a295de..47a121d 100644
--- a/skins/default/functions.js
+++ b/skins/default/functions.js
@@ -13,7 +13,7 @@
     tab = '#settingstab' + (rcmail.env.action=='preferences' ? 'default' : (rcmail.env.action.indexOf('identity')>0 ? 'identities' : rcmail.env.action.replace(/\./g, '')));
 
   $(tab).addClass('tablink-selected');
-  $(tab + '> a').removeAttr('onclick').unbind('click').bind('click', function(){return false});
+  $(tab + '> a').removeAttr('onclick').unbind('click').bind('click', function(){return false;});
 }
 
 function rcube_show_advanced(visible)
@@ -39,12 +39,12 @@
     
   link.style.display = 'none';
 
-  if (row = document.getElementById('compose-' + id))
+  if ((row = document.getElementById('compose-' + id)))
     {
     var div = document.getElementById('compose-div');
     var headers_div = document.getElementById('compose-headers-div');
     row.style.display = (document.all && !window.opera) ? 'block' : 'table-row';
-    div.style.top = (parseInt(headers_div.offsetHeight)) + 'px';
+    div.style.top = parseInt(headers_div.offsetHeight, 10) + 'px';
     }
 
   return false;
@@ -52,7 +52,7 @@
 
 function rcmail_hide_header_form(id)
 {
-  var row, parent, ns, ps, link, links;
+  var row, parent, ns, link, links;
 
   link = document.getElementById(id + '-link');
   link.style.display = '';
@@ -71,12 +71,12 @@
 
   document.getElementById('_' + id).value = '';
 
-  if (row = document.getElementById('compose-' + id))
+  if ((row = document.getElementById('compose-' + id)))
     {
     var div = document.getElementById('compose-div');
     var headers_div = document.getElementById('compose-headers-div');
     row.style.display = 'none';
-    div.style.top = (parseInt(headers_div.offsetHeight)) + 'px';
+    div.style.top = parseInt(headers_div.offsetHeight, 10) + 'px';
     }
 
   return false;
@@ -175,7 +175,7 @@
 set_searchmod: function(elem)
 {
   if (!rcmail.env.search_mods)
-    rcmail.env.search_mods = new Object();
+    rcmail.env.search_mods = {};
   
   if (!rcmail.env.search_mods[rcmail.env.mailbox])
     rcmail.env.search_mods[rcmail.env.mailbox] = rcube_clone_object(rcmail.env.search_mods['*']);

--
Gitblit v1.9.1