Ampersand should be escaped

Bug #191038 reported by Philipp Kern
2
Affects Status Importance Assigned to Milestone
Me TV
Fix Released
High
Unassigned

Bug Description

(me-tv:9127): Gtk-WARNING **: Failed to set text from markup due to error parsing markup: Error on line 1: Character ' ' is not valid at the start of an entity name; the & character begins an entity; if this ampersand isn't supposed to be an entity, escape it as &

You should escape ampersands from HTML-unaware sources with &...

Revision history for this message
Michael Lamothe (lamothe-deactivatedaccount-deactivatedaccount) wrote :

Hi Philipp, I can only assume that this is from the channels.conf. I'm doing this properly everywhere else. There might be a translated string that has a ampersand in it.

Do you know where it's coming from?

Changed in me-tv:
assignee: nobody → michael-lamothe
importance: Undecided → High
milestone: none → station-5
status: New → In Progress
Revision history for this message
Philipp Kern (pkern) wrote :

At least channels.conf contains some ampersands, so it's likely that they are the origin. (All channel names are copied verbatim from the scan.) So probably quoting should be added here? (I doubt that people should be allowed to tweak the channel name with HTML here...)

Revision history for this message
Michael Lamothe (lamothe-deactivatedaccount-deactivatedaccount) wrote :

Have confirmed that when you put an ampersand in the channel name then you get the error. I have confirmed that this fixes the issue.

Changed in me-tv:
status: In Progress → Fix Committed
Revision history for this message
Michael Lamothe (lamothe-deactivatedaccount-deactivatedaccount) wrote :

Fixed in 0.5.16 release

Changed in me-tv:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.