Continue refactoring

Author: SergeiPavlov

Orm/Xtensive.Orm/Sql/Ddl/Actions/SqlAddColumn.cs

@@ -12,10 +12,9 @@ public class SqlAddColumn : SqlAction
   {
     public TableColumn Column { get; private set; }
 
-    internal override object Clone(SqlNodeCloneContext context) =>
-      context.NodeMapping.TryGetValue(this, out var clone)
-        ? clone
-        : context.NodeMapping[this] = new SqlAddColumn(Column);
+    internal override SqlAddColumn Clone(SqlNodeCloneContext context) =>
+      context.GetOrAdd(this, static (t, c) =>
+        new SqlAddColumn(t.Column));
 
     // Constructors
 

Orm/Xtensive.Orm/Sql/Ddl/Actions/SqlAddConstraint.cs

@@ -12,10 +12,9 @@ public class SqlAddConstraint : SqlAction
   {
     public Constraint Constraint { get; private set; }
 
-    internal override object Clone(SqlNodeCloneContext context) =>
-      context.NodeMapping.TryGetValue(this, out var clone)
-        ? clone
-        : context.NodeMapping[this] = new SqlAddConstraint(Constraint);
+    internal override SqlAddConstraint Clone(SqlNodeCloneContext context) =>
+      context.GetOrAdd(this, static (t, c) =>
+        new SqlAddConstraint(t.Constraint));
 
     // Constructors
 

Orm/Xtensive.Orm/Sql/Ddl/Actions/SqlAlterIdentityInfo.cs

@@ -14,10 +14,9 @@ public class SqlAlterIdentityInfo : SqlAction
     public SequenceDescriptor SequenceDescriptor { get; set; }
     public SqlAlterIdentityInfoOptions InfoOption { get; set; }
 
-    internal override object Clone(SqlNodeCloneContext context) =>
-      context.NodeMapping.TryGetValue(this, out var clone)
-        ? clone
-        : context.NodeMapping[this] = new SqlAlterIdentityInfo(Column, (SequenceDescriptor) SequenceDescriptor.Clone(), InfoOption);
+    internal override SqlAlterIdentityInfo Clone(SqlNodeCloneContext context) =>
+      context.GetOrAdd(this, static (t, c) =>
+        new SqlAlterIdentityInfo(t.Column, (SequenceDescriptor) t.SequenceDescriptor.Clone(), t.InfoOption));
 
     internal SqlAlterIdentityInfo(TableColumn column, SequenceDescriptor sequenceDescriptor, SqlAlterIdentityInfoOptions infoOption)
     {

Orm/Xtensive.Orm/Sql/Ddl/Actions/SqlDropColumn.cs

@@ -12,10 +12,9 @@ public class SqlDropColumn : SqlCascadableAction
   {
     public TableColumn Column { get; private set; }
 
-    internal override object Clone(SqlNodeCloneContext context) =>
-      context.NodeMapping.TryGetValue(this, out var clone)
-        ? clone
-        : context.NodeMapping[this] = new SqlDropColumn(Column);
+    internal override SqlDropColumn Clone(SqlNodeCloneContext context) =>
+      context.GetOrAdd(this, static (t, c) =>
+        new SqlDropColumn(t.Column));
 
     // Constructors
 

Orm/Xtensive.Orm/Sql/Ddl/Actions/SqlDropConstraint.cs

@@ -12,10 +12,9 @@ public class SqlDropConstraint : SqlCascadableAction
   {
     public Constraint Constraint { get; private set; }
 
-    internal override object Clone(SqlNodeCloneContext context) =>
-      context.NodeMapping.TryGetValue(this, out var clone)
-        ? clone
-        : context.NodeMapping[this] = new SqlDropConstraint(Constraint, Cascade);
+    internal override SqlDropConstraint Clone(SqlNodeCloneContext context) =>
+      context.GetOrAdd(this, static (t, c) =>
+        new SqlDropConstraint(t.Constraint, t.Cascade));
 
     // Constructors
 

Orm/Xtensive.Orm/Sql/Ddl/Actions/SqlDropDefault.cs

@@ -12,10 +12,9 @@ public class SqlDropDefault : SqlAction
   {
     public TableColumn Column { get; private set; }
 
-    internal override object Clone(SqlNodeCloneContext context) =>
-      context.NodeMapping.TryGetValue(this, out var clone)
-        ? clone
-        : context.NodeMapping[this] = new SqlDropDefault(Column);
+    internal override SqlDropDefault Clone(SqlNodeCloneContext context) =>
+      context.GetOrAdd(this, static (t, c) =>
+        new SqlDropDefault(t.Column));
 
     // Constructors
 

Orm/Xtensive.Orm/Sql/Ddl/Actions/SqlRenameColumn.cs

@@ -15,10 +15,9 @@ public class SqlRenameColumn : SqlAction
     public TableColumn Column { get; private set; }
     public string NewName { get; private set; }
 
-    internal override object Clone(SqlNodeCloneContext context) =>
-      context.NodeMapping.TryGetValue(this, out var clone)
-        ? clone
-        : context.NodeMapping[this] = new SqlRenameColumn(Column, NewName);
+    internal override SqlRenameColumn Clone(SqlNodeCloneContext context) =>
+      context.GetOrAdd(this, static (t, c) =>
+        new SqlRenameColumn(t.Column, t.NewName));
 
 
     // Constructors

Orm/Xtensive.Orm/Sql/Ddl/Actions/SqlSetDefault.cs

@@ -14,10 +14,9 @@ public class SqlSetDefault : SqlAction
     public TableColumn Column { get; private set; }
     public SqlExpression DefaultValue { get; private set; }
 
-    internal override object Clone(SqlNodeCloneContext context) =>
-      context.NodeMapping.TryGetValue(this, out var clone)
-        ? clone
-        : context.NodeMapping[this] = new SqlSetDefault(DefaultValue.Clone(context), Column);
+    internal override SqlSetDefault Clone(SqlNodeCloneContext context) =>
+      context.GetOrAdd(this, static (t, c) =>
+        new SqlSetDefault(t.DefaultValue.Clone(c), t.Column));
 
     internal SqlSetDefault(SqlExpression defaultValue, TableColumn column)
     {

Orm/Xtensive.Orm/Sql/Ddl/SqlAlterDomain.cs

@@ -25,11 +25,9 @@ public Domain Domain {
       }
     }
 
-
-    internal override object Clone(SqlNodeCloneContext context) =>
-      context.NodeMapping.TryGetValue(this, out var clone)
-        ? clone
-        : context.NodeMapping[this] = new SqlAlterDomain(domain, (SqlAction)action.Clone(context));
+    internal override SqlAlterDomain Clone(SqlNodeCloneContext context) =>
+      context.GetOrAdd(this, static (t, c) =>
+        new SqlAlterDomain(t.domain, (SqlAction)t.action.Clone(c)));
 
     public override void AcceptVisitor(ISqlVisitor visitor)
     {

Orm/Xtensive.Orm/Sql/Ddl/SqlAlterPartitionFunction.cs

@@ -31,10 +31,9 @@ public SqlAlterPartitionFunctionOption Option
       set { option = value; }
     }
 
-    internal override object Clone(SqlNodeCloneContext context) =>
-      context.NodeMapping.TryGetValue(this, out var clone)
-        ? clone
-        : context.NodeMapping[this] = new SqlAlterPartitionFunction(partitionFunction, boundary, option);
+    internal override SqlAlterPartitionFunction Clone(SqlNodeCloneContext context) =>
+      context.GetOrAdd(this, static (t, c) =>
+        new SqlAlterPartitionFunction(t.partitionFunction, t.boundary, t.option));
 
     public override void AcceptVisitor(ISqlVisitor visitor)
     {