[ 
https://issues.apache.org/jira/browse/OFBIZ-3774?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13165111#comment-13165111
 ] 

Adrian Crum commented on OFBIZ-3774:
------------------------------------

Yes, we need a separate Jira issue for the package rename. My hope is that we 
can stay in scope and provide a clear change trail for others to follow.

With that in mind, I would like to see the redesign occur with small, easily 
defined steps:

1. Add the visitor interfaces and accept methods.
2. Create abstract classes for the interfaces.
3. Extend the abstract classes to rendering classes and artifact-info-gatherer 
classes.
4. Move existing widget model code to the new classes.
5. Merge all changes to the trunk.
6. Close this issue and start on the next one.

The package rename would be a good candidate for the next redesign effort, and 
we will merge those changes to the trunk when they are completed.

                
> 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
>            Assignee: Adrian Crum
>            Priority: Minor
>         Attachments: OFBIZ-3774.patch, OFBIZ-3774.patch, OFBIZ-3774.patch, 
> OFBIZ-3774_ScreenWidgetVisitor.patch, OFBIZ-3774_ScreenWidgetVisitor.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.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to