Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Problema salvataggio scadenze su scadenzario #3

Closed
loviuz opened this issue Aug 8, 2017 · 7 comments
Closed

Problema salvataggio scadenze su scadenzario #3

loviuz opened this issue Aug 8, 2017 · 7 comments
Labels
bug Bug generali del progetto
Milestone

Comments

@loviuz
Copy link
Contributor

loviuz commented Aug 8, 2017

Creata fattura di acquisto da 103,70 euro con 2 scadenze (tipo pagamento: bonifico 30/60 gg d.f). Vado sullo scadenziario e cerco di inserire il pagamento della prima rata. Inserisco l'importo e salvo. Non salva dicendomi "Il totale degli importi inseriti non corrisponde al totale da pagare!", stesso problema anche con 1 sola scadenza. Su fatture di vendita non accade.

@loviuz loviuz added the bug Bug generali del progetto label Aug 8, 2017
@loviuz loviuz added this to the Versione 2.3 STABILE milestone Aug 8, 2017
@Dasc3er
Copy link
Contributor

Dasc3er commented Aug 9, 2017

Apparentemente, il problema è relativo al fatto che per le fatture di acquisto il valore salvato nel campo da_pagare nella tabella co_scadenziario è negativo...

@loviuz E' corretto questo comportamento?
Se si, basterebbe un controllo sul valore assoluto del totale da pagare.

Un'altra cosa: all'interno del file actions.php del modulo Prima Nota viene aggiunto al totale da pagare il valore presente in $dare sia nel caso questo sia assegnato che quando viene utilizzata la variabile $avere. Questo comportamento è corretto?

if ($dare != '' && $dare != 0) {
    $totale = $dare;
    $totale_pagato += $dare;
} elseif ($avere != '' && $avere != 0) {
    $totale = -$avere;
    $totale_pagato -= $dare;
} 

Dasc3er added a commit that referenced this issue Aug 11, 2017
Risoluzione del bug #3 grazie all'utilizzo della funziona abs() sul totale da pagare, con fix di un bug nella prima nota.
Miglioramento nella gestione dei numeri  dagli input a livello JS (nuove funzioni integrate toEnglish() e toLocale() da utilizzare sul valore dell'input).
@loviuz
Copy link
Contributor Author

loviuz commented Aug 11, 2017

Apparentemente, il problema è relativo al fatto che per le fatture di acquisto il valore salvato nel campo da_pagare nella tabella co_scadenziario è negativo...

Sì, è corretto

Un'altra cosa: all'interno del file actions.php del modulo Prima Nota viene aggiunto al totale da pagare il valore presente in $dare sia nel caso questo sia assegnato che quando viene utilizzata la variabile $avere. Questo comportamento è corretto?

Devo verificare

Dasc3er added a commit that referenced this issue Aug 12, 2017
Risoluzione completa del bug #3, causato da alcuni problemi con l'inputmask inizializzata con valori negativi.
@Dasc3er
Copy link
Contributor

Dasc3er commented Aug 12, 2017

@loviuz Dovrei aver risolto il primo problema, che era relativo all'inizializzazione di InputMask con valori negativi con la nuova configurazione...
In attesa di chiarimenti per il secondo comportamento.

@loviuz
Copy link
Contributor Author

loviuz commented Aug 12, 2017

Un'altra cosa: all'interno del file actions.php del modulo Prima Nota viene aggiunto al totale da pagare il valore presente in $dare sia nel caso questo sia assegnato che quando viene utilizzata la variabile $avere. Questo comportamento è corretto?

ho verificato nelle precedenti versioni, ed è così. per il fatto che la prima nota deve avere il totale di dare e avere uguali, ovviamente il $totale_pagato torna, però concettualmente è sbagliato :-)
meglio cambiarlo così:

if ($dare != '' && $dare != 0) {
    $totale = $dare;
    $totale_pagato += $dare;
} elseif ($avere != '' && $avere != 0) {
    $totale = -$avere;
    $totale_pagato -= $avere;
}

Una cosa che ho visto non funzionare invece è che, se emetto una fattura e poi creo la prima nota, che serve per segnare l'incasso della fattura, la trovo ancora nello scadenzario perché non si aggiorna il campo "pagato" su co_scadenzario.
Nella data di scadenza dello scadenzario poi c'è ancora il formato DATETIME, per cui vedo ad esempio
01/09/2017 00:00

@Dasc3er
Copy link
Contributor

Dasc3er commented Aug 23, 2017

Riguardo le date nel database, all'interno di co_scadenziario sono presenti tre attributi impostati a DATETIME: data_emissione, scadenza, data_pagamento.
Sarebbero da convertire tutti e tre?

Per maggiori informazioni, #19.

@loviuz
Copy link
Contributor Author

loviuz commented Aug 26, 2017

yesss. basta solo la data, non l'ora. se li converti nel file di update, possiamo chiudere anche questa segnalazione

@Dasc3er
Copy link
Contributor

Dasc3er commented Aug 26, 2017

Li ho inseriti anche nella segnalazione #19, quindi lavoriamo su quella per i DATETIME.

@loviuz Servirebbe anche un controllo sulla risoluzione di questo bug, per sicurezza...

@Dasc3er Dasc3er closed this as completed Aug 28, 2017
Dasc3er pushed a commit that referenced this issue Mar 7, 2019
* Merge master (#3)

* Correzione duplica fattura

Corretta la funzione di duplicazione fattura che precedentemente non copiava correttamente le righe "Descrizione"

* Aggiunta estensione al file allegato alle email (correzione)

* Corretto inserimento metodo di pagamento nei preventivi

* Migliorie minori

* Aggiunti campi nome e cognome + Fix vari FE

* Correzione codice intervento

Il codice non veniva trovato se presentava nel formato uno dei modificatori come 'yy' perché mancava la conversione di questi modificatori nel template da cercare

* Corretto codice intervento

Aggiunta correzione per inserire correttamente i modificatori al codice intervento per la ricerca ed elaborazione corretta della sequenza codici

* Completamento blocchi denominazione e nome-cognome

* Migliorata visualizzazione XML da importare con dati sede

* Aggiunta preview XML acquisto + migliorie minori

* Migliorie grafiche su pulsanti

* Completamento integrazione cassa previdenziale in fattura elettronica

* Ulteriori correzioni su calcolo totale fattura

* Risolto calcolo statistiche fatturato per cliente, issue #400

* Fix grafiche minori

* Correzioni strutturali

@loviuz Ho corretto alcune modifiche degli ultimi commit.
Il metodo calcola è esclusivamente interno e deve essere utilizzato come tale (per le modifiche di comportamento ci sono gli attributi appositi).

* Fix #415

* Corretta stampa filtro riepilogo interventi

* Fix minori

* Fix minori

* Aumento percentuali rate pagamenti a 2

* Bugfix formattazione numeri con decimali al salvataggio con "Invio"

* Fix riferimenti in fattura per contratti, preventivi e interventi

* Fix minori

* Update 2_4_5.sql

* Bugfix: non inseriva in XML le rate pagamento complete ma solo l'ultima

* Fix minori

* Update edit.php

* Modifica stati fatturabili dagli ordini

* Fix minori

* Aggiunta eccezione per XML

* Aggiunti pagamenti mancanti + Fix DatiPagamento import FE

* Export XML massivo

* Miglioramento codice
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug generali del progetto
Projects
None yet
Development

No branches or pull requests

2 participants