Bug 99500

Summary: digikam file property toolstip window: comments text interprets 'html' tags but it should not
Product: [Applications] digikam Reporter: Achim Bohnet <ach>
Component: Usability-i18nAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: caulier.gilles
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed/Implemented In: 7.5.0
Sentry Crash Report:

Description Achim Bohnet 2005-02-16 01:41:09 UTC 
Version:           0.7.2-beta1 (using KDE 3.3.2,  (3.1))
Compiler:          gcc version 3.3.5 (Debian 1:3.3.5-6)
OS:                Linux (i686) release 2.6.10-1-686

In early 0.7 cvs days there was a discussion how to interpret
contents of comments field and it was agreed to display
as is.   Comments displayed below thumbnail and html gallery
still use this convention but thumbnail tooltip that shows the
image properties does not.

My test case:   <b>bold</b> <it>italics</it>

Achim
Comment 1 Renchi Raju 2005-02-16 02:13:41 UTC 
CVS commit by pahlibar: 



show tooltip text as actual text. also set format in commentseditor to 
use plaintext
BUGS: 99500


  M +3 -1      albumfiletip.cpp   1.21
  M +1 -0      imagedescedit.cpp   1.41


--- kdeextragear-3/digikam/digikam/albumfiletip.cpp  #1.20:1.21
@@ -385,4 +385,7 @@ QString AlbumFileTip::breakString(const 
     QString str = input.simplifyWhiteSpace();
 
+    str.replace(QChar('<'), "&lt;");
+    str.replace(QChar('>'), "&gt;");
+                 
     uint maxLen = 30;
 

--- kdeextragear-3/digikam/digikam/imagedescedit.cpp  #1.40:1.41
@@ -105,4 +105,5 @@ ImageDescEdit::ImageDescEdit(AlbumIconVi
     QVGroupBox* commentsBox = new QVGroupBox(i18n("Comments"), plainPage());
     m_commentsEdit = new KTextEdit(commentsBox);
+    m_commentsEdit->setTextFormat(QTextEdit::PlainText);
 #if KDE_IS_VERSION(3,2,0)
     m_commentsEdit->setCheckSpellingEnabled(true);
Comment 2 Renchi Raju 2005-02-16 19:49:06 UTC 
CVS commit by pahlibar: 



Better solution. escape all html characters
CCBUGS: 99500


  M +2 -2      albumfiletip.cpp   1.22


--- kdeextragear-3/digikam/digikam/albumfiletip.cpp  #1.21:1.22
@@ -31,4 +31,5 @@
 #include <qpainter.h>
 #include <qapplication.h>
+#include <qstylesheet.h>
 
 // KDE includes.
@@ -385,6 +386,5 @@ QString AlbumFileTip::breakString(const 
     QString str = input.simplifyWhiteSpace();
 
-    str.replace(QChar('<'), "&lt;");
-    str.replace(QChar('>'), "&gt;");
+    str = QStyleSheet::escape(str);
                  
     uint maxLen = 30;
Comment 3 caulier.gilles 2021-12-30 15:50:22 UTC 
Fixed with works from bug #438701 for next 7.5.0 release

Gilles Caulier