From d9e2c4afd31d678673fea9ec5cd59ff4ccec3487 Mon Sep 17 00:00:00 2001 From: Thomas Zilio Date: Tue, 2 Oct 2018 18:25:52 +0200 Subject: [PATCH] Fix #315 --- core.php | 2 +- modules/anagrafiche/actions.php | 2 +- modules/anagrafiche/src/Anagrafica.php | 4 ++-- modules/ddt/bulk.php | 2 +- modules/fatture/actions.php | 12 ++++++------ modules/fatture/src/Articolo.php | 4 ++-- modules/fatture/src/Descrizione.php | 4 ++-- modules/fatture/src/Fattura.php | 4 ++-- modules/fatture/src/Riga.php | 4 ++-- modules/interventi/actions.php | 2 +- modules/interventi/api/create.php | 2 +- modules/interventi/bulk.php | 2 +- modules/interventi/src/Articolo.php | 4 ++-- modules/interventi/src/Riga.php | 4 ++-- plugins/importPA/src/FatturaElettronica.php | 8 ++++---- src/Base/Description.php | 4 ++-- src/Base/Model.php | 2 +- src/base/Article.php | 4 ++-- src/base/Row.php | 4 ++-- tests/unit/FatturaTest.php | 2 +- 20 files changed, 38 insertions(+), 38 deletions(-) diff --git a/core.php b/core.php index ab3db4082..dd9c4dfd9 100644 --- a/core.php +++ b/core.php @@ -70,7 +70,7 @@ if (!API::isAPIRequest()) { // Impostazioni di debug if (App::debug()) { // Ignora gli avvertimenti e le informazioni relative alla deprecazione di componenti - error_reporting(E_ALL & ~E_WARNING & ~E_NOTICE & ~E_USER_DEPRECATED); + error_reporting(E_ALL & ~E_WARNING & ~E_NOTICE & ~E_USER_DEPRECATED & ~E_STRICT); // File di log ordinato in base alla data $handlers[] = new RotatingFileHandler($docroot.'/logs/error.log', 0, Monolog\Logger::ERROR); diff --git a/modules/anagrafiche/actions.php b/modules/anagrafiche/actions.php index bc747244e..60630f9a7 100644 --- a/modules/anagrafiche/actions.php +++ b/modules/anagrafiche/actions.php @@ -111,7 +111,7 @@ switch (post('op')) { $idtipoanagrafica = post('idtipoanagrafica'); $ragione_sociale = post('ragione_sociale'); - $anagrafica = Anagrafica::new($ragione_sociale, $idtipoanagrafica); + $anagrafica = Anagrafica::make($ragione_sociale, $idtipoanagrafica); $id_record = $anagrafica->id; // Se ad aggiungere un cliente è un agente, lo imposto come agente di quel cliente diff --git a/modules/anagrafiche/src/Anagrafica.php b/modules/anagrafiche/src/Anagrafica.php index 825ea8792..35a0c476e 100644 --- a/modules/anagrafiche/src/Anagrafica.php +++ b/modules/anagrafiche/src/Anagrafica.php @@ -30,9 +30,9 @@ class Anagrafica extends Model * * @return self */ - public static function new($ragione_sociale, array $tipologie = []) + public static function make($ragione_sociale, array $tipologie = []) { - $model = parent::new(); + $model = parent::make(); $model->ragione_sociale = $ragione_sociale; diff --git a/modules/ddt/bulk.php b/modules/ddt/bulk.php index 147b24a16..1317e2e0e 100644 --- a/modules/ddt/bulk.php +++ b/modules/ddt/bulk.php @@ -56,7 +56,7 @@ switch (post('op')) { // Se non c'è già una fattura appena creata per questo cliente, creo una fattura nuova if (empty($id_documento)) { $anagrafica = Anagrafica::find($id_anagrafica); - $fattura = Fattura::new($anagrafica, $tipo_documento, $data, $id_segment); + $fattura = Fattura::make($anagrafica, $tipo_documento, $data, $id_segment); $id_documento = $fattura->id; $id_documento_cliente[$id_anagrafica] = $id_documento; diff --git a/modules/fatture/actions.php b/modules/fatture/actions.php index e6911adca..1f22c5e2e 100644 --- a/modules/fatture/actions.php +++ b/modules/fatture/actions.php @@ -34,7 +34,7 @@ switch (post('op')) { $anagrafica = Anagrafica::find($idanagrafica); $tipo = Tipo::find($idtipodocumento); - $fattura = Fattura::new($anagrafica, $tipo, $data, $id_segment); + $fattura = Fattura::make($anagrafica, $tipo, $data, $id_segment); $id_record = $fattura->id; flash()->info(tr('Aggiunta fattura numero _NUM_!', [ @@ -484,7 +484,7 @@ switch (post('op')) { } $originale = ArticoloOriginale::find(post('idarticolo')); - $articolo = Articolo::new($fattura, $originale); + $articolo = Articolo::make($fattura, $originale); $articolo->descrizione = post('descrizione'); $um = post('um'); @@ -522,7 +522,7 @@ switch (post('op')) { $qta = -$qta; } - $riga = Riga::new($fattura); + $riga = Riga::make($fattura); $riga->descrizione = post('descrizione'); $um = post('um'); @@ -556,7 +556,7 @@ switch (post('op')) { break; case 'add_descrizione': - $riga = Descrizione::new($fattura); + $riga = Descrizione::make($fattura); $riga->descrizione = post('descrizione'); $riga->save(); @@ -1257,7 +1257,7 @@ switch (post('op')) { $anagrafica = $fattura->anagrafica()->first(); $tipo = Tipo::where('descrizione', 'Nota di credito')->where('dir', 'entrata')->first(); - $nota = Fattura::new($anagrafica, $tipo, $data, $id_segment); + $nota = Fattura::make($anagrafica, $tipo, $data, $id_segment); $nota->ref_documento = $fattura->id; $nota->idconto = $fattura->idconto; $nota->idpagamento = $fattura->idpagamento; @@ -1352,7 +1352,7 @@ if (get('op') == 'nota_addebito') { $tipo = Tipo::where('descrizione', 'Nota di debito')->where('dir', 'entrata')->first(); $data = $fattura->data; - $nota = Fattura::new($anagrafica, $tipo, $data, $id_segment); + $nota = Fattura::make($anagrafica, $tipo, $data, $id_segment); $nota->ref_documento = $fattura->id; $nota->idconto = $fattura->idconto; $nota->idpagamento = $fattura->idpagamento; diff --git a/modules/fatture/src/Articolo.php b/modules/fatture/src/Articolo.php index c6347085c..c7f8815a0 100644 --- a/modules/fatture/src/Articolo.php +++ b/modules/fatture/src/Articolo.php @@ -17,9 +17,9 @@ class Articolo extends Article * * @return self */ - public static function new(Fattura $fattura, Original $articolo) + public static function make(Fattura $fattura, Original $articolo) { - $model = parent::new($articolo); + $model = parent::make($articolo); $model->fattura()->associate($fattura); diff --git a/modules/fatture/src/Descrizione.php b/modules/fatture/src/Descrizione.php index 1fbcbf142..098f9b037 100644 --- a/modules/fatture/src/Descrizione.php +++ b/modules/fatture/src/Descrizione.php @@ -15,9 +15,9 @@ class Descrizione extends Description * * @return self */ - public static function new(Fattura $fattura) + public static function make(Fattura $fattura) { - $model = parent::new(); + $model = parent::make(); $model->fattura()->associate($fattura); diff --git a/modules/fatture/src/Fattura.php b/modules/fatture/src/Fattura.php index 1d82a33aa..a8d6e4cc6 100644 --- a/modules/fatture/src/Fattura.php +++ b/modules/fatture/src/Fattura.php @@ -23,9 +23,9 @@ class Fattura extends Model * * @return self */ - public static function new(Anagrafica $anagrafica, Tipo $tipo_documento, $data, $id_segment) + public static function make(Anagrafica $anagrafica, Tipo $tipo_documento, $data, $id_segment) { - $model = parent::new(); + $model = parent::make(); $stato_documento = Stato::where('descrizione', 'Bozza')->first(); diff --git a/modules/fatture/src/Riga.php b/modules/fatture/src/Riga.php index bca25c863..bb58d9c0c 100644 --- a/modules/fatture/src/Riga.php +++ b/modules/fatture/src/Riga.php @@ -15,9 +15,9 @@ class Riga extends Row * * @return self */ - public static function new(Fattura $fattura) + public static function make(Fattura $fattura) { - $model = parent::new(); + $model = parent::make(); $model->fattura()->associate($fattura); diff --git a/modules/interventi/actions.php b/modules/interventi/actions.php index 8b63df8b7..d5494165d 100644 --- a/modules/interventi/actions.php +++ b/modules/interventi/actions.php @@ -522,7 +522,7 @@ switch (post('op')) { case 'addarticolo': $originale = ArticoloOriginale::find(post('idarticolo')); $intervento = Intervento::find($id_record); - $articolo = Articolo::new($intervento, $originale, post('idautomezzo')); + $articolo = Articolo::make($intervento, $originale, post('idautomezzo')); $articolo->qta = post('qta'); $articolo->descrizione = post('descrizione'); diff --git a/modules/interventi/api/create.php b/modules/interventi/api/create.php index b2d558a1e..2a4945768 100644 --- a/modules/interventi/api/create.php +++ b/modules/interventi/api/create.php @@ -63,7 +63,7 @@ switch ($resource) { $originale = ArticoloOriginale::find($data['id_articolo']); $intervento = Intervento::find($data['id_intervento']); - $articolo = Articolo::new($intervento, $originale, $data['id_automezzo']); + $articolo = Articolo::make($intervento, $originale, $data['id_automezzo']); $articolo->qta = $data['qta']; $articolo->um = $data['um']; diff --git a/modules/interventi/bulk.php b/modules/interventi/bulk.php index 178d72729..b59942c0e 100644 --- a/modules/interventi/bulk.php +++ b/modules/interventi/bulk.php @@ -100,7 +100,7 @@ switch (post('op')) { if (empty($id_documento)) { $anagrafica = Anagrafica::find($id_anagrafica); - $fattura = Fattura::new($anagrafica, $tipo_documento, $data, $id_segment); + $fattura = Fattura::make($anagrafica, $tipo_documento, $data, $id_segment); $id_documento = $fattura->id; $id_documento_cliente[$id_anagrafica] = $id_documento; diff --git a/modules/interventi/src/Articolo.php b/modules/interventi/src/Articolo.php index 429a59118..f129495b0 100644 --- a/modules/interventi/src/Articolo.php +++ b/modules/interventi/src/Articolo.php @@ -19,9 +19,9 @@ class Articolo extends Article * * @return self */ - public static function new(Intervento $intervento, Original $articolo, $id_automezzo = null) + public static function make(Intervento $intervento, Original $articolo, $id_automezzo = null) { - $model = parent::new($articolo); + $model = parent::make($articolo); $model->intervento()->associate($intervento); diff --git a/modules/interventi/src/Riga.php b/modules/interventi/src/Riga.php index dfab5963b..a2d2ed225 100644 --- a/modules/interventi/src/Riga.php +++ b/modules/interventi/src/Riga.php @@ -15,9 +15,9 @@ class Riga extends Row * * @return self */ - public static function new(Intervento $intervento) + public static function make(Intervento $intervento) { - $model = parent::new(); + $model = parent::make(); $model->intervento()->associate($intervento); diff --git a/plugins/importPA/src/FatturaElettronica.php b/plugins/importPA/src/FatturaElettronica.php index 9ecddf726..9a05e44af 100644 --- a/plugins/importPA/src/FatturaElettronica.php +++ b/plugins/importPA/src/FatturaElettronica.php @@ -88,7 +88,7 @@ class FatturaElettronica } $ragione_sociale = $xml['DatiAnagrafici']['Anagrafica']['Denominazione'] ?: $xml['DatiAnagrafici']['Anagrafica']['Nome'].' '.$xml['DatiAnagrafici']['Anagrafica']['Cognome']; - $anagrafica = Anagrafica::new($ragione_sociale, [ + $anagrafica = Anagrafica::make($ragione_sociale, [ TipoAnagrafica::where('descrizione', 'Fornitore')->first()->id, ]); @@ -161,9 +161,9 @@ class FatturaElettronica $articolo = ArticoloOriginale::find($articoli[$key]); if (!empty($articolo)) { - $obj = Articolo::new($this->getFattura(), $articolo); + $obj = Articolo::make($this->getFattura(), $articolo); } else { - $obj = Riga::new($this->getFattura()); + $obj = Riga::make($this->getFattura()); } $obj->descrizione = $riga['Descrizione']; @@ -245,7 +245,7 @@ class FatturaElettronica $descrizione_tipo = empty($this->getBody()['DatiGenerali']['DatiTrasporto']) ? 'Fattura immediata di acquisto' : 'Fattura accompagnatoria di acquisto'; $tipo = TipoFattura::where('descrizione', $descrizione_tipo)->first(); - $fattura = Fattura::new($anagrafica, $tipo, $data, $this->id_sezionale); + $fattura = Fattura::make($anagrafica, $tipo, $data, $this->id_sezionale); $this->fattura = $fattura; $fattura->numero = $numero; diff --git a/src/Base/Description.php b/src/Base/Description.php index 73a63c3b2..1d471f31a 100644 --- a/src/Base/Description.php +++ b/src/Base/Description.php @@ -17,9 +17,9 @@ abstract class Description extends Model } } - public static function new($bypass = false) + public static function make($bypass = false) { - $model = parent::new(); + $model = parent::make(); if (!$bypass) { $model->is_descrizione = 1; diff --git a/src/Base/Model.php b/src/Base/Model.php index bd5a6f15c..a48de45fa 100644 --- a/src/Base/Model.php +++ b/src/Base/Model.php @@ -11,7 +11,7 @@ abstract class Model extends Original * * @return static */ - public static function new() + public static function make() { return new static(); } diff --git a/src/base/Article.php b/src/base/Article.php index 37afd2d45..9a9ab7c36 100644 --- a/src/base/Article.php +++ b/src/base/Article.php @@ -18,9 +18,9 @@ abstract class Article extends Row }); } - public static function new(Original $articolo) + public static function make(Original $articolo) { - $model = parent::new(true); + $model = parent::make(true); $model->articolo()->associate($articolo); diff --git a/src/base/Row.php b/src/base/Row.php index f56088055..fcf656839 100644 --- a/src/base/Row.php +++ b/src/base/Row.php @@ -19,9 +19,9 @@ abstract class Row extends Description } } - public static function new($bypass = false) + public static function make($bypass = false) { - return parent::new(true); + return parent::make(true); } public function getTotaleAttribute() diff --git a/tests/unit/FatturaTest.php b/tests/unit/FatturaTest.php index 5d99fa2ce..86128b4c1 100644 --- a/tests/unit/FatturaTest.php +++ b/tests/unit/FatturaTest.php @@ -17,7 +17,7 @@ class FatturaTest extends \Codeception\Test\Unit $anagrafica = Anagrafica::find($id_anagrafica); $tipo = Tipo::find($id_tipo); - $fattura = Fattura::new($anagrafica, $tipo, $data, $id_segment); + $fattura = Fattura::make($anagrafica, $tipo, $data, $id_segment); $this->assertEquals($fattura->idanagrafica, 1); $this->assertEquals($fattura->idtipodocumento, 2);