[improvement] another rename to better reflect AST visiion.

pharo-project · Apr 18, 2020 · f8c9d73 · f8c9d73
1 parent 3f0061d
commit f8c9d73
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 13 deletions.
diff --git a/...ls-Core/SycRenameTempCommand.extension.st → ...re/SycRenameArgOrTempCommand.extension.st b/...ls-Core/SycRenameTempCommand.extension.st → ...re/SycRenameArgOrTempCommand.extension.st
@@ -1,14 +1,14 @@
-Extension { #name : #SycRenameTempCommand }
+Extension { #name : #SycRenameArgOrTempCommand }
 
 { #category : #'*Calypso-SystemTools-Core' }
-SycRenameTempCommand classSide >> sourceCodeMenuActivation [
+SycRenameArgOrTempCommand class >> sourceCodeMenuActivation [
 	<classAnnotation>
 
 	^SycSourceCodeMenuActivation byRootGroupItemOrder: 1.2 for: ClyMethodSourceCodeContext
 ]
 
 { #category : #'*Calypso-SystemTools-Core' }
-SycRenameTempCommand classSide >> sourceCodeShortcutActivation [
+SycRenameArgOrTempCommand class >> sourceCodeShortcutActivation [
 	<classAnnotation>
 
 	^CmdShortcutActivation renamingFor: ClyMethodSourceCodeContext

diff --git a/src/GeneralRules/ReTemporaryVariableCapitalizationRule.class.st b/src/GeneralRules/ReTemporaryVariableCapitalizationRule.class.st
@@ -42,7 +42,7 @@ ReTemporaryVariableCapitalizationRule >> critiqueFor: aNode [
 	crit	
 		tinyHint: aNode name;
 		refactoring: (
-			RBRenameTemporaryRefactoring
+			RBRenameArgumentOrTemporaryRefactoring
 				renameTemporaryFrom: aNode sourceInterval
 				to: aNode name uncapitalized
 				in: aNode methodNode methodClass

diff --git a/src/NautilusRefactoring/NautilusRefactoring.class.st b/src/NautilusRefactoring/NautilusRefactoring.class.st
@@ -999,7 +999,7 @@ NautilusRefactoring >> privateRenameMethodFor: aMethod withNewName: newMethodNam
 { #category : #'private-source' }
 NautilusRefactoring >> privateRenameTemporaryNamed: oldname Between: anInterval from: aMethod [
 
-	^ RBRenameTemporaryRefactoring
+	^ RBRenameArgumentOrTemporaryRefactoring
 		model: environment
 		renameTemporaryFrom: anInterval
 		to: (self request: self newVariableRequestText initialAnswer: oldname)

diff --git a/...ore/RBRenameTemporaryRefactoring.class.st → ...meArgumentOrTemporaryRefactoring.class.st b/...ore/RBRenameTemporaryRefactoring.class.st → ...meArgumentOrTemporaryRefactoring.class.st
@@ -7,7 +7,7 @@ The variable declaration an all references in this method are renamed.
 My precondition verify that the new name is a valid variable name and not an existing instance or a class variable name
 "
 Class {
-	#name : #RBRenameTemporaryRefactoring,
+	#name : #RBRenameArgumentOrTemporaryRefactoring,
 	#superclass : #RBMethodRefactoring,
 	#instVars : [
 		'selector',
@@ -20,7 +20,7 @@ Class {
 }
 
 { #category : #'instance creation' }
-RBRenameTemporaryRefactoring class >> model: aRBSmalltalk renameTemporaryFrom: anInterval to: newName in: aClass selector: aSelector [ 
+RBRenameArgumentOrTemporaryRefactoring class >> model: aRBSmalltalk renameTemporaryFrom: anInterval to: newName in: aClass selector: aSelector [ 
 	^(self new)
 		model: aRBSmalltalk;
 		class: aClass
@@ -31,7 +31,7 @@ RBRenameTemporaryRefactoring class >> model: aRBSmalltalk renameTemporaryFrom: a
 ]
 
 { #category : #'instance creation' }
-RBRenameTemporaryRefactoring class >> renameTemporaryFrom: anInterval to: newName in: aClass selector: aSelector [ 
+RBRenameArgumentOrTemporaryRefactoring class >> renameTemporaryFrom: anInterval to: newName in: aClass selector: aSelector [ 
 	^self new
 		class: aClass
 		selector: aSelector
@@ -40,15 +40,15 @@ RBRenameTemporaryRefactoring class >> renameTemporaryFrom: anInterval to: newNam
 ]
 
 { #category : #initialization }
-RBRenameTemporaryRefactoring >> class: aClass selector: aSelector interval: anInterval newName: aString [ 
+RBRenameArgumentOrTemporaryRefactoring >> class: aClass selector: aSelector interval: anInterval newName: aString [ 
 	class := self classObjectFor: aClass.
 	selector := aSelector.
 	interval := anInterval.
 	newName := aString
 ]
 
 { #category : #preconditions }
-RBRenameTemporaryRefactoring >> preconditions [
+RBRenameArgumentOrTemporaryRefactoring >> preconditions [
 	^ (RBCondition
 		withBlock: [ | methodSource |
 			interval first > interval last
@@ -62,7 +62,7 @@ RBRenameTemporaryRefactoring >> preconditions [
 ]
 
 { #category : #tranforming }
-RBRenameTemporaryRefactoring >> renameNode: aParseTree [
+RBRenameArgumentOrTemporaryRefactoring >> renameNode: aParseTree [
 	(aParseTree whoDefines: newName)
 		ifNotNil: [ self refactoringError: newName asString , ' is already defined' ].
 	(aParseTree allDefinedVariables includes: newName)
@@ -71,7 +71,7 @@ RBRenameTemporaryRefactoring >> renameNode: aParseTree [
 ]
 
 { #category : #printing }
-RBRenameTemporaryRefactoring >> storeOn: aStream [ 
+RBRenameArgumentOrTemporaryRefactoring >> storeOn: aStream [ 
 	aStream nextPut: $(.
 	self class storeOn: aStream.
 	aStream nextPutAll: ' renameTemporaryFrom: '.
@@ -88,7 +88,7 @@ RBRenameTemporaryRefactoring >> storeOn: aStream [
 ]
 
 { #category : #tranforming }
-RBRenameTemporaryRefactoring >> transform [
+RBRenameArgumentOrTemporaryRefactoring >> transform [
 	| definingNode variableNode |
 	parseTree := class parseTreeFor: selector.
 	variableNode := self whichVariableNode: parseTree inInterval: interval name: oldName.