Skip to content

JavadocHelper: Support Automatic-Module-Name for Javadoc URL lookup - #9528

Merged
matthiasblaesing merged 1 commit into
apache:masterfrom
NicolaIsotta:amn-javadoc
Sep 13, 2026
Merged

matthiasblaesing merged 1 commit into
apache:masterfrom
NicolaIsotta:amn-javadoc

Conversation

@NicolaIsotta

@NicolaIsotta NicolaIsotta commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

When resolving module-prefixed Javadoc URLs for libraries lacking a module-info.class, JavadocHelper now checks also the jar manifest for Automatic-Module-Name. This allows automatic modules without an explicit module-info.class to correctly resolve module-prefixed Javadoc paths.

When resolving module-prefixed Javadoc URLs for libraries lacking a `module-info.class`,
JavadocHelper now checks also the jar manifest for `Automatic-Module-Name`.
This allows automatic modules without an explicit `module-info.class` to correctly resolve module-prefixed Javadoc paths.
@NicolaIsotta

NicolaIsotta commented Jul 20, 2026

Copy link
Copy Markdown
Contributor Author

To test:

  • create a java maven project
  • add omnifaces 4.7.10 as a dependency
  • from the project tab, expand the "Dependecies" node
  • Right click the omnifaces node and click "Download Javadoc"
  • Open the node, open a package and right click a .class file
  • Choose "Show Javadoc" from the context menu

On NB30 a toast notification will appear, stating the no javadoc was found.
With the PR applied, the javadoc should open correctly

@mbien mbien added Java [ci] enable extra Java tests (java.completion, java.source.base, java.hints, refactoring.java, form) JavaDoc [ci] enable java/javadoc tests and build-javadoc target ci:dev-build [ci] produce a dev-build zip artifact (7 days expiration, see link on workflow summary page) labels Jul 21, 2026
@apache apache locked and limited conversation to collaborators Jul 21, 2026
@apache apache unlocked this conversation Jul 21, 2026
@mbien
mbien requested a review from neilcsmith-net August 29, 2026 03:48

@matthiasblaesing matthiasblaesing left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works and looks sane to me. Thank you!

@matthiasblaesing
matthiasblaesing merged commit ca84efd into apache:master Sep 13, 2026
34 checks passed
@mbien mbien added this to the NB32 milestone Sep 13, 2026
@NicolaIsotta
NicolaIsotta deleted the amn-javadoc branch September 13, 2026 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:dev-build [ci] produce a dev-build zip artifact (7 days expiration, see link on workflow summary page) Java [ci] enable extra Java tests (java.completion, java.source.base, java.hints, refactoring.java, form) JavaDoc [ci] enable java/javadoc tests and build-javadoc target

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants