Tryton - Issues

 

Issue8104

Title Use by default equals operator for range
Priority feature Status resolved
Superseder Nosy List ced, reviewbot, roundup-bot
Type feature request Components sao, tryton
Assigned To ced Keywords review
Reviews 50761002
View: 50761002

Created on 2019-02-11.18:19:28 by ced, last changed by roundup-bot.

Messages
New changeset e9cf27ff2d63 by Cédric Krier in branch 'default':
Fill 'to' with 'from' value on filter box
https://hg.tryton.org/tryton-env/rev/e9cf27ff2d63
New changeset 27dcc6013dd5 by Cédric Krier in branch 'default':
Fill 'to' with 'from' value on filter box
https://hg.tryton.org/tryton/rev/27dcc6013dd5
New changeset 2e49820602fb by Cédric Krier in branch 'default':
Fill 'to' with 'from' value on filter box
https://hg.tryton.org/sao/rev/2e49820602fb
review50761002 updated at https://codereview.tryton.org/50761002/#ps20001
msg47113 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2019-02-19.15:46:13
I updated the patch to always fill the to part when the from is changed.
It solves the problem of the popup which change the date when a calendar cell is picked. Such that in previous version, the to date stayed to the first picked date.
Also I think it gives a more predictable behavior and enforce the behavior to always fill the from then the to.
review50761002 updated at https://codereview.tryton.org/50761002/#ps1
msg46946 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2019-02-11.18:21:44
Here is review50761002 which also include a refactoring to separate the widget and the "between" behavior to allow to reuse it later for numeral fields.
msg46945 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2019-02-11.18:19:27
From https://discuss.tryton.org/t/improve-search-filter-on-client/1093/9?u=ced

User may not always understand that the range should be filled with the same value on both side to be an equal. But it is often the operator that users want at first. So the idea is to fill the second part when the first is changed and the second is empty.
History
Date User Action Args
2019-02-26 16:22:19roundup-botsetmessages: + msg47228
2019-02-26 16:22:17roundup-botsetmessages: + msg47227
2019-02-26 16:22:15roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg47226
2019-02-19 16:08:51reviewbotsetmessages: + msg47116
2019-02-19 15:46:13cedsetmessages: + msg47113
2019-02-11 18:44:09reviewbotsetnosy: + reviewbot
messages: + msg46947
2019-02-11 18:21:44cedsetstatus: in-progress -> testing
reviews: 50761002
messages: + msg46946
keyword: + review
2019-02-11 18:19:28cedcreate