java.jmx

jmx/readable? makes incorrect call to .isReadable

Details

  • Type: Defect Defect
  • Status: Resolved Resolved
  • Priority: Minor Minor
  • Resolution: Completed
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None
  • Patch:
    Code and Test

Description

In the definition of jmx/readable?, copied below, .isReadable is called.

(defn readable?
"Is attribute readable?"
[n attr]
(.isReadable (mbean-info n)))

My guess is that the intended isReadable method is from the class MBeanAttributeInfo http://docs.oracle.com/javase/1.5.0/docs/api/javax/management/MBeanAttributeInfo.html#isReadable%28%29

However, mbean-info returns an instance of class MBeanInfo. Also note that the attr argument of readable? isn't even used.

Activity

Andy Fingerhut made changes -
Field Original Value New Value
Description In the definition of jmx/readable?, copied below, .isReadable is called.

(defn readable?
  "Is attribute readable?"
  [n attr]
  (.isReadable (mbean-info n)))

My guess is that the method is from the class MBeanAttributeInfo http://docs.oracle.com/javase/1.5.0/docs/api/javax/management/MBeanAttributeInfo.html#isReadable%28%29

However, mbean-info returns an instance of class MBeanInfo. Also note that the attr argument of readable? isn't even used.
In the definition of jmx/readable?, copied below, .isReadable is called.

(defn readable?
  "Is attribute readable?"
  [n attr]
  (.isReadable (mbean-info n)))

My guess is that the intended isReadable method is from the class MBeanAttributeInfo http://docs.oracle.com/javase/1.5.0/docs/api/javax/management/MBeanAttributeInfo.html#isReadable%28%29

However, mbean-info returns an instance of class MBeanInfo. Also note that the attr argument of readable? isn't even used.
Nick Bailey made changes -
Patch Accepted [ 10012 ]
Resolution Completed [ 1 ]
Assignee Nick Bailey [ nickmbailey ]
Status Open [ 1 ] Resolved [ 5 ]
Alex Miller made changes -
Patch Accepted [ 10012 ] Code and Test [ 10002 ]

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: