At the present there's no way to hook into the form created by media_file_page_edit_multiple(). I need this to add some form elements that apply to all files being edited.

CommentFileSizeAuthor
#3 multiform-1889652.patch793 bytesericduran
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

czigor’s picture

czigor CreditAttribution: czigor commented
Project: D7 Media » Multiple forms
Version: 7.x-2.0-unstable7 » 7.x-1.x-dev

I think this is a Multiple forms issue instead.

firfin’s picture

firfin CreditAttribution: firfin commented

@czigor did you find a way to do this yet?

Anybody have any hints?

ericduran’s picture

ericduran CreditAttribution: ericduran commented
Issue summary: View changes
FileSize
multiform-1889652.patch793 bytes

This actually adds a hook in the multiform_get_form right before the redirect.

There is literally no way of altering anything in that form. The params can probably be changed a bit we can probably also move it to above the form_set_cache but I didn't really need that for my purposes.

Elijah Lynn’s picture

Elijah Lynn
Location Portland, Oregon
CreditAttribution: Elijah Lynn commented
Related issues: +#1944062: hook_form_multiform_alter() doesn't work as expected
Elijah Lynn’s picture

Elijah Lynn
Location Portland, Oregon
CreditAttribution: Elijah Lynn commented
Related issues: +#2067091: Sticky Edit Actions doesn't work with the Media module's multiform "Save" and "cancel" buttons.

This would be useful, my memory is faint but I think we ended up submitting a patch to media to alter the multiform in Media.

https://drupal.org/node/2067091

kalman.hosszu’s picture

kalman.hosszu CreditAttribution: kalman.hosszu commented
Status: Active » Needs review
mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman commented
Status: Needs review » Reviewed & tested by the community

This allows us to finally alter the redirect of a multiform. Due to the lack of form_state it's getting a cached form state which prevents any form_alters from working (and I wasn't able to find a way to attach custom submit handler to multiform.)

Proteo’s picture

Proteo CreditAttribution: Proteo as a volunteer commented

Patch from #3 worked for me. Media Bulk Upload need this to allow bulk field editing:

Bulk field editing after bulk uploading

Any chance to get it commited?

byronveale’s picture

byronveale CreditAttribution: byronveale as a volunteer commented

I can also confirm this is working, presently with media-7.x-2.0-beta5 with patch, multiform-7.x-1.1, and plupload-7.x-1.7.

byronveale’s picture

byronveale CreditAttribution: byronveale as a volunteer commented

Looking for maintainer input: the Media patch mentioned in my last comment has been committed (to 7.x-2.x-dev), but it depends on the patch in comment #3, which, wow, is from two and a half years ago; is there any intent, one way or the other, to get this committed?

joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented

This looks like a project that needs a maintainer. I'll make a new issue offering to maintain. Feel free to join me.

#2826351: Offer to maintain the 'multiform' project 7.x branch

joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented

@byronveale has been following up with this issue.
RTBC++ still waiting for maintainer access.

joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented
Status: Reviewed & tested by the community » Fixed

this is in release 7.x-1.2

Status: Fixed » Closed (fixed)

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