Created on 2020-04-13.09:42:25 by josesalvador, last changed 1 month ago by roundup-bot.
New changeset 41e2ccca80de by José Salvador in branch '5.8': Exclude cancelled invoices from EC Operation list https://hg.tryton.org/modules/account_es/rev/41e2ccca80de New changeset db70896e8633 by Cédric Krier in branch '5.8': Test amount of Spanish EC Operation List without digits https://hg.tryton.org/modules/account_es/rev/db70896e8633
New changeset fd56d576e9ac by Cédric Krier in branch 'default': Test amount of Spanish EC Operation List without digits https://hg.tryton.org/tryton-env/rev/fd56d576e9ac
New changeset 3c9b72a55846 by Cédric Krier in branch 'default': Test amount of Spanish EC Operation List without digits https://hg.tryton.org/modules/account_es/rev/3c9b72a55846
New changeset 75c0483baed0 by Sergi Almacellas Abellana in branch 'default': Exclude cancelled invoices from EC Operation list https://hg.tryton.org/tryton-env/rev/75c0483baed0
New changeset a0eaa91048cd by José Salvador in branch 'default': Exclude cancelled invoices from EC Operation list https://hg.tryton.org/modules/account_es/rev/a0eaa91048cd
New review pushed   https://codereview.tryton.org/295401002/#ps298421002
Code review ready for testing. But this patch set  uploaded with upload.py does not include the removed file as the screenshot attached  of mercurial state shows. Why? What have I done wrong?  https://codereview.tryton.org/295401002/#ps309901003  mercurial_state.png PS -> I'm conscious of the disaster of previous patches and comments. That will not occur again since I'm be able to manage partial not ready developments without the need to push to code review tool. Excuse me for that.
Relying on in other issues I think the right state is testing. Correct me if I am wrong, please.
The issue is not finished but I need some guidance/responses to the code in the review so is for that I set to chatting this issue. If it is not the right workflow please help to know how I have to do it.
Here is review295401002 which adds the required filters also improve the cancell invoice scenario to test this case.
> Yes, but as far as customer invoices can also be cancelled (when refunded they are marked as cancelled) we should also take them in account. > I will not do any distinction here and fix both cases. This means adding the cancel state filter and also exclude "canceled moves" as we did on issue9097 Understood, so in this case, just for information, are you still ensuring it will be needed to update account_eu too?
>> I'm talking all time about canceled (and not refund) **purchase** (from supplier) invoices **not canceled customer invoices**. Yes, but as far as customer invoices can also be cancelled (when refunded they are marked as cancelled) we should also take them in account. I will not do any distintion here and fix both cases. This means adding the cancel state filter and also exclude "canceled moves" as we did on issue9097
@pokoli @ced remember I'm talking all time about canceled (and not refund) **purchase** (from supplier) invoices **not canceled customer invoices**.
On 2020-04-15 10:10, Sergi Almacellas Abellana wrote: > > I do not see why account_eu should be modified? > As far as we allow to cancel custom invoices I think it's safer to also exclude the cancelled invoices. For me, it is a big problem because (except Spanish) I do not know a country that allow to "update" such report after sending it. Canceling a customer invoice should always be done with a credit note in Europe.
> I do not see why account_eu should be modified? As far as we allow to cancel custom invoices I think it's safer to also exclude the cancelled invoices.
I do not see why account_eu should be modified?
Ok, so we should add a cancel filter on Spanish and European EC Operation list. But if we make this filter we should also include the cancel invoices like we did in issue9097
Not exactly... In issue9097 we're talking about a customer refund so in this case all the invoices/moves must be taken into account. In this case I'm talking about a cancel (not refund) of purchase invoice which was accounted as mistake. In other words I think that: - A cancel purchase invoice must not be taken into account - A refund purchase invoice must be taken into account
@josesalvador is this similar to issue9097 but for Spanish EC Operation List?
IIUC when you cancel a purchase invoice (eg to correct a purchase invoice accounted as mistake) Tryton cancel it generating a counterpart move. So IMHO that operation must not be showed in EC Operation List or at least it should be showed as zero amount (although I don't like the last one alternative...)
As long as they have accounting move, for me it is expected.
EC Operation List shows canceled purchase invoices
messages: + msg65536
|2021-02-27 10:54:09||roundup-bot||set||messages: + msg64905|
|2021-02-27 10:54:05||roundup-bot||set||messages: + msg64904|
|2021-02-26 09:22:14||roundup-bot||set||messages: + msg64874|
nosy: + roundup-bot
status: testing -> resolved
|2021-02-16 21:02:15||reviewbot||set||messages: + msg64648|
|2021-02-16 20:01:42||reviewbot||set||messages: + msg64647|
|2021-02-13 11:00:11||reviewbot||set||messages: + msg64532|
|2020-12-13 13:50:25||reviewbot||set||messages: + msg63067|
|2020-12-13 13:20:41||josesalvador||set||status: in-progress -> testing|
|2020-10-15 20:28:17||josesalvador||set||status: testing -> in-progress|
|2020-10-15 19:11:36||reviewbot||set||messages: + msg60977|
|2020-10-15 18:07:02||reviewbot||set||messages: + msg60972|
|2020-10-14 17:15:32||reviewbot||set||messages: + msg60962|
|2020-10-14 16:38:47||reviewbot||set||messages: + msg60958|
|2020-10-14 16:31:22||josesalvador||set||status: in-progress -> testing|
component: - account_eu
|2020-09-14 19:24:09||josesalvador||set||status: testing -> in-progress|
|2020-08-18 12:59:21||reviewbot||set||messages: + msg59776|
|2020-08-18 12:50:46||josesalvador||set||messages: + msg59775|
|2020-06-23 18:48:29||reviewbot||set||messages: + msg58845|
|2020-06-23 18:33:44||josesalvador||set||status: in-progress -> testing|
files: + mercurial_state.png
messages: + msg58843
|2020-06-12 22:10:33||reviewbot||set||messages: + msg58674|
|2020-06-12 17:42:00||josesalvador||set||status: testing -> in-progress|
|2020-06-11 20:53:24||josesalvador||set||status: chatting -> testing|
messages: + msg58655
|2020-06-11 20:37:18||reviewbot||set||messages: + msg58654|
|2020-06-11 20:24:29||josesalvador||set||status: in-progress -> chatting|
messages: + msg58653
|2020-06-10 22:07:49||reviewbot||set||messages: + msg58640|
|2020-06-10 19:37:37||reviewbot||set||messages: + msg58639|
|2020-06-02 21:08:15||reviewbot||set||messages: + msg58426|
|2020-05-22 08:18:46||josesalvador||set||status: testing -> in-progress|
|2020-05-21 11:10:28||pokoli||set||assignedto: pokoli -> josesalvador|
messages: + msg57196
|2020-04-16 10:42:05||reviewbot||set||reviews: 295401002|
keyword: + review
|2020-04-16 10:26:45||pokoli||set||status: chatting -> testing|
messages: + msg57195
keyword: + backport
|2020-04-15 12:47:00||josesalvador||set||messages: + msg57170|
|2020-04-15 11:42:57||pokoli||set||messages: + msg57169|
|2020-04-15 11:17:24||josesalvador||set||messages: + msg57168|
|2020-04-15 11:00:05||ced||set||messages: + msg57167|
|2020-04-15 10:10:08||pokoli||set||messages: + msg57165|
|2020-04-15 10:00:34||ced||set||messages: + msg57164|
component: + account_eu
|2020-04-13 13:09:39||josesalvador||set||messages: + msg57091|
messages: + msg57076
|2020-04-13 10:18:12||josesalvador||set||status: need-eg -> chatting|
messages: + msg57075
|2020-04-13 10:04:05||ced||set||status: unread -> need-eg|
nosy: + ced
messages: + msg57074