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

[ADD] eCDF financial reports wizard #13

Closed
wants to merge 30 commits into from

Conversation

ThomasBinsfeld
Copy link

Reports

This wizard allows to generate three types of financial reports :

  • Profit & Loss (P&L)
  • Balance Sheet (BS)
  • Chart of Accounts (CA)

Profit & Loss and Balance Sheet can be generated in abbreviated version or not. The selected reports (max. 3) are written in a downloadable XML file for eCDF.

Templates

Five MIS Builder templates are used to generate the rapports :

  • Luxembourg Balance Sheet
  • Luxembourg Balance Sheet (abbreviated)
  • Luxembourg Profit & Loss
  • Luxembourg Profit & Loss (abbreviated)
  • Luxembourg Chart of Accounts

View

ecdf_mis_report_gui

@ThomasBinsfeld
Copy link
Author

@sbidoul @adrienpeiffer

@coveralls
Copy link

coveralls commented May 12, 2016

Coverage Status

Coverage decreased (-79.9%) to 20.11% when pulling dac03bf on ThomasBinsfeld:8.0-l10n_lu_mis_ecdf into 9df05a6 on OCA:8.0.

@coveralls
Copy link

coveralls commented May 13, 2016

Coverage Status

Coverage decreased (-79.9%) to 20.055% when pulling 41a2550 on ThomasBinsfeld:8.0-l10n_lu_mis_ecdf into 9df05a6 on OCA:8.0.

@moylop260
Copy link

moylop260 commented May 17, 2016

Could you check the following output?

l10n_lu_ecdf/__init__.py:1: [W8201(incoherent-interpreter-exec-perm), ] Incoherent interpreter comment and executable permission. Interpreter: [] Exec perm: True
l10n_lu_ecdf/__init__.py:1: [C7902(missing-readme), ] Missing ./README.rst file. Template here: https://github.com/OCA/maintainer-tools/blob/master/template/module/README.rst
l10n_lu_ecdf/__init__.py:1: [W7908(missing-newline-extrafiles), ] wizard/ecdf_report_view.xml Missing newline
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:3 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:4 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:5 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:6 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:7 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:8 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:9 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:10 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:11 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:12 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:13 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:14 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:15 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:16 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:17 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:18 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:19 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:20 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:21 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:22 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:23 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:24 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:25 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:26 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:27 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:28 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:29 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:30 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:31 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:32 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:33 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:34 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:35 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:36 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:37 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:38 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:39 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:40 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:41 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:42 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:43 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:44 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:45 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:46 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:47 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:48 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:49 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:50 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:51 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:52 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:53 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:54 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:55 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:63 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:64 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:65 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:66 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:67 Use wrong tabs indentation instead of four spaces
l10n_lu_ecdf/__init__.py:1: [W7910(wrong-tabs-instead-of-spaces), ] wizard/ecdf_report_view.xml:68 Use wrong tabs indentation instead of four spaces
************* Module l10n_lu_ecdf.__openerp__
l10n_lu_ecdf/__openerp__.py:1: [C8201(no-utf8-coding-comment), ] No UTF-8 coding comment found: Use `# coding: utf-8` or `# -*- coding: utf-8 -*-`
l10n_lu_ecdf/__openerp__.py:1: [W8201(incoherent-interpreter-exec-perm), ] Incoherent interpreter comment and executable permission. Interpreter: [] Exec perm: True
l10n_lu_ecdf/__openerp__.py:1: [C8101(manifest-required-author), ] Missing author required "Odoo Community Association (OCA)" in manifest file
************* Module l10n_lu_ecdf.models.__init__
l10n_lu_ecdf/models/__init__.py:1: [W8201(incoherent-interpreter-exec-perm), ] Incoherent interpreter comment and executable permission. Interpreter: [] Exec perm: True
************* Module l10n_lu_ecdf.models.res_company
l10n_lu_ecdf/models/res_company.py:1: [C8201(no-utf8-coding-comment), ] No UTF-8 coding comment found: Use `# coding: utf-8` or `# -*- coding: utf-8 -*-`
l10n_lu_ecdf/models/res_company.py:1: [W8201(incoherent-interpreter-exec-perm), ] Incoherent interpreter comment and executable permission. Interpreter: [] Exec perm: True
l10n_lu_ecdf/models/res_company.py:4: [C8104(class-camelcase), res_company] Use `CamelCase` "ResCompany" in class name "res_company". You can use oca-autopep8 of https://github.com/OCA/maintainer-tools to auto fix it.
************* Module l10n_lu_ecdf.wizard.__init__
l10n_lu_ecdf/wizard/__init__.py:1: [W8201(incoherent-interpreter-exec-perm), ] Incoherent interpreter comment and executable permission. Interpreter: [] Exec perm: True
************* Module l10n_lu_ecdf.wizard.ecdf_report
l10n_lu_ecdf/wizard/ecdf_report.py:1: [W8201(incoherent-interpreter-exec-perm), ] Incoherent interpreter comment and executable permission. Interpreter: [] Exec perm: True
l10n_lu_ecdf/wizard/ecdf_report.py:11: [R8101(openerp-exception-warning), ] Import `Warning` should be renamed as UserError `from openerp.exceptions import Warning as UserError`
l10n_lu_ecdf/wizard/ecdf_report.py:12: [W0402(deprecated-module), ] Uses of a deprecated module 'openerp.osv'
l10n_lu_ecdf/wizard/ecdf_report.py:90: [C8107(translation-required), EcdfReport.check_matr] String parameter of raise "ValidationError" requires translation. Use _('Matricule must be 11 or 13 characters                 long.')
l10n_lu_ecdf/wizard/ecdf_report.py:105: [C8107(translation-required), EcdfReport.check_rcs] String parameter of raise "ValidationError" requires translation. Use _('RCS number must begin with an uppercase                 letter followed by 2 to 6 digits. The first digit must not be                  0.')
l10n_lu_ecdf/wizard/ecdf_report.py:121: [C8107(translation-required), EcdfReport.check_vat] String parameter of raise "ValidationError" requires translation. Use _('VAT number must begin with two                 uppercase letters followed by 8 digits.')
l10n_lu_ecdf/wizard/ecdf_report.py:208: [C8107(translation-required), EcdfReport._check_prev_fiscyear] String parameter of raise "ValidationError" requires translation. Use _('Previous fiscal year must be before current fiscal year')
l10n_lu_ecdf/wizard/ecdf_report.py:235: [C8107(translation-required), EcdfReport.get_matr_declarer] String parameter of raise "ValueError" requires translation. Use _('Matricule not present')
l10n_lu_ecdf/wizard/ecdf_report.py:707: [C8107(translation-required), EcdfReport.print_xml] String parameter of raise "Warning" requires translation. Use _('MIS Template(s) not found :')
  • You should run chmod -x l10n_lu_ecdf/__init__.py && chmod -x l10n_lu_ecdf/__openerp__.py
  • Use four spaces in all your technical files.
  • Add translation method raise Warning(_('my msg'))

