Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#17245 closed defect (invalid)

[regression] DateTextBox: value (set upon initialisation) is no longer submitted to server

Reported by: alan.kay Owned by: Douglas Hays
Priority: undecided Milestone: 1.9.1
Component: Dijit - Form Version: 1.9.0
Keywords: Cc:
Blocked By: Blocking:

Description

I create a DateTextBox? and set its value as a date object upon initialisation as follows

new DateTextBox({ name: "test", value: new Date(2014, 05, 11) }, "testNode");

But the hidden field associated with the DateTextBox? (test) does not have its value set.

This is a cross browser issue and as far as I can tell was introduced in this version of dojo (I checked against 1.8.4 and did not encounter this problem)

Attachments (1)

17245.html (464 bytes) - added by bill 6 years ago.
test case, after loading type "dojo.query('input[name="test"]')[0].value" in console. it should show a date, not ""

Download all attachments as: .zip

Change History (9)

comment:1 Changed 6 years ago by alan.kay

Also seems to affect NumberTextBox? (and possibly more)

comment:2 Changed 6 years ago by bill

Milestone: tbd1.9.1
Owner: set to Douglas Hays
Status: newassigned

Started in 73f44da76b7b09cd80e3785e193cccd798158c92, from #16194.

Changed 6 years ago by bill

Attachment: 17245.html added

test case, after loading type "dojo.query('input[name="test"]')[0].value" in console. it should show a date, not ""

comment:3 Changed 6 years ago by bill

Summary: DateTextBox value (set upon initialisation) is no longer submitted to server[regression] DateTextBox: value (set upon initialisation) is no longer submitted to server

comment:4 Changed 6 years ago by Douglas Hays

Resolution: invalid
Status: assignedclosed

Programmatic widget instantiations require .startup() calls.

comment:5 Changed 6 years ago by alan.kay

But they didn't technically require .startup() calls before. They worked in previous versions of dojo and now they don't. Even the dojo reference guide doesn't doesn't use a startup call: http://dojotoolkit.org/reference-guide/1.9/dijit/form/TimeTextBox.html#programmatic-example

So nothing's going to get "fixed" on this to make it backward compatible?

comment:6 Changed 6 years ago by alan.kay

Just found http://dojotoolkit.org/documentation/tutorials/1.9/understanding_widgetbase/#startup that does say you have to call startup, and I know what I'm about to say isn't really a bug, but the discrepancies between the different forms of documentation is confusing at best. If the tutorial mentions calling startup then the reference guide probably should conform to that requirement as well.

comment:7 Changed 6 years ago by freddefisk

I guess anyone is free to do changes in the reference documentation, right?

Just login at http://livedocs.dojotoolkit.org/dijit/form/TimeTextBox with the same username and password as used here in trac.

comment:8 Changed 6 years ago by gbranchaud

I just wanted to say I also got burned with this issue. Thanks for the detailed comments and for the livedocs update.

Note: See TracTickets for help on using tickets.