From a23aab9e56680df4179024d2713f9dd1a6a76f64 Mon Sep 17 00:00:00 2001 From: Thomas Zilio Date: Thu, 4 Oct 2018 17:41:31 +0200 Subject: [PATCH] Fix dello stile del codice --- core.php | 2 +- include/common/articolo.php | 1 - include/init/requirements.php | 2 +- lib/functions.php | 1 - lib/util.php | 1 - modules/articoli/ajax/select.php | 31 +++++++++---------- modules/fatture/actions.php | 2 +- modules/fatture/edit.php | 3 +- modules/fatture/src/Articolo.php | 2 +- modules/fatture/src/Fattura.php | 1 - modules/interventi/modutil.php | 2 +- modules/ordini/variables.php | 2 +- src/HTMLBuilder/Manager/EmailManager.php | 1 - src/Mail.php | 2 +- src/Models/Module.php | 1 - src/base/Article.php | 5 +-- .../pdfgen.riepilogo_interventi.php | 1 - 17 files changed, 25 insertions(+), 35 deletions(-) diff --git a/core.php b/core.php index dd9c4dfd9..61640a138 100644 --- a/core.php +++ b/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 diff --git a/include/common/articolo.php b/include/common/articolo.php index 873e5d146..e6e77f4ce 100644 --- a/include/common/articolo.php +++ b/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 ' diff --git a/include/init/requirements.php b/include/init/requirements.php index ef4d6fba8..88ace39a5 100644 --- a/include/init/requirements.php +++ b/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', diff --git a/lib/functions.php b/lib/functions.php index cf5227dea..32c194e7b 100644 --- a/lib/functions.php +++ b/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); diff --git a/lib/util.php b/lib/util.php index d8a13116c..767df2828 100644 --- a/lib/util.php +++ b/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. diff --git a/modules/articoli/ajax/select.php b/modules/articoli/ajax/select.php index 43a64a760..c3ce12c09 100644 --- a/modules/articoli/ajax/select.php +++ b/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 - if (!empty($superselect['dir']) && $superselect['dir'] == 'entrata') { - if (!empty($superselect['idanagrafica'])) { - $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']; - } - } - - //in alternativa, per tutti gli altri casi, prendo quella da impostazioni - if (empty($idiva_predefinita)){ - $idiva_predefinita = get_var('Iva predefinita'); - $rs = $dbo->fetchArray("SELECT descrizione FROM co_iva WHERE id='".$idiva_predefinita."'"); - $iva_predefinita = $rs[0]['descrizione']; - } - - + //per le vendite leggo iva predefinita da anagrafica, se settata + if (!empty($superselect['dir']) && $superselect['dir'] == 'entrata') { + if (!empty($superselect['idanagrafica'])) { + $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']; + } + } + + //in alternativa, per tutti gli altri casi, prendo quella da impostazioni + if (empty($idiva_predefinita)) { + $idiva_predefinita = get_var('Iva predefinita'); + $rs = $dbo->fetchArray("SELECT descrizione FROM co_iva WHERE id='".$idiva_predefinita."'"); + $iva_predefinita = $rs[0]['descrizione']; + } foreach ($elements as $element) { $filter[] = 'mg_articoli.id='.prepare($element); diff --git a/modules/fatture/actions.php b/modules/fatture/actions.php index 27e84c484..dfa6a1e6b 100644 --- a/modules/fatture/actions.php +++ b/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!')); } diff --git a/modules/fatture/edit.php b/modules/fatture/edit.php index 9c1d5ae5a..7a3ec68ea 100644 --- a/modules/fatture/edit.php +++ b/modules/fatture/edit.php @@ -184,8 +184,7 @@ if ($dir == 'uscita') { 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)); diff --git a/modules/fatture/src/Articolo.php b/modules/fatture/src/Articolo.php index 25e74ada9..820ea5301 100644 --- a/modules/fatture/src/Articolo.php +++ b/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; diff --git a/modules/fatture/src/Fattura.php b/modules/fatture/src/Fattura.php index a8d6e4cc6..d7730817f 100644 --- a/modules/fatture/src/Fattura.php +++ b/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) { diff --git a/modules/interventi/modutil.php b/modules/interventi/modutil.php index 2592c0132..3f3f14d74 100644 --- a/modules/interventi/modutil.php +++ b/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)); diff --git a/modules/ordini/variables.php b/modules/ordini/variables.php index 0e18543cb..6d9af8947 100644 --- a/modules/ordini/variables.php +++ b/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']), diff --git a/src/HTMLBuilder/Manager/EmailManager.php b/src/HTMLBuilder/Manager/EmailManager.php index 795ed41e5..877837188 100644 --- a/src/HTMLBuilder/Manager/EmailManager.php +++ b/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']; } diff --git a/src/Mail.php b/src/Mail.php index cc9cef57e..62254efa9 100644 --- a/src/Mail.php +++ b/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; }; diff --git a/src/Models/Module.php b/src/Models/Module.php index 695f78ece..c57492128 100644 --- a/src/Models/Module.php +++ b/src/Models/Module.php @@ -2,7 +2,6 @@ namespace Models; -use App; use Auth; use Traits\RecordTrait; use Traits\UploadTrait; diff --git a/src/base/Article.php b/src/base/Article.php index ef8a17c91..c1e918fbf 100644 --- a/src/base/Article.php +++ b/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) { diff --git a/templates/riepilogo_interventi/pdfgen.riepilogo_interventi.php b/templates/riepilogo_interventi/pdfgen.riepilogo_interventi.php index eba4aaa5d..6d023d649 100644 --- a/templates/riepilogo_interventi/pdfgen.riepilogo_interventi.php +++ b/templates/riepilogo_interventi/pdfgen.riepilogo_interventi.php @@ -115,7 +115,6 @@ foreach ($interventi as $intervento) { array_push($idinterventi, "'".$intervento['id']."'"); } - $body .= 'RIEPILOGO INTERVENTI DAL '.Translator::dateToLocale($_SESSION['period_start']).' al '.Translator::dateToLocale($_SESSION['period_end'])."

\n"; // Sostituisco i valori tra | | con il valore del campo del db