Use IQueryBuilder::PARAM_* instead of \PDO::PARAM_*

This commit is contained in:
Joas Schilling 2016-02-29 09:44:40 +01:00
parent 6526c122af
commit 2a0cda74d4
15 changed files with 76 additions and 69 deletions

View File

@ -607,7 +607,7 @@ class CalDavBackend extends AbstractBackend implements SyncSupport, Subscription
->values([ ->values([
'calendarid' => $query->createNamedParameter($calendarId), 'calendarid' => $query->createNamedParameter($calendarId),
'uri' => $query->createNamedParameter($objectUri), 'uri' => $query->createNamedParameter($objectUri),
'calendardata' => $query->createNamedParameter($calendarData, \PDO::PARAM_LOB), 'calendardata' => $query->createNamedParameter($calendarData, IQueryBuilder::PARAM_LOB),
'lastmodified' => $query->createNamedParameter(time()), 'lastmodified' => $query->createNamedParameter(time()),
'etag' => $query->createNamedParameter($extraData['etag']), 'etag' => $query->createNamedParameter($extraData['etag']),
'size' => $query->createNamedParameter($extraData['size']), 'size' => $query->createNamedParameter($extraData['size']),
@ -646,7 +646,7 @@ class CalDavBackend extends AbstractBackend implements SyncSupport, Subscription
$query = $this->db->getQueryBuilder(); $query = $this->db->getQueryBuilder();
$query->update('calendarobjects') $query->update('calendarobjects')
->set('calendardata', $query->createNamedParameter($calendarData, \PDO::PARAM_LOB)) ->set('calendardata', $query->createNamedParameter($calendarData, IQueryBuilder::PARAM_LOB))
->set('lastmodified', $query->createNamedParameter(time())) ->set('lastmodified', $query->createNamedParameter(time()))
->set('etag', $query->createNamedParameter($extraData['etag'])) ->set('etag', $query->createNamedParameter($extraData['etag']))
->set('size', $query->createNamedParameter($extraData['size'])) ->set('size', $query->createNamedParameter($extraData['size']))

View File

@ -489,7 +489,7 @@ class CardDavBackend implements BackendInterface, SyncSupport {
$query = $this->db->getQueryBuilder(); $query = $this->db->getQueryBuilder();
$query->insert('cards') $query->insert('cards')
->values([ ->values([
'carddata' => $query->createNamedParameter($cardData, \PDO::PARAM_LOB), 'carddata' => $query->createNamedParameter($cardData, IQueryBuilder::PARAM_LOB),
'uri' => $query->createNamedParameter($cardUri), 'uri' => $query->createNamedParameter($cardUri),
'lastmodified' => $query->createNamedParameter(time()), 'lastmodified' => $query->createNamedParameter(time()),
'addressbookid' => $query->createNamedParameter($addressBookId), 'addressbookid' => $query->createNamedParameter($addressBookId),
@ -542,7 +542,7 @@ class CardDavBackend implements BackendInterface, SyncSupport {
$etag = md5($cardData); $etag = md5($cardData);
$query = $this->db->getQueryBuilder(); $query = $this->db->getQueryBuilder();
$query->update('cards') $query->update('cards')
->set('carddata', $query->createNamedParameter($cardData, \PDO::PARAM_LOB)) ->set('carddata', $query->createNamedParameter($cardData, IQueryBuilder::PARAM_LOB))
->set('lastmodified', $query->createNamedParameter(time())) ->set('lastmodified', $query->createNamedParameter(time()))
->set('size', $query->createNamedParameter(strlen($cardData))) ->set('size', $query->createNamedParameter(strlen($cardData)))
->set('etag', $query->createNamedParameter($etag)) ->set('etag', $query->createNamedParameter($etag))

View File

@ -26,6 +26,7 @@ use InvalidArgumentException;
use OCA\DAV\CardDAV\AddressBook; use OCA\DAV\CardDAV\AddressBook;
use OCA\DAV\CardDAV\CardDavBackend; use OCA\DAV\CardDAV\CardDavBackend;
use OCA\DAV\Connector\Sabre\Principal; use OCA\DAV\Connector\Sabre\Principal;
use OCP\DB\QueryBuilder\IQueryBuilder;
use OCP\IDBConnection; use OCP\IDBConnection;
use Sabre\DAV\PropPatch; use Sabre\DAV\PropPatch;
use Sabre\VObject\Component\VCard; use Sabre\VObject\Component\VCard;
@ -480,7 +481,7 @@ class CardDavBackendTest extends TestCase {
->values( ->values(
[ [
'addressbookid' => $query->createNamedParameter(0), 'addressbookid' => $query->createNamedParameter(0),
'carddata' => $query->createNamedParameter($vCards[$i]->serialize(), \PDO::PARAM_LOB), 'carddata' => $query->createNamedParameter($vCards[$i]->serialize(), IQueryBuilder::PARAM_LOB),
'uri' => $query->createNamedParameter('uri' . $i), 'uri' => $query->createNamedParameter('uri' . $i),
'lastmodified' => $query->createNamedParameter(time()), 'lastmodified' => $query->createNamedParameter(time()),
'etag' => $query->createNamedParameter('etag' . $i), 'etag' => $query->createNamedParameter('etag' . $i),
@ -558,7 +559,7 @@ class CardDavBackendTest extends TestCase {
->values( ->values(
[ [
'addressbookid' => $query->createNamedParameter(1), 'addressbookid' => $query->createNamedParameter(1),
'carddata' => $query->createNamedParameter('carddata', \PDO::PARAM_LOB), 'carddata' => $query->createNamedParameter('carddata', IQueryBuilder::PARAM_LOB),
'uri' => $query->createNamedParameter('uri'), 'uri' => $query->createNamedParameter('uri'),
'lastmodified' => $query->createNamedParameter(5489543), 'lastmodified' => $query->createNamedParameter(5489543),
'etag' => $query->createNamedParameter('etag'), 'etag' => $query->createNamedParameter('etag'),
@ -586,7 +587,7 @@ class CardDavBackendTest extends TestCase {
->values( ->values(
[ [
'addressbookid' => $query->createNamedParameter($i), 'addressbookid' => $query->createNamedParameter($i),
'carddata' => $query->createNamedParameter('carddata' . $i, \PDO::PARAM_LOB), 'carddata' => $query->createNamedParameter('carddata' . $i, IQueryBuilder::PARAM_LOB),
'uri' => $query->createNamedParameter('uri' . $i), 'uri' => $query->createNamedParameter('uri' . $i),
'lastmodified' => $query->createNamedParameter(5489543), 'lastmodified' => $query->createNamedParameter(5489543),
'etag' => $query->createNamedParameter('etag' . $i), 'etag' => $query->createNamedParameter('etag' . $i),

View File

@ -65,7 +65,7 @@ class DBConfigService {
$builder = $this->connection->getQueryBuilder(); $builder = $this->connection->getQueryBuilder();
$query = $builder->select(['mount_id', 'mount_point', 'storage_backend', 'auth_backend', 'priority', 'type']) $query = $builder->select(['mount_id', 'mount_point', 'storage_backend', 'auth_backend', 'priority', 'type'])
->from('external_mounts', 'm') ->from('external_mounts', 'm')
->where($builder->expr()->eq('mount_id', $builder->createNamedParameter($mountId, \PDO::PARAM_INT))); ->where($builder->expr()->eq('mount_id', $builder->createNamedParameter($mountId, IQueryBuilder::PARAM_INT)));
$mounts = $this->getMountsFromQuery($query); $mounts = $this->getMountsFromQuery($query);
if (count($mounts) > 0) { if (count($mounts) > 0) {
return $mounts[0]; return $mounts[0];
@ -83,7 +83,7 @@ class DBConfigService {
$builder = $this->connection->getQueryBuilder(); $builder = $this->connection->getQueryBuilder();
$query = $builder->select(['mount_id', 'mount_point', 'storage_backend', 'auth_backend', 'priority', 'type']) $query = $builder->select(['mount_id', 'mount_point', 'storage_backend', 'auth_backend', 'priority', 'type'])
->from('external_mounts') ->from('external_mounts')
->where($builder->expr()->eq('type', $builder->expr()->literal(self::MOUNT_TYPE_ADMIN, \PDO::PARAM_INT))); ->where($builder->expr()->eq('type', $builder->expr()->literal(self::MOUNT_TYPE_ADMIN, IQueryBuilder::PARAM_INT)));
return $this->getMountsFromQuery($query); return $this->getMountsFromQuery($query);
} }
@ -91,7 +91,7 @@ class DBConfigService {
$query = $builder->select(['m.mount_id', 'mount_point', 'storage_backend', 'auth_backend', 'priority', 'm.type']) $query = $builder->select(['m.mount_id', 'mount_point', 'storage_backend', 'auth_backend', 'priority', 'm.type'])
->from('external_mounts', 'm') ->from('external_mounts', 'm')
->innerJoin('m', 'external_applicable', 'a', 'm.mount_id = a.mount_id') ->innerJoin('m', 'external_applicable', 'a', 'm.mount_id = a.mount_id')
->where($builder->expr()->eq('a.type', $builder->createNamedParameter($type, \PDO::PARAM_INT))); ->where($builder->expr()->eq('a.type', $builder->createNamedParameter($type, IQueryBuilder::PARAM_INT)));
if (is_null($value)) { if (is_null($value)) {
$query = $query->andWhere($builder->expr()->isNull('a.value')); $query = $query->andWhere($builder->expr()->isNull('a.value'));
@ -126,7 +126,7 @@ class DBConfigService {
public function getAdminMountsFor($type, $value) { public function getAdminMountsFor($type, $value) {
$builder = $this->connection->getQueryBuilder(); $builder = $this->connection->getQueryBuilder();
$query = $this->getForQuery($builder, $type, $value); $query = $this->getForQuery($builder, $type, $value);
$query->andWhere($builder->expr()->eq('m.type', $builder->expr()->literal(self::MOUNT_TYPE_ADMIN, \PDO::PARAM_INT))); $query->andWhere($builder->expr()->eq('m.type', $builder->expr()->literal(self::MOUNT_TYPE_ADMIN, IQueryBuilder::PARAM_INT)));
return $this->getMountsFromQuery($query); return $this->getMountsFromQuery($query);
} }
@ -141,15 +141,15 @@ class DBConfigService {
public function getAdminMountsForMultiple($type, array $values) { public function getAdminMountsForMultiple($type, array $values) {
$builder = $this->connection->getQueryBuilder(); $builder = $this->connection->getQueryBuilder();
$params = array_map(function ($value) use ($builder) { $params = array_map(function ($value) use ($builder) {
return $builder->createNamedParameter($value, \PDO::PARAM_STR); return $builder->createNamedParameter($value, IQueryBuilder::PARAM_STR);
}, $values); }, $values);
$query = $builder->select(['m.mount_id', 'mount_point', 'storage_backend', 'auth_backend', 'priority', 'm.type']) $query = $builder->select(['m.mount_id', 'mount_point', 'storage_backend', 'auth_backend', 'priority', 'm.type'])
->from('external_mounts', 'm') ->from('external_mounts', 'm')
->innerJoin('m', 'external_applicable', 'a', 'm.mount_id = a.mount_id') ->innerJoin('m', 'external_applicable', 'a', 'm.mount_id = a.mount_id')
->where($builder->expr()->eq('a.type', $builder->createNamedParameter($type, \PDO::PARAM_INT))) ->where($builder->expr()->eq('a.type', $builder->createNamedParameter($type, IQueryBuilder::PARAM_INT)))
->andWhere($builder->expr()->in('a.value', $params)); ->andWhere($builder->expr()->in('a.value', $params));
$query->andWhere($builder->expr()->eq('m.type', $builder->expr()->literal(self::MOUNT_TYPE_ADMIN, \PDO::PARAM_INT))); $query->andWhere($builder->expr()->eq('m.type', $builder->expr()->literal(self::MOUNT_TYPE_ADMIN, IQueryBuilder::PARAM_INT)));
return $this->getMountsFromQuery($query); return $this->getMountsFromQuery($query);
} }
@ -164,7 +164,7 @@ class DBConfigService {
public function getUserMountsFor($type, $value) { public function getUserMountsFor($type, $value) {
$builder = $this->connection->getQueryBuilder(); $builder = $this->connection->getQueryBuilder();
$query = $this->getForQuery($builder, $type, $value); $query = $this->getForQuery($builder, $type, $value);
$query->andWhere($builder->expr()->eq('m.type', $builder->expr()->literal(self::MOUNT_TYPE_PERSONAl, \PDO::PARAM_INT))); $query->andWhere($builder->expr()->eq('m.type', $builder->expr()->literal(self::MOUNT_TYPE_PERSONAl, IQueryBuilder::PARAM_INT)));
return $this->getMountsFromQuery($query); return $this->getMountsFromQuery($query);
} }
@ -186,11 +186,11 @@ class DBConfigService {
$builder = $this->connection->getQueryBuilder(); $builder = $this->connection->getQueryBuilder();
$query = $builder->insert('external_mounts') $query = $builder->insert('external_mounts')
->values([ ->values([
'mount_point' => $builder->createNamedParameter($mountPoint, \PDO::PARAM_STR), 'mount_point' => $builder->createNamedParameter($mountPoint, IQueryBuilder::PARAM_STR),
'storage_backend' => $builder->createNamedParameter($storageBackend, \PDO::PARAM_STR), 'storage_backend' => $builder->createNamedParameter($storageBackend, IQueryBuilder::PARAM_STR),
'auth_backend' => $builder->createNamedParameter($authBackend, \PDO::PARAM_STR), 'auth_backend' => $builder->createNamedParameter($authBackend, IQueryBuilder::PARAM_STR),
'priority' => $builder->createNamedParameter($priority, \PDO::PARAM_INT), 'priority' => $builder->createNamedParameter($priority, IQueryBuilder::PARAM_INT),
'type' => $builder->createNamedParameter($type, \PDO::PARAM_INT) 'type' => $builder->createNamedParameter($type, IQueryBuilder::PARAM_INT)
]); ]);
$query->execute(); $query->execute();
return (int)$this->connection->lastInsertId('external_mounts'); return (int)$this->connection->lastInsertId('external_mounts');
@ -204,19 +204,19 @@ class DBConfigService {
public function removeMount($mountId) { public function removeMount($mountId) {
$builder = $this->connection->getQueryBuilder(); $builder = $this->connection->getQueryBuilder();
$query = $builder->delete('external_mounts') $query = $builder->delete('external_mounts')
->where($builder->expr()->eq('mount_id', $builder->createNamedParameter($mountId, \PDO::PARAM_INT))); ->where($builder->expr()->eq('mount_id', $builder->createNamedParameter($mountId, IQueryBuilder::PARAM_INT)));
$query->execute(); $query->execute();
$query = $builder->delete('external_applicable') $query = $builder->delete('external_applicable')
->where($builder->expr()->eq('mount_id', $builder->createNamedParameter($mountId, \PDO::PARAM_INT))); ->where($builder->expr()->eq('mount_id', $builder->createNamedParameter($mountId, IQueryBuilder::PARAM_INT)));
$query->execute(); $query->execute();
$query = $builder->delete('external_config') $query = $builder->delete('external_config')
->where($builder->expr()->eq('mount_id', $builder->createNamedParameter($mountId, \PDO::PARAM_INT))); ->where($builder->expr()->eq('mount_id', $builder->createNamedParameter($mountId, IQueryBuilder::PARAM_INT)));
$query->execute(); $query->execute();
$query = $builder->delete('external_options') $query = $builder->delete('external_options')
->where($builder->expr()->eq('mount_id', $builder->createNamedParameter($mountId, \PDO::PARAM_INT))); ->where($builder->expr()->eq('mount_id', $builder->createNamedParameter($mountId, IQueryBuilder::PARAM_INT)));
$query->execute(); $query->execute();
} }
@ -229,7 +229,7 @@ class DBConfigService {
$query = $builder->update('external_mounts') $query = $builder->update('external_mounts')
->set('mount_point', $builder->createNamedParameter($newMountPoint)) ->set('mount_point', $builder->createNamedParameter($newMountPoint))
->where($builder->expr()->eq('mount_id', $builder->createNamedParameter($mountId, \PDO::PARAM_INT))); ->where($builder->expr()->eq('mount_id', $builder->createNamedParameter($mountId, IQueryBuilder::PARAM_INT)));
$query->execute(); $query->execute();
} }
@ -243,7 +243,7 @@ class DBConfigService {
$query = $builder->update('external_mounts') $query = $builder->update('external_mounts')
->set('auth_backend', $builder->createNamedParameter($newAuthBackend)) ->set('auth_backend', $builder->createNamedParameter($newAuthBackend))
->where($builder->expr()->eq('mount_id', $builder->createNamedParameter($mountId, \PDO::PARAM_INT))); ->where($builder->expr()->eq('mount_id', $builder->createNamedParameter($mountId, IQueryBuilder::PARAM_INT)));
$query->execute(); $query->execute();
} }
@ -265,9 +265,9 @@ class DBConfigService {
if ($count === 0) { if ($count === 0) {
$builder = $this->connection->getQueryBuilder(); $builder = $this->connection->getQueryBuilder();
$query = $builder->update('external_config') $query = $builder->update('external_config')
->set('value', $builder->createNamedParameter($value, \PDO::PARAM_STR)) ->set('value', $builder->createNamedParameter($value, IQueryBuilder::PARAM_STR))
->where($builder->expr()->eq('mount_id', $builder->createNamedParameter($mountId, \PDO::PARAM_INT))) ->where($builder->expr()->eq('mount_id', $builder->createNamedParameter($mountId, IQueryBuilder::PARAM_INT)))
->andWhere($builder->expr()->eq('key', $builder->createNamedParameter($key, \PDO::PARAM_STR))); ->andWhere($builder->expr()->eq('key', $builder->createNamedParameter($key, IQueryBuilder::PARAM_STR)));
$query->execute(); $query->execute();
} }
} }
@ -287,9 +287,9 @@ class DBConfigService {
if ($count === 0) { if ($count === 0) {
$builder = $this->connection->getQueryBuilder(); $builder = $this->connection->getQueryBuilder();
$query = $builder->update('external_options') $query = $builder->update('external_options')
->set('value', $builder->createNamedParameter(json_encode($value), \PDO::PARAM_STR)) ->set('value', $builder->createNamedParameter(json_encode($value), IQueryBuilder::PARAM_STR))
->where($builder->expr()->eq('mount_id', $builder->createNamedParameter($mountId, \PDO::PARAM_INT))) ->where($builder->expr()->eq('mount_id', $builder->createNamedParameter($mountId, IQueryBuilder::PARAM_INT)))
->andWhere($builder->expr()->eq('key', $builder->createNamedParameter($key, \PDO::PARAM_STR))); ->andWhere($builder->expr()->eq('key', $builder->createNamedParameter($key, IQueryBuilder::PARAM_STR)));
$query->execute(); $query->execute();
} }
} }
@ -305,13 +305,13 @@ class DBConfigService {
public function removeApplicable($mountId, $type, $value) { public function removeApplicable($mountId, $type, $value) {
$builder = $this->connection->getQueryBuilder(); $builder = $this->connection->getQueryBuilder();
$query = $builder->delete('external_applicable') $query = $builder->delete('external_applicable')
->where($builder->expr()->eq('mount_id', $builder->createNamedParameter($mountId, \PDO::PARAM_INT))) ->where($builder->expr()->eq('mount_id', $builder->createNamedParameter($mountId, IQueryBuilder::PARAM_INT)))
->andWhere($builder->expr()->eq('type', $builder->createNamedParameter($type, \PDO::PARAM_INT))); ->andWhere($builder->expr()->eq('type', $builder->createNamedParameter($type, IQueryBuilder::PARAM_INT)));
if (is_null($value)) { if (is_null($value)) {
$query = $query->andWhere($builder->expr()->isNull('value')); $query = $query->andWhere($builder->expr()->isNull('value'));
} else { } else {
$query = $query->andWhere($builder->expr()->eq('value', $builder->createNamedParameter($value, \PDO::PARAM_STR))); $query = $query->andWhere($builder->expr()->eq('value', $builder->createNamedParameter($value, IQueryBuilder::PARAM_STR)));
} }
$query->execute(); $query->execute();
@ -354,7 +354,7 @@ class DBConfigService {
$builder = $this->connection->getQueryBuilder(); $builder = $this->connection->getQueryBuilder();
$fields[] = 'mount_id'; $fields[] = 'mount_id';
$placeHolders = array_map(function ($id) use ($builder) { $placeHolders = array_map(function ($id) use ($builder) {
return $builder->createPositionalParameter($id, \PDO::PARAM_INT); return $builder->createPositionalParameter($id, IQueryBuilder::PARAM_INT);
}, $mountIds); }, $mountIds);
$query = $builder->select($fields) $query = $builder->select($fields)
->from($table) ->from($table)

View File

@ -25,6 +25,7 @@
namespace OC\AppFramework\Db; namespace OC\AppFramework\Db;
use OCP\DB\QueryBuilder\IQueryBuilder;
use OCP\IDb; use OCP\IDb;
use OCP\IDBConnection; use OCP\IDBConnection;
@ -240,7 +241,7 @@ class Db implements IDb {
* @param int $type Type of the parameter. * @param int $type Type of the parameter.
* @return string The quoted parameter. * @return string The quoted parameter.
*/ */
public function quote($input, $type = \PDO::PARAM_STR) { public function quote($input, $type = IQueryBuilder::PARAM_STR) {
return $this->connection->quote($input, $type); return $this->connection->quote($input, $type);
} }

View File

@ -28,6 +28,7 @@ use OCP\AppFramework\QueryException;
use OCP\BackgroundJob\IJob; use OCP\BackgroundJob\IJob;
use OCP\BackgroundJob\IJobList; use OCP\BackgroundJob\IJobList;
use OCP\AutoloadNotAllowedException; use OCP\AutoloadNotAllowedException;
use OCP\DB\QueryBuilder\IQueryBuilder;
class JobList implements IJobList { class JobList implements IJobList {
/** @var \OCP\IDBConnection */ /** @var \OCP\IDBConnection */
@ -69,7 +70,7 @@ class JobList implements IJobList {
->values([ ->values([
'class' => $query->createNamedParameter($class), 'class' => $query->createNamedParameter($class),
'argument' => $query->createNamedParameter($argument), 'argument' => $query->createNamedParameter($argument),
'last_run' => $query->createNamedParameter(0, \PDO::PARAM_INT), 'last_run' => $query->createNamedParameter(0, IQueryBuilder::PARAM_INT),
]); ]);
$query->execute(); $query->execute();
} }
@ -102,7 +103,7 @@ class JobList implements IJobList {
protected function removeById($id) { protected function removeById($id) {
$query = $this->connection->getQueryBuilder(); $query = $this->connection->getQueryBuilder();
$query->delete('jobs') $query->delete('jobs')
->where($query->expr()->eq('id', $query->createNamedParameter($id, \PDO::PARAM_INT))); ->where($query->expr()->eq('id', $query->createNamedParameter($id, IQueryBuilder::PARAM_INT)));
$query->execute(); $query->execute();
} }
@ -171,7 +172,7 @@ class JobList implements IJobList {
$query = $this->connection->getQueryBuilder(); $query = $this->connection->getQueryBuilder();
$query->select('*') $query->select('*')
->from('jobs') ->from('jobs')
->where($query->expr()->gt('id', $query->createNamedParameter($lastId, \PDO::PARAM_INT))) ->where($query->expr()->gt('id', $query->createNamedParameter($lastId, IQueryBuilder::PARAM_INT)))
->orderBy('id', 'ASC') ->orderBy('id', 'ASC')
->setMaxResults(1); ->setMaxResults(1);
$result = $query->execute(); $result = $query->execute();
@ -216,7 +217,7 @@ class JobList implements IJobList {
$query = $this->connection->getQueryBuilder(); $query = $this->connection->getQueryBuilder();
$query->select('*') $query->select('*')
->from('jobs') ->from('jobs')
->where($query->expr()->eq('id', $query->createNamedParameter($id, \PDO::PARAM_INT))); ->where($query->expr()->eq('id', $query->createNamedParameter($id, IQueryBuilder::PARAM_INT)));
$result = $query->execute(); $result = $query->execute();
$row = $result->fetch(); $row = $result->fetch();
$result->closeCursor(); $result->closeCursor();
@ -286,8 +287,8 @@ class JobList implements IJobList {
public function setLastRun($job) { public function setLastRun($job) {
$query = $this->connection->getQueryBuilder(); $query = $this->connection->getQueryBuilder();
$query->update('jobs') $query->update('jobs')
->set('last_run', $query->createNamedParameter(time(), \PDO::PARAM_INT)) ->set('last_run', $query->createNamedParameter(time(), IQueryBuilder::PARAM_INT))
->where($query->expr()->eq('id', $query->createNamedParameter($job->getId(), \PDO::PARAM_INT))); ->where($query->expr()->eq('id', $query->createNamedParameter($job->getId(), IQueryBuilder::PARAM_INT)));
$query->execute(); $query->execute();
} }
} }

View File

@ -25,6 +25,7 @@ use OCP\Comments\CommentsEvent;
use OCP\Comments\IComment; use OCP\Comments\IComment;
use OCP\Comments\ICommentsManager; use OCP\Comments\ICommentsManager;
use OCP\Comments\NotFoundException; use OCP\Comments\NotFoundException;
use OCP\DB\QueryBuilder\IQueryBuilder;
use OCP\IDBConnection; use OCP\IDBConnection;
use OCP\IConfig; use OCP\IConfig;
use OCP\ILogger; use OCP\ILogger;
@ -233,7 +234,7 @@ class Manager implements ICommentsManager {
$resultStatement = $qb->select('*') $resultStatement = $qb->select('*')
->from('comments') ->from('comments')
->where($qb->expr()->eq('id', $qb->createParameter('id'))) ->where($qb->expr()->eq('id', $qb->createParameter('id')))
->setParameter('id', $id, \PDO::PARAM_INT) ->setParameter('id', $id, IQueryBuilder::PARAM_INT)
->execute(); ->execute();
$data = $resultStatement->fetch(); $data = $resultStatement->fetch();
@ -675,9 +676,9 @@ class Manager implements ICommentsManager {
->where($qb->expr()->eq('user_id', $qb->createParameter('user_id'))) ->where($qb->expr()->eq('user_id', $qb->createParameter('user_id')))
->andWhere($qb->expr()->eq('object_type', $qb->createParameter('object_type'))) ->andWhere($qb->expr()->eq('object_type', $qb->createParameter('object_type')))
->andWhere($qb->expr()->eq('object_id', $qb->createParameter('object_id'))) ->andWhere($qb->expr()->eq('object_id', $qb->createParameter('object_id')))
->setParameter('user_id', $user->getUID(), \PDO::PARAM_STR) ->setParameter('user_id', $user->getUID(), IQueryBuilder::PARAM_STR)
->setParameter('object_type', $objectType, \PDO::PARAM_STR) ->setParameter('object_type', $objectType, IQueryBuilder::PARAM_STR)
->setParameter('object_id', $objectId, \PDO::PARAM_STR) ->setParameter('object_id', $objectId, IQueryBuilder::PARAM_STR)
->execute(); ->execute();
if ($affectedRows > 0) { if ($affectedRows > 0) {
@ -707,9 +708,9 @@ class Manager implements ICommentsManager {
->where($qb->expr()->eq('user_id', $qb->createParameter('user_id'))) ->where($qb->expr()->eq('user_id', $qb->createParameter('user_id')))
->andWhere($qb->expr()->eq('object_type', $qb->createParameter('object_type'))) ->andWhere($qb->expr()->eq('object_type', $qb->createParameter('object_type')))
->andWhere($qb->expr()->eq('object_id', $qb->createParameter('object_id'))) ->andWhere($qb->expr()->eq('object_id', $qb->createParameter('object_id')))
->setParameter('user_id', $user->getUID(), \PDO::PARAM_STR) ->setParameter('user_id', $user->getUID(), IQueryBuilder::PARAM_STR)
->setParameter('object_type', $objectType, \PDO::PARAM_STR) ->setParameter('object_type', $objectType, IQueryBuilder::PARAM_STR)
->setParameter('object_id', $objectId, \PDO::PARAM_STR) ->setParameter('object_id', $objectId, IQueryBuilder::PARAM_STR)
->execute(); ->execute();
$data = $resultStatement->fetch(); $data = $resultStatement->fetch();

View File

@ -168,7 +168,7 @@ class QueryBuilder implements IQueryBuilder {
* *
* @param string|integer $key The parameter position or name. * @param string|integer $key The parameter position or name.
* @param mixed $value The parameter value. * @param mixed $value The parameter value.
* @param string|null $type One of the PDO::PARAM_* constants. * @param string|null $type One of the IQueryBuilder::PARAM_* constants.
* *
* @return \OCP\DB\QueryBuilder\IQueryBuilder This QueryBuilder instance. * @return \OCP\DB\QueryBuilder\IQueryBuilder This QueryBuilder instance.
*/ */
@ -988,7 +988,7 @@ class QueryBuilder implements IQueryBuilder {
* *
* @return IParameter the placeholder name used. * @return IParameter the placeholder name used.
*/ */
public function createNamedParameter($value, $type = \PDO::PARAM_STR, $placeHolder = null) { public function createNamedParameter($value, $type = IQueryBuilder::PARAM_STR, $placeHolder = null) {
return new Parameter($this->queryBuilder->createNamedParameter($value, $type, $placeHolder)); return new Parameter($this->queryBuilder->createNamedParameter($value, $type, $placeHolder));
} }
@ -1005,8 +1005,8 @@ class QueryBuilder implements IQueryBuilder {
* $qb = $conn->getQueryBuilder(); * $qb = $conn->getQueryBuilder();
* $qb->select('u.*') * $qb->select('u.*')
* ->from('users', 'u') * ->from('users', 'u')
* ->where('u.username = ' . $qb->createPositionalParameter('Foo', PDO::PARAM_STR)) * ->where('u.username = ' . $qb->createPositionalParameter('Foo', IQueryBuilder::PARAM_STR))
* ->orWhere('u.username = ' . $qb->createPositionalParameter('Bar', PDO::PARAM_STR)) * ->orWhere('u.username = ' . $qb->createPositionalParameter('Bar', IQueryBuilder::PARAM_STR))
* </code> * </code>
* *
* @param mixed $value * @param mixed $value
@ -1014,7 +1014,7 @@ class QueryBuilder implements IQueryBuilder {
* *
* @return IParameter * @return IParameter
*/ */
public function createPositionalParameter($value, $type = \PDO::PARAM_STR) { public function createPositionalParameter($value, $type = IQueryBuilder::PARAM_STR) {
return new Parameter($this->queryBuilder->createPositionalParameter($value, $type)); return new Parameter($this->queryBuilder->createPositionalParameter($value, $type));
} }
@ -1027,7 +1027,7 @@ class QueryBuilder implements IQueryBuilder {
* $qb->select('u.*') * $qb->select('u.*')
* ->from('users', 'u') * ->from('users', 'u')
* ->where('u.username = ' . $qb->createParameter('name')) * ->where('u.username = ' . $qb->createParameter('name'))
* ->setParameter('name', 'Bar', PDO::PARAM_STR)) * ->setParameter('name', 'Bar', IQueryBuilder::PARAM_STR))
* </code> * </code>
* *
* @param string $name * @param string $name

View File

@ -202,7 +202,7 @@ abstract class Mapper {
/** /**
* Returns the correct PDO constant based on the value type * Returns the correct PDO constant based on the value type
* @param $value * @param $value
* @return PDO constant * @return int PDO constant
* @since 8.1.0 * @since 8.1.0
*/ */
private function getPDOType($value) { private function getPDOType($value) {

View File

@ -156,7 +156,7 @@ interface IQueryBuilder {
* *
* @param string|integer $key The parameter position or name. * @param string|integer $key The parameter position or name.
* @param mixed $value The parameter value. * @param mixed $value The parameter value.
* @param string|null $type One of the PDO::PARAM_* constants. * @param string|null $type One of the IQueryBuilder::PARAM_* constants.
* *
* @return \OCP\DB\QueryBuilder\IQueryBuilder This QueryBuilder instance. * @return \OCP\DB\QueryBuilder\IQueryBuilder This QueryBuilder instance.
* @since 8.2.0 * @since 8.2.0
@ -780,7 +780,7 @@ interface IQueryBuilder {
* @return IParameter * @return IParameter
* @since 8.2.0 * @since 8.2.0
*/ */
public function createNamedParameter($value, $type = \PDO::PARAM_STR, $placeHolder = null); public function createNamedParameter($value, $type = self::PARAM_STR, $placeHolder = null);
/** /**
* Creates a new positional parameter and bind the given value to it. * Creates a new positional parameter and bind the given value to it.
@ -795,8 +795,8 @@ interface IQueryBuilder {
* $qb = $conn->getQueryBuilder(); * $qb = $conn->getQueryBuilder();
* $qb->select('u.*') * $qb->select('u.*')
* ->from('users', 'u') * ->from('users', 'u')
* ->where('u.username = ' . $qb->createPositionalParameter('Foo', PDO::PARAM_STR)) * ->where('u.username = ' . $qb->createPositionalParameter('Foo', IQueryBuilder::PARAM_STR))
* ->orWhere('u.username = ' . $qb->createPositionalParameter('Bar', PDO::PARAM_STR)) * ->orWhere('u.username = ' . $qb->createPositionalParameter('Bar', IQueryBuilder::PARAM_STR))
* </code> * </code>
* *
* @param mixed $value * @param mixed $value
@ -805,7 +805,7 @@ interface IQueryBuilder {
* @return IParameter * @return IParameter
* @since 8.2.0 * @since 8.2.0
*/ */
public function createPositionalParameter($value, $type = \PDO::PARAM_STR); public function createPositionalParameter($value, $type = self::PARAM_STR);
/** /**
* Creates a new parameter * Creates a new parameter
@ -816,7 +816,7 @@ interface IQueryBuilder {
* $qb->select('u.*') * $qb->select('u.*')
* ->from('users', 'u') * ->from('users', 'u')
* ->where('u.username = ' . $qb->createParameter('name')) * ->where('u.username = ' . $qb->createParameter('name'))
* ->setParameter('name', 'Bar', PDO::PARAM_STR)) * ->setParameter('name', 'Bar', IQueryBuilder::PARAM_STR))
* </code> * </code>
* *
* @param string $name * @param string $name

View File

@ -32,6 +32,7 @@
// use OCP namespace for all classes that are considered public. // use OCP namespace for all classes that are considered public.
// This means that they should be used by apps instead of the internal ownCloud classes // This means that they should be used by apps instead of the internal ownCloud classes
namespace OCP; namespace OCP;
use OCP\DB\QueryBuilder\IQueryBuilder;
/** /**
* Interface IDBConnection * Interface IDBConnection
@ -193,7 +194,7 @@ interface IDBConnection {
* @return string The quoted parameter. * @return string The quoted parameter.
* @since 8.0.0 * @since 8.0.0
*/ */
public function quote($input, $type = \PDO::PARAM_STR); public function quote($input, $type = IQueryBuilder::PARAM_STR);
/** /**
* Gets the DatabasePlatform instance that provides all the metadata about * Gets the DatabasePlatform instance that provides all the metadata about

View File

@ -68,7 +68,7 @@ abstract class MapperTestUtility extends \Test\TestCase {
/** /**
* Returns the correct PDO constant based on the value type * Returns the correct PDO constant based on the value type
* @param $value * @param $value
* @return PDO constant * @return int PDO constant
*/ */
private function getPDOType($value) { private function getPDOType($value) {
switch (gettype($value)) { switch (gettype($value)) {

View File

@ -11,6 +11,7 @@ namespace Test\DB;
use Doctrine\DBAL\Platforms\SqlitePlatform; use Doctrine\DBAL\Platforms\SqlitePlatform;
use OC\DB\MDB2SchemaManager; use OC\DB\MDB2SchemaManager;
use OCP\DB\QueryBuilder\IQueryBuilder;
/** /**
* Class Connection * Class Connection
@ -94,7 +95,7 @@ class Connection extends \Test\TestCase {
$builder = $this->connection->getQueryBuilder(); $builder = $this->connection->getQueryBuilder();
$query = $builder->select('textfield') $query = $builder->select('textfield')
->from('table') ->from('table')
->where($builder->expr()->eq('integerfield', $builder->createNamedParameter($integerField, \PDO::PARAM_INT))); ->where($builder->expr()->eq('integerfield', $builder->createNamedParameter($integerField, IQueryBuilder::PARAM_INT)));
$result = $query->execute(); $result = $query->execute();
return $result->fetchColumn(); return $result->fetchColumn();

View File

@ -327,7 +327,7 @@ class ExpressionBuilderTest extends TestCase {
[1, null], [1, null],
[1, 'string'], [1, 'string'],
[1, 'integer'], [1, 'integer'],
[1, \PDO::PARAM_INT], [1, IQueryBuilder::PARAM_INT],
]; ];
} }

View File

@ -7,6 +7,7 @@
*/ */
namespace Test\Repair; namespace Test\Repair;
use OCP\DB\QueryBuilder\IQueryBuilder;
/** /**
* Tests for the cleaning the tags tables * Tests for the cleaning the tags tables
@ -123,8 +124,8 @@ class CleanTags extends \Test\TestCase {
$qb = $this->connection->getQueryBuilder(); $qb = $this->connection->getQueryBuilder();
$qb->insert('vcategory_to_object') $qb->insert('vcategory_to_object')
->values([ ->values([
'objid' => $qb->createNamedParameter($objectId, \PDO::PARAM_INT), 'objid' => $qb->createNamedParameter($objectId, IQueryBuilder::PARAM_INT),
'categoryid' => $qb->createNamedParameter($category, \PDO::PARAM_INT), 'categoryid' => $qb->createNamedParameter($category, IQueryBuilder::PARAM_INT),
'type' => $qb->createNamedParameter($type), 'type' => $qb->createNamedParameter($type),
]) ])
->execute(); ->execute();