Select elements which are required can be submitted without any user input

These sorts of changes need to happen in the current development version, not a stable release.

Moving to normal priority since nothing is fundamentally broken.

I made small code lifting with testing and then I roll a patch.

Thanks for re-rolling and improving. Don't mark your own patches as RTBC -- I'd like to get another person to review this as well.

I tested it.
it works fine with ( | ). But I would expect not to need to use it :(

It also works fine if I get my values by a code in allowed values - PHP code. The only problem is that the
value is not showed in field settings default value select list. On the form is it than available.

It does not work with a node reference. There is still no
default value available :(

any ideas to get it work with node reference?

I would expect to have the
value at each time as an available option that cant be submited if the field is required.

Once again, thx for your contribution, it brought me forward.

Hi,
please can you tell me why the patch is not in 6.12?
thx

Is there anything further to do on this, it is marked as needs work, but the patch in #9 works. Presumably too late to get it in 7, but has it made it into 6 yet??

@DeeLay: #9 was ignored by simpletest so it can't go forward..

Ok so it's done for d7dev, please check..

new approach

WOW! it's seems it's bigger issue then I thought in the begging :)
40 fail(s), and 36,738 exception(es).

.. . Don't mark your own patches as RTBC -- I'd like to get another person to review this as well.

sure

2nd approach

I'm proposing new sub-select-#options element
#unselected

Example:

  $form['foo'] = array(
    '#type' => 'select',
    '#title' => t('Foo title'),
    '#required' => TRUE,
    '#options' => array(
      '#unselected' => TRUE,
      'title' => t('Titles only'),
      'teaser' => t('Titles plus teaser'),
      'fulltext' => t('Full text'),
    ),
  );

someone can make a review?! please. I think it's worth...

+++ includes/form.patch-please-select.inc	2010-03-11 13:42:42.507418314 +0100
@@ -1201,6 +1201,25 @@ function form_builder($form_id, $element
+  /**
+   * If element #type exist and #type == 'select' ¶
+   * and '#select > #unselected' == TRUE // mean that developer ask drupal form api to use '<Please select>' message

trailing white space

Powered by Dreditor.

1) I'm relatively sure that this bug does not exist in Drupal 7.

2) The patch implements something entirely different than this issue is about.

Most likely won't fix.

#9 was ignored because:

Drupal core patches: Run the diff command in the root Drupal directory where you're making a patch, the same directory that contains the index.php, cron.php, etc. files. Core patches should always be made from Drupal root.

see more: http://drupal.org/patch/create

so I create this again from drupal root folder

@sun:

1) I'm relatively sure that this bug does not exist in Drupal 7.

I agree > changing

2) The patch implements something entirely different than this issue is about.

yes to much coffee :)

This is technically a duplicate of #140783: A select list without #default_value always passes form validation and cannot be fixed for D6, as it has a potential impact on every select element in all forms throughout Drupal. To understand what I'm saying, make sure to read the entire aforementioned issue.

For D6, you need to implement a custom workaround for this problem in your module; i.e., manually validate that a user selected something on your own.

Tricky hint for how to achieve this in a simple way:

  '#required' => TRUE,
  '#options' => array(
    NULL => t('- Select -'),
  ),

@sun: ok I understand now. Sorry for hassle...