Fix #785 e stile del codice

core.php

@@ -3,7 +3,7 @@
 // Rimozione header X-Powered-By
 header_remove('X-Powered-By');
 
-ini_set ('session.cookie_samesite', 'strict');
+ini_set('session.cookie_samesite', 'strict');
 
 // Impostazioni di configurazione PHP
 date_default_timezone_set('Europe/Rome');

modules/articoli/actions.php

@@ -154,6 +154,20 @@ switch (post('op')) {
         $new->qta = 0;
         $new->save();
 
+        // Copia degli allegati
+        $allegati = $articolo->uploads();
+        foreach ($allegati as $allegato) {
+            $allegato->copia([
+                'id_module' => $new->getModule()->id,
+                'id_record' => $new->id,
+            ]);
+        }
+
+        // Salvataggio immagine relativa
+        $nome_immagine = $articolo->immagine_upload->name;
+        $new->immagine = $new->uploads()->where('name', $nome_immagine)->first()->filename;
+        $new->save();
+
         $id_record = $new->id;
 
         flash()->info(tr('Articolo duplicato correttamente!'));

modules/articoli/src/Articolo.php

@@ -6,11 +6,13 @@ use Common\Model;
 use Illuminate\Database\Eloquent\SoftDeletes;
 use Modules;
 use Modules\Interventi\Components\Articolo as ArticoloIntervento;
+use Traits\RecordTrait;
 use Uploads;
 
 class Articolo extends Model
 {
     use SoftDeletes;
+    use RecordTrait;
 
     protected $table = 'mg_articoli';
 
@@ -93,6 +95,15 @@ class Articolo extends Model
 
     // Attributi Eloquent
 
+    public function getImmagineUploadAttribute()
+    {
+        if (empty($this->immagine)) {
+            return null;
+        }
+
+        return $this->uploads()->where('filename', $this->immagine)->first();
+    }
+
     public function getImageAttribute()
     {
         if (empty($this->immagine)) {

modules/articoli/widgets/stampa_inventario.php

@@ -2,8 +2,6 @@
 
 include_once __DIR__.'/../../../core.php';
 
-use Carbon\Carbon;
-
 // Trovo id_print della stampa
 $id_print = $dbo->fetchOne('SELECT zz_prints.id FROM zz_prints INNER JOIN zz_modules ON zz_prints.id_module=zz_modules.id WHERE zz_modules.name="Articoli" AND zz_prints.name="Inventario magazzino"')['id'];
 

modules/contratti/widgets/contratti.ratecontrattuali.php

@@ -99,7 +99,6 @@ if (!empty($rsp)) {
         // $importo = ($r['budget_contratto'] * $n_sedi_pianificate / $n_rate[ $r['idcontratto'] ][ $r['idzona'] ]) - ($gia_fatturato[ $r['idcontratto'] ][ $r['idzona'] ] * $n_sedi_pianificate / sizeof($gia_fatturato[ $r['idcontratto'] ][ $r['idzona'] ]) );
         $importo = ($r['budget_contratto'] * $n_sedi_pianificate / $n_rate[$r['idcontratto']][$r['idzona']]);
 
-
         // Visualizzo solo le rate non pagate
         if ($r['iddocumento'] == 0) {
             echo "

modules/preventivi/edit.php

@@ -15,7 +15,7 @@ if (count($preventivo->revisioni) > 1) {
         <i class="fa fa-info-circle"></i> 
         '.tr('Questo preventivo presenta _N_ revisioni',
         [
-            '_N_' => count($preventivo->revisioni)
+            '_N_' => count($preventivo->revisioni),
         ]).'
     </div>
     ';
@@ -35,7 +35,7 @@ if (count($preventivo->revisioni) > 1) {
 		<div class="panel-body">
 			<div class="row">
 				<div class="col-md-3">
-					{[ "type": "text", "label": "<?php echo tr('Numero'); ?>", "name": "numero", "required": 1, "class": "text-center", "value": "$numero$", "icon-after": "<?php echo (count($preventivo->revisioni) > 1) ? tr('rev.').' '.$preventivo->numero_revision : '' ?>" ]}
+					{[ "type": "text", "label": "<?php echo tr('Numero'); ?>", "name": "numero", "required": 1, "class": "text-center", "value": "$numero$", "icon-after": "<?php echo (count($preventivo->revisioni) > 1) ? tr('rev.').' '.$preventivo->numero_revision : ''; ?>" ]}
 				</div>
 
                 <div class="col-md-3">

modules/preventivi/src/Preventivo.php

@@ -281,5 +281,4 @@ class Preventivo extends Document
     {
         $this->idstato = Stato::where('descrizione', $stato)->first()['id'];
     }
-
 }

templates/magazzino_inventario/body.php

@@ -9,9 +9,9 @@ $id_module = Modules::get('Articoli')['id'];
 // Valori di ricerca
 $where['servizio'] = '0';
 
-foreach( $_SESSION['module_'.$id_module] as $name => $value ){
+foreach ($_SESSION['module_'.$id_module] as $name => $value) {
-    if( preg_match('/^search_(.+?)$/', $name, $m) ){
+    if (preg_match('/^search_(.+?)$/', $name, $m)) {
-        $where[ $m[1] ] = $value;
+        $where[$m[1]] = $value;
     }
 }
 

templates/partitario_mastrino/pdfgen.partitario_mastrino.php

@@ -170,7 +170,7 @@ elseif (get('lev') == '1') {
             // Ciclo fra i sotto-conti di livello 2
             $rs3 = $dbo->fetchArray('SELECT id, numero, descrizione, movimenti.totale FROM co_pianodeiconti3 LEFT JOIN (SELECT SUM(totale) AS totale, idconto FROM co_movimenti GROUP BY idconto) AS movimenti ON co_pianodeiconti3.id=movimenti.idconto WHERE idpianodeiconti2="'.$rs2[$y]['id'].'" AND movimenti.totale != 0');
 
-            if( !empty($rs3)){
+            if (!empty($rs3)) {
                 $body .= "		<tr><th class='bb padded' colspan='4'><b>".$rs2[$y]['numero'].' '.$rs2[$y]['descrizione']."</b></th></tr>\n";
             }