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

[16.0] [FIX] l10n_it_declaration_of_intent: fixed manual declaration of intent selection #4587

Open
wants to merge 1 commit into
base: 16.0
Choose a base branch
from

Conversation

sherpya
Copy link
Member

@sherpya sherpya commented Jan 30, 2025

the form allows only add declarations of intent,
this fix clears the set before adding new ones

closes #4576

the form allows only add declarations of intent,
this fix clears the set before adding new ones

closes OCA#4576
@francesco-ooops
Copy link
Contributor

@primes2h riesci a fare un test?

Copy link
Contributor

@primes2h primes2h left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ho provato a fare un test di modifica della DI assegnata alla fattura.

  • Ho creato 2 dichiarazioni di intento
  • Ho creato una fattura assegnandogli la DI n. 1, quindi ho confermato la fattura.
  • Ho rimesso in bozza la fattura sostituendo la DI n. 1 con la DI n. 2 e ho riconfermato la fattura.

L'importo viene scalato da entrambe le dichiarazioni di intento.

Vedi

Peek 30-01-2025 17-17

@sherpya
Copy link
Member Author

sherpya commented Jan 30, 2025

Ho provato a fare un test di modifica della DI assegnata alla fattura.

* Ho creato 2 dichiarazioni di intento

* Ho creato una fattura assegnandogli la DI n. 1, quindi ho confermato la fattura.

* Ho rimesso in bozza la fattura sostituendo la DI n. 1 con la DI n. 2 e ho riconfermato la fattura.

L'importo viene scalato da entrambe le dichiarazioni di intento.

Vedi

dovrebbe in qualche modo "ritornare indietro" l'importo di quelle levate? quando scala esattamente il valore? perché mi viene in mente che potrebbe scalarlo più volte anche se ne hai solo uno se fai draft -> approva

@primes2h
Copy link
Contributor

primes2h commented Jan 31, 2025

Ho provato a fare un test di modifica della DI assegnata alla fattura.

* Ho creato 2 dichiarazioni di intento

* Ho creato una fattura assegnandogli la DI n. 1, quindi ho confermato la fattura.

* Ho rimesso in bozza la fattura sostituendo la DI n. 1 con la DI n. 2 e ho riconfermato la fattura.

L'importo viene scalato da entrambe le dichiarazioni di intento.
Vedi

dovrebbe in qualche modo "ritornare indietro" l'importo di quelle levate?

Certo, l'importo deve essere scalato solo da quelle selezionate, partendo dalla prima.

quando scala esattamente il valore? perché mi viene in mente che potrebbe scalarlo più volte anche se ne hai solo uno se fai draft -> approva

No, se rimetti la fattura in bozza l'importo disponibile della DI viene ripristinato.

@primes2h
Copy link
Contributor

Tra l'altro noto un altro problema.
Per cambiare la DI associata alla fattura è necessario rimetterla prima in bozza.

Queste bozze restano "appese" alle dichiarazioni di intento anche dopo la riconferma.

immagine

@primes2h
Copy link
Contributor

Tra l'altro noto un altro problema. Per cambiare la DI associata alla fattura è necessario rimetterla prima in bozza.

Queste bozze restano "appese" alle dichiarazioni di intento anche dopo la riconferma.

Mi rispondo da solo.
Vedo che sono bozze relative ad altre fatture e non vanno a scalare l'importo. Quindi il comportamento qui è corretto.

@sherpya
Copy link
Member Author

sherpya commented Jan 31, 2025

quando scala esattamente il valore? perché mi viene in mente che potrebbe scalarlo più volte anche se ne hai solo uno se fai draft -> approva

No, se rimetti la fattura in bozza l'importo disponibile della DI viene ripristinato.

nel video test con due che hai fatto perché non lo fa?

@primes2h
Copy link
Contributor

primes2h commented Jan 31, 2025

quando scala esattamente il valore? perché mi viene in mente che potrebbe scalarlo più volte anche se ne hai solo uno se fai draft -> approva

No, se rimetti la fattura in bozza l'importo disponibile della DI viene ripristinato.

nel video test con due che hai fatto perché non lo fa?

Nel video la fattura viene messa in bozza, cambiata la DI 1 con la 2 e poi riconfermata. Non si vede come cambiano gli importi delle DI mentre la fattura è in bozza.

Se provi a fare un test con runboat puoi verificare anche tu che l'importo viene correttamente ripristinato.

Vedi

Peek 31-01-2025 15-20

@sherpya
Copy link
Member Author

sherpya commented Jan 31, 2025

quando scala esattamente il valore? perché mi viene in mente che potrebbe scalarlo più volte anche se ne hai solo uno se fai draft -> approva

No, se rimetti la fattura in bozza l'importo disponibile della DI viene ripristinato.

nel video test con due che hai fatto perché non lo fa?

Nel video la fattura viene messa in bozza, cambiata la DI 1 con la 2 e poi riconfermata. Non si vede come cambiano gli importi delle DI mentre la fattura è in bozza.

Se provi a fare un test con runboat puoi verificare anche tu che l'importo viene correttamente ripristinato.

A me sembra un bug differente, se senza la patch date due dichiarazioni di intenti e una fattura:

  • aggiungi una dichiarazione di intenti
  • confermi la fattura
  • metti in bozza la fattura
  • aggiungi un'altra dichiarazione di intenti (puoi fare solo questo senza fix)
  • confermi

la problematica si presenta lo stesso? perché a logica non vedo come la fix proposta possa cambiare questo comportamento

@primes2h
Copy link
Contributor

quando scala esattamente il valore? perché mi viene in mente che potrebbe scalarlo più volte anche se ne hai solo uno se fai draft -> approva

No, se rimetti la fattura in bozza l'importo disponibile della DI viene ripristinato.

nel video test con due che hai fatto perché non lo fa?

Nel video la fattura viene messa in bozza, cambiata la DI 1 con la 2 e poi riconfermata. Non si vede come cambiano gli importi delle DI mentre la fattura è in bozza.
Se provi a fare un test con runboat puoi verificare anche tu che l'importo viene correttamente ripristinato.

A me sembra un bug differente,

Mi sa che non ci siamo capiti. :-)

Il fatto di ripristinare l'importo dopo aver messo la fattura in bozza è il comportamento corretto, non è un bug.

L'unico bug che ho rilevato è che sostituendo una DI con un'altra (che questa patch consente di fare), l'importo viene scalato da entrambe le dichiarazioni di intento (anche da quella che è stata tolta).

#4587 (review)

@sherpya
Copy link
Member Author

sherpya commented Jan 31, 2025

quando scala esattamente il valore? perché mi viene in mente che potrebbe scalarlo più volte anche se ne hai solo uno se fai draft -> approva

No, se rimetti la fattura in bozza l'importo disponibile della DI viene ripristinato.

nel video test con due che hai fatto perché non lo fa?

Nel video la fattura viene messa in bozza, cambiata la DI 1 con la 2 e poi riconfermata. Non si vede come cambiano gli importi delle DI mentre la fattura è in bozza.
Se provi a fare un test con runboat puoi verificare anche tu che l'importo viene correttamente ripristinato.

A me sembra un bug differente,

Mi sa che non ci siamo capiti. :-)

Il fatto di ripristinare l'importo dopo aver messo la fattura in bozza è il comportamento corretto, non è un bug.

L'unico bug che ho rilevato è che sostituendo una DI con un'altra (che questa patch consente di fare), l'importo viene scalato da entrambe le dichiarazioni di intento (anche da quella che è stata tolta).

#4587 (review)

probabilmenente non ho capito ma se metti in bozza non ritorna l'amount alla dichiarazione di intenti?
se ne metti un'altra eliminando quella prima perché dovrebbe scalare da quella prima?
non vengono sottratti nel momento in cui si fa conferma prendendo le dichiarazioni da invoice.declaration_of_intent_ids ?
Quello che dico è che se lo fa con la fix, perché non lo fa fa se aggiungi due volte? c'è qualche altro field che va aggiornato?

@sherpya
Copy link
Member Author

sherpya commented Jan 31, 2025

a proposito sto guardando le altre fix sulle versioni precedenti (simili a questa) ma sono andate chiuse per inattività

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

l10n_it_declaration_of_intent: strano funzionamento della selezione dichiarazione intenti
3 participants