[16.0][MIG] l10n_it_delivery_note by Borruso · Pull Request #3020 · OCA/l10n-italy

Borruso · 2022-11-10T15:03:31Z

Migration l10n_it_delivery_note from 14.0 to 16.0
Dipende da #3019

--
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

tafaRU · 2022-11-18T09:19:30Z

@Borruso devi aggiungere nel file test-requirements.txt: #3019
In questo modo sia i tests che runboat dovrebbero diventare 🟢

Borruso · 2022-11-18T09:53:48Z

@Borruso devi aggiungere nel file test-requirements.txt: #3019 In questo modo sia i tests che runboat dovrebbero diventare green_circle

provato a inserire il file .txt mi dici se ho fatto giusto o sbagliato qualcosa?

tafaRU · 2022-11-18T10:10:39Z

dici se ho fatto giusto o sbagliato qualcosa?

la sintassi non è giusta, puoi prendere spunto da 42641f0
altra cosa: aggiungilo in un commit separato così lo eliminerai una volta mergiata la PR dipendenza.

Grazie.

MarcoCalcagni · 2022-11-25T11:48:58Z

Test Funzionali
in questa form/tab
qualsiasi cosa inserisco non viene salvata

stampa ddt i numeri vanno allineati a dx. sia qty che prezzo.

manca l'azione che consente di fatturare dalla tree ddt

in fattura, la nota "DDt xx del xx " dovrebbe essere inserita una sola volta e non su tutte le righe.
inoltre se ho un ddt con 2 picking, lo stesso prodotto è presente nei due picking, quando creo la fattura aggiunge 2 volte la riga nota "DDt xx del xx "

per i trasferimenti interno tra 2 magazzini della stessa company la form è disallineata

nel report

MarcoCalcagni · 2022-11-25T12:04:46Z

Riflessione funzionale.

odoo installa automaticamente l10n_it_stock_ddt se poi si installa l10n_it_delivery_note nella form ci sono 2 button per generare il report che chiamano modelli diversi di report. Questa cosa più generare confusione per gli user.
Quindi suggerisco di :
inserire incompatibilità tra i 2 moduli
oppure
visto, che il modulo l10n_it_stock_ddt la funzione di l10n_it_delivery_note non avanzata, di escludere questa modalità nel delivery note. chi ha bisogno un pick un ddt userà il modulo std di odoo che invece deve accorpare più picking in un ddt userà il modulo delivery note.

TonyMasciI · 2022-11-25T09:46:05Z

+#~ "confermato."
+
+#~ msgid "Notify if delivery note isn't validated"
+#~ msgstr "Notifica se il DdT non è validato"


Come mai sono commentati questi termini?

TonyMasciI · 2022-11-25T10:28:57Z

+
+        changed |= self._update_generic_shipping_information(partner)
+
+        return changed


Perché i campi:

transport_condition_id

goods_appearance_id

transport_reason_id

transport_method_id

delivery_method_id

non sono definiti qui nel Mixin e ogni volta nei singoli Model che ereditano questo Mixin?

TonyMasciI · 2022-11-25T14:31:23Z

+            <!--<p>-->
+                <!--Bla, bla, bla...-->
+            <!--</p>-->
+        <!--</field>-->


O si elimina o si scrive la descrizione

TonyMasciI · 2022-11-25T14:39:32Z

+        return res
+
+
+class AccountInvoiceLine(models.Model):


Non ha senso portare iu un file tutto suo la classe?

TonyMasciI · 2022-11-25T14:47:51Z

+    default_transport_condition_id = fields.Many2one(
+        "stock.picking.transport.condition",
+        string="Condition of transport",
+        default=False,


Non penso sia necessario il defaul in questa definizione

TonyMasciI · 2022-11-25T14:52:05Z

+        return action
+
+
+class SaleOrderLine(models.Model):


Sposterei in un altro file dedicato al model

TonyMasciI · 2022-11-25T15:19:17Z

+        return location_address
+
+
+class StockDeliveryNoteLine(models.Model):


Farei un'altro file dedicato a questo Model

TonyMasciI · 2022-11-25T21:40:40Z

+                validator(pickings)
+
+            except ValidationError as exc:
+                errors.append(exc.name)


UserError attribute 'name' is a deprecated alias to args[0]

TonyMasciI · 2022-11-25T23:00:24Z

Inoltre avviato il test con questo comando:

--config=PATH\odoo.conf --test-tags=/l10n_it_delivery_note --init=l10n_it_delivery_note --database=NOME_DATABASE --test-enable --stop-after-init

Questo è il risultato:

0 failed, 5 error(s) of 7 tests when loading database 'NOME_DATABASE'

Con questi errori:

odoo.exceptions.UserError: No journal could be found in company My Company (San Francisco) for any of those types: sale
odoo.exceptions.UserError: No journal could be found in company My Company (San Francisco) for any of those types: sale
odoo.exceptions.UserError: No journal could be found in company My Company (San Francisco) for any of those types: sale
odoo.exceptions.UserError: No journal could be found in company My Company (San Francisco) for any of those types: sale
odoo.exceptions.UserError: No journal could be found in company My Company (San Francisco) for any of those types: sale

andando in errore in questa linea

Borruso · 2022-12-02T11:11:30Z

Inoltre avviato il test con questo comando:

--config=PATH\odoo.conf --test-tags=/l10n_it_delivery_note --init=l10n_it_delivery_note --database=NOME_DATABASE --test-enable --stop-after-init

Questo è il risultato:

0 failed, 5 error(s) of 7 tests when loading database 'NOME_DATABASE'

Con questi errori:
* `odoo.exceptions.UserError: No journal could be found in company My Company (San Francisco) for any of those types: sale`

* `odoo.exceptions.UserError: No journal could be found in company My Company (San Francisco) for any of those types: sale`

* `odoo.exceptions.UserError: No journal could be found in company My Company (San Francisco) for any of those types: sale`

* `odoo.exceptions.UserError: No journal could be found in company My Company (San Francisco) for any of those types: sale`

* `odoo.exceptions.UserError: No journal could be found in company My Company (San Francisco) for any of those types: sale`
andando in errore in questa linea

come se il database che stai usando non abbia caricato i dati di test

TonyMasciI · 2022-12-02T22:48:27Z

Inoltre avviato il test con questo comando:
--config=PATH\odoo.conf --test-tags=/l10n_it_delivery_note --init=l10n_it_delivery_note --database=NOME_DATABASE --test-enable --stop-after-init
Questo è il risultato:
0 failed, 5 error(s) of 7 tests when loading database 'NOME_DATABASE'
Con questi errori:
* `odoo.exceptions.UserError: No journal could be found in company My Company (San Francisco) for any of those types: sale`

* `odoo.exceptions.UserError: No journal could be found in company My Company (San Francisco) for any of those types: sale`

* `odoo.exceptions.UserError: No journal could be found in company My Company (San Francisco) for any of those types: sale`

* `odoo.exceptions.UserError: No journal could be found in company My Company (San Francisco) for any of those types: sale`

* `odoo.exceptions.UserError: No journal could be found in company My Company (San Francisco) for any of those types: sale`
andando in errore in questa linea
come se il database che stai usando non abbia caricato i dati di test

In teoria il DB che ho usato è stato generato al momento dell'avvio del comando dato che non esisteva, data la combinazione dei comandi --init, --database e --test-enable come da guida

andreampiovesana · 2022-12-16T12:21:53Z

avete aggiunto le modifiche di #3087 ?

MarcoCalcagni · 2022-12-23T10:56:36Z

avete aggiunto le modifiche di #3087 ?

dopo il merge di questa

MarcoCalcagni · 2022-12-23T11:05:06Z

/ocabot merge nobump

OCA-git-bot · 2022-12-23T11:05:15Z

Sorry @As400it you are not allowed to merge.

To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons.

If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the maintainers key of its manifest.

andreampiovesana

LGTM
tested in staging 16

tafaRU · 2022-12-23T14:05:56Z

I commenti di @TonyMasciI sono stati tutti risolti? Non mi pare di vedere la sua approvazione.

Borruso · 2022-12-23T14:21:18Z

I commenti di @TonyMasciI sono stati tutti risolti? Non mi pare di vedere la sua approvazione.

tutti i commenti sono stati risolti almeno quelli.
per i commenti dei test come risposto solo i dati di demo che non sono presenti.
per il commento del *.po non infliuisce con la PR.
mentre per il discorso del mixin in questa sede non è possibile dato che è una migrazione, ma con un PR successiva con beckport anche alla 12 e 14

TheMule71 · 2022-12-26T10:01:24Z

@Borruso
ti segnalo #3099 per
https://github.com/Borruso/l10n-italy/blob/b35b3940603a4d64034e4bb45ec2f320fdaf45e3/l10n_it_delivery_note/models/stock_delivery_note.py#L466

tafaRU · 2022-12-27T08:10:00Z

tutti i commenti sono stati risolti almeno quelli.

ok grazie @Borruso grazie del feedback, attendiamo comunque l'approvazione di @TonyMasciI prima di procedere con il merge.

…rder Co-authored-by: TheMule71 <marco.colombo@gmail.com>

Currently translated at 100.0% (258 of 258 strings) Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_delivery_note Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_delivery_note/it/

Borruso · 2023-02-10T10:31:22Z

I test sono red_circle @Borruso puoi verificare come mai?

@tafaRU @SimoRubi @TheMule71 @eLBati
l'errore lo da sulla creazione del prodotto nella company2
product2 = self.product_product_10.copy({"company_id": self.company2.id})
se però la copy la facciamo con il with_company(company=self.company2.id)
quindi:
product2 = self.product_product_10.with_company(company=self.company2.id).copy({"company_id": self.company2.id})
oppure
product2 = self.product_product_10.with_company(company=self.company2.id).copy()
in questo caso non da errore
non sono io a causarlo perché nelle funzioni del DdT non passa
forse è cambiato qualcosa su Odoo
oppure nel merge non abbiamo visto il test rosso ma questo lo escludo

tafaRU · 2023-02-10T10:44:54Z

@Borruso applica le modifiche su un commit separato e documentalo bene. Grazie.

Borruso · 2023-02-10T12:11:10Z

@Borruso applica le modifiche su un commit separato e documentalo bene. Grazie.

commit fatto puoi verificare??

tafaRU · 2023-02-10T13:07:19Z

commit fatto puoi verificare??

grazie, meglio se il commit message lo scrivi in inglese: potresti modificarlo?

Borruso · 2023-02-10T14:08:02Z

commit fatto puoi verificare??

grazie, meglio se il commit message lo scrivi in inglese: potresti modificarlo?

fatto

tafaRU · 2023-02-13T14:01:05Z

        "res.partner",
        string="Recipient",
        states=DRAFT_EDITABLE_STATE,
+        default=_default_company,


Qui e nei due campi successivi come mai usi _default_company come default?

errore durante la migrazione correggo

tafaRU · 2023-02-13T16:36:15Z

@Borruso le ultime correzioni non dovrebbero andare a finire in f862812 bensì nel commit di migrazione.

…'with_company()' instead of putting the company in 'copy()' https://github.com/OCA/l10n-italy/actions/runs/4082549051/jobs/7037035918#step:8:381 Traceback (most recent call last): File "/__w/l10n-italy/l10n-italy/l10n_it_fatturapa_out/tests/test_fatturapa_xml_validation.py", line 816, in test_multicompany_fail product2 = self.product_product_10.copy( File "/opt/odoo/addons/product/models/product_product.py", line 393, in copy template = self.product_tmpl_id.copy(default=default) File "/opt/odoo/addons/stock/models/product.py", line 878, in copy res = super().copy(default=default) File "/opt/odoo/addons/product/models/product_template.py", line 474, in copy return super(ProductTemplate, self).copy(default=default) File "/opt/odoo/odoo/models.py", line 4776, in copy record_copy = self.create(vals) File "<decorator-gen-169>", line 2, in create File "/opt/odoo/odoo/api.py", line 409, in _model_create_multi return create(self, [arg]) File "/opt/odoo/addons/product/models/product_template.py", line 428, in create templates = super(ProductTemplate, self).create(vals_list) File "<decorator-gen-142>", line 2, in create File "/opt/odoo/odoo/api.py", line 410, in _model_create_multi return create(self, arg) File "/opt/odoo/addons/mail/models/mail_thread.py", line 258, in create threads = super(MailThread, self).create(vals_list) File "<decorator-gen-65>", line 2, in create File "/opt/odoo/odoo/api.py", line 410, in _model_create_multi return create(self, arg) File "/opt/odoo/odoo/addons/base/models/ir_fields.py", line 665, in create recs = super().create(vals_list) File "<decorator-gen-13>", line 2, in create File "/opt/odoo/odoo/api.py", line 410, in _model_create_multi return create(self, arg) File "/opt/odoo/odoo/models.py", line 3971, in create records._check_company() File "/opt/odoo/odoo/models.py", line 3413, in _check_company raise UserError("\n".join(lines)) odoo.exceptions.UserError: Incompatible companies on records: - 'Cabinet with Doors (copy)' belongs to company 'YourCompany (copy)' and 'Production Location' (property_stock_production: 'Virtual Locations/Production') belongs to another company. - 'Cabinet with Doors (copy)' belongs to company 'YourCompany (copy)' and 'Inventory Location' (property_stock_inventory: 'Virtual Locations/Inventory adjustment') belongs to another company.

tafaRU · 2023-02-14T08:30:28Z

/ocabot merge nobump

OCA-git-bot · 2023-02-14T08:31:23Z

This PR looks fantastic, let's merge it!
Prepared branch 16.0-ocabot-merge-pr-3020-by-tafaRU-bump-nobump, awaiting test results.

OCA-git-bot · 2023-02-14T08:41:44Z

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

This was referenced Nov 10, 2022

Migration to version 16.0 #2967

Open

[16.0] [MIG] l10n_it_delivery_note_order_link #3022

Merged

Borruso force-pushed the 16.0-mig-l10n_it_delivery_note branch from 18363e6 to 63e671d Compare November 11, 2022 10:03

jado95 mentioned this pull request Nov 11, 2022

16.0 mig l10n_it_delivery_note_batch #3032

Closed

Borruso force-pushed the 16.0-mig-l10n_it_delivery_note branch from 63e671d to a3599e1 Compare November 18, 2022 09:32

Borruso force-pushed the 16.0-mig-l10n_it_delivery_note branch from a3599e1 to d899a81 Compare November 18, 2022 10:16

tafaRU added the needs functional review label Nov 25, 2022

tafaRU requested a review from MarcoCalcagni November 25, 2022 08:40

TonyMasciI suggested changes Nov 25, 2022

View reviewed changes

Borruso force-pushed the 16.0-mig-l10n_it_delivery_note branch 3 times, most recently from e91a6da to b35b394 Compare December 2, 2022 11:15

MarcoCalcagni approved these changes Dec 23, 2022

View reviewed changes

andreampiovesana approved these changes Dec 23, 2022

View reviewed changes

fredzamoabg and others added 6 commits February 10, 2023 10:47

[14.0][FIX] l10n_it_delivery_note: force currency in DN line_note

278e158

[14.0][FIX] l10n_it_delivery_note: unable to create DN without sale o…

9babd9d

…rder Co-authored-by: TheMule71 <marco.colombo@gmail.com>

[FIX][l10n_it_delivery_note] fix return super

80bfec6

Translated using Weblate (Italian)

a938547

Currently translated at 100.0% (258 of 258 strings) Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_delivery_note Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_delivery_note/it/

Remove maintainer

24d48e0

[IMP] l10n_it_delivery_note: pre-commit stuff

339aa98

Borruso force-pushed the 16.0-mig-l10n_it_delivery_note branch from 2772af2 to 8bb119c Compare February 10, 2023 12:03

Borruso force-pushed the 16.0-mig-l10n_it_delivery_note branch from 8bb119c to 21e1d6c Compare February 10, 2023 14:07

tafaRU reviewed Feb 10, 2023

View reviewed changes

Comment thread l10n_it_delivery_note/tests/test_stock_delivery_note_invoicing.py

tafaRU reviewed Feb 13, 2023

View reviewed changes

Borruso force-pushed the 16.0-mig-l10n_it_delivery_note branch from 21e1d6c to f862812 Compare February 13, 2023 16:12

Borruso added 2 commits February 14, 2023 09:09

[MIG] l10n_it_delivery_note: Migration to 16.0

f3cff07

Borruso force-pushed the 16.0-mig-l10n_it_delivery_note branch from f862812 to ad4b4b2 Compare February 14, 2023 08:09

OCA-git-bot added the bot is merging ⏳ label Feb 14, 2023

OCA-git-bot merged commit c81f4eb into OCA:16.0 Feb 14, 2023

OCA-git-bot added merged 🎉 and removed bot is merging ⏳ labels Feb 14, 2023

This was referenced Dec 22, 2023

l10n_it_delivery_note: force currency in DN line_note #2700

Closed

l10n_it_delivery_note: fix print depending on picking_type instead of type_code #2692

Closed

francesco-ooops added the migration label Jan 10, 2024


		changed \|= self._update_generic_shipping_information(partner)

		return changed

		return location_address


		class StockDeliveryNoteLine(models.Model):

Uh oh!

Conversation

Borruso commented Nov 10, 2022

Uh oh!

tafaRU commented Nov 18, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Borruso commented Nov 18, 2022

Uh oh!

tafaRU commented Nov 18, 2022

Uh oh!

MarcoCalcagni commented Nov 25, 2022

Uh oh!

MarcoCalcagni commented Nov 25, 2022

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

TonyMasciI commented Nov 25, 2022

Uh oh!

Borruso commented Dec 2, 2022

Uh oh!

TonyMasciI commented Dec 2, 2022

Uh oh!

andreampiovesana commented Dec 16, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

MarcoCalcagni commented Dec 23, 2022

Uh oh!

MarcoCalcagni commented Dec 23, 2022

Uh oh!

OCA-git-bot commented Dec 23, 2022

Uh oh!

andreampiovesana left a comment

Choose a reason for hiding this comment

Uh oh!

tafaRU commented Dec 23, 2022

Uh oh!

Borruso commented Dec 23, 2022

Uh oh!

TheMule71 commented Dec 26, 2022

Uh oh!

tafaRU commented Dec 27, 2022

Uh oh!

Borruso commented Feb 10, 2023

Uh oh!

tafaRU commented Feb 10, 2023

Uh oh!

Borruso commented Feb 10, 2023

Uh oh!

tafaRU commented Feb 10, 2023

Uh oh!

Borruso commented Feb 10, 2023

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

tafaRU commented Feb 13, 2023

Uh oh!

tafaRU commented Feb 14, 2023

Uh oh!

OCA-git-bot commented Feb 14, 2023

Uh oh!

OCA-git-bot commented Feb 14, 2023

Uh oh!

tafaRU commented Nov 18, 2022 •

edited

Loading

andreampiovesana commented Dec 16, 2022 •

edited

Loading