In there is a '; hanging on the end of line 43. Because of the way this block of text is it shouldn't be there and is coming through in the display. This needs to be removed.

Members fund testing for the Drupal project. Drupal Association Learn more


pixelmord’s picture

I noticed that also, but only when Javascript is disabled.

See attached patch against alpha-4

mfer’s picture

Status: Active » Needs work

You can't put a # in the file name. Isn't letting me get to it.

pixelmord’s picture

Sorry, I wasn't thinking....
I will attach a new patch as soon as I get back in the office, but until then, just remove the two mentioned characters in Line 43 by hand.

pixelmord’s picture

Version: 7.x-1.x-dev » 7.x-1.0-alpha4
1003 bytes

so here is the patch with a url-safe name ...

aspilicious’s picture

Status: Needs work » Reviewed & tested by the community

Used this on my dev site, works :)

silkogelman’s picture

confirmed: #4 works on a fresh D7 install with these modules:

Thomas Bosviel’s picture

#4 works. Thanks.

effulgentsia’s picture

Title: Dangling '; » Video markup incorrectly appends '; after embed. Breaks html5 (iPad) playback for video embedded in wysiwyg.
Version: 7.x-1.0-alpha4 » 7.x-1.x-dev

Thanks! #4 looks good. I'll commit it when I get my GIT setup, or maybe another maintainer will beat me to it.

clemens.tolboom’s picture

Status: Reviewed & tested by the community » Needs review
2.16 KB

The patch from #4 is not complete. The $variables['output'] is set twice too.

Osiris Indriya’s picture

After using patch in #9, I am still seeing the ' in a views panel pane. Only the ; has been removed.

Osiris Indriya’s picture

23.97 KB

Here is a photo of the pane I am speaking of.

clemens.tolboom’s picture

You are correct.

Please try again.

aquariumtap’s picture

The patch from #12 worked for me. Thanks!

clemens.tolboom’s picture

Status: Needs review » Reviewed & tested by the community

Tested by aquariumtap

katbailey’s picture

Sorry for the noise, but I need a version of the patch that drush make can apply...

effulgentsia’s picture

Status: Reviewed & tested by the community » Fixed

Hey, I'm starting to get the hang of git. Committed to 7.x-1.x. Please re-open this issue if I screwed up the commit. Thanks.

Status: Fixed » Closed (fixed)

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