Skip to content

Return useful data from /pluginManager/api/json by default #26351

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.

Sign up for GitHub

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

Open
himanshu748 wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Return useful data from /pluginManager/api/json by default #26351

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
9193501
[JENKINS-27993] Increase @Exported visibility on PluginWrapper proper…
himanshu748 Feb 22, 2026
ee34d75
Address PR feedback
himanshu748 Feb 24, 2026
a54f9eb
Merge upstream/master
himanshu748 Feb 24, 2026
ca4cdf2
Revert unrelated formatting changes in PluginManagerTest.java
himanshu748 Feb 24, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 7 additions & 6 deletions core/src/main/java/hudson/PluginWrapper.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -522,6 +522,7 @@ public PluginWrapper(PluginManager parent, File archive, Manifest manifest, URL
this.archive = archive;
}

@Exported(visibility = 2)
@Override
public String getDisplayName() {
String displayName = getLongName();
Expand Down Expand Up @@ -595,7 +596,7 @@ public List<Dependency> getOptionalDependencies() {
/**
* Returns the short name suitable for URL.
*/
@Exported
@Exported(visibility = 2)
public String getShortName() {
return shortName;
}
Expand Down Expand Up @@ -627,7 +628,7 @@ public String getShortName() {
* null if this information is unavailable.
* @since 1.283
*/
@Exported
@Exported(visibility = 2)
public String getUrl() {
// first look in update center metadata
List<UpdateSite.Plugin> siteMetadataList = getInfoFromAllSites();
Expand Down Expand Up @@ -683,7 +684,7 @@ public YesNoMaybe supportsDynamicLoad() {
/**
* Returns the version number of this plugin
*/
@Exported
@Exported(visibility = 2)
public String getVersion() {
return getVersionOf(manifest);
}
Expand Down Expand Up @@ -894,7 +895,7 @@ private Set<String> dependentsToCheck(PluginDisableStrategy strategy) {
/**
* Returns true if this plugin is enabled for this session.
*/
@Exported
@Exported(visibility = 2)
public boolean isActive() {
return active && !hasCycleDependency();
}
Expand All @@ -921,7 +922,7 @@ public boolean isBundled() {
* If true, the plugin is going to be activated next time
* Jenkins runs.
*/
@Exported
@Exported(visibility = 2)
public boolean isEnabled() {
return !disableFile.exists();
}
Expand Down Expand Up @@ -1080,7 +1081,7 @@ private List<UpdateSite.Plugin> getInfoFromAllSites() {
* This method is conservative in the sense that if the version number is incomprehensible,
* it always returns false.
*/
@Exported
@Exported(visibility = 2)
public boolean hasUpdate() {
return getUpdateInfo() != null;
}
Expand Down
Loading