feat: fixed github warning - Cannot convert null literal to non-nullable reference type.

OMpawar-21 · OMpawar-21 · commit 658680421ba0 · 2026-05-20T06:53:22.000+05:30
diff --git a/Contentstack.Management.Core/Models/BaseModel.cs b/Contentstack.Management.Core/Models/BaseModel.cs
@@ -15,7 +15,7 @@ public class BaseModel<T>
         public string Uid { get; set; }
 
 
-        public BaseModel(Stack stack, string fieldName, string uid = null)
+        public BaseModel(Stack stack, string fieldName, string? uid = null)
         {
             stack.ThrowIfAPIKeyEmpty();
             if (fieldName == null)
@@ -27,15 +27,15 @@ public BaseModel(Stack stack, string fieldName, string uid = null)
             Uid = uid;
         }
 
-        public virtual ContentstackResponse Create(T model, ParameterCollection collection = null)
+        public virtual ContentstackResponse Create(T model, ParameterCollection? collection = null)
         {
             ThrowIfUidNotEmpty();
 
             var service = new CreateUpdateService<T>(stack, resourcePath, model, this.fieldName, collection: collection, stjOptions: stack.client.SerializerOptions);
             return stack.client.InvokeSync(service);
         }
 
-        public virtual Task<ContentstackResponse> CreateAsync(T model, ParameterCollection collection = null)
+        public virtual Task<ContentstackResponse> CreateAsync(T model, ParameterCollection? collection = null)
         {
             ThrowIfUidNotEmpty();
             stack.ThrowIfNotLoggedIn();
@@ -45,15 +45,15 @@ public virtual Task<ContentstackResponse> CreateAsync(T model, ParameterCollecti
             return stack.client.InvokeAsync<CreateUpdateService<T>, ContentstackResponse>(service);
         }
 
-        public virtual ContentstackResponse Update(T model, ParameterCollection collection = null)
+        public virtual ContentstackResponse Update(T model, ParameterCollection? collection = null)
         {
             ThrowIfUidEmpty();
 
             var service = new CreateUpdateService<T>(stack, resourcePath, model, this.fieldName, "PUT", collection: collection, stjOptions: stack.client.SerializerOptions);
             return stack.client.InvokeSync(service);
         }
 
-        public virtual Task<ContentstackResponse> UpdateAsync(T model, ParameterCollection collection = null)
+        public virtual Task<ContentstackResponse> UpdateAsync(T model, ParameterCollection? collection = null)
         {
             stack.ThrowIfNotLoggedIn();
             ThrowIfUidEmpty();
@@ -63,7 +63,7 @@ public virtual Task<ContentstackResponse> UpdateAsync(T model, ParameterCollecti
             return stack.client.InvokeAsync<CreateUpdateService<T>, ContentstackResponse>(service);
         }
 
-        public virtual ContentstackResponse Fetch(ParameterCollection collection = null)
+        public virtual ContentstackResponse Fetch(ParameterCollection? collection = null)
         {
             stack.ThrowIfNotLoggedIn();
             ThrowIfUidEmpty();
@@ -72,7 +72,7 @@ public virtual ContentstackResponse Fetch(ParameterCollection collection = null)
             return stack.client.InvokeSync(service);
         }
 
-        public virtual Task<ContentstackResponse> FetchAsync(ParameterCollection collection = null)
+        public virtual Task<ContentstackResponse> FetchAsync(ParameterCollection? collection = null)
         {
             stack.ThrowIfNotLoggedIn();
             ThrowIfUidEmpty();
@@ -81,7 +81,7 @@ public virtual Task<ContentstackResponse> FetchAsync(ParameterCollection collect
             return stack.client.InvokeAsync<FetchDeleteService, ContentstackResponse>(service);
         }
 
-        public virtual ContentstackResponse Delete(ParameterCollection collection = null)
+        public virtual ContentstackResponse Delete(ParameterCollection? collection = null)
         {
             stack.ThrowIfNotLoggedIn();
             ThrowIfUidEmpty();
@@ -90,7 +90,7 @@ public virtual ContentstackResponse Delete(ParameterCollection collection = null
             return stack.client.InvokeSync(service);
         }
 
-        public virtual Task<ContentstackResponse> DeleteAsync(ParameterCollection collection = null)
+        public virtual Task<ContentstackResponse> DeleteAsync(ParameterCollection? collection = null)
         {
             stack.ThrowIfNotLoggedIn();
             ThrowIfUidEmpty();
diff --git a/Contentstack.Management.Core/Models/ContentType.cs b/Contentstack.Management.Core/Models/ContentType.cs
@@ -39,7 +39,7 @@ public Query Query()
         /// </example>
         /// <param name="model">IContentType for updating Content Type.</param>
         /// <returns></returns>
-        public override ContentstackResponse Create(ContentModelling model, ParameterCollection collection = null)
+        public override ContentstackResponse Create(ContentModelling model, ParameterCollection? collection = null)
         {
             return base.Create(model, collection);
         }
@@ -56,7 +56,7 @@ public override ContentstackResponse Create(ContentModelling model, ParameterCol
         /// </example>
         /// <param name="model">IContentType for updating Content Type.</param>
         /// <returns></returns>
-        public override Task<ContentstackResponse> CreateAsync(ContentModelling model, ParameterCollection collection = null)
+        public override Task<ContentstackResponse> CreateAsync(ContentModelling model, ParameterCollection? collection = null)
         {
             return base.CreateAsync(model, collection);
         }
@@ -73,7 +73,7 @@ public override Task<ContentstackResponse> CreateAsync(ContentModelling model, P
         /// </example>
         /// <param name="model">IContentType for updating Content Type.</param>
         /// <returns></returns>
-        public override ContentstackResponse Update(ContentModelling model, ParameterCollection collection = null)
+        public override ContentstackResponse Update(ContentModelling model, ParameterCollection? collection = null)
         {
             return base.Update(model, collection);
         }
@@ -90,7 +90,7 @@ public override ContentstackResponse Update(ContentModelling model, ParameterCol
         /// </example>
         /// <param name="model">IContentType for updating Content Type.</param>
         /// <returns></returns>
-        public override Task<ContentstackResponse> UpdateAsync(ContentModelling model, ParameterCollection collection = null)
+        public override Task<ContentstackResponse> UpdateAsync(ContentModelling model, ParameterCollection? collection = null)
         {
             return base.UpdateAsync(model, collection);
         }
@@ -105,7 +105,7 @@ public override Task<ContentstackResponse> UpdateAsync(ContentModelling model, P
         /// </code></pre>
         /// </example>
         /// <returns>The <see cref="ContentstackResponse"/>.</returns>
-        public override ContentstackResponse Fetch(ParameterCollection collection = null)
+        public override ContentstackResponse Fetch(ParameterCollection? collection = null)
         {
             return base.Fetch(collection);
         }
@@ -120,7 +120,7 @@ public override ContentstackResponse Fetch(ParameterCollection collection = null
         /// </code></pre>
         /// </example>
         /// <returns>The Task.</returns>
-        public override Task<ContentstackResponse> FetchAsync(ParameterCollection collection = null)
+        public override Task<ContentstackResponse> FetchAsync(ParameterCollection? collection = null)
         {
             return base.FetchAsync(collection);
         }
@@ -135,7 +135,7 @@ public override Task<ContentstackResponse> FetchAsync(ParameterCollection collec
         /// </code></pre>
         /// </example>
         /// <returns>The <see cref="ContentstackResponse"/>.</returns>
-        public override ContentstackResponse Delete(ParameterCollection collection = null)
+        public override ContentstackResponse Delete(ParameterCollection? collection = null)
         {
             return base.Delete(collection);
         }
@@ -150,7 +150,7 @@ public override ContentstackResponse Delete(ParameterCollection collection = nul
         /// </code></pre>
         /// </example>
         /// <returns>The Task.</returns>
-        public override Task<ContentstackResponse> DeleteAsync(ParameterCollection collection = null)
+        public override Task<ContentstackResponse> DeleteAsync(ParameterCollection? collection = null)
         {
             return base.DeleteAsync(collection);
         }

Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@ public class BaseModel<T>`
`15`	`15`	`public string Uid { get; set; }`
`16`	`16`
`17`	`17`
`18`		`- public BaseModel(Stack stack, string fieldName, string uid = null)`
	`18`	`+ public BaseModel(Stack stack, string fieldName, string? uid = null)`
`19`	`19`	`{`
`20`	`20`	`stack.ThrowIfAPIKeyEmpty();`
`21`	`21`	`if (fieldName == null)`
`@@ -27,15 +27,15 @@ public BaseModel(Stack stack, string fieldName, string uid = null)`
`27`	`27`	`Uid = uid;`
`28`	`28`	`}`
`29`	`29`
`30`		`- public virtual ContentstackResponse Create(T model, ParameterCollection collection = null)`
	`30`	`+ public virtual ContentstackResponse Create(T model, ParameterCollection? collection = null)`
`31`	`31`	`{`
`32`	`32`	`ThrowIfUidNotEmpty();`
`33`	`33`
`34`	`34`	`var service = new CreateUpdateService<T>(stack, resourcePath, model, this.fieldName, collection: collection, stjOptions: stack.client.SerializerOptions);`
`35`	`35`	`return stack.client.InvokeSync(service);`
`36`	`36`	`}`
`37`	`37`
`38`		`- public virtual Task<ContentstackResponse> CreateAsync(T model, ParameterCollection collection = null)`
	`38`	`+ public virtual Task<ContentstackResponse> CreateAsync(T model, ParameterCollection? collection = null)`
`39`	`39`	`{`
`40`	`40`	`ThrowIfUidNotEmpty();`
`41`	`41`	`stack.ThrowIfNotLoggedIn();`
`@@ -45,15 +45,15 @@ public virtual Task<ContentstackResponse> CreateAsync(T model, ParameterCollecti`
`45`	`45`	`return stack.client.InvokeAsync<CreateUpdateService<T>, ContentstackResponse>(service);`
`46`	`46`	`}`
`47`	`47`
`48`		`- public virtual ContentstackResponse Update(T model, ParameterCollection collection = null)`
	`48`	`+ public virtual ContentstackResponse Update(T model, ParameterCollection? collection = null)`
`49`	`49`	`{`
`50`	`50`	`ThrowIfUidEmpty();`
`51`	`51`
`52`	`52`	`var service = new CreateUpdateService<T>(stack, resourcePath, model, this.fieldName, "PUT", collection: collection, stjOptions: stack.client.SerializerOptions);`
`53`	`53`	`return stack.client.InvokeSync(service);`
`54`	`54`	`}`
`55`	`55`
`56`		`- public virtual Task<ContentstackResponse> UpdateAsync(T model, ParameterCollection collection = null)`
	`56`	`+ public virtual Task<ContentstackResponse> UpdateAsync(T model, ParameterCollection? collection = null)`
`57`	`57`	`{`
`58`	`58`	`stack.ThrowIfNotLoggedIn();`
`59`	`59`	`ThrowIfUidEmpty();`
`@@ -63,7 +63,7 @@ public virtual Task<ContentstackResponse> UpdateAsync(T model, ParameterCollecti`
`63`	`63`	`return stack.client.InvokeAsync<CreateUpdateService<T>, ContentstackResponse>(service);`
`64`	`64`	`}`
`65`	`65`
`66`		`- public virtual ContentstackResponse Fetch(ParameterCollection collection = null)`
	`66`	`+ public virtual ContentstackResponse Fetch(ParameterCollection? collection = null)`
`67`	`67`	`{`
`68`	`68`	`stack.ThrowIfNotLoggedIn();`
`69`	`69`	`ThrowIfUidEmpty();`
`@@ -72,7 +72,7 @@ public virtual ContentstackResponse Fetch(ParameterCollection collection = null)`
`72`	`72`	`return stack.client.InvokeSync(service);`
`73`	`73`	`}`
`74`	`74`
`75`		`- public virtual Task<ContentstackResponse> FetchAsync(ParameterCollection collection = null)`
	`75`	`+ public virtual Task<ContentstackResponse> FetchAsync(ParameterCollection? collection = null)`
`76`	`76`	`{`
`77`	`77`	`stack.ThrowIfNotLoggedIn();`
`78`	`78`	`ThrowIfUidEmpty();`
`@@ -81,7 +81,7 @@ public virtual Task<ContentstackResponse> FetchAsync(ParameterCollection collect`
`81`	`81`	`return stack.client.InvokeAsync<FetchDeleteService, ContentstackResponse>(service);`
`82`	`82`	`}`
`83`	`83`
`84`		`- public virtual ContentstackResponse Delete(ParameterCollection collection = null)`
	`84`	`+ public virtual ContentstackResponse Delete(ParameterCollection? collection = null)`
`85`	`85`	`{`
`86`	`86`	`stack.ThrowIfNotLoggedIn();`
`87`	`87`	`ThrowIfUidEmpty();`
`@@ -90,7 +90,7 @@ public virtual ContentstackResponse Delete(ParameterCollection collection = null`
`90`	`90`	`return stack.client.InvokeSync(service);`
`91`	`91`	`}`
`92`	`92`
`93`		`- public virtual Task<ContentstackResponse> DeleteAsync(ParameterCollection collection = null)`
	`93`	`+ public virtual Task<ContentstackResponse> DeleteAsync(ParameterCollection? collection = null)`
`94`	`94`	`{`
`95`	`95`	`stack.ThrowIfNotLoggedIn();`
`96`	`96`	`ThrowIfUidEmpty();`
Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ public Query Query()`
`39`	`39`	`/// </example>`
`40`	`40`	`/// <param name="model">IContentType for updating Content Type.</param>`
`41`	`41`	`/// <returns></returns>`
`42`		`- public override ContentstackResponse Create(ContentModelling model, ParameterCollection collection = null)`
	`42`	`+ public override ContentstackResponse Create(ContentModelling model, ParameterCollection? collection = null)`
`43`	`43`	`{`
`44`	`44`	`return base.Create(model, collection);`
`45`	`45`	`}`
`@@ -56,7 +56,7 @@ public override ContentstackResponse Create(ContentModelling model, ParameterCol`
`56`	`56`	`/// </example>`
`57`	`57`	`/// <param name="model">IContentType for updating Content Type.</param>`
`58`	`58`	`/// <returns></returns>`
`59`		`- public override Task<ContentstackResponse> CreateAsync(ContentModelling model, ParameterCollection collection = null)`
	`59`	`+ public override Task<ContentstackResponse> CreateAsync(ContentModelling model, ParameterCollection? collection = null)`
`60`	`60`	`{`
`61`	`61`	`return base.CreateAsync(model, collection);`
`62`	`62`	`}`
`@@ -73,7 +73,7 @@ public override Task<ContentstackResponse> CreateAsync(ContentModelling model, P`
`73`	`73`	`/// </example>`
`74`	`74`	`/// <param name="model">IContentType for updating Content Type.</param>`
`75`	`75`	`/// <returns></returns>`
`76`		`- public override ContentstackResponse Update(ContentModelling model, ParameterCollection collection = null)`
	`76`	`+ public override ContentstackResponse Update(ContentModelling model, ParameterCollection? collection = null)`
`77`	`77`	`{`
`78`	`78`	`return base.Update(model, collection);`
`79`	`79`	`}`
`@@ -90,7 +90,7 @@ public override ContentstackResponse Update(ContentModelling model, ParameterCol`
`90`	`90`	`/// </example>`
`91`	`91`	`/// <param name="model">IContentType for updating Content Type.</param>`
`92`	`92`	`/// <returns></returns>`
`93`		`- public override Task<ContentstackResponse> UpdateAsync(ContentModelling model, ParameterCollection collection = null)`
	`93`	`+ public override Task<ContentstackResponse> UpdateAsync(ContentModelling model, ParameterCollection? collection = null)`
`94`	`94`	`{`
`95`	`95`	`return base.UpdateAsync(model, collection);`
`96`	`96`	`}`
`@@ -105,7 +105,7 @@ public override Task<ContentstackResponse> UpdateAsync(ContentModelling model, P`
`105`	`105`	`/// </code></pre>`
`106`	`106`	`/// </example>`
`107`	`107`	`/// <returns>The <see cref="ContentstackResponse"/>.</returns>`
`108`		`- public override ContentstackResponse Fetch(ParameterCollection collection = null)`
	`108`	`+ public override ContentstackResponse Fetch(ParameterCollection? collection = null)`
`109`	`109`	`{`
`110`	`110`	`return base.Fetch(collection);`
`111`	`111`	`}`
`@@ -120,7 +120,7 @@ public override ContentstackResponse Fetch(ParameterCollection collection = null`
`120`	`120`	`/// </code></pre>`
`121`	`121`	`/// </example>`
`122`	`122`	`/// <returns>The Task.</returns>`
`123`		`- public override Task<ContentstackResponse> FetchAsync(ParameterCollection collection = null)`
	`123`	`+ public override Task<ContentstackResponse> FetchAsync(ParameterCollection? collection = null)`
`124`	`124`	`{`
`125`	`125`	`return base.FetchAsync(collection);`
`126`	`126`	`}`
`@@ -135,7 +135,7 @@ public override Task<ContentstackResponse> FetchAsync(ParameterCollection collec`
`135`	`135`	`/// </code></pre>`
`136`	`136`	`/// </example>`
`137`	`137`	`/// <returns>The <see cref="ContentstackResponse"/>.</returns>`
`138`		`- public override ContentstackResponse Delete(ParameterCollection collection = null)`
	`138`	`+ public override ContentstackResponse Delete(ParameterCollection? collection = null)`
`139`	`139`	`{`
`140`	`140`	`return base.Delete(collection);`
`141`	`141`	`}`
`@@ -150,7 +150,7 @@ public override ContentstackResponse Delete(ParameterCollection collection = nul`
`150`	`150`	`/// </code></pre>`
`151`	`151`	`/// </example>`
`152`	`152`	`/// <returns>The Task.</returns>`
`153`		`- public override Task<ContentstackResponse> DeleteAsync(ParameterCollection collection = null)`
	`153`	`+ public override Task<ContentstackResponse> DeleteAsync(ParameterCollection? collection = null)`
`154`	`154`	`{`
`155`	`155`	`return base.DeleteAsync(collection);`
`156`	`156`	`}`