Using file_entity 7.x-2.0-beta1 and Media 7.x-2.x-dev (2014-Oct-14)

Attempting to save settings for a new Multimedia asset field, I get these 2 errors:

Notice: Undefined index: settings in media_field_instance_validate() (line 382 of [...]\sites\all\modules\media\media.module).
The upload destination must be one of the allowed schemes.

No matter Allowed URI schemes are checked or not, it has no effect.

CommentFileSizeAuthor
#11 2361113-media-impossible_to_save_field_settings-11.patch760 bytesandypost
#9 Imposible_to_save_field_settings-2361113-9.patch813 bytesrv0
#8 2361113-media-impossible_to_save_field_settings-8.patch685 bytesandreymaximov

Comments

anrikun’s picture

anrikun commented
Issue summary: View changes
anrikun’s picture

anrikun commented
Title: Impossible to save settings for a new Media field » Impossible to save settings for a new Multimedia asset field
Issue summary: View changes
troybthompson’s picture

troybthompson commented

Also with Media 7.x-2.0-alpha4 and File Entity 7.x-2.0-beta1 when saving existing field.

samueljon’s picture

samueljon commented

I can also confirm this error with Media module 7.x-2.0-alpha4 and file entity 7.x-2.0-beta1

baronmunchowsen’s picture

baronmunchowsen commented

I have this issue after upgrading to:

  • File Entity 7.x-2.0-beta1+1-dev
  • Media 7.x-2.0-alpha4+7-dev
jackhutton’s picture

jackhutton commented

getting this error message as well.

Notice: Undefined index: settings in media_field_instance_validate() (line 382 of /srv/bindings/4ff3da15687b43feb291615c28308d9e/code/sites/all/modules/media/media.module).
The upload destination must be one of the allowed schemes.

file entity 7.x-2.0-beta1, media 7.x-2.0-alpha4+7-dev

anrikun’s picture

anrikun commented
Priority: Normal » Major
andreymaximov’s picture

andreymaximov commented
Status: Active » Needs review
StatusFileSize
new 2361113-media-impossible_to_save_field_settings-8.patch685 bytes

I have the same issue and made small investigation.

This issue was introduced by the commit http://cgit.drupalcode.org/media/commit/media.module?id=e4c529a28990c1da... related to the issue #1289680: Widget does not respect Allowed URI Schemes

I've made patch that simply excludes multimedia asset fields from proposed validation.
Not sure this is correct handling of the issue but it should help you to continue using multimedia asset fields since it completely broken now.

rv0’s picture

rv0 commented
StatusFileSize
new Imposible_to_save_field_settings-2361113-9.patch813 bytes

If I understand correctly, above patch just bypasses the validation, so not really a fix.
Attached patch falls back to file_default_scheme().

I guess it's also not a valid fix though.

stephesk8s’s picture

stephesk8s commented

#8 worked for me. Thank you for posting.

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new 2361113-media-impossible_to_save_field_settings-11.patch760 bytes

This validation should be limited by file and image types so #9 is wrong.
But there could be any field type with this settings defined.

podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
commented

+1 RTBC #11

  • aaron committed b192a0e on 7.x-2.x 
    Issue #2361113 by andypost, rv0, AndreyMaximov: fixed Impossible to save...
aaron’s picture

aaron commented
Status: Reviewed & tested by the community » Fixed

Committed to http://drupalcode.org/project/media.git/commit/b192a0e.

A simple patch: my favorite kind!

Status: Fixed » Closed (fixed)

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

fazni’s picture

fazni commented

work for me tks

Status: Closed (fixed) » Needs review

aetinti queued 11: 2361113-media-impossible_to_save_field_settings-11.patch for re-testing.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs review » Closed (fixed)

Status: Closed (fixed) » Needs work

The last submitted patch, 11: 2361113-media-impossible_to_save_field_settings-11.patch, failed testing.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs work » Closed (fixed)