Make configurable the class created by the PortletRequestHelperFactory
----------------------------------------------------------------------

                 Key: DISPL-428
                 URL: http://jira.codehaus.org/browse/DISPL-428
             Project: DisplayTag
          Issue Type: Improvement
          Components: Configuration
    Affects Versions: 1.1
            Reporter: Raúl Herranz


To make configurable the class created by the PortletRequestHelperFactory, 
there should be a new config line in the displaytag.properties file with the 
complete name (package name + class name) of the class. Then, the 
PortletRequestHelperFactory's getRequestHelperInstance method should be changed 
to something similar to this:

        String className = //GET THE CLASSNAME FROM THE displaytag.properties
        if (className!=null && !"".equals(className)) {
                try {
                        Class requestHelperClass = Class.forName(className);
                        Constructor requestHelperConstructor = 
requestHelperClass.getConstructor(new 
Class[]{javax.servlet.jsp.PageContext.class});
                        retorno = 
(RequestHelper)requestHelperConstructor.newInstance(new Object[]{pageContext});
                } catch (Exception e) {
                        //do whatever is needed (by example, the exception 
should be logged)
                }
        } else {
                //do whatever is needed (by example, className not finded 
should be logged)
        }



-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
displaytag-devel mailing list
displaytag-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/displaytag-devel

Reply via email to