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

Issue 37401002: timesheet: Use root to retrieve company and do not fail on missing line for delete/update (Closed)

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

Description

The delete needs first the company/employee before delete the instance. The delete/update operation should not fail if the line has been deleted before. issue6506

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -5 lines) Patch
M routes.py View 2 chunks +13 lines, -5 lines 0 comments Download

Messages

Total messages: 4
ced
2 months, 1 week ago (2017-05-13 13:12:17 UTC) #1
reviewbot
flake8 OK URL: https://codereview.tryton.org/37401002
2 months, 1 week ago (2017-05-13 13:37:39 UTC) #2
rietveld-bot_tryton.org
New changeset 8e58e96bd54b by C?dric Krier in branch 'default': Use root to retrieve company and ...
2 months ago (2017-05-22 16:04:16 UTC) #3
rietveld-bot_tryton.org
1 month, 3 weeks ago (2017-05-30 22:29:01 UTC) #4
New changeset 5f7c3160dff0 by Cédric Krier in branch '4.4':
Use root to retrieve company and do not fail on missing line for delete/update
http://hg.tryton.org/modules/timesheet/rev/5f7c3160dff0


New changeset a297cb0e3da0 by Cédric Krier in branch '4.2':
Use root to retrieve company and do not fail on missing line for delete/update
http://hg.tryton.org/modules/timesheet/rev/a297cb0e3da0
Sign in to reply to this message.

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