The function coherent_access_form_after_build makes two calls to form_builder(), which in Drupal 6 expects 3 arguments, but only two arguments are given. Adding $form_values as the third argument seems to fix it.

Comments

Anonymous’s picture

Anonymous (not verified) commented

Huh??

Sorry, unfortunately I'm not a coder and would not know where to begin with this task

Adding $form_values as the third argument seems to fix it.

.

I'm have modest home photography business and I recently put a site up using drupal. I like Coherent Access so that I can create content only clients to view only their photos online. But each time I create a new 'private' page I get

warning: Missing argument 3 for form_builder(), called in /home/mccammon/public_html/modules/coherent_access/coherent_access.module on line 362 and defined in /home/mccammon/public_html/includes/form.inc on line 841.

.

Any ideas? I'm currently using Drupal 6.3.

chuntley’s picture

chuntley commented

In my patched copy the calls to form_builder() are at the end of lines 363 and 376.

On line 363 I changed
form_builder($_POST['form_id'], $form['shared_editing']['edit']['remove_editors'])
to
form_builder($_POST['form_id'], $form['shared_editing']['edit']['remove_editors'],$form_values)

On line 376 I changed
form_builder($_POST['form_id'], $form['shared_editing']['view']['remove_viewers'])
to
form_builder($_POST['form_id'], $form['shared_editing']['view']['remove_viewers'],$form_values)

That made the warning messages go away.

jgraham’s picture

jgraham commented

Thanks for the help chuntley. I've been busy over here and will get to cleaning up the 6.x port and doing a proper release as soon as possible.

The form processing in 6 needs to be reworked and be cleaner given the AHAH options for forms I need to read up on this a bit. Unfortunately I have not had much spare time to come up to speed on this and do a proper port of this module to 6.

jgraham’s picture

jgraham commented
Status: Active » Fixed
Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.