Adjustments and limits (#64)

Added statistics questions when creating an application

Author: jgauffin

src/Server/Coderr.Server.Api/Core/Applications/Commands/CreateApplication.cs

@@ -45,5 +45,15 @@ public CreateApplication(string name, TypeOfApplication typeOfApplication)
         /// </summary>
         [IgnoreField]
         public int UserId { get; set; }
+
+        /// <summary>
+        /// Estimated number of errors
+        /// </summary>
+        public int? NumberOfErrors { get; set; }
+
+        /// <summary>
+        /// Number of developers that work full time with this application.
+        /// </summary>
+        public decimal? NumberOfDevelopers { get; set; }
     }
 }

src/Server/Coderr.Server.Api/Core/Applications/Commands/MuteStatisticsQuestion.cs

@@ -0,0 +1,15 @@
+using System;
+using System.Collections.Generic;
+using System.Text;
+
+namespace Coderr.Server.Api.Core.Applications.Commands
+{
+    /// <summary>
+    /// Do not pester the user with the question about adding info for statistics.
+    /// </summary>
+    [Command]
+    public class MuteStatisticsQuestion 
+    {
+        public int ApplicationId { get; set; }
+    }
+}

src/Server/Coderr.Server.Api/Core/Applications/Queries/GetApplicationInfoResult.cs

@@ -39,5 +39,10 @@ public class GetApplicationInfoResult
         /// Versions that we have received error reports for.
         /// </summary>
         public string[] Versions { get; set; }
+
+        /// <summary>
+        /// Got information to be able to compare how the team is performing with other teams.
+        /// </summary>
+        public bool ShowStatsQuestion { get; set; }
     }
 }

src/Server/Coderr.Server.Api/Web/Overview/Queries/GetOverviewResult.cs

@@ -31,5 +31,10 @@ public GetOverviewResult()
         ///     Labels for the time axis (X-axis) in the chart.
         /// </summary>
         public string[] TimeAxisLabels { get; set; }
+
+        /// <summary>
+        /// Number of reports that 
+        /// </summary>
+        public int MissedReports { get; set; }
     }
 }

src/Server/Coderr.Server.App/Core/Applications/CommandHandlers/CreateApplicationHandler.cs

@@ -28,8 +28,13 @@ public async Task HandleAsync(IMessageContext context, CreateApplication command
                 ApplicationType =
                     (TypeOfApplication) Enum.Parse(typeof(TypeOfApplication), command.TypeOfApplication.ToString())
             };
-            var creator = await _userRepository.GetUserAsync(command.UserId);
 
+            if (command.NumberOfDevelopers > 0)
+            {
+                app.AddStatsBase(command.NumberOfDevelopers, command.NumberOfErrors);
+            }
+
+            var creator = await _userRepository.GetUserAsync(command.UserId);
             await _repository.CreateAsync(app);
             await _repository.CreateAsync(new ApplicationTeamMember(app.Id, creator.AccountId, creator.UserName)
             {

src/Server/Coderr.Server.App/Core/Applications/CommandHandlers/MuteStatsHandler.cs

@@ -0,0 +1,25 @@
+using System;
+using System.Threading.Tasks;
+using Coderr.Server.Api.Core.Applications.Commands;
+using Coderr.Server.Domain.Core.Applications;
+using DotNetCqs;
+
+namespace Coderr.Server.App.Core.Applications.CommandHandlers
+{
+    class MuteStatisticsQuestionHandler : IMessageHandler<MuteStatisticsQuestion>
+    {
+        private readonly IApplicationRepository _applicationRepository;
+
+        public MuteStatisticsQuestionHandler(IApplicationRepository applicationRepository)
+        {
+            _applicationRepository = applicationRepository ?? throw new ArgumentNullException(nameof(applicationRepository));
+        }
+
+        public async Task HandleAsync(IMessageContext context, MuteStatisticsQuestion message)
+        {
+            var app = await _applicationRepository.GetByIdAsync(message.ApplicationId);
+            app.MuteStatisticsQuestion = true;
+            await _applicationRepository.UpdateAsync(app);
+        }
+    }
+}

src/Server/Coderr.Server.App/Core/Applications/QueryHandlers/GetApplicationInfoHandler.cs

@@ -8,7 +8,6 @@
 using Coderr.Server.Domain.Core.Incidents;
 using Coderr.Server.Domain.Modules.ApplicationVersions;
 using DotNetCqs;
-
 using TypeOfApplication = Coderr.Server.Api.Core.Applications.TypeOfApplication;
 
 namespace Coderr.Server.App.Core.Applications.QueryHandlers
@@ -64,7 +63,8 @@ public async Task<GetApplicationInfoResult> HandleAsync(IMessageContext context,
                 Name = app.Name,
                 SharedSecret = app.SharedSecret,
                 TotalIncidentCount = totalCount,
-                Versions = versions.ToArray()
+                Versions = versions.ToArray(),
+                ShowStatsQuestion = !app.MuteStatisticsQuestion
             };
         }
     }

src/Server/Coderr.Server.App/Modules/MonthlyStats/ApplicationUsageStatisticsDto.cs

@@ -8,5 +8,7 @@ public class ApplicationUsageStatisticsDto
         public int ReOpenedCount { get; set; }
         public int ClosedCount { get; set; }
         public int IgnoredCount { get; set; }
+        public decimal? NumberOfDevelopers { get; set; }
+        public int? EstimatedNumberOfErrors { get; set; }
     }
 }

src/Server/Coderr.Server.App/Modules/MonthlyStats/CollectStatsJob.cs

@@ -67,7 +67,8 @@ from incidents
                         var item = new AppResult
                         {
                             ApplicationId = reader.GetInt32(0),
-                            Count = reader.GetInt32(1)
+                            Count = reader.GetInt32(1),
+                            
                         };
                         results.Add(item);
                     }
@@ -211,6 +212,8 @@ private async Task<bool> ReportMonth(DateTime lastMonth)
         {
             var apps = new Dictionary<int, ApplicationUsageStatisticsDto>();
 
+            await GetApps(apps);
+
             var values = await GetIncidentCounts(lastMonth);
             MergeStats(values, apps, (stat, value) => stat.IncidentCount = value);
 
@@ -249,6 +252,27 @@ private async Task<bool> ReportMonth(DateTime lastMonth)
             return true;
         }
 
+        private async Task GetApps(Dictionary<int, ApplicationUsageStatisticsDto> apps)
+        {
+            using (var cmd = _unitOfWork.CreateDbCommand())
+            {
+                cmd.CommandText = @"select Id, EstimatedNumberOfErrors, NumberOfFtes FROM Applications";
+                using (var reader = await cmd.ExecuteReaderAsync())
+                {
+                    while (await reader.ReadAsync())
+                    {
+                        var item = new ApplicationUsageStatisticsDto()
+                        {
+                            ApplicationId = reader.GetInt32(0),
+                            EstimatedNumberOfErrors= reader.GetFieldValue<int?>(1),
+                            NumberOfDevelopers = reader.GetFieldValue<decimal?>(2)
+                        };
+                        apps[item.ApplicationId] = item;
+                    }
+                }
+            }
+        }
+
         private static void MergeStats(IEnumerable<AppResult> appResults, IDictionary<int, ApplicationUsageStatisticsDto> apps, Action<ApplicationUsageStatisticsDto, int> assignMethod)
         {
             foreach (var count in appResults)

src/Server/Coderr.Server.Domain/Core/Applications/Application.cs

@@ -8,6 +8,8 @@ namespace Coderr.Server.Domain.Core.Applications
     /// </summary>
     public class Application
     {
+        private bool _muteStatisticsQuestion;
+
         /// <summary>
         ///     Initializes a new instance of the <see cref="Application" /> class.
         /// </summary>
@@ -78,6 +80,31 @@ protected Application()
         /// </summary>
         /// <remarks>The user have to manually configure the secret.</remarks>
         public string SharedSecret { get; private set; }
+
+        /// <summary>
+        /// Entered when the user created the application
+        /// </summary>
+        public int? EstimatedNumberOfErrors { get; private set; }
+
+        /// <summary>
+        /// Number of full time developers
+        /// </summary>
+        public decimal? NumberOfFtes { get; private set; }
+
+        public void AddStatsBase(decimal? fte, int? numberOfErrors)
+        {
+            EstimatedNumberOfErrors = numberOfErrors;
+            NumberOfFtes = fte;
+        }
+
+        /// <summary>
+        /// Don't ask for statistics information
+        /// </summary>
+        public bool MuteStatisticsQuestion
+        {
+            get => _muteStatisticsQuestion || NumberOfFtes > 0 || EstimatedNumberOfErrors > 0;
+            set => _muteStatisticsQuestion = value;
+        }
     }