You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Philip May (JIRA)" <ji...@apache.org> on 2008/02/20 10:38:43 UTC

[jira] Created: (MODELER-29) Improve the reusability and separation of concerns

Improve the reusability and separation of concerns
--------------------------------------------------

                 Key: MODELER-29
                 URL: https://issues.apache.org/jira/browse/MODELER-29
             Project: Commons Modeler
          Issue Type: Improvement
            Reporter: Philip May


The level of reusability and the separation of concerns of some classes is insufficient. 
I would like to reuse the MbeansDescriptorsDOMSource class to parse some xml and to build the MBean stuff.
To do this I have to call the MbeansDescriptorsDOMSource#loadDescriptors( Registry registry, String location, String type, Object source) method.
To call this method I must have registry, location and type. These references are not even needed in the class and could be removed.

MbeansDescriptorsDOMSource and the other ModelerSource classes do need some refactoring to make them reusable.

When I am looking at the code of commons.modeler somehow I get the impression that this commons component is just made for tomcat and not
for the usage in other projects.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (MODELER-29) Improve the reusability and separation of concerns

Posted by "Philip May (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/MODELER-29?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Philip May updated MODELER-29:
------------------------------

    Description: 
The level of reusability and the separation of concerns of some classes is insufficient. 
I would like to reuse the MbeansDescriptorsDOMSource class to parse some xml and to build the MBean stuff.
To do this I have to call the MbeansDescriptorsDOMSource#loadDescriptors( Registry registry, String location, String type, Object source) method.
To call this method I must have registry, location and type. These references are not even needed in the class and could be removed.

MbeansDescriptorsDOMSource and the other ModelerSource classes do need some refactoring to make them reusable.
Also they need more documentation (Javadoc).

When I am looking at the code of commons.modeler somehow I get the impression that this commons component is just made for tomcat and not
for the usage in other projects. Registry id the entry (main) class of commons.modeler. But it could and should be possible to reuse the other classes
like the parser too.

  was:
The level of reusability and the separation of concerns of some classes is insufficient. 
I would like to reuse the MbeansDescriptorsDOMSource class to parse some xml and to build the MBean stuff.
To do this I have to call the MbeansDescriptorsDOMSource#loadDescriptors( Registry registry, String location, String type, Object source) method.
To call this method I must have registry, location and type. These references are not even needed in the class and could be removed.

MbeansDescriptorsDOMSource and the other ModelerSource classes do need some refactoring to make them reusable.

When I am looking at the code of commons.modeler somehow I get the impression that this commons component is just made for tomcat and not
for the usage in other projects. Registry id the entry (main) class of commons.modeler. But it could and should be possible to reuse the other classes
like the parser too.


> Improve the reusability and separation of concerns
> --------------------------------------------------
>
>                 Key: MODELER-29
>                 URL: https://issues.apache.org/jira/browse/MODELER-29
>             Project: Commons Modeler
>          Issue Type: Improvement
>            Reporter: Philip May
>
> The level of reusability and the separation of concerns of some classes is insufficient. 
> I would like to reuse the MbeansDescriptorsDOMSource class to parse some xml and to build the MBean stuff.
> To do this I have to call the MbeansDescriptorsDOMSource#loadDescriptors( Registry registry, String location, String type, Object source) method.
> To call this method I must have registry, location and type. These references are not even needed in the class and could be removed.
> MbeansDescriptorsDOMSource and the other ModelerSource classes do need some refactoring to make them reusable.
> Also they need more documentation (Javadoc).
> When I am looking at the code of commons.modeler somehow I get the impression that this commons component is just made for tomcat and not
> for the usage in other projects. Registry id the entry (main) class of commons.modeler. But it could and should be possible to reuse the other classes
> like the parser too.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (MODELER-29) Improve the reusability and separation of concerns

Posted by "Philip May (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/MODELER-29?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Philip May updated MODELER-29:
------------------------------

    Description: 
The level of reusability and the separation of concerns of some classes is insufficient. 
I would like to reuse the MbeansDescriptorsDOMSource class to parse some xml and to build the MBean stuff.
To do this I have to call the MbeansDescriptorsDOMSource#loadDescriptors( Registry registry, String location, String type, Object source) method.
To call this method I must have registry, location and type. These references are not even needed in the class and could be removed.

MbeansDescriptorsDOMSource and the other ModelerSource classes do need some refactoring to make them reusable.

When I am looking at the code of commons.modeler somehow I get the impression that this commons component is just made for tomcat and not
for the usage in other projects. Registry id the entry (main) class of commons.modeler. But it could and should be possible to reuse the other classes
like the parser too.

  was:
The level of reusability and the separation of concerns of some classes is insufficient. 
I would like to reuse the MbeansDescriptorsDOMSource class to parse some xml and to build the MBean stuff.
To do this I have to call the MbeansDescriptorsDOMSource#loadDescriptors( Registry registry, String location, String type, Object source) method.
To call this method I must have registry, location and type. These references are not even needed in the class and could be removed.

MbeansDescriptorsDOMSource and the other ModelerSource classes do need some refactoring to make them reusable.

When I am looking at the code of commons.modeler somehow I get the impression that this commons component is just made for tomcat and not
for the usage in other projects.


> Improve the reusability and separation of concerns
> --------------------------------------------------
>
>                 Key: MODELER-29
>                 URL: https://issues.apache.org/jira/browse/MODELER-29
>             Project: Commons Modeler
>          Issue Type: Improvement
>            Reporter: Philip May
>
> The level of reusability and the separation of concerns of some classes is insufficient. 
> I would like to reuse the MbeansDescriptorsDOMSource class to parse some xml and to build the MBean stuff.
> To do this I have to call the MbeansDescriptorsDOMSource#loadDescriptors( Registry registry, String location, String type, Object source) method.
> To call this method I must have registry, location and type. These references are not even needed in the class and could be removed.
> MbeansDescriptorsDOMSource and the other ModelerSource classes do need some refactoring to make them reusable.
> When I am looking at the code of commons.modeler somehow I get the impression that this commons component is just made for tomcat and not
> for the usage in other projects. Registry id the entry (main) class of commons.modeler. But it could and should be possible to reuse the other classes
> like the parser too.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.