The parameter to date_validate is actually a form field, not a whole form. See patch for D5.3

Files: 
CommentFileSizeAuthor
date_validate.patch817 byteslyricnz
PASSED: [[SimpleTest]]: [MySQL] 31,995 pass(es).
[ View ]

Comments

lyricnz’s picture

Patch also applies to D6 with an offset.

lyricnz’s picture

Status:Active» Needs review

Also in D7. Sheesh.

cafuego’s picture

Status:Needs review» Reviewed & tested by the community

Yup, works for me!

cweagans’s picture

Version:5.3» 7.x-dev

+1 from me as well.

Moving to 7.x. 5.x is no longer maintained. This actually might need to be set to 8.x and then backported to 7.x, but I'm guessing that it'll still apply the same, so I guess it doesn't matter =P

scor’s picture

date_validate.patch queued for re-testing.

Status:Reviewed & tested by the community» Needs work

The last submitted patch, date_validate.patch, failed testing.

lyricnz’s picture

Status:Needs work» Needs review

date_validate.patch queued for re-testing.

lyricnz’s picture

Status:Needs review» Reviewed & tested by the community

Resetting to RTBC, stupid patchbot.

webchick’s picture

Version:7.x-dev» 8.x-dev
Issue tags:+needs backport to D7
webchick’s picture

Version:8.x-dev» 6.x-dev

Committed to 8.x and 7.x.

Looks like this applies to 6.x too (with patch -p0).

webchick’s picture

lyricnz’s picture

If it applies, it's good to go. Change has no external impact.

Gábor Hojtsy’s picture

Status:Reviewed & tested by the community» Fixed

Superb, committed, thanks. I never imagined we even have a date field type in Drupal 6 :)

cweagans’s picture

That was probably the coolest patch ever.

Applied to three different major versions of Drupal? Impressive. =P

Status:Fixed» Closed (fixed)
Issue tags:-needs backport to D6

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