Description: ------------ If default value for an element within repeated item is a null (which may easilly come from a database), instead of setting an empty value for it the value from a previous repeated item is kept. Possible fix: when iterating over items' indexes, should probably reset elements' values on each iteration instead of once in restoreChildAttributes() Test script: --------------- $form = new HTML_QuickForm2('repeat-bug'); $form->addDataSource(new HTML_QuickForm2_DataSource_Array(array( 'buggy' => array( 'name' => array( 1 => 'First', 2 => 'Second' ), 'extra' => array( 1 => 'Has extra', 2 => null ) ) ))); $group = new HTML_QuickForm2_Container_Group('buggy'); $group->addText('name'); $group->addText('extra'); $repeat = $form->addRepeat(null, array('id' => 'buggy-repeat'), array('prototype' => $group)); $value = $repeat->getValue(); var_dump($value['buggy']['extra'][2]); Expected result: ---------------- NULL or maybe string(0) "" Actual result: -------------- string(9) "Has extra"

Fixed in Git. https://github.com/pear/HTML_QuickForm2/commit/e58cb3af31af0f00875bc8a6c919a1fd166c201a

Implemented a proper fix: elements now can check whether a DataSource contains a value for an element even if the value is null and use that value. https://github.com/pear/HTML_QuickForm2/commit/c07dd7ffa3d0468cd4f7df8be5ed62e9ae49e1fe