From 6436dd16f94415dc120e2d563c0f480f1f002183 Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <noreply@fakecake.org>
Date: Mon, 6 Jun 2022 09:29:16 +0300
Subject: [PATCH] filter-tree: move external dojo/method to PrefFilterTree
 class

---
 classes/pref/filters.php |  8 --------
 js/PrefFilterTree.js     | 11 ++++++++++-
 2 files changed, 10 insertions(+), 9 deletions(-)

diff --git a/classes/pref/filters.php b/classes/pref/filters.php
index 79dd78993..8f1c578b6 100755
--- a/classes/pref/filters.php
+++ b/classes/pref/filters.php
@@ -703,14 +703,6 @@ class Pref_Filters extends Handler_Protected {
 				</div>
 				<div dojoType="fox.PrefFilterTree" id="filterTree" dndController="dijit.tree.dndSource"
 					betweenThreshold="5" model="filterModel" openOnClick="true">
-					<script type="dojo/method" event="onClick" args="item">
-						var id = String(item.id);
-						var bare_id = id.substr(id.indexOf(':')+1);
-
-						if (id.match('FILTER:')) {
-							Filters.edit(bare_id);
-						}
-					</script>
 				</div>
 			</div>
 			<?php PluginHost::getInstance()->run_hooks(PluginHost::HOOK_PREFS_TAB, "prefFilters") ?>
diff --git a/js/PrefFilterTree.js b/js/PrefFilterTree.js
index d4496b647..149261abd 100644
--- a/js/PrefFilterTree.js
+++ b/js/PrefFilterTree.js
@@ -1,5 +1,5 @@
 /* eslint-disable prefer-rest-params */
-/* global __, define, lib, dijit, dojo, xhr, App, Notify */
+/* global __, define, lib, dijit, dojo, xhr, App, Notify, Filters */
 
 define(["dojo/_base/declare", "dojo/dom-construct", "lib/CheckBoxTree"], function (declare, domConstruct) {
 
@@ -10,6 +10,15 @@ define(["dojo/_base/declare", "dojo/dom-construct", "lib/CheckBoxTree"], functio
 			dijit.byId('filterTree').hideOrShowFilterRules(
 				parseInt(localStorage.getItem("ttrss:hide-filter-rules"))
 			);
+
+			dojo.connect(this, 'onClick', (item) => {
+				const id = String(item.id);
+				const bare_id = id.substr(id.indexOf(':')+1);
+
+				if (id.match('FILTER:')) {
+					Filters.edit(bare_id);
+				}
+			});
 		},
 		_createTreeNode: function(args) {
 			const tnode = this.inherited(arguments);