More info here:
https://travis-ci.org/OCA/l10n-luxemburg/jobs/129921440#L421

@coveralls
Copy link

coveralls commented May 17, 2016

Coverage Status

Coverage decreased (-79.7%) to 20.274% when pulling 36f90b4 on ThomasBinsfeld:8.0-l10n_lu_mis_ecdf into 9df05a6 on OCA:8.0.

@coveralls
Copy link

coveralls commented May 17, 2016

Coverage Status

Coverage decreased (-79.7%) to 20.274% when pulling 8ebabf8 on ThomasBinsfeld:8.0-l10n_lu_mis_ecdf into 9df05a6 on OCA:8.0.

@moylop260
Copy link

@ThomasBinsfeld
Thanks for fixing and contribution.

Could you review the following checks?

l10n_lu_ecdf/__init__.py:1: [C7902(missing-readme), ] Missing ./README.rst file. Template here: https://github.com/OCA/maintainer-tools/blob/master/template/module/README.rst
l10n_lu_ecdf/__init__.py:1: [W7908(missing-newline-extrafiles), ] wizard/ecdf_report_view.xml Missing newline
************* Module l10n_lu_ecdf.models.__init__
l10n_lu_ecdf/models/__init__.py:1: [W8201(incoherent-interpreter-exec-perm), ] Incoherent interpreter comment and executable permission. Interpreter: [] Exec perm: True
************* Module l10n_lu_ecdf.models.res_company
l10n_lu_ecdf/models/res_company.py:1: [W8201(incoherent-interpreter-exec-perm), ] Incoherent interpreter comment and executable permission. Interpreter: [] Exec perm: True
l10n_lu_ecdf/models/res_company.py:6: [C8104(class-camelcase), res_company] Use `CamelCase` "ResCompany" in class name "res_company". You can use oca-autopep8 of https://github.com/OCA/maintainer-tools to auto fix it.
************* Module l10n_lu_ecdf.wizard.__init__
l10n_lu_ecdf/wizard/__init__.py:1: [W8201(incoherent-interpreter-exec-perm), ] Incoherent interpreter comment and executable permission. Interpreter: [] Exec perm: True
************* Module l10n_lu_ecdf.wizard.ecdf_report
l10n_lu_ecdf/wizard/ecdf_report.py:1: [W8201(incoherent-interpreter-exec-perm), ] Incoherent interpreter comment and executable permission. Interpreter: [] Exec perm: True

The W8201(incoherent-interpreter-exec-perm) could be fixed with chmod -x file.py

@coveralls
Copy link

coveralls commented May 17, 2016

Coverage Status

Coverage decreased (-79.7%) to 20.274% when pulling fef55b5 on ThomasBinsfeld:8.0-l10n_lu_mis_ecdf into 9df05a6 on OCA:8.0.

@coveralls
Copy link

coveralls commented May 20, 2016

Coverage Status

Coverage decreased (-61.4%) to 38.63% when pulling 275b53c on ThomasBinsfeld:8.0-l10n_lu_mis_ecdf into 9df05a6 on OCA:8.0.

@coveralls
Copy link

coveralls commented May 20, 2016

Coverage Status

Coverage decreased (-60.0%) to 40.0% when pulling d184a51 on ThomasBinsfeld:8.0-l10n_lu_mis_ecdf into 9df05a6 on OCA:8.0.

@coveralls
Copy link

coveralls commented May 24, 2016

Coverage Status

Coverage decreased (-8.4%) to 91.644% when pulling a409449 on ThomasBinsfeld:8.0-l10n_lu_mis_ecdf into 9df05a6 on OCA:8.0.

@sbidoul
Copy link
Member

sbidoul commented May 31, 2016

Superseded by #14

@sbidoul sbidoul closed this May 31, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants