Fix #415

include/src/Components/Row.php

@@ -212,6 +212,21 @@ abstract class Row extends Description
         return parent::save($options);
     }
 
+    public function aliquota()
+    {
+        return $this->belongsTo(Aliquota::class, 'idiva');
+    }
+
+    public function rivalsa()
+    {
+        return $this->belongsTo(RivalsaINPS::class, 'idrivalsainps');
+    }
+
+    public function ritenuta()
+    {
+        return $this->belongsTo(RitenutaAcconto::class, 'idritenutaacconto');
+    }
+
     protected static function boot($bypass = false)
     {
         parent::boot(true);
@@ -277,19 +292,4 @@ abstract class Row extends Description
     {
         $this->attributes['sconto'] = $this->sconto;
     }
-
-    public function aliquota()
-    {
-        return $this->belongsTo(Aliquota::class, 'idiva');
-    }
-
-    public function rivalsa()
-    {
-        return $this->belongsTo(RivalsaINPS::class, 'idrivalsainps');
-    }
-
-    public function ritenuta()
-    {
-        return $this->belongsTo(RitenutaAcconto::class, 'idritenutaacconto');
-    }
 }

include/src/Document.php

@@ -26,24 +26,6 @@ abstract class Document extends Model
 
     abstract public function scontoGlobale();
 
