Hi all Last week I did these tasks:
- Add preview to select variables when running the generate accessors command (Fix issue #5270 <https://github.com/pharo-project/pharo/issues/5270>). - Separate the link between views and refactorings (setUp and pushDown method refactoring). - Add generic previews and remove specific previews of PushUp and PushDown refactoring (https://github.com/pharo-project/pharo/pull/8553). - I started the modification to make refactorings only in some packages, add a searcher for classes and methods in packages (these have reference to the package internally), I still have to finish implementing this part so that it is updated correctly when there are changes, deletions and additions. Fix issues #8513 <https://github.com/pharo-project/pharo/issues/8513>, #6366 <https://github.com/pharo-project/pharo/issues/6366>. - Improve "Move method to class side" Refactoring, the new version catch broken references (method senders and direct access to instVar) and fix them ( https://github.com/pharo-project/pharo/compare/Pharo9.0...Lin777:NewMoveToClassSideRefactoring?expand=1), for instance: Before refactoring: *RBTransformationRuleTestData* >> rewriteUsing: searchReplacer rewriteRule := searchReplacer. self resetResult. After refactoring: *RBTransformationRuleTestData* >> rewriteUsing: searchReplacer ^ self class rewriteUsing: searchReplace. *RBTransformationRuleTestData* *class* >> rewriteUsing: searchReplacer | aRBTransformationRuleTestData | aRBTransformationRuleTestData := self new. aRBTransformationRuleTestData rewriteRule: searchReplacer. aRBTransformationRuleTestData resetResult'). ----------------------------------------------------------------------- Tasks for this week: - Improve extract setUp refactoring. - Finish to rename methods in specific packages (Fix issues #8513 <https://github.com/pharo-project/pharo/issues/8513>, #6366 <https://github.com/pharo-project/pharo/issues/6366>.) - Improve messages when exists a Warnings or Expections in refactorings. - Fix move to component refactoring issue ( #8499 <https://github.com/pharo-project/pharo/issues/8499>) - Fix extract method issue (#5855 <https://github.com/pharo-project/pharo/issues/5855>) - Check if it is better to change the behavior of the "Remove method" command, using a refactoring instead of a strategy Cheers, Evelyn