[V11] l10n_nl_tax_statement: Update of past undeclared invoices - Update of BTW statement doesn't work

[oca]

[V11] l10n_nl_tax_statement: Update of past undeclared invoices - Update of BTW statement doesn't work

[astirpe]

@tv-openbig Thank you for reporting this issue!
I published a fix in #220, would you give it a try?

[tv-openbig]

@astirpe Thanks for your quick intervention.
I have checked you fix. It is nearly fixed (50%).
The omzet and btw in the list was updated as they should !

If i click on the lense to see the invoices behind i would expect to see the
"regular" invoice from the given date range + the invoice from the past.
If i open the lense of the line "1a Leveringen/diensten belast met hoog tarief"
i get just the regular invoice, not the invoice from the past.

Anyway big thank you so far.

[astirpe]

@tv-openbig right, I will fix it next week. Thanks!
Would you keep Onestein in the list of the authors when you clone this module?

[tv-openbig]

@astirpe

Hello, yes sure i will.

For the V11 version i plan a PR towards OCA/l10n-germany project. So far it was was not possible as Odoo rejected my PR to the V10 german accounting plan (odoo/odoo#18571 (comment)).

For the V11 we do not have anymore missing tax tags, so i am positive for the V11 PR.
I will update the authors lists for V11 and also for the V10 "clone".
I mentioned you already as a contributor in the README but for sure i will complete it,
if i am ready with the V11, and for sure the "author" entry in the manifest is more relevant
as the readme ;-)

[astirpe]

@tv-openbig Should be working fine now. Could you check?

[tv-openbig]

@astirpe in the list of journal items i still don't get the invoice lines from the past.
I have tested on this runbot instance http://3357967-220-05ec37.runbot3.odoo-community.org/?db=3357967-220-05ec37-all.
Best regards
Thorsten

[astirpe]

@tv-openbig It's because you didn't set the Country to Netherlands in YourCompany. The report works fine when the Country is Netherlands for the actual company.
I can understand that there is a strange behavior when the country is not set properly and that could be confusing. But the report is meant for Dutch only companies. There is space for future improvement to avoid confusions, but I don't want to spend time on it right now, since it's not part of the issue you opened and is a low prio for me.
Thanks!

[tv-openbig]

@astirpe OK, that is indeed not a big issue in that case;-)

[astirpe]

@tv-openbig in #220 I just committed a fix for getting the invoice lines from the past when the Country is not set to Netherlands. Thanks!