[
https://issues.apache.org/jira/browse/OFBIZ-10091?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16825840#comment-16825840 ]
Michael Brohl commented on OFBIZ-10091:
---------------------------------------
{quote}Are you still interested in working on that?
{quote}
Yes, I think this should be corrected. Might need some time to do it because of other activities though.
> Unnecessary throws clauses in renderer classes?
> -----------------------------------------------
>
> Key: OFBIZ-10091
> URL:
https://issues.apache.org/jira/browse/OFBIZ-10091> Project: OFBiz
> Issue Type: Improvement
> Components: framework
> Affects Versions: Trunk
> Reporter: Michael Brohl
> Priority: Minor
>
> During the refactoring work I found that there are several classes in org.apache.ofbiz.widget.renderer.macro which state in their method declarations that they throw TemplateException and/or IOException.
> In many cases, there is either no evidence of these Exceptions or they are catched inside the methods.
> One example is the MacroFormRenderer constructor which says it throws TemplateException but does not.
> The method declarations in the interfaces of org.apache.ofbiz.widget.renderer all say that IOException is thrown.
> It might be that I overlook something and I did not dig much deeper into the code so I first want to ask what the reasons are or if it would not be reasonable to make a difference in interfaces and implementations and only declare that a method throws a specific exception if it really does?
> Thanks for your help,
> regards,
> Michael
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)