From a740f4b51d4810ea10022915051284a812232c18 Mon Sep 17 00:00:00 2001 From: Andrew Dolgov Date: Thu, 18 Nov 2010 15:49:26 +0300 Subject: [PATCH] tweak feed-prefs; rework filter prefs to use dijit.Tree --- PrefFilterTree.js | 57 +++++++ modules/pref-feeds.php | 4 + modules/pref-filters.php | 312 ++++++++++++++------------------------- prefs.js | 17 ++- tt-rss.css | 28 +++- 5 files changed, 212 insertions(+), 206 deletions(-) create mode 100644 PrefFilterTree.js diff --git a/PrefFilterTree.js b/PrefFilterTree.js new file mode 100644 index 000000000..b938517d5 --- /dev/null +++ b/PrefFilterTree.js @@ -0,0 +1,57 @@ +dojo.provide("fox.PrefFilterTree"); + +dojo.require("lib.CheckBoxTree"); + +dojo.declare("fox.PrefFilterTree", lib.CheckBoxTree, { + _createTreeNode: function(args) { + var tnode = this.inherited(arguments); + + var enabled = this.model.store.getValue(args.item, 'enabled'); + var param = this.model.store.getValue(args.item, 'param'); + + if (param) { + param = dojo.doc.createElement('span'); + param.className = (enabled != false) ? 'labelParam' : 'labelParam Disabled'; + param.innerHTML = args.item.param[0]; + dojo.place(param, tnode.labelNode, 'after'); + } + + return tnode; + }, + + getLabel: function(item) { + var label = item.name; + + var feed = this.model.store.getValue(item, 'feed'); + var inverse = this.model.store.getValue(item, 'inverse'); + + if (feed) + label += " (" + __("Feed:") + " " + feed + ")"; + + if (inverse) + label += " (" + __("Inverse") + ")"; + +/* if (item.param) + label = "" + label + + "" + item.param[0]; */ + + return label; + }, + getLabelClass: function (item, opened) { + var enabled = this.model.store.getValue(item, 'enabled'); + return (enabled != false) ? "dijitTreeLabel labelFixedLength" : "dijitTreeLabel labelFixedLength Disabled"; + }, + getRowClass: function (item, opened) { + return (!item.error || item.error == '') ? "dijitTreeRow" : + "dijitTreeRow Error"; + }, + onClick: function (item) { + var id = String(item.id); + var bare_id = id.substr(id.indexOf(':')+1); + + if (id.match('FILTER:')) { + editFilter(bare_id, event); + } + }, +}); + diff --git a/modules/pref-feeds.php b/modules/pref-feeds.php index 67adeb66d..7671d3a67 100644 --- a/modules/pref-feeds.php +++ b/modules/pref-feeds.php @@ -1247,6 +1247,10 @@ "; + print "
+ Hint: you can drag feeds and categories around. +
"; + print ""; # feeds pane print "
"; diff --git a/modules/pref-filters.php b/modules/pref-filters.php index 153707f6f..e419fa991 100644 --- a/modules/pref-filters.php +++ b/modules/pref-filters.php @@ -3,6 +3,100 @@ $subop = $_REQUEST["subop"]; $quiet = $_REQUEST["quiet"]; + if ($subop == "getfiltertree") { + $root = array(); + $root['id'] = 'root'; + $root['name'] = __('Filters'); + $root['items'] = array(); + + $result = db_query($link, "SELECT + ttrss_filters.id AS id,reg_exp, + ttrss_filter_types.name AS filter_type_name, + ttrss_filter_types.description AS filter_type_descr, + enabled, + inverse, + feed_id, + action_id, + filter_param, + filter_type, + ttrss_filter_actions.description AS action_description, + ttrss_feeds.title AS feed_title, + ttrss_filter_actions.name AS action_name, + ttrss_filters.action_param AS action_param + FROM + ttrss_filter_types,ttrss_filter_actions,ttrss_filters LEFT JOIN + ttrss_feeds ON (ttrss_filters.feed_id = ttrss_feeds.id) + WHERE + filter_type = ttrss_filter_types.id AND + ttrss_filter_actions.id = action_id AND + ttrss_filters.owner_uid = ".$_SESSION["uid"]." + ORDER by action_description, reg_exp"); + + $cat = false; + $cur_action_description = ""; + + while ($line = db_fetch_assoc($result)) { + if ($cur_action_description != $line['action_description']) { + + if ($cat) + array_push($root['items'], $cat); + + $cat = array(); + $cat['id'] = 'ACTION:' . $line['action_id']; + $cat['name'] = $line['action_description']; + $cat['items'] = array(); + + $cur_action_description = $line['action_description']; + } + + if (array_search($line["action_name"], + array("score", "tag", "label")) === false) { + + $line["action_param"] = ''; + } else { + if ($line['action_name'] == 'label') { + + $tmp_result = db_query($link, "SELECT fg_color, bg_color + FROM ttrss_labels2 WHERE caption = '". + db_escape_string($line["action_param"])."' AND + owner_uid = " . $_SESSION["uid"]); + + $fg_color = db_fetch_result($tmp_result, 0, "fg_color"); + $bg_color = db_fetch_result($tmp_result, 0, "bg_color"); + + $tmp = "α " . $line['action_param']; + + $line['action_param'] = $tmp; + } + } + + $filter = array(); + $filter['id'] = 'FILTER:' . $line['id']; + $filter['bare_id'] = $line['id']; + $filter['name'] = $line['reg_exp']; + $filter['type'] = $line['filter_type']; + $filter['enabled'] = sql_bool_to_bool($line['enabled']); + $filter['param'] = $line['action_param']; + $filter['inverse'] = sql_bool_to_bool($line['inverse']); + $filter['checkbox'] = false; + + if ($line['feed_id']) + $filter['feed'] = $line['feed_title']; + + array_push($cat['items'], $filter); + } + + array_push($root['items'], $cat); + + $fl = array(); + $fl['identifier'] = 'id'; + $fl['label'] = 'name'; + $fl['items'] = array($root); + + print json_encode($fl); + return; + } + if ($subop == "edit") { $filter_id = db_escape_string($_REQUEST["id"]); @@ -309,9 +403,9 @@ print "
". "" . __('Select').""; print "
"; - print "
".__('All')."
"; - print "
".__('None')."
"; print "
"; @@ -333,205 +427,23 @@ print "
"; # toolbar-frame print "
"; - if ($filter_search) { - $filter_search = split(' ', db_escape_string($filter_search)); + print "
+ ". + __("Loading, please wait...")."
"; - $tokens = array(); - - foreach ($filter_search as $token) { - $token = trim($token); - - array_push($tokens, "( - UPPER(ttrss_filter_actions.description) LIKE UPPER('%$token%') OR - UPPER(reg_exp) LIKE UPPER('%$token%') OR - UPPER(action_param) LIKE UPPER('%$token%') OR - UPPER(ttrss_feeds.title) LIKE UPPER('%$token%') OR - UPPER(ttrss_filter_types.description) LIKE UPPER('%$token%'))"); - } - - $filter_search_query = "(" . join($tokens, " AND ") . ") AND "; - - } else { - $filter_search_query = ""; - } - - $result = db_query($link, "SELECT - ttrss_filters.id AS id,reg_exp, - ttrss_filter_types.name AS filter_type_name, - ttrss_filter_types.description AS filter_type_descr, - enabled, - inverse, - feed_id, - filter_param, - filter_type, - ttrss_filter_actions.description AS action_description, - ttrss_feeds.title AS feed_title, - ttrss_filter_actions.name AS action_name, - ttrss_filters.action_param AS action_param - FROM - ttrss_filter_types,ttrss_filter_actions,ttrss_filters LEFT JOIN - ttrss_feeds ON (ttrss_filters.feed_id = ttrss_feeds.id) - WHERE - filter_type = ttrss_filter_types.id AND - $filter_search_query - ttrss_filter_actions.id = action_id AND - ttrss_filters.owner_uid = ".$_SESSION["uid"]." - ORDER by action_description, $sort"); - - if (db_num_rows($result) != 0) { - - print "

