[ActivatorUtilitiesConstructor] is dependent on constructor ordering (#6959)

Author: david-driscoll

src/HotChocolate/Core/src/Types.NodaTime/DurationType.cs

@@ -13,14 +13,6 @@ public class DurationType : StringToStructBaseType<Duration>
     private readonly IPattern<Duration>[] _allowedPatterns;
     private readonly IPattern<Duration> _serializationPattern;
 
-    /// <summary>
-    /// Initializes a new instance of <see cref="DurationType"/>.
-    /// </summary>
-    [ActivatorUtilitiesConstructor]
-    public DurationType() : this(DurationPattern.Roundtrip)
-    {
-    }
-
     /// <summary>
     /// Initializes a new instance of <see cref="DurationType"/>.
     /// </summary>
@@ -36,6 +28,14 @@ public DurationType(params IPattern<Duration>[] allowedPatterns) : base("Duratio
         Description = NodaTimeResources.DurationType_Description;
     }
 
+    /// <summary>
+    /// Initializes a new instance of <see cref="DurationType"/>.
+    /// </summary>
+    [ActivatorUtilitiesConstructor]
+    public DurationType() : this(DurationPattern.Roundtrip)
+    {
+    }
+
     /// <inheritdoc />
     protected override string Serialize(Duration runtimeValue)
         => _serializationPattern

src/HotChocolate/Core/src/Types.NodaTime/InstantType.cs

@@ -13,14 +13,6 @@ public class InstantType : StringToStructBaseType<Instant>
     private readonly IPattern<Instant>[] _allowedPatterns;
     private readonly IPattern<Instant> _serializationPattern;
 
-    /// <summary>
-    /// Initializes a new instance of <see cref="InstantType"/>.
-    /// </summary>
-    [ActivatorUtilitiesConstructor]
-    public InstantType() : this(InstantPattern.ExtendedIso)
-    {
-    }
-
     /// <summary>
     /// Initializes a new instance of <see cref="InstantType"/>.
     /// </summary>
@@ -36,6 +28,14 @@ public InstantType(params IPattern<Instant>[] allowedPatterns) : base("Instant")
         Description = NodaTimeResources.InstantType_Description;
     }
 
+    /// <summary>
+    /// Initializes a new instance of <see cref="InstantType"/>.
+    /// </summary>
+    [ActivatorUtilitiesConstructor]
+    public InstantType() : this(InstantPattern.ExtendedIso)
+    {
+    }
+
     /// <inheritdoc />
     protected override string Serialize(Instant runtimeValue)
         => _serializationPattern

src/HotChocolate/Core/src/Types.NodaTime/LocalDateTimeType.cs

@@ -13,14 +13,6 @@ public class LocalDateTimeType : StringToStructBaseType<LocalDateTime>
     private readonly IPattern<LocalDateTime>[] _allowedPatterns;
     private readonly IPattern<LocalDateTime> _serializationPattern;
 
-    /// <summary>
-    /// Initializes a new instance of <see cref="LocalDateTimeType"/>.
-    /// </summary>
-    [ActivatorUtilitiesConstructor]
-    public LocalDateTimeType() : this(LocalDateTimePattern.ExtendedIso)
-    {
-    }
-
     /// <summary>
     /// Initializes a new instance of <see cref="LocalDateTimeType"/>.
     /// </summary>
@@ -36,6 +28,14 @@ public LocalDateTimeType(params IPattern<LocalDateTime>[] allowedPatterns) : bas
         Description = NodaTimeResources.LocalDateTimeType_Description;
     }
 
+    /// <summary>
+    /// Initializes a new instance of <see cref="LocalDateTimeType"/>.
+    /// </summary>
+    [ActivatorUtilitiesConstructor]
+    public LocalDateTimeType() : this(LocalDateTimePattern.ExtendedIso)
+    {
+    }
+
     /// <inheritdoc />
     protected override string Serialize(LocalDateTime runtimeValue)
         => _serializationPattern

src/HotChocolate/Core/src/Types.NodaTime/LocalDateType.cs

@@ -14,14 +14,6 @@ public class LocalDateType : StringToStructBaseType<LocalDate>
     private readonly IPattern<LocalDate>[] _allowedPatterns;
     private readonly IPattern<LocalDate> _serializationPattern;
 
-    /// <summary>
-    /// Initializes a new instance of <see cref="LocalDateType"/>.
-    /// </summary>
-    [ActivatorUtilitiesConstructor]
-    public LocalDateType() : this(LocalDatePattern.Iso)
-    {
-    }
-
     /// <summary>
     /// Initializes a new instance of <see cref="LocalDateType"/>.
     /// </summary>
@@ -37,6 +29,14 @@ public LocalDateType(params IPattern<LocalDate>[] allowedPatterns) : base("Local
         Description = NodaTimeResources.LocalDateType_Description;
     }
 
+    /// <summary>
+    /// Initializes a new instance of <see cref="LocalDateType"/>.
+    /// </summary>
+    [ActivatorUtilitiesConstructor]
+    public LocalDateType() : this(LocalDatePattern.Iso)
+    {
+    }
+
     /// <inheritdoc />
     protected override string Serialize(LocalDate runtimeValue)
         => _serializationPattern

src/HotChocolate/Core/src/Types.NodaTime/LocalTimeType.cs

@@ -14,14 +14,6 @@ public class LocalTimeType : StringToStructBaseType<LocalTime>
     private readonly IPattern<LocalTime>[] _allowedPatterns;
     private readonly IPattern<LocalTime> _serializationPattern;
 
-    /// <summary>
-    /// Initializes a new instance of <see cref="LocalTimeType"/>.
-    /// </summary>
-    [ActivatorUtilitiesConstructor]
-    public LocalTimeType() : this(LocalTimePattern.ExtendedIso)
-    {
-    }
-
     /// <summary>
     /// Initializes a new instance of <see cref="LocalTimeType"/>.
     /// </summary>
@@ -37,6 +29,14 @@ public LocalTimeType(params IPattern<LocalTime>[] allowedPatterns) : base("Local
         Description = NodaTimeResources.LocalTimeType_Description;
     }
 
+    /// <summary>
+    /// Initializes a new instance of <see cref="LocalTimeType"/>.
+    /// </summary>
+    [ActivatorUtilitiesConstructor]
+    public LocalTimeType() : this(LocalTimePattern.ExtendedIso)
+    {
+    }
+
     /// <inheritdoc />
     protected override string Serialize(LocalTime runtimeValue)
         => _serializationPattern

src/HotChocolate/Core/src/Types.NodaTime/OffsetDateTimeType.cs

@@ -13,17 +13,6 @@ public class OffsetDateTimeType : StringToStructBaseType<OffsetDateTime>
     private readonly IPattern<OffsetDateTime>[] _allowedPatterns;
     private readonly IPattern<OffsetDateTime> _serializationPattern;
 
-    /// <summary>
-    /// Initializes a new instance of <see cref="OffsetDateTimeType"/>.
-    /// </summary>
-    [ActivatorUtilitiesConstructor]
-    public OffsetDateTimeType() : this(OffsetDateTimePattern.ExtendedIso)
-    {
-        // Backwards compatibility with the original code's behavior
-        _serializationPattern = OffsetDateTimePattern.GeneralIso;
-        _allowedPatterns = [OffsetDateTimePattern.ExtendedIso,];
-    }
-
     /// <summary>
     /// Initializes a new instance of <see cref="OffsetDateTimeType"/>.
     /// </summary>
@@ -40,6 +29,17 @@ public OffsetDateTimeType(params IPattern<OffsetDateTime>[] allowedPatterns)
         Description = NodaTimeResources.OffsetDateTimeType_Description;
     }
 
+    /// <summary>
+    /// Initializes a new instance of <see cref="OffsetDateTimeType"/>.
+    /// </summary>
+    [ActivatorUtilitiesConstructor]
+    public OffsetDateTimeType() : this(OffsetDateTimePattern.ExtendedIso)
+    {
+        // Backwards compatibility with the original code's behavior
+        _serializationPattern = OffsetDateTimePattern.GeneralIso;
+        _allowedPatterns = [OffsetDateTimePattern.ExtendedIso,];
+    }
+
     /// <inheritdoc />
     protected override string Serialize(OffsetDateTime runtimeValue)
         => _serializationPattern

src/HotChocolate/Core/src/Types.NodaTime/OffsetDateType.cs

@@ -15,14 +15,6 @@ public class OffsetDateType : StringToStructBaseType<OffsetDate>
     private readonly IPattern<OffsetDate>[] _allowedPatterns;
     private readonly IPattern<OffsetDate> _serializationPattern;
 
-    /// <summary>
-    /// Initializes a new instance of <see cref="OffsetDateType"/>.
-    /// </summary>
-    [ActivatorUtilitiesConstructor]
-    public OffsetDateType() : this(OffsetDatePattern.GeneralIso)
-    {
-    }
-
     /// <summary>
     /// Initializes a new instance of <see cref="OffsetDateType"/>.
     /// </summary>
@@ -38,6 +30,14 @@ public OffsetDateType(params IPattern<OffsetDate>[] allowedPatterns) : base("Off
         Description = NodaTimeResources.OffsetDateType_Description;
     }
 
+    /// <summary>
+    /// Initializes a new instance of <see cref="OffsetDateType"/>.
+    /// </summary>
+    [ActivatorUtilitiesConstructor]
+    public OffsetDateType() : this(OffsetDatePattern.GeneralIso)
+    {
+    }
+
     /// <inheritdoc />
     protected override string Serialize(OffsetDate runtimeValue)
         => _serializationPattern

src/HotChocolate/Core/src/Types.NodaTime/OffsetTimeType.cs

@@ -14,14 +14,6 @@ public class OffsetTimeType : StringToStructBaseType<OffsetTime>
     private readonly IPattern<OffsetTime>[] _allowedPatterns;
     private readonly IPattern<OffsetTime> _serializationPattern;
 
-    /// <summary>
-    /// Initializes a new instance of <see cref="OffsetTimeType"/>.
-    /// </summary>
-    [ActivatorUtilitiesConstructor]
-    public OffsetTimeType() : this(OffsetTimePattern.GeneralIso)
-    {
-    }
-
     /// <summary>
     /// Initializes a new instance of <see cref="OffsetTimeType"/>.
     /// </summary>
@@ -37,6 +29,14 @@ public OffsetTimeType(params IPattern<OffsetTime>[] allowedPatterns) : base("Off
         Description = NodaTimeResources.OffsetTimeType_Description;
     }
 
+    /// <summary>
+    /// Initializes a new instance of <see cref="OffsetTimeType"/>.
+    /// </summary>
+    [ActivatorUtilitiesConstructor]
+    public OffsetTimeType() : this(OffsetTimePattern.GeneralIso)
+    {
+    }
+
     /// <inheritdoc />
     protected override string Serialize(OffsetTime runtimeValue)
         => _serializationPattern

src/HotChocolate/Core/src/Types.NodaTime/OffsetType.cs

@@ -15,14 +15,6 @@ public class OffsetType : StringToStructBaseType<Offset>
     private readonly IPattern<Offset>[] _allowedPatterns;
     private readonly IPattern<Offset> _serializationPattern;
 
-    /// <summary>
-    /// Initializes a new instance of <see cref="OffsetType"/>.
-    /// </summary>
-    [ActivatorUtilitiesConstructor]
-    public OffsetType() : this(OffsetPattern.GeneralInvariantWithZ)
-    {
-    }
-
     /// <summary>
     /// Initializes a new instance of <see cref="OffsetType"/>.
     /// </summary>
@@ -38,6 +30,14 @@ public OffsetType(params IPattern<Offset>[] allowedPatterns) : base("Offset")
         Description = NodaTimeResources.OffsetType_Description;
     }
 
+    /// <summary>
+    /// Initializes a new instance of <see cref="OffsetType"/>.
+    /// </summary>
+    [ActivatorUtilitiesConstructor]
+    public OffsetType() : this(OffsetPattern.GeneralInvariantWithZ)
+    {
+    }
+
     /// <inheritdoc />
     protected override string Serialize(Offset runtimeValue)
         => _serializationPattern

src/HotChocolate/Core/src/Types.NodaTime/PeriodType.cs

@@ -14,14 +14,6 @@ public class PeriodType : StringToClassBaseType<Period>
     private readonly IPattern<Period>[] _allowedPatterns;
     private readonly IPattern<Period> _serializationPattern;
 
-    /// <summary>
-    /// Initializes a new instance of <see cref="PeriodType"/>.
-    /// </summary>
-    [ActivatorUtilitiesConstructor]
-    public PeriodType() : this(PeriodPattern.Roundtrip)
-    {
-    }
-
     /// <summary>
     /// Initializes a new instance of <see cref="PeriodType"/>.
     /// </summary>
@@ -37,6 +29,14 @@ public PeriodType(params IPattern<Period>[] allowedPatterns) : base("Period")
         Description = NodaTimeResources.PeriodType_Description;
     }
 
+    /// <summary>
+    /// Initializes a new instance of <see cref="PeriodType"/>.
+    /// </summary>
+    [ActivatorUtilitiesConstructor]
+    public PeriodType() : this(PeriodPattern.Roundtrip)
+    {
+    }
+
     /// <inheritdoc />
     protected override string Serialize(Period runtimeValue)
         => _serializationPattern.Format(runtimeValue);