Notice: Undefined index: attributes in media_wysiwyg_token_to_markup() (line 121 of /html/sites/all/modules/media/modules/media_wysiwyg/includes/media_wysiwyg.filter.inc).

CommentFileSizeAuthor
#6 2327329-media_undefined_index-6.patch891 bytescircuscowboy
#5 2327329-media_undefined_index-5.patch966 bytesrecrit
#1 2327329-media_undefined_index.patch1.02 KBdrnikki
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

drnikki’s picture

drnikki CreditAttribution: drnikki commented
FileSize
2327329-media_undefined_index.patch1.02 KB
rooby’s picture

rooby CreditAttribution: rooby commented
Status: Active » Needs review

There is a patch here, setting to needs review.

I would probably say use isset() instead of array_key_exists() in this case but either works.

rooby’s picture

rooby CreditAttribution: rooby commented

I wonder why lots of other people aren't complaining about this.

It makes me wonder if there is some other problem causing this and this is just masking the real cause?

Status: Needs review » Needs work

The last submitted patch, 1: 2327329-media_undefined_index.patch, failed testing.

recrit’s picture

recrit CreditAttribution: recrit commented
Status: Needs work » Needs review
FileSize
2327329-media_undefined_index-5.patch966 bytes

Re-roll of the patch with latest 7.x-2.x cba92a0.

circuscowboy’s picture

circuscowboy CreditAttribution: circuscowboy commented
Status: Needs review » Reviewed & tested by the community
FileSize
2327329-media_undefined_index-6.patch891 bytes

Rerolled - also reviewed - solves the error for me.

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

Status: Fixed » Closed (fixed)

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