webform_localization_webform_submission_load fatals on empty $submissions

I had the same error multi-page forms in 7.x-1.x when "saving a draft between pages"
patch does not fix it (well, it fixes the error, but does not target the problem)
fixed with #2007756: Unable to save draft between pages on a multipage webform using a single form across the translation set

Ran into this bug by calling webform_get_submission() from my module code. Any time you attempt to get a non-existent submission, webform_localization_webform_submission_load() gets passed an empty array, and fatals. I can confirm that the patch fixes this issue.

@coleman.sean.c
please try the patch in the linked issue in #1
the patch here does not target the problem.

Respectfully, the problem you describe in the other thread is unrelated to this issue, although they have similar symptoms. I've done some more thorough testing, and found that this patch addresses the bug, and the patch in the other issue does not.

To isolate this particular bug, from a stock Drupal install with the minimal profile, enable webform, and a module that does the following:

function foo_init() {
  module_load_include('inc', 'webform', 'includes/webform.submissions');
  webform_get_submission(1000, 1000);
}

With just webform, the 'minimal' profile, and the dependencies for webform_localization, webform_get_submission() returns FALSE, and the page loads.

With webform_localization enabled, the page crashes.

With webform_localization enabled and the patch from #2007756-1: Unable to save draft between pages on a multipage webform using a single form across the translation set applied, the page still crashes.

With webform_localization enabled and the patch from this issue applied, the page loads correctly.

I've attached my test module if anyone wants to replicate my results.

edit: nvm

this patch wil indeed fix your issue (though not the one this thread is about, but it wont do any harm to have this code in there)

Thanks :)

I'm not being able to replicate this issue, seems to me that looking into current webform code the submissions array should never be empty in first place. Anyway the patch will do any harm ;) , thanks you all guys.