Tryton - Issues

 

Issue6506

Title Delete timesheet line from route fails
Priority bug Status resolved
Superseder Nosy List ced, reviewbot, roundup-bot
Type crash Components timesheet
Assigned To ced Keywords review
Reviews 37401002
View: 37401002

Created on 2017-05-13.15:09:08 by ced, last changed by roundup-bot.

Messages
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
New changeset 8e58e96bd54b by C?dric Krier in branch 'default':
Use root to retrieve company and do not fail on missing line for delete/update
http://hg.tryton.org/modules/timesheet/rev/8e58e96bd54b
review37401002 updated at https://codereview.tryton.org/37401002/#ps1
msg33652 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2017-05-13.15:09:07
The delete operation needs to read the line to get the company/employee to browse it.
So we should first browse as root and then re-browse with the right context.
Also it should not fail to delete or update a line that does not exist any-more.
History
Date User Action Args
2017-05-31 00:29:01roundup-botsetmessages: + msg33830
2017-05-22 18:04:16roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg33735
2017-05-13 15:37:35reviewbotsetnosy: + reviewbot
messages: + msg33653
2017-05-13 15:12:23cedsetstatus: in-progress -> testing
reviews: 37401002
keyword: + review
2017-05-13 15:09:08cedcreate

Showing 10 items. Show all history (warning: this could be VERY long)