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

CommentFileSizeAuthor
date_validate.patch817 byteslyricnz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lyricnz’s picture

lyricnz CreditAttribution: lyricnz commented

Patch also applies to D6 with an offset.

lyricnz’s picture

lyricnz CreditAttribution: lyricnz commented
Status: Active » Needs review

Also in D7. Sheesh.

cafuego’s picture

cafuego CreditAttribution: 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
CreditAttribution: cweagans 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 CreditAttribution: 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 CreditAttribution: lyricnz commented
Status: Needs work » Needs review

date_validate.patch queued for re-testing.

lyricnz’s picture

lyricnz CreditAttribution: 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 🇨🇦
CreditAttribution: webchick 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 🇨🇦
CreditAttribution: webchick 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 🇨🇦
CreditAttribution: webchick commented
Issue tags: -Needs backport to D7 +Needs backport to D6

x

lyricnz’s picture

lyricnz CreditAttribution: 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
CreditAttribution: Gábor Hojtsy 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
CreditAttribution: cweagans 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.