Edit media/multiedit is broken since 2143469 was commited in media module [#2149013]

The function media_file_page_edit_multiple() called in media_gallery/media_gallery.pages.inc (around line 413) has been renamed and moved to a sub module in media 2.x with the commit found in #2143469: Spin-off bulk file upload support in dedicated module / project. As a result the following error now shows on the edit media page: Fatal error: Call to undefined function media_file_page_edit_multiple() in PATH/media_gallery/media_gallery.pages.inc on line 413.

Suspect this means that media_bulk_upload and plupload now need to be dependencies of media_gallery.

Comment	File	Size	Author
#1	Media_function_name_changes-2143469-1.patch	1 KB	Mirabuck

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

Mirabuck CreditAttribution: Mirabuck commented 4 December 2013 at 15:21

Status:

Active

» Needs review

File	Size
Media_function_name_changes-2143469-1.patch	1 KB

The attached patch should fix this issue.

Comment #2

spgd01 CreditAttribution: spgd01 commented 10 December 2013 at 14:40

Patch #1 works for me.

Comment #3

fibigerg CreditAttribution: fibigerg commented 15 December 2013 at 00:51

Working for me but not for 100%. I get redirect back to node page when trying to access multi edit page. from contextual link on gallery wrapper. "Edit media" tab at edit page works fine.

I have my gallery appended to article node. I think that could be a problem.

Comment #4

fibigerg CreditAttribution: fibigerg commented 15 December 2013 at 02:11

Solved, see https://drupal.org/node/2156869

Comment #5

saltednut

she/her

English

Chicago

CreditAttribution: saltednut commented 3 January 2014 at 21:49

I can confirm this does fix it for me on a fresh build. On another site I had running, I ran into an error where I also needed to change the include above the function call.

  // Load the Media module include file containing the form callback.
  // @todo To support AJAX enabled widgets within the form, we need to also add
  //   media.pages.inc to $form_state['build_info']['files']['menu']. Figure out
  //   how to integrate that properly with the Multiform module.
  module_load_include('inc', 'media', 'includes/media.pages');

Unsure about the @todo, but it seems like the safe route here might be also to change the module_load_include to module_load_include('inc', 'media_bulk_upload', 'includes/media_bulk_upload.pages');

If anyone who is more familiar with the Media Gallery system - and the specific @todo mentioned, that would be helpful.