Tryton - Issues

 

Issue6946

Title wrong converter evaluation on timedelta using 'sum'
Priority bug Status chatting
Superseder Nosy List ced, perilla, reviewbot
Type Components tryton
Assigned To perilla Keywords review
Reviews 41691002
View: 41691002

Created on 2017-11-17.15:16:30 by perilla, last changed by reviewbot.

Files
File name Uploaded Type Edit Remove
after.png perilla, 2017-11-17.15:23:14 image/png
before.png perilla, 2017-11-17.15:23:01 image/png
Messages
review41691002 updated at https://codereview.tryton.org/41691002/#ps40001
review41691002 updated at https://codereview.tryton.org/41691002/#ps20001
msg36926 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2017-11-17.16:14:27
I does not make sense to use the domain parser context.
Indeed it should do like the TimeDeltaField (and maybe we should fix the TODO).
msg36925 (view) Author: [hidden] (perilla) Date: 2017-11-17.16:02:43
When evaluate timesheet lines, to get sum work time,
tryton use screen context to get converter and this
result on a void converter.  Indeed I think the correct
way to do this is using domain_parser context.
msg36922 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2017-11-17.15:37:49
Please provide a explanation of the issue.
review41691002 updated at https://codereview.tryton.org/41691002/#ps1
msg36920 (view) Author: [hidden] (perilla) Date: 2017-11-17.15:17:51
I think we should use domain_parser.
History
Date User Action Args
2017-11-24 15:51:42reviewbotsetmessages: + msg37033
2017-11-17 17:02:33reviewbotsetmessages: + msg36927
2017-11-17 16:14:27cedsetmessages: + msg36926
2017-11-17 16:02:43perillasetmessages: + msg36925
2017-11-17 15:48:31perillasetassignedto: perilla
2017-11-17 15:37:49cedsetnosy: + ced
messages: + msg36922
2017-11-17 15:35:56reviewbotsetnosy: + reviewbot
messages: + msg36921
2017-11-17 15:26:36perillasetreviews: 41691002
keyword: + review
2017-11-17 15:23:14perillasetfiles: + after.png
2017-11-17 15:23:01perillasetfiles: + before.png

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