l10n_it_declaration_of_intent: strano funzionamento della selezione dichiarazione intenti

[sherpya]

Nella form per aggiungere manualmente le dichiarazioni di intenti ad una fattura, l'action python aggiunge sempre (comando 4)
quindi se ho sbagliato a scegliere non posso più eliminare un elemento dalla selezione, il codice in l10n_it_declaration_of_intent/wizard/select_declarations.py:

    def confirm(self):
        self.ensure_one()
        res = True
        # Link declaration to invoice
        invoice_id = self.env.context.get("active_id", False)
        if not invoice_id:
            return res
        invoice = self.env["account.move"].browse(invoice_id)
        for declaration in self.declaration_ids:
            invoice.declaration_of_intent_ids = [(4, declaration.id)]
        return True

io lo modificherei così, che ne pensate?

    def confirm(self):
        self.ensure_one()
        res = True
        # Link declaration to invoice
        invoice_id = self.env.context.get("active_id", False)
        if not invoice_id:
            return res
        invoice = self.env["account.move"].browse(invoice_id)
        declaration_of_intent_ids = [(5, 0)]
        for declaration in self.declaration_ids:
            declaration_of_intent_ids.append((4, declaration.id))
        invoice.declaration_of_intent_ids = declaration_of_intent_ids
        return True

(anche se preferisco usare Command.clear() Command.create())

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

[francesco-ooops]

@sherpya grazie della segnalazione! Riusciresti a indicare gli step funzionali per ripetere l'errore nel runboat?

qual è il risultato corrente e quale quello atteso?

[francesco-ooops]

Inoltre in quale versione si presenta il problema?

[8080git]

Con runboat, se provo prima di tutto a mettere un "limit amount" nella company mi da errore perchè non ho i permessi: Due to security restrictions, you are not allowed to create 'Yearly limit for declarations' (l10n_it_declaration_of_intent.yearly_limit) records.

Ma detto questo il bug riscontrato è sulla ver. 16.
Una volta che ti trovi nella fattura con una lettera di intenti associata, e vuole cambiarla con un altra, per quel partner, ti aggiunge ma non ti fa sganciare quella precedente, in sostanza a aggiunge sempre e non riesci ad eliminare una selezionata in precedenza magari erroneamente.

[francesco-ooops]

@8080git testato ora su runboat, con utente admin riesco ad impostare il limite nella company:

[8080git]

questo mi serviva per riprodurre la problematica sulla lettera d'intenti, non è un bug.

[primes2h]

@francesco-ooops
È un problema noto che si trascina ancora dalla v.12.0. Esistono altre issue/PR a riguardo, anche se la problematica non è mai stata tracciata puntualmente.

#2333
#2437
#3310
#3426
#3430
#3498

[sherpya]

@francesco-ooops È un problema noto che si trascina ancora dalla v.12.0. Esistono altre issue/PR a riguardo, anche se la problematica non è mai stata tracciata puntualmente.

#2333 #2437 #3310 #3426 #3430 #3498

la fix che ho proposto mi sembra safe, la stiamo usando in produzione, che ne dite?

[francesco-ooops]

@sherpya apri pure una PR e facciamo fare review sia funzionale che di codice

La cosa importante è che la issue di tracciamento sia chiara, per questo domandavo cosa andiamo a testare