[10.0][11.0][WIP] Nova versão do plano de contas #2
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Por favor tb acompanhem a discussão do tópico: odoo#17367 |
mileo
force-pushed
the
feature/10.0-new_l10n_br
branch
2 times, most recently
from
a94b127
to
a4a436d
Compare
|
Pessoal, Extrai algumas coisas do sped_account e passei pra cá, ainda tem alguns defeitos: Sem dados de demonstração as coisas estão funcionando bem: Mas com dados de demonstração ainda estou tendo alguns problemas. @renatonlima este hack que fazem as coisas funcionarem com dados de demonstração? []s |
|
Favor desconsiderar o comentário acima, o problema era nas dependências do sped_account. Módulo que estou fechando e já faço o PR. |
mileo
force-pushed
the
feature/10.0-new_l10n_br
branch
from
6e7fae1
to
18bc674
Compare
|
Pessoal será q vc conseguem dar uma olhada neste PR? Ele esta bloqueando os testes do odoo-brazil/l10n-brazil#94 e os próximos testes que eu estou montando do contábil. |
|
valeu, accredito que seja util @mileo. Vamos analisar isso sim... |
aricaldeira
suggested changes
Jun 29, 2017
Before this commit, there was a trash icon button in the project sharing view to delete record. In this commit, removed trash icon by adding 'delete=0' in timesheet tree biew. task-3014420x Part-of: odoo#104601
…s widget Before this commit, missing borders of remaining hours in project sharing kanban view. So in this commit, added border and respected color boder classes like 'border-success','border-warning' and 'border-danger' on remaining hours. task-3014420 Part-of: odoo#104601
Before this commit, the progress bar was missing since the progress bar CSS wasn't loaded to the project sharing webclient. So in this commit, the progress bar CSS path has been added to the manifest file for the project web client. task-3014420 Part-of: odoo#104601
Before this commit, when remaining hours is negative then it the label was displayed as muted instead of danger. So in this commit, display the label danger when remaning hours are negative. task-3014420 closes odoo#104601 Related: odoo/enterprise#33447 Signed-off-by: Xavier Bol (xbo) <[email protected]>
The Belgian tax report was only available in French. Now the tax report is available in English by default and there are official translations in Dutch and French. Resources: NL: https://financien.belgium.be/sites/default/files/downloads/165-625-formulier-2022.pdf FR: https://finances.belgium.be/sites/default/files/downloads/165-625-formulaire-2022.pdf [task-3074748](https://www.odoo.com/web#id=3074748&cids=1&menu_id=4720&action=4043&model=project.task&view_type=form) closes odoo#106258 Signed-off-by: Josse Colpaert <[email protected]>
The user could make the mistake to put a large number in the `Guests` input in the POS order and this is not validated, even an number more large for the capacity of an integer, raising an uncontrolled error: `psycopg2.errors.NumericValueOutOfRange: integer out of range` this commit is adding a validation error in order to limit the number of `Guests` with the maximum number for an integer: 2**31 - 1 closes odoo#106523 X-original-commit: 0ff14cc Signed-off-by: Trinh Jacky (trj) <[email protected]>
During the forward-port of odoo#106388 I changed the way the popoverClass is taken into account in the `onPositioned` callback and should have written a test. (from 16.0 branch up to current master) This commit brings the same change (splitting and spreading the prop value) and adds a test. closes odoo#106724 X-original-commit: ddaffdb Signed-off-by: Luca Vitali <[email protected]> Signed-off-by: Bruno Boi (boi) <[email protected]>
closes odoo#106742 X-original-commit: ad9dca9 Related: odoo/enterprise#34476 Signed-off-by: Victor Feyens (vfe) <[email protected]>
Purpose: Make the command palette easier to discover - indeed, unless you know about CTRL+K, it's difficult to know you can use that elsewhere than the home screen. Solution: Add CTRL+K next to the Shortcut menu in the user menu. closes odoo#106594 Tackid: 3074492 Signed-off-by: Aaron Bohy (aab) <[email protected]>
Currently in Sale/Orders/Sales Teams menu , The Sales Analysis dashboard have text overflow bug for large numbers in USD/EUR this bug might be rare but in currency like Yuan or VND its common closes odoo#106737 X-original-commit: 2e58bc2 Signed-off-by: Victor Feyens (vfe) <[email protected]>
Current behavior: If you create an empty Db and just create one coupon program for the PoS , the `enter_code` button is not displayed. This happens because the condition to show the button was looking for program type `coupon` instead of `coupons`. Steps to reproduce: - Create an empty Db (Important) - Install PoS and PoS Loyalty - Create a coupon program for the PoS - Open the PoS, the `enter_code` button is not displayed opw-3078726 closes odoo#106585 Signed-off-by: Trinh Jacky (trj) <[email protected]>
Issue: At anypoint we should be able to call the method "_set_lot_ids" from stock.move on moves that are already done and the quantities should not change. This is not the case for lots that get their quantity changes to 1 as if the product was tracked by serial number. This commit should be seen a a complement to the commit: odoo#79565 Fix: Check the product is tracked by SN before trying to change the quantities to 1. Related ticket: 2689724 closes odoo#100640 X-original-commit: e6fa1e2 Signed-off-by: Arnold Moyaux (arm) <[email protected]>
closes odoo#106745 Signed-off-by: Masereel Pierre <[email protected]>
closes odoo#106779 Signed-off-by: Trinh Jacky (trj) <[email protected]>
Current behavior: In the PoS if you apply a coupon on an order that contains 2 products with different taxes, it will create 2 discount lines on the order. If you change the pricelist, the discount lines won't have the correct values. Steps to reproduce: - Create 2 different products with different taxes. - Create a pricelist with a discount on the 2 products. - Create a coupon that apply a 100% discount on the order. - Start a PoS session - Add the 2 products to the order and apply the coupon. - The order total is now 0€. - Change the pricelist to the one with the discount. - The order total is different than 0€. opw-3049098 closes odoo#106612 X-original-commit: 2f29ca9 Signed-off-by: Trinh Jacky (trj) <[email protected]> Signed-off-by: Engels Robin (roen) <[email protected]>
Since odoo/odoo@69c3d5a the readonly and required modifiers were no longer passed for non-editable views. Which means: - kanban: never passed, - tree: passed only when `editable="1"` or `multi_edit="1"`, - form: always passed. The rationale behind that is that a view which is considered as readonly doesn't need to know if its field are readonly or required, as by definition, if the view is readonly, the user shouldn't be able to modify anything in the view. This was to make the views sent to the web client more light-weight, not sending unused information, and save KB of transfers. However, some widgets, used in the kanban and non-editable tree, allow to modify fields, even in readonly considered views, and therefore need to know these readonly/required modifiers from the field definition in the Python model. e.g. the `widget="color"`, odoo#103478 Ideally, these readonly/required modifiers should be transferred only when the widget requires it, to avoid transferring the readonly/required modifiers when they are not useful in kanban/tree views (which is 99% of the time). However, this would require a bigger refactoring, which is considered too risky compared to the added value for a stable release. task-3013110 closes odoo#106766 Signed-off-by: Denis Ledoux (dle) <[email protected]>
Journal Items menu was not visible out of debug mode, now it is visible Closes PR odoo#105704 Related: odoo/enterprise#33924 Signed-off-by: Laurent Smet <[email protected]>
### Contains the following commits: odoo/o-spreadsheet@6c89e266 [FIX] collaborative: don't endlessly dispatch new revision odoo/o-spreadsheet@1de19d7d [FIX] GridOverlay: Only react to self targeted double clicks odoo/o-spreadsheet@40fc6151 [FIX] figure: apply dragging style on actual drag, not on mousedown odoo/o-spreadsheet@aee5f929 [FIX] figure: Avoid scroll on a focused figure odoo/o-spreadsheet@37b97b18 [FIX] dashboard: click cells to close filter menu closes odoo#107644 Signed-off-by: Pierre Rousseau (pro) <[email protected]>
closes odoo#107328 Signed-off-by: Xavier Morel (xmo) <[email protected]> Co-authored-by: Maralai <[email protected]> Co-authored-by: Matt Harrison <[email protected]>
When website resolves its menu to the backend, it calls `load_menus_root`. Unlike `load_menus`, that does not nicely handle missing menuitems, so *if* one of the root menus was created manually (hence doesn't have a xid) the website will crash either immediately or when trying to open the backend menu (depending on version). Fix by adding a similar fallback to that of `load_menus`. closes odoo#107575 X-original-commit: 3a68265 Signed-off-by: Xavier Morel (xmo) <[email protected]>
currently on clicking the given link return 404 response. closes odoo#107704 Signed-off-by: Victor Feyens (vfe) <[email protected]>
This commit adds the possibility to change the product tag color on a product and also to edit those tags from a submenu of the ecommerce. task_id=2904805 closes odoo#100717 Backport of e91b15a opw:3088158 closes odoo#107515 Signed-off-by: William Braeckman (wbr) <[email protected]>
Reproduction: 1. Install Sales, Stock 2. Place an order with any product, in the Other Info tab, type in anything for Customer Reference 3. Confirm the Order, Go to Delivery, Validate the delivery 4. Print the Delivery Slip, and the customer reference part is overlapping with other text Reason: the unnecessary CSS attribute <row> is causing an overlapping when rendering the delivery template Fix: remove <row> and reformatting the code A correct example of adding extra info (done without row): https://github.com/odoo-dev/odoo/blob/16.0/addons/delivery/views/report_deliveryslip.xml#L4-L17 closes odoo#107659 Signed-off-by: Yannick Tivisse (yti) <[email protected]>
Since [this commit], when dropping several "table of content" blocks on the same page, it was impossible to save. This commit solves this problem. Steps to reproduce the resolved bug: - In edit mode, drop two TOC blocks - Click on Save => the UI is blocked [this commit]: odoo@06b03ba Related to opw-3047375 Related to opw-3035502 task-2948895 Part-of: odoo#98977
The table of content (TOC) has two mechanisms that interest us here: The first one is scrollSpy (from Bootstrap) which allows (among other things) to bold the menu elements according to the scroll. The second one is a mutation observer which is implemented to regenerate the menu when the content of the TOC changes. This being said, when we edit the TOC and scroll at the same time, there is a race condition that makes scrollSpy want to add a class to a menu element while this menu has just been regenerated by the observer. The error is only visible since the migration from Bootstrap 4 to Bootstrap 5 because to add the class, Bootstrap 4 did it with the jQuery `addClass()` function which does not cause an error if the element on which it is called does not exist. Now, Bootstrap 5 does the same thing in pure JS with `classList.add()` which causes an error if the element on which it is called is not defined. This commit fixes this error by disposing the scrollSpy when the menu is regenerated. Steps to reproduce the error: - drop a table of content block - drag a snippet around and go over the drop zones => traceback. It can appear directly or after some tries. opw-3047375 opw-3035502 task-2948895 Part-of: odoo#98977
Since the Bootstrap 5 merge, when there were several TOCs on the same page the scrollSpy only worked for one of them. This commit fixes that. Steps to reproduce the bug: - Drop two TOC blocks on the same page - Save the page => The second TOC is not working properly, it does not highlight the current section. Related to opw-3047375 Related to opw-3035502 task-2948895 Part-of: odoo#98977
This commit removes unnecessary spaces from the XML of the table of content block. Related to opw-3047375 Related to opw-3035502 task-2948895 closes odoo#98977 Signed-off-by: Quentin Smetz (qsm) <[email protected]>
install purchase_requisition module, and create an RFQ in db, check the alternatives tab in the notebook, currently the button Create Alternatives will not be visible unless user click the save button manually. button no longer needs to be hidden when creating a new PO since v16 added in the auto-save feature closes odoo#107679 Signed-off-by: Tiffany Chang <[email protected]>
The field pos_session_id has been moved from account.bank.statement to account.bank.statement.line. See: odoo@7e23eda closes odoo#106509 Signed-off-by: Olivier Colson (oco) <[email protected]>
closes odoo#106974 Signed-off-by: Julien Castiaux <[email protected]>
Commit [1] improved the "sanitation" of this field for special character. For instance, when copy pasting the following terms: | Input | Before | After | |-----------------|------------------|-----------------| | fée d'été à 40€ | f-e-d-t-40- | fee-dete-a-40 | | Nội dung có Dấu | n-i-dung-c-d-u | noi-dung-co-dau | But it actually came with a bad behavior which was not noticed: it prevents to type `-` at the end of the input, which sounds good but is not. Indeed, when typing `a-word`, you will type `a` then try to type `-` which won't work as considering a (forbidden) trailing slash, even if you actually want to type something after. This commit allows trailing slashes again, it's not a big deal and one can remove it if he wants to. [1]: odoo@bb43d4d opw-3075419 closes odoo#107675 X-original-commit: 427ce76 Signed-off-by: Romain Derie (rde) <[email protected]>
One can very well select a "auth=user route" as homepage for his
website, like /my.
There would then be an issue with such an URL being set as homepage:
- As the user landed in the homepage controller which is auth=public,
the system will add the public user as env user (see
`_auth_method_public()`).
```
@http.route('/', type='http', auth="public", website=True, sitemap=True)
def index(self, **kw):
```
- Then, that controller will reroute to the homepage url (/my). The
request.httprequest.path will now be /my
- Then, that controller will recall the dispatcher stack:
`request._serve_ir_http()`
- From there, this call won't fail as it should because the user is
considered as logged in as it went already through the
`_auth_method_public()`, adding the public user as env.user.
`_auth_method_user()` won't raise its error.
- The /my page will be rendered despite not being logged in.
Once the user land on that page, the system will actually detect him as
logged out on a page supposed to be accessed when logged in.
It will then:
1. Show a toaster to inform the user
2. Auto reload the page
As the current URL is still `/` (due to the reroute and not a redirect),
the same flow will happen again, and again, looping forever.
--------
Note that accessing /my directly won't be an issue, as it won't go
through the homepage controller (which is auth=public), so there won't
be a user_id set on the env (the public user), meaning that the dispatch
layer will reject the access and raise an access error (through
`_auth_method_user()`.
Also note that the same behavior will occur when going through the first
menu fallback mechanism:
- if the user didn't setup any homepage_url
- and deleted his / website.page
- and his first website menu is /my
In that case, it will go through the first menu redirect fallback, which
will be working fine (as it's a redirect and not a reroute).
With both those 2 flows (going through redirect), the user will
correctly land on the login page (which will redirect to /my once logged
in).
------
Finally, the other solution would be to prevent such a configuration
(/my as homepage_url) but it was not easily doable (if doable at all),
see odoo#99100 (comment)
------
A test is also added and over the more complexe case (which should cover
everything):
- With /my as homepage_url
- With the / website.page deleted
- With /my as first menu URL
-> Accessing / as public user should:
1. Reroute to /my (because of the homepage_url set to it) which
should fail now thanks to this commit
2. Since the reroute / re-serve failed, it should reach the
"first menu fallback" mechanism, which is also /my
3. That fallback should be a redirect, not a reroute, so the user
should actually land on the login page
4. Once logged in on that page, it should properly redirect to /my
opw-3077339
Part-of: odoo#106591
Commit [1] introduced the creation of a website in the tests, but wrongly set `user_id` to the current environment user. It doesn't make any sense as this field is supposed to hold the website public user, which is not a real user and is supposed to be a public user, as the field name hints.. On top of that, the current environment user (the demo user for tests, which is wrongly set as the website public user) is then used to login into the website, which is then making things even more weird/wrong: you are not supposed to login with the public user (which is not really a public user here but still marked as the public user from the website). It was preventing the fix from this same PR (see previous commit) to be working, as it was making this tour crash. Not setting the `user_id` property will simply let the system create one for you (a real public user..). [1]: odoo@3611aaf closes odoo#106591 Signed-off-by: Julien Castiaux <[email protected]>
On clicking "add to cart" button, there is a simple animation of product image moving to the cart icon. However, if the cart icon is hidden because of disabled option "Show Empty Cart", the animation is broken: the image is moving to left corner of the navbar. Fix it by showing cart icon before starting the animation. opw-3078396 closes odoo#106777 Signed-off-by: William Braeckman (wbr) <[email protected]>
Steps to reproduce: - Create a price list with different currency and discount with "show price and discount to the customer" - On the website select this pricelist and register for the event. Issue: The price in the cart is shown in the main currency Solution: [website_event_sale] There is an initial issue which when it calls '_compute_price_reduce'. We compare, 'product.lst_price' (in product.currency) and 'product.price' (which has been converted to the pricelist.currency). In order to compare apples with apples, a conversion is applied to have the 'product.lst_price' in the same currency. After that, we have kind of a coherent behaviour in the sense that 'ticket.price_reduce' is in the same currency as 'ticket.price'. Thereafter, a conversion is applied (if the pricelist.currency is different) to get the expected amount. The same reasoning is applied to [website_event_booth_sale]. Note: 'list_price' has been changed to 'lst_price' in Booth to have the same logic between Event and Booth opw-2766997 X-original-commit: 2a023ca Part-of: odoo#101375
Steps to reproduce: - Create a price list with different currency and discount with "show price and discount to the customer" - On the website select this pricelist and try to select the booth Issue: The displayed price will not be the correct one Note: This is an issue discovered during the correction of odoo#101375 (forward-port of odoo#85640) It allows to have the correct price depending of the currency of the pricelist applied. Now the unlink of the rate makes the new rate directlt effective. There is no need of having a `new_company` anymore. Summary: - view modification in `website_event_booth_sale` -> price of selected booth, simplification of comparison for the `<del>` - view modification in `website_event_sale` : simplification of comparison for the `<del>` - backend test modification in `website_event_[booth_]sale` common: addapt the rate; take out useless `new_env`; simplified pricelists creation - tour test addition: added the tour for essential use cases in event and event_booth; simplified the command so it is more readable related ticket: opw-2766997 closes odoo#106593 closes odoo#101375 X-original-commit: db764e9 Signed-off-by: Thibault Delavallee (tde) <[email protected]> Signed-off-by: Nicolas Lempereur (nle) <[email protected]> Signed-off-by: Yolann Sabaux (yosa) <[email protected]>
STEPS: Create a Pricelist with price rule discount-based on parent product category. Result: the price rule discount is not applied if the product is not directly attached to a parent category. Fix it by correcting domain in `_get_applicable_rules_domain`. Also, update tests: use parent category in the pricelist. odoo@dc8db07 opw-3080836 closes odoo#107715 Signed-off-by: Victor Feyens (vfe) <[email protected]>
The kanban cards did not take the class oe_kanban_color. This is because it was set on the child div instead of the parent one. opw-3069024 closes odoo#106690 Signed-off-by: Brice Bartoletti (bib) <[email protected]>
From the commit 2ab231b We introduce a error in the path to read Linux file In this commit we correct the access to the file which is in the "home" of the raspberry closes odoo#107762 Signed-off-by: Quentin Lejeune (qle) <[email protected]>
…a reference is missing When a record is `forcecreate=0` and a reference of one of its fields is missing, the update of the corresponding module fails. In this case, we can skip the creation of this record. An example of this issue is this [record](https://github.com/odoo/enterprise/blob/6411ae071ace980834befeb8040c94b1a05c8034/documents_account/data/data.xml#L17) in `documents_account` module (enterprise) when the reference of this [field](https://github.com/odoo/enterprise/blob/6411ae071ace980834befeb8040c94b1a05c8034/documents_account/data/data.xml#L20) is missing. This can be reproduced as follows: 1. Create a DB with `documents_account` module 2. Uninstall `documents_account` module 3. Remove `documents.documents_finance_status` 4. Reinstall `documents_account` ``` Traceback (most recent call last): File "/home/ayman/src/odoo/15.0/odoo/tools/convert.py", line 680, in _tag_root f(rec) File "/home/ayman/src/odoo/15.0/odoo/tools/convert.py", line 567, in _tag_record f_val = self.id_get(f_ref) File "/home/ayman/src/odoo/15.0/odoo/tools/convert.py", line 663, in id_get res = self.model_id_get(id_str, raise_if_not_found) File "/home/ayman/src/odoo/15.0/odoo/tools/convert.py", line 669, in model_id_get return self.env['ir.model.data']._xmlid_to_res_model_res_id(id_str, raise_if_not_found=raise_if_not_found) File "/home/ayman/src/odoo/15.0/odoo/addons/base/models/ir_model.py", line 1943, in _xmlid_to_res_model_res_id return self._xmlid_lookup(xmlid)[1:3] File "<decorator-gen-35>", line 2, in _xmlid_lookup File "/home/ayman/src/odoo/15.0/odoo/tools/cache.py", line 90, in lookup value = d[key] = self.method(*args, **kwargs) File "/home/ayman/src/odoo/15.0/odoo/addons/base/models/ir_model.py", line 1936, in _xmlid_lookup raise ValueError('External ID not found in the system: %s' % xmlid) ValueError: External ID not found in the system: documents.documents_finance_status ``` closes odoo#107756 X-original-commit: 2e4f066 Signed-off-by: Christophe Simonis <[email protected]>
Follow-up reports were previously not handled by the commit fixing invoices. This commit aims to fix that. closes odoo#107784 X-original-commit: 768cfbe Signed-off-by: Florian Daloze (fda) <[email protected]> Signed-off-by: Solan Delvenne (sode) <[email protected]>
mileo
force-pushed
the
feature/10.0-new_l10n_br
branch
from
18bc674
to
543bb4d
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
@rvalyi @renatonlima @rvalyi @mbcosta
Pessoal comecei a extrair algumas partes do sped_account para criar uma nova versão do l10n_br para propormos a master antes do lançamento da versão 11.
E também mantermos esse fork como um mirror da OCA/ocb e odoo/odoo.
De começo proponho 2 módulos:
O primeiro módulo deve servir tb como base para empresas que desejam importar seu próprio plano de contas.
--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr