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 684781 - Mainpage sections are not shown as children in the treeview
Mainpage sections are not shown as children in the treeview
Status: RESOLVED FIXED
Product: doxygen
Classification: Other
Component: general
1.8.2-SVN
Other Linux
: Normal normal
: ---
Assigned To: Dimitri van Heesch
Dimitri van Heesch
Depends on:
Blocks:
 
 
Reported: 2012-09-25 13:12 UTC by Philipp Moeller
Modified: 2012-12-03 15:01 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Philipp Moeller 2012-09-25 13:12:17 UTC
Considering

    /*!
    \mainpage Mainpage
    
    \section intro Introduction
    Text
    \section more More Info
    More Text
    */

and

    /*!
    \page my_page A Page
    
    \section page_intro Introduction
    Text
    \section page_more More Info
    More Text
    */

and a default doxygen configuration with GENERATE_TREEVIEW = YES,
DISABLE_INDEX = YES the two section of the mainpage are not added as
children to the tree while the children of the normal page are.

The expected outcome is for the mainpage and page to behave
in the same way.
Comment 1 Dimitri van Heesch 2012-09-30 08:30:21 UTC
Confirmed. Should be fixed in the next subversion update.
Comment 2 Philipp Moeller 2012-10-01 11:40:10 UTC
The recent update shows the sections in the treeview, but they are not correctly nested. This is due to checks for Doxygen::mainPage in Line 3306 and Line 3325 in index.cpp, when the mainpage has been given a name. It works fine when the page has been given a name.

Replacing the checks with

  if ((hasSections || hasSubPages) && (pd!=Doxygen::mainPage || mainPageHasTitle()))

fixes this.
Comment 3 Dimitri van Heesch 2012-10-07 18:13:00 UTC
Confirmed. I'll include your fix in the next subversion update.
Comment 4 Philipp Moeller 2012-12-03 15:01:06 UTC
Fixed now.