From a0d8aee48eb92addd7a4af47e1e0f480dec4c176 Mon Sep 17 00:00:00 2001
From: Thomas Zilio <Dasc3er@users.noreply.github.com>
Date: Sun, 18 Feb 2018 19:35:33 +0100
Subject: [PATCH] Cleanup

---
 core.php                         | 14 ----------
 include/top.php                  | 14 +++++++++-
 modules/contratti/add_riga.php   | 44 ++------------------------------
 modules/ordini/add_riga.php      |  2 +-
 modules/preventivi/edit_riga.php |  6 ++---
 5 files changed, 19 insertions(+), 61 deletions(-)

diff --git a/core.php b/core.php
index eca370414..74009a14b 100644
--- a/core.php
+++ b/core.php
@@ -178,20 +178,6 @@ if (!API::isAPIRequest()) {
     $jscript_modules = $assets['js'];
 
     if ($continue) {
-        // Istanziamento della barra di debug
-        if (!empty($debug)) {
-            $debugbar = new DebugBar\DebugBar();
-
-            $debugbar->addCollector(new DebugBar\DataCollector\MemoryCollector());
-            $debugbar->addCollector(new DebugBar\DataCollector\PhpInfoCollector());
-
-            $debugbar->addCollector(new DebugBar\DataCollector\RequestDataCollector());
-            $debugbar->addCollector(new DebugBar\DataCollector\TimeDataCollector());
-
-            $debugbar->addCollector(new DebugBar\Bridge\MonologCollector($logger));
-            $debugbar->addCollector(new DebugBar\DataCollector\PDO\PDOCollector($dbo->getPDO()));
-        }
-
         $id_module = filter('id_module');
         $id_record = filter('id_record');
         $id_plugin = filter('id_plugin');
diff --git a/include/top.php b/include/top.php
index c8c5a0f32..0d32fedab 100644
--- a/include/top.php
+++ b/include/top.php
@@ -4,7 +4,19 @@ include_once __DIR__.'/../core.php';
 
 $paths = App::getPaths();
 
-if (!empty($debugbar)) {
+// Istanziamento della barra di debug
+if (!empty($debug)) {
+    $debugbar = new DebugBar\DebugBar();
+
+    $debugbar->addCollector(new DebugBar\DataCollector\MemoryCollector());
+    $debugbar->addCollector(new DebugBar\DataCollector\PhpInfoCollector());
+
+    $debugbar->addCollector(new DebugBar\DataCollector\RequestDataCollector());
+    $debugbar->addCollector(new DebugBar\DataCollector\TimeDataCollector());
+
+    $debugbar->addCollector(new DebugBar\Bridge\MonologCollector($logger));
+    $debugbar->addCollector(new DebugBar\DataCollector\PDO\PDOCollector($dbo->getPDO()));
+
     $debugbarRenderer = $debugbar->getJavascriptRenderer();
     $debugbarRenderer->setIncludeVendors(false);
     $debugbarRenderer->setBaseUrl($paths['assets'].'/php-debugbar');
diff --git a/modules/contratti/add_riga.php b/modules/contratti/add_riga.php
index 287e6e615..193cedb0a 100644
--- a/modules/contratti/add_riga.php
+++ b/modules/contratti/add_riga.php
@@ -54,7 +54,7 @@ echo '
     <input type="hidden" name="backto" value="record-edit">';
 
 // Elenco articoli raggruppati per gruppi e sottogruppi
-if($rsr[0]['is_descrizione']!=1){
+if (empty($rsr[0]['is_descrizione'])) {
     echo '
     <div class="col-md-12">
         {[ "type": "select", "label": "'.tr('Articolo').'", "name": "idarticolo", "value": "'.$idarticolo.'", "ajax-source": "articoli" ]}
@@ -67,7 +67,7 @@ echo '
         {[ "type": "textarea", "label": "'.tr('Descrizione').'", "name": "descrizione", "id": "descrizione_riga", "value": '.json_encode($descrizione).', "required": 1 ]}
     </div>';
 
-if($rsr[0]['is_descrizione']!=1){
+if (empty($rsr[0]['is_descrizione'])) {
     // Iva
     echo '
     <div class="col-md-4">
@@ -86,46 +86,6 @@ if($rsr[0]['is_descrizione']!=1){
         {[ "type": "select", "label": "'.tr('Unità di misura').'", "icon-after": "add|'.Modules::get('Unità di misura')['id'].'", "name": "um", "value": "'.$um.'", "ajax-source": "misure" ]}
     </div>';
 
-    /*
-    if (!empty($idriga)) {
-    //Rivalsa INPS
-    if( get_var("Percentuale rivalsa INPS") != "" ){
-    echo "		<div class='col-md-3'>\n";
-    echo "			<label>Rivalsa INPS:</label>\n";
-    echo "			<select id='idrivalsainps' class='superselect' name=\"idrivalsainps\">\n";
-    echo "				<option value=''>-</option>\n";
-
-    $query = "SELECT * FROM co_rivalsainps";
-    $rs = $dbo->fetchArray($query);
-    for( $i=0; $i<sizeof($rs); $i++ ){
-        ( $rs[$i]['id'] == $rsr[$i]['idrivalsainps'] ) ? $attr='selected="true"' : $attr='';
-        echo "				<option value='".$rs[$i]['id']."' ".$attr.">".$rs[$i]['descrizione']."</option>\n";
-    }
-
-    echo "			</select>\n";
-    echo "		</div>\n";
-    }
-
-    //Ritenuta d'acconto
-    if( get_var("Percentuale ritenuta d'acconto") != "" ){
-    echo "		<div class='col-md-3'>\n";
-    echo "			<label>Ritenuta d'acconto:</label>\n";
-    echo "			<select id='idritenutaacconto' class='superselect' name=\"idritenutaacconto\">\n";
-    echo "				<option value=''>-</option>\n";
-
-    $query = "SELECT * FROM co_ritenutaacconto";
-    $rs = $dbo->fetchArray($query);
-    for( $i=0; $i<sizeof($rs); $i++ ){
-        ( $rs[$i]['id'] == $rsr[$i]['idritenutaacconto'] ) ? $attr='selected="true"' : $attr='';
-        echo "				<option value='".$rs[$i]['id']."' ".$attr.">".$rs[$i]['descrizione']."</option>\n";
-    }
-
-    echo "			</select>\n";
-    echo "		</div>\n";
-    }
-    }
-    */
-
     // Costo unitario
     echo '
     <div class="col-md-6">
diff --git a/modules/ordini/add_riga.php b/modules/ordini/add_riga.php
index 558b23137..669c17644 100644
--- a/modules/ordini/add_riga.php
+++ b/modules/ordini/add_riga.php
@@ -79,7 +79,7 @@ echo '
         </div>
     </div>';
 
-if($rsr[0]['is_descrizione']!=1){
+if(empty($rsr[0]['is_descrizione'])){
     // Iva
     echo '
     <div class="row">
diff --git a/modules/preventivi/edit_riga.php b/modules/preventivi/edit_riga.php
index 1043092ec..e17b2a845 100644
--- a/modules/preventivi/edit_riga.php
+++ b/modules/preventivi/edit_riga.php
@@ -61,7 +61,7 @@ echo '
     <input type="hidden" name="idriga" value="'.$idriga.'">
     <input type="hidden" name="backto" value="record-edit">';
 
-if($rsr[0]['is_descrizione']!=1){
+if(empty($rsr[0]['is_descrizione'])){
     // Elenco articoli raggruppati per gruppi e sottogruppi
     echo '
     <div class="row">
@@ -79,7 +79,7 @@ echo '
         </div>
     </div>';
 
-if($rsr[0]['is_descrizione']!=1){
+if(empty($rsr[0]['is_descrizione'])){
     // Quantità
     echo '
     <div class="row">
@@ -113,7 +113,7 @@ if($rsr[0]['is_descrizione']!=1){
             {[ "type": "number", "label": "'.tr('Sconto unitario').'", "name": "sconto", "value": "'.$sconto.'", "icon-after": "choice|untprc|'.$tipo_sconto.'" ]}
         </div>
     </div>';
-    
+
 }
 
 echo '