toolbar toggle link rendered without escaping ampersand as an html entity [#1095498]

<a href="/drupal7/?q=toolbar/toggle&destination=node" title="Hide shortcuts" class="toggle toggle-active">Hide shortcuts</a> </div>

is rendered when the following should be rendered (which is what I get with my patch to be attached as soon as I know the bug number):

<a href="/drupal7/?q=toolbar/toggle&destination=node" title="Hide shortcuts" class="toggle toggle-active">Hide shortcuts</a> </div>

This toggle link is rendered at least when logged in as administrator on the default drupal8 and drupal7 setups (patch applies to both branches).

Comment	File	Size	Author
#1	drupal-1095498.patch	1.1 KB	ohnobinki

Comments

ohnobinki commented 17 March 2011 at 02:55

Assigned:	ohnobinki	» Unassigned
Status:	Active	» Needs review

Status	File	Size
new	drupal-1095498.patch	1.1 KB

David_Rothstein commented 7 April 2011 at 04:14

Status:

Needs review

» Reviewed & tested by the community

I confirmed that this patch works as intended, and yeah, using l() here just makes good sense anyway. RTBC.

dries commented 12 April 2011 at 20:34

Status:

Reviewed & tested by the community

» Fixed

Committed to 7.x and 8.x.

26 April 2011 at 20:41

Status:	Fixed	» Closed (fixed)
Issue tags:	-xhtml compliance

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

26 April 2011 at 20:41

Issue tags: