Incidents can now be deleted

Author: jgauffin

src/Server/Coderr.Server.Api/Core/Incidents/Commands/DeleteIncident.cs

@@ -0,0 +1,31 @@
+using System;
+
+namespace Coderr.Server.Api.Core.Incidents.Commands
+{
+    /// <summary>
+    ///     Delete incidents (old, created in dev environment etc)
+    /// </summary>
+    [Message]
+    public class DeleteIncident
+    {
+        /// <summary>
+        ///     Creates a new instance of <see cref="DeleteIncident" />.
+        /// </summary>
+        /// <param name="incidentId">incident to delete</param>
+        /// <param name="areYouSure">should be "yes"</param>
+        public DeleteIncident(int incidentId, string areYouSure)
+        {
+            if (incidentId <= 0) throw new ArgumentOutOfRangeException(nameof(incidentId));
+            if (areYouSure != "yes")
+                throw new ArgumentOutOfRangeException(nameof(areYouSure), areYouSure, "Must be 'yes'.");
+            IncidentId = incidentId;
+            AreYouSure = areYouSure;
+        }
+
+        public int IncidentId { get; private set; }
+        public int? UserId { get; set; }
+
+        public DateTime? DeletedAtUtc { get; set; }
+        public string AreYouSure { get; private set; }
+    }
+}

src/Server/Coderr.Server.Api/Core/Incidents/Events/IncidentDeleted.cs

@@ -0,0 +1,30 @@
+using System;
+using System.Collections.Generic;
+using System.Text;
+
+namespace Coderr.Server.Api.Core.Incidents.Events
+{
+    [Event]
+    public class IncidentDeleted
+    {
+        public IncidentDeleted(int incidentId, int userId, DateTime deletedAtUtc)
+        {
+            if (incidentId <= 0) throw new ArgumentOutOfRangeException(nameof(incidentId));
+            if (userId <= 0) throw new ArgumentOutOfRangeException(nameof(userId));
+
+            IncidentId = incidentId;
+            DeletedById = userId;
+            DeletedAtUtc = deletedAtUtc;
+        }
+
+        protected IncidentDeleted()
+        {
+
+        }
+
+        public int IncidentId { get;private  set; }
+
+        public int DeletedById { get; set; }
+        public DateTime DeletedAtUtc { get; private set; }
+    }
+}

src/Server/Coderr.Server.App/Core/Incidents/Commands/DeleteIncidentHandler.cs

@@ -0,0 +1,33 @@
+using System;
+using System.Threading.Tasks;
+using Coderr.Server.Abstractions.Security;
+using Coderr.Server.Api.Core.Incidents.Commands;
+using Coderr.Server.Api.Core.Incidents.Events;
+using Coderr.Server.Domain.Core.Incidents;
+using DotNetCqs;
+
+namespace Coderr.Server.App.Core.Incidents.Commands
+{
+    /// <summary>
+    ///     Handler for <see cref="DeleteIncident" />
+    /// </summary>
+    public class DeleteIncidentHandler : IMessageHandler<DeleteIncident>
+    {
+        private readonly IIncidentRepository _repository;
+
+        public DeleteIncidentHandler(IIncidentRepository repository)
+        {
+            _repository = repository;
+        }
+
+        public async Task HandleAsync(IMessageContext context, DeleteIncident message)
+        {
+            var incident = await _repository.GetAsync(message.IncidentId);
+            await _repository.Delete(message.IncidentId);
+
+            var evt = new IncidentDeleted(message.IncidentId, message.UserId ?? context.Principal.GetAccountId(),
+                message.DeletedAtUtc ?? DateTime.UtcNow);
+            await context.SendAsync(evt);
+        }
+    }
+}

src/Server/Coderr.Server.App/Modules/History/Events/IncidentAssignedHandler.cs

@@ -28,4 +28,4 @@ public async Task HandleAsync(IMessageContext context, IncidentAssigned message)
             await _repository.CreateAsync(entry);
         }
     }
-}
+}git

src/Server/Coderr.Server.Domain/Core/Incidents/IIncidentRepository.cs

@@ -9,6 +9,13 @@ namespace Coderr.Server.Domain.Core.Incidents
     /// </summary>
     public interface IIncidentRepository
     {
+        /// <summary>
+        /// Delete an incident
+        /// </summary>
+        /// <param name="incidentId">id</param>
+        /// <returns></returns>
+        Task Delete(int incidentId);
+
         /// <summary>
         ///     Get incident
         /// </summary>

src/Server/Coderr.Server.SqlServer/Core/Incidents/IncidentRepository.cs

@@ -24,7 +24,7 @@ public IncidentRepository(IAdoNetUnitOfWork uow)
 
         public async Task UpdateAsync(Incident incident)
         {
-            using (var cmd = (DbCommand) _uow.CreateCommand())
+            using (var cmd = (DbCommand)_uow.CreateCommand())
             {
                 cmd.CommandText =
                     @"UPDATE Incidents SET 
@@ -59,12 +59,12 @@ public async Task UpdateAsync(Incident incident)
 
         public async Task<int> GetTotalCountForAppInfoAsync(int applicationId)
         {
-            using (var cmd = (DbCommand) _uow.CreateCommand())
+            using (var cmd = (DbCommand)_uow.CreateCommand())
             {
                 cmd.CommandText =
                     @"SELECT CAST(count(*) as int) FROM Incidents WHERE ApplicationId = @ApplicationId";
                 cmd.AddParameter("ApplicationId", applicationId);
-                var result = (int) await cmd.ExecuteScalarAsync();
+                var result = (int)await cmd.ExecuteScalarAsync();
                 return result;
             }
         }
@@ -84,9 +84,20 @@ public Task<IList<Incident>> GetManyAsync(IEnumerable<int> incidentIds)
             }
         }
 
+        public async Task Delete(int incidentId)
+        {
+            using (var cmd = (DbCommand)_uow.CreateCommand())
+            {
+                cmd.CommandText =
+                    @"DELETE FROM Incidents WHERE Id = @id";
+                cmd.AddParameter("Id", incidentId);
+                await cmd.ExecuteNonQueryAsync();
+            }
+        }
+
         public Task<Incident> GetAsync(int id)
         {
-            using (var cmd = (DbCommand) _uow.CreateCommand())
+            using (var cmd = (DbCommand)_uow.CreateCommand())
             {
                 cmd.CommandText =
                     "SELECT TOP 1 * FROM Incidents WHERE Id = @id";
@@ -107,7 +118,7 @@ public Incident Find(int id)
                 return cmd.FirstOrDefault(new IncidentMapper());
             }
         }
-        
+
         public Incident Get(int id)
         {
             using (var cmd = _uow.CreateCommand())

src/Server/Coderr.Server.Web/ClientApp/components/analyze/incidents/incident.ts

@@ -27,6 +27,7 @@ export default class AnalyzeIncidentComponent extends Vue {
     currentCollectionName: string = '';
 
     created() {
+        this.incidentId = parseInt(this.$route.params.incidentId, 10);
     }
 
     mounted() {

src/Server/Coderr.Server.Web/ClientApp/components/discover/incidents/search.ts

@@ -320,13 +320,13 @@ export default class IncidentSearchComponent extends Vue {
         setTimeout(() => { this.search() }, 1000);
     }
 
-    closeSelectedIncidents() {
+    deleteSelectedIncidents() {
         const elems = document.querySelectorAll('#searchTable tbody input[type="checkbox"]:checked');
         for (let i = 0; i < elems.length; i++) {
             const elem = <HTMLInputElement>elems[i];
-            this.incidentService$.close(parseInt(elem.value), "Mass close from search.");
+            this.incidentService$.delete(parseInt(elem.value), "yes");
         }
-        AppRoot.notify('All selected incidents have closed.');
+        AppRoot.notify('All selected incidents have deleted.');
         setTimeout(() => { this.search() }, 1000);
     }
 

src/Server/Coderr.Server.Web/ClientApp/components/discover/incidents/search.vue.html

@@ -125,7 +125,7 @@
                 </div>
                 <div v-else>
                     <button class="btn btn-blue" v-on:click.prevent="assignAllToMe">Assign selected incidents to me</button>
-                    <button class="btn btn-light" v-on:click.prevent="closeSelectedIncidents">Close selected incidents</button>
+                    <button class="btn btn-light" v-on:click.prevent="deleteSelectedIncidents">Delete selected incidents</button>
                 </div>
             </div>
         </div>

src/Server/Coderr.Server.Web/ClientApp/dto/Core/Incidents.ts

@@ -198,6 +198,11 @@ export class CloseIncident
     public UserId: number;
     public ApplicationVersion: string;
 }
+export class DeleteIncident {
+    public static TYPE_NAME: string = 'DeleteIncident';
+    public AreYouSure: string;
+    public IncidentId: number;
+}
 export class IgnoreIncident
 {
     public static TYPE_NAME: string = 'IgnoreIncident';