Further simplifications to viewable(). Stop trying to optimize for

the case where we just have one restriction, it's unnecessary.

1 files changed, 1 insertions, 9 deletions

diff --git a/modules/gallery/helpers/item.php b/modules/gallery/helpers/item.php
index 300a6942..b7be23cd 100644
--- a/modules/gallery/helpers/item.php
+++ b/modules/gallery/helpers/item.php
@@ -158,17 +158,9 @@ class item_Core {
         $view_restrictions["items.view_$id"] = access::ALLOW;
       }
     }
-    switch (count($view_restrictions)) {
-    case 0:
-      break;
 
-    case 1:
-      $model->where($view_restrictions[0], "=", access::ALLOW);
-      break;
-
-    default:
+    if (count($view_restrictions)) {
       $model->and_open()->or_where($view_restrictions)->close();
-      break;
     }
 
     return $model;