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

CommentFileSizeAuthor
date_validate.patch817 byteslyricnz

Comments

lyricnz’s picture

lyricnz commented

Patch also applies to D6 with an offset.

lyricnz’s picture

lyricnz commented
Status: Active » Needs review

Also in D7. Sheesh.

cafuego’s picture

cafuego commented
Status: Needs review » Reviewed & tested by the community

Yup, works for me!

cweagans’s picture

cweagans
He/Him
Primary language English
Location Boise, ID, USA
commented
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

scor commented

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

lyricnz commented
Status: Needs work » Needs review

date_validate.patch queued for re-testing.

lyricnz’s picture

lyricnz commented
Status: Needs review » Reviewed & tested by the community

Resetting to RTBC, stupid patchbot.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Version: 7.x-dev » 8.x-dev
Issue tags: +Needs backport to D7
webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
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

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Issue tags: -Needs backport to D7 +Needs backport to D6

x

lyricnz’s picture

lyricnz commented

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

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
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

cweagans
He/Him
Primary language English
Location Boise, ID, USA
commented

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.

Issue tags: +Needs backport to D6

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.