Bug 491340

Summary: BoardGameGeek description not excaping HTML.
Product: [Applications] tellico Reporter: Stewart Webb <stew>
Component: generalAssignee: Robby Stephenson <robby>
Status: RESOLVED FIXED    
Severity: minor    
Priority: NOR    
Version First Reported In: 3.5.5   
Target Milestone: ---   
Platform: Arch Linux   
OS: Linux   
Latest Commit: Version Fixed/Implemented In: 4.0
Sentry Crash Report:

Description Stewart Webb 2024-08-06 08:37:40 UTC
SUMMARY
BoardGameGeek fetcher results in "description" field holding unescaped HTML.


STEPS TO REPRODUCE
1. Search any item using BGG plugin
2. Add to library
3. view "Description" field.

OBSERVED RESULT
presents a single, long row of text, with HTML escape sequences

EXPECTED RESULT
presents HTML decorated text

SOFTWARE/OS VERSIONS
All

ADDITIONAL INFORMATION
I've tested a presumed fix on my local machine, but adding the xsl attribute, `disable-output-escaping="yes"` to `<xsl:value-of select="description"/>` elements.
But I don't know your codebase, so didn't want to just supply a PR as-of-yet.
Comment 1 Robby Stephenson 2024-08-06 11:34:55 UTC
Git commit a6a8e8a0ec36a85257c5e8f07193895ea8a80553 by Robby Stephenson.
Committed on 06/08/2024 at 11:34.
Pushed by rstephenson into branch '4.0'.

Update description for *Geek sites to replace HTML entities

Corrections for BoardGameGeek, VideoGameGeek, and RPGGeek data sources
FIXED-IN: 4.0

M  +4    -0    ChangeLog
M  +5    -1    src/fetch/boardgamegeekfetcher.cpp
M  +5    -1    src/fetch/rpggeekfetcher.cpp
M  +5    -1    src/fetch/videogamegeekfetcher.cpp
M  +1    -0    src/tests/boardgamegeekfetchertest.cpp
M  +1    -0    src/tests/rpggeekfetchertest.cpp
M  +1    -0    src/tests/videogamegeekfetchertest.cpp

https://invent.kde.org/office/tellico/-/commit/a6a8e8a0ec36a85257c5e8f07193895ea8a80553
Comment 2 Robby Stephenson 2024-08-06 11:42:52 UTC
Git commit 53e89960080903644fd35ffe94b9c869a129076c by Robby Stephenson.
Committed on 06/08/2024 at 11:42.
Pushed by rstephenson into branch '3.5'.

Update description for *Geek sites to replace HTML entities

Corrections for BoardGameGeek, VideoGameGeek, and RPGGeek data sources
FIXED-IN: 4.0

M  +4    -0    ChangeLog
M  +5    -1    src/fetch/boardgamegeekfetcher.cpp
M  +5    -1    src/fetch/rpggeekfetcher.cpp
M  +5    -1    src/fetch/videogamegeekfetcher.cpp
M  +1    -0    src/tests/boardgamegeekfetchertest.cpp
M  +1    -0    src/tests/rpggeekfetchertest.cpp
M  +1    -0    src/tests/videogamegeekfetchertest.cpp

https://invent.kde.org/office/tellico/-/commit/53e89960080903644fd35ffe94b9c869a129076c