Simplify getUsedBy() in Packet View controller

Author: carlbennett

src/controllers/Packet/View.php

@@ -32,7 +32,9 @@ public function &run(Router &$router, ViewLib &$view, array &$args) {
         Comment::PARENT_TYPE_PACKET,
         $model->packet_id
       );
-      $model->used_by = $this->getUsedBy($model->packet);
+      $model->used_by = Product::getProductsFromIds(
+        $model->packet->getUsedBy()
+      );
     } else {
       $model->used_by = null;
     }
@@ -47,9 +49,4 @@ public function &run(Router &$router, ViewLib &$view, array &$args) {
 
   }
 
-  protected function getUsedBy(Packet &$packet) {
-    if (is_null($packet)) return null;
-    return Product::getProductsFromIds($packet->getUsedBy());
-  }
-
 }