Bug 47918 - Wrong description in o.a.c.startup.mbeans-desriptors.xml for the HostConfig operations
Summary: Wrong description in o.a.c.startup.mbeans-desriptors.xml for the HostConfig o...
Status: RESOLVED FIXED
Alias: None
Product: Tomcat 6
Classification: Unclassified
Component: Catalina (show other bugs)
Version: 6.0.20
Hardware: All All
: P2 normal (vote)
Target Milestone: default
Assignee: Tomcat Developers Mailing List
URL:
Keywords: PatchAvailable
Depends on:
Blocks:
 
Reported: 2009-09-30 00:42 UTC by Uwe Günther
Modified: 2009-11-03 15:23 UTC (History)
1 user (show)


Attachments
Patch for Tomcat 7 trunk (1.33 KB, patch)
2009-09-30 00:42 UTC, Uwe Günther 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Uwe Günther 2009-09-30 00:42:57 UTC 
Created attachment 24324 [details]
Patch for Tomcat 7 trunk

I just came across this one yesterday while exploring how the whole deployment
process works.

The the following mbean operations for the mbean HostConfig in
o.a.c.startup.mbeans-desriptors.xml have all the same description "Add a web
application name to the serviced list":

addServiced, isServiced, removeServiced


This just looks ugly if somebody tries to discover the mbeans with for example
jconsole.

This bug is filed for Tomcate 7 but I have seen this in the lastes Tomcat 6
branch as well.

addServiced: Add a web application to the serviced list to show it gets
serviced by another component
isServiced: Is a web application serviced by another component
removeServiced: Remove a web application from the serviced list to show it
isn't serviced by another component


Patch is attached. I have done it with:

svn diff -x -u java\org\apache\catalina\startup\mbeans-descriptors.xml >
java.org.apache.catalina.startup.mbeans-descriptors.xml

under Windows.

-- 
Kind Regards
Uwe Günther
Comment 1 Mark Thomas 2009-11-02 16:22:17 UTC 
Fixed in trunk and proposed for 6.0.x

Thanks for the patch.
Comment 2 Mark Thomas 2009-11-03 15:23:58 UTC 
This has been fixed in 6.0.x and will be included in 6.0.21 onwards.