Hi all
Last week I did these tasks:
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:
Cheers,
Evelyn
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