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

Rex Wang commented on GERONIMO-4306:
------------------------------------


I think Jarek's advice is good, but currently I just choose solution #1 to make 
this patch for 2.1.x. Because, as you know, Lin is working on the new server 
assembly portlets, I think we could improve this in 2.2.

Thanks Donald & Jarek

> Plugin list won't be updated after installing a new server plugin
> -----------------------------------------------------------------
>
>                 Key: GERONIMO-4306
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4306
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: console
>    Affects Versions: 2.1.3
>            Reporter: Rex Wang
>             Fix For: 2.1.4, 2.2
>
>         Attachments: GERONIMO-4306.patch
>
>
> Steps:
> 1. Login web console
> 2. Go to "Plugin -> Assemble Server" portlet, and click the button - 
> "Assemble a server", then get the current plugin list.
> 3. Go to "Plugin -> Install Plugins" portlet to install a new plugin
> 4. Go back to the "Assemble Server", and click the button, then the list is 
> not updated.
> Proposed fix:
> 1.
> In "AssemblyListHandler.java", we cached the plugin list in portlet session, 
> and that won't be updated after installing a new plugin by web console.
> If we add codes to refresh the session after installing a plugin in "Install 
> Plugins" portlet, it seems to solve this problem. But it does not work for 
> using the gsh to install new plugins.
> So, we have to let the AssemblyListHandler load the plugin list each time we 
> open the portlet. And good news is that I did not see any significant 
> performance falling.
> 2.
> And another method to solve this problem is that  "ask user to logout and 
> login back when they install some plugins if they want to see their plugins 
> list in the 'assemble Server' portlet", but is it acceptable?
> If the #1 is more reasonable, I can provide a patch. 
> Thanks
> Rex

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