refactor: Use is (not) null

Author: linkdotnet

src/LinkDotNet.Blog.Infrastructure/Persistence/InMemory/Repository.cs

@@ -39,12 +39,12 @@ public ValueTask<IPagedList<TProjection>> GetAllByProjectionAsync<TProjection>(
     {
         ArgumentNullException.ThrowIfNull(selector);
         var result = entities.AsEnumerable();
-        if (filter != null)
+        if (filter is not null)
         {
             result = result.Where(filter.Compile());
         }
 
-        if (orderBy != null)
+        if (orderBy is not null)
         {
             result = descending
                 ? result.OrderByDescending(orderBy.Compile())
@@ -64,7 +64,7 @@ public ValueTask StoreAsync(TEntity entity)
         }
 
         var entry = entities.SingleOrDefault(b => b.Id == entity.Id);
-        if (entry != null)
+        if (entry is not null)
         {
             entities.Remove(entry);
         }
@@ -76,7 +76,7 @@ public ValueTask StoreAsync(TEntity entity)
     public ValueTask DeleteAsync(string id)
     {
         var blogPostToDelete = entities.SingleOrDefault(b => b.Id == id);
-        if (blogPostToDelete != null)
+        if (blogPostToDelete is not null)
         {
             entities.Remove(blogPostToDelete);
         }

src/LinkDotNet.Blog.Infrastructure/Persistence/MongoDB/Repository.cs

@@ -62,12 +62,12 @@ public async ValueTask<IPagedList<TProjection>> GetAllByProjectionAsync<TProject
     {
         var query = Collection.AsQueryable();
 
-        if (filter != null)
+        if (filter is not null)
         {
             query = query.Where(filter);
         }
 
-        if (orderBy != null)
+        if (orderBy is not null)
         {
             query = descending ? query.OrderByDescending(orderBy) : query.OrderBy(orderBy);
         }

src/LinkDotNet.Blog.Infrastructure/Persistence/RavenDb/Repository.cs

@@ -59,12 +59,12 @@ public async ValueTask<IPagedList<TProjection>> GetAllByProjectionAsync<TProject
         using var session = documentStore.OpenAsyncSession();
 
         var query = session.Query<TEntity>();
-        if (filter != null)
+        if (filter is not null)
         {
             query = query.Where(filter);
         }
 
-        if (orderBy != null)
+        if (orderBy is not null)
         {
             query = descending
                 ? query.OrderByDescending(orderBy)

src/LinkDotNet.Blog.Infrastructure/Persistence/Sql/Repository.cs

@@ -62,12 +62,12 @@ public async ValueTask<IPagedList<TProjection>> GetAllByProjectionAsync<TProject
         var blogDbContext = await dbContextFactory.CreateDbContextAsync();
         var entity = blogDbContext.Set<TEntity>().AsNoTracking().AsQueryable();
 
-        if (filter != null)
+        if (filter is not null)
         {
             entity = entity.Where(filter);
         }
 
-        if (orderBy != null)
+        if (orderBy is not null)
         {
             entity = descending
                 ? entity.OrderByDescending(orderBy)
@@ -97,7 +97,7 @@ public async ValueTask StoreAsync(TEntity entity)
     public async ValueTask DeleteAsync(string id)
     {
         var entityToDelete = await GetByIdAsync(id);
-        if (entityToDelete != null)
+        if (entityToDelete is not null)
         {
             var blogDbContext = await dbContextFactory.CreateDbContextAsync();
             blogDbContext.Remove(entityToDelete);

src/LinkDotNet.Blog.Web/Features/AboutMe/Components/Profile.razor

@@ -98,7 +98,7 @@ OnYesPressed="DeleteItem"></ConfirmDialog>
 
     private async Task HandleDrop(ProfileInformationEntry dropTarget)
     {
-        if (currentDragItem == null || dropTarget  == currentDragItem)
+        if (currentDragItem is null || dropTarget == currentDragItem)
         {
             return;
         }

src/LinkDotNet.Blog.Web/Features/AboutMe/Components/Skill/SkillTable.razor

@@ -83,7 +83,7 @@
 
     private async Task HandleDrop(ProficiencyLevel proficiencyLevel)
     {
-        if (currentDragItem == null || currentDragItem.ProficiencyLevel == proficiencyLevel)
+        if (currentDragItem is null || currentDragItem.ProficiencyLevel == proficiencyLevel)
         {
             return;
         }

src/LinkDotNet.Blog.Web/Features/Admin/BlogPostEditor/Components/CreateNewBlogPost.razor

@@ -65,7 +65,7 @@
 	            <label for="tags">Tags</label>
 	            <InputText class="form-control" id="tags" @bind-Value="model.Tags"/>
             </div>
-            @if (BlogPost != null && !IsScheduled)
+            @if (BlogPost is not null && !IsScheduled)
             {
                 <div class="form-check">
                     <InputCheckbox class="form-check-input" id="updatedate" @bind-Value="model.ShouldUpdateDate" />
@@ -117,7 +117,7 @@
 
     protected override void OnParametersSet()
     {
-        if (BlogPost == null)
+        if (BlogPost is null)
         {
             return;
         }

src/LinkDotNet.Blog.Web/Features/Admin/BlogPostEditor/UpdateBlogPostPage.razor

@@ -6,7 +6,7 @@
 @inject IRepository<BlogPost> BlogPostRepository
 @inject IToastService ToastService
 
-@if (blogPostFromDb != null)
+@if (blogPostFromDb is not null)
 {
     <CreateNewBlogPost
         Title="Update BlogPost"
@@ -38,4 +38,4 @@ else
         await BlogPostRepository.StoreAsync(blogPostFromDb);
         ToastService.ShowInfo($"Updated BlogPost {blogPost.Title}");
     }
-}
+}

src/LinkDotNet.Blog.Web/Features/Admin/Dashboard/Components/VisitCountPerPage.razor

@@ -16,7 +16,7 @@
     <div class="card-body">
         <div class="row">
             <DateRangeSelector FilterChanged="RefreshVisitCount"></DateRangeSelector>
-            @if (visitData != null)
+            @if (visitData is not null)
             {
                 <p id="total-clicks">@visitData.Sum(c => c.ClickCount) clicks in total</p>
             }
@@ -31,7 +31,7 @@
                     <th>Clicks</th>
                     <th>Likes</th>
                 </tr>
-                @if (visitData != null)
+                @if (visitData is not null)
                 {
                     @foreach (var date in visitData)
                     {

src/LinkDotNet.Blog.Web/Features/Admin/Sitemap/SitemapPage.razor

@@ -14,7 +14,7 @@
     {
         <Loading></Loading>
     }
-    @if (sitemapUrlSet != null)
+    @if (sitemapUrlSet is not null)
     {
         <table class="table table-striped table-hover h-50">
             <thead>