From 886f368fd9062ec20d34cb579a78e6b99aa451e1 Mon Sep 17 00:00:00 2001
From: Leon Styhre <leon@leonstyhre.com>
Date: Mon, 30 Jan 2023 19:41:24 +0100
Subject: [PATCH] Hex-encoded ampersand characters in game names are now
 converted correctly when scraping with ScreenScraper.

---
 es-app/src/scrapers/ScreenScraper.cpp | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/es-app/src/scrapers/ScreenScraper.cpp b/es-app/src/scrapers/ScreenScraper.cpp
index d3a14775c..2ad4e126f 100644
--- a/es-app/src/scrapers/ScreenScraper.cpp
+++ b/es-app/src/scrapers/ScreenScraper.cpp
@@ -351,6 +351,8 @@ void ScreenScraperRequest::processGame(const pugi::xml_document& xmldoc,
                                                            {region, "wor", "us", "ss", "eu", "jp"})
                                   .text()
                                   .get()};
+        // Game names sometimes contain ampersands encoded as hex codes.
+        gameName = Utils::String::replace(gameName, "&#x26;", "&");
         // In some very rare cases game names contain newline characters that we need to remove.
         result.mdl.set("name", Utils::String::replace(gameName, "\n", ""));
         LOG(LogDebug) << "ScreenScraperRequest::processGame(): Name: " << result.mdl.get("name");