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

Issue 49681002: tryton: Do not recycle first record as product result (Closed)

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

Description

The value of the field of the first record, which is edited in the list, will be lost. Because the editable widget is started before the search popup and it override the new default value with its content when the popup is closed. So we can not reuse record create before the popup. issue7941

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -7 lines) Patch
M tryton/gui/window/view_form/view/form_gtk/one2many.py View 1 chunk +3 lines, -7 lines 0 comments Download

Messages

Total messages: 4
ced
3 months ago (2018-12-21 15:42:07 UTC) #1
reviewbot
flake8 OK URL: https://codereview.tryton.org/49681002
3 months ago (2018-12-21 15:43:17 UTC) #2
rietveld-bot_tryton.org
New changeset 2ddba0a8db33 by Cédric Krier in branch 'default': Do not recycle first record as ...
2 months, 3 weeks ago (2019-01-02 17:54:18 UTC) #3
rietveld-bot_tryton.org
2 months, 1 week ago (2019-01-14 18:03:03 UTC) #4
New changeset b15c7470ba71 by Cédric Krier in branch '5.0':
Do not recycle first record as product result
https://hg.tryton.org/tryton/rev/b15c7470ba71


New changeset 384e1ff56d21 by Cédric Krier in branch '4.8':
Do not recycle first record as product result
https://hg.tryton.org/tryton/rev/384e1ff56d21


New changeset 253eda3e413d by Cédric Krier in branch '4.6':
Do not recycle first record as product result
https://hg.tryton.org/tryton/rev/253eda3e413d


New changeset 24fc4060eb80 by Cédric Krier in branch '4.4':
Do not recycle first record as product result
https://hg.tryton.org/tryton/rev/24fc4060eb80


New changeset b99942da46dc by Cédric Krier in branch '4.2':
Do not recycle first record as product result
https://hg.tryton.org/tryton/rev/b99942da46dc
Sign in to reply to this message.

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