| Age | Commit message (Collapse) | Author |
|---|
|
Main
-
[FIX] l10n_mx: Repartition account for refund must be the same
than invoice's one.
Explanation
-
Current Behavior
=
Purchase Tax (16%)
-
|field|value
|-|-
|Transition Account|Transtion Tax
|Invoice Repartition|Paid Tax
|Refund Repartition|Transtion Tax
|CABA Base|Base Account
Invoice
-
|account|debit|credit|
|-|-:|-:|
|Expense|1000||
|Transtion Tax|160||
|Payables||1160|
Payment
-
|account|debit|credit|
|-|-:|-:|
|Payables|580||
|Bank||580|
CABA on Payment
-
|account|debit|credit|
|-|-:|-:|
|Base Account|500||
|Base Account||500|
|Paid Tax|80||
|Transtion Tax||80|
Refund
-
|account|debit|credit|
|-|-:|-:|
|Payables|290||
|Discounts||250|
|Transtion Tax||40|
CABA on Refund
-
|account|debit|credit|
|-|-:|-:|
|Base Account|250||
|Base Account||250|
|Paid Tax|40||
|Transtion Tax||40|
|Base Account|250||
|Base Account||250|
|Transtion Tax|40||
|Transtion Tax (*)||40|
(*)Wrong Entry Line
Trail Balance
-
|account|debit|credit|balance|
|-|-:|-:|-:|
|Bank|0|580|-580|
|Base Account|1000|1000|0|
|Discounts|0|250|-250|
|Expense|1000|0|1000|
|Paid Tax|120|0|120|
|Payables|870|1160|-290|
|Transtion Tax|200|200|0|
|Total Sum|3190|3190|0|
Expected Behavior
=
Purchase Tax (16%)
-
|field|value
|-|-
|Transition Account|Transtion Tax
|Invoice Repartition|Paid Tax
|Refund Repartition|Paid Tax
|CABA Base|Base Account
Invoice
-
|account|debit|credit|
|-|-:|-:|
|Expense|1000||
|Transtion Tax|160||
|Payables||1160|
Payment
-
|account|debit|credit|
|-|-:|-:|
|Payables|580||
|Bank||580|
CABA on Payment
-
|account|debit|credit|
|-|-:|-:|
|Base Account|500||
|Base Account||500|
|Paid Tax|80||
|Transtion Tax||80|
Refund
-
|account|debit|credit|
|-|-:|-:|
|Payables|290||
|Discounts||250|
|Transtion Tax||40|
CABA on Refund
-
|account|debit|credit|
|-|-:|-:|
|Base Account|250||
|Base Account||250|
|Paid Tax|40||
|Transtion Tax||40|
|Base Account|250||
|Base Account||250|
|Transtion Tax|40||
|Paid Tax (*)||40|
(*)Right Entry Line
Trail Balance
-
|account|debit|credit|balance|
|-|-:|-:|-:|
|Bank|0|580|-580|
|Base Account|1000|1000|0|
|Discounts|0|250|-250|
|Expense|1000|0|1000|
|Paid Tax|120|40|80|
|Payables|870|1160|-290|
|Transtion Tax|200|160|40|
|Total Sum|3190|3190|0|
Tax Review
=
It is expected that Payables for 290 divided by 1.16 (100% + tax) yields the tax base equal to 250
Given that tax base of 250 multiplied by 0.16 (tax) should yield the Outstanding Transition Tax.
Transition Tax must be 40. Currently it is 0.
Paid Tax must be 80. Currently is it 120.
Refund must cancel out 40 in Paid Tax because Refund is not an actual Cash Flow.
closes odoo/odoo#67323
X-original-commit: 18f9cbea59b4a9bcacb5eee237c531a666b1dc46
Signed-off-by: Jorge Pinna Puissant (jpp) <jpp@odoo.com>
|
|
The overrides in l10n_ modules are broken since 14.0 because the hook no longer exists.
closes odoo/odoo#61564
Signed-off-by: oco-odoo <oco-odoo@users.noreply.github.com>
|
|
Ensure the code works fine even if countries are deleted.
Also consider "re"-created countries after deletion, by only considering
the country code, not the data reference.
This should reduce support requests related to deleted countries, and
ease the resolution of such problems by the users themselves.
TASK ID - 2368842
closes odoo/odoo#57837
Related: odoo/enterprise#13275
Signed-off-by: Victor Feyens (vfe) <vfe@odoo.com>
|
|
Since record creation is faster when done from a CSV file,
hence, we move account tags definitions into CSV file.
Also, there was a tag with XMLid 'account_tag_174' defined
twice in XML file, which has been removed.
Total 1082 records of account.account.tag will be created
after removing one duplicated tag.
closes odoo/odoo#56153
Task: 2150637
X-original-commit: 5eef487f194b10ec2a1c0eb452eb22eac6098fc7
Signed-off-by: Josse Colpaert <jco@openerp.com>
|
|
closes odoo/odoo#52407
Task: 2247368
Related: odoo/enterprise#12226
Related: odoo/upgrade#1571
Signed-off-by: oco-odoo <oco-odoo@users.noreply.github.com>
|
|
Miscellaneous changes aiming to improve the user experience in the accounting module,
mainly in the journal model and its form view.
Also aim to improve the bank account creation wizard.
Task ID: #2250349
closes odoo/odoo#54494
Related: odoo/upgrade#1546
Related: odoo/enterprise#11862
Signed-off-by: oco-odoo <oco-odoo@users.noreply.github.com>
|
|
Task 2309613
We have a new hierarchy:
* Accounting
* [Generic, not changed]
* Localization
* Account Chart
* Check
* EDI
* Point of Sale
* Purchase
* Reporting
* Sale
This helps in displaying only the chart of accounts when clicking on
"Install more Packages" from the accounting settings in the Fiscal
Localization section.
We can also refine the search in the _auto_install_l10n post init hook
of account.
closes odoo/odoo#55384
Related: odoo/enterprise#12179
Related: odoo/upgrade#1568
Signed-off-by: Cedric Snauwaert (csn) <csn@openerp.com>
|
|
This commits refactors the code of the reconciliation, in order to facilitate the process of complex use cases, namely multi-currencies or cash-basis-taxes related (see details below). It also prepares the code for a second refactoring where we will save on each journal items the amount_currency and currency_id field (even in case of operation made in company currency), also in the sake of simplification.
1) Multi-currencies:
- the account.partial.reconcile model now will have dedicated columns to specify the amount of the partial reconciliation in the debit_line_id currency and the credit_line_id currency. That comes in handy when dealing with journal items having different secondary currencies, but also allows some simplification.
- residual_amount_currency computation changed accordingly
- moved models account.full.reconcile and account.partial.reconcile in their dedicated .py file
2) Cash basis taxes
- cash basis entries now handle correctly rounding errors to make sure the exact amount of gets reported when the reconciliation becomes full.
- the account for the base amount of cash basis entries has to be set, now, on the company instead of on each cash basis tax.
- that new 'property' field can be set at the CoA installation via the field property_cash_basis_base_account_id of account.chart.template, or going through the accounting settings.
Was task task: 2243420
Was PR #50308
Related: odoo/upgrade#1121
Related: odoo/enterprise#10252
Signed-off-by: Quentin De Paoli (qdp) <qdp@openerp.com>
|
|
In a general manner,
the chart templates should not be set to noupdate,
that way, if a new company starts in an existing database,
it starts with an up-to-date chart of accounts.
For instance,
if the chart of template is not updated from
12.0 to 13.0, the `default_pos_receivable_account_id`
is not added in the chart template,
and when a new company is created,
the default pos receivable account is not set on the company.
Besides, the field `res.company``account_default_pos_receivable_account_id`
is not displayed on any form view,
whether or not the full accounting is installed,
so we must especially pay attention this default account is well set
from start, as the user as no opportunity to correct this by himself.
In addition,
in upgrade scripts,
the default pos receivable account on the chart template
is used to correctly set the pos receivable account on the company
and on the pos payment methods,
so it must be well set for the upgrade script to do its job correctly.
Technically, it means, without this revision, the fields:
- `account.chart.template``default_pos_receivable_account_id`,
- `res.company``account_default_pos_receivable_account_id`,
- `pos.payment.method``receivable_account_id`
were not filled properly on upgrade,
causing critical issues in the accounting entries on pos session closing.
Related to #52786
Related to odoo/upgrade@ccfd2371ef89ae67dc9bd68bb0d02f2a440551a2
closes odoo/odoo#52870
X-original-commit: 3258ba015ff75b7c176d83a44261d24ebfcc26de
Signed-off-by: Denis Ledoux (dle) <dle@odoo.com>
|
|
Task 2198388
When testing and demoing localization, it can but cumbersome to create a
Company with the right credentials and install the chart of account on
it.
This commit aims to shorten the process. The first version only contains
a valid vat (or similar) number, but we should add fields specific to
localization in the future so that we have a working environment as soon
as we install the localization.
change manifest
closes odoo/odoo#48102
Signed-off-by: Josse Colpaert <jco@openerp.com>
|
|
There are few fields added on settings and on company form view
by various l10n modules which are meant for particular country's
user and are of no use to others. Such fields should be hidden
from unrelated users. With this commit, it will be the case.
closes odoo/odoo#35755
Task: 2049977
Closes: #35755
Related: odoo/enterprise#5158
Related: odoo/enterprise#5158
Signed-off-by: Josse Colpaert <jco@openerp.com>
|
|
Purpose
=======
The current kanban view is messy. It is difficult to identify which
apps are installed or not. The user can completely miss a module
that might have interested him. A search panel would make things way
more readable.
closes odoo/odoo#44401
Taskid: 2181557
Related: odoo/enterprise#8144
Related: odoo/upgrade#879
Signed-off-by: Yannick Tivisse (yti) <yti@odoo.com>
|
|
Fix account type on 108.01.01 & 108.02.01
According to the SAT catalog must be an account of Current Assets
Fix the account type and the tag on 811.01.01
According to the SAT catalog must be an account of Expenses, and the tag
must be 811.01
The correct account for unaffected earnings in the SAT catalog is 305.01,
then, was added the account and assigned the correct type.
closes odoo/odoo#46046
X-original-commit: 632cc5417796ff375d47767fd7cdb358a5d4ce94
Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>
|
|
Was still using the old "selection:" form instead of ir.model.field.selection
closes odoo/odoo#44322
X-original-commit: fdf9b06d76f95965ed173b1631305f8c8c3a8d9d
Related: odoo/enterprise#8050
Signed-off-by: Martin Trigaux (mat) <mat@odoo.com>
|
|
Followup of a425695e
The terms were back in 12.0
Courtesy of Juan José Scarafía
closes odoo/odoo#41624
X-original-commit: 85d0c7001a997748d7691205bbb8d066597591a5
Signed-off-by: Martin Trigaux (mat) <mat@odoo.com>
|
|
This will improve the extendibility of view inheritance
closes odoo/odoo#41398
Signed-off-by: Martin Trigaux (mat) <mat@odoo.com>
|
|
These changes were announced in the new Annex 24 RFM 2019 [1] and define
the new code for accounts for the northern border zone. This change affects a
part of the CoA because these accounts were changed from the middile to the
end of the group.
[1] https://www.sat.gob.mx/cs/Satellite?blobcol=urldata&blobkey=id&blobtable=MungoBlobs&blobwhere=1461173776384&ssbinary=true
closes odoo/odoo#39556
X-original-commit: cb163992ff5aea0d5e0dff7d31c38fc50ec52ab6
Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>
|
|
This issue is relevant for MX Localization because Cash Basis is created
with the rate of the payment, and not with the rate of the invoice as is
done for odoo core.
closes odoo/odoo#38399
X-original-commit: 10bb81c837c0d88ac4ec396442bd86a654c78646
Signed-off-by: Quentin De Paoli (qdp) <qdp@openerp.com>
|
|
Before this commit when installing l10n_mx chart of accounts taxes for
0% and 4% were the ones assigned by default in products or in invoices
for sales and purchase, respectively.
Reason was wrong sequence in the Template of Taxes, combined with order
of creation, ids.
now the two ones with lowest sequence are the 16% Taxes.
closes odoo/odoo#37642
X-original-commit: 357943e9440683f2bff01dc2f1ef074e5104769a
Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>
|
|
The domain for the tags in the taxes indicates that the country must be
the same that the tax country:
domain="[('applicability', '=', 'taxes'), ('country_id', '=', country_id)]"
For this reason is necessary assign the country in the tags, to allow
register new taxes or change the tag in the records created.
closes odoo/odoo#37482
Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>
|
|
In https://github.com/odoo/odoo/pull/36304 was removed the module
account_cancel, but missing remove the dependence. Now is
removed
closes odoo/odoo#37455
Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>
|
|
This commit deletes account_cancel module and squash it into account.
The 'update_posted' field is changed by 'restrict_mode_hash_table' field.
If 'restrict_mode_hash_table' is true, you have a hash chain on your account journal.
These hash chains prove the inalterability of your accounting.
l10n_fr_certification module is deleted and all hashing method are moved in account module.
Now, you can download a PDF report about your inalterability in the company settings.
If you have l10n_fr or l10n_post_cert installed you have more information on this report
like the inalterability of pos orders, etc.
Task ID: 2039160
|
|
The tags in the tax are created for the Mexican taxes (IVA, ISR & IEPS)
This was assigned in the taxes on 3efefd2171fffaf9e6433cb4f2eb4ef26de7a298,
but in the commit 333c22edd9eee246214259be587e10b8202f40a0
was removed from the tax 0%.
Now was returned that data, to be consistent with the tax for 16%
closes odoo/odoo#36206
Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>
|
|
When is used the option to get the CoA reports based on groups, is
necessary set the tag in each group, then, is necessary load the tags
for second level in the SAT catalog for accounts.
|
|
This module provide you a data from Mexican Bank data extracted
from `SAT Bank catalog
<http://www.sat.gob.mx/fichas_tematicas/buzon_tributario/Documents/catalogo_bancos.pdf>`_.
Additionally was added the following fields:
- ASM code in Banks: to identify banking institutions by ASM standard
- CLABE code in Bank Accounts: required to the sending and receiving
of domestic inter-bank electronic funds transfer.
closes odoo/odoo#35742
Signed-off-by: Josse Colpaert <jco@openerp.com>
|
|
|
|
Before, we only had a public method that installed
the CoA for the current active company.
With the multi-company changes, it was not
possible anymore to install a module with a
demo company and then have the CoA installed
in that demo company correctly.
We changed that public method to be able to
put an extra optional parameter and shortened
its name to try_loading instead of
try_loading_for_current_company. The method that
it calls when there is no chart installed
is made private and renamed to _load.
closes odoo/odoo#35703
Signed-off-by: Quentin De Paoli (qdp) <qdp@openerp.com>
|
|
|
|
[IMP] l10n_*: don't define accounts or tags on tax repartition line templates anymore for 0% taxes
[FIX] account: remove old function used to migrate the former tax model, incompatible with the new one
|
|
closes odoo/odoo#35101
Signed-off-by: oco-odoo <oco-odoo@users.noreply.github.com>
|
|
A new feature[*] in point_of_sale (PoS) which minimizes the
creation of account.move records in closing a pos.session relies
on a receivable account made specifically for PoS.
This commit addresses this feature's requirement by adding a
new receivable account to each localization.
[*] point_of_sale: single AE for a pos.session
TASK-ID: 1862388
|
|
closes odoo/odoo#35505
Signed-off-by: Josse Colpaert <jco@openerp.com>
|
|
Running the following 12.0 tests: https://github.com/odoo/enterprise/blob/b7768337d88990e403338c39e461ecb1796413ab/l10n_mx_edi_landing/tests/test_landing.py#L126
raise the following error using anglo-saxon:
```bash
File stock_account/models/account_invoice.py, line 60, in invoice_validate
File stock_account/models/account_invoice.py, line 89, in _anglo_saxon_reconcile_valuation
File 10n_mx_edi/models/account_move.py, line 12, in reconcile
File account/models/account_move.py, line 957, in reconcile
File account/models/account_move.py, line 948, in _check_reconcile_validity
odoo.exceptions.UserError: ('Account Mercancías en tránsito (115.05.01) does not allow reconciliation. First change the configuration of this account to allow it.', '')
```
closes odoo/odoo#34463
Signed-off-by: Josse Colpaert <jco@openerp.com>
|
|
|
|
|
|
|
|
|
|
New account tags were added to use with the incoming of the northern border
zone of Mexico
These changes were announced in the new Annex 24 RFM 2019 [1] and define
the new accounts for the northern border zone. This change affects a
part of the CoA because these accounts are added in the middle instead
of at the end of the group.
[1] https://www.sat.gob.mx/cs/Satellite?blobcol=urldata&blobkey=id&blobtable=MungoBlobs&blobwhere=1461173688346&ssbinary=true
Closes #33631
closes odoo/odoo#33734
Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>
|
|
|
|
Multi is the default api for methods, it is not necessary to explicitly
decorate methods with it, adds clutter and most people use it because
they see that the rest of the code uses it.
Done with `find . -type f -name '*.py' | xargs sed -i '/@api.multi/d'`
|
|
Fix mexico accounts nature of the following SAT accounts:
- 119.01
- 119.02
- 119.03
- 199.04
- 180.01
- 503
- 505.01
- 505.02
- 607.01
- 610.01
- 612.01
Currently some debit accounts are being labeled as creditors and vice
versa. For example the account 119.01 IVA pendiente de pago should be
labeled as a debit account but is set as a credit account.
closes odoo/odoo#33287
Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>
|
|
- Add repartition lines on taxes
- Link account tags directly to account.move.line; remove the tag_ids field from account.tag
- Add a new report engine dedicated to tax reports, directly generating account tags. It is called as an alternate mode of generic tax report, with a dedicated "Use tax grids" toggle.
>> The biggest change lies in the way the new tax report computes its values.
Everything is now aggregated directly using the tags set on the account move lines. Thanks to that,
modifying the configuration of a tax today will not impact the report for the previous periods anymore.
This is a big improvement, as it means the report will keep on reflecting the values that were submitted
to the state before, whatever the configuration change.
- Add an audit char field to account.move.line telling with tax grids are impacted by the line, with the corresponding amount
- Modify the behavior of cash basis taxes: the cash basis account is now used as the transition account, while the regular account given in tax declaration is used to store the final entry (it was the opposite before)
- Modify every l10n_* module in order to keep them consistent with these changes
closes odoo/odoo#32833
Signed-off-by: Quentin De Paoli (qdp) <qdp@openerp.com>
|
|
|
|
|
|
|
|
closes odoo/odoo#32224
Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>
|
|
|
|
Since those taxes are missing a sequence and they're at the bottom of
the data, they're being taken as the default ones for the mexican chart
of accounts. However, the default taxes for sales and purchases should
be IVA 16%.
closes odoo/odoo#31651
Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>
|
|
|
|
Include commit 572d628 again, as it was lost with a forward port.
closes odoo/odoo#31479
|
