Date popup fields get wiped out, when there is a validation error and sometimes on submit.
Reproduce example: Edit a Timetracking with a date and clear the title. Now there is a validation error (no title) and the date is wiped out. (date_popup Module has to be enabled.)

Patch follows.

Files: 
CommentFileSizeAuthor
#4 storm_date-1418656-4.patch1.27 KBkfritsche
PASSED: [[SimpleTest]]: [MySQL] 260 pass(es).
[ View ]
#1 storm_date-1418656-1.patch1.27 KBkfritsche
PASSED: [[SimpleTest]]: [MySQL] 612 pass(es).
[ View ]

Comments

Title:Date field get wiped outDate popup fields get wiped out
Status:Active» Needs review
StatusFileSize
new1.27 KB
PASSED: [[SimpleTest]]: [MySQL] 612 pass(es).
[ View ]

I checked the date module. When the date popup is added through a process function, it sets the default_value and not the value.
If the value is set, the edit array in date_popup_process is empty.

After the patch, the default_value is used instead of the value.
This fixes the problem for me.

The date handling is an area I don't know very much about.

Just to ask, does this affect 1.x, or is it a recently introduced bug?

I think this affects 1.x too.
The bug is introduced in date 6.x-2.8. Somehow they handle things differently now. But this patch works for date 6.x-2.7 as well. As far as i understand date module, now it is the way it should be done...

I would commit this today and create another patch for 6.x-1.x...

Version:6.x-2.x-dev» 6.x-1.x-dev
StatusFileSize
new1.27 KB
PASSED: [[SimpleTest]]: [MySQL] 260 pass(es).
[ View ]

committed for 6.x-2.x

Here a rerolled patch against 6.x-1.x

Status:Needs review» Fixed

committed to 6.x-1.x

Status:Fixed» Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Issue summary:View changes

Added reproduce example