X-Git-Url: https://git.wh0rd.org/?a=blobdiff_plain;f=classes%2Fpref%2Fusers.php;h=ab8ac775b54319c77ff51d680a701efe4af6d59b;hb=3a0292303e453f38204279b5d1c978a4b9c367e9;hp=d97362f9f8e64b20e09cd4d7def354d0c213b26e;hpb=2a3e8d7f3449c372d79ec525c251e0eeb791e170;p=tt-rss.git

diff --git a/classes/pref/users.php b/classes/pref/users.php
index d97362f9..ab8ac775 100644
--- a/classes/pref/users.php
+++ b/classes/pref/users.php
@@ -12,226 +12,236 @@ class Pref_Users extends Handler_Protected {
 		}
 
 		function csrf_ignore($method) {
-			$csrf_ignored = array("index");
+			$csrf_ignored = array("index", "edit", "userdetails");
 
 			return array_search($method, $csrf_ignored) !== false;
 		}
 
-		function userdetails() {
+		function edit() {
+			global $access_level_names;
 
-			$uid = sprintf("%d", $_REQUEST["id"]);
+			print "<form id=\"user_edit_form\" onsubmit='return false' dojoType=\"dijit.form.Form\">";
 
-			$result = $this->dbh->query("SELECT login,
-				".SUBSTRING_FOR_DATE."(last_login,1,16) AS last_login,
-				access_level,
-				(SELECT COUNT(int_id) FROM ttrss_user_entries
-					WHERE owner_uid = id) AS stored_articles,
-				".SUBSTRING_FOR_DATE."(created,1,16) AS created
-				FROM ttrss_users
-				WHERE id = '$uid'");
+			print '<div dojoType="dijit.layout.TabContainer" style="height : 400px">
+        		<div dojoType="dijit.layout.ContentPane" title="'.__('Edit user').'">';
 
-			if ($this->dbh->num_rows($result) == 0) {
-				print "<h1>".__('User not found')."</h1>";
-				return;
-			}
+			//print "<form id=\"user_edit_form\" onsubmit='return false' dojoType=\"dijit.form.Form\">";
 
-			// print "<h1>User Details</h1>";
+			$id = (int) clean($_REQUEST["id"]);
 
-			$login = $this->dbh->fetch_result($result, 0, "login");
+			print_hidden("id", "$id");
+			print_hidden("op", "pref-users");
+			print_hidden("method", "editSave");
 
-			print "<table width='100%'>";
+			$sth = $this->pdo->prepare("SELECT * FROM ttrss_users WHERE id = ?");
+			$sth->execute([$id]);
 
-			$last_login = make_local_datetime(
-				$this->dbh->fetch_result($result, 0, "last_login"), true);
+			if ($row = $sth->fetch()) {
 
-			$created = make_local_datetime(
-				$this->dbh->fetch_result($result, 0, "created"), true);
+				$login = $row["login"];
+				$access_level = $row["access_level"];
+				$email = $row["email"];
 
-			$access_level = $this->dbh->fetch_result($result, 0, "access_level");
-			$stored_articles = $this->dbh->fetch_result($result, 0, "stored_articles");
+				$sel_disabled = ($id == $_SESSION["uid"] || $login == "admin") ? "disabled" : "";
 
-			print "<tr><td>".__('Registered')."</td><td>$created</td></tr>";
-			print "<tr><td>".__('Last logged in')."</td><td>$last_login</td></tr>";
+				print "<div class=\"dlgSec\">".__("User")."</div>";
+				print "<div class=\"dlgSecCont\">";
 
-			$result = $this->dbh->query("SELECT COUNT(id) as num_feeds FROM ttrss_feeds
-				WHERE owner_uid = '$uid'");
+				if ($sel_disabled) {
+					print_hidden("login", "$login");
+				}
 
-			$num_feeds = $this->dbh->fetch_result($result, 0, "num_feeds");
+				print "<input size=\"30\" style=\"font-size : 16px\"
+					dojoType=\"dijit.form.ValidationTextBox\" required=\"1\"
+					$sel_disabled
+					name=\"login\" value=\"$login\">";
 
-			print "<tr><td>".__('Subscribed feeds count')."</td><td>$num_feeds</td></tr>";
+				print "</div>";
 
-			print "</table>";
+				print "<div class=\"dlgSec\">".__("Authentication")."</div>";
+				print "<div class=\"dlgSecCont\">";
 
-			print "<h1>".__('Subscribed feeds')."</h1>";
+				print __('Access level: ') . " ";
 
-			$result = $this->dbh->query("SELECT id,title,site_url FROM ttrss_feeds
-				WHERE owner_uid = '$uid' ORDER BY title");
+				if (!$sel_disabled) {
+					print_select_hash("access_level", $access_level, $access_level_names,
+						"dojoType=\"dijit.form.Select\" $sel_disabled");
+				} else {
+					print_select_hash("", $access_level, $access_level_names,
+						"dojoType=\"dijit.form.Select\" $sel_disabled");
+					print_hidden("access_level", "$access_level");
+				}
 
-			print "<ul class=\"userFeedList\">";
+				print "<hr/>";
 
-			while ($line = $this->dbh->fetch_assoc($result)) {
+				print "<input dojoType=\"dijit.form.TextBox\" type=\"password\" size=\"20\" placeholder=\"Change password\"
+				name=\"password\">";
 
-				$icon_file = ICONS_URL."/".$line["id"].".ico";
+				print "</div>";
 
-				if (file_exists($icon_file) && filesize($icon_file) > 0) {
-					$feed_icon = "<img class=\"tinyFeedIcon\" src=\"$icon_file\">";
-				} else {
-					$feed_icon = "<img class=\"tinyFeedIcon\" src=\"images/blank_icon.gif\">";
-				}
+				print "<div class=\"dlgSec\">".__("Options")."</div>";
+				print "<div class=\"dlgSecCont\">";
 
-				print "<li>$feed_icon&nbsp;<a href=\"".$line["site_url"]."\">".$line["title"]."</a></li>";
+				print "<input dojoType=\"dijit.form.TextBox\" size=\"30\" name=\"email\" placeholder=\"E-mail\"
+				value=\"$email\">";
 
-			}
+				print "</div>";
+
+				print "</table>";
 
-			if ($this->dbh->num_rows($result) < $num_feeds) {
-				// FIXME - add link to show ALL subscribed feeds here somewhere
-				print "<li><img
-					class=\"tinyFeedIcon\" src=\"images/blank_icon.gif\">&nbsp;...</li>";
 			}
 
-			print "</ul>";
+			print '</div>'; #tab
+			print "<div href=\"backend.php?op=pref-users&method=userdetails&id=$id\"
+				dojoType=\"dijit.layout.ContentPane\" title=\"".__('User details')."\">";
+
+			print '</div>';
+			print '</div>';
+
+			print "<div class=\"dlgButtons\">
+				<button dojoType=\"dijit.form.Button\" class=\"btn-primary\" type=\"submit\" onclick=\"dijit.byId('userEditDlg').execute()\">".
+				__('Save')."</button>
+				<button dojoType=\"dijit.form.Button\" onclick=\"dijit.byId('userEditDlg').hide()\">".
+				__('Cancel')."</button></div>";
 
-			print "<div align='center'>
-				<button onclick=\"closeInfoBox()\">".__("Close this window").
-				"</button></div>";
+			print "</form>";
 
 			return;
 		}
 
-		function edit() {
-			global $access_level_names;
+		function userdetails() {
+			$id = (int) clean($_REQUEST["id"]);
 
-			$id = $this->dbh->escape_string($_REQUEST["id"]);
-			print "<form id=\"user_edit_form\" onsubmit='return false'>";
+			$sth = $this->pdo->prepare("SELECT login,
+				".SUBSTRING_FOR_DATE."(last_login,1,16) AS last_login,
+				access_level,
+				(SELECT COUNT(int_id) FROM ttrss_user_entries
+					WHERE owner_uid = id) AS stored_articles,
+				".SUBSTRING_FOR_DATE."(created,1,16) AS created
+				FROM ttrss_users
+				WHERE id = ?");
+			$sth->execute([$id]);
 
-			print "<input type=\"hidden\" name=\"id\" value=\"$id\">";
-			print "<input type=\"hidden\" name=\"op\" value=\"pref-users\">";
-			print "<input type=\"hidden\" name=\"method\" value=\"editSave\">";
+			if ($row = $sth->fetch()) {
+				print "<table width='100%'>";
 
-			$result = $this->dbh->query("SELECT * FROM ttrss_users WHERE id = '$id'");
+				$last_login = make_local_datetime(
+					$row["last_login"], true);
 
-			$login = $this->dbh->fetch_result($result, 0, "login");
-			$access_level = $this->dbh->fetch_result($result, 0, "access_level");
-			$email = $this->dbh->fetch_result($result, 0, "email");
+				$created = make_local_datetime(
+					$row["created"], true);
 
-			$sel_disabled = ($id == $_SESSION["uid"]) ? "disabled" : "";
+				$stored_articles = $row["stored_articles"];
 
-			print "<div class=\"dlgSec\">".__("User")."</div>";
-			print "<div class=\"dlgSecCont\">";
+				print "<tr><td>".__('Registered')."</td><td>$created</td></tr>";
+				print "<tr><td>".__('Last logged in')."</td><td>$last_login</td></tr>";
 
-			if ($sel_disabled) {
-				print "<input type=\"hidden\" name=\"login\" value=\"$login\">";
-				print "<input size=\"30\" style=\"font-size : 16px\"
-					onkeypress=\"return filterCR(event, userEditSave)\" $sel_disabled
-					value=\"$login\">";
-			} else {
-				print "<input size=\"30\" style=\"font-size : 16px\"
-					onkeypress=\"return filterCR(event, userEditSave)\" $sel_disabled
-					name=\"login\" value=\"$login\">";
-			}
+				$sth = $this->pdo->prepare("SELECT COUNT(id) as num_feeds FROM ttrss_feeds
+					WHERE owner_uid = ?");
+				$sth->execute([$id]);
+				$row = $sth->fetch();
+				$num_feeds = $row["num_feeds"];
 
-			print "</div>";
+				print "<tr><td>".__('Subscribed feeds count')."</td><td>$num_feeds</td></tr>";
+				print "<tr><td>".__('Stored articles')."</td><td>$stored_articles</td></tr>";
 
-			print "<div class=\"dlgSec\">".__("Authentication")."</div>";
-			print "<div class=\"dlgSecCont\">";
+				print "</table>";
 
-			print __('Access level: ') . " ";
+				print "<h1>".__('Subscribed feeds')."</h1>";
 
-			if (!$sel_disabled) {
-				print_select_hash("access_level", $access_level, $access_level_names,
-					$sel_disabled);
-			} else {
-				print_select_hash("", $access_level, $access_level_names,
-					$sel_disabled);
-				print "<input type=\"hidden\" name=\"access_level\" value=\"$access_level\">";
-			}
+				$sth = $this->pdo->prepare("SELECT id,title,site_url FROM ttrss_feeds
+					WHERE owner_uid = ? ORDER BY title");
+				$sth->execute([$id]);
 
-			print "<br/>";
+				print "<ul class=\"userFeedList\">";
 
-			print __('Change password to') .
-				" <input type=\"password\" size=\"20\" onkeypress=\"return filterCR(event, userEditSave)\"
-				name=\"password\">";
+				while ($line = $sth->fetch()) {
 
-			print "</div>";
+					$icon_file = ICONS_URL."/".$line["id"].".ico";
 
-			print "<div class=\"dlgSec\">".__("Options")."</div>";
-			print "<div class=\"dlgSecCont\">";
+					if (file_exists($icon_file) && filesize($icon_file) > 0) {
+						$feed_icon = "<img class=\"tinyFeedIcon\" src=\"$icon_file\">";
+					} else {
+						$feed_icon = "<img class=\"tinyFeedIcon\" src=\"images/blank_icon.gif\">";
+					}
 
-			print __('E-mail: ').
-				" <input size=\"30\" name=\"email\" onkeypress=\"return filterCR(event, userEditSave)\"
-				value=\"$email\">";
+					print "<li>$feed_icon&nbsp;<a href=\"".$line["site_url"]."\">".$line["title"]."</a></li>";
 
-			print "</div>";
+				}
 
-			print "</table>";
+				print "</ul>";
 
-			print "</form>";
 
-			print "<div class=\"dlgButtons\">
-				<button onclick=\"return userEditSave()\">".
-					__('Save')."</button>
-				<button onclick=\"return userEditCancel()\">".
-					__('Cancel')."</button></div>";
+			} else {
+				print "<h1>".__('User not found')."</h1>";
+			}
 
-			return;
 		}
 
 		function editSave() {
-			$login = $this->dbh->escape_string(trim($_REQUEST["login"]));
-			$uid = $this->dbh->escape_string($_REQUEST["id"]);
-			$access_level = (int) $_REQUEST["access_level"];
-			$email = $this->dbh->escape_string(trim($_REQUEST["email"]));
-			$password = $_REQUEST["password"];
+			$login = trim(clean($_REQUEST["login"]));
+			$uid = clean($_REQUEST["id"]);
+			$access_level = (int) clean($_REQUEST["access_level"]);
+			$email = trim(clean($_REQUEST["email"]));
+			$password = clean($_REQUEST["password"]);
 
 			if ($password) {
 				$salt = substr(bin2hex(get_random_bytes(125)), 0, 250);
 				$pwd_hash = encrypt_password($password, $salt, true);
-				$pass_query_part = "pwd_hash = '$pwd_hash', salt = '$salt',";
+				$pass_query_part = "pwd_hash = ".$this->pdo->quote($pwd_hash).",
+					salt = ".$this->pdo->quote($salt).",";
 			} else {
 				$pass_query_part = "";
 			}
 
-			$this->dbh->query("UPDATE ttrss_users SET $pass_query_part login = '$login',
-				access_level = '$access_level', email = '$email', otp_enabled = false
-				WHERE id = '$uid'");
+			$sth = $this->pdo->prepare("UPDATE ttrss_users SET $pass_query_part login = ?,
+				access_level = ?, email = ?, otp_enabled = false WHERE id = ?");
+			$sth->execute([$login, $access_level, $email, $uid]);
 
 		}
 
 		function remove() {
-			$ids = explode(",", $this->dbh->escape_string($_REQUEST["ids"]));
+			$ids = explode(",", clean($_REQUEST["ids"]));
 
 			foreach ($ids as $id) {
 				if ($id != $_SESSION["uid"] && $id != 1) {
-					$this->dbh->query("DELETE FROM ttrss_tags WHERE owner_uid = '$id'");
-					$this->dbh->query("DELETE FROM ttrss_feeds WHERE owner_uid = '$id'");
-					$this->dbh->query("DELETE FROM ttrss_users WHERE id = '$id'");
+					$sth = $this->pdo->prepare("DELETE FROM ttrss_tags WHERE owner_uid = ?");
+					$sth->execute([$id]);
+
+					$sth = $this->pdo->prepare("DELETE FROM ttrss_feeds WHERE owner_uid = ?");
+					$sth->execute([$id]);
+
+					$sth = $this->pdo->prepare("DELETE FROM ttrss_users WHERE id = ?");
+					$sth->execute([$id]);
 				}
 			}
 		}
 
 		function add() {
 
-			$login = $this->dbh->escape_string(trim($_REQUEST["login"]));
+			$login = trim(clean($_REQUEST["login"]));
 			$tmp_user_pwd = make_password(8);
 			$salt = substr(bin2hex(get_random_bytes(125)), 0, 250);
 			$pwd_hash = encrypt_password($tmp_user_pwd, $salt, true);
 
-			$result = $this->dbh->query("SELECT id FROM ttrss_users WHERE
-				login = '$login'");
+			$sth = $this->pdo->prepare("SELECT id FROM ttrss_users WHERE
+				login = ?");
+			$sth->execute([$login]);
 
-			if ($this->dbh->num_rows($result) == 0) {
+			if (!$sth->fetch()) {
 
-				$this->dbh->query("INSERT INTO ttrss_users
+				$sth = $this->pdo->prepare("INSERT INTO ttrss_users
 					(login,pwd_hash,access_level,last_login,created, salt)
-					VALUES ('$login', '$pwd_hash', 0, null, NOW(), '$salt')");
-
+					VALUES (?, ?, 0, null, NOW(), ?)");
+				$sth->execute([$login, $pwd_hash, $salt]);
 
-				$result = $this->dbh->query("SELECT id FROM ttrss_users WHERE
-					login = '$login' AND pwd_hash = '$pwd_hash'");
+				$sth = $this->pdo->prepare("SELECT id FROM ttrss_users WHERE
+					login = ? AND pwd_hash = ?");
+				$sth->execute([$login, $pwd_hash]);
 
-				if ($this->dbh->num_rows($result) == 1) {
+				if ($row = $sth->fetch()) {
 
-					$new_uid = $this->dbh->fetch_result($result, 0, "id");
+					$new_uid = $row['id'];
 
 					print format_notice(T_sprintf("Added user <b>%s</b> with password <b>%s</b>",
 						$login, $tmp_user_pwd));
@@ -248,59 +258,67 @@ class Pref_Users extends Handler_Protected {
 			}
 		}
 
-		static function resetUserPassword($link, $uid, $show_password) {
+		static function resetUserPassword($uid, $show_password) {
 
-			$result = db_query($link, "SELECT login,email
-				FROM ttrss_users WHERE id = '$uid'");
+			$pdo = Db::pdo();
 
-			$login = db_fetch_result($result, 0, "login");
-			$email = db_fetch_result($result, 0, "email");
-			$salt = db_fetch_result($result, 0, "salt");
+			$sth = $pdo->prepare("SELECT login, email
+				FROM ttrss_users WHERE id = ?");
+			$sth->execute([$uid]);
 
-			$new_salt = substr(bin2hex(get_random_bytes(125)), 0, 250);
-			$tmp_user_pwd = make_password(8);
+			if ($row = $sth->fetch()) {
 
-			$pwd_hash = encrypt_password($tmp_user_pwd, $new_salt, true);
+				$login = $row["login"];
+				$email = $row["email"];
 
-			db_query($link, "UPDATE ttrss_users SET pwd_hash = '$pwd_hash', salt = '$new_salt'
-				WHERE id = '$uid'");
+				$new_salt = substr(bin2hex(get_random_bytes(125)), 0, 250);
+				$tmp_user_pwd = make_password(8);
 
-			if ($show_password) {
-				print T_sprintf("Changed password of user <b>%s</b> to <b>%s</b>", $login, $tmp_user_pwd);
-			} else {
-				print_notice(T_sprintf("Sending new password of user <b>%s</b> to <b>%s</b>", $login, $email));
-			}
+				$pwd_hash = encrypt_password($tmp_user_pwd, $new_salt, true);
+
+				$sth = $pdo->prepare("UPDATE ttrss_users
+					  SET pwd_hash = ?, salt = ?, otp_enabled = false
+					WHERE id = ?");
+				$sth->execute([$pwd_hash, $new_salt, $uid]);
+
+				if ($show_password) {
+					print T_sprintf("Changed password of user <b>%s</b> to <b>%s</b>", $login, $tmp_user_pwd);
+				} else {
+					print_notice(T_sprintf("Sending new password of user <b>%s</b> to <b>%s</b>", $login, $email));
+				}
 
-			require_once 'classes/ttrssmailer.php';
+				require_once 'classes/ttrssmailer.php';
 
-			if ($email) {
-				require_once "lib/MiniTemplator.class.php";
+				if ($email) {
+					require_once "lib/MiniTemplator.class.php";
 
-				$tpl = new MiniTemplator;
+					$tpl = new MiniTemplator;
 
-				$tpl->readTemplateFromFile("templates/resetpass_template.txt");
+					$tpl->readTemplateFromFile("templates/resetpass_template.txt");
 
-				$tpl->setVariable('LOGIN', $login);
-				$tpl->setVariable('NEWPASS', $tmp_user_pwd);
+					$tpl->setVariable('LOGIN', $login);
+					$tpl->setVariable('NEWPASS', $tmp_user_pwd);
 
-				$tpl->addBlock('message');
+					$tpl->addBlock('message');
 
-				$message = "";
+					$message = "";
 
-				$tpl->generateOutputToString($message);
+					$tpl->generateOutputToString($message);
 
-				$mail = new ttrssMailer();
+					$mail = new ttrssMailer();
 
-				$rc = $mail->quickMail($email, $login,
-					__("[tt-rss] Password change notification"),
-					$message, false);
+					$rc = $mail->quickMail($email, $login,
+						__("[tt-rss] Password change notification"),
+						$message, false);
+
+					if (!$rc) print_error($mail->ErrorInfo);
+				}
 
-				if (!$rc) print_error($mail->ErrorInfo);
 			}
 		}
 
 		function resetPass() {
-			$uid = $this->dbh->escape_string($_REQUEST["id"]);
+			$uid = clean($_REQUEST["id"]);
 			Pref_Users::resetUserPassword($uid, true);
 		}
 
@@ -313,7 +331,7 @@ class Pref_Users extends Handler_Protected {
 
 			print "<div id=\"pref-user-toolbar\" dojoType=\"dijit.Toolbar\">";
 
-			$user_search = $this->dbh->escape_string($_REQUEST["search"]);
+			$user_search = trim(clean($_REQUEST["search"]));
 
 			if (array_key_exists("search", $_REQUEST)) {
 				$_SESSION["prefs_user_search"] = $user_search;
@@ -324,11 +342,11 @@ class Pref_Users extends Handler_Protected {
 			print "<div style='float : right; padding-right : 4px;'>
 				<input dojoType=\"dijit.form.TextBox\" id=\"user_search\" size=\"20\" type=\"search\"
 					value=\"$user_search\">
-				<button dojoType=\"dijit.form.Button\" onclick=\"javascript:updateUsersList()\">".
+				<button dojoType=\"dijit.form.Button\" onclick=\"updateUsersList()\">".
 					__('Search')."</button>
 				</div>";
 
-			$sort = $this->dbh->escape_string($_REQUEST["sort"]);
+			$sort = clean($_REQUEST["sort"]);
 
 			if (!$sort || $sort == "undefined") {
 				$sort = "login";
@@ -343,66 +361,55 @@ class Pref_Users extends Handler_Protected {
 				dojoType=\"dijit.MenuItem\">".__('None')."</div>";
 			print "</div></div>";
 
-			print "<button dojoType=\"dijit.form.Button\" onclick=\"javascript:addUser()\">".__('Create user')."</button>";
+			print "<button dojoType=\"dijit.form.Button\" onclick=\"addUser()\">".__('Create user')."</button>";
 
 			print "
-				<button dojoType=\"dijit.form.Button\" onclick=\"javascript:selectedUserDetails()\">".
-				__('Details')."</button dojoType=\"dijit.form.Button\">
-				<button dojoType=\"dijit.form.Button\" onclick=\"javascript:editSelectedUser()\">".
+				<button dojoType=\"dijit.form.Button\" onclick=\"editSelectedUser()\">".
 				__('Edit')."</button dojoType=\"dijit.form.Button\">
-				<button dojoType=\"dijit.form.Button\" onclick=\"javascript:removeSelectedUsers()\">".
+				<button dojoType=\"dijit.form.Button\" onclick=\"removeSelectedUsers()\">".
 				__('Remove')."</button dojoType=\"dijit.form.Button\">
-				<button dojoType=\"dijit.form.Button\" onclick=\"javascript:resetSelectedUserPass()\">".
+				<button dojoType=\"dijit.form.Button\" onclick=\"resetSelectedUserPass()\">".
 				__('Reset password')."</button dojoType=\"dijit.form.Button\">";
 
+			PluginHost::getInstance()->run_hooks(PluginHost::HOOK_PREFS_TAB_SECTION,
+				"hook_prefs_tab_section", "prefUsersToolbar");
+
 			print "</div>"; #toolbar
 			print "</div>"; #pane
 			print "<div id=\"pref-user-content\" dojoType=\"dijit.layout.ContentPane\" region=\"center\">";
 
-			print "<div id=\"sticky-status-msg\"></div>";
+			$sort = validate_field($sort,
+				["login", "access_level", "created", "num_feeds", "created", "last_login"], "login");
 
-			if ($user_search) {
+			if ($sort != "login") $sort = "$sort DESC";
 
-				$user_search = explode(" ", $user_search);
-				$tokens = array();
-
-				foreach ($user_search as $token) {
-					$token = trim($token);
-					array_push($tokens, "(UPPER(login) LIKE UPPER('%$token%'))");
-				}
-
-				$user_search_query = "(" . join($tokens, " AND ") . ") AND ";
-
-			} else {
-				$user_search_query = "";
-			}
-
-			$result = $this->dbh->query("SELECT
-					id,login,access_level,email,
+			$sth = $this->pdo->prepare("SELECT
+					tu.id,
+					login,access_level,email,
 					".SUBSTRING_FOR_DATE."(last_login,1,16) as last_login,
-					".SUBSTRING_FOR_DATE."(created,1,16) as created
+					".SUBSTRING_FOR_DATE."(created,1,16) as created,
+					(SELECT COUNT(id) FROM ttrss_feeds WHERE owner_uid = tu.id) AS num_feeds
 				FROM
-					ttrss_users
+					ttrss_users tu
 				WHERE
-					$user_search_query
-					id > 0
+					(:search = '' OR login LIKE :search) AND tu.id > 0
 				ORDER BY $sort");
-
-			if ($this->dbh->num_rows($result) > 0) {
+			$sth->execute([":search" => $user_search ? "%$user_search%" : ""]);
 
 			print "<p><table width=\"100%\" cellspacing=\"0\"
 				class=\"prefUserList\" id=\"prefUserList\">";
 
 			print "<tr class=\"title\">
 						<td align='center' width=\"5%\">&nbsp;</td>
-						<td width='30%'><a href=\"#\" onclick=\"updateUsersList('login')\">".__('Login')."</a></td>
-						<td width='30%'><a href=\"#\" onclick=\"updateUsersList('access_level')\">".__('Access Level')."</a></td>
+						<td width='20%'><a href=\"#\" onclick=\"updateUsersList('login')\">".__('Login')."</a></td>
+						<td width='20%'><a href=\"#\" onclick=\"updateUsersList('access_level')\">".__('Access Level')."</a></td>
+						<td width='10%'><a href=\"#\" onclick=\"updateUsersList('num_feeds')\">".__('Subscribed feeds')."</a></td>
 						<td width='20%'><a href=\"#\" onclick=\"updateUsersList('created')\">".__('Registered')."</a></td>
 						<td width='20%'><a href=\"#\" onclick=\"updateUsersList('last_login')\">".__('Last login')."</a></td></tr>";
 
 			$lnum = 0;
 
-			while ($line = $this->dbh->fetch_assoc($result)) {
+			while ($line = $sth->fetch()) {
 
 				$uid = $line["id"];
 
@@ -419,11 +426,12 @@ class Pref_Users extends Handler_Protected {
 
 				$onclick = "onclick='editUser($uid, event)' title='".__('Click to edit')."'";
 
-				print "<td $onclick>" . $line["login"] . "</td>";
+				print "<td $onclick><img src='images/user.png' class='markedPic' alt=''> " . $line["login"] . "</td>";
 
 				if (!$line["email"]) $line["email"] = "&nbsp;";
 
 				print "<td $onclick>" .	$access_level_names[$line["access_level"]] . "</td>";
+				print "<td $onclick>" . $line["num_feeds"] . "</td>";
 				print "<td $onclick>" . $line["created"] . "</td>";
 				print "<td $onclick>" . $line["last_login"] . "</td>";
 
@@ -434,15 +442,12 @@ class Pref_Users extends Handler_Protected {
 
 			print "</table>";
 
-			} else {
-				print "<p>";
+			if ($lnum == 0) {
 				if (!$user_search) {
 					print_warning(__('No users defined.'));
 				} else {
 					print_warning(__('No matching users found.'));
 				}
-				print "</p>";
-
 			}
 
 			print "</div>"; #pane
@@ -453,6 +458,4 @@ class Pref_Users extends Handler_Protected {
 			print "</div>"; #container
 
 		}
-
-	}
-?>
+	}
\ No newline at end of file