CodeIgniter Forums
Form Helper set_radio unexpected default value - Printable Version

+- CodeIgniter Forums (https://forum.codeigniter.com)
+-- Forum: Archived Discussions (https://forum.codeigniter.com/forumdisplay.php?fid=20)
+--- Forum: Archived Development & Programming (https://forum.codeigniter.com/forumdisplay.php?fid=23)
+--- Thread: Form Helper set_radio unexpected default value (/showthread.php?tid=21454)

Form Helper set_radio unexpected default value - El Forum - 08-10-2009

[eluser]jpi[/eluser]
Hi,
From line 749 file form_helper.php
Code:
function set_radio($field = '', $value = '', $default = FALSE)
    {
        $OBJ =& _get_validation_object();

        if ($OBJ === FALSE)
        {
            if ( ! isset($_POST[$field]))
            {
                if (count($_POST) === 0)
                {
                    return ' checked="checked"';
                }
                return '';
            }

            $field = $_POST[$field];
            
            if (is_array($field))
            {
                if ( ! in_array($value, $field))
                {
                    return '';
                }
            }
            else
            {
                if (($field == '' OR $value == '') OR ($field != $value))
                {
                    return '';
                }
            }

            return ' checked="checked"';
        }

        return $OBJ->set_radio($field, $value, $default);
    }
Let's say I haven't initialized a form_validation object and have not yet submited the form that use set_radio() (so $_POST[] is empty). Then this function will return 'checked="checked"' whatever $default argument is.

I don't think it is the expected behavior.

Form Helper set_radio unexpected default value - El Forum - 08-18-2009

[eluser]fritzthecat[/eluser]
IAWTP - surely if $_POST is empty the function should use $default

Form Helper set_radio unexpected default value - El Forum - 08-27-2009

[eluser]Muser[/eluser]
I think it is a bug too. I am using Codeigniter 1.7.1 and for all the radios the attribute checked="checked" is parsed.