From eaa5226e549114d9289568b3ae5ea1312b7c98bd Mon Sep 17 00:00:00 2001
From: Mike Richards <mrichards42@gmx.com>
Date: Mon, 6 May 2024 21:58:17 -0400
Subject: [PATCH] Strip html tags from completion dialog

This is a native message ctrl that can't contain html tags, so strip
them out like we do titles.
---
 src/MyFrame.cpp    | 9 +++++----
 src/html/parse.hpp | 6 ++++++
 2 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/src/MyFrame.cpp b/src/MyFrame.cpp
index 36e9a6f7..d1362d8d 100644
--- a/src/MyFrame.cpp
+++ b/src/MyFrame.cpp
@@ -982,8 +982,7 @@ MyFrame::ShowMetadata()
         SetTitle(XWORD_APP_NAME);
     else {
         // Puzzle titles can contain HTML, so parse out the plain text.
-        PlainTextHtmlParser parser;
-        wxString title = * (wxString*) (parser.Parse(puz2wx(m_puz.GetTitle())));
+        wxString title = PlainTextHtmlParser::StripTags(puz2wx(m_puz.GetTitle()));
         SetTitle(title + _T(" - ") XWORD_APP_NAME);
     }
 
@@ -1095,8 +1094,10 @@ MyFrame::CheckPuzzle()
             m_status->SetAlert(_T("The puzzle is filled correctly!"),
                 wxGetApp().GetConfigManager().Status.completeColor());
             // Show completion message for acrostics since it is often the formatted quote.
-            if (m_puz.GetGrid().IsAcrostic() && m_puz.HasMeta(puzT("completion")))
-                XWordMessage(this, m_puz.GetMeta(puzT("completion")));
+            if (m_puz.GetGrid().IsAcrostic() && m_puz.HasMeta(puzT("completion"))) {
+                wxString message = PlainTextHtmlParser::StripTags(puz2wx(m_puz.GetMeta(puzT("completion"))));
+                XWordMessage(this, message);
+            }
             break;
         case UNCHECKABLE_PUZZLE:
             StopTimer();
diff --git a/src/html/parse.hpp b/src/html/parse.hpp
index 574acedf..142d2013 100644
--- a/src/html/parse.hpp
+++ b/src/html/parse.hpp
@@ -50,6 +50,12 @@ class PlainTextHtmlParser : public wxHtmlParser
         return (wxObject*) &text;
     }
 
+    static wxString StripTags (const wxString & input) {
+        PlainTextHtmlParser parser;
+        wxString text = * (wxString*) parser.Parse(input);
+        return text;
+    }
+
 private:
     wxString text;
 };