From d10033112d057b1d8770f9c36b23c7d748e44e7e Mon Sep 17 00:00:00 2001
From: Thomas Zilio <Dasc3er@users.noreply.github.com>
Date: Thu, 1 Aug 2019 10:08:34 +0200
Subject: [PATCH] Fix JS prima nota

---
 assets/src/js/custom.js                       |  2 -
 assets/src/js/functions/init.js               | 12 ++--
 include/bottom.php                            | 60 ++++++++++---------
 modules/primanota/actions.php                 | 22 +++----
 modules/primanota/add.php                     | 12 +---
 modules/primanota/edit.php                    | 11 +---
 modules/primanota/init.php                    |  4 +-
 .../src/{PrimaNota.php => Mastrino.php}       |  2 +-
 modules/primanota/src/Movimento.php           | 12 ++--
 9 files changed, 63 insertions(+), 74 deletions(-)
 rename modules/primanota/src/{PrimaNota.php => Mastrino.php} (99%)

diff --git a/assets/src/js/custom.js b/assets/src/js/custom.js
index b358188c9..928c16f66 100644
--- a/assets/src/js/custom.js
+++ b/assets/src/js/custom.js
@@ -63,8 +63,6 @@ $(document).ready(function () {
     // Richiamo alla generazione di Datatables
     start_datatables();
 
-    init();
-
     // Calendario principale
     start_complete_calendar("#daterange", function (start, end) {
         // Esegue il submit del periodo selezionato e ricarica la pagina
diff --git a/assets/src/js/functions/init.js b/assets/src/js/functions/init.js
index cc7e9906c..866f52574 100644
--- a/assets/src/js/functions/init.js
+++ b/assets/src/js/functions/init.js
@@ -27,14 +27,10 @@ function init() {
         $('form').not('.no-check').parsley();
     }
 
-    window.Parsley.on('field:success', function () {
-        this.$element.removeClass('parsley-success');
-    });
-
     // Aggiunta nell'URL del nome tab su cui tornare dopo il submit
     // Blocco del pulsante di submit dopo il primo submit
-    $("form").submit(function () {
-        if ($(this).parsley().validate()) {
+    $('form').on("submit", function(e) {
+        if ($(this).parsley().validate() && e.result) {
             $(this).submit(function () {
                 return false;
             });
@@ -60,5 +56,9 @@ function init() {
         return false;
     });
 
+    window.Parsley.on('field:success', function () {
+        this.$element.removeClass('parsley-success');
+    });
+
     restart_inputs();
 }
diff --git a/include/bottom.php b/include/bottom.php
index f847b806f..ac9575111 100644
--- a/include/bottom.php
+++ b/include/bottom.php
@@ -51,37 +51,41 @@ if (Auth::check()) {
 
     // Hooks
     echo '
-    <script>
-    $(document).ready(function() {
-        alertPush();
-        
-        $.ajax({
-            url: globals.rootdir + "/ajax.php",
-            type: "get",
-            data: {
-                op: "hooks",
-            },
-            success: function(data) {
-                hooks = JSON.parse(data);
-
-                $("#hooks-header").text(globals.translations.hooksExecuting);
-
-                if (hooks.length == 0) {
-                    $("#hooks-loading").hide();
-                    $("#hooks-number").text(0);
-                    $("#hooks-header").text(globals.translations.hookNone);
-                }
-                
-                hooks.forEach(function(item, index){
-                    executeHook(item, hooks.length);
-                });
-            },
-        });
-    });
+        <script>
+        $(document).ready(function() {
+            alertPush();
+            
+            $.ajax({
+                url: globals.rootdir + "/ajax.php",
+                type: "get",
+                data: {
+                    op: "hooks",
+                },
+                success: function(data) {
+                    hooks = JSON.parse(data);
     
-    </script>';
+                    $("#hooks-header").text(globals.translations.hooksExecuting);
+    
+                    if (hooks.length == 0) {
+                        $("#hooks-loading").hide();
+                        $("#hooks-number").text(0);
+                        $("#hooks-header").text(globals.translations.hookNone);
+                    }
+                    
+                    hooks.forEach(function(item, index){
+                        executeHook(item, hooks.length);
+                    });
+                },
+            });
+        });
+        
+        </script>';
 }
 
+echo '
+        <script>$(document).ready(init)</script>';
+
+
 echo '
 	</body>
 </html>';
diff --git a/modules/primanota/actions.php b/modules/primanota/actions.php
index e72df5f71..9ec3f9f2d 100644
--- a/modules/primanota/actions.php
+++ b/modules/primanota/actions.php
@@ -3,7 +3,7 @@
 include_once __DIR__.'/../../core.php';
 
 use Modules\PrimaNota\Movimento;
-use Modules\PrimaNota\PrimaNota;
+use Modules\PrimaNota\Mastrino;
 use Modules\Scadenzario\Scadenza;
 
 switch (post('op')) {
@@ -12,7 +12,7 @@ switch (post('op')) {
         $descrizione = post('descrizione');
         $is_insoluto = post('is_insoluto');
 
-        $prima_nota = PrimaNota::build($descrizione, $data, $is_insoluto, true);
+        $mastrino = Mastrino::build($descrizione, $data, $is_insoluto, true);
 
         $conti = post('idconto');
         foreach ($conti as $i => $id_conto) {
@@ -22,14 +22,14 @@ switch (post('op')) {
 
             $scadenza = Scadenza::find($id_scadenza);
 
-            $movimento = Movimento::build($prima_nota, $id_conto, $scadenza);
+            $movimento = Movimento::build($mastrino, $id_conto, $scadenza);
             $movimento->setTotale($avere, $dare);
             $movimento->save();
         }
 
-        $prima_nota->aggiornaScadenzario();
+        $mastrino->aggiornaScadenzario();
 
-        $id_record = $prima_nota->id;
+        $id_record = $mastrino->id;
 
         flash()->info(tr('Movimento aggiunto in prima nota!'));
 
@@ -55,10 +55,10 @@ switch (post('op')) {
         $data = post('data');
         $descrizione = post('descrizione');
 
-        $prima_nota->descrizione = $descrizione;
-        $prima_nota->data = $data;
+        $mastrino->descrizione = $descrizione;
+        $mastrino->data = $data;
 
-        $prima_nota->cleanup();
+        $mastrino->cleanup();
 
         $conti = post('idconto');
         foreach ($conti as $i => $id_conto) {
@@ -68,18 +68,18 @@ switch (post('op')) {
 
             $scadenza = Scadenza::find($id_scadenza);
 
-            $movimento = Movimento::build($prima_nota, $id_conto, $scadenza);
+            $movimento = Movimento::build($mastrino, $id_conto, $scadenza);
             $movimento->setTotale($avere, $dare);
             $movimento->save();
         }
 
-        $prima_nota->aggiornaScadenzario();
+        $mastrino->aggiornaScadenzario();
 
         flash()->info(tr('Movimento modificato in prima nota!'));
         break;
 
     // eliminazione movimento prima nota
     case 'delete':
-        $prima_nota->delete();
+        $mastrino->delete();
         break;
 }
diff --git a/modules/primanota/add.php b/modules/primanota/add.php
index faa9d57bf..c8018b0ac 100644
--- a/modules/primanota/add.php
+++ b/modules/primanota/add.php
@@ -237,15 +237,9 @@ include $structure->filepath('movimenti.php');
     var nuovo_modello = "<?php echo tr('Aggiungi e crea modello'); ?>";
     var modifica_modello = "<?php echo tr('Aggiungi e modifica modello'); ?>";
 
-    $(document).ready(function() {
-        $("#bs-popup #add-form").submit(function() {
-            var result = calcolaBilancio();
-
-            if(!result) {
-                setTimeout(function(){ bloccaZeri(); }, 300);
-            }
-
-            return result;
+    $(document).ready(function(e) {
+        $("#bs-popup #add-form").on("submit", function(e) {
+            return calcolaBilancio();
         });
 
         $('#bs-popup #modello_primanota').change(function() {
diff --git a/modules/primanota/edit.php b/modules/primanota/edit.php
index a6644e1e6..a7db782a2 100644
--- a/modules/primanota/edit.php
+++ b/modules/primanota/edit.php
@@ -57,7 +57,7 @@ include_once __DIR__.'/../../core.php';
 	</div>
 <?php
 
-$righe = $prima_nota->movimenti->toArray();
+$righe = $mastrino->movimenti->toArray();
 
 include $structure->filepath('movimenti.php');
 
@@ -66,14 +66,7 @@ include $structure->filepath('movimenti.php');
 
 <script>
     $("#edit-form").submit(function(e) {
-        var result = calcolaBilancio();
-
-        if(!result) {
-            setTimeout(function(){ bloccaZeri(); }, 300);
-            e.preventDefault();
-        }
-
-        return result;
+        return calcolaBilancio();
     });
 </script>
 
diff --git a/modules/primanota/init.php b/modules/primanota/init.php
index 0d77b7169..213ee5846 100644
--- a/modules/primanota/init.php
+++ b/modules/primanota/init.php
@@ -2,10 +2,10 @@
 
 include_once __DIR__.'/../../core.php';
 
-use Modules\PrimaNota\PrimaNota;
+use Modules\PrimaNota\Mastrino;
 
 if (isset($id_record)) {
-    $prima_nota = PrimaNota::find($id_record);
+    $mastrino = Mastrino::find($id_record);
 
     $record = $dbo->fetchOne('SELECT * FROM co_movimenti WHERE idmastrino='.prepare($id_record));
 }
diff --git a/modules/primanota/src/PrimaNota.php b/modules/primanota/src/Mastrino.php
similarity index 99%
rename from modules/primanota/src/PrimaNota.php
rename to modules/primanota/src/Mastrino.php
index ffb5d9a7f..e84cccdbd 100644
--- a/modules/primanota/src/PrimaNota.php
+++ b/modules/primanota/src/Mastrino.php
@@ -6,7 +6,7 @@ use Common\Model;
 use Modules\Fatture\Fattura;
 use Modules\Scadenzario\Scadenza;
 
-class PrimaNota extends Model
+class Mastrino extends Model
 {
     public $incrementing = false;
     protected $table = 'co_movimenti';
diff --git a/modules/primanota/src/Movimento.php b/modules/primanota/src/Movimento.php
index b58638166..c5e09b8bf 100644
--- a/modules/primanota/src/Movimento.php
+++ b/modules/primanota/src/Movimento.php
@@ -16,15 +16,15 @@ class Movimento extends Model
         'dare',
     ];
 
-    public static function build(PrimaNota $prima_nota, $id_conto, Scadenza $scadenza = null)
+    public static function build(Mastrino $mastrino, $id_conto, Scadenza $scadenza = null)
     {
         $model = parent::build();
 
-        $model->idmastrino = $prima_nota->idmastrino;
-        $model->data = $prima_nota->data;
-        $model->descrizione = $prima_nota->descrizione;
-        $model->primanota = $prima_nota->primanota;
-        $model->is_insoluto = $prima_nota->is_insoluto;
+        $model->idmastrino = $mastrino->idmastrino;
+        $model->data = $mastrino->data;
+        $model->descrizione = $mastrino->descrizione;
+        $model->primanota = $mastrino->primanota;
+        $model->is_insoluto = $mastrino->is_insoluto;
 
         $model->id_scadenza = $scadenza ? $scadenza->id : null;