Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 362767 - [MVC] The RetrieveViewHelper and CommitHelper cannot be set to the controller
Summary: [MVC] The RetrieveViewHelper and CommitHelper cannot be set to the controller
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: EDT (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Justin Spadea CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-11-03 04:18 EDT by fahua jin CLA
Modified: 2017-02-23 14:18 EST (History)
2 users (show)

See Also:

Attachments
The sample project. (409.98 KB, multipart/x-zip)
2011-11-03 04:19 EDT, fahua jin CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description fahua jin CLA 2011-11-03 04:18:19 EDT 
Build Identifier: 0.7.0.v201111012101

1) Setup the attached project, and open CommitFunctionInvoked.egl with VE.
2) The source code attaches RetrieveViewHelper and CommitHelper functions to the controller, but they're not invoked. 

Reproducible: Always
Comment 1 fahua jin CLA 2011-11-03 04:19:20 EDT 
Created attachment 206385 [details]
The sample project.
Comment 2 Justin Spadea CLA 2011-11-03 16:31:10 EDT 
Fixed EGLClassTemplate.java. The MVC code needed to be generated before field initializers, so that initializers would override default helpers.
Comment 3 fahua jin CLA 2011-11-04 02:11:28 EDT 
(In reply to comment #2)
> Fixed EGLClassTemplate.java. The MVC code needed to be generated before field
> initializers, so that initializers would override default helpers.

Justin, did u release the code? The problem still existed in my build. 

Build #: 0.7.0.v201111032101
Comment 4 fahua jin CLA 2011-11-04 02:19:42 EDT 
Reopen the defect to track status.
Comment 5 Justin Spadea CLA 2011-11-04 13:19:11 EDT 
You're right, I forgot to release the change. The build server is having problems but the next build should contain the fix.
Comment 6 fahua jin CLA 2011-11-07 03:25:55 EST 
Verified in 0.7.0.v201111060901.