]> git.wh0rd.org - tt-rss.git/blobdiff - classes/pref/users.php
php: remove trailing whitespaces
[tt-rss.git] / classes / pref / users.php
index 49dd896e9f0a2036a7ce8fb3b96f346a51df2c16..ab8ac775b54319c77ff51d680a701efe4af6d59b 100644 (file)
@@ -12,253 +12,241 @@ 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;
 
-                       header("Content-Type: text/xml");
-                       print "<dlg>";
+                       print "<form id=\"user_edit_form\" onsubmit='return false' dojoType=\"dijit.form.Form\">";
 
-                       $uid = sprintf("%d", $_REQUEST["id"]);
+                       print '<div dojoType="dijit.layout.TabContainer" style="height : 400px">
+                       <div dojoType="dijit.layout.ContentPane" title="'.__('Edit user').'">';
 
-                       print "<title>".__('User details')."</title>";
+                       //print "<form id=\"user_edit_form\" onsubmit='return false' dojoType=\"dijit.form.Form\">";
 
-                       print "<content><![CDATA[";
+                       $id = (int) clean($_REQUEST["id"]);
 
-                       $result = db_query($this->link, "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_hidden("id", "$id");
+                       print_hidden("op", "pref-users");
+                       print_hidden("method", "editSave");
 
-                       if (db_num_rows($result) == 0) {
-                               print "<h1>".__('User not found')."</h1>";
-                               return;
-                       }
+                       $sth = $this->pdo->prepare("SELECT * FROM ttrss_users WHERE id = ?");
+                       $sth->execute([$id]);
 
-                       // print "<h1>User Details</h1>";
+                       if ($row = $sth->fetch()) {
 
-                       $login = db_fetch_result($result, 0, "login");
+                               $login = $row["login"];
+                               $access_level = $row["access_level"];
+                               $email = $row["email"];
 
-                       print "<table width='100%'>";
+                               $sel_disabled = ($id == $_SESSION["uid"] || $login == "admin") ? "disabled" : "";
 
-                       $last_login = make_local_datetime($this->link,
-                               db_fetch_result($result, 0, "last_login"), true);
+                               print "<div class=\"dlgSec\">".__("User")."</div>";
+                               print "<div class=\"dlgSecCont\">";
 
-                       $created = make_local_datetime($this->link,
-                               db_fetch_result($result, 0, "created"), true);
-
-                       $access_level = db_fetch_result($result, 0, "access_level");
-                       $stored_articles = db_fetch_result($result, 0, "stored_articles");
-
-                       print "<tr><td>".__('Registered')."</td><td>$created</td></tr>";
-                       print "<tr><td>".__('Last logged in')."</td><td>$last_login</td></tr>";
-
-                       $result = db_query($this->link, "SELECT COUNT(id) as num_feeds FROM ttrss_feeds
-                               WHERE owner_uid = '$uid'");
+                               if ($sel_disabled) {
+                                       print_hidden("login", "$login");
+                               }
 
-                       $num_feeds = db_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 = db_query($this->link, "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/>";
 
-                       $row_class = "odd";
+                               print "<input dojoType=\"dijit.form.TextBox\" type=\"password\" size=\"20\" placeholder=\"Change password\"
+                               name=\"password\">";
 
-                       while ($line = db_fetch_assoc($result)) {
+                               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 "<input dojoType=\"dijit.form.TextBox\" size=\"30\" name=\"email\" placeholder=\"E-mail\"
+                               value=\"$email\">";
 
-                               print "<li class=\"$row_class\">$feed_icon&nbsp;<a href=\"".$line["site_url"]."\">".$line["title"]."</a></li>";
+                               print "</div>";
 
-                               $row_class = $row_class == "even" ? "odd" : "even";
+                               print "</table>";
 
                        }
 
-                       if (db_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 '</div>'; #tab
+                       print "<div href=\"backend.php?op=pref-users&method=userdetails&id=$id\"
+                               dojoType=\"dijit.layout.ContentPane\" title=\"".__('User details')."\">";
 
-                       print "</ul>";
+                       print '</div>';
+                       print '</div>';
 
-                       print "<div align='center'>
-                               <button onclick=\"closeInfoBox()\">".__("Close this window").
-                               "</button></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 "]]></content></dlg>";
+                       print "</form>";
 
                        return;
                }
 
-               function edit() {
-                       global $access_level_names;
+               function userdetails() {
+                       $id = (int) clean($_REQUEST["id"]);
 
-                       header("Content-Type: text/xml");
+                       $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]);
 
-                       $id = db_escape_string($_REQUEST["id"]);
+                       if ($row = $sth->fetch()) {
+                               print "<table width='100%'>";
 
-                       print "<dlg id=\"$method\">";
-                       print "<title>".__('User Editor')."</title>";
-                       print "<content><![CDATA[";
+                               $last_login = make_local_datetime(
+                                       $row["last_login"], true);
 
-                       print "<form id=\"user_edit_form\" onsubmit='return false'>";
+                               $created = make_local_datetime(
+                                       $row["created"], true);
 
-                       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\">";
+                               $stored_articles = $row["stored_articles"];
 
-                       $result = db_query($this->link, "SELECT * FROM ttrss_users WHERE id = '$id'");
+                               print "<tr><td>".__('Registered')."</td><td>$created</td></tr>";
+                               print "<tr><td>".__('Last logged in')."</td><td>$last_login</td></tr>";
 
-                       $login = db_fetch_result($result, 0, "login");
-                       $access_level = db_fetch_result($result, 0, "access_level");
-                       $email = db_fetch_result($result, 0, "email");
+                               $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"];
 
-                       $sel_disabled = ($id == $_SESSION["uid"]) ? "disabled" : "";
+                               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\">".__("User")."</div>";
-                       print "<div class=\"dlgSecCont\">";
+                               print "</table>";
 
-                       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\">";
-                       }
+                               print "<h1>".__('Subscribed feeds')."</h1>";
 
-                       print "</div>";
+                               $sth = $this->pdo->prepare("SELECT id,title,site_url FROM ttrss_feeds
+                                       WHERE owner_uid = ? ORDER BY title");
+                               $sth->execute([$id]);
 
-                       print "<div class=\"dlgSec\">".__("Authentication")."</div>";
-                       print "<div class=\"dlgSecCont\">";
+                               print "<ul class=\"userFeedList\">";
 
-                       print __('Access level: ') . " ";
+                               while ($line = $sth->fetch()) {
 
-                       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\">";
-                       }
-
-                       print "<br/>";
-
-                       print __('Change password to') .
-                               " <input size=\"20\" onkeypress=\"return filterCR(event, userEditSave)\"
-                               name=\"password\">";
-
-                       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>";
 
-                       print "]]></content></dlg>";
+                       } else {
+                               print "<h1>".__('User not found')."</h1>";
+                       }
 
-                       return;
                }
 
                function editSave() {
-                       $login = db_escape_string(trim($_REQUEST["login"]));
-                       $uid = db_escape_string($_REQUEST["id"]);
-                       $access_level = (int) $_REQUEST["access_level"];
-                       $email = db_escape_string(trim($_REQUEST["email"]));
-                       $password = db_escape_string(trim($_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 = "";
                        }
 
-                       db_query($this->link, "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 = split(",", db_escape_string($_REQUEST["ids"]));
+                       $ids = explode(",", clean($_REQUEST["ids"]));
 
                        foreach ($ids as $id) {
                                if ($id != $_SESSION["uid"] && $id != 1) {
-                                       db_query($this->link, "DELETE FROM ttrss_tags WHERE owner_uid = '$id'");
-                                       db_query($this->link, "DELETE FROM ttrss_feeds WHERE owner_uid = '$id'");
-                                       db_query($this->link, "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 = db_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 = db_query($this->link, "SELECT id FROM ttrss_users WHERE
-                               login = '$login'");
+                       $sth = $this->pdo->prepare("SELECT id FROM ttrss_users WHERE
+                               login = ?");
+                       $sth->execute([$login]);
 
-                       if (db_num_rows($result) == 0) {
+                       if (!$sth->fetch()) {
 
-                               db_query($this->link, "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 = db_query($this->link, "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 (db_num_rows($result) == 1) {
+                               if ($row = $sth->fetch()) {
 
-                                       $new_uid = db_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));
 
-                                       initialize_user($this->link, $new_uid);
+                                       initialize_user($new_uid);
 
                                } else {
 
@@ -270,78 +258,68 @@ class Pref_Users extends Handler_Protected {
                        }
                }
 
-               function resetPass() {
+               static function resetUserPassword($uid, $show_password) {
 
-                       $uid = db_escape_string($_REQUEST["id"]);
+                       $pdo = Db::pdo();
 
-                       $result = db_query($this->link, "SELECT login,email
-                               FROM ttrss_users WHERE id = '$uid'");
+                       $sth = $pdo->prepare("SELECT login, email
+                               FROM ttrss_users WHERE id = ?");
+                       $sth->execute([$uid]);
 
-                       $login = db_fetch_result($result, 0, "login");
-                       $email = db_fetch_result($result, 0, "email");
-                       $salt = db_fetch_result($result, 0, "salt");
-
-                       $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($this->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);
 
-                       print T_sprintf("Changed password of user <b>%s</b>
-                                to <b>%s</b>", $login, $tmp_user_pwd);
+                               $pwd_hash = encrypt_password($tmp_user_pwd, $new_salt, true);
 
-                       require_once 'lib/phpmailer/class.phpmailer.php';
+                               $sth = $pdo->prepare("UPDATE ttrss_users
+                                         SET pwd_hash = ?, salt = ?, otp_enabled = false
+                                       WHERE id = ?");
+                               $sth->execute([$pwd_hash, $new_salt, $uid]);
 
-                       if ($email) {
-                               print " ";
-                               print T_sprintf("Notifying <b>%s</b>.", $email);
-
-                               require_once "lib/MiniTemplator.class.php";
+                               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));
+                               }
 
-                               $tpl = new MiniTemplator;
+                               require_once 'classes/ttrssmailer.php';
 
-                               $tpl->readTemplateFromFile("templates/resetpass_template.txt");
+                               if ($email) {
+                                       require_once "lib/MiniTemplator.class.php";
 
-                               $tpl->setVariable('LOGIN', $login);
-                               $tpl->setVariable('NEWPASS', $tmp_user_pwd);
+                                       $tpl = new MiniTemplator;
 
-                               $tpl->addBlock('message');
+                                       $tpl->readTemplateFromFile("templates/resetpass_template.txt");
 
-                               $message = "";
+                                       $tpl->setVariable('LOGIN', $login);
+                                       $tpl->setVariable('NEWPASS', $tmp_user_pwd);
 
-                               $tpl->generateOutputToString($message);
+                                       $tpl->addBlock('message');
 
-                               $mail = new PHPMailer();
+                                       $message = "";
 
-                               $mail->PluginDir = "lib/phpmailer/";
-                               $mail->SetLanguage("en", "lib/phpmailer/language/");
+                                       $tpl->generateOutputToString($message);
 
-                               $mail->CharSet = "UTF-8";
+                                       $mail = new ttrssMailer();
 
-                               $mail->From = SMTP_FROM_ADDRESS;
-                               $mail->FromName = SMTP_FROM_NAME;
-                               $mail->AddAddress($email, $login);
+                                       $rc = $mail->quickMail($email, $login,
+                                               __("[tt-rss] Password change notification"),
+                                               $message, false);
 
-                               if (SMTP_HOST) {
-                                       $mail->Host = SMTP_HOST;
-                                       $mail->Mailer = "smtp";
-                                       $mail->SMTPAuth = SMTP_LOGIN != '';
-                                       $mail->Username = SMTP_LOGIN;
-                                       $mail->Password = SMTP_PASSWORD;
+                                       if (!$rc) print_error($mail->ErrorInfo);
                                }
 
-                               $mail->IsHTML(false);
-                               $mail->Subject = __("[tt-rss] Password change notification");
-                               $mail->Body = $message;
-
-                               $rc = $mail->Send();
-
-                               if (!$rc) print_error($mail->ErrorInfo);
                        }
+               }
 
-                       print "</div>";
+               function resetPass() {
+                       $uid = clean($_REQUEST["id"]);
+                       Pref_Users::resetUserPassword($uid, true);
                }
 
                function index() {
@@ -353,7 +331,7 @@ class Pref_Users extends Handler_Protected {
 
                        print "<div id=\"pref-user-toolbar\" dojoType=\"dijit.Toolbar\">";
 
-                       $user_search = db_escape_string($_REQUEST["search"]);
+                       $user_search = trim(clean($_REQUEST["search"]));
 
                        if (array_key_exists("search", $_REQUEST)) {
                                $_SESSION["prefs_user_search"] = $user_search;
@@ -364,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 = db_escape_string($_REQUEST["sort"]);
+                       $sort = clean($_REQUEST["sort"]);
 
                        if (!$sort || $sort == "undefined") {
                                $sort = "login";
@@ -383,68 +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>";
-
-                       if ($user_search) {
-
-                               $user_search = split(" ", $user_search);
-                               $tokens = array();
+                       $sort = validate_field($sort,
+                               ["login", "access_level", "created", "num_feeds", "created", "last_login"], "login");
 
-                               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 = "";
-                       }
+                       if ($sort != "login") $sort = "$sort DESC";
 
-                       $result = db_query($this->link, "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 (db_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 = db_fetch_assoc($result)) {
-
-                               $class = ($lnum % 2) ? "even" : "odd";
+                       while ($line = $sth->fetch()) {
 
                                $uid = $line["id"];
 
@@ -452,8 +417,8 @@ class Pref_Users extends Handler_Protected {
 
                                $line["login"] = htmlspecialchars($line["login"]);
 
-                               $line["created"] = make_local_datetime($this->link, $line["created"], false);
-                               $line["last_login"] = make_local_datetime($this->link, $line["last_login"], false);
+                               $line["created"] = make_local_datetime($line["created"], false);
+                               $line["last_login"] = make_local_datetime($line["last_login"], false);
 
                                print "<td align='center'><input onclick='toggleSelectRow2(this);'
                                        dojoType=\"dijit.form.CheckBox\" type=\"checkbox\"
@@ -461,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>";
 
@@ -476,26 +442,20 @@ 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
 
-                       global $pluginhost;
-                       $pluginhost->run_hooks($pluginhost::HOOK_PREFS_TAB,
+                       PluginHost::getInstance()->run_hooks(PluginHost::HOOK_PREFS_TAB,
                                "hook_prefs_tab", "prefUsers");
 
                        print "</div>"; #container
 
                }
-
-       }
-?>
+       }
\ No newline at end of file