Move [NotMapped] after field attribute

Author: bkoelman

test/JsonApiDotNetCoreTests/IntegrationTests/AtomicOperations/Playlist.cs

@@ -12,8 +12,8 @@ public sealed class Playlist : Identifiable<long>
     [Attr]
     public string Name { get; set; } = null!;
 
-    [NotMapped]
     [Attr]
+    [NotMapped]
     public bool IsArchived => false;
 
     [HasMany]

test/JsonApiDotNetCoreTests/IntegrationTests/EagerLoading/Building.cs

@@ -14,12 +14,12 @@ public sealed class Building : Identifiable<int>
     [Attr]
     public string Number { get; set; } = null!;
 
-    [NotMapped]
     [Attr]
+    [NotMapped]
     public int WindowCount => Windows.Count;
 
-    [NotMapped]
     [Attr(Capabilities = AttrCapabilities.AllowView | AttrCapabilities.AllowChange)]
+    [NotMapped]
     public string PrimaryDoorColor
     {
         get
@@ -50,8 +50,8 @@ public string PrimaryDoorColor
         }
     }
 
-    [NotMapped]
     [Attr(Capabilities = AttrCapabilities.AllowView)]
+    [NotMapped]
     public string? SecondaryDoorColor => SecondaryDoor?.Color;
 
     [EagerLoad]

test/JsonApiDotNetCoreTests/IntegrationTests/EagerLoading/Street.cs

@@ -12,16 +12,16 @@ public sealed class Street : Identifiable<int>
     [Attr]
     public string Name { get; set; } = null!;
 
-    [NotMapped]
     [Attr(Capabilities = AttrCapabilities.AllowView)]
+    [NotMapped]
     public int BuildingCount => Buildings.Count;
 
-    [NotMapped]
     [Attr(Capabilities = AttrCapabilities.AllowView)]
+    [NotMapped]
     public int DoorTotalCount => Buildings.Sum(building => building.SecondaryDoor == null ? 1 : 2);
 
-    [NotMapped]
     [Attr(Capabilities = AttrCapabilities.AllowView)]
+    [NotMapped]
     public int WindowTotalCount => Buildings.Sum(building => building.WindowCount);
 
     [EagerLoad]

test/JsonApiDotNetCoreTests/IntegrationTests/ReadWrite/WorkItem.cs

@@ -18,8 +18,8 @@ public sealed class WorkItem : Identifiable<int>
     [Attr]
     public WorkItemPriority Priority { get; set; }
 
-    [NotMapped]
     [Attr(Capabilities = AttrCapabilities.All & ~(AttrCapabilities.AllowCreate | AttrCapabilities.AllowChange))]
+    [NotMapped]
     public bool IsImportant
     {
         get => Priority == WorkItemPriority.High;

test/JsonApiDotNetCoreTests/IntegrationTests/ReadWrite/WorkItemGroup.cs

@@ -15,8 +15,8 @@ public sealed class WorkItemGroup : Identifiable<Guid>
     [Attr]
     public bool IsPublic { get; set; }
 
-    [NotMapped]
     [Attr]
+    [NotMapped]
     public bool IsDeprecated => !string.IsNullOrEmpty(Name) && Name.StartsWith("DEPRECATED:", StringComparison.OrdinalIgnoreCase);
 
     [HasOne]

test/JsonApiDotNetCoreTests/IntegrationTests/ResourceInheritance/Models/AlwaysMovingTandem.cs

@@ -9,8 +9,8 @@ namespace JsonApiDotNetCoreTests.IntegrationTests.ResourceInheritance.Models;
 [Resource(ControllerNamespace = "JsonApiDotNetCoreTests.IntegrationTests.ResourceInheritance", GenerateControllerEndpoints = JsonApiEndpoints.None)]
 public sealed class AlwaysMovingTandem : Bike
 {
-    [NotMapped]
     [Attr]
+    [NotMapped]
     public Guid LocationToken
     {
         get => Guid.NewGuid();

test/JsonApiDotNetCoreTests/IntegrationTests/ZeroKeys/Game.cs

@@ -13,8 +13,8 @@ public sealed class Game : Identifiable<int?>
     [Attr]
     public string Title { get; set; } = null!;
 
-    [NotMapped]
     [Attr]
+    [NotMapped]
     public Guid SessionToken => Guid.NewGuid();
 
     [HasOne]