From cd10f93e0d271840caff92b88195385a2a3f15b2 Mon Sep 17 00:00:00 2001 From: Bernhard Posselt Date: Thu, 26 Jun 2014 11:33:44 +0200 Subject: style fixes --- db/feedmapper.php | 2 +- db/imapper.php | 4 +++- db/itemmapper.php | 2 +- db/mapperfactory.php | 1 - 4 files changed, 5 insertions(+), 4 deletions(-) (limited to 'db') diff --git a/db/feedmapper.php b/db/feedmapper.php index 21fbd1d12..dde806208 100644 --- a/db/feedmapper.php +++ b/db/feedmapper.php @@ -85,7 +85,7 @@ class FeedMapper extends Mapper implements IMapper { 'LEFT JOIN `*PREFIX*news_items` `items` ' . 'ON `feeds`.`id` = `items`.`feed_id` ' . // WARNING: this is a desperate attempt at making this query work - // because prepared statements dont work. This is a possible + // because prepared statements don't work. This is a possible // SQL INJECTION RISK WHEN MODIFIED WITHOUT THOUGHT. // think twice when changing this 'AND (`items`.`status` & ' . StatusFlag::UNREAD . ') = ' . diff --git a/db/imapper.php b/db/imapper.php index 9882060f0..18a924e24 100644 --- a/db/imapper.php +++ b/db/imapper.php @@ -20,13 +20,15 @@ interface IMapper { /** * @param int $id the id of the feed * @param string $userId the id of the user + * @return \OCP\AppFramework\Db\Entity */ public function find($id, $userId); /** * Delete an entity * @param Entity $entity the entity that should be deleted - * @throws ServiceNotFoundException if the entity does not exist, or there + * @throws \OCP\AppFramework\Db\DoesNotExistException if the entity does + * not exist, or there * are more than one of it */ public function delete(Entity $entity); diff --git a/db/itemmapper.php b/db/itemmapper.php index 477848a32..c7a0902ca 100644 --- a/db/itemmapper.php +++ b/db/itemmapper.php @@ -80,7 +80,7 @@ class ItemMapper extends Mapper implements IMapper { 'ON `items`.`feed_id` = `feeds`.`id` ' . 'AND `feeds`.`user_id` = ? ' . // WARNING: this is a desperate attempt at making this query work - // because prepared statements dont work. This is a possible + // because prepared statements don't work. This is a possible // SQL INJECTION RISK WHEN MODIFIED WITHOUT THOUGHT. // think twice when changing this 'WHERE ((`items`.`status` & ' . StatusFlag::STARRED . ') = ' . diff --git a/db/mapperfactory.php b/db/mapperfactory.php index 339ff30b8..4af75cefa 100644 --- a/db/mapperfactory.php +++ b/db/mapperfactory.php @@ -13,7 +13,6 @@ namespace OCA\News\Db; -use \OCP\IConfig; use \OCP\IDb; use \OCA\News\Db\Postgres\ItemMapper as PostgresItemMapper; -- cgit v1.2.3