PHP notice in form_builder function

Any progress? I'm having this on Pressflow as well (along with some other bugs)

just corrected a bit the patch

again

Can you elaborate on why you feel this is necessary? There is no reason for #post not to be defined.

Because it produces a lot of notices in error_reporting(E_ALL) mode.
Perhaps it's an HS issue. I didnt investigate it deeply.

subscribe from pressflow. Something like this would probably be better.

if (isset($form['#post'])) {
  $form[$key]['#post'] = $form['#post'];
}

Setting to needs review for the testbot. Once it passes will switch this issue over to hierarchical select so we can come up with a proper fix in that module.

Patch above was green before moving this to the HS issue queue.
I believe the root issue is somewhere in the HS code base; thus I'm moving this issue into this project.

Having the same issue, the problem is on hierarchical_select.module, line 690:

  // This prevents values from in $element['#post'] to be used instead of the
  // generated default values (#default_value).
  // For example: $element['hierarchical_select']['selects']['0']['#default_value']
  // is set to 'label_0' after an "Add" operation. When $element['#post'] is
  // NOT unset, the corresponding value in $element['#post'] will be used
  // instead of the default value that was set. This is undesired behavior.
  if (isset($element['#post'])) {
    unset($element['#post']);  
  }

Removing this piece removes the problem, but I don't fully understand the implications.

OK, if I have understood the comment in code, the #post values where used instead of the #default_value.

If I remove this code, and load an edit form of a node with a hierarchical select, values are correctly set, so maybe those lines are not needed anymore?

Adding a patch.

Tidying up the issue queue. The 6.x version is no longer supported. Only reopen this if it also applies to the 7.x.