Skip to content

Commit

Permalink
Merge pull request #16659 from jecisc/deprecated-aliases
Browse files Browse the repository at this point in the history
Remove deprecated aliases of P12
  • Loading branch information
tesonep authored Jun 3, 2024
2 parents 92c5115 + 3596de0 commit 0ac6afe
Show file tree
Hide file tree
Showing 74 changed files with 36 additions and 424 deletions.
3 changes: 0 additions & 3 deletions bootstrap/scripts/03-metacello-bootstrap/01-loadMetacello.st
Original file line number Diff line number Diff line change
Expand Up @@ -34,9 +34,6 @@ MCFileTreeFileSystemUtils initialize.
ConfigurationOf initialize.
MetacelloPlatform initialize.

"Init deprecated aliases for Pharo 12. To remove in Pharo 13"
{ MetacelloPlatform . MetacelloProject . MetacelloProjectSpec . MetacelloTarget . MetacelloVersion . MetacelloVersionSpec } do: [ :class | class initialize ].

STONWriter initialize.

MCGitHubRepository initialize.
Expand Down
2 changes: 1 addition & 1 deletion src/BaselineOfPharo/BaselineOfPharo.class.st
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ BaselineOfPharo class >> roassal [
newName: 'Roassal'
owner: 'pharo-graphics'
project: 'Roassal'
version: 'v1.06d'
version: 'v1.06e'
]

{ #category : 'repository urls' }
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ ClyPackageRichTextCommentEditorToolMorph class >> browserTabActivation [
"This declaration specifies that in any browser when packages are selected, a comment editor will be available in a tab."

<classAnnotation>
^ClyTabActivationStrategyAnnotation for: RPackage asCalypsoItemContext
^ClyTabActivationStrategyAnnotation for: Package asCalypsoItemContext
]

{ #category : 'activation' }
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,12 +12,6 @@ Class {
#tag : 'Tabs'
}

{ #category : 'class initialization' }
ClyTabActivationStrategyAnnotation class >> initialize [

self deprecatedAliases: { #ClyTabActivationStrategy }
]

{ #category : 'operations' }
ClyTabActivationStrategyAnnotation >> createToolFor: aBrowser inContext: aBrowserContext [

Expand Down
5 changes: 0 additions & 5 deletions src/Calypso-SystemTools-Core/ClyProtocolEditorMorph.class.st
Original file line number Diff line number Diff line change
Expand Up @@ -30,11 +30,6 @@ Class {
#tag : 'Editors-Methods'
}

{ #category : 'class initialization' }
ClyProtocolEditorMorph class >> initialize [
self deprecatedAliases: { #ClyProtocolAndPackageEditorMorph }
]

{ #category : 'building' }
ClyProtocolEditorMorph >> build [
self buildResetButton.
Expand Down
36 changes: 18 additions & 18 deletions src/Kernel-CodeModel-Tests/PackageAndClassesTest.class.st
Original file line number Diff line number Diff line change
Expand Up @@ -426,15 +426,15 @@ PackageAndClassesTest >> testRenameClass [

| xPackage class |
xPackage := self ensureXPackage.
class := self newClassNamed: #RPackageOldStubClass in: xPackage.
class := self newClassNamed: #PackageOldStubClass in: xPackage.

class rename: #RPackageNewStubClass.
class rename: #PackageNewStubClass.

self assert: (self organizer packageOf: class) equals: xPackage.
self assert: (self organizer packageOfClassNamed: #RPackageNewStubClass) equals: xPackage.
self assert: (self organizer packageOfClassNamed: #RPackageOldStubClass) equals: self organizer undefinedPackage.
self assert: (xPackage includesClassNamed: 'RPackageNewStubClass').
self deny: (xPackage includesClassNamed: 'RPackageOldStubClass')
self assert: (self organizer packageOfClassNamed: #PackageNewStubClass) equals: xPackage.
self assert: (self organizer packageOfClassNamed: #PackageOldStubClass) equals: self organizer undefinedPackage.
self assert: (xPackage includesClassNamed: 'PackageNewStubClass').
self deny: (xPackage includesClassNamed: 'PackageOldStubClass')
]

{ #category : 'tests - operations on classes' }
Expand All @@ -443,10 +443,10 @@ PackageAndClassesTest >> testRenameClassUpdateClassDefinedSelectorsInTheParentPa

| xPackage class |
xPackage := self ensureXPackage.
class := self newClassNamed: 'RPackageOldStubClass' asSymbol in: xPackage.
self createMethodNamed: 'stubMethod' inClass: class inProtocol: 'classic category'.
class := self newClassNamed: 'PackageOldStubClass' asSymbol in: xPackage.
self createMethodNamed: 'stubMethod' inClass: class inProtocol: 'classic protocol'.

class rename: 'RPackageNewStubClass'.
class rename: 'PackageNewStubClass'.

self assert: (class >> #stubMethod) package equals: xPackage.
self deny: (class >> #stubMethod) isExtension
Expand All @@ -459,10 +459,10 @@ PackageAndClassesTest >> testRenameClassUpdateClassExtensionSelectorsInTheExtend
| xPackage yPackage class |
xPackage := self ensureXPackage.
yPackage := self ensureYPackage.
class := self newClassNamed: #RPackageOldStubClass in: xPackage.
class := self newClassNamed: #PackageOldStubClass in: xPackage.
self createMethodNamed: 'stubMethod' inClass: class asExtensionOf: yPackage.

class rename: 'RPackageNewStubClass'.
class rename: 'PackageNewStubClass'.

self assert: (yPackage includesExtensionSelector: #stubMethod ofClass: class)
]
Expand All @@ -473,10 +473,10 @@ PackageAndClassesTest >> testRenameClassUpdateMetaclassDefinedSelectorsInThePare

| xPackage class |
xPackage := self ensureXPackage.
class := self newClassNamed: #RPackageOldStubClass in: xPackage.
self createMethodNamed: 'stubMethod' inClass: class class inProtocol: 'classic category'.
class := self newClassNamed: #PackageOldStubClass in: xPackage.
self createMethodNamed: 'stubMethod' inClass: class class inProtocol: 'classic protocol'.

class rename: 'RPackageNewStubClass'.
class rename: 'PackageNewStubClass'.

self assert: (class class >> #stubMethod) package equals: xPackage.
self deny: (class class >> #stubMethod) isExtension
Expand All @@ -489,10 +489,10 @@ PackageAndClassesTest >> testRenameClassUpdateMetaclassExtensionSelectorsInTheEx
| xPackage yPackage class |
xPackage := self ensureXPackage.
yPackage := self ensureYPackage.
class := self newClassNamed: #RPackageOldStubClass in: xPackage.
class := self newClassNamed: #PackageOldStubClass in: xPackage.
self createMethodNamed: 'stubMethod' inClass: class class asExtensionOf: yPackage.

class rename: 'RPackageNewStubClass'.
class rename: 'PackageNewStubClass'.

self assert: (yPackage includesExtensionSelector: #stubMethod ofClass: class class)
]
Expand All @@ -504,10 +504,10 @@ PackageAndClassesTest >> testRenameClassUpdateOrganizerClassExtendingPackagesMap
| xPackage yPackage class |
xPackage := self ensureXPackage.
yPackage := self ensureYPackage.
class := self newClassNamed: #RPackageOldStubClass in: xPackage.
class := self newClassNamed: #PackageOldStubClass in: xPackage.
self createMethodNamed: #stubMethod inClass: class asExtensionOf: yPackage.

class rename: 'RPackageNewStubClass'.
class rename: 'PackageNewStubClass'.

self assert: (class extendingPackages includes: yPackage)
]
Expand Down
2 changes: 0 additions & 2 deletions src/Kernel-CodeModel-Tests/PackageOrganizerTest.class.st
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,6 @@ PackageOrganizerTest >> testExtensionMethodNotExactlyTheName [
PackageOrganizerTest >> testFullRegistration [

| xPackage yPackage zPackage a1 a2 b1 b2 a3 |
"taken from setup of RPackageReadOnlyCompleteSetup"
xPackage := self ensureXPackage.
yPackage := self ensureYPackage.
zPackage := self ensureZPackage.
Expand Down Expand Up @@ -360,7 +359,6 @@ PackageOrganizerTest >> testRemoveEmptyPackagesAndTags [
PackageOrganizerTest >> testRemovePackage [

| xPackage yPackage zPackage a1 a2 b1 b2 a3 |
"taken from setup of RPackageReadOnlyCompleteSetup"
xPackage := self ensureXPackage.
yPackage := self ensureYPackage.
zPackage := self ensureZPackage.
Expand Down
16 changes: 8 additions & 8 deletions src/Kernel-CodeModel-Tests/PackageTest.class.st
Original file line number Diff line number Diff line change
Expand Up @@ -131,10 +131,10 @@ PackageTest >> testAnonymousClassAndSelector [
| ghost method |
ghost := Object newAnonymousSubclassInEnvironment: testEnvironment.
method := ghost compiler compile: 'rpackagetest'.
ghost addSelector: #rpackagetest withMethod: method.
self deny: (self organizer undefinedPackage includesSelector: #rpackagetest ofClass: ghost).
ghost addSelector: #packagetest withMethod: method.
self deny: (self organizer undefinedPackage includesSelector: #packagetest ofClass: ghost).
self assert: (self organizer packageOfClassNamed: ghost name) equals: self organizer undefinedPackage.
ghost classify: #rpackagetest under: '*rpackagetest'
ghost classify: #packagetest under: '*packagetest'
]

{ #category : 'tests - extensions' }
Expand All @@ -154,7 +154,7 @@ PackageTest >> testAsPackageWithError [
]

{ #category : 'tests - demotion' }
PackageTest >> testDemoteToRPackageNamed [
PackageTest >> testDemoteToPackageNamed [

| package1 package2 class |
package1 := self ensurePackage: #'Test1-TAG1'.
Expand All @@ -171,7 +171,7 @@ PackageTest >> testDemoteToRPackageNamed [
]

{ #category : 'tests - demotion' }
PackageTest >> testDemoteToRPackageNamedExistingPackage [
PackageTest >> testDemoteToPackageNamedExistingPackage [

| package1 package2 packageExisting class |
package1 := self ensurePackage: #'Test1-TAG1'.
Expand All @@ -190,7 +190,7 @@ PackageTest >> testDemoteToRPackageNamedExistingPackage [
]

{ #category : 'tests - demotion' }
PackageTest >> testDemoteToRPackageNamedKeepOrganizer [
PackageTest >> testDemoteToPackageNamedKeepOrganizer [

| newOrganizer package renamedPackage |
newOrganizer := PackageOrganizer new.
Expand All @@ -203,7 +203,7 @@ PackageTest >> testDemoteToRPackageNamedKeepOrganizer [
]

{ #category : 'tests - demotion' }
PackageTest >> testDemoteToRPackageNamedMultilevelPackage [
PackageTest >> testDemoteToPackageNamedMultilevelPackage [

| package1 package2 class |
package1 := self ensurePackage: #'Test1-TAG1-X1'.
Expand All @@ -220,7 +220,7 @@ PackageTest >> testDemoteToRPackageNamedMultilevelPackage [
]

{ #category : 'tests - demotion' }
PackageTest >> testDemoteToRPackageNamedWithExtension [
PackageTest >> testDemoteToPackageNamedWithExtension [

| packageOriginal packageDemoted class classOther |
packageOriginal := self ensurePackage: #'Test1-TAG1'.
Expand Down
9 changes: 4 additions & 5 deletions src/Kernel-CodeModel/Package.class.st
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,6 @@ Class {

{ #category : 'class initialization' }
Package class >> initialize [
self deprecatedAliases: { #RPackage }.
Properties
ifNil: [ self initializeProperties ]
ifNotNil: [
Expand Down Expand Up @@ -403,10 +402,10 @@ Package >> isTestPackage [
2. Or test package contains '-Tests-' in middle.
Some examples: "

"(RPackage named: 'MockPackage-Tests') isTestPackage >>> true"
"(RPackage named: 'MockPackage-tests') isTestPackage >>> true"
"(RPackage named: 'MockPackage') isTestPackage >>> false"
"(RPackage named: 'MockPackage-Tests-Package') isTestPackage >>> true"
"(Package named: 'MockPackage-Tests') isTestPackage >>> true"
"(Package named: 'MockPackage-tests') isTestPackage >>> true"
"(Package named: 'MockPackage') isTestPackage >>> false"
"(Package named: 'MockPackage-Tests-Package') isTestPackage >>> true"

^ (self name endsWith: '-Tests' caseSensitive: false) or: [ self name includesSubstring: '-Tests-' caseSensitive: false ]
]
Expand Down
6 changes: 0 additions & 6 deletions src/Kernel-CodeModel/PackageConflictError.class.st
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,3 @@ Class {
#package : 'Kernel-CodeModel',
#tag : 'Exceptions'
}

{ #category : 'class initialization' }
PackageConflictError class >> initialize [

self deprecatedAliases: { #RPackageConflictError }
]
6 changes: 0 additions & 6 deletions src/Kernel-CodeModel/PackageOrganizer.class.st
Original file line number Diff line number Diff line change
Expand Up @@ -24,12 +24,6 @@ PackageOrganizer class >> default [
^ self packageOrganizer
]

{ #category : 'class initialization' }
PackageOrganizer class >> initialize [

self deprecatedAliases: { #RPackageOrganizer }
]

{ #category : 'registration' }
PackageOrganizer >> addPackage: aPackage [

Expand Down
6 changes: 0 additions & 6 deletions src/Kernel-CodeModel/PackageTag.class.st
Original file line number Diff line number Diff line change
Expand Up @@ -15,12 +15,6 @@ Class {
#tag : 'Packages'
}

{ #category : 'class initialization' }
PackageTag class >> initialize [

self deprecatedAliases: { #RPackageTag }
]

{ #category : 'instance creation' }
PackageTag class >> package: aPackage name: aString [

Expand Down
6 changes: 0 additions & 6 deletions src/Kernel-CodeModel/PseudoVariable.class.st
Original file line number Diff line number Diff line change
Expand Up @@ -12,12 +12,6 @@ Class {
#tag : 'Variables'
}

{ #category : 'class initialization' }
PseudoVariable class >> initialize [

self deprecatedAliases: { #ReservedVariable }
]

{ #category : 'testing' }
PseudoVariable class >> isAbstract [
^self = PseudoVariable
Expand Down
6 changes: 0 additions & 6 deletions src/Kernel/MessageNotUnderstood.class.st
Original file line number Diff line number Diff line change
Expand Up @@ -14,12 +14,6 @@ Class {
#tag : 'Exceptions'
}

{ #category : 'class initialization' }
MessageNotUnderstood class >> initialize [

self deprecatedAliases: { #VariableNotDeclared }
]

{ #category : 'accessing' }
MessageNotUnderstood >> defaultAction [

Expand Down
6 changes: 0 additions & 6 deletions src/Metacello-Base/ConfigurationOf.class.st
Original file line number Diff line number Diff line change
Expand Up @@ -67,12 +67,6 @@ Class {
#package : 'Metacello-Base'
}

{ #category : 'class initialization' }
ConfigurationOf class >> initialize [

self deprecatedAliases: { #MetacelloBaseConfiguration }
]

{ #category : 'accessing' }
ConfigurationOf class >> project [

Expand Down
3 changes: 1 addition & 2 deletions src/Metacello-Core/MetacelloPlatform.class.st
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,7 @@ MetacelloPlatform class >> current [
{ #category : 'class initialization' }
MetacelloPlatform class >> initialize [

self select.
self deprecatedAliases: { #MetacelloPharoPlatform }
self select
]

{ #category : 'initialize-release' }
Expand Down
6 changes: 0 additions & 6 deletions src/Metacello-Core/MetacelloProject.class.st
Original file line number Diff line number Diff line change
Expand Up @@ -15,12 +15,6 @@ Class {
#tag : 'Model'
}

{ #category : 'class initialization' }
MetacelloProject class >> initialize [

self deprecatedAliases: { #MetacelloMCProject }
]

{ #category : 'instance creation' }
MetacelloProject class >> new [

Expand Down
6 changes: 0 additions & 6 deletions src/Metacello-Core/MetacelloProjectSpec.class.st
Original file line number Diff line number Diff line change
Expand Up @@ -19,12 +19,6 @@ Class {
#tag : 'Specs'
}

{ #category : 'class initialization' }
MetacelloProjectSpec class >> initialize [

self deprecatedAliases: { #MetacelloMCProjectSpec. #MetacelloGenericProjectSpec }
]

{ #category : 'instance creation' }
MetacelloProjectSpec class >> new [
self name == #MetacelloProjectSpec
Expand Down
6 changes: 0 additions & 6 deletions src/Metacello-Core/MetacelloTarget.class.st
Original file line number Diff line number Diff line change
Expand Up @@ -13,12 +13,6 @@ Class {
#tag : 'Targets'
}

{ #category : 'class initialization' }
MetacelloTarget class >> initialize [

self deprecatedAliases: { #MetacelloCommonMCSpecLoader }
]

{ #category : 'utilities' }
MetacelloTarget class >> nameComponentsFrom: aVersionName [
| ar |
Expand Down
5 changes: 0 additions & 5 deletions src/Metacello-Core/MetacelloVersion.class.st
Original file line number Diff line number Diff line change
Expand Up @@ -25,11 +25,6 @@ MetacelloVersion class >> fromString: aString [
^self new versionNumber: aString asMetacelloVersionNumber
]

{ #category : 'class initialization' }
MetacelloVersion class >> initialize [
self deprecatedAliases: { #MetacelloMCVersion }
]

{ #category : 'comparing' }
MetacelloVersion >> < aMetacelloVersion [

Expand Down
6 changes: 0 additions & 6 deletions src/Metacello-Core/MetacelloVersionSpec.class.st
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,6 @@ Class {
#tag : 'Specs'
}

{ #category : 'class initialization' }
MetacelloVersionSpec class >> initialize [

self deprecatedAliases: { #MetacelloMCVersionSpec }
]

{ #category : 'visiting' }
MetacelloVersionSpec >> acceptVisitor: aVisitor [

Expand Down
Loading

0 comments on commit 0ac6afe

Please sign in to comment.