[jira] Issue Comment Edited: (GERONIMO-4353) An uniform way to show error messages on the console

["Ivan (JIRA)" <ji...@apache.org>]

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

xuhaihong edited comment on GERONIMO-4353 at 10/28/08 8:08 AM:
----------------------------------------------------------

    I try to provide a solution to using a uniform way to handle and display error messages in the portlets. 

1. Some helper methods are added in the class BasePortlet, so any portlets that extend it could use them to add error messages.
    eg. getWebMessages(request, response).addErrorMessage(message, detailedMessage);
2. While rendering the portlets, those messages are generated as html texts in a hidden block on the html pages.
3. A dojo extension GMessages.js will load the messages and show them  on the header of  each portlet. The reason of using Javascript to display messages is that I wish to show messages on the header of  the portlet page, and until finishing rendering, we could not know how many messages are added by the portlet.
4. For how to display the messages, I refer to the way used by the deployment portlet, please check the snapshot attached.
5. I update the class DeploymentPortlet  for a demo.

Before trying this patch, dojo plugin should be installed, and make sure the reference to dojo.js is /dojo/dojo/dojo.js
Plase help to review it, and if it is feasible, I wish to continue to work on it, and begin to do some changes to existing portlets.
Thanks !


      was (Author: xuhaihong):
        I try to provide a solution to using a uniform way to handle and display error messages in the portlets. 

1. Some helper methods are added in the class BasePortlet, so any portlets that extend it could use them to add error messages.
    eg. getWebMessages(request, response).addErrorMessage(message, detailedMessage);
2. While rendering the portlets, those messages are generated as html texts in a hidden block on the html pages.
3. A dojo extension GMessages.js will load the messages and show them  on the header of the page for each portlet. The reason why using js to display messages is for we wish to show messages on the header of  the portlet page, and until finishing rendering, we could not which messages are needed to display.
4. For how to display the messages, I refer to way used by the deployment portlet.
5. I update the class DeploymentPortlet  for a demo.

Before trying this patch, dojo plugin should be installed, and make sure the reference to dojo.js is /dojo/dojo/dojo.js
Plase help to review it, and if it is feasible, I wish to continue to work on it, and begin to do some changes to existing portlets.
Thanks !

  
> An uniform way to show error messages on the console
> ----------------------------------------------------
>
>                 Key: GERONIMO-4353
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4353
>             Project: Geronimo
>          Issue Type: Improvement
>      Security Level: public(Regular issues) 
>          Components: console
>    Affects Versions: 2.1.3
>            Reporter: Ivan
>         Attachments: Geronimo-4353.20081028.patch, snapshot.JPG
>
>
> Currently, different portlets uses different way to show those error messages to the end user. We need a uniform way to do it.  The simliar issue is opened in GERONIMO-2621, but it was closed due to overwork on it in the past.
> I am trying to provide a solution, and will provide a demo in the next few days. Thanks !

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