[
https://issues.apache.org/jira/browse/OFBIZ-3774?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12868503#action_12868503
]
Adrian Crum commented on OFBIZ-3774:
------------------------------------
One thing I would like to clarify: the patch only includes screen sub-widgets,
but the goal would be apply the visitor pattern to all screen widget model
classes - like Marc mentioned.
An abstract class for renderers that navigates the model structure probably
won't work - because each rendering format will require its own sequence of
events, or its own mode of navigation. You can't determine in advance how a
particular rendering format will need to navigate the structure. The existing
widget models try to do this form of prediction and it has caused problems in
the past.
I'm currently working on porting the macro renderer over to the visitor
interface to see what kind of problems we might encounter.
> Add The Visitor Pattern To Screen Widget Model Classes
> ------------------------------------------------------
>
> Key: OFBIZ-3774
> URL: https://issues.apache.org/jira/browse/OFBIZ-3774
> Project: OFBiz
> Issue Type: Improvement
> Components: framework
> Reporter: Adrian Crum
> Priority: Minor
> Attachments: OFBIZ-3774.patch
>
>
> From time to time there has been interest expressed in introducing the
> visitor pattern to the screen widget model classes. This issue is intended to
> provide a forum on the subject and a means to vote on it. Details are in the
> comments.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
