Tryton Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(29)

Issue 45861002: trytond: Remove actual stopped self from the transactions stack (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 months, 1 week ago by ced
Modified:
1 month, 1 week ago
Reviewers:
pokoli, rietveld-bot, reviewbot
Visibility:
Public.

Description

Stopping non-current transaction should not remove the last transaction but itself. issue7973

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -2 lines) Patch
M trytond/transaction.py View 1 chunk +5 lines, -2 lines 2 comments Download

Messages

Total messages: 7
ced
2 months, 1 week ago (2019-01-07 17:53:12 UTC) #1
reviewbot
flake8 OK URL: https://codereview.tryton.org/45861002
2 months, 1 week ago (2019-01-07 18:07:55 UTC) #2
pokoli
https://codereview.tryton.org/45861002/diff/1/trytond/transaction.py File trytond/transaction.py (right): https://codereview.tryton.org/45861002/diff/1/trytond/transaction.py#newcode155 trytond/transaction.py:155: transactions.reverse() Why reversing and then undoing the reverse?
2 months, 1 week ago (2019-01-11 17:51:51 UTC) #3
ced
https://codereview.tryton.org/45861002/diff/1/trytond/transaction.py File trytond/transaction.py (right): https://codereview.tryton.org/45861002/diff/1/trytond/transaction.py#newcode155 trytond/transaction.py:155: transactions.reverse() On 2019/01/11 17:51:51, pokoli wrote: > Why reversing ...
2 months, 1 week ago (2019-01-11 18:33:00 UTC) #4
rietveld-bot_tryton.org
New changeset 677c4fc1c843 by Cédric Krier in branch 'default': Remove actual stopped self from the ...
2 months ago (2019-01-17 16:13:45 UTC) #5
rietveld-bot_tryton.org
New changeset 36e666e80791 by Cédric Krier in branch 'default': Remove actual stopped self from the ...
2 months ago (2019-01-17 16:13:53 UTC) #6
rietveld-bot_tryton.org
1 month, 1 week ago (2019-02-10 22:51:26 UTC) #7
New changeset cf07753010cd by Cédric Krier in branch '5.0':
Remove actual stopped self from the transactions stack
https://hg.tryton.org/trytond/rev/cf07753010cd


New changeset 64050e1ead7a by Cédric Krier in branch '4.8':
Remove actual stopped self from the transactions stack
https://hg.tryton.org/trytond/rev/64050e1ead7a


New changeset 31e1a8c2d984 by Cédric Krier in branch '4.6':
Remove actual stopped self from the transactions stack
https://hg.tryton.org/trytond/rev/31e1a8c2d984


New changeset fe396c8fee1c by Cédric Krier in branch '4.4':
Remove actual stopped self from the transactions stack
https://hg.tryton.org/trytond/rev/fe396c8fee1c


New changeset 456d4f18f424 by Cédric Krier in branch '4.2':
Remove actual stopped self from the transactions stack
https://hg.tryton.org/trytond/rev/456d4f18f424
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 0147766