use array_unshift() instead of array_reverse()

something is clobbering the #submit array prior to the form getting to LT.

i suspect it's the way inline reg. is handling the #submit array in it's form_alter function. it's a bug that inline reg is using array_reverse() to put it's submit handler first -- that completely wrecks the original order of the handlers, which might be important depending on what else is installed. instead, the module should do:

array_unshift($form['#submit'],'inline_registration_submit'); 

that adds the handler to the front of the array without disturbing the other elements.

you're using array_unshift() incorrectly. replace your array_reverse() line with the exact line of code i provide in #1.

fyi, it's very easy to search php.net for function help: simply visit php.net/<name_of_function>, like:
http://php.net/array_unshift

what's the exact error message?

it's not a bug in the code i've given you. something other module is doing nasty things with the $form['#submit'] array, and it shouldn't be.

your issue is out of scope for the bug reported. i suggest you start disabling other contrib modules until the problem goes away, then file another bug with that module.

let's make this issue about fixing the spotted problem with inline reg.

disable *all* other contrib modules, including inline registration. the only contrib module that should stay enabled for this test is logintoboggan.

if you still have the error after you perform this test, then my only other guess is that there's something seriously broken with your drupal base installation.

This happen when login tobogan and inline registration modules are instaled together. I'm also not very skilled programmer, but this error happen when the array is empty, so simple add condition into login tobogan module:

i covered this

$key = array_search('user_register_submit', $form['#submit']);
        if ($key !== FALSE) {
          unset($form['#submit'][$key]);
        }
        array_unshift($form['#submit'],'logintoboggan_user_register_submit');

into this simple condition:

      if ($form['#submit']) {
        $key = array_search('user_register_submit', $form['#submit']);
        if ($key !== FALSE) {
          unset($form['#submit'][$key]);
        }
        array_unshift($form['#submit'],'logintoboggan_user_register_submit');
      }

That works for me well!

Would be great if someone will find better solution. Rather change code in inline registration module, cause its still in development? Anyway looks like its more login tobogan problem, because we always have to assume empty variables :)
Sorry for my english :\

Anyway looks like its more login tobogan problem, because we always have to assume empty variables :)

no, we shouldn't have to assume that's an empty variable. core should *always* populate that variable, and no module should ever completely unset it.

as a test, enable just core and the logintoboggan module, and see if you get any errors. then enable core, logintoboggan, and inline registration, and see if you get any errors. if you get no errors in the first case, but errors in the second case, then you've triangulated the problem ;)

Rather change code in inline registration module, cause its still in development?

this has nothing to do with anything ;) a bug is a bug, no matter what stage of development a module is in...

Here is a patch that uses array_unshift() in the right position.

In addition to this line, that uses array_reverse

    // And ensure our submit function is called first (so the node is authored by the newly created user)
    $form['#submit'] = array_reverse($form['#submit']);

there's another line that has to be changed, otherwise the 'inline_registration_submit' callback gets added twice:

    // Add our own validation and submit function to the node_form
    $form['#validate'][] = 'inline_registration_validate';
    $form['#submit'][] = 'inline_registration_submit';

Issue is very old. Closing it.