-    /**
-     * Calcola la somma degli attributi indicati come parametri.
-     * Il metodo **non** deve essere adattato per ulteriori funzionalità: deve esclusivamente calcolare la somma richiesta in modo esplicito dagli argomenti.
-     *
-     * @param mixed ...$args
-     *
-     * @return float
-     */
-    protected function calcola(...$args)
-    {
-        $result = 0;
-        foreach ($args as $arg) {
-            $result += $this->getRigheContabili()->sum($arg);
-        }
-
-        return $this->round($result);
-    }
-
     /**
      * Calcola l'imponibile della fattura.
      *
@@ -154,6 +136,24 @@ abstract class Document extends Model
         return $this->calcola('guadagno');
     }
 
+    /**
+     * Calcola la somma degli attributi indicati come parametri.
+     * Il metodo **non** deve essere adattato per ulteriori funzionalità: deve esclusivamente calcolare la somma richiesta in modo esplicito dagli argomenti.
+     *
+     * @param mixed ...$args
+     *
+     * @return float
+     */
+    protected function calcola(...$args)
+    {
+        $result = 0;
+        foreach ($args as $arg) {
+            $result += $this->getRigheContabili()->sum($arg);
+        }
+
+        return $this->round($result);
+    }
+
     /**
      * Restituisce la collezione di righe e articoli con valori rilevanti per i conti.
      *

modules/anagrafiche/actions.php

@@ -26,9 +26,9 @@ switch (post('op')) {
 
         $sede->save();
 
-		if (!empty(post('nome')) and !empty(post('cognome')) ){
+        if (!empty(post('nome')) and !empty(post('cognome'))) {
             $ragione_sociale = post('nome').' '.post('cognome');
-		}else{
+        } else {
             $ragione_sociale = post('ragione_sociale');
         }
         // Informazioni sull'anagrafica

modules/anagrafiche/add.php

@@ -32,7 +32,6 @@ echo '
 		</div>
 	</div>';
 
-
 echo '
     <div class="box box-info collapsed-box">
 	    <div class="box-header with-border">

modules/anagrafiche/plugins/statistiche.php

@@ -134,10 +134,10 @@ echo '
 				<div class="box-body">';
 // Fatture di vendita
 $totale_fatture_vendita = 0;
-$fatture = database()->fetchArray("SELECT id FROM co_documenti WHERE idanagrafica=".prepare($id_record));
+$fatture = database()->fetchArray('SELECT id FROM co_documenti WHERE idanagrafica='.prepare($id_record));
 
 foreach ($fatture as $fattura) {
-    $totale_fatture_vendita = sum( $totale_fatture_vendita, Modules\Fatture\Fattura::find( $fattura['id'] )->netto );
+    $totale_fatture_vendita = sum($totale_fatture_vendita, Modules\Fatture\Fattura::find($fattura['id'])->netto);
 }
 
 $data_start = strtotime('now');

modules/primanota/actions.php

@@ -92,10 +92,9 @@ switch (post('op')) {
         //Creo il modello di prima nota
 
         if (!empty(post('crea_modello'))) {
-			
+            if (empty(post('idmastrino'))) {
-			if (empty(post('idmastrino'))){
                 $idmastrino = get_new_idmastrino('co_movimenti_modelli');
-			}else{
+            } else {
                 $dbo->query('DELETE FROM co_movimenti_modelli WHERE idmastrino='.prepare(post('idmastrino')));
                 $idmastrino = post('idmastrino');
             }
@@ -105,7 +104,6 @@ switch (post('op')) {
                 $query = 'INSERT INTO co_movimenti_modelli(idmastrino, descrizione, idconto) VALUES('.prepare($idmastrino).', '.prepare($descrizione).', '.prepare($idconto).')';
                 $dbo->query($query);
             }
-			
         }
 
         break;

modules/primanota/add.php

@@ -351,9 +351,11 @@ include_once __DIR__.'/../../core.php';
 					var causale = $(this).find('option:selected').text();
 					
 					//aggiornava erroneamente anche la causale ed eventuale numero di fattura e data
-					<?php if (empty($iddocumento)) {?>
+					<?php if (empty($iddocumento)) {
+        ?>
 						$('#bs-popup #desc').val(causale);
-					<?php } ?>
+					<?php
+    } ?>
 					
 					$.get('<?php echo $rootdir; ?>/ajax_complete.php?op=get_conti&idmastrino='+idmastrino, function(data){
 						var conti = data.split(',');

plugins/exportFE/src/FatturaElettronica.php

@@ -637,8 +637,6 @@ class FatturaElettronica
         return $this->intermediario;
     }
 
-
-
     /**
      * Restituisce le informazioni riguardanti un anagrafica sulla base dell'identificativo fornito.
      *
@@ -1077,7 +1075,6 @@ class FatturaElettronica
         return $result;
     }
 
-
     /**
      * Restituisce l'array responsabile per la generazione del tag TerzoIntermediarioOSoggettoEmittente (1.5).
      *
@@ -1094,7 +1091,6 @@ class FatturaElettronica
         return $result;
     }
 
-
     /**
      * Restituisce l'array responsabile per la generazione del tag DatiGeneraliDocumento.
      *

plugins/importFE/rows.php

@@ -30,8 +30,8 @@ echo '
     <h4>'.
         $ragione_sociale.'<br>
         <small>
-            '.( !empty($codice_fiscale) ? (tr('Codice Fiscale').': '.$codice_fiscale.'<br>') : '' ).'
+            '.(!empty($codice_fiscale) ? (tr('Codice Fiscale').': '.$codice_fiscale.'<br>') : '').'
-            '.( !empty($partita_iva) ? (tr('Partita IVA').': '.$partita_iva.'<br>') : '' ).'
+            '.(!empty($partita_iva) ? (tr('Partita IVA').': '.$partita_iva.'<br>') : '').'
             '.$cap.' '.$citta.' ('.$provincia.')<br>
         </small>
     </h4><br>';
@@ -41,9 +41,14 @@ $pagamenti = $fattura_pa->getBody()['DatiPagamento'];
 
 $metodi = $pagamenti['DettaglioPagamento'];
 $metodi = isset($metodi[0]) ? $metodi : [$metodi];
+$codice_modalita_pagamento = $metodi[0]['ModalitaPagamento'];
 
 // prc '.($pagamenti['CondizioniPagamento'] == 'TP01' ? '!' : '').'= 100 AND
-$query = 'SELECT id, descrizione FROM co_pagamenti WHERE codice_modalita_pagamento_fe = '.prepare($metodi[0]['ModalitaPagamento']).' GROUP BY descrizione ORDER BY descrizione ASC';
+$query = 'SELECT id, descrizione FROM co_pagamenti';
+if (!empty($codice_modalita_pagamento)) {
+    $query .= ' WHERE codice_modalita_pagamento_fe = '.prepare($codice_modalita_pagamento);
+}
+$query .= ' GROUP BY descrizione ORDER BY descrizione ASC';
 
 echo '
     <h4>'.tr('Pagamento').'</h4>

src/HTMLBuilder/Manager/FileManager.php

@@ -112,7 +112,7 @@ class FileManager implements ManagerInterface
                 <button class="btn btn-xs btn-info" data-target="#bs-popup2" type="button" data-title="'.prepareToField($r['name']).' <small><em>('.$r['filename'].')</em></small>" data-href="#view-'.$r['id'].'">
                     <i class="fa fa-eye"></i>
                 </button>';
-                    } elseif (strtolower($extension) == 'xml' ) {
+                    } elseif (strtolower($extension) == 'xml') {
                         $result .= '
                         <a class="btn btn-xs btn-info" href="'.ROOTDIR.'/plugins/exportFE/view.php?id_record='.$r['id'].'" target="_blank">
                             <i class="fa fa-eye"></i>