date_popup wipes out external validation function

Gribnif - August 8, 2008 - 17:11
Project:Date
Version:6.x-2.0-beta4
Component:Date Popup
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

Currently, the date_popup_process() function sets the '#element_validate' part of the form array to be a new array. This wipes out any value that might have been set by the code that generated the form element in the first place. The solution is to treat this as an array, and append the value. If that element of the form array does not already exist, an array is created. See the attached patch.

AttachmentSize
date_popup_validate.patch.txt1.6 KB

#1

KarenS - August 13, 2008 - 07:16
Status:patch (code needs review)» fixed

This was committed.

#2

Anonymous (not verified) - August 27, 2008 - 07:22
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.