YakumoLabs/user_external
4
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

use $stmt = \OC::$server->getDatabaseConnection()->prepare() for the remaining two

Browse Source 
Signed-off-by: Jonas Sulzer <jonas@violoncello.ch>
This commit is contained in:
Jonas Sulzer
2019-01-11 14:08:10 +01:00
parent cfd23bc01e
commit 757c651995
1 changed files with 16 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files

36
lib/base.php
View File

@@ -6,7 +6,6 @@
* See the COPYING-README file. * See the COPYING-README file.
*/ */
namespace OCA\user_external; namespace OCA\user_external;
use \OC_DB;
/** /**
* Base class for external auth implementations that stores users * Base class for external auth implementations that stores users
@@ -74,19 +73,17 @@ abstract class Base extends \OC\User\Backend{
* @return array with all displayNames (value) and the corresponding uids (key) * @return array with all displayNames (value) and the corresponding uids (key)
*/ */
public function getDisplayNames($search = '', $limit = null, $offset = null) { public function getDisplayNames($search = '', $limit = null, $offset = null) {
$result = OC_DB::executeAudited( $stmt = \OC::$server->getDatabaseConnection()->prepare(
array( 'SELECT `uid`, `displayname` FROM `*PREFIX*users_external`'
'sql' => 'SELECT `uid`, `displayname` FROM `*PREFIX*users_external`' . ' WHERE (LOWER(`displayname`) LIKE LOWER(?) '
. ' WHERE (LOWER(`displayname`) LIKE LOWER(?) ' . ' OR LOWER(`uid`) LIKE LOWER(?)) AND `backend` = ?',
. ' OR LOWER(`uid`) LIKE LOWER(?)) AND `backend` = ?', $limit, $offset
'limit' => $limit,
'offset' => $offset
),
array('%' . $search . '%', '%' . $search . '%', $this->backend)
); );
$stmt->execute(['%' . $search . '%', '%' . $search . '%', $this->backend]);
$displayNames = array(); $displayNames = array();
while ($row = $result->fetchRow()) { while ($row = $stmt->fetch()) {
$displayNames[$row['uid']] = $row['displayname']; $displayNames[$row['uid']] = $row['displayname'];
} }
@@ -99,17 +96,16 @@ abstract class Base extends \OC\User\Backend{
* @return array with all uids * @return array with all uids
*/ */
public function getUsers($search = '', $limit = null, $offset = null) { public function getUsers($search = '', $limit = null, $offset = null) {
$result = OC_DB::executeAudited( $stmt = \OC::$server->getDatabaseConnection()->prepare(
array( 'SELECT `uid` FROM `*PREFIX*users_external`'
'sql' => 'SELECT `uid` FROM `*PREFIX*users_external`' . ' WHERE LOWER(`uid`) LIKE LOWER(?) AND `backend` = ?',
. ' WHERE LOWER(`uid`) LIKE LOWER(?) AND `backend` = ?', $limit, $offset
'limit' => $limit,
'offset' => $offset
),
array($search . '%', $this->backend)
); );
$stmt->execute([$search . '%', $this->backend]);
$users = array(); $users = array();
while ($row = $result->fetchRow()) { while ($row = $stmt->fetch()) {
$users[] = $row['uid']; $users[] = $row['uid'];
} }
return $users; return $users;