Daily updates from Odoo
Wednesday, December 5, 2018
4 changes · master
Resolved issues and error corrections
This change rolls back a previous settings search and ordering update because it could interfere with saving changes across multiple settings tabs. The rollback prioritizes reliable configuration saving until a safer framework-level solution can be built.
Original PR description
This reverts commit 01173f374581845651495e28ecb2ea04dd83b5f3. This commit added improvements in search and ordering of settings. However due to DOM manipulation some events are not bound correctly. Notably saving settings could be broken if several different tabs are modified before saving. After checking more in-depth settings code it appears it will be difficult to ensure a proper re-rendering and re-ordering of settings without having to modify deeply the code. This is therefore more a task for JS framework team.
This fixes issues that prevented survey walkthrough tests from launching unless another website component happened to be installed first. Survey tours now load the right resources and dependencies, making automated checks more reliable for the Survey app.
Original PR description
Purpose of this merge is basically to allow launching survey tour in browser and command line. Several fixes are necessary to achieve that purpose. See sub commits for more details. This merge is related to task ID 1915356.
Stock movements marked as owned by the same company will now create the expected accounting entries. This helps businesses that record an owner on every stock move keep inventory valuation and financial records accurate.
Original PR description
Before this commit, a user establishing owner for his own compay would have produced no accounting moves. This is not correct, as ownly owned products should indeed create accounting moves. After…
Before this commit, a user establishing owner for his own compay would have
produced no accounting moves.
This is not correct, as ownly owned products should indeed create accounting
moves.
After this commit, "ownly owned" moves do create accounting entries.
**Description of the issue/feature this PR addresses:**
- When own company was recorded as owner of stock move
- for example because of a simplification policy ("all moves have owner")
- Stock moves with an owner did not record accounting entries ...
- ... _even_ if this owner is the same as the move's company
**Current behavior before PR:**
- No accounting entries for "ownly" owned stock moves
**Desired behavior after PR is merged:**
- Accounting entries for "ownly" owned stock moves
**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-labsSpanish tax reports no longer crash when opened from a manual data wizard, allowing users to access required reporting workflows reliably. The update also handles cash-basis settings for the 347 report more cleanly, reducing the risk of incorrect report behavior.
Original PR description
The fix introduced in 12 by PR https://github.com/odoo/enterprise/pull/3206#pullrequestreview-181239042 reveals a bug in master: Spanish reports crash when opened from a manual data wizard. This commit fixes the issue. Also, this commit disables the cash basis in a cleaner and more correct way for mod 347.