Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

11.0 mig intrastat base #28

Closed
wants to merge 73 commits into from
Closed

11.0 mig intrastat base #28

wants to merge 73 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Mar 9, 2018

#23

Alexis de Lattre and others added 30 commits March 9, 2018 16:22
…tat_product/service object is stored in database, we don't need the legal_intrastat state on invoices any more, because we will be able to create entry lines for DEB for a repair of equipment under warranty without using a legal_intrastat. Report_intrastat_type is now only required for DEB -> object moved to module l10n_fr_intrastat_product. For DEB, the button "regenerate lines" only regenerate DEB lines related to invoices.

Renamed intrastat base module, because it doesn't have France-specific parameters any more.

Add demo data.
…port_intrastat_type table, restarting OpenERP and re-creating intrastat types. Moved intrastat departments from stock.warehouse to stock.location. Dropped SQL queries ; replaced by traditionnal python code logic. No more need to have one rate per day for invoices with foreign currency. Add total fiscal value for DEB More code factorization. Prepare translation work.
… Moved configuration about taxes from company form to tax form Some modifications to ease v5 -> v6 migration : - object report_intrastat_code now belong to group account manager - button functions now return True Tried to implement the following feature : open attachement form when the XML file as been generated : works on v6, but make client crash en v5 -> code has been commented DEB lines with procedure code = 25 are now deducted from the fiscal total. Round invoice total.
… l10n_fr_intrastat_product to intrastat_base, because it should also be used in the module l10n_fr_intrastat_service. Take this field into account in the generation of DEB lines (module l10n_fr_intrastat_service).
…to the EU but invoice outside of the EU, your customer needs to have a fiscal representative inside the EU, which will be used for the DEB . depend on base_vat instead of account.
…hysical person in the EU with VAT, the move is not declared in DEB, so it must not block with a "raise" if the partner doesn't have a VAT number.
…as is_accessory_cost services but no regular product -> DES If the invoice has is_accessory_cost services and regular product -> added to the cost of products in DEB

Now allows "pricelist for statistical value" which is not in EUR (the currency conversion will be made from the pricelist currency to EUR)

Usability improvements :
- Order for DEB and DES tree view : "the more recent at the top"
- distinction between "Information to declare" and "Additionnal information" in intrastat lines
…ive" field in order to avoid the "compression" of the VAT field that made it too small.
…ncluding your own country - When generating lines for Intrastat Product/Service, all invoices for which country == Company's country are excluded
…ligation_level on res.company Add group group_detailed_intrastat_product, so that companies that only use obligation = simplified don't see all the additionnal fields. Remove transaction code corresponding to repairs in intrastat types Better on_change on intrastat types (code is mutualised with field.function) Update syntax : demo_xml/update_xml/init_xml -> data/demo
…astat_service (hope that Akretion France won't forget it's own declarations now !!!) On report.intrastat.product and report.intrastat.service : add copy() fonctions, tracking of important fields, a year_month function field and enhance views. Remove date_done field (the tracking in the chatter does the job). Remove class instanciation in the code.
…p "Detailed intrastat product". Remove dead code and fields that was used when we had to put DEB lines for repair operations (a thing of the past !). Update coding style. Reduce the number of flake8 warnings.
…nload the attachement via the drop down list on the form view, the name of the file will be the name of the attachement and not datas_fname ; so we need to have name = datas_fname.
alexis-via and others added 18 commits March 9, 2018 16:22
Copy incoterms and destination country from SO to invoice when invoicing from SO
We need weight even when supplementary units is used
Small cleanups and enhancements
Add default values for intrastat transaction on company
Code cleanup
Take into account the taxes for B2C
Small code cleanup
Re-organise view of intrastat.product.declaration
Set more fields as invisible (localisation should put them visible if they need it)
Fix handling of suppl. units when hs_code is empty on invoice line (but set on product)
Small usability enhancements
Add transport mode in computation tree view
Rename variables
Set 2 other modules to uninstallable
Update README.rst: switch to new intrastat project
…g > Report > PDF Reports (because intrastat reports are not in PDF !)
intrastat_base: Move company view params to account config page
@pedrobaeza pedrobaeza added this to the 11.0 milestone Mar 9, 2018
@pedrobaeza pedrobaeza mentioned this pull request Mar 9, 2018
4 tasks
@ghost ghost mentioned this pull request Mar 12, 2018
@feketemihai
Copy link
Member

@BT-kaberer Why we are keeping the incompatibility between report_intrastat from main branch and this, just that this module adds the same field in countries and data, doesn't mean that we should double that, and not to depends on that module...

@ghost
Copy link
Author

ghost commented Apr 4, 2018

@feketemihai
intrastat_base act as abstract module for localization. it not only adds a new field and data in res.country.
it defines an abstract report model that is used in the localization modules. l10n_fr_intrastat_service,, l10n_fr_intrastat_product, l10n_be_intrastat_product:

now my assumption is that if it would depend on standard instrastat_report there would be confusion between two different intrastat reporting from the original base module and the localization. Because both modules would be installed. And that's why it's marked as conflicting.
https://github.com/odoo/odoo/blob/11.0/addons/report_intrastat/report/report_intrastat_report.py
vs.
https://github.com/OCA/intrastat/blob/10.0/intrastat_base/models/intrastat_common.py

maybe @alexis-via can shed more light to this

@@ -0,0 +1,80 @@
.. image:: https://img.shields.io/badge/licence-AGPL--3-blue.svg
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.


.. image:: https://odoo-community.org/website/image/ir.attachment/5784_f2813bd/datas
:alt: Try me on Runbot
:target: https://runbot.odoo-community.org/runbot/227/10.0
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

11.0

@@ -0,0 +1,2 @@
# -*- coding: utf-8 -*-
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove coding utf-8 line, since it's already the standard for Python3

@@ -0,0 +1,29 @@
# -*- coding: utf-8 -*-
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove coding utf-8 line, since it's already the standard for Python3

@@ -0,0 +1,8 @@
# -*- coding: utf-8 -*-
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove coding utf-8 line, since it's already the standard for Python3

@@ -0,0 +1,168 @@
# -*- coding: utf-8 -*-
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove coding utf-8 line, since it's already the standard for Python3

@@ -0,0 +1,13 @@
# -*- coding: utf-8 -*-
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove coding utf-8 line, since it's already the standard for Python3

@@ -0,0 +1,13 @@
# -*- coding: utf-8 -*-
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove coding utf-8 line, since it's already the standard for Python3

'license': 'AGPL-3',
'summary': 'Base module for Intrastat reporting',
'author': 'Akretion,Odoo Community Association (OCA), brain-tec AG',
'website': 'http://www.akretion.com',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change the website to https://github.com/OCA/intrastat

@astirpe
Copy link
Member

astirpe commented Jun 12, 2018

Also it would be nice to have some tests (for example the tests proposed in #9) included in this module.

@rvalyi rvalyi mentioned this pull request Jul 28, 2018
@rvalyi
Copy link
Member

rvalyi commented Jul 30, 2018

closing this one as it is superseded by #37 which takes the last review into account...

@rvalyi rvalyi closed this Jul 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants