diff options
author | Bernhard Posselt <nukeawhale@gmail.com> | 2013-09-07 23:00:34 +0200 |
---|---|---|
committer | Bernhard Posselt <nukeawhale@gmail.com> | 2013-09-07 23:00:34 +0200 |
commit | ee861fa6dd80378be60cb76d2fb28c5e0e808ab6 (patch) | |
tree | ee068e63498b9bfddff3cda366607b43cef688de /db | |
parent | 4a36e0e4af8df6971be40f08c8ab0fd65c78fdd6 (diff) |
extra work for postgres, fix #338
Diffstat (limited to 'db')
-rw-r--r-- | db/itemmapper.php | 2 | ||||
-rw-r--r-- | db/mapperfactory.php | 52 | ||||
-rw-r--r-- | db/postgres/itemmapper.php | 80 |
3 files changed, 132 insertions, 2 deletions
diff --git a/db/itemmapper.php b/db/itemmapper.php index eaf24dc49..0a78b02df 100644 --- a/db/itemmapper.php +++ b/db/itemmapper.php @@ -257,7 +257,6 @@ class ItemMapper extends Mapper implements IMapper { $params = array($status, $threshold); $result = $this->execute($sql, $params); - /* FIXME: this is broken on posgres while($row = $result->fetchRow()) { $size = (int) $row['size']; @@ -274,7 +273,6 @@ class ItemMapper extends Mapper implements IMapper { $this->execute($sql, $params, $limit); } } - */ } diff --git a/db/mapperfactory.php b/db/mapperfactory.php new file mode 100644 index 000000000..cf64f90d6 --- /dev/null +++ b/db/mapperfactory.php @@ -0,0 +1,52 @@ +<?php + +/** +* ownCloud - News +* +* @author Alessandro Cosentino +* @author Bernhard Posselt +* @copyright 2012 Alessandro Cosentino cosenal@gmail.com +* @copyright 2012 Bernhard Posselt dev@bernhard-posselt.com +* +* This library is free software; you can redistribute it and/or +* modify it under the terms of the GNU AFFERO GENERAL PUBLIC LICENSE +* License as published by the Free Software Foundation; either +* version 3 of the License, or any later version. +* +* This library is distributed in the hope that it will be useful, +* but WITHOUT ANY WARRANTY; without even the implied warranty of +* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +* GNU AFFERO GENERAL PUBLIC LICENSE for more details. +* +* You should have received a copy of the GNU Affero General Public +* License along with this library. If not, see <http://www.gnu.org/licenses/>. +* +*/ + +namespace OCA\News\Db; + +use \OCA\AppFramework\Core\API; + + +class MapperFactory { + + private $api; + + public function __construct(API $api) { + $this->api = $api; + } + + + public function getItemMapper() { + switch($this->api->getSystemValue('dbtype')) { + case 'pgsql': + return new \OCA\News\Db\Postgres\ItemMapper($this->api); + break; + default: + return new ItemMapper($this->api); + break; + } + } + + +}
\ No newline at end of file diff --git a/db/postgres/itemmapper.php b/db/postgres/itemmapper.php new file mode 100644 index 000000000..ddf83ae10 --- /dev/null +++ b/db/postgres/itemmapper.php @@ -0,0 +1,80 @@ +<?php +/** +* ownCloud - News app +* +* @author Alessandro Cosentino +* @author Bernhard Posselt +* @copyright 2012 Alessandro Cosentino cosenal@gmail.com +* @copyright 2012 Bernhard Posselt dev@bernhard-posselt.com +* +* This library is free software; you can redistribute it and/or +* modify it under the terms of the GNU AFFERO GENERAL PUBLIC LICENSE +* License as published by the Free Software Foundation; either +* version 3 of the License, or any later version. +* +* This library is distributed in the hope that it will be useful, +* but WITHOUT ANY WARRANTY; without even the implied warranty of +* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +* GNU AFFERO GENERAL PUBLIC LICENSE for more details. +* +* You should have received a copy of the GNU Affero General Public +* License along with this library. If not, see <http://www.gnu.org/licenses/>. +* +*/ + +namespace OCA\News\Db\Postgres; + +use \OCA\AppFramework\Db\DoesNotExistException; +use \OCA\AppFramework\Db\MultipleObjectsReturnedException; +use \OCA\AppFramework\Db\Mapper; +use \OCA\AppFramework\Core\API; + +use \OCA\News\Db\StatusFlag; + + +class ItemMapper extends \OCA\News\Db\ItemMapper { + + public function __construct(API $api){ + parent::__construct($api); + } + + + /** + * Delete all items for feeds that have over $threshold unread and not + * starred items + */ + public function deleteReadOlderThanThreshold($threshold){ + $status = StatusFlag::STARRED | StatusFlag::UNREAD; + $sql = 'SELECT COUNT(*) `size`, `feed_id` ' . + 'FROM `*PREFIX*news_items` ' . + 'WHERE NOT ((`status` & ?) > 0) ' . + 'GROUP BY `feed_id` ' . + 'HAVING COUNT(*) > ?'; + $params = array($status, $threshold); + $result = $this->execute($sql, $params); + + while($row = $result->fetchRow()) { + + $size = (int) $row['size']; + $limit = $size - $threshold; + + if($limit > 0) { + $params = array($status, $row['feed_id'], $limit); + + $sql = 'DELETE FROM `*PREFIX*news_items` ' . + 'WHERE `id` IN (' . + 'SELECT `id` FROM `*PREFIX*news_items` ' . + 'WHERE NOT ((`status` & ?) > 0) ' . + 'AND `feed_id` = ? ' . + 'ORDER BY `id` ASC ' . + 'LIMIT ?' . + ')'; + + $this->execute($sql, $params); + } + } + + } + + +}
\ No newline at end of file |