Hello,

This seems to be a regression of issue 2307993 with the latest version of the Media module for Drupal 7. I have re-rolled and attached a simpler patch to fix this issue. Hope it helps anyone who is experiencing this issue.

Cheers,

Michael Crump

CommentFileSizeAuthor
#2 media-2853617-preserve-script-tags-2.patch845 bytesmcrump
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mcrump created an issue. See original summary.

mcrump’s picture

mcrump CreditAttribution: mcrump commented
FileSize
media-2853617-preserve-script-tags-2.patch845 bytes

Here is a patch file for this issue.

mcrump’s picture

mcrump CreditAttribution: mcrump commented
Issue summary: View changes
joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented
Status: Active » Needs review

looks ok at first glance, haven't yet tested it.

Viktor-T’s picture

Viktor-T CreditAttribution: Viktor-T commented

Hi,

The patch was tested for about a month, no issue spotted, works like a charm. Still applicable to 2.1 security release.

Regards

joseph.olstad’s picture

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

Q: what version(s) of jQuery was (this issue observed on, and) the patch tested on?

  • mcrump authored e664d93 on 7.x-3.x 
    Issue #2853617 by mcrump, Viktor-T: WYSIWYG filter removes script tags...
joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented
Version: 7.x-2.x-dev » 7.x-3.x-dev
Assigned: Unassigned » joseph.olstad

committed to the 7.x-3.x dev branch

wait a bit before putting it in 7.x-2.x branch

joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented
Version: 7.x-3.x-dev » 7.x-2.x-dev
joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented
Issue tags: +minor version target

  • mcrump authored e664d93 on 7.x-2.x 
    Issue #2853617 by mcrump, Viktor-T: WYSIWYG filter removes script tags...
joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented
Status: Needs review » Fixed

This was just merged into 7.x-2.x and is included in release 7.x-2.6

joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented
Issue tags: -minor version target

Status: Fixed » Closed (fixed)

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