diff options
author | Andrew Dolgov <[email protected]> | 2010-11-08 09:56:08 +0300 |
---|---|---|
committer | Andrew Dolgov <[email protected]> | 2010-11-08 09:56:08 +0300 |
commit | b83d72b140b7810b834b2fd29242170878ff4994 (patch) | |
tree | 08ebd14e02e8f83affd316e2f65f43333838f3da /viewfeed.js | |
parent | 2a5c136e963a67906444821582973c09b180db7a (diff) |
js: code cleanup
Diffstat (limited to 'viewfeed.js')
-rw-r--r-- | viewfeed.js | 57 |
1 files changed, 9 insertions, 48 deletions
diff --git a/viewfeed.js b/viewfeed.js index 2ba2f637d..1a40c9649 100644 --- a/viewfeed.js +++ b/viewfeed.js @@ -1003,13 +1003,7 @@ function selectionAssignLabel(id) { function selectionToggleUnread(set_state, callback_func, no_error) { try { - var rows; - - if (isCdmMode()) { - rows = cdmGetSelectedArticles(); - } else { - rows = getSelectedTableRowIds("headlinesList", "RROW", "RCHK"); - } + var rows = getSelectedArticleIds2(); if (rows.length == 0 && !no_error) { alert(__("No articles are selected.")); @@ -1088,14 +1082,8 @@ function selectionToggleUnread(set_state, callback_func, no_error) { function selectionToggleMarked() { try { - var rows; + var rows = getSelectedArticleIds2(); - if (isCdmMode()) { - rows = cdmGetSelectedArticles(); - } else { - rows = getSelectedTableRowIds("headlinesList", "RROW", "RCHK"); - } - if (rows.length == 0) { alert(__("No articles are selected.")); return; @@ -1132,13 +1120,7 @@ function selectionToggleMarked() { function selectionTogglePublished() { try { - var rows; - - if (isCdmMode()) { - rows = cdmGetSelectedArticles(); - } else { - rows = getSelectedTableRowIds("headlinesList", "RROW", "RCHK"); - } + var rows = getSelectedArticleIds2(); if (rows.length == 0) { alert(__("No articles are selected.")); @@ -1278,21 +1260,14 @@ function catchupPage() { function deleteSelection() { try { - - var rows; - - if ($("headlinesList")) { - rows = getSelectedTableRowIds("headlinesList", "RROW", "RCHK"); - } else { - rows = cdmGetSelectedArticles(); - } + var rows = getSelectedArticleIds2(); + if (rows.length == 0) { alert(__("No articles are selected.")); return; } - var fn = getFeedName(getActiveFeedId(), activeFeedIsCat()); var str; var op; @@ -1329,20 +1304,13 @@ function archiveSelection() { try { - var rows; - - if ($("headlinesList")) { - rows = getSelectedTableRowIds("headlinesList", "RROW", "RCHK"); - } else { - rows = cdmGetSelectedArticles(); - } - + var rows = getSelectedArticleIds2(); + if (rows.length == 0) { alert(__("No articles are selected.")); return; } - var fn = getFeedName(getActiveFeedId(), activeFeedIsCat()); var str; var op; @@ -1385,20 +1353,13 @@ function catchupSelection() { try { - var rows; - - if ($("headlinesList")) { - rows = getSelectedTableRowIds("headlinesList", "RROW", "RCHK"); - } else { - rows = cdmGetSelectedArticles(); - } - + var rows = getSelectedArticleIds2(); + if (rows.length == 0) { alert(__("No articles are selected.")); return; } - var fn = getFeedName(getActiveFeedId(), activeFeedIsCat()); var str = __("Mark %d selected articles in %s as read?"); |