Required not being respected [#1189814]

When you check the 'required' box, it is not being respected. The form validates with no input and the asterisk does not appear. Furthermore, when you add a default option such as '|Please select,' to force the user to select a different option, '|Please select' validates.

Comment	File	Size	Author
#2	0001-1189814-by-mradcliffe.-Add-additional-assertion-for-.patch	2.13 KB	mradcliffe
#2	0001-1189814-by-mradcliffe.-Fixed-required-property-not-b.patch	4.55 KB	mradcliffe

Comments

Comment #1

mradcliffe

he/him

English

commented 15 June 2011 at 23:46

I have a test in 7.x-1.x, and I know this behaves differently in 6.x-1.x. I'm porting over the test and will look into it (confirmed bug).

Comment #2

mradcliffe

he/him

English

commented 16 June 2011 at 00:57

Status:

Active

» Needs review

Status	File	Size
new	0001-1189814-by-mradcliffe.-Fixed-required-property-not-b.patch	4.55 KB
new	0001-1189814-by-mradcliffe.-Add-additional-assertion-for-.patch	2.13 KB

I can't do the asterisk at the moment, but I ported the Drupal 7 test to Drupal 6 and finished logic to respect the required property.

The first patch is for drupal 7 version to add an additional assertion I added when I backported the test to Drupal 6. The second patch is for drupal 6 dev. version.