[ 
http://issues.apache.org/jira/browse/GERONIMO-2471?page=comments#action_12442446
 ] 
            
Anita Kulshreshtha commented on GERONIMO-2471:
----------------------------------------------

applied the patch with following minor modifications in rev 464362. Thanks 
Chris!
. Reordered the columns to Name, Value, ...., and Getters. The names and values 
are now visible without scrolling.
. Dropped Writable and Readable column. AFAIK, a getter implies a readable and 
a setter implies a writable attribute.
. Renamed Setter column to Writable.
I have left the Getters column because even though we follow the convention 
getX/isX for an attribute x, the kernel allows getx to be a getter. Please 
review. 
   The deploymentDescriptor looks good! I have few more items on my wish list :)
. Is it possible to color code the attribute names as suggested in the code?
. Is it possible to go to login page after the login expires? Currently 
clicking on '* MBeans' gives "error: invalid reply from the server". Clicking 
on 'help' does go to login page. 
   Most of this in nit picking.. I am looking forward to using it instead of 
jconsole!

> JMX Portlet doesn't display all the attributes of a web module
> --------------------------------------------------------------
>
>                 Key: GERONIMO-2471
>                 URL: http://issues.apache.org/jira/browse/GERONIMO-2471
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: console
>    Affects Versions: 1.2
>            Reporter: Christopher M. Cardona
>         Assigned To: Anita Kulshreshtha
>             Fix For: 1.2
>
>         Attachments: GERONIMO-2471_2472-trunk.patch
>
>
> Viewing a web module doesn't display all its attributes.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to