Skip to content

[14.0] [FIX] l10n_it_withholding_tax: fix for #2345 + multicompany#2353

Merged
OCA-git-bot merged 1 commit into
OCA:14.0from
TheMule71:14.0-fix-l10n_it_withholding_tax-multicompany
Sep 17, 2021
Merged

[14.0] [FIX] l10n_it_withholding_tax: fix for #2345 + multicompany#2353
OCA-git-bot merged 1 commit into
OCA:14.0from
TheMule71:14.0-fix-l10n_it_withholding_tax-multicompany

Conversation

@TheMule71

@TheMule71 TheMule71 commented Jul 2, 2021

Copy link
Copy Markdown
Contributor

Usa self.env.company invece di self.env.user.company_id. La seconda, l'azienda di default dell'utente collegato, raramente è quella giusta da usare, visto che comunque si opera nel contesto dell'azienda indicata da self.env.company.

Questa fix fa parte di una serie di fix relative alla gestione del multicompany.

Edit: aggiunto anche fix per la #2345 (era #2351)

Edit2: da verificare se dipende da #2352

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

@TheMule71 TheMule71 force-pushed the 14.0-fix-l10n_it_withholding_tax-multicompany branch from a5637a4 to 4d63ebf Compare July 2, 2021 11:50
currency = self.env["res.currency"].browse(self.env.context["currency_id"])
else:
currency = self.env.user.company_id.currency_id
currency = self.env.company

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Manca currency_id mi pare

@TheMule71 TheMule71 Jul 2, 2021

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

yes
grazie, sistemato

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@TheMule71 TheMule71 changed the title [FIX] l10n_it_withholding_tax: multicompany [14.0] [FIX] l10n_it_withholding_tax: multicompany Jul 2, 2021
@TheMule71 TheMule71 force-pushed the 14.0-fix-l10n_it_withholding_tax-multicompany branch from 4d63ebf to 7960409 Compare July 2, 2021 12:35
@TheMule71 TheMule71 force-pushed the 14.0-fix-l10n_it_withholding_tax-multicompany branch from 7960409 to ebafaf8 Compare July 2, 2021 14:09
@TheMule71 TheMule71 changed the title [14.0] [FIX] l10n_it_withholding_tax: multicompany [14.0] [FIX] l10n_it_withholding_tax: fix for #2345 + multicompany Jul 2, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 2, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 2, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 17, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 20, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 30, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 30, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Aug 6, 2021
@francescapenso

Copy link
Copy Markdown

provata e funzionante

TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Aug 20, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Sep 3, 2021
@TheMule71

Copy link
Copy Markdown
Contributor Author

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-2353-by-TheMule71-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 67c265c into OCA:14.0 Sep 17, 2021
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@TheMule71 TheMule71 deleted the 14.0-fix-l10n_it_withholding_tax-multicompany branch September 17, 2021 09:06
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Sep 17, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Sep 17, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 1, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 1, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 8, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 8, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 15, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 15, 2021
TheMule71 added a commit to TheMule71/l10n-italy that referenced this pull request Jan 21, 2022
TheMule71 added a commit to TheMule71/l10n-italy that referenced this pull request Jan 21, 2022
TheMule71 added a commit to TheMule71/l10n-italy that referenced this pull request Jan 21, 2022
TheMule71 added a commit to TheMule71/l10n-italy that referenced this pull request Jan 21, 2022
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 29, 2022
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants