https://codereview.tryton.org/413951003/diff/439801003/src/view/form.js File src/view/form.js (right): https://codereview.tryton.org/413951003/diff/439801003/src/view/form.js#newcode1830 src/view/form.js:1830: this.get_format(), this.date.val()); For me this is wrong to parse ...
https://codereview.tryton.org/413951003/diff/439801003/src/view/form.js
File src/view/form.js (right):
https://codereview.tryton.org/413951003/diff/439801003/src/view/form.js#newco...
src/view/form.js:1830: this.get_format(), this.date.val());
On 2022/11/23 14:03:16, ced wrote:
> On 2022/11/23 13:44:34, pokoli wrote:
> > On 2022/11/23 13:38:00, ced wrote:
> > > For me this is wrong to parse twice the date value.
> >
> > THis is not parsing the value twice, this is parsing the current value of
the
> > text to read the time.
>
> When called from get_value.
>
get_value calls this._parse, so to avoid calling the current value I just
updated the code to avoid the recursive call.
Issue 413951003: sao: Use date input for DateTime picker on Firefox
Created 10 months ago by pokoli
Modified 10 months ago
Reviewers: reviewbot, ced
Base URL:
Comments: 4