Skip to content

Commit a7e789c

Browse files
Pek5892Pek5892
committed
fix: gestione valori a null database
1 parent 1a96114 commit a7e789c

9 files changed

Lines changed: 144 additions & 65 deletions

File tree

‎modules/anagrafiche/actions.php‎

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -63,38 +63,38 @@
6363
$anagrafica->idsede_fatturazione = post('idsede_fatturazione');
6464
$anagrafica->note = post('note');
6565
$anagrafica->codiceri = post('codiceri');
66-
$anagrafica->codicerea = strtoupper(post('codicerea'));
66+
$anagrafica->codicerea = strtoupper(post('codicerea'))?:null;
6767
$anagrafica->appoggiobancario = post('appoggiobancario');
6868
$anagrafica->filiale = post('filiale');
6969
$anagrafica->codiceiban = post('codiceiban');
7070
$anagrafica->bic = post('bic');
7171
$anagrafica->diciturafissafattura = post('diciturafissafattura');
72-
$anagrafica->idpagamento_acquisti = post('idpagamento_acquisti');
73-
$anagrafica->idpagamento_vendite = post('idpagamento_vendite');
74-
$anagrafica->id_piano_sconto_acquisti = post('id_piano_sconto_acquisti');
75-
$anagrafica->id_piano_sconto_vendite = post('id_piano_sconto_vendite');
76-
$anagrafica->idiva_acquisti = post('idiva_acquisti');
77-
$anagrafica->idiva_vendite = post('idiva_vendite');
78-
$anagrafica->idbanca_acquisti = post('idbanca_acquisti');
79-
$anagrafica->idbanca_vendite = post('idbanca_vendite');
72+
$anagrafica->idpagamento_acquisti = post('idpagamento_acquisti')?:null;
73+
$anagrafica->idpagamento_vendite = post('idpagamento_vendite')?:null;
74+
$anagrafica->id_piano_sconto_acquisti = post('id_piano_sconto_acquisti')?:null;
75+
$anagrafica->id_piano_sconto_vendite = post('id_piano_sconto_vendite')?:null;
76+
$anagrafica->idiva_acquisti = post('idiva_acquisti')?:null;
77+
$anagrafica->idiva_vendite = post('idiva_vendite')?:null;
78+
$anagrafica->idbanca_acquisti = post('idbanca_acquisti')?:null;
79+
$anagrafica->idbanca_vendite = post('idbanca_vendite') ?: null;
8080
$anagrafica->id_settore = post('id_settore');
8181
$anagrafica->marche = post('marche');
8282
$anagrafica->dipendenti = post('dipendenti');
8383
$anagrafica->macchine = post('macchine');
8484
$anagrafica->idagente = post('idagente');
85-
$anagrafica->id_provenienza = post('id_provenienza');
85+
$anagrafica->id_provenienza = post('id_provenienza') ?: null;
8686
$anagrafica->idrelazione = post('idrelazione');
8787
$anagrafica->sitoweb = post('sitoweb');
8888
$anagrafica->iscrizione_tribunale = post('iscrizione_tribunale');
89-
$anagrafica->n_alboartigiani = post('n_alboartigiani');
89+
$anagrafica->n_alboartigiani = post('n_alboartigiani') ?: null;
9090
$anagrafica->foro_competenza = post('foro_competenza');
91-
$anagrafica->riferimento_amministrazione = post('riferimento_amministrazione');
91+
$anagrafica->riferimento_amministrazione = post('riferimento_amministrazione')?:null;
9292
$anagrafica->colore = post('colore');
9393
$anagrafica->idtipointervento_default = post('idtipointervento_default') ?: null;
9494
$anagrafica->id_dichiarazione_intento_default = post('id_dichiarazione_intento_default') ?: null;
9595
$anagrafica->provvigione_default = post('provvigione_default');
96-
$anagrafica->id_ritenuta_acconto_acquisti = post('id_ritenuta_acconto_acquisti');
97-
$anagrafica->id_ritenuta_acconto_vendite = post('id_ritenuta_acconto_vendite');
96+
$anagrafica->id_ritenuta_acconto_acquisti = post('id_ritenuta_acconto_acquisti') ?: null;
97+
$anagrafica->id_ritenuta_acconto_vendite = post('id_ritenuta_acconto_vendite') ?: null;
9898
$anagrafica->split_payment = post('split_payment');
9999
$anagrafica->id_listino = post('id_listino');
100100
$anagrafica->tipologie = (array) post('idtipoanagrafica');
@@ -219,11 +219,11 @@
219219
$anagrafica->telefono = post('telefono');
220220
$anagrafica->cellulare = post('cellulare');
221221
$anagrafica->email = post('email');
222-
$anagrafica->idagente = $idagente;
222+
$anagrafica->idagente = $idagente ?: null;
223223
$anagrafica->pec = post('pec');
224224
$anagrafica->tipo = post('tipo');
225225
$anagrafica->id_nazione = post('id_nazione') ?: null;
226-
$anagrafica->codice_destinatario = strtoupper(post('codice_destinatario'));
226+
$anagrafica->codice_destinatario = strtoupper(post('codice_destinatario')) ?: null;
227227

228228
$anagrafica->codice_fiscale = strtoupper(post('codice_fiscale'));
229229
$anagrafica->partita_iva = strtoupper(post('piva'));

‎modules/contratti/actions.php‎

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -91,8 +91,8 @@
9191
$contratto->idreferente = post('idreferente');
9292
$contratto->condizioni_fornitura = post('condizioni_fornitura');
9393
$contratto->informazioniaggiuntive = post('informazioniaggiuntive');
94-
$contratto->id_categoria = post('id_categoria');
95-
$contratto->id_sottocategoria = post('id_sottocategoria');
94+
$contratto->id_categoria = post('id_categoria') ?: null;
95+
$contratto->id_sottocategoria = post('id_sottocategoria') ?: null;
9696

9797
// Informazioni sulle date del documento
9898
$contratto->data_bozza = post('data_bozza') ?: null;
@@ -206,6 +206,7 @@
206206
$articolo->setPrezzoUnitario(post('prezzo_unitario'), post('idiva'));
207207
$articolo->setSconto(post('sconto'), post('tipo_sconto'));
208208
$articolo->setProvvigione(post('provvigione'), post('tipo_provvigione'));
209+
$articolo->idconto = post('idconto') ?: null;
209210

210211
try {
211212
$articolo->qta = $qta;
@@ -545,7 +546,7 @@
545546
if (post('create_document') == 'on') {
546547
$contratto = Contratto::build($documento->anagrafica, $documento->nome, post('id_segment'));
547548

548-
$contratto->idpagamento = $documento->idpagamento;
549+
$contratto->idpagamento = $documento->idpagamento?:setting('Tipo di pagamento predefinito');
549550
$contratto->idsede_partenza = $idsede_partenza;
550551
$contratto->idsede_destinazione = $idsede_destinazione;
551552
$contratto->rinnovabile = setting('Crea contratto rinnovabile di default');

‎modules/ddt/actions.php‎

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -91,16 +91,16 @@
9191
$ddt->idanagrafica = $id_anagrafica;
9292
$ddt->idreferente = post('idreferente');
9393
$ddt->idagente = post('idagente');
94-
$ddt->idspedizione = post('idspedizione');
95-
$ddt->idcausalet = post('idcausalet');
94+
$ddt->idspedizione = post('idspedizione') ?: null;
95+
$ddt->idcausalet = post('idcausalet') ?: null;
9696
$ddt->idsede_partenza = post('idsede_partenza');
9797
$ddt->idsede_destinazione = post('idsede_destinazione');
98-
$ddt->idvettore = post('idvettore');
98+
$ddt->idvettore = post('idvettore') ?: null;
9999
$ddt->data_ora_trasporto = post('data_ora_trasporto') ?: null;
100-
$ddt->idporto = post('idporto');
101-
$ddt->idaspettobeni = post('idaspettobeni');
102-
$ddt->idrivalsainps = $idrivalsainps;
103-
$ddt->idritenutaacconto = $idritenutaacconto;
100+
$ddt->idporto = post('idporto') ?: null;
101+
$ddt->idaspettobeni = post('idaspettobeni') ?: null;
102+
$ddt->idrivalsainps = $idrivalsainps ?: null;
103+
$ddt->idritenutaacconto = $idritenutaacconto ?: null;
104104

105105
$ddt->n_colli = post('n_colli');
106106
$ddt->peso = post('peso');
@@ -174,6 +174,7 @@
174174
$articolo->costo_unitario = post('costo_unitario') ?: 0;
175175
$articolo->setPrezzoUnitario(post('prezzo_unitario'), post('idiva'));
176176
$articolo->setSconto(post('sconto'), post('tipo_sconto'));
177+
$articolo->idconto = post('idconto') ?: null;
177178
if ($dir == 'entrata') {
178179
$articolo->setProvvigione(post('provvigione'), post('tipo_provvigione'));
179180
}
@@ -298,7 +299,7 @@
298299
$tipo = Tipo::where('dir', $documento->direzione)->first();
299300

300301
$ddt = DDT::build($documento->anagrafica, $tipo, post('data'), post('id_segment'));
301-
$ddt->idpagamento = $documento->idpagamento;
302+
$ddt->idpagamento = $documento->idpagamento?:setting('Tipo di pagamento predefinito');
302303

303304
$ddt->id_documento_fe = $documento->id_documento_fe;
304305
$ddt->codice_cup = $documento->codice_cup;
@@ -497,7 +498,7 @@
497498
$copia->idcausalet = $ddt->idcausalet;
498499
$copia->idspedizione = $ddt->idspedizione;
499500
$copia->n_colli = $ddt->n_colli;
500-
$copia->idpagamento = $ddt->idpagamento;
501+
$copia->idpagamento = $ddt->idpagamento?:setting('Tipo di pagamento predefinito');
501502
$copia->idporto = $ddt->idporto;
502503
$copia->idvettore = $ddt->idvettore;
503504
$copia->data_ora_trasporto = $ddt->data_ora_trasporto;

‎modules/fatture/actions.php‎

Lines changed: 23 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -137,19 +137,19 @@
137137
$fattura->note_aggiuntive = post('note_aggiuntive');
138138

139139
$fattura->idanagrafica = post('idanagrafica');
140-
$fattura->idagente = post('idagente') ?: '';
141-
$fattura->idreferente = post('idreferente');
142-
$fattura->idpagamento = post('idpagamento');
143-
$fattura->id_banca_azienda = post('id_banca_azienda');
144-
$fattura->id_banca_controparte = post('id_banca_controparte');
145-
$fattura->idcausalet = post('idcausalet');
146-
$fattura->idspedizione = post('idspedizione');
147-
$fattura->idporto = post('idporto');
148-
$fattura->idaspettobeni = post('idaspettobeni');
149-
$fattura->idvettore = post('idvettore');
150-
$fattura->idsede_partenza = post('idsede_partenza');
151-
$fattura->idsede_destinazione = post('idsede_destinazione');
152-
$fattura->idconto = post('idconto');
140+
$fattura->idagente = post('idagente') ?: null;
141+
$fattura->idreferente = post('idreferente') ?: null;
142+
$fattura->idpagamento = post('idpagamento') ?: null;
143+
$fattura->id_banca_azienda = post('id_banca_azienda') ?: null;
144+
$fattura->id_banca_controparte = post('id_banca_controparte') ?: null;
145+
$fattura->idcausalet = post('idcausalet') ?: null;
146+
$fattura->idspedizione = post('idspedizione') ?: null;
147+
$fattura->idporto = post('idporto') ?: null;
148+
$fattura->idaspettobeni = post('idaspettobeni') ?: null;
149+
$fattura->idvettore = post('idvettore') ?: null;
150+
$fattura->idsede_partenza = post('idsede_partenza') ?: null;
151+
$fattura->idsede_destinazione = post('idsede_destinazione') ?: null;
152+
$fattura->idconto = post('idconto') ?: null;
153153
$fattura->split_payment = post('split_payment') ?: 0;
154154
$fattura->is_fattura_conto_terzi = post('is_fattura_conto_terzi') ?: 0;
155155
$fattura->n_colli = post('n_colli');
@@ -171,8 +171,8 @@
171171
if ($dir == 'uscita') {
172172
$fattura->numero = post('numero');
173173
$fattura->numero_esterno = post('numero_esterno');
174-
$fattura->idrivalsainps = post('id_rivalsa_inps');
175-
$fattura->idritenutaacconto = post('id_ritenuta_acconto');
174+
$fattura->idrivalsainps = post('id_rivalsa_inps') ?: null;
175+
$fattura->idritenutaacconto = post('id_ritenuta_acconto') ?: null;
176176
}
177177

178178
// Operazioni sul bollo
@@ -533,7 +533,7 @@
533533
$articolo->data_fine_competenza = post('data_fine_competenza') ?: null;
534534

535535
$articolo->id_iva = post('idiva');
536-
$articolo->idconto = post('idconto');
536+
$articolo->idconto = post('idconto') ?: null;
537537

538538
$articolo->calcolo_ritenuta_acconto = post('calcolo_ritenuta_acconto') ?: null;
539539
$articolo->id_ritenuta_acconto = post('id_ritenuta_acconto') ?: null;
@@ -784,17 +784,17 @@
784784

785785
$fattura = Fattura::build($documento->anagrafica, $tipo, post('data'), post('id_segment'));
786786

787-
if (!empty($documento->idpagamento)) {
787+
if (!empty($documento->idpagamento) && $documento->idpagamento != 0) {
788788
$fattura->idpagamento = $documento->idpagamento;
789789
} else {
790790
$fattura->idpagamento = setting('Tipo di pagamento predefinito');
791791
}
792792

793-
$fattura->idsede_partenza = $idsede_partenza;
794-
$fattura->idsede_destinazione = $idsede_destinazione;
793+
$fattura->idsede_partenza = $idsede_partenza ?: null;
794+
$fattura->idsede_destinazione = $idsede_destinazione ?: null;
795795
$fattura->id_ritenuta_contributi = post('id_ritenuta_contributi') ?: null;
796-
$fattura->idreferente = $documento->idreferente;
797-
$fattura->idagente = $documento->idagente ?: '';
796+
$fattura->idreferente = $documento->idreferente ?: null;
797+
$fattura->idagente = $documento->idagente ?: null;
798798

799799
$fattura->save();
800800

@@ -1159,8 +1159,8 @@
11591159

11601160
$provvigione = $dbo->selectOne('an_anagrafiche', 'provvigione_default', ['idanagrafica' => $fattura->idagente])['provvigione_default'];
11611161

1162-
$articolo->id_rivalsa_inps = setting('Cassa previdenziale predefinita') ?: '';
1163-
$articolo->id_ritenuta_acconto = setting('Ritenuta d\'acconto predefinita') ?: '';
1162+
$articolo->id_rivalsa_inps = setting('Cassa previdenziale predefinita') ?: null;
1163+
$articolo->id_ritenuta_acconto = setting('Ritenuta d\'acconto predefinita') ?: null;
11641164
$articolo->setPrezzoUnitario($prezzo_unitario, $id_iva);
11651165
$articolo->setSconto($sconto, 'PRC');
11661166
$articolo->setProvvigione($provvigione ?: 0, 'PRC');

‎modules/interventi/actions.php‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,6 @@
4747

4848
switch (post('op')) {
4949
case 'update':
50-
$idcontratto = post('idcontratto') ?: null;
5150
$id_promemoria = post('idcontratto_riga');
5251
$tecnici_assegnati_array = post('tecnici_assegnati') ?: [];
5352

@@ -65,7 +64,7 @@
6564
$intervento->informazioniaggiuntive = post('informazioniaggiuntive');
6665

6766
$intervento->idanagrafica = post('idanagrafica');
68-
$intervento->idclientefinale = post('idclientefinale');
67+
$intervento->idclientefinale = post('idclientefinale') ?: null;
6968
$intervento->idreferente = post('idreferente');
7069
$intervento->idagente = post('idagente');
7170
$intervento->idtipointervento = post('idtipointervento');
@@ -74,7 +73,7 @@
7473
$intervento->idsede_partenza = post('idsede_partenza');
7574
$intervento->idsede_destinazione = post('idsede_destinazione');
7675
$intervento->id_preventivo = post('idpreventivo') ?: null;
77-
$intervento->id_contratto = $idcontratto;
76+
$intervento->id_contratto = post('idcontratto') ?: null;
7877
$intervento->id_ordine = post('idordine') ?: null;
7978
$intervento->idpagamento = post('idpagamento');
8079

@@ -334,7 +333,7 @@
334333
OperationLog::setInfo('id_record', $id_record);
335334
OperationLog::setInfo('level', 'info');
336335
OperationLog::build('add_sessione');
337-
336+
338337
}
339338

340339
// Assegnazione dei tecnici all'intervento
@@ -667,6 +666,7 @@
667666
$articolo->setPrezzoUnitario(post('prezzo_unitario'), post('idiva'));
668667
$articolo->setSconto(post('sconto'), post('tipo_sconto'));
669668
$articolo->setProvvigione(post('provvigione'), post('tipo_provvigione'));
669+
$articolo->idconto = post('idconto') ?: null;
670670

671671
try {
672672
$articolo->qta = $qta;

‎modules/ordini/actions.php‎

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -93,14 +93,14 @@
9393
$ordine->idagente = post('idagente');
9494
$ordine->idstatoordine = $idstatoordine;
9595
$ordine->idpagamento = $idpagamento;
96-
$ordine->idspedizione = post('idspedizione');
97-
$ordine->idporto = post('idporto');
98-
$ordine->idvettore = post('idvettore');
96+
$ordine->idspedizione = post('idspedizione') ?: null;
97+
$ordine->idporto = post('idporto') ?: null;
98+
$ordine->idvettore = post('idvettore') ?: null;
9999
$ordine->idsede_partenza = post('idsede_partenza');
100100
$ordine->idsede_destinazione = post('idsede_destinazione');
101101
$ordine->idconto = post('idconto');
102-
$ordine->idrivalsainps = $idrivalsainps;
103-
$ordine->idritenutaacconto = $idritenutaacconto;
102+
$ordine->idrivalsainps = $idrivalsainps ?: null;
103+
$ordine->idritenutaacconto = $idritenutaacconto ?: null;
104104

105105
$ordine->bollo = 0;
106106
$ordine->rivalsainps = 0;
@@ -185,6 +185,7 @@
185185
$articolo->confermato = post('confermato') ?: 0;
186186
$articolo->setPrezzoUnitario(post('prezzo_unitario'), post('idiva'));
187187
$articolo->setSconto(post('sconto'), post('tipo_sconto'));
188+
$articolo->idconto = post('idconto') ?: null;
188189
if ($dir == 'entrata') {
189190
$articolo->setProvvigione(post('provvigione'), post('tipo_provvigione'));
190191
}
@@ -431,7 +432,7 @@
431432
$tipo = Tipo::where('dir', $documento->direzione)->first();
432433

433434
$ordine = Ordine::build($documento->anagrafica, $tipo, post('data'), post('id_segment'));
434-
$ordine->idpagamento = $documento->idpagamento;
435+
$ordine->idpagamento = $documento->idpagamento?:setting('Tipo di pagamento predefinito');
435436
$ordine->idsede_partenza = $idsede_partenza;
436437
$ordine->idsede_destinazione = $idsede_destinazione;
437438

‎modules/preventivi/actions.php‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -71,8 +71,8 @@
7171
$preventivo->idsede_destinazione = post('idsede_destinazione');
7272
$preventivo->idagente = post('idagente');
7373
$preventivo->idreferente = post('idreferente');
74-
$preventivo->idpagamento = post('idpagamento');
75-
$preventivo->idporto = post('idporto');
74+
$preventivo->idpagamento = post('idpagamento') ?: null;
75+
$preventivo->idporto = post('idporto') ?: null;
7676
$preventivo->tempi_consegna = post('tempi_consegna');
7777
$preventivo->numero = post('numero');
7878
$preventivo->condizioni_fornitura = post('condizioni_fornitura');
@@ -211,6 +211,7 @@
211211
$articolo->setPrezzoUnitario(post('prezzo_unitario'), post('idiva'));
212212
$articolo->setSconto(post('sconto'), post('tipo_sconto'));
213213
$articolo->setProvvigione(post('provvigione'), post('tipo_provvigione'));
214+
$articolo->idconto = post('idconto') ?: null;
214215

215216
try {
216217
$articolo->qta = $qta;

‎templates/preventivi/body.php‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,11 +30,11 @@
3030
$anagrafica_azienda = Anagrafica::find(setting('Azienda predefinita'));
3131
$prezzi_ivati = setting('Utilizza prezzi di vendita comprensivi di IVA');
3232

33-
$pagamento = Pagamento::find($documento['idpagamento']);
33+
$pagamento = $documento['idpagamento'] ? Pagamento::find($documento['idpagamento']) : null;
3434

3535
// Banca dell'Azienda corrente impostata come predefinita per il Cliente
3636
$banca_azienda = Banca::where('id_anagrafica', '=', $anagrafica_azienda->id)
37-
->where('id_pianodeiconti3', '=', $pagamento['idconto_vendite'] ?: 0);
37+
->where('id_pianodeiconti3', '=', $pagamento ? ($pagamento['idconto_vendite'] ?: 0) : 0);
3838
try {
3939
$banca = (clone $banca_azienda)
4040
->findOrFail($anagrafica->idbanca_vendite);

0 commit comments

Comments
 (0)