added reload icon and fixed a bug where random could redirect to deleted articles

[github/wallabag/wallabag.git] / inc / poche / Poche.class.php

diff --git a/inc/poche/Poche.class.php b/inc/poche/Poche.class.php
index fb74ab9a0aaa23ec1e0ba0eb7b89e4de8cda384a..f992814583ba7f7fb56f9805c1219b59910177c6 100755 (executable)
--- a/inc/poche/Poche.class.php
+++ b/inc/poche/Poche.class.php
@@ -349,8 +349,11 @@ class Poche
                 
             /* For some unknown reason I can't get displayView() to work here (it redirects to home view afterwards). So here's a dirty fix which redirects directly to URL */
             case 'random':
-                $count = $this->store->getEntriesByViewCount($view, $this->user->getId());
-                $id = rand(1,$count);
+                $id = 0;
+                while ($this->store->retrieveOneById($id,$this->user->getId()) == null) {
+                    $count = $this->store->getEntriesByViewCount($view, $this->user->getId());
+                    $id = rand(1,$count);
+                }
                 Tools::logm('get a random article');
                 Tools::redirect('?view=view&id=' . $id);
                 //$this->displayView('view', $id);