I have a vocabulary called Topics. I have a term called Science & Nature. I have View setup with contextual filters to override the title with %1 where the URL is topics/science-nature. Both the title and h1 are overwritten but display the & and not the &. I cannot get the ampersand to display correctly.

Thanks.

CommentFileSizeAuthor
#4 2361065-tvi-html-titles.patch520 bytesdrupal_jon

Comments

joegl’s picture

joegl commented
Issue summary: View changes
duaelfr’s picture

duaelfr
he/him
Primary language French
Location Montpellier, France
commented

Could you, please, add steps to reproduce?
https://www.drupal.org/contributor-tasks/add-steps-to-reproduce

joegl’s picture

joegl commented

Create a new vocabulary.
Create a term that has an ampersand ("&") in it.
Create a new content type with title and body.
Add the vocabulary as a term reference.
Create a dummy piece of content (or two).
Select the term with the ampersand ("&") in it.
Create a view for the content type.
Create a block display in that view.
Add contextual filter for "Has Taxonomy Term ID".
"When the filter is not available": Provide Default Value: Taxonomy Term ID from URL. Check "Load default filter from term page."
"When the filter is available...": Override Title: %1. Check "Reduce duplicates."
Enable TVI module.
Edit the vocabulary and use TVI to override the display with the view we created.
Load the taxonomy page for the term with the ampersand ("&") in it.

And I believe the actual HTML is:
& (like it's getting escaped twice).

drupal_jon’s picture

drupal_jon commented
StatusFileSize
new 2361065-tvi-html-titles.patch520 bytes

I can confirm this happens with similar steps to create. It seems like this is a recurrence of #1817570: Page titles are santitized, double encoding HTML entities (& to &).

I've attached a patch that uses the same approach as the older fix.

capfive’s picture

capfive commented

Just applied this and it worked :)

kevinquillen’s picture

kevinquillen commented
Version: 7.x-1.0-beta4 » 7.x-1.x-dev
Assigned: Unassigned » kevinquillen

kevinquillen’s picture

kevinquillen commented

Looks good. Committed.

kevinquillen’s picture

kevinquillen commented
Status: Active » Fixed
kevinquillen’s picture

kevinquillen commented

Status: Fixed » Closed (fixed)

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

krischanski’s picture

krischanski commented
Version: 7.x-1.x-dev » 8.x-1.x-dev

Have the same problem now with Version 8.x, so the patch is not useable.
Edit: It's not a problem with TVI, by using the standard taxonomy term view, the title is with & too.