Daily updates from Odoo
Tuesday, August 21, 2018
7 changes · master
Resolved issues and error corrections
This fix changes accounting reconciliation so it uses the company on the transaction line to determine currency, rather than looking through the related account's company. This removes an unnecessary detour that could block downstream customizations where access to the line's company is available but access to the account's company is not.
Original PR description
- [x] Test failure unrelated This indirection is the only place in the whole codebase, which indirects in this way. This block downstream code which relies on: - access to the line's company - while…
- [x] Test failure unrelated This indirection is the only place in the whole codebase, which indirects in this way. This block downstream code which relies on: - access to the line's company - while not necesarily having access to the account's company (being line's company different than account's company) Disclaimer: This use case is not officially supported. However, this indirection does not make sense under the currently supported model, neither. **Description of the issue/feature this PR addresses:** - Downstream module dependency - Unnecessary indirection **Current behavior before PR:** - The currency is taken from the line's account's company **Desired behavior after PR is merged:** - The currency is taken from the line's company, directly **Note:** Please do not close, even if not immediately accepted (`wishlist` tag). This PR is meant to be carried along and actively maintained, until this semantic will be completed this way or another. -- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr @xoe-labs
This fixes an issue in the website editor where custom text or background colors chosen from the color picker were not applied correctly to the editor buttons. It also improves color suggestions by recognizing colors already used on the page, making editing more consistent for users.
Original PR description
Description of the issue/feature this PR addresses: related to task: https://www.odoo.com/web?#id=37974&action=333&active_id=131&model=project.task&view_type=form&menu_id=4720 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
Voucher-generated accounting entries now retain the selected product and quantity, matching the behavior users expect from invoices. This helps businesses analyze costs and revenues by product when vouchers are used for smaller expenses or similar transactions.
Original PR description
- [x] Test failure unrelated - [x] Manual QA locally, can only be validated on an ee runbot (eg. by group journal entries by product) Possibly as an oversight, before this commit, product_id was not…
- [x] Test failure unrelated - [x] Manual QA locally, can only be validated on an ee runbot (eg. by group journal entries by product) Possibly as an oversight, before this commit, product_id was not passed to the aml on account vouchers. This caused inexpected behaviour, if products are a relevant dimension in cost or revenue analysis analysis: Whereas on invoices, this product is copied to aml on voucher, it is not. Now products are copied to the amls thereby enabling this reporting dimension. This is especially usefull for people who use vouchers for minor expenses, but need some additional visibility to what general or analytic accounts provide. **Description of the issue/feature this PR addresses:** - Products are not available on voucher generated amls - This is a missing analytic dimension as compared to normal invoices **Current behavior before PR:** - Products are not copied on vouchers **Desired behavior after PR is merged:** - As vouchers permit to choose a product - Users expects information not to be lost on aml as compared to similar behavior on invoices. **Note:** Please do not close, even if not immediately accepted (`wishlist` tag). This PR is meant to be carried along and actively maintained, until this semantic will be completed this way or another. -- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr @xoe-labs
This fixes an error that prevented companies from installing the Mexican chart of accounts. The correction restores the expected setup flow for Mexican accounting localization, helping users complete configuration without a crash.
Original PR description
The name of **_prepare_transfer_account** method was changed by **_prepare_transfer_account_for_direct_creation**…
The name of **_prepare_transfer_account** method was changed by **_prepare_transfer_account_for_direct_creation** [here](https://github.com/odoo/odoo/commit/87f0d2eefb77bfc6a9a0fa7f7dcd1475c7c34639#diff-ce53c5491cb2c9ab3dbbd41ecabf594f)
But, I think by mistake was changed the name of **_prepare_transfer_account** method by **_get_default_banck_journals_data** and that it's wrong and occurs the following error when I try install **l10n_mx** module:
``` python
File "/.repo_requirements/odoo/odoo/tools/convert.py", line 750, in parse
self._tags[rec.tag](rec, de, mode=mode)
File "/.repo_requirements/odoo/odoo/tools/convert.py", line 342, in _tag_function
_eval_xml(self, rec, env)
File "/.repo_requirements/odoo/odoo/tools/convert.py", line 193, in _eval_xml
return odoo.api.call_kw(model, method, args, {})
File "/.repo_requirements/odoo/odoo/api.py", line 749, in call_kw
return _call_kw_multi(method, model, args, kwargs)
File "/.repo_requirements/odoo/odoo/api.py", line 736, in _call_kw_multi
result = method(recs, *args, **kwargs)
File "<decorator-gen-174>", line 2, in try_loading_for_current_company
File "/.repo_requirements/odoo/odoo/api.py", line 372, in loop
result = [method(rec, *args, **kwargs) for rec in self]
File "/.repo_requirements/odoo/odoo/api.py", line 372, in <listcomp>
result = [method(rec, *args, **kwargs) for rec in self]
File "/home/odoo/odoo-master/addons/account/models/chart_template.py", line 170, in try_loading_for_current_company
self.load_for_current_company(15.0, 15.0)
File "/home/odoo/enterprise/account_reports/models/chart_template.py", line 9, in load_for_current_company
res = super(AccountChartTemplate, self).load_for_current_company(sale_tax_rate, purchase_tax_rate)
File "/home/odoo/odoo-master/addons/account/models/chart_template.py", line 240, in load_for_current_company
self._create_bank_journals(company, acc_template_ref)
File "/home/odoo/odoo-master/addons/payment/models/chart_template.py", line 11, in _create_bank_journals
res = super(AccountChartTemplate, self)._create_bank_journals(company, acc_template_ref)
File "/home/odoo/odoo-master/addons/account/models/chart_template.py", line 311, in _create_bank_journals
for acc in self._get_default_bank_journals_data():
File "/home/odoo/odoo-master/addons/account/models/chart_template.py", line 311, in _create_bank_journals
for acc in self._get_default_bank_journals_data():
File "/usr/lib/python3.5/bdb.py", line 48, in trace_dispatch
return self.dispatch_line(frame)
File "/usr/lib/python3.5/bdb.py", line 67, in dispatch_line
if self.quitting: raise BdbQuit
odoo.tools.convert.ParseError: "" while parsing /home/odoo/odoo-master/addons/l10n_mx/data/account_chart_template_data.xml:4, near
<function model="account.chart.template" name="try_loading_for_current_company">
<value eval="[ref('l10n_mx.mx_coa')]"/>
</function>
2018-08-17 17:26:25,632 3917 INFO test odoo.service.server: Initiating shutdown
2018-08-17 17:26:25,633 3917 INFO test odoo.service.server: Hit CTRL-C again or send a second signal to force the shutdown.
```:
--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-prPurchase orders created from purchase agreements now handle vendor names according to the agreement type. Blanket orders keep the agreed vendor locked, while calls for tender allow users to choose or change the vendor during the request-for-quotation stage.
Original PR description
…se agreements. Task link : https://www.odoo.com/web?debug#id=1866975&action=327&model=project.task&view_type=form&menu_id=4720 Pad link : https://pad.odoo.com/p/r.42624a44d71b668ebc959b5243a9f646 This commit is related to task ID 1866975. 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
Users can now return using the breadcrumb after opening translations from a template editor. This prevents a navigation dead end and makes multilingual template editing smoother.
Original PR description
Issue-Link: https://www.odoo.com/web#id=1869397&action=333&active_id=1278&model=project.task&view_type=form&menu_id=4720 Pad-Link: https://pad.odoo.com/p/r.2cb72c3b77d36a10b27fffc195e08e03 Before this commit: User was not be able to go back through breadcrumb after below steps: - enable another language to have access to translations. - go in Settings - Technical - Template. - select a template then Edit. - click on the earth icon (translation) next to the body - click on the previous link in the breadcrumb After this commit: on_reverse_breadcrumb if condition has been improved. Issue-ID: 1869397 -- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
The Partner Ledger report menu now displays its dropdown option correctly. This prevents a small layout issue when users open the report, making navigation clearer and more consistent.
Original PR description
Task ID: 1866900 Pad : https://pad.odoo.com/p/r.9009e7ab618ac76d4442905d98645fdc