sean add an entry and assign to camille. We got confused with igor and it took us a while to find. I think that the implementation in Analyse was not necessary. (so probably having 2 and 3 is ok) even if I never saw the difference. Stef On 4/8/14 15:08, Sean P. DeNigris wrote:
GitHub wrote
13729 3 menu item to generate inst var accessors https://pharo.fogbugz.com/f/cases/13729 I left a comment in the issue, but maybe no one is checking there since it's closed...
From the original comment: There is currently 3 menu items to generate iv accessors: Analyse > Create inst var accessors Refactoring > Class Refactoring > Generate accessors Refactoring > Inst Var Refactoring > Accessors
Each one has it's own underlying logic. I keep the second that has a shortcut associated and remove the two others.
Please put back #3. It is a different use case. #2 is a batch operation adding accessors for all inst vars. #3 adds accessors for one inst var. When many inst vars are involved, it is tedious to deselect all the other accessors...
The main problem was not that there was multiple options, but that they were poorly organized (see http://forum.world.st/Nautilus-accessor-over-generation-td4679480.html)
----- Cheers, Sean -- View this message in context: http://forum.world.st/pharo-project-pharo-core-ea9df7-40138-tp4770903p477172... Sent from the Pharo Smalltalk Developers mailing list archive at Nabble.com.