Make IntegerSlider the only AKA for migration (#11360)

mmisol · web-flow · commit 78f563fc275a · 2020-12-14T13:53:32.000-05:00
There were two classes that were declaring an 'AlsoKnownAs' attribute
for 'DSCoreNodesUI.Input.IntegerSlider' in 'CoreNodeModels.Input':
'IntegerSlider' and 'IntegerSlider64Bit'. While the latter obsoletes
the former, it is not possible to migrate to it from XML, so only 
'IntegerSlider' should continue declaring an 'AlsoKnownAs' attribute.
Note that currently both classes are trying to be registered, which
results in only one of them being registered but not really making any
guarantees of which one it is. This also generates an exception catched
internally but shown in the console.
diff --git a/src/Libraries/CoreNodeModels/Input/IntegerSlider.cs b/src/Libraries/CoreNodeModels/Input/IntegerSlider.cs
@@ -22,8 +22,8 @@ namespace CoreNodeModels.Input
     [OutPortTypes("int")]
     [SupressImportIntoVM]
     [IsDesignScriptCompatible]
-    [AlsoKnownAs("DSCoreNodesUI.Input.IntegerSlider")]
     [IsVisibleInDynamoLibrary(false)]
+    [AlsoKnownAs("DSCoreNodesUI.Input.IntegerSlider")]
     public class IntegerSlider : SliderBase<int>
     {
         /// <summary>
@@ -218,7 +218,6 @@ protected override void DeserializeCore(XmlElement element, SaveContext context)
     [OutPortTypes("int")]
     [SupressImportIntoVM]
     [IsDesignScriptCompatible]
-    [AlsoKnownAs("DSCoreNodesUI.Input.IntegerSlider")]
     public class IntegerSlider64Bit : SliderBase<long>
     {
         /// <summary>