"; - - print ""; - - print " - - - - - "; - - $lnum = 0; - } - - $class = ($lnum % 2) ? "even" : "odd"; - - print ""; - - $line["reg_exp"] = htmlspecialchars($line["reg_exp"]); - - if (!$line["feed_title"]) $line["feed_title"] = __("All feeds"); - - if (array_search($line["action_name"], - array("score", "tag", "label")) === false) { - - $line["action_param"] = false; - } - - if (!$line["action_param"]) { - $line["action_param"] = "—"; - } else if ($line["action_name"] == "score") { - - $score_pic = theme_image($link, - "images/" . get_score_pic($line["action_param"])); - - $score_pic = ""; - - $line["action_param"] = "$score_pic " . $line["action_param"]; - - } - - $line["feed_title"] = htmlspecialchars($line["feed_title"]); - - print ""; - - $filter_params = array( - "before" => __("before"), - "after" => __("after")); - - if ($line["action_name"] == 'label') { - - $tmp_result = db_query($link, "SELECT fg_color, bg_color - FROM ttrss_labels2 WHERE caption = '". - db_escape_string($line["action_param"])."' AND - owner_uid = " . $_SESSION["uid"]); - - $fg_color = db_fetch_result($tmp_result, 0, "fg_color"); - $bg_color = db_fetch_result($tmp_result, 0, "bg_color"); - - $tmp = "
- α"; - $tmp .= "
"; - - $line["action_param"] = "$tmp " . $line["action_param"]; - } - - if ($line["filter_type"] == 5) { - - if (!strtotime($line["reg_exp"])) { - $line["reg_exp"] = "" . - $line["reg_exp"] . ""; - } - - $line["reg_exp"] = __("Date") . " " . - $filter_params[$line['filter_param']] . " " . - $line["reg_exp"]; - } - - if (!$enabled) { - $line["reg_exp"] = "" . - $line["reg_exp"] . " " . __("(Disabled)").""; - $line["feed_title"] = "" . - $line["feed_title"] . ""; - $line["filter_type_descr"] = "" . - $line["filter_type_descr"] . ""; - $line["action_description"] = "" . - $line["action_description"] . ""; - $line["action_param"] = "" . - $line["action_param"] . ""; - } - - $onclick = "onclick='editFilter($filter_id, event)' title='".__('Click to edit')."'"; - - $inverse_label = ""; - - if ($inverse) { - $inverse_label = " ".__('(Inverse)').""; - } - - print ""; - print ""; - - print ""; - print ""; - - print ""; - - ++$lnum; - } - - print "
- ".__('Select:')." - ".__('All').", - ".__('None')." - "; - - $lnum = 0; - - $cur_action_description = ""; - - while ($line = db_fetch_assoc($result)) { - - $filter_id = $line["id"]; - $edit_filter_id = $_REQUEST["id"]; - - $enabled = sql_bool_to_bool($line["enabled"]); - $inverse = sql_bool_to_bool($line["inverse"]); - - $this_row_id = "id=\"FILRR-$filter_id\""; - - $line["filter_type_descr"] = __($line["filter_type_descr"]); - $line["action_description"] = __($line["action_description"]); - - if ($line["action_description"] != $cur_action_description) { - $cur_action_description = $line["action_description"]; - - print "
$cur_action_description
 ".__('Match')."".__('Feed')."".__('Field')."".__('Params')."
" . $line["reg_exp"] . "$inverse_label" . $line["feed_title"] . "" . $line["filter_type_descr"] . "" . $line["action_param"] . "
"; - - } else { - - print "

"; - if (!$filter_search) { - print_warning(__('No filters defined.')); - } else { - print_warning(__('No matching filters found.')); - } - print "

"; - - } + print "
+
+
+
+
+ +
"; print "
"; #pane print ""; #container diff --git a/prefs.js b/prefs.js index 0e77118db..5dc9e7702 100644 --- a/prefs.js +++ b/prefs.js @@ -356,10 +356,7 @@ function editFilter(id, event) { if (!event || !event.ctrlKey) { - notify_progress("Loading, please wait..."); - - selectTableRows('prefFilterList', 'none'); - selectTableRowById('FILRR-'+id, 'FICHK-'+id, true); + notify_progress("Loading, please wait...", true); var query = "?op=pref-filters&subop=edit&id=" + param_escape(id); @@ -435,7 +432,16 @@ function getSelectedFeeds() { } function getSelectedFilters() { - return getSelectedTableRowIds("prefFilterList"); + var tree = dijit.byId("filterTree"); + var items = tree.model.getCheckedItems(); + var rv = []; + + items.each(function(item) { + rv.push(tree.model.store.getValue(item, 'bare_id')); + }); + + return rv; + } function getSelectedFeedCats() { @@ -1172,6 +1178,7 @@ function init() { dojo.require("lib.CheckBoxTree"); dojo.require("fox.PrefFeedTree"); + dojo.require("fox.PrefFilterTree"); loading_set_progress(30); diff --git a/tt-rss.css b/tt-rss.css index 4cab27e9f..0eba89c13 100644 --- a/tt-rss.css +++ b/tt-rss.css @@ -1271,6 +1271,19 @@ span.prefsLabelEntry { padding : 2px; } +span.labelColorIndicator { + height : 14px; + width : 14px; + line-height : 14px; + font-height : 9px; + display : inline-block; + border : 1px solid black; + background-color : #fff7d5; + color : #063064; + text-align : center; + margin-right : 2px; +} + div.labelColorIndicator { height : 14px; width : 14px; @@ -1513,6 +1526,10 @@ div.fatalError textarea { font-weight : bold; } +.dijitTreeLabel.Disabled, .labelParam.Disabled { + color : gray; +} + .dijitTreeRow.Error { color : red; } @@ -1526,12 +1543,16 @@ img.feedIcon, img.tinyFeedIcon { height : 16px; } -div#feedlistLoading { +div#feedlistLoading, div#filterlistLoading { text-align : center; padding : 5px; color : gray; } +div#feedlistLoading img, div#filterlistLoading img { + margin-right : 5px; +} + div#pref-tabs .dijitContentPane { font-size : 12px; } @@ -1562,3 +1583,8 @@ div#pref-tabs .dijitContentPane h3 { #errorButton { color : red; } + +span.labelFixedLength { + display : inline-block; + width : 70%; +}