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

"Packages" has a "Packages" in the tree #9192

Closed
buhtz opened this issue Mar 8, 2022 · 9 comments
Closed

"Packages" has a "Packages" in the tree #9192

buhtz opened this issue Mar 8, 2022 · 9 comments

Comments

@buhtz
Copy link

buhtz commented Mar 8, 2022

In the tree of my generated documentation the node/entry Packages appears two times.

image

Sidenode: This is not about mypackage which I know why it appears two times in the tree.

The full project including generated html is in that archive: project.zip.
.

I use doxygen 1.9.4 (57efad44bfc02744ba5fd8ea820b54a2443e3771*) from git on Debian 11 (Linux TONNE 5.10.0-11-amd64 #1 SMP Debian 5.10.92-1 (2022-01-18) x86_64 GNU/Linux) with Python 3.9.

This is the file and folder structure of the project:

project
├── docs
│   ├── Doxyfile
│   └── html
│       ├── ...
│       └── ...
├── LICENSE
├── README.md
├── src
│   ├── mypackage
│   │   ├── __init__.py
│   │   └── mypackage.py
│   ├── setup.cfg
│   └── setup.py
└── tests

This is doxygen -x:

# Difference with default Doxyfile 1.9.4 (57efad44bfc02744ba5fd8ea820b54a2443e3771*)
PROJECT_NAME           = Project
OPTIMIZE_OUTPUT_JAVA   = YES
EXTRACT_ALL            = YES
EXTRACT_PRIVATE        = YES
EXTRACT_STATIC         = YES
EXTRACT_LOCAL_METHODS  = YES
SORT_MEMBER_DOCS       = NO
WARN_AS_ERROR          = YES
INPUT                  = ../src/mypackage
FILE_PATTERNS          =
RECURSIVE              = YES
EXCLUDE                = ./_trash
EXAMPLE_PATTERNS       =
HTML_TIMESTAMP         = YES
GENERATE_TREEVIEW      = YES
MATHJAX_RELPATH        = https://cdn.jsdelivr.net/npm/mathjax@3
GENERATE_LATEX         = NO
HAVE_DOT               = YES
UML_LOOK               = YES
DOT_UML_DETAILS        = YES
@buhtz buhtz changed the title "package" has a "package" in the tree "Packages" has a "Packages" in the tree Mar 8, 2022
@albert-github
Copy link
Collaborator

Which version did you use, 1.9.4 is the current master so the full id is necessary from doxygen -v

@buhtz
Copy link
Author

buhtz commented Mar 8, 2022

Dear Albert,
thanks for asking back.

Which version did you use, 1.9.4 is the current master so the full id is necessary from doxygen -v

It is 1.9.4 (57efad44bfc02744ba5fd8ea820b54a2443e3771*). You can see the ID in the doxygen -x output but I also updated the whole Issue.

@albert-github
Copy link
Collaborator

Oops, I missed the doxygen -x output.

@buhtz
Copy link
Author

buhtz commented Mar 10, 2022

I am wondering if I am the only person who observe this phenomenon?
Maybe I did something wrong?

albert-github added a commit to albert-github/doxygen that referenced this issue Mar 10, 2022
The problem here is the setting `OPTIMIZE_OUTPUT_JAVA   = YES, when  not having this setting we see "Namespaces" and "Namespace list" though for the "Package list" there is no translation function.
A function `trPackageList` has been introduced.
@albert-github
Copy link
Collaborator

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

@buhtz
Copy link
Author

buhtz commented Mar 10, 2022

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

Not gone in my case. #9197 (comment)

@albert-github
Copy link
Collaborator

For completeness: Discussion is running at with the pull request #9197, looks like patch has not been applied locally.

albert-github added a commit to albert-github/doxygen that referenced this issue Mar 14, 2022
- renaming original trPackageMembers to trPackageFunctions
- renaming trStaticPackageMembers to trStaticPackageFunctions
- introducing new function trPackageMembers with best / educated guess for the translation.

(based on comment with doxygen#9192 / doxygen#9197)
doxygen added a commit that referenced this issue Apr 10, 2022
issue #9192 "Packages" has a "Packages" in the tree
@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 Apr 11, 2022
@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 May 5, 2022

This issue was previously marked 'fixed but not released',
which means it should be fixed in doxygen version 1.9.4.
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 May 5, 2022
@doxygen doxygen closed this as completed May 5, 2022
petyovsky pushed a commit to petyovsky/doxygen that referenced this issue Aug 26, 2022
removed all implicit conversion from `QCString` to `const char *`,
fixed issues: doxygen#7434, doxygen#8404, doxygen#9192,
fixed typo and spacing in: `trLegendDocs()`,
fixed plural in: `trTemplateParameters()`,
better translation considering the context: `trMemberTypedefDocumentation()`,
  `trMemberEnumerationDocumentation()`, `trMemberFunctionDocumentation()`,
  `trMemberDataDocumentation()`, `trDefineDocumentation()`, `trDirDocumentation()`,
  `trTypeDocumentation()`.
updated translation in context of non OO languages: `trHierarchicalIndex()`
  `trGotoGraphicalHierarchy()`, `trGotoTextualHierarchy()`, `trCompoundMembersFortran()`.
updated translation in: `trNoMatches()`, trPackageAttribs(), trStaticPackageAttribs().
fixed wrong translation in: `trCompoundReferenceFortran()` of `case ClassDef::Class:`.
unified translation string in: `trGeneratedFromFilesFortran` according to `trGeneratedFromFiles()`.
fixed wrong wording in: `trCompoundReferenceSlice` of `case ClassDef::Interface:` when `isLocal==true`.
minor spacing and comments changes for better convergence to the reference file: `translator_en.h`.
all unclear or fuzzy czech translations marked with ???.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants