From f80878422d3e6c951334c499ef14e019ce7130eb Mon Sep 17 00:00:00 2001 From: Frederic Guillot Date: Tue, 27 Dec 2016 21:16:40 -0500 Subject: [PATCH] Avoid useless SQL query when viewing item --- app/controllers/item.php | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/app/controllers/item.php b/app/controllers/item.php index 6708471..7bf060b 100644 --- a/app/controllers/item.php +++ b/app/controllers/item.php @@ -40,8 +40,10 @@ Router\get_action('show', function () { $feed = Model\Feed\get_feed($user_id, $item['feed_id']); $group_id = Request\int_param('group_id', null); - Model\Item\change_item_status($user_id, $item_id, Model\Item\STATUS_READ); - $item['status'] = 'read'; + if ($item['status'] !== Model\Item\STATUS_READ) { + Model\Item\change_item_status($user_id, $item_id, Model\Item\STATUS_READ); + $item['status'] = Model\Item\STATUS_READ; + } switch ($menu) { case 'unread':