Fix #70. The CurrentStep can be null.

jsgoupil · Jun 6, 2023 · 5ce3a40 · 5ce3a40
1 parent d35c331
commit 5ce3a40
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 8 deletions.
diff --git a/src/WebConnector.AspNetCore/QbManager.cs b/src/WebConnector.AspNetCore/QbManager.cs
@@ -552,7 +552,7 @@ protected internal virtual Task<string> GetCompanyFileAsync(IAuthenticatedTicket
         /// </summary>
         /// <param name="step">Step name.</param>
         /// <returns>A IStepQueryRequest.</returns>
-        protected internal IStepQueryRequest? FindStepRequest(string step)
+        protected internal IStepQueryRequest? FindStepRequest(string? step)
         {
             if (string.IsNullOrEmpty(step))
             {
@@ -567,7 +567,7 @@ protected internal virtual Task<string> GetCompanyFileAsync(IAuthenticatedTicket
         /// </summary>
         /// <param name="step">Step name.</param>
         /// <returns>A IStepQueryResponse.</returns>
-        protected internal IStepQueryResponse? FindStepResponse(string step)
+        protected internal IStepQueryResponse? FindStepResponse(string? step)
         {
             if (string.IsNullOrEmpty(step))
             {
@@ -582,7 +582,7 @@ protected internal virtual Task<string> GetCompanyFileAsync(IAuthenticatedTicket
         /// </summary>
         /// <param name="step">Step name.</param>
         /// <returns>The next step name or null if none.</returns>
-        protected internal string FindNextStepName(string step)
+        protected internal string FindNextStepName(string? step)
         {
             var hasStep = false;
             using var enumerator = stepRequest.GetEnumerator();

diff --git a/src/WebConnector.AspNetCore/WebConnector.AspNetCore.csproj b/src/WebConnector.AspNetCore/WebConnector.AspNetCore.csproj
@@ -8,7 +8,7 @@
     <Author>Jean-Sébastien Goupil</Author>
     <Authors>Jean-Sébastien Goupil</Authors>
     <Copyright>Copyright © Jean-Sébastien Goupil 2015-2021</Copyright>
-    <Version>3.1.2</Version>
+    <Version>3.1.3</Version>
     <AssemblyName>QbSync.WebConnector.AspNetCore</AssemblyName>
     <RootNamespace>QbSync.WebConnector.AspNetCore</RootNamespace>
     <PackageId>QbSync.WebConnector.AspNetCore</PackageId>

diff --git a/src/WebConnector/Core/AuthenticatedTicket.cs b/src/WebConnector/Core/AuthenticatedTicket.cs
@@ -13,7 +13,7 @@ public interface IAuthenticatedTicket
         /// <summary>
         /// State indicating what to exchange with the Web Connector.
         /// </summary>
-        string CurrentStep { get; set; }
+        string? CurrentStep { get; set; }
 
         /// <summary>
         /// Simple boolean indicating if the ticket is authenticated.

diff --git a/src/WebConnector/WebConnector.csproj b/src/WebConnector/WebConnector.csproj
@@ -8,7 +8,7 @@
     <Author>Jean-Sébastien Goupil</Author>
     <Authors>Jean-Sébastien Goupil</Authors>
     <Copyright>Copyright © Jean-Sébastien Goupil 2015-2021</Copyright>
-    <Version>3.1.2</Version>
+    <Version>3.1.3</Version>
     <AssemblyName>QbSync.WebConnector</AssemblyName>
     <RootNamespace>QbSync.WebConnector</RootNamespace>
     <PackageId>QbSync.WebConnector</PackageId>

diff --git a/test/WebApplication.Sample/Db/ApplicationDbContext.cs b/test/WebApplication.Sample/Db/ApplicationDbContext.cs
@@ -42,15 +42,15 @@ public class QbKvpState
         public string Ticket { get; set; } = default!;
         public string Key { get; set; } = default!;
         public string? Value { get; set; }
-        public string CurrentStep { get; set; } = default!;
+        public string? CurrentStep { get; set; }
     }
 
     public class QbTicket : IAuthenticatedTicket
     {
         [Key]
         public string Ticket { get; set; } = default!;
 
-        public string CurrentStep { get; set; } = default!;
+        public string? CurrentStep { get; set; }
 
         public bool Authenticated { get; set; }