This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.
Controller: $this->view->something View: $this->something NetBeans currently offers just $something which is incorrect. Filipe, please confirm. Radku, we need to improve editor part to support this scenario. This is definitely a candidate for NB 6.9.1.
Project's part fixed, reassigning to the editor. Thanks. http://hg.netbeans.org/web-main/rev/3dc91d8afaf8
(In reply to comment #0) > Controller: > $this->view->something > > View: > $this->something > > NetBeans currently offers just $something which is incorrect. > > Filipe, please confirm. Radku, we need to improve editor part to support this > scenario. > > This is definitely a candidate for NB 6.9.1. you guys probably already strated working on this .. however, confirmed. the problem occurred because of difference between symfony and zend call in view. You use $something in symfony but $this->something in zend. I agree, that this is 6.9.1 candidate
This is proposed patch for 6.9.1: http://hg.netbeans.org/web-main/rev/1b115734c4c2 - in CC for views offered $this - after this completes fields (taken from $this->view->*) as $this->* - CC after $this->*->| doesn't complete - no goto declaration and so on Relatively safe, might be transplanted, but incomplete (see above) Better impl. deserves more changes and might cause poor quality. I will start to work on it, but not sure if it will be safe enough to put it into 6.9.1
Integrated into 'main-golden', will be available in build *201006030001* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress) Changeset: http://hg.netbeans.org/main/rev/3dc91d8afaf8 User: Tomas Mysik <tmysik@netbeans.org> Log: Project's part of #186858: Incorrect CC in Views Editor's part is to be done now.
Filipe, please verify. Thanks.
Reopening beause we must first agreed whether it is fixed or not (the editor's part).
Added the 69patch1 status whiteboard keyword. I've already ported changesets into releasee691 repository and the status whiteboard keyword helps me keep this bug on "PORTED UNVERIFIED" list, please do not remove 69patch1 status whiteboard keyword without backing out the changesets first. Related changesets are: release691 #4f436738ee7a (Tomas' project's part) release691 #b8924fc2892d (Radek's part [editor?])
In case we would have agreed on rollback from release691 repository, there exist also one more related changeset in release691 #7d2f9a18e0b6 which is a fix for specification version and module dependency issue #187769 "BUILD FAILED". The rollback TASK is covered by issue 187962. If we would have agreed on not rolling back from release691 repository, then please remove 6.9.1_CANDIDATE keyword from issue 187962 and mark it INVALID. Thanks, -R
Keep in 6.9.1, no rollback! Filip is going to test for regressions again(though already tested by myself and users), close this issue as fixed(and create new one for net release) This fix is real improvement(although not ideal and 100%), any better fix can't be delivered in guaranteed quality for 6.9.1 and should come into next release.
(In reply to comment #9) > Keep in 6.9.1, no rollback! Filip is going to test for regressions again(though > already tested by myself and users), close this issue as fixed(and create new > one for net release) > > This fix is real improvement(although not ideal and 100%), any better fix can't > be delivered in guaranteed quality for 6.9.1 and should come into next release. verified in Product Version: NetBeans IDE Dev (Build 100624-37b8a71917c5) Java: 1.6.0_16; Java HotSpot(TM) 64-Bit Server VM 14.2-b01 two additional bugs were filed related to this functionality BUG 188055 BUG 188056 even though two additional bugs have been filed this one can be transplanted into 6.9.1
(In reply to comment #9) > even though two additional bugs have been filed this one can be transplanted > into 6.9.1 well, not transplanted but kept as it is I meant
Re-seting the 6.9.1_CANDIDATE keyword. Updated the status whiteboard.
Since Filip has verified in trunk I checked that fixes are in 6.9.1 clone and I am marking as verified in 6.9.1.