[16.0][FIX] l10n_it_declaration_of_intent: doppio scarico fattura #3467

Borruso · 2023-07-07T11:13:02Z

Risolve #3116

--
Confermo di aver firmato il CLA https://odoo-community.org/page/cla e di aver letto le linee guida su https://odoo-community.org/page/contributing

scigghia · 2023-09-15T08:26:33Z

@Borruso ma questa PR è per la 14 o 16 ? perchè nel titolo c'è 14.0 ma il merge evidenzia il repo 16
tnx

apolitano27 · 2023-09-15T09:45:02Z

@Borruso dai miei test su Odoo 16EE l'obiettivo preposto viene raggiunto, ovvero l'importo non viene più scaricato da entrambe le dichiarazioni d'intento.
Tuttavia, se l'importo della fattura nella prima dichiarazione non è sufficiente, il comportamento atteso sarebbe quello di scaricare l'importo residuo dalla prima dichiarazione e il restante dalla seconda dichiarazione. Allo stato attuale, se l'importo sulla prima dichiarazione non è sufficiente a chiudere la fattura, alla conferma l'operazione viene bloccata e si impedisce di proseguire.

Borruso · 2023-09-15T15:44:09Z

@Borruso ma questa PR è per la 14 o 16 ? perchè nel titolo c'è 14.0 ma il merge evidenzia il repo 16 tnx

scusi presidente

SirAionTech

Grazie della PR!
Grande che hai aggiunto il test!
Ho fatto solo revisione del codice

P.S. puoi modificare il messaggio del commit in modo che rispetti le linee guida OCA (rif. https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#71commit-message)?
Ad esempio il primo punto è:

Commit messages are in English

SirAionTech · 2023-12-07T11:40:05Z

l10n_it_declaration_of_intent/models/account_move.py

+    def update_declarations(
+        self, declarations, grouped_lines, declarations_used_amounts
+    ):


Suggested change

def update_declarations(

self, declarations, grouped_lines, declarations_used_amounts

):

def update_declarations(

self, declarations_used_amounts, grouped_lines

):

declarations non è più usato dal metodo e comunque le sue informazioni sono recuperabili da declarations_used_amounts

SirAionTech · 2023-12-07T11:42:45Z

l10n_it_declaration_of_intent/models/account_move.py

-                    declarations = [force_declaration]
+                    declarations = {force_declaration.id: amount}
                else:
-                    declarations = declarations
+                    declarations = declarations_used_amounts


La variabile declarations non sono più semplici dichiarazioni ma un dizionario, si può chiarire nel nome della variabile?
Ad esempio declaration_id_to_amount_dict

SirAionTech · 2023-12-07T11:48:32Z

l10n_it_declaration_of_intent/tests/test_declaration_of_intent.py

+        records = declaration_model.get_valid(
+            type_d="out", partner_id=self.partner2.id, date=self.today_date
+        )
+        self.assertEqual(len(records), 2)


Suggested change

records = declaration_model.get_valid(

type_d="out", partner_id=self.partner2.id, date=self.today_date

)

self.assertEqual(len(records), 2)

valid_declarations = declaration_model.get_valid(

type_d="out", partner_id=self.partner2.id, date=self.today_date

)

self.assertEqual(valid_declarations, self.declaration2 | self.declaration3)

o almeno dare un nome più significativo invece di records.

SirAionTech · 2023-12-07T11:54:02Z

l10n_it_declaration_of_intent/tests/test_declaration_of_intent.py

@@ -503,3 +503,22 @@ def test_action_register_payment(self):
        else:
            payments = action["domain"][0][2]
            self.assertTrue(len(payments) > 1)
+
+    def test_multiple_valid(self):
+        declaration_model = self.env["l10n_it_declaration_of_intent.declaration"].sudo()


Puoi aggiungere una breve descrizione di cosa deve verificare questo test?

Borruso · 2023-12-11T08:07:00Z

@SirAionTech fatto puoi controllare?

SirAionTech

Revisione del codice, per me è ok.

Grazie di aver tradotto il messaggio del commit, ora però viene tagliato

P.S. puoi modificare il messaggio del commit in modo che rispetti le linee guida OCA (rif. https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#71commit-message)?
Ad esempio il primo punto è:

Commit messages are in English

C'è anche:

please check if the commit message is cut with ellipsis

SirAionTech

Revisione del codice, per me è ok.

SirAionTech · 2023-12-13T09:03:18Z

@Borruso dai miei test su Odoo 16EE l'obiettivo preposto viene raggiunto, ovvero l'importo non viene più scaricato da entrambe le dichiarazioni d'intento. Tuttavia, se l'importo della fattura nella prima dichiarazione non è sufficiente, il comportamento atteso sarebbe quello di scaricare l'importo residuo dalla prima dichiarazione e il restante dalla seconda dichiarazione. Allo stato attuale, se l'importo sulla prima dichiarazione non è sufficiente a chiudere la fattura, alla conferma l'operazione viene bloccata e si impedisce di proseguire.

@apolitano27 grazie di aver provato le modifiche, puoi evidenziare se secondo te la PR va bene così o sono necessarie modifiche?
In https://www.odoo-italia.org/documentazione/14.0/sviluppo/review.html trovi una breve guida di come fare una revisione in Github, alla fine potrai decidere se approvare (Approve) o richiedere modifiche (Request changes).

github-actions · 2024-05-19T12:28:35Z

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

SirAionTech · 2024-06-26T09:45:16Z

/ocabot rebase

OCA-git-bot · 2024-06-26T09:45:25Z

Congratulations, PR rebased to 16.0.

SirAionTech · 2024-06-26T09:48:49Z

@sergiocorato il commit di questa PR ha te come autore, vuoi controllare che sia tutto ok?

MaurizioPellegrinet

Funtional test: OK

OCA-git-bot · 2024-06-26T11:00:01Z

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

SirAionTech · 2024-07-09T10:29:29Z

/ocabot merge patch

OCA-git-bot · 2024-07-09T10:29:39Z

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-3467-by-SirAionTech-bump-patch, awaiting test results.

Signed-off-by SirAionTech

OCA-git-bot · 2024-07-09T10:52:07Z

It looks like something changed on 16.0 in the meantime.
Let me try again (no action is required from you).
Prepared branch 16.0-ocabot-merge-pr-3467-by-SirAionTech-bump-patch, awaiting test results.

OCA-git-bot · 2024-07-09T11:16:15Z

Congratulations, your PR was merged at 012e8a0. Thanks a lot for contributing to OCA. ❤️

Borruso changed the title ~~[14.0][FIX] l10n_it_declaration_of_intent: doppio scarico fattura~~ [16.0][FIX] l10n_it_declaration_of_intent: doppio scarico fattura Sep 15, 2023

francesco-ooops mentioned this pull request Sep 15, 2023

Doppio scarico dichiarazione d'intento in caso di più dichiarazioni valide per lo stesso partner #3116

Open

3 tasks

Borruso force-pushed the 16.0-fix-l10n_it_declaration_of_intent-doppio-utilizzo branch from e878803 to e3df2d4 Compare September 15, 2023 16:24

Borruso force-pushed the 16.0-fix-l10n_it_declaration_of_intent-doppio-utilizzo branch from e3df2d4 to d392f0d Compare November 17, 2023 10:44

SirAionTech requested changes Dec 7, 2023

View reviewed changes

Borruso force-pushed the 16.0-fix-l10n_it_declaration_of_intent-doppio-utilizzo branch from d392f0d to de86906 Compare December 11, 2023 08:06

Borruso requested a review from SirAionTech December 11, 2023 08:06

Borruso force-pushed the 16.0-fix-l10n_it_declaration_of_intent-doppio-utilizzo branch from de86906 to 716fdca Compare December 11, 2023 08:12

SirAionTech approved these changes Dec 12, 2023

View reviewed changes

Borruso force-pushed the 16.0-fix-l10n_it_declaration_of_intent-doppio-utilizzo branch from 716fdca to 3e51527 Compare December 13, 2023 07:23

Borruso requested a review from SirAionTech December 13, 2023 07:23

SirAionTech approved these changes Dec 13, 2023

View reviewed changes

francesco-ooops added the 16.0 label Dec 20, 2023

francesco-ooops linked an issue Dec 20, 2023 that may be closed by this pull request

Doppio scarico dichiarazione d'intento in caso di più dichiarazioni valide per lo stesso partner #3116

Open

3 tasks

github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label May 19, 2024

[FIX] l10n_it_declaration_of_intent: amounts declarations validation

6785350

OCA-git-bot force-pushed the 16.0-fix-l10n_it_declaration_of_intent-doppio-utilizzo branch from 3e51527 to 6785350 Compare June 26, 2024 09:45

MaurizioPellegrinet approved these changes Jun 26, 2024

View reviewed changes

OCA-git-bot added approved ready to merge labels Jun 26, 2024

OCA-git-bot added a commit that referenced this pull request Jul 9, 2024

Merge PR #3467 into 16.0

d1dbeaf

Signed-off-by SirAionTech

OCA-git-bot added the bot is merging ⏳ label Jul 9, 2024

OCA-git-bot merged commit 284f68e into OCA:16.0 Jul 9, 2024
7 checks passed

OCA-git-bot added merged 🎉 and removed bot is merging ⏳ labels Jul 9, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[16.0][FIX] l10n_it_declaration_of_intent: doppio scarico fattura #3467

[16.0][FIX] l10n_it_declaration_of_intent: doppio scarico fattura #3467

Borruso commented Jul 7, 2023

scigghia commented Sep 15, 2023

apolitano27 commented Sep 15, 2023 •

edited

Loading

Borruso commented Sep 15, 2023

SirAionTech left a comment •

edited

Loading

SirAionTech Dec 7, 2023

SirAionTech Dec 7, 2023

SirAionTech Dec 7, 2023

SirAionTech Dec 7, 2023

Borruso commented Dec 11, 2023

SirAionTech left a comment

SirAionTech left a comment

SirAionTech commented Dec 13, 2023

github-actions bot commented May 19, 2024

SirAionTech commented Jun 26, 2024

OCA-git-bot commented Jun 26, 2024

SirAionTech commented Jun 26, 2024

MaurizioPellegrinet left a comment

OCA-git-bot commented Jun 26, 2024

SirAionTech commented Jul 9, 2024

OCA-git-bot commented Jul 9, 2024

OCA-git-bot commented Jul 9, 2024

OCA-git-bot commented Jul 9, 2024

[16.0][FIX] l10n_it_declaration_of_intent: doppio scarico fattura #3467

[16.0][FIX] l10n_it_declaration_of_intent: doppio scarico fattura #3467

Conversation

Borruso commented Jul 7, 2023

scigghia commented Sep 15, 2023

apolitano27 commented Sep 15, 2023 • edited Loading

Borruso commented Sep 15, 2023

SirAionTech left a comment • edited Loading

Choose a reason for hiding this comment

SirAionTech Dec 7, 2023

Choose a reason for hiding this comment

SirAionTech Dec 7, 2023

Choose a reason for hiding this comment

SirAionTech Dec 7, 2023

Choose a reason for hiding this comment

SirAionTech Dec 7, 2023

Choose a reason for hiding this comment

Borruso commented Dec 11, 2023

SirAionTech left a comment

Choose a reason for hiding this comment

SirAionTech left a comment

Choose a reason for hiding this comment

SirAionTech commented Dec 13, 2023

github-actions bot commented May 19, 2024

SirAionTech commented Jun 26, 2024

OCA-git-bot commented Jun 26, 2024

SirAionTech commented Jun 26, 2024

MaurizioPellegrinet left a comment

Choose a reason for hiding this comment

OCA-git-bot commented Jun 26, 2024

SirAionTech commented Jul 9, 2024

OCA-git-bot commented Jul 9, 2024

OCA-git-bot commented Jul 9, 2024

OCA-git-bot commented Jul 9, 2024

apolitano27 commented Sep 15, 2023 •

edited

Loading

SirAionTech left a comment •

edited

Loading