[ 
https://issues.apache.org/jira/browse/GERONIMO-4318?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Lin Sun updated GERONIMO-4318:
------------------------------

    Attachment: G4318.patch

Sorry it was not formatted nicely in previous comment, so attaching a file here.

> All the plugins are marked as installable on the install plugins portlet
> ------------------------------------------------------------------------
>
>                 Key: GERONIMO-4318
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4318
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: console
>    Affects Versions: 2.2
>            Reporter: Lin Sun
>            Assignee: Lin Sun
>             Fix For: 2.2
>
>         Attachments: G4318.patch
>
>
> All the plugins are marked as installable on the install plugins portlet.   
> We check if a plugin is installable by using pluginInstaller.validatePlugin.  
>  If an exception is thrown, then we set the installable to false.   The throw 
> of MissingDependencyException in validatePlugin method is commented out 
> during rev 696105.
> A proposed fix is to throw ConfigurationAlreadyExistsException when 
> validatePlugin fails because of the configuration is already installed.   
> This seems more reasonable and will also get rid of the confusion message of 
> "Missing Dependency: XXX" when a user attempts to install a plugin that has 
> already been installed using the "deploy install-plugin" command.

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

Reply via email to