Add ability to edit the products a packet is used with

* Uses a series of checkboxes passed as an array of bnet product ID's
* Adds Packet->setUsedBy($values) function, which actually updates the database.
* Adds Product::getProductsFromIds($ids) function which returns an array of product objects from the given array of product ID's. This replaces most of the Packet\Edit controller's getUsedBy() function.

Author: Davnit

src/controllers/Packet/Edit.php

@@ -9,6 +9,7 @@
 use \BNETDocs\Libraries\Logger;
 use \BNETDocs\Libraries\Packet;
 use \BNETDocs\Libraries\User;
+use \BNETDocs\Libraries\Product;
 use \BNETDocs\Models\Packet\Edit as PacketEditModel;
 
 use \CarlBennett\MVC\Libraries\Common;
@@ -39,6 +40,7 @@ public function &run(Router &$router, View &$view, array &$args) {
     $model->published  = null;
     $model->remarks    = null;
     $model->user       = Authentication::$user;
+    $model->used_by    = null;
 
     $model->acl_allowed = ($model->user && $model->user->getAcl(
       User::OPTION_ACL_PACKET_MODIFY
@@ -59,6 +61,7 @@ public function &run(Router &$router, View &$view, array &$args) {
       $model->remarks   = $model->packet->getPacketRemarks(false);
       $model->markdown  = ($flags & Packet::OPTION_MARKDOWN);
       $model->published = ($flags & Packet::OPTION_PUBLISHED);
+      $model->used_by   = $this->getUsedBy($model->packet);
 
       if ($router->getRequestMethod() == "POST") {
         $this->handlePost($router, $model);
@@ -96,6 +99,7 @@ protected function handlePost(Router &$router, PacketEditModel &$model) {
     $content    = (isset($data["content"   ]) ? $data["content"   ] : null);
     $publish    = (isset($data["publish"   ]) ? $data["publish"   ] : null);
     $save       = (isset($data["save"      ]) ? $data["save"      ] : null);
+    $used_by    = (isset($data["used_by"   ]) ? $data["used_by"   ] : null);
 
     $model->id       = $id;
     $model->name     = $name;
@@ -135,6 +139,8 @@ protected function handlePost(Router &$router, PacketEditModel &$model) {
       $model->packet->setEditedDateTime(
         new DateTime("now", new DateTimeZone("Etc/UTC"))
       );
+      
+      $model->packet->setUsedBy($used_by);
 
       $success = $model->packet->update();
 
@@ -166,8 +172,14 @@ protected function handlePost(Router &$router, PacketEditModel &$model) {
         "name"            => $model->packet->getPacketName(),
         "format"          => $model->packet->getPacketFormat(),
         "remarks"         => $model->packet->getPacketRemarks(false),
+        "used_by"         => $used_by
       ])
     );
   }
 
+  protected function getUsedBy(Packet &$packet) {
+    if (is_null($packet)) return null;
+    return Product::getProductsFromIds($packet->getUsedBy());
+  }
+
 }

src/controllers/Packet/View.php

@@ -49,12 +49,7 @@ public function &run(Router &$router, ViewLib &$view, array &$args) {
 
   protected function getUsedBy(Packet &$packet) {
     if (is_null($packet)) return null;
-    $used_by = $packet->getUsedBy();
-    $products = [];
-    foreach ($used_by as $bnet_product_id) {
-      $products[] = new Product($bnet_product_id);
-    }
-    return $products;
+    return Product::getProductsFromIds($packet->getUsedBy());
   }
 
 }

src/libraries/Packet.php

@@ -702,6 +702,49 @@ public function setPublished( $value ) {
       $this->options_bitmask &= ~self::OPTION_PUBLISHED;
     }
   }
+  
+  public function setUsedBy( $value ) {
+    if (!isset(Common::$database)) {
+      Common::$database = DatabaseDriver::getDatabaseObject();
+    }
+    try {
+      Common::$database->beginTransaction();
+      $stmt = Common::$database->prepare('
+        DELETE FROM `packet_used_by`
+        WHERE `id` = :id;
+      ');
+      $stmt->bindParam(':id', $this->id, PDO::PARAM_INT);
+      if (!$stmt->execute()) {
+        throw new QueryException('Cannot update packet used by');
+      }
+      if ($value !== null && count($value) > 0) {
+        $insert = [];
+        $placeholders = [];
+        foreach ($value as $v) {
+          array_push($insert, $this->id, (int)$v);
+          $placeholders[] = '(?, ?)';
+        }
+        $stmt = Common::$database->prepare('
+          INSERT INTO `packet_used_by`
+            (`id`, `bnet_product_id`)
+          VALUES
+            ' . implode(', ', $placeholders) . '; 
+        ');
+        if (!$stmt->execute($insert)) {
+          Common::$database->rollBack();
+          throw new QueryException('Cannot update packet used by');
+        }
+      }
+
+      Common::$database->commit();
+
+      $cache_key = 'bnetdocs-packetusedby-' . $this->id;
+      Common::$cache->set($cache_key, serialize($value), self::CACHE_TTL);
+    } catch (PDOException $e) {
+      Common::$database->rollBack();
+      throw new QueryException('Cannot update packet used by', $e);
+    }
+  }
 
   public function update() {
     if (!isset(Common::$database)) {

src/libraries/Product.php

@@ -90,6 +90,16 @@ public static function getAllProducts() {
     return null;
   }
 
+  public static function getProductsFromIds($product_ids) {
+    $products = [];
+    if ($product_ids !== null) {
+      foreach ($product_ids as $bnet_product_id) {
+        $products[] = new Product($bnet_product_id);
+      }
+    }
+    return $products;
+  }
+
   public function getBnetProductId() {
     return $this->bnet_product_id;
   }

src/models/Packet/Edit.php

@@ -19,5 +19,6 @@ class Edit extends Model {
   public $published;
   public $remarks;
   public $user;
+  public $used_by;
 
 }

src/templates/Packet/Edit.phtml

@@ -2,6 +2,8 @@
 
 namespace BNETDocs\Templates\Packet;
 
+use \BNETDocs\Libraries\Product;
+
 use \CarlBennett\MVC\Libraries\Common;
 use \CarlBennett\MVC\Libraries\Pair;
 
@@ -82,6 +84,23 @@ require('./header.inc.phtml');
                 filter_var( $this->getContext()->format, FILTER_SANITIZE_FULL_SPECIAL_CHARS );
               ?></textarea>
           </section>
+          <section>
+          	<label for="usedby">Used by:</label>
+          	<table><?php
+          	  $all_products = Product::getAllProducts();
+          	  $used_by = $this->getContext()->used_by;
+          	  $j = count($all_products);
+          	  for ($i = 0; $i < $j; ++$i) { ?>
+          	  <tr><td><input type="checkbox" name="used_by[]" value="<?php 
+          	    echo $all_products[$i]->getBnetProductId() . "\"" . ($used_by !== null && in_array($all_products[$i], $used_by) ? " checked> " : "> ") . filter_var($all_products[$i]->getLabel(), FILTER_SANITIZE_STRING) . "</td><td>";
+          	    if ($i + 1 < $j) {
+          	      ++$i;
+          	      ?><input type="checkbox" name="used_by[]" value="<?php 
+          	      echo $all_products[$i]->getBnetProductId() . "\"" . ($used_by !== null && in_array($all_products[$i], $used_by) ? " checked> " : "> ") . filter_var($all_products[$i]->getLabel(), FILTER_SANITIZE_STRING); ?></td></tr>
+<?php      	    }
+              } ?>
+			</table>
+          </section>
           <section>
             <label for="remarks">Remarks:</label>
             <span style="float:right;">