From 48de0cfb46d6561aa9b34ca084ac7c7edef24575 Mon Sep 17 00:00:00 2001
From: Thomas Zilio <Dasc3er@users.noreply.github.com>
Date: Wed, 10 Jul 2019 16:31:10 +0200
Subject: [PATCH] Miglioramento strutturale statistiche Articoli

---
 plugins/statistiche_articoli/edit.php         | 136 ++++++++++--------
 plugins/statistiche_articoli/js/calendar.js   |  25 ++++
 plugins/statistiche_articoli/js/prezzo.js     |  51 +++++++
 plugins/statistiche_articoli/js/stat.js       |  11 ++
 .../{add_prezzo.php => manage_prezzi.php}     |  15 +-
 5 files changed, 168 insertions(+), 70 deletions(-)
 create mode 100644 plugins/statistiche_articoli/js/calendar.js
 create mode 100644 plugins/statistiche_articoli/js/prezzo.js
 create mode 100644 plugins/statistiche_articoli/js/stat.js
 rename plugins/statistiche_articoli/{add_prezzo.php => manage_prezzi.php} (63%)

diff --git a/plugins/statistiche_articoli/edit.php b/plugins/statistiche_articoli/edit.php
index 629d40fb5..f51cdbe9f 100644
--- a/plugins/statistiche_articoli/edit.php
+++ b/plugins/statistiche_articoli/edit.php
@@ -10,7 +10,7 @@ echo '
             '.tr('Periodi temporali').'
         </h4>
         <div class="box-tools pull-right">
-            <button class="btn btn-warning btn-xs" onclick="add_period()">
+            <button class="btn btn-warning btn-xs" onclick="add_calendar()">
                 <i class="fa fa-plus"></i> '.tr('Aggiungi periodo').'
             </button>
             <button type="button" class="btn btn-box-tool" data-widget="collapse">
@@ -33,6 +33,7 @@ echo '
         <table class="table table-striped table-condensed table-bordered">
             <thead>
                 <tr>
+                    <th class="text-center">#</th>
                     <th>'.tr('Perido').'</th>
                     <th>'.tr('Prezzo minimo').'</th>
                     <th>'.tr('Prezzio medio').'</th>
@@ -58,6 +59,7 @@ echo '
         <table class="table table-striped table-condensed table-bordered">
             <thead>
                 <tr>
+                    <th>#</th>
                     <th>'.tr('Perido').'</th>
                     <th>'.tr('Prezzo minimo').'</th>
                     <th>'.tr('Prezzio medio').'</th>
@@ -74,81 +76,89 @@ echo '
     </div>
 </div>
 
+<script src="'.$structure->fileurl('js/stat.js').'"></script>
+<script src="'.$structure->fileurl('js/calendar.js').'"></script>
+<script src="'.$structure->fileurl('js/prezzo.js').'"></script>
+
 <script>
+var calendars = {};
+var info = {
+    url: "'.str_replace('edit.php', '', $structure->fileurl('edit.php')).'",
+    id_module: globals.id_module,
+    id_record: globals.id_record,
+};
+
 $(document).ready(function() {
-    add_period();
+    add_calendar();
 });
 
-function change_period(calendar, start, end) {
-    add_prezzo("#prezzi_acquisto", calendar, "uscita", start, end);
-    add_prezzo("#prezzi_vendita", calendar, "entrata", start, end);
+function remove_calendar(button) {
+    if (Object.keys(calendars).length > 1){
+        var name = $(button).parent().find("input").attr("id");
+    
+        calendars[name].remove();
+        delete calendars[name];
+        
+        $("#group-" + name).remove();
+    } else {
+        swal({
+            title: "'.tr("E' presente un solo calendario!").'",
+            type: "info",
+        });
+    }
 }
 
-function add_prezzo(id, calendar, direzione, start, end) {
-    $.ajax({
-        url: "'.$structure->fileurl('add_prezzo.php').'",
-        type: "get",
-        data: {
-            id_module: globals.id_module,
-            id_record: globals.id_record,
-            calendar: calendar,
-            dir: direzione,
-            start: start,
-            end: end,
-        },
-        success: function(data){
-            var row = $(id).find("#row-" + calendar);
-
-            if (!row.length) {
-                $(id).append(data);
-            } else {
-                row.after(data);
-                row.remove();
-            }
-            
-            $("#row-" + calendar).effect("highlight", {}, 3000);
-        }
-	});
-}
-
-function add_period() {
+function add_calendar() {
     var last = $("#calendars").find("input").last().attr("id");
     var last_id = last ? last.split("-")[1] : 0;
     last_id = parseInt(last_id) + 1;
 
     var name = "calendar-" + last_id;
     
-    $("#calendars").append(\'<div class="col-md-4"><input class="form-control" type="text" name="\' + name + \'" id="\' + name + \'"/><br></div>\');
-
-    $("body").on("focus", "#" + name, function() {
-        $(this).daterangepicker({
-            locale: {
-                customRangeLabel: globals.translations.custom,
-                applyLabel: globals.translations.apply,
-                cancelLabel: globals.translations.cancel,
-                fromLabel: globals.translations.from,
-                toLabel: globals.translations.to,
-            },
-            startDate: globals.start_date,
-            endDate: globals.end_date,
-            applyClass: "btn btn-success btn-sm",
-            cancelClass: "btn btn-danger btn-sm",
-            linkedCalendars: false
-        });
-        
-        var id = $(this).attr("id").split("-")[1];
-        change_period(id, globals.start_date, globals.end_date);
-                
-        $(this).on("apply.daterangepicker", function(ev, picker) {
-            var id = $(this).attr("id").split("-")[1];
-            var start = picker.startDate.format("YYYY-MM-DD");
-            var end = picker.endDate.format("YYYY-MM-DD");
-
-            console.log(id, start, end);
-            change_period(id, start, end);
-        });
+    $("#calendars").append(`<div class="col-md-4" id="group-` + name + `">
+    <div class="input-group">
+        <span class="input-group-addon before">` + last_id + `</span>
+        <input class="form-control calendar-input text-center" type="text" name="` + name + `" id="` + name + `"/>
+        <span class="input-group-addon after clickable btn btn-danger" onclick="remove_calendar(this)">
+            <i class="fa fa-trash-o"></i>
+        </span>
+    </div>
+    <br>
+</div>`);
+    
+    $("#" + name).daterangepicker({
+        locale: {
+            customRangeLabel: globals.translations.custom,
+            applyLabel: globals.translations.apply,
+            cancelLabel: globals.translations.cancel,
+            fromLabel: globals.translations.from,
+            toLabel: globals.translations.to,
+        },
+        startDate: globals.start_date,
+        endDate: globals.end_date,
+        applyClass: "btn btn-success btn-sm",
+        cancelClass: "btn btn-danger btn-sm",
+        linkedCalendars: false
     });
     
-    $("#" + name).focus();
+    $("#" + name).on("apply.daterangepicker", function(ev, picker) {
+        var name = $(this).attr("id");
+        var start = picker.startDate.format("YYYY-MM-DD");
+        var end = picker.endDate.format("YYYY-MM-DD");
+    
+        calendars[name].update(start, end);
+    });
+    
+    // Inizializzazone calendario
+    var calendar = new Calendar(info, last_id);
+    calendars[name] = calendar;
+
+    var prezzo_acquisto = new Prezzo(calendar, "#prezzi_acquisto", "uscita");
+    var prezzo_vendita = new Prezzo(calendar, "#prezzi_vendita", "entrata");
+    
+    calendar.addElement(prezzo_acquisto);
+    calendar.addElement(prezzo_vendita);
+    
+    calendar.update(globals.start_date, globals.end_date);
 }
 </script>';
diff --git a/plugins/statistiche_articoli/js/calendar.js b/plugins/statistiche_articoli/js/calendar.js
new file mode 100644
index 000000000..4e0bc7f7b
--- /dev/null
+++ b/plugins/statistiche_articoli/js/calendar.js
@@ -0,0 +1,25 @@
+class Calendar {
+    constructor(info, id){
+        this.info = info;
+        this.id = id;
+
+        this.elements = new Array();
+    }
+
+    addElement(object){
+        this.elements.push(object);
+    }
+
+    update(start, end) {
+        this.elements.forEach(function (element) {
+            element.update(start, end);
+        });
+    }
+
+    remove() {
+        this.elements.forEach(function (element) {
+            element.remove();
+        });
+    }
+
+}
diff --git a/plugins/statistiche_articoli/js/prezzo.js b/plugins/statistiche_articoli/js/prezzo.js
new file mode 100644
index 000000000..d7f32e8ba
--- /dev/null
+++ b/plugins/statistiche_articoli/js/prezzo.js
@@ -0,0 +1,51 @@
+class Prezzo extends Stat {
+    constructor(calendar, id, direzione){
+        super(calendar, id);
+
+        this.direzione = direzione;
+    }
+
+    getData(start, end, callback) {
+        $.ajax({
+            url: this.calendar.info.url + "/manage_prezzi.php",
+            type: "get",
+            data: {
+                id_module: this.calendar.info.id_module,
+                id_record: this.calendar.info.id_record,
+                calendar_id: this.calendar.id,
+                dir: this.direzione,
+                start: start,
+                end: end,
+            },
+            success: function(data){
+                callback(data)
+            }
+        });
+    }
+
+    add(start, end) {
+        var id = this.id;
+        var calendar_id = this.calendar.id;
+
+        this.getData(start, end, function(data) {
+            var row = $(id).find("#row-" + calendar_id);
+
+            if (!row.length) {
+                $(id).append(data);
+            } else {
+                row.after(data);
+                row.remove();
+            }
+
+            $(id).find("#row-" + calendar_id).effect("highlight", {}, 3000);
+        });
+    }
+
+    update(start, end) {
+        this.add(start, end)
+    }
+
+    remove(start, end) {
+        $(this.id).find("#row-" + this.calendar.id).remove();
+    }
+}
diff --git a/plugins/statistiche_articoli/js/stat.js b/plugins/statistiche_articoli/js/stat.js
new file mode 100644
index 000000000..42786d574
--- /dev/null
+++ b/plugins/statistiche_articoli/js/stat.js
@@ -0,0 +1,11 @@
+class Stat {
+    constructor(calendar, id){
+        this.calendar = calendar;
+
+        this.id = id;
+    }
+
+    add(start, end){}
+    update(start, end){}
+    remove(){}
+}
diff --git a/plugins/statistiche_articoli/add_prezzo.php b/plugins/statistiche_articoli/manage_prezzi.php
similarity index 63%
rename from plugins/statistiche_articoli/add_prezzo.php
rename to plugins/statistiche_articoli/manage_prezzi.php
index c41956413..7b39ead81 100644
--- a/plugins/statistiche_articoli/add_prezzo.php
+++ b/plugins/statistiche_articoli/manage_prezzi.php
@@ -4,7 +4,7 @@ include_once __DIR__.'/../../core.php';
 
 use Plugins\StatisticheArticoli\Stats;
 
-$calendar = filter('calendar');
+$calendar_id = filter('calendar_id');
 $direzione = filter('dir');
 $start = filter('start');
 $end = filter('end');
@@ -28,12 +28,13 @@ if ($data_min == $data_max) {
 }
 
 echo '
-<tr id="row-'.$calendar.'">
+<tr id="row-'.$calendar_id.'">
+    <td class="text-center">'.$calendar_id.'</td>
     <td>'.dateFormat($start).' - '.dateFormat($end).'</td>
-    <td>'.moneyFormat($prezzo_min['prezzo']).'</td>
-    <td>'.moneyFormat($prezzo_medio).'</td>
-    <td>'.moneyFormat($prezzo_max['prezzo']).'</td>
-    <td>'.moneyFormat($oscillazione).'</td>
-    <td>'.Translator::numberToLocale($oscillazione_percentuale, '2').' %</td>
+    <td class="text-right">'.moneyFormat($prezzo_min['prezzo']).'</td>
+    <td class="text-right">'.moneyFormat($prezzo_medio).'</td>
+    <td class="text-right">'.moneyFormat($prezzo_max['prezzo']).'</td>
+    <td class="text-right">'.moneyFormat($oscillazione).'</td>
+    <td class="text-right">'.Translator::numberToLocale($oscillazione_percentuale, '2').' %</td>
     <td>'.$andamento.'</td>
 </tr>';