Convert Node.Name into init setter to avoid on-change subscriptions

Author: SergeiPavlov

Orm/Xtensive.Orm/Orm/Model/ColumnInfo.cs

@@ -249,8 +249,7 @@ public override bool Equals(object obj) =>
     /// </summary>
     public ColumnInfo Clone()
     {
-      ColumnInfo clone = new ColumnInfo(fld);
-      clone.Name = Name;
+      ColumnInfo clone = new(fld) { Name = Name };
       clone.attributes = attributes;
       clone.valueType = valueType;
       clone.length = length;

Orm/Xtensive.Orm/Orm/Model/Node.cs

@@ -15,7 +15,7 @@ namespace Xtensive.Orm.Model
   ///An abstract base class for model node.
   /// </summary>
   [Serializable]
-  public abstract class Node : LockableBase, IChangeNotifier
+  public abstract class Node : LockableBase
   {
     private string name;
 
@@ -24,23 +24,16 @@ public abstract class Node : LockableBase, IChangeNotifier
     /// </summary>
     public string Name
     {
-      get { return name; }
+      get => name;
       set {
         EnsureNotLocked();
+        if (name is not null)
+          throw new InvalidOperationException("The node Name is locked.");
         ValidateName(value);
-        ChangeState("Name", delegate { name = value; });
+        name = value;
       }
     }
 
-    private void ChangeState(string property, Action onChangeStateDelegate)
-    {
-      if (Changing != null)
-        Changing(this, new ChangeNotifierEventArgs(property));
-      onChangeStateDelegate();
-      if (Changed != null)
-        Changed(this, new ChangeNotifierEventArgs(property));
-    }
-
     /// <summary>
     /// Performs additional custom processes before setting new name to this instance.
     /// </summary>
@@ -49,16 +42,6 @@ protected virtual void ValidateName(string newName)
     {
     }
 
-    #region IChangeNotifier Members
-
-    /// <inheritdoc/>
-    public event EventHandler<ChangeNotifierEventArgs> Changing;
-
-    /// <inheritdoc/>
-    public event EventHandler<ChangeNotifierEventArgs> Changed;
-
-    #endregion
-
     /// <summary>
     /// Updates the internal state of this instance.
     /// </summary>