Vauxoo / odoo

Fork of Odoo (formerly OpenERP). [This project is not publically mantained just born for internal usage with some little patches] go to official repository on github.com/odoo/odoo
https://www.odoo.com
Other
9 stars 9 forks source link

[FIX] account: error in caba entry with stock valuation account #393

Closed hbto closed 2 years ago

hbto commented 3 years ago

Why this MR?

This PR was create only as last result to avoid odoo's new changes that are breaking stable feature. At the time of creating this PR this commit was already in stable, @nhomar suggested to only cherry-pick the needed fix, and later updating one step at a time the next commits in stable until deleting this PR.

Main

Before this commit, the cash basis journal entry was created with an incorrect amount of 0.05 of Base Tax Received Account.

Now, the cash basis journal entry is created with the amount paid of 13095.65 of Base Tax Received Account

fine-tuning of e25f857e672a645bfc46a9bac9b871c107b0a8db

opw-2411516

closes odoo/odoo#63738

Signed-off-by: Jorge Pinna Puissant (jpp) jpp@odoo.com Co-authored-by: Andrea Grazioso agr@odoo.com

Description of the issue/feature this PR addresses:

Current behavior before PR:

Desired behavior after PR is merged:

-- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

luisg123v commented 3 years ago

Since vauxoo:12.0 has been updated, the commit cherry-picked here is now included into the stable branch, so this patch is no longer required. Hence closing.

hbto commented 3 years ago

@luisg123v This MR must not be close There are other projects that depend on it.

Regards.