-
-
Notifications
You must be signed in to change notification settings - Fork 346
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add subclass driver #16353
Add subclass driver #16353
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work! I really like the UI for adding new subclass! Left some comments inline, based on our discussion
src/SystemCommands-ClassCommands/SycCmAddSubclassCommand.class.st
Outdated
Show resolved
Hide resolved
Add tests Migrate the Syc user pick e5f0165 Add RBAddNewClassRefactoring and ReAddSubclassDriver Add tests Migrate the Syc user pick 6e9c5a4 Add StRefactoringAddClassPresenter Update #SycCmAddSubclassCommand pick a763b30 Update privateTransform so we don't reparent classes in add subclass. Add event hooks in add class presenter.
Update #SycCmAddSubclassCommand
Add event hooks in add class presenter.
Enhance Add subclass UI
…m the driver. Fix the layout so comment presenter is bigger. Add use comment template checkbox. Refactoring initialize presenters. Show superclass in title dialog. Fix updating tag items. Temporary fix of onCancel: block using CmCommandAborted or CmdCommandAborted, both raising a Debugger when signaled.
5422119
to
0344b88
Compare
…RBInsertNewClassRefactoring Remove unnecessary line in ReAddSubclassDriver>>configureRefactoring Remove unnecessary instance variable newClassPresenter in StRefactoringAddClassPresenter. Re-use 'textInput' from superclass. Update initialExtent setting. Remove unnecessary #example method.
src/SystemCommands-ClassCommands/SycCmAddSubclassCommand.class.st
Outdated
Show resolved
Hide resolved
Provide two instance creation helpers to provide a dialog with the superclass and its packages. Refactor ReAddSubclassDriver methods.
Two tests failing due to:
|
@hernanmd can you take the point of balsa into account so that we can integrate this issue. |
- the following classes have unused instance variables and should be cleaned: SycCmAddSubclassCommand - the following classes have uncategorized methods: an OrderedCollection(RBInteractionDriver) Re-use RBInsertNewClassRefactoring also for adding new refactoring
Done |
This PR implements a first version of "Add subclass" refactoring as a driver:
add_new_subclass_edit.mp4
Main features
blankCommentTemplate
in the class selected as superclass.Implementation notes
RBAddNewClassRefactoring
which differs from the Insert New Class refactoring in that does not reparent the subclasses to the new class.ReAddSubclassDriver
andReAddSubclassDriverTest
.Known issues
CmdCommandAborted signal
andCmCommandAborted signal
are failing right now and it should be investigated why (debugger keep opening which could mean the exception is not properly handled).