[ 
https://issues.apache.org/jira/browse/GERONIMO-4318?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12634937#action_12634937
 ] 

David Jencks commented on GERONIMO-4318:
----------------------------------------

rev 699420 fixes these problems Lin noticed:
-- c-m-p InstallModulesMojo needs to actually track the installed plugins
-- plugin plugin PluginInstallerGBean needs to be configured with the tracking 
file location
-- bug in save method in PluginInstallerGBean

> 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
>
>
> 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