Changeset 2797

Timestamp:

03/02/2010 11:18:53 PM (16 years ago)

Author:

johnjamesjacoby

Message:

All field types should only return false if they are required and are empty.

File:

• trunk/bp-xprofile.php (modified) (2 diffs)

trunk/bp-xprofile.php

@@ -727 +727 @@
         $field = new BP_XProfile_Field( $field_id );
 
-        /* Check the value is an acceptable value */
+        /* Check that value is acceptable */
         if ( 'checkbox' == $field->type || 'radio' == $field->type || 'selectbox' == $field->type || 'multiselectbox' == $field->type ) {
                 $options = $field->get_children();
+
+                /* Not required fields can have empty value */
+                if ( !$is_required )
+                        $possible_values[] = '';
 
                 foreach( $options as $option )
@@ -740 +744 @@
                         }
 
-                        if ( empty( $value ) )
+                        /* Field must be required to trigger error */
+                        if ( $is_required && empty( $value ) )
                                 return false;