StéphaneDucasse

KMDescriptionPresenter open bring an empty list of bindings

2

It is strange We should also promote it as a tool accessible in a menu too.

Review the welcome on the discord channel because it loos outdated.

[RB] safeMethodNameFor: aClass basedOn: aString could be covered by some tests.

1

In RBRefactoring.

[RB] Replace senders does not work from the code pane

1

We could not find a way to make ReplaceMessageSendTransformation (menu item (T) Replace senders to work because the context is different from the one we get when we click on...

And we should replace them with the `StRefactoringPreviewPresenter`

The first you use "doit" in the Playground it takes around 5 seconds

13

Seb showed me this today in P11

I verified and I found a bug - I have a class which uses a trait - I apply insert superclass - and my class does not have the trait...

[RB] RBRemoveAllSenders improvements: name is bad, semantics is fuzzy

1

- first it should renamed RBRemoveAllSelfSendTransformation because we can have no warrantee. - second the API is really strange: why passing a range and not just a selector (note that...

[RB] checkMethodName: aString in: aClass has an extra unused argument

1

``` checkMethodName: aString in: aClass "Return whether the argument aName is can represent a selector" "You probably look for checkMethodName: since the second argument is ignored" ^aString isString and: [...

In the following getNewSelector is actually doing a side effect so if we compute superclass preconditions and local one, they fail of course. ``` localPreconditions | imps | self getNewSelector....