Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

qhp file corrupt when subsection with no parent section #8753

Closed
ferdymercury opened this issue Sep 1, 2021 · 5 comments
Closed

qhp file corrupt when subsection with no parent section #8753

ferdymercury opened this issue Sep 1, 2021 · 5 comments
Labels

Comments

@ferdymercury
Copy link
Contributor

ferdymercury commented Sep 1, 2021

Describe the bug
Consider this markdown page:

\mainpage %Test Page

### MySmallTitle
hello here

### AnotherTitle
hello there

Doxygen issues this warning,

mainpage.md:4: warning: found subsubsection command outside of subsection context!
mainpage.md:7: warning: found subsubsection command outside of subsection context!

but then, the QHP generation fails:

Error in line 14: Opening and ending tag mismatch.
error: failed to run qhelpgenerator on index.qhp

Expected behavior
Doxygen generates a valid QHP file even if there is a subsubsection with no parent section.

To Reproduce
Add the snippet above to a mainpage.md file, and add it as INPUT in a Doxyfile. Set GENERATE_QHP to YES

Version
1.9.2 built from sources

Additional context
Partially related #5880
#5892
root-project/root#8781 (comment)

@albert-github
Copy link
Collaborator

Looks like there are 2 </section>s too many, this is independent on the number of sections beforehand as when I use:

\mainpage %Test Page

### MySmallTitle
hello here

there are 2 </section>s too many as well.

albert-github added a commit to albert-github/doxygen that referenced this issue Sep 1, 2021
…ction

Decoupled the section level logic from the open/close logic.
@albert-github
Copy link
Collaborator

I've just pushed a proposed patch, pull request #8754

@ferdymercury
Copy link
Contributor Author

Thanks for the swift reply and fix!!

@albert-github albert-github added the fixed but not released Bug is fixed in github, but still needs to make its way to an official release label Sep 3, 2021
@albert-github
Copy link
Collaborator

Code has been integrated in master on GitHub (please don't close the issue as this will be done at the moment of an official release).

@doxygen
Copy link
Owner

doxygen commented Dec 31, 2021

This issue was previously marked 'fixed but not released',
which means it should be fixed in doxygen version 1.9.3.
Please verify if this is indeed the case. Reopen the
issue if you think it is not fixed and please include any additional information
that you think can be relevant (preferably in the form of a self-contained example).

@doxygen doxygen removed the fixed but not released Bug is fixed in github, but still needs to make its way to an official release label Dec 31, 2021
@doxygen doxygen closed this as completed Dec 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants