Throw ServiceNotFoundException when marking non-existent item as read (#61)

* Throw ServiceNotFoundException when marking non-existent item as read

Fixes #58

* Add test case for marking non-existent item as read

1 files changed, 6 insertions, 2 deletions

diff --git a/lib/Service/ItemService.php b/lib/Service/ItemService.php
index fdf97c27b..f657b3eff 100644
--- a/lib/Service/ItemService.php
+++ b/lib/Service/ItemService.php
@@ -147,8 +147,12 @@ class ItemService extends Service {
      * @throws ServiceNotFoundException if the item does not exist
      */
     public function read($itemId, $isRead, $userId){
-        $lastModified = $this->timeFactory->getMicroTime();
-        $this->itemMapper->readItem($itemId, $isRead, $lastModified, $userId);
+        try {
+            $lastModified = $this->timeFactory->getMicroTime();
+            $this->itemMapper->readItem($itemId, $isRead, $lastModified, $userId);
+        } catch(DoesNotExistException $ex) {
+            throw new ServiceNotFoundException($ex->getMessage());
+        }
     }