After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 719389 - HTML escaping incorrectly applied in source tooltips
HTML escaping incorrectly applied in source tooltips
Status: RESOLVED FIXED
Product: doxygen
Classification: Other
Component: general
1.8.5
Other Mac OS
: Normal major
: ---
Assigned To: Dimitri van Heesch
Dimitri van Heesch
Depends on:
Blocks:
 
 
Reported: 2013-11-27 03:05 UTC by Jason Erb
Modified: 2013-12-24 18:59 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Minimal example based on the information provided (27.03 KB, application/octet-stream)
2013-12-15 18:01 UTC, albert
Details

Description Jason Erb 2013-11-27 03:05:56 UTC
HTML entities in source browser tooltips get HTML escaped, resulting in tooltips containing HTML entities such as " and > instead of double quote and greater than, respectively.
Comment 1 albert 2013-12-01 19:24:13 UTC
Can you please attach a self-contained example (source+config file in a tar or
zip) that allows reproducing the problem?
Comment 2 Jason Erb 2013-12-15 17:25:28 UTC
Here's a page that you can visit that demonstrates the behaviour:

http://sparist.github.io/Om/front__pull__term__operation_8hpp_source.html

To reproduce, hover the class name FrontPullTermOperation in the source code.  In the popup, you will see "The <-[terms] operation implementation.", which is taken from the brief comment for FrontPullTermOperation.

Source file for FrontPullTermOperation: https://github.com/sparist/Om/blob/master/code/om/operation/front_pull_term_operation.hpp

The om__operation__front_pull_term_operation__ reference is defined here:
https://github.com/sparist/Om/blob/master/code/om/operation/front_pull_term_operation.dox
Comment 3 albert 2013-12-15 18:01:01 UTC
Created attachment 264232 [details]
Minimal example based on the information provided

Minimal example based on the information provided
Comment 4 albert 2013-12-15 18:32:53 UTC
Looks like the docify function is called twice.
The input for docify is already:
  The <-[terms] operation implementation.
after docify call (line 1420 of htmlgen.cpp: docify(desc);) this is:
  The <-[terms] operation implementation.

Possible problem / solution around line 1314 of htmlgen.cpp (case '&':  m_t << "&amp;"; break;)

I'm not sure if the docify call should be removed or the docify subroutine shouldn't do the second substitution or ...
Comment 5 Dimitri van Heesch 2013-12-23 21:52:16 UTC
Confirmed. Should be fixed in the next GIT update.
Comment 6 Dimitri van Heesch 2013-12-24 18:59:27 UTC
This bug was previously marked ASSIGNED, which means it should be fixed in
doxygen version 1.8.6. Please verify if this is indeed the case. Reopen the
bug if you think it is not fixed and please include any additional information 
that you think can be relevant (preferrably in the form of a self-contained example).