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

Comments

drnikki’s picture

drnikki commented
StatusFileSize
new 2327329-media_undefined_index.patch1.02 KB
rooby’s picture

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 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 commented
Status: Needs work » Needs review
StatusFileSize
new 2327329-media_undefined_index-5.patch966 bytes

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

circuscowboy’s picture

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

Rerolled - also reviewed - solves the error for me.

joseph.olstad’s picture

joseph.olstad
Primary language French
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.