Comments

thedavidmeister’s picture

Issue tags: +Novice
InternetDevels’s picture

Assigned: Unassigned » InternetDevels

We are working today with this issue during Code Sprint UA.

InternetDevels’s picture

Assigned: InternetDevels » Unassigned
Status: Active » Needs review
FileSize
2.69 KB
PASSED: [[SimpleTest]]: [MySQL] 57,067 pass(es). View

Patch attached.

InternetDevels’s picture

Issue tags: +CodeSprintUA
thedavidmeister’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: -CodeSprintUA

This looks ok to me.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed b61918f and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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

thedavidmeister’s picture

Status: Closed (fixed) » Needs work

There is still a theme() call in FilterCaption.php

StephaneQ’s picture

Status: Needs work » Needs review
FileSize
1.16 KB
PASSED: [[SimpleTest]]: [MySQL] 57,886 pass(es). View
siccababes’s picture

Status: Needs review » Reviewed & tested by the community

I created an article and it worked fine.

xjm’s picture

webchick’s picture

Status: Reviewed & tested by the community » Fixed

C14N(), eh? And the award for the silliest method name goes to... :D

At any rate, this does seem indeed to just be taking what's there and moving it to a drupal_render() call, so...

Committed and pushed to 8.x. Thanks!

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