diff --git a/editor.php b/editor.php index 2630c0f69..2cf47276b 100755 --- a/editor.php +++ b/editor.php @@ -106,7 +106,7 @@ if (empty($record) || !$has_access) {
'; - if(!empty($record['deleted_at'])){ + if (!empty($record['deleted_at'])) { echo '

'.tr('Ripristinare il record?').'

diff --git a/modules/anagrafiche/actions.php b/modules/anagrafiche/actions.php index a2733483f..70dd760d2 100644 --- a/modules/anagrafiche/actions.php +++ b/modules/anagrafiche/actions.php @@ -8,6 +8,7 @@ switch (post('op')) { case 'restore': $anagrafica->restore(); + // no break case 'update': // Informazioni sulla sede $sede = $anagrafica->sedeLegale; diff --git a/modules/anagrafiche/src/Anagrafica.php b/modules/anagrafiche/src/Anagrafica.php index 3fa20b8b3..f67211615 100644 --- a/modules/anagrafiche/src/Anagrafica.php +++ b/modules/anagrafiche/src/Anagrafica.php @@ -3,11 +3,11 @@ namespace Modules\Anagrafiche; use Common\Model; +use Illuminate\Database\Eloquent\SoftDeletes; use Modules\Fatture\Fattura; use Settings; use Traits\RecordTrait; use Util\Generator; -use Illuminate\Database\Eloquent\SoftDeletes; class Anagrafica extends Model { @@ -56,7 +56,8 @@ class Anagrafica extends Model return $model; } - public static function fromTipo($type){ + public static function fromTipo($type) + { $tipologia = Tipo::where('descrizione', 'Tecnico')->first(); $anagrafiche = self::whereHas('tipi', function ($query) use ($tipologia) { @@ -119,9 +120,9 @@ class Anagrafica extends Model $query = 'INSERT INTO in_tariffe(idtecnico, idtipointervento, costo_ore, costo_km, costo_dirittochiamata, costo_ore_tecnico, costo_km_tecnico, costo_dirittochiamata_tecnico) SELECT '.prepare($anagrafica->id).', idtipointervento, costo_orario, costo_km, costo_diritto_chiamata, costo_orario_tecnico, costo_km_tecnico, costo_diritto_chiamata_tecnico FROM in_tipiintervento'; $presenti = $database->fetchArray('SELECT idtipointervento AS id FROM in_tariffe WHERE idtecnico = '.prepare($anagrafica->id)); - if(!empty($presenti)){ + if (!empty($presenti)) { $list = []; - foreach ($presenti as $presente){ + foreach ($presenti as $presente) { $list[] = prepare($presente['id']); } diff --git a/modules/articoli/ajax/select.php b/modules/articoli/ajax/select.php index 23ee99fff..a10ba579c 100644 --- a/modules/articoli/ajax/select.php +++ b/modules/articoli/ajax/select.php @@ -80,7 +80,7 @@ switch ($resource) { } $where[] = 'attivo = 1'; - + // Filtro articolo solo per documenti di vendita if ($superselect['dir'] == 'entrata' && isset($superselect['idsede_partenza'])) { $where[] = '(idsede_azienda='.prepare($superselect['idsede_partenza']).' OR idsede_controparte='.prepare($superselect['idsede_partenza']).')'; @@ -134,10 +134,10 @@ switch ($resource) { $qta_azienda = $dbo->fetchOne("SELECT SUM(mg_movimenti.qta) AS qta, IF(mg_movimenti.idsede_azienda= 0,'Sede legale',(CONCAT_WS(' - ',an_sedi.nomesede,an_sedi.citta))) as sede FROM mg_movimenti LEFT JOIN an_sedi ON an_sedi.id = mg_movimenti.idsede_azienda WHERE mg_movimenti.idarticolo=".prepare($r['id']).' AND idsede_azienda='.prepare($superselect['idsede_partenza']).' GROUP BY idsede_azienda'); // Lettura eventuali movimenti ad una propria sede (nel caso di movimenti fra sedi della mia azienda) per il calcolo corretto delle quantità - if($superselect['idsede_partenza'] != 0){ - $qta_controparte = $dbo->fetchOne("SELECT SUM(mg_movimenti.qta) AS qta, IF(mg_movimenti.idsede_controparte= 0,'Sede legale',(CONCAT_WS(' - ',an_sedi.nomesede,an_sedi.citta))) as sede FROM mg_movimenti LEFT JOIN an_sedi ON an_sedi.id = mg_movimenti.idsede_controparte WHERE mg_movimenti.idarticolo=".prepare($r['id'])." AND idsede_controparte=".prepare($superselect['idsede_partenza'])." GROUP BY idsede_controparte"); - }else{ - $qta_controparte = $dbo->fetchOne("SELECT SUM(mg_movimenti.qta) AS qta, IF(mg_movimenti.idsede_controparte= 0,'Sede legale',(CONCAT_WS(' - ',an_sedi.nomesede,an_sedi.citta))) as sede FROM ((( mg_movimenti LEFT JOIN an_sedi ON an_sedi.id = mg_movimenti.idsede_controparte ) LEFT JOIN dt_ddt ON mg_movimenti.idddt = dt_ddt.id ) LEFT JOIN co_documenti ON mg_movimenti.iddocumento = co_documenti.id ) WHERE mg_movimenti.idarticolo=".prepare($r['id'])." AND idsede_controparte=".prepare($superselect['idsede_partenza'])." AND IFNULL( dt_ddt.idanagrafica, co_documenti.idanagrafica ) = ".prepare(setting('Azienda predefinita'))." GROUP BY idsede_controparte"); + if ($superselect['idsede_partenza'] != 0) { + $qta_controparte = $dbo->fetchOne("SELECT SUM(mg_movimenti.qta) AS qta, IF(mg_movimenti.idsede_controparte= 0,'Sede legale',(CONCAT_WS(' - ',an_sedi.nomesede,an_sedi.citta))) as sede FROM mg_movimenti LEFT JOIN an_sedi ON an_sedi.id = mg_movimenti.idsede_controparte WHERE mg_movimenti.idarticolo=".prepare($r['id']).' AND idsede_controparte='.prepare($superselect['idsede_partenza']).' GROUP BY idsede_controparte'); + } else { + $qta_controparte = $dbo->fetchOne("SELECT SUM(mg_movimenti.qta) AS qta, IF(mg_movimenti.idsede_controparte= 0,'Sede legale',(CONCAT_WS(' - ',an_sedi.nomesede,an_sedi.citta))) as sede FROM ((( mg_movimenti LEFT JOIN an_sedi ON an_sedi.id = mg_movimenti.idsede_controparte ) LEFT JOIN dt_ddt ON mg_movimenti.idddt = dt_ddt.id ) LEFT JOIN co_documenti ON mg_movimenti.iddocumento = co_documenti.id ) WHERE mg_movimenti.idarticolo=".prepare($r['id']).' AND idsede_controparte='.prepare($superselect['idsede_partenza']).' AND IFNULL( dt_ddt.idanagrafica, co_documenti.idanagrafica ) = '.prepare(setting('Azienda predefinita')).' GROUP BY idsede_controparte'); } $qta = $qta_azienda['qta'] - $qta_controparte['qta']; diff --git a/modules/articoli/plugins/articoli.lotti.php b/modules/articoli/plugins/articoli.lotti.php index f4a0bf9b4..1055aaf42 100644 --- a/modules/articoli/plugins/articoli.lotti.php +++ b/modules/articoli/plugins/articoli.lotti.php @@ -7,7 +7,7 @@ include_once __DIR__.'/../../../core.php'; $record['abilita_serial'] = ($record['serial'] > 0) ? 1 : $record['abilita_serial']; if (empty($record['abilita_serial'])) { echo ' -'; +'; } echo ' diff --git a/modules/contratti/actions.php b/modules/contratti/actions.php index e0e0266a9..9841dcdba 100644 --- a/modules/contratti/actions.php +++ b/modules/contratti/actions.php @@ -57,7 +57,7 @@ switch (post('op')) { $contratto->id_documento_fe = post('id_documento_fe'); $contratto->num_item = post('num_item'); $contratto->codice_cig = post('codice_cig'); - $contratto->codice_cup = post('codice_cup');; + $contratto->codice_cup = post('codice_cup'); $contratto->save(); diff --git a/modules/interventi/modutil.php b/modules/interventi/modutil.php index a3daf0281..ac233861b 100644 --- a/modules/interventi/modutil.php +++ b/modules/interventi/modutil.php @@ -91,7 +91,7 @@ function add_tecnico($idintervento, $idtecnico, $inizio, $fine, $idcontratto = n // Leggo i costi unitari dalle tariffe se almeno un valore è stato impostato $rsc = $dbo->fetchArray('SELECT * FROM in_tariffe WHERE idtecnico='.prepare($idtecnico).' AND idtipointervento='.prepare($idtipointervento)); - + $costo_ore = $rsc[0]['costo_ore']; $costo_km = $rsc[0]['costo_km']; $costo_dirittochiamata = $rsc[0]['costo_dirittochiamata']; diff --git a/modules/interventi/src/Intervento.php b/modules/interventi/src/Intervento.php index 0d3eafc70..a9863fd43 100644 --- a/modules/interventi/src/Intervento.php +++ b/modules/interventi/src/Intervento.php @@ -6,8 +6,8 @@ use Common\Document; use Modules\Anagrafiche\Anagrafica; use Modules\Contratti\Contratto; use Modules\Preventivi\Preventivo; -use Util\Generator; use Modules\TipiIntervento\Tipo as TipoSessione; +use Util\Generator; class Intervento extends Document { diff --git a/modules/preventivi/actions.php b/modules/preventivi/actions.php index 47e8ad3fb..7f46eebd4 100644 --- a/modules/preventivi/actions.php +++ b/modules/preventivi/actions.php @@ -3,11 +3,11 @@ include_once __DIR__.'/../../core.php'; use Modules\Anagrafiche\Anagrafica; -use Modules\TipiIntervento\Tipo as TipoSessione; use Modules\Preventivi\Components\Articolo; use Modules\Preventivi\Components\Riga; use Modules\Preventivi\Components\Sconto; use Modules\Preventivi\Preventivo; +use Modules\TipiIntervento\Tipo as TipoSessione; switch (post('op')) { case 'add': diff --git a/modules/tecnici_tariffe/actions.php b/modules/tecnici_tariffe/actions.php index 404fdf65a..7b5db3749 100644 --- a/modules/tecnici_tariffe/actions.php +++ b/modules/tecnici_tariffe/actions.php @@ -9,7 +9,7 @@ switch (post('op')) { $values = [ 'costo_ore' => post('costo_ore')[$id_tipo_intervento], - 'costo_km' => post('costo_km')[$id_tipo_intervento], + 'costo_km' => post('costo_km')[$id_tipo_intervento], 'costo_dirittochiamata' => post('costo_dirittochiamata')[$id_tipo_intervento], 'costo_ore_tecnico' => post('costo_ore_tecnico')[$id_tipo_intervento], 'costo_km_tecnico' => post('costo_km_tecnico')[$id_tipo_intervento], @@ -31,10 +31,10 @@ switch (post('op')) { $id_tipo_intervento = post('idtipointervento'); $importi = $dbo->fetchOne('SELECT * FROM in_tipiintervento WHERE idtipointervento='.prepare($id_tipo_intervento)); - + $values = [ 'costo_ore' => $importi['costo_orario'], - 'costo_km' => $importi['costo_km'], + 'costo_km' => $importi['costo_km'], 'costo_dirittochiamata' => $importi['costo_diritto_chiamata'], 'costo_ore_tecnico' => $importi['costo_orario_tecnico'], 'costo_km_tecnico' => $importi['costo_km_tecnico'], @@ -42,7 +42,9 @@ switch (post('op')) { ]; foreach ($tipi_interventi as $tipo_intervento) { - if($tipo_intervento['id'] == $id_tipo_intervento) break; + if ($tipo_intervento['id'] == $id_tipo_intervento) { + break; + } } // Aggiorno il record diff --git a/modules/tecnici_tariffe/edit.php b/modules/tecnici_tariffe/edit.php index 4f2b35aac..8ef4ec7a2 100644 --- a/modules/tecnici_tariffe/edit.php +++ b/modules/tecnici_tariffe/edit.php @@ -2,7 +2,6 @@ include_once __DIR__.'/../../core.php'; - echo '
diff --git a/modules/tecnici_tariffe/init.php b/modules/tecnici_tariffe/init.php index 0bc3272e7..271a99bb1 100644 --- a/modules/tecnici_tariffe/init.php +++ b/modules/tecnici_tariffe/init.php @@ -3,7 +3,7 @@ include_once __DIR__.'/../../core.php'; if (isset($id_record)) { - $record = $dbo->fetchOne("SELECT idanagrafica, ragione_sociale, colore FROM an_anagrafiche WHERE idanagrafica = ".prepare($id_record)); + $record = $dbo->fetchOne('SELECT idanagrafica, ragione_sociale, colore FROM an_anagrafiche WHERE idanagrafica = '.prepare($id_record)); $tipi_interventi = $dbo->fetchArray('SELECT *, in_tipiintervento.idtipointervento AS id, in_tariffe.idtipointervento AS esiste FROM in_tipiintervento LEFT JOIN in_tariffe ON in_tipiintervento.idtipointervento = in_tariffe.idtipointervento AND in_tariffe.idtecnico = '.prepare($id_record).' ORDER BY descrizione'); } diff --git a/modules/tipi_intervento/actions.php b/modules/tipi_intervento/actions.php index 18bdc5992..a49799be9 100644 --- a/modules/tipi_intervento/actions.php +++ b/modules/tipi_intervento/actions.php @@ -1,6 +1,7 @@ costo_km_tecnico = post('costo_km_tecnico'); $tipo->costo_diritto_chiamata_tecnico = post('costo_diritto_chiamata_tecnico'); $tipo->tempo_standard = post('tempo_standard'); - + $tipo->save(); flash()->info(tr('Informazioni tipo intervento salvate correttamente!')); @@ -47,7 +48,7 @@ switch (post('op')) { case 'import': $values = [ 'costo_ore' => $record['costo_orario'], - 'costo_km' => $record['costo_km'], + 'costo_km' => $record['costo_km'], 'costo_dirittochiamata' => $record['costo_diritto_chiamata'], 'costo_ore_tecnico' => $record['costo_orario_tecnico'], 'costo_km_tecnico' => $record['costo_km_tecnico'], diff --git a/modules/tipi_intervento/init.php b/modules/tipi_intervento/init.php index 4cc9fc037..3aff99998 100644 --- a/modules/tipi_intervento/init.php +++ b/modules/tipi_intervento/init.php @@ -2,6 +2,7 @@ include_once __DIR__.'/../../core.php'; use Modules\TipiIntervento\Tipo; + if (isset($id_record)) { $record = $dbo->fetchOne('SELECT * FROM in_tipiintervento WHERE idtipointervento='.prepare($id_record)); diff --git a/modules/tipi_intervento/src/Tipo.php b/modules/tipi_intervento/src/Tipo.php index 0a48635aa..6ce9cd5d5 100644 --- a/modules/tipi_intervento/src/Tipo.php +++ b/modules/tipi_intervento/src/Tipo.php @@ -4,7 +4,6 @@ namespace Modules\TipiIntervento; use Common\Model; use Modules\Anagrafiche\Anagrafica; -use Modules\Contratti\Contratto; class Tipo extends Model { @@ -15,9 +14,9 @@ class Tipo extends Model /** * Crea un nuovo tipo di intervento. * - * @param string $codice - * @param string $descrizione - * @param string $tempo_standard + * @param string $codice + * @param string $descrizione + * @param string $tempo_standard * * @return self */ @@ -37,10 +36,11 @@ class Tipo extends Model return $model; } - public function fixTecnici(){ + public function fixTecnici() + { // Fix per le relazioni con i tecnici $tecnici = Anagrafica::fromTipo('Tecnico')->get(); - foreach ($tecnici as $tecnico){ + foreach ($tecnici as $tecnico) { Anagrafica::fixTecnico($tecnico); } } @@ -67,7 +67,6 @@ class Tipo extends Model $this->attributes['tempo_standard'] = $result; } - public function preventivi() { return $this->hasMany(Preventivo::class, 'idtipointervento'); @@ -77,6 +76,4 @@ class Tipo extends Model { return $this->hasMany(Intervento::class, 'idtipointervento'); } - - } diff --git a/plugins/rinnovi_contratti/edit.php b/plugins/rinnovi_contratti/edit.php index 6964af4b2..8f9132265 100644 --- a/plugins/rinnovi_contratti/edit.php +++ b/plugins/rinnovi_contratti/edit.php @@ -4,7 +4,7 @@ include_once __DIR__.'/../../../core.php'; $id_contratto_precedente = $record['idcontratto_prev']; -if(empty($id_contratto_precedente)){ +if (empty($id_contratto_precedente)) { echo ' '; } @@ -21,7 +21,7 @@ echo ' while (!empty($id_contratto_precedente)) { $rs = $dbo->fetchArray('SELECT nome, numero, data_accettazione, data_conclusione, budget, idcontratto_prev FROM co_contratti WHERE id='.prepare($id_contratto_precedente)); -echo ' + echo ' '.Modules::link($id_module, $id_contratto_precedente, tr('Contratto num. _NUM_', [ diff --git a/update/2_4_10.php b/update/2_4_10.php index 72a717a4d..a43a32bd0 100644 --- a/update/2_4_10.php +++ b/update/2_4_10.php @@ -10,7 +10,7 @@ foreach ($fatture as $fattura) { // Fix per le relazioni tariffe-tecnici $tecnici = Anagrafica::fromTipo('Tecnico')->get(); -foreach ($tecnici as $tecnico){ +foreach ($tecnici as $tecnico) { Anagrafica::fixTecnico($tecnico); }