remove some obsolete stuff, fix minor issues found by code analyzer

diff --git a/js/deprecated.js b/js/deprecated.js
deleted file mode 100644 (file)
index dd286aa..0000000
--- a/js/deprecated.js
+++ /dev/null
@@ -1,23 +0,0 @@
-function selectTableRow(r, do_select) {
-       
-       if (do_select) {
-               r.addClassName("Selected");
-       } else {
-               r.removeClassName("Selected");
-       }
-}
-
-function selectTableRowById(elem_id, check_id, do_select) {
-       var row = $(elem_id);
-
-       if (row) {
-               selectTableRow(row, do_select);
-       }
-
-       var check = $(check_id);
-
-       if (check) {
-               check.checked = do_select;
-       }
-}
-

diff --git a/js/functions.js b/js/functions.js
index d52fee8944a53bb150bbc9118a6a6adc6a1c77f5..4c5d67f6c646a7cc1423dd8ababa5d8095a0b9c0 100755 (executable)
--- a/js/functions.js
+++ b/js/functions.js
@@ -96,17 +96,7 @@ function exception_error(e, e_compat, filename, lineno, colno) {
 }
 
 function param_escape(arg) {
-       if (typeof encodeURIComponent != 'undefined')
-               return encodeURIComponent(arg);
-       else
-               return escape(arg);
-}
-
-function param_unescape(arg) {
-       if (typeof decodeURIComponent != 'undefined')
-               return decodeURIComponent(arg);
-       else
-               return unescape(arg);
+       return encodeURIComponent(arg);
 }
 
 function notify_real(msg, no_hide, n_type) {
@@ -255,50 +245,11 @@ function gotoMain() {
        document.location.href = "index.php";
 }
 
-/** * @(#)isNumeric.js * * Copyright (c) 2000 by Sundar Dorai-Raj
-  * * @author Sundar Dorai-Raj
-  * * Email: sdoraira@vt.edu
-  * * This program is free software; you can redistribute it and/or
-  * * modify it under the terms of the GNU General Public License
-  * * as published by the Free Software Foundation; either version 2
-  * * of the License, or (at your option) any later version,
-  * * provided that any use properly credits the author.
-  * * This program is distributed in the hope that it will be useful,
-  * * but WITHOUT ANY WARRANTY; without even the implied warranty of
-  * * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
-  * * GNU General Public License for more details at http://www.gnu.org * * */
-
-  var numbers=".0123456789";
-  function isNumeric(x) {
-    // is x a String or a character?
-    if(x.length>1) {
-      // remove negative sign
-      x=Math.abs(x)+"";
-      for(var j=0;j<x.length;j++) {
-        // call isNumeric recursively for each character
-        number=isNumeric(x.substring(j,j+1));
-        if(!number) return number;
-      }
-      return number;
-    }
-    else {
-      // if x is number return true
-      if(numbers.indexOf(x)>=0) return true;
-      return false;
-    }
-  }
-
-
 function toggleSelectRowById(sender, id) {
        var row = $(id);
        return toggleSelectRow(sender, row);
 }
 
-function toggleSelectListRow(sender) {
-       var row = sender.parentNode;
-       return toggleSelectRow(sender, row);
-}
-
 /* this is for dijit Checkbox */
 function toggleSelectListRow2(sender) {
        var row = sender.domNode.parentNode;
@@ -345,20 +296,11 @@ function checkboxToggleElement(elem, id) {
        }
 }
 
-function dropboxSelect(e, v) {
-       for (var i = 0; i < e.length; i++) {
-               if (e[i].value == v) {
-                       e.selectedIndex = i;
-                       break;
-               }
-       }
-}
-
 function getURLParam(param){
        return String(window.location.href).parseQuery()[param];
 }
 
-function closeInfoBox(cleanup) {
+function closeInfoBox() {
        dialog = dijit.byId("infoBox");
 
        if (dialog)     dialog.hide();
@@ -530,13 +472,6 @@ function strip_tags(s) {
        return s.replace(/<\/?[^>]+(>|$)/g, "");
 }
 
-function truncate_string(s, length) {
-       if (!length) length = 30;
-       var tmp = s.substring(0, length);
-       if (s.length > length) tmp += "&hellip;";
-       return tmp;
-}
-
 function hotkey_prefix_timeout() {
 
        var date = new Date();
@@ -739,7 +674,6 @@ function quickAddFeed() {
                                                                dialog.show_error(__("XML validation failed: %s").
                                                                                replace("%s", rc['message']));
                                                                break;
-                                                               break;
                                                        case 0:
                                                                dialog.show_error(__("You are already subscribed to this feed."));
                                                                break;
@@ -1308,7 +1242,7 @@ function getSelectedTableRowIds(id) {
        return rows;
 }
 
-function editFeed(feed, event) {
+function editFeed(feed) {
        if (feed <= 0)
                return alert(__("You can't edit this kind of feed."));
 

diff --git a/js/prefs.js b/js/prefs.js
index 0512e0e34e76d69adbfde254c426c70447d46db2..a3f32dcbc46c12bd6d10fcb87ba1d22339453497 100755 (executable)
--- a/js/prefs.js
+++ b/js/prefs.js
@@ -9,7 +9,7 @@ function notify_callback2(transport, sticky) {
        notify_info(transport.responseText, sticky);
 }
 
-function updateFeedList(sort_key) {
+function updateFeedList() {
 
        var user_search = $("feed_search");
        var search = "";
@@ -83,7 +83,7 @@ function addUser() {
 
 }
 
-function editUser(id, event) {
+function editUser(id) {
 
        var query = "backend.php?op=pref-users&method=edit&id=" +
                param_escape(id);
@@ -749,7 +749,7 @@ function updateSystemList() {
                } });
 }
 
-function selectTab(id, noupdate, method) {
+function selectTab(id, noupdate) {
        if (!noupdate) {
                notify_progress("Loading, please wait...");
 
@@ -1319,7 +1319,7 @@ function resetCatOrder() {
        });
 }
 
-function editCat(id, item, event) {
+function editCat(id, item) {
        var new_name = prompt(__('Rename category to:'), item.name);
 
        if (new_name && new_name != item.name) {
@@ -1340,7 +1340,7 @@ function editCat(id, item, event) {
        }
 }
 
-function editLabel(id, event) {
+function editLabel(id) {
        var query = "backend.php?op=pref-labels&method=edit&id=" +
                param_escape(id);
 

diff --git a/js/tt-rss.js b/js/tt-rss.js
index b498b5e998e005bb80c2dbe570a21b36cf71d229..4df1b04eeded6bf503477494b069c2f6d9e7b72e 100644 (file)
--- a/js/tt-rss.js
+++ b/js/tt-rss.js
@@ -365,7 +365,6 @@ function init_hotkey_actions() {
        hotkey_actions["open_in_new_window"] = function() {
                if (getActiveArticleId()) {
                        openArticleInNewWindow(getActiveArticleId());
-                       return;
                }
        };
        hotkey_actions["catchup_below"] = function() {
@@ -375,13 +374,9 @@ function init_hotkey_actions() {
                catchupRelativeToArticle(0);
        };
        hotkey_actions["article_scroll_down"] = function() {
-               var ctr = $("content_insert") ? $("content_insert") : $("headlines-frame");
-
                scrollArticle(40);
        };
        hotkey_actions["article_scroll_up"] = function() {
-               var ctr = $("content_insert") ? $("content_insert") : $("headlines-frame");
-
                scrollArticle(-40);
        };
        hotkey_actions["close_article"] = function() {