From 0dd95da8de54110ce93edba08e0bd5d405a7676a Mon Sep 17 00:00:00 2001
From: Hannes Matuschek <hmatuschek@gmail.com>
Date: Sun, 30 Aug 2020 15:51:27 +0200
Subject: [PATCH] Added low-stock part listing.

---
 src/Controller/PartListsController.php        | 17 +++++++++++++++++
 src/DataTables/PartsDataTable.php             | 15 ++++++++++++++-
 src/Services/Trees/ToolsTreeBuilder.php       |  5 +++++
 templates/Parts/lists/lowstock_list.html.twig | 11 +++++++++++
 4 files changed, 47 insertions(+), 1 deletion(-)
 create mode 100644 templates/Parts/lists/lowstock_list.html.twig

diff --git a/src/Controller/PartListsController.php b/src/Controller/PartListsController.php
index ce5e9495..63e35d83 100644
--- a/src/Controller/PartListsController.php
+++ b/src/Controller/PartListsController.php
@@ -276,4 +276,21 @@ class PartListsController extends AbstractController
 
         return $this->render('Parts/lists/all_list.html.twig', ['datatable' => $table]);
     }
+
+    /**
+     * @Route("/lowstock", name="parts_show_low_stock")
+     *
+     * @return JsonResponse|Response
+     */
+    public function showLowStock(Request $request, DataTableFactory $dataTable)
+    {
+        $table = $dataTable->createFromType(PartsDataTable::class, ['lowstock'=>true])
+            ->handleRequest($request);
+
+        if ($table->isCallback()) {
+            return $table->getResponse();
+        }
+
+        return $this->render('Parts/lists/lowstock_list.html.twig', ['datatable' => $table]);
+    }
 }
diff --git a/src/DataTables/PartsDataTable.php b/src/DataTables/PartsDataTable.php
index 0290b29e..89a79d99 100644
--- a/src/DataTables/PartsDataTable.php
+++ b/src/DataTables/PartsDataTable.php
@@ -53,6 +53,7 @@ use App\Entity\Parts\Category;
 use App\Entity\Parts\Footprint;
 use App\Entity\Parts\Manufacturer;
 use App\Entity\Parts\Part;
+use App\Entity\Parts\PartLot;
 use App\Entity\Parts\Storelocation;
 use App\Entity\Parts\Supplier;
 use App\Services\AmountFormatter;
@@ -107,6 +108,7 @@ final class PartsDataTable implements DataTableTypeInterface
             'storelocation' => null,
             'supplier' => null,
             'tag' => null,
+            'lowstock' => null,
             'search' => null,
         ]);
 
@@ -115,6 +117,7 @@ final class PartsDataTable implements DataTableTypeInterface
         $optionsResolver->setAllowedTypes('manufacturer', ['null', Manufacturer::class]);
         $optionsResolver->setAllowedTypes('supplier', ['null', Supplier::class]);
         $optionsResolver->setAllowedTypes('tag', ['null', 'string']);
+        $optionsResolver->setAllowedTypes('lowstock', ['null', 'bool']);
         $optionsResolver->setAllowedTypes('search', ['null', 'string']);
 
         //Configure search options
@@ -233,7 +236,7 @@ final class PartsDataTable implements DataTableTypeInterface
             ])
             ->add('minamount', TextColumn::class, [
                 'label' => $this->translator->trans('part.table.minamount'),
-                'visible' => false,
+                'visible' => isset($options["lowstock"]),
                 'render' => function ($value, Part $context) {
                     return $this->amountFormatter->format($value, $context->getPartUnit());
                 },
@@ -395,6 +398,16 @@ final class PartsDataTable implements DataTableTypeInterface
             $builder->andWhere('part.tags LIKE :tag')->setParameter('tag', '%'.$options['tag'].'%');
         }
 
+        if (isset($options['lowstock'])) {
+            $in = $builder->getEntityManager()->createQueryBuilder();
+            $in->select("parts.id")
+                ->from(Part::class, 'parts')
+                ->innerJoin("parts.partLots", "lots")
+                ->groupBy('parts.id')
+                ->having('SUM(lots.amount)<parts.minamount');
+            $builder->andWhere($builder->expr()->in('part', $in->getDQL()));
+        }
+        
         if (!empty($options['search'])) {
             if (!$options['search_options']['regex']) {
                 //Dont show results, if no things are selected
diff --git a/src/Services/Trees/ToolsTreeBuilder.php b/src/Services/Trees/ToolsTreeBuilder.php
index 759d19cc..6c9795f2 100644
--- a/src/Services/Trees/ToolsTreeBuilder.php
+++ b/src/Services/Trees/ToolsTreeBuilder.php
@@ -245,6 +245,11 @@ class ToolsTreeBuilder
             $this->urlGenerator->generate('parts_show_all')
         );
 
+        $show_nodes[] = new TreeViewNode(
+            $this->translator->trans('tree.tools.show.low_stock_parts'),
+            $this->urlGenerator->generate('parts_show_low_stock')
+        );
+
         if ($this->security->isGranted('read', new PartAttachment())) {
             $show_nodes[] = new TreeViewNode(
                 $this->translator->trans('tree.tools.show.all_attachments'),
diff --git a/templates/Parts/lists/lowstock_list.html.twig b/templates/Parts/lists/lowstock_list.html.twig
new file mode 100644
index 00000000..18112254
--- /dev/null
+++ b/templates/Parts/lists/lowstock_list.html.twig
@@ -0,0 +1,11 @@
+{% extends "base.html.twig" %}
+
+{% block  title %}
+{% trans %}parts_list.lowstock.title{% endtrans %}
+{% endblock %}
+
+{% block content %}
+
+    {% include "Parts/lists/_parts_list.html.twig" %}
+
+{% endblock %}