ZF-3018: The Method Zend_Form_Element_Checkbox::isChecked() requires an param which is not used.

Issue Type:
Bug
Created:
2008-04-02T03:50:16.000+0000
Last Updated:
2008-05-05T11:44:48.000+0000
Status:
Resolved
Fix version(s):
  • 1.5.2 (15/May/08)
Reporter:
Nick Lo (nicklo)
Assignee:
Matthew Weier O'Phinney (matthew)
Tags:
  • Zend_Form
Related issues:
Attachments:

Description

Line 188:


    /**
     * Get checked flag
     * 
     * @return bool
     */
    public function isChecked($flag)
    {
        return $this->checked;
    }
  • Submitted on behalf of Benjamin Cremer (crem0r on #zftalk)

Comments

Posted by Wil Sinclair (wil) on 2008-04-18T16:33:55.000+0000

Please evaluate and categorize/assign as necessary.

Posted by Matthew Weier O'Phinney (matthew) on 2008-04-22T13:08:26.000+0000

The public property 'checked' is defined in the source, and defaults to false; setting the element value and/or the flag will change its value.

Posted by Janek Noest (smoove) on 2008-04-23T05:41:10.000+0000

The problem is the method's parameter $flag, which is clearly never used, but required.

Posted by Matthew Weier O'Phinney (matthew) on 2008-04-23T06:54:19.000+0000

Re-opening based on user feedback.

Posted by Matthew Weier O'Phinney (matthew) on 2008-04-23T06:56:05.000+0000

Fixed in trunk and release-1.5 branch