Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Non ci sono già le record rules che mostrano solo i record della company corrente?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Voi dite questa:
?
Magari capisco male io ma qui company_id viene matchato in company_ids, ossia le company alle quali l'utente ha accesso, come permessi. Nella issue in questione, l'utente ha effettivamente accesso a più company.
Il problema qui non è di permessi, ma il fatto che viene beccato il primo conto con code _("MISC") ossia "VARIE" per noi, indipendentemente dalla company sulla quale si sta operando. Se l'utente ha accesso ad una sola company, o se per caso il conto che becca è quello giusto, il problema non si manifesta.
Dobbiamo trovare il conto giusto, quello della company corrente su cui l'iutente sta operando.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok quindi l'utente ha accesso a più company quindi vengono beccati tutti i sezionali delle company cui può accedere.
Onestamente ricordavo che la rule facesse vedere solo i sezionali dell'azienda dell'utente (ma è in v12: https://github.com/odoo/odoo/blob/49eb24a46771792bec454f6c4693c19d2b5b05b6/addons/account/security/account_security.xml#L105-L110).
In questo caso secondo me va bene filtrare per la company corrente ma dovresti usare
self.env.company
perché durante il calcolo dei default,self
non ha dati al suo interno; o meglio: dipendono solo dagli altri default https://github.com/odoo/odoo/blob/72fea2fe11244a55e9a630049fc75f147e26e473/odoo/models.py#L1263-L1328 ma per semplicità lo supporrei senza datiperché https://github.com/odoo/odoo/blob/72fea2fe11244a55e9a630049fc75f147e26e473/odoo/models.py#L1263-L1328 è unapi.model