Merge pull request #88 from Davnit/packet-edit-usedby

Add editor for the list of products a packet is used by

Author: carlbennett

src/controllers/Packet/Edit.php

@@ -8,6 +8,7 @@
 use \BNETDocs\Libraries\Exceptions\PacketNotFoundException;
 use \BNETDocs\Libraries\Logger;
 use \BNETDocs\Libraries\Packet;
+use \BNETDocs\Libraries\Product;
 use \BNETDocs\Libraries\User;
 use \BNETDocs\Models\Packet\Edit as PacketEditModel;
 
@@ -38,6 +39,7 @@ public function &run(Router &$router, View &$view, array &$args) {
     $model->packet_id  = (isset($data["id"]) ? $data["id"] : null);
     $model->published  = null;
     $model->remarks    = null;
+    $model->used_by    = null;
     $model->user       = Authentication::$user;
 
     $model->acl_allowed = ($model->user && $model->user->getAcl(
@@ -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;
@@ -138,6 +142,10 @@ protected function handlePost(Router &$router, PacketEditModel &$model) {
 
       $success = $model->packet->update();
 
+      // Used-by is stored in a different table than packet data so it is
+      // updated separately.
+      $model->packet->setUsedBy($used_by);
+
     } catch (QueryException $e) {
 
       // SQL error occurred. We can show a friendly message to the user while
@@ -166,8 +174,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

@@ -725,6 +725,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

@@ -65,7 +65,7 @@ public static function getAllProducts() {
           `bnls_product_id`,
           `label`,
           `sort`,
-          `version_byte
+          `version_byte`
         FROM `products`
         ORDER BY `sort` ASC;
       ");
@@ -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 self($bnet_product_id);
+      }
+    }
+    return $products;
+  }
+
   public function getBnetProductId() {
     return $this->bnet_product_id;
   }

src/models/Packet/Edit.php

@@ -18,6 +18,7 @@ class Edit extends Model {
   public $packet_id;
   public $published;
   public $remarks;
+  public $used_by;
   public $user;
 
 }

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;">