I added field "description" to image file type. Everything went ok but when i tried to fill this field under admin/content/media and saved it i see this message:
Notice: Undefined index: file in media_edit_submit() (line 366 of (...)\drupal\sites\all\modules\media\includes\media.pages.inc).
EntityMalformedException: Missing bundle property on entity of type file. in entity_extract_ids() (line 7562 of (...)\drupal\includes\common.inc).

CommentFileSizeAuthor
#2 media-pass_form_state_byref_media_edit-1723820-2.patch489 bytestedbow

Comments

rychu90’s picture

rychu90 commented
Priority: Normal » Major
tedbow’s picture

tedbow
he/him
Primary language English
Location Ithaca, NY, USA
commented
Version: 7.x-1.2 » 7.x-1.x-dev
Category: support » bug
Status: Active » Needs review
StatusFileSize
new media-pass_form_state_byref_media_edit-1723820-2.patch489 bytes

I have attached a patch that fixes this issue for me.

I believe this is caused $form_state is not passed by reference to the function media_edit. So right now:

function media_edit($form, &$form_state, $file) {
  $form_state['file'] = $file;

Doesn't affect $form_state outside of the form. Then in 

function media_edit_submit($form, &$form_state) {
  $file = $form_state['file'];

The file is not actually in $form_state.

Status: Needs review » Needs work

The last submitted patch, media-pass_form_state_byref_media_edit-1723820-2.patch, failed testing.

bleen’s picture

bleen commented

confirmed that this is an issue in 7.x-1.-dev.

Also, I tried the patch in #2 and I believe it is the correct solution. Obviously we need to figure out why its failing though...

tedbow’s picture

tedbow
he/him
Primary language English
Location Ithaca, NY, USA
commented

Ok, If you click view details for the patch you will see that all the exceptions(no errors found) are similar to this:

Use of undefined constant FILE_TYPE_NONE - assumed 'FILE_TYPE_NONE' Notice media.types.inc 245 media_type_invalid_files_count()

With different line numbers.

Is this an existing problem in Media? I don't see how my patch could have caused this. But this is the first time I have looked at Media's code.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs work » Fixed

Thanks, I committed #2 to Git. http://drupalcode.org/project/media.git/commit/6d25709

We can fix the test failures separately.

Status: Fixed » Closed (fixed)

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