Media tries to deal with a form that is not module-specific therefore the warning/notice shows up:

Notice: Undefined index: instance in media_form_field_ui_field_edit_form_alter() (Line 364

Shouldn't these lines be wrapped by a condition to check that this really is a media form?

I'm using commerce and dpm says here field is of type commerce_line_item_reference.

CommentFileSizeAuthor
#3 media-commerce-price-field-2597754-3.patch687 bytesy_h
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tsc_a created an issue. See original summary.

tasc’s picture

tasc
Primary language German
Location Berlin
CreditAttribution: tasc commented
Issue summary: View changes
y_h’s picture

y_h CreditAttribution: y_h commented
FileSize
media-commerce-price-field-2597754-3.patch687 bytes

I've got a similar issue in the same function when editing a commerce_price field:

Notice: Undefined index: widget in media_form_field_ui_field_edit_form_alter() (line 371

so i thought i'd post it here with a patch.

joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented
Status: Active » Needs review
donapis’s picture

donapis CreditAttribution: donapis commented
Status: Needs review » Reviewed & tested by the community

Hi,

Patch #3 is working for me.

joseph.olstad’s picture

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

patch looks good thanks.

  • y_h authored 54cbc4b on 7.x-2.x 
    Issue #2597754 by y_h: Undefined index: instance in...
joseph.olstad’s picture

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

fixed in 7.x-2.x dev ... Not serious enough to warrant a release tag on its own. But safe enough to commit to dev , harmless check.

Thanks!

  • y_h authored 54cbc4b on 7.x-3.x 
    Issue #2597754 by y_h: Undefined index: instance in...
joseph.olstad’s picture

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

This was included in 7.x-2.0 , tagged and released today, april 6th 2017

Status: Fixed » Closed (fixed)

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