From 2e7c233f6c9bf6430672b39f970a3324deba79dd Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <Pathoschild@users.noreply.github.com>
Date: Wed, 6 Apr 2022 21:48:55 -0400
Subject: enable nullable annotations by default (#837)

This adds `#nullable disable` to all existing code (except where null is impossible like enum files), so it can be migrated incrementally.
---
 .../Framework/Clients/CurseForge/ResponseModels/ModFileModel.cs         | 2 ++
 src/SMAPI.Web/Framework/Clients/CurseForge/ResponseModels/ModModel.cs   | 2 ++
 2 files changed, 4 insertions(+)

(limited to 'src/SMAPI.Web/Framework/Clients/CurseForge/ResponseModels')

diff --git a/src/SMAPI.Web/Framework/Clients/CurseForge/ResponseModels/ModFileModel.cs b/src/SMAPI.Web/Framework/Clients/CurseForge/ResponseModels/ModFileModel.cs
index 9de74847..eabef9f0 100644
--- a/src/SMAPI.Web/Framework/Clients/CurseForge/ResponseModels/ModFileModel.cs
+++ b/src/SMAPI.Web/Framework/Clients/CurseForge/ResponseModels/ModFileModel.cs
@@ -1,3 +1,5 @@
+#nullable disable
+
 namespace StardewModdingAPI.Web.Framework.Clients.CurseForge.ResponseModels
 {
     /// <summary>Metadata from the CurseForge API about a mod file.</summary>
diff --git a/src/SMAPI.Web/Framework/Clients/CurseForge/ResponseModels/ModModel.cs b/src/SMAPI.Web/Framework/Clients/CurseForge/ResponseModels/ModModel.cs
index 48cd185b..a95df7f1 100644
--- a/src/SMAPI.Web/Framework/Clients/CurseForge/ResponseModels/ModModel.cs
+++ b/src/SMAPI.Web/Framework/Clients/CurseForge/ResponseModels/ModModel.cs
@@ -1,3 +1,5 @@
+#nullable disable
+
 namespace StardewModdingAPI.Web.Framework.Clients.CurseForge.ResponseModels
 {
     /// <summary>An mod from the CurseForge API.</summary>
-- 
cgit 


From 0b48c1748b354458059c7607415288de072b01e9 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <Pathoschild@users.noreply.github.com>
Date: Tue, 12 Apr 2022 19:15:39 -0400
Subject: enable nullable annotations in the web project & related code (#837)

---
 .../CurseForge/ResponseModels/ModFileModel.cs      | 22 +++++++++++++---
 .../Clients/CurseForge/ResponseModels/ModModel.cs  | 30 +++++++++++++++++-----
 2 files changed, 42 insertions(+), 10 deletions(-)

(limited to 'src/SMAPI.Web/Framework/Clients/CurseForge/ResponseModels')

diff --git a/src/SMAPI.Web/Framework/Clients/CurseForge/ResponseModels/ModFileModel.cs b/src/SMAPI.Web/Framework/Clients/CurseForge/ResponseModels/ModFileModel.cs
index eabef9f0..e9adcf20 100644
--- a/src/SMAPI.Web/Framework/Clients/CurseForge/ResponseModels/ModFileModel.cs
+++ b/src/SMAPI.Web/Framework/Clients/CurseForge/ResponseModels/ModFileModel.cs
@@ -1,14 +1,28 @@
-#nullable disable
-
 namespace StardewModdingAPI.Web.Framework.Clients.CurseForge.ResponseModels
 {
     /// <summary>Metadata from the CurseForge API about a mod file.</summary>
     public class ModFileModel
     {
+        /*********
+        ** Accessors
+        *********/
         /// <summary>The file name as downloaded.</summary>
-        public string FileName { get; set; }
+        public string FileName { get; }
 
         /// <summary>The file display name.</summary>
-        public string DisplayName { get; set; }
+        public string? DisplayName { get; }
+
+
+        /*********
+        ** Public methods
+        *********/
+        /// <summary>Construct an instance.</summary>
+        /// <param name="fileName">The file name as downloaded.</param>
+        /// <param name="displayName">The file display name.</param>
+        public ModFileModel(string fileName, string? displayName)
+        {
+            this.FileName = fileName;
+            this.DisplayName = displayName;
+        }
     }
 }
diff --git a/src/SMAPI.Web/Framework/Clients/CurseForge/ResponseModels/ModModel.cs b/src/SMAPI.Web/Framework/Clients/CurseForge/ResponseModels/ModModel.cs
index a95df7f1..fd7796f2 100644
--- a/src/SMAPI.Web/Framework/Clients/CurseForge/ResponseModels/ModModel.cs
+++ b/src/SMAPI.Web/Framework/Clients/CurseForge/ResponseModels/ModModel.cs
@@ -1,20 +1,38 @@
-#nullable disable
-
 namespace StardewModdingAPI.Web.Framework.Clients.CurseForge.ResponseModels
 {
     /// <summary>An mod from the CurseForge API.</summary>
     public class ModModel
     {
+        /*********
+        ** Accessors
+        *********/
         /// <summary>The mod's unique ID on CurseForge.</summary>
-        public int ID { get; set; }
+        public int ID { get; }
 
         /// <summary>The mod name.</summary>
-        public string Name { get; set; }
+        public string Name { get; }
 
         /// <summary>The web URL for the mod page.</summary>
-        public string WebsiteUrl { get; set; }
+        public string WebsiteUrl { get; }
 
         /// <summary>The available file downloads.</summary>
-        public ModFileModel[] LatestFiles { get; set; }
+        public ModFileModel[] LatestFiles { get; }
+
+
+        /*********
+        ** Public methods
+        *********/
+        /// <summary>Construct an instance.</summary>
+        /// <param name="id">The mod's unique ID on CurseForge.</param>
+        /// <param name="name">The mod name.</param>
+        /// <param name="websiteUrl">The web URL for the mod page.</param>
+        /// <param name="latestFiles">The available file downloads.</param>
+        public ModModel(int id, string name, string websiteUrl, ModFileModel[] latestFiles)
+        {
+            this.ID = id;
+            this.Name = name;
+            this.WebsiteUrl = websiteUrl;
+            this.LatestFiles = latestFiles;
+        }
     }
 }
-- 
cgit