Fix dello stile del codice

core.php

@@ -96,7 +96,7 @@ if (!API::isAPIRequest()) {
 // Disabilita la segnalazione degli errori (se il debug è disabilitato)
 if (!App::debug()) {
     error_reporting(0);
-    ini_set("display_errors", 0);
+    ini_set('display_errors', 0);
 }
 
 // Imposta il formato di salvataggio dei log

include/common/articolo.php

@@ -6,7 +6,6 @@ unset($_SESSION['superselect']['dir']);
 unset($_SESSION['superselect']['idanagrafica']);
 unset($_SESSION['superselect']['idarticolo']);
 
-
 // Articolo
 if (!isset($options['edit_articolo']) || !empty($options['edit_articolo'])) {
     echo '

include/init/requirements.php

@@ -50,7 +50,7 @@ $settings = [
     ],
     'dom' => [
         'type' => 'ext',
-        'description' => tr("Permette la gestione dei file standard per la Fatturazione Elettronica"),
+        'description' => tr('Permette la gestione dei file standard per la Fatturazione Elettronica'),
     ],
     'soap' => [
         'type' => 'ext',

lib/functions.php

@@ -215,7 +215,6 @@ function translateTemplate()
     $id_module = Modules::getCurrent()['id'];
     $id_plugin = Plugins::getCurrent()['id'];
 
-
     $template = ob_get_clean();
 
     $template = str_replace('$id_module$', $id_module, $template);

lib/util.php

@@ -45,7 +45,6 @@ if (!function_exists('array_clean')) {
     }
 }
 
-
 if (!function_exists('starts_with')) {
     /**
      * Check if a string starts with the given string.

modules/articoli/ajax/select.php

@@ -6,23 +6,20 @@ switch ($resource) {
     case 'articoli':
         $query = 'SELECT mg_articoli.*, (SELECT CONCAT(co_pianodeiconti2.numero, ".", co_pianodeiconti3.numero, " ", co_pianodeiconti3.descrizione) FROM co_pianodeiconti3 INNER JOIN co_pianodeiconti2 ON co_pianodeiconti3.idpianodeiconti2=co_pianodeiconti2.id WHERE co_pianodeiconti3.id = idconto_vendita) AS idconto_vendita_title, (SELECT CONCAT(co_pianodeiconti2.numero, ".", co_pianodeiconti3.numero, " ", co_pianodeiconti3.descrizione) FROM co_pianodeiconti3 INNER JOIN co_pianodeiconti2 ON co_pianodeiconti3.idpianodeiconti2=co_pianodeiconti2.id WHERE co_pianodeiconti3.id = idconto_acquisto) AS idconto_acquisto_title, co_iva.descrizione AS iva_vendita FROM mg_articoli LEFT OUTER JOIN co_iva ON mg_articoli.idiva_vendita=co_iva.id |where| ORDER BY mg_articoli.id_categoria ASC, mg_articoli.id_sottocategoria ASC';
 
- 
+        //per le vendite leggo iva predefinita da anagrafica, se settata
-		//per le vendite leggo iva predefinita da anagrafica, se settata
+         if (!empty($superselect['dir']) && $superselect['dir'] == 'entrata') {
-		 if (!empty($superselect['dir']) && $superselect['dir'] == 'entrata') {
+             if (!empty($superselect['idanagrafica'])) {
-			 if (!empty($superselect['idanagrafica'])) {
+                 $idiva_predefinita = $dbo->fetchArray("SELECT idiva_vendite FROM an_anagrafiche WHERE idanagrafica = '".$superselect['idanagrafica']."' ")[0]['idiva_vendite'];
-				$idiva_predefinita = $dbo->fetchArray("SELECT idiva_vendite FROM an_anagrafiche WHERE idanagrafica = '".$superselect['idanagrafica']."' ")[0]['idiva_vendite'];
+                 $iva_predefinita = $dbo->fetchArray("SELECT descrizione FROM co_iva WHERE id = '".$superselect['idiva_vendita']."' ")[0]['descrizione'];
-				$iva_predefinita = $dbo->fetchArray("SELECT descrizione FROM co_iva WHERE id = '".$superselect['idiva_vendita']."' ")[0]['descrizione'];
+             }
-			 }
+         }
-		 }
+
-		 
+         //in alternativa, per tutti gli altri casi, prendo quella da impostazioni
-		 //in alternativa, per tutti gli altri casi, prendo quella da impostazioni
+          if (empty($idiva_predefinita)) {
-		  if (empty($idiva_predefinita)){
+              $idiva_predefinita = get_var('Iva predefinita');
-			$idiva_predefinita = get_var('Iva predefinita');
+              $rs = $dbo->fetchArray("SELECT descrizione FROM co_iva WHERE id='".$idiva_predefinita."'");
-			$rs = $dbo->fetchArray("SELECT descrizione FROM co_iva WHERE id='".$idiva_predefinita."'");
+              $iva_predefinita = $rs[0]['descrizione'];
-			$iva_predefinita = $rs[0]['descrizione'];
+          }
-		}
-		
-		
 
         foreach ($elements as $element) {
             $filter[] = 'mg_articoli.id='.prepare($element);

modules/fatture/actions.php

@@ -513,7 +513,7 @@ switch (post('op')) {
 
         try {
             $articolo->qta = $qta;
-        } catch(UnexpectedValueException $e) {
+        } catch (UnexpectedValueException $e) {
             flash()->error(tr('Alcuni serial number sono già stati utilizzati!'));
         }
 

modules/fatture/edit.php

@@ -184,8 +184,7 @@ if ($dir == 'uscita') {
 <?php
 
 if (!empty($record['is_fiscale'])) {
-
+    // Aggiunta prima nota solo se non c'è già, se non si è in bozza o se il pagamento non è completo
-// Aggiunta prima nota solo se non c'è già, se non si è in bozza o se il pagamento non è completo
     $n2 = $dbo->fetchNum('SELECT id FROM co_movimenti WHERE iddocumento='.prepare($id_record).' AND primanota=1');
 
     $rs3 = $dbo->fetchArray('SELECT SUM(da_pagare-pagato) AS differenza, SUM(da_pagare) FROM co_scadenziario GROUP BY iddocumento HAVING iddocumento='.prepare($id_record));

modules/fatture/src/Articolo.php

@@ -42,7 +42,7 @@ class Articolo extends Article
         $data = $fattura->data;
 
         $carico = ($tipo->dir == 'entrata') ? tr('Ripristino articolo da _TYPE_ _NUM_') : tr('Carico magazzino da _TYPE_ numero _NUM_');
-        $scarico = ($tipo->dir == 'entrata') ? tr('Scarico magazzino per _TYPE_ numero _NUM_') : tr('Rimozione articolo da _TYPE_ _NUM_') ;
+        $scarico = ($tipo->dir == 'entrata') ? tr('Scarico magazzino per _TYPE_ numero _NUM_') : tr('Rimozione articolo da _TYPE_ _NUM_');
 
         $qta = ($tipo->dir == 'uscita') ? -$qta : $qta;
         $movimento = ($qta < 0) ? $carico : $scarico;

modules/fatture/src/Fattura.php

@@ -93,7 +93,6 @@ class Fattura extends Model
      * **Attenzione**: la data deve inserita prima!
      *
      * @param [type] $value
-     * @return void
      */
     public function setIdSegmentAttribute($value)
     {

modules/interventi/modutil.php

@@ -358,7 +358,7 @@ function aggiungi_intervento_in_fattura($id_intervento, $id_fattura, $descrizion
 
     $id_rivalsa_inps = $id_rivalsa_inps !== false ? $id_rivalsa_inps : setting('Percentuale rivalsa INPS');
     $id_ritenuta_acconto = $id_ritenuta_acconto !== false ? $id_ritenuta_acconto : setting("Percentuale ritenuta d'acconto");
-    $calcolo_ritenuta_acconto = $calcolo_ritenuta_acconto !== false ? $calcolo_ritenuta_acconto :  setting("Metodologia calcolo ritenuta d'acconto predefinito");
+    $calcolo_ritenuta_acconto = $calcolo_ritenuta_acconto !== false ? $calcolo_ritenuta_acconto : setting("Metodologia calcolo ritenuta d'acconto predefinito");
 
     // Leggo l'anagrafica del cliente
     $rs = $dbo->fetchArray('SELECT idanagrafica, codice, (SELECT MIN(orario_inizio) FROM in_interventi_tecnici WHERE idintervento='.prepare($id_intervento).') AS data FROM `in_interventi` WHERE id='.prepare($id_intervento));

modules/ordini/variables.php

@@ -7,7 +7,7 @@ FROM or_ordini WHERE id='.prepare($id_record));
 // Variabili da sostituire
 return [
     'email' => $r['email'],
-	'id_anagrafica' => $r['idanagrafica'],
+    'id_anagrafica' => $r['idanagrafica'],
     'numero' => empty($r['numero_esterno']) ? $r['numero'] : $r['numero_esterno'],
     'note' => $r['note'],
     'data' => Translator::dateToLocale($r['data']),

src/HTMLBuilder/Manager/EmailManager.php

@@ -59,7 +59,6 @@ class EmailManager implements ManagerInterface
             $attachments = [];
             foreach ($options['attachments'] as $attachment) {
                 $attachment = $database->selectOne('zz_files', '*', ['id' => $attachment]);
-                ;
 
                 $attachments[] = $attachment['name'];
             }

src/Mail.php

@@ -161,7 +161,7 @@ class Mail extends PHPMailer\PHPMailer\PHPMailer
             $this->IsSMTP(true);
 
             // Impostazioni di debug
-            $this->SMTPDebug = App::debug() ? 2: 0;
+            $this->SMTPDebug = App::debug() ? 2 : 0;
             $this->Debugoutput = function ($str, $level) {
                 $this->infos[] = $str;
             };

src/Models/Module.php

@@ -2,7 +2,6 @@
 
 namespace Models;
 
-use App;
 use Auth;
 use Traits\RecordTrait;
 use Traits\UploadTrait;

src/base/Article.php

@@ -33,6 +33,7 @@ abstract class Article extends Row
     }
 
     abstract public function movimenta($qta);
+
     abstract public function getDirection();
 
     /**
@@ -83,7 +84,7 @@ abstract class Article extends Row
     {
         // Se la nuova quantità è minore della precedente
         if ($this->qta > $new_qta) {
-            $seriali_usati  = $this->usedSerials();
+            $seriali_usati = $this->usedSerials();
             $count_seriali_usati = count($seriali_usati);
 
             // Controllo sulla possibilità di rimuovere i seriali (se non utilizzati da documenti di vendita)
@@ -110,7 +111,7 @@ abstract class Article extends Row
     /**
      * Modifica la quantità dell'articolo e movimenta automaticamente il magazzino.
      *
-     * @param double $value
+     * @param float $value
      */
     public function setQtaAttribute($value)
     {

templates/riepilogo_interventi/pdfgen.riepilogo_interventi.php

@@ -115,7 +115,6 @@ foreach ($interventi as $intervento) {
     array_push($idinterventi, "'".$intervento['id']."'");
 }
 
-
 $body .= '<big><big><b>RIEPILOGO INTERVENTI DAL '.Translator::dateToLocale($_SESSION['period_start']).' al '.Translator::dateToLocale($_SESSION['period_end'])."</b></big></big><br/><br/>\n";
 
 // Sostituisco i valori tra | | con il valore del campo del db