Merge pull request 'feat: Allow full survey editing & restrict editing to unpublished surveys' (#2) from Feat/HttpPut into main

Reviewed-on: #2

survey-beta/Controllers/AnalyticsController.cs

@@ -1,8 +1,6 @@
-using Microsoft.AspNetCore.Mvc;
-using survey_beta.Models;
-using System;
-using System.Threading.Tasks;
-using survey_beta.DTOs.Update;
+using Microsoft.AspNetCore.Authorization;
+using Microsoft.AspNetCore.Http;
+using Microsoft.AspNetCore.Mvc;
 
 namespace survey_beta.Controllers
 {
@@ -10,37 +8,23 @@ namespace survey_beta.Controllers
     [ApiController]
     public class AnalyticsController : ControllerBase
     {
-        private readonly AnalyticsServices _analyticsService;
+        private readonly AnalyticsServices _analyticsServices;
 
-        public AnalyticsController(AnalyticsServices analyticsService)
+        public AnalyticsController(AnalyticsServices analyticsServices)
         {
-            _analyticsService = analyticsService;
+            _analyticsServices = analyticsServices;
         }
-
-        [HttpGet("{surveyId}/statistics")]
-        public async Task<ActionResult<SurveyStats>> GetSurveyStatistics(string surveyId)
+        [HttpGet("survey/{surveyId}")]
+        public IActionResult GetSurveyAnalytics(string surveyId)
         {
-            if (string.IsNullOrWhiteSpace(surveyId))
-                return BadRequest(new { error = "Survey ID is required." });
-
-            var statistics = await Task.Run(() => _analyticsService.GetAggregatedSurveyResponses(surveyId));
-
-            if (statistics == null)
-                return NotFound(new { error = "No data available for the given survey." });
-
-            return Ok(statistics);
+            var analytics = _analyticsServices.GetAggregatedSurveyResponses(surveyId);
+            if (analytics == null) return NotFound("Survey analytics not found.");
+            return Ok(analytics);
         }
-        [HttpGet("{surveyId}/export")]
-        public async Task<IActionResult> ExportSurveyStatistics(string surveyId)
+        [HttpGet("export/{surveyId}")]
+        public async Task<IActionResult> ExportSurveyData(string surveyId)
         {
-            if (string.IsNullOrWhiteSpace(surveyId))
-                return BadRequest(new { error = "Survey ID is required." });
-            var fileResult = await _analyticsService.ExportResponsesToCsv(surveyId);
-
-            if (fileResult == null)
-                return NotFound(new { error = "No data available to export." });
-
-            return fileResult;
+            return await _analyticsServices.ExportResponsesToCsv(surveyId);
         }
     }
-}
+}

survey-beta/Controllers/ResponsesController.cs

@@ -23,7 +23,7 @@ namespace survey_beta.Controllers
             try
             {
                 await _responsesService.AddResponseAsync(request);
-                return Ok(new { message = "Your response has been submitted successfully." });
+                return Ok(new { message = "Response added successfully." });
             }
             catch (Exception ex)
             {

survey-beta/Controllers/UsersController.cs

@@ -38,7 +38,7 @@ public class UserController : ControllerBase
         }
         catch (Exception ex)
         {
-            return Unauthorized(new { error = "Invalid credentials. Please check your username and password." });
+            return Unauthorized(ex.Message);
         }
     }
     //[Authorize]
@@ -81,7 +81,7 @@ public class UserController : ControllerBase
     {
         var users = await _usersServices.GetUserByIdAsync(id);
         if (users == null)
-            return NotFound(new { message = "The specified user does not exist." });
+            return NotFound();
         var result = await _usersServices.DeleteUsersAsync(id);
         return Ok(result);
     }

survey-beta/Program.cs

@@ -87,4 +87,5 @@ app.UseHttpsRedirection();
 app.UseAuthentication();
 app.UseAuthorization();
 app.MapControllers();
+
 app.Run();

survey-beta/Services/AnalyticsServices.cs

@@ -1,5 +1,4 @@
 using AutoMapper;
-using Microsoft.AspNetCore.Http.HttpResults;
 using Microsoft.AspNetCore.Mvc;
 using Microsoft.EntityFrameworkCore;
 using survey_beta.DataBaseContext;
@@ -34,6 +33,7 @@ public class AnalyticsServices
 
         SaveAnalytics(surveyId, analyticsData);
     }
+
     private SurveyStats CalculateAnalytics(List<Response> responses)
     {
         var answerFrequency = new Dictionary<string, int>();
@@ -89,6 +89,7 @@ public class AnalyticsServices
 
         return CalculateAnalytics(responses);
     }
+
     public async Task<IActionResult> ExportResponsesToCsv(string surveyId)
     {
         var responses = await _context.Responses
@@ -99,43 +100,21 @@ public class AnalyticsServices
             .ThenInclude(a => a.Choice)
             .ToListAsync();
 
-        if (!responses.Any())
-        {
-            return null;
-        }
         var csvLines = new List<string>
     {
-        "Survey Analysis",
-        $"Total Responses: {responses.Count}",
-        "",
-        "ResponseId,Question,ChoiceText,ChoiceCount,ChoicePercentage"
+        "ResponseId,Question,ChoiceText"
     };
-        var choiceStats = responses
-            .SelectMany(r => r.Answers)
-            .Where(a => a.Choice != null)
-            .GroupBy(a => new { a.Question.Content, a.Choice.Letter })
-            .Select(g => new
-            {
-                Question = g.Key.Content,
-                Choice = g.Key.Letter,
-                Count = g.Count(),
-                Percentage = (double)g.Count() / responses.Count * 100
-            })
-            .OrderByDescending(s => s.Count)
-            .ToList();
+
         foreach (var response in responses)
         {
             foreach (var answer in response.Answers)
             {
                 var questionText = answer.Question.Content;
-                var choiceText = answer.Choice?.Letter ?? "N/A";
-
-                var choiceStat = choiceStats.FirstOrDefault(s => s.Question == questionText && s.Choice == choiceText);
-
-                csvLines.Add($"{response.Id},{questionText},{choiceText},{choiceStat?.Count ?? 0},{choiceStat?.Percentage.ToString("0.00") ?? "0"}%");
+                csvLines.Add($"{response.Id},{questionText}");
             }
         }
-        var fileName = $"Survey_{surveyId}_Analysis.csv";
+
+        var fileName = "Survey_Responses.csv";
         var fileBytes = Encoding.UTF8.GetBytes(string.Join(Environment.NewLine, csvLines));
 
         return new FileContentResult(fileBytes, "text/csv")

survey-beta/Services/ResponsesServices.cs

@@ -29,11 +29,15 @@ public class ResponsesService
             CreatedAt = DateTime.UtcNow,
         };
 
-        response.Answers = responseDto.Answers.Select(answer => new Answer
+        foreach (var answer in responseDto.Answers)
         {
-            QuestionId = answer.QuestionId,
-            ResponseId = response.Id
-        }).ToList();
+            var answerEntity = new Answer
+            {
+                QuestionId = answer.QuestionId,
+                ResponseId = response.Id
+            };
+            response.Answers.Add(answerEntity);
+        }
 
         await _context.Responses.AddAsync(response);
         await _context.SaveChangesAsync();

survey-beta/Services/SurveyServices.cs

@@ -121,17 +121,17 @@ public class SurveyService
         await _context.SaveChangesAsync();
         return true;
     }
-    public async Task<bool> DeleteSurveyAsync(string id, CancellationToken cancellationToken)
+    public async Task<bool> DeleteSurveyAsync(string id)
     {
-        var survey = await _context.Surveys
+        var survey = await _context.Surveys.AsNoTracking()
             .Include(s => s.Questions)
-                .ThenInclude(q => q.Choices)
-            .FirstOrDefaultAsync(s => s.Id == id, cancellationToken);
+            .ThenInclude(q => q.Choices)
+            .FirstOrDefaultAsync(s => s.Id == id);
 
         if (survey == null) return false;
 
         _context.Surveys.Remove(survey);
-        await _context.SaveChangesAsync(cancellationToken);
+        await _context.SaveChangesAsync();
         return true;
     }
 }

survey-beta/Services/UsersServices.cs

@@ -36,11 +36,9 @@ public class UsersServices
         var result = await _userManager.CreateAsync(user, createUserDto.Password);
         if (!result.Succeeded)
         {
-            if (!result.Succeeded)
-            {
-                throw new Exception("Failed to create user. Please check your inputs and try again.");
-            }
+            throw new Exception("Failed to create user: " + string.Join(", ", result.Errors.Select(e => e.Description)));
         }
+
         var userDto = _mapper.Map<UserDto>(user);
         userDto.Token = GenerateJwtToken(user);
 
@@ -87,17 +85,13 @@ public class UsersServices
     }
     private string GenerateJwtToken(User user)
     {
-        var userRoles = _userManager.GetRolesAsync(user).Result;
-
-        var claims = new List<Claim>
-    {
-        new Claim(JwtRegisteredClaimNames.Sub, user.UserName),
-        new Claim(JwtRegisteredClaimNames.Jti, Guid.NewGuid().ToString()),
-        new Claim(ClaimTypes.Email, user.Email),
-        new Claim(ClaimTypes.Name, user.Fullname)
-    };
-
-        claims.AddRange(userRoles.Select(role => new Claim(ClaimTypes.Role, role)));
+        var claims = new[]
+        {
+            new Claim(JwtRegisteredClaimNames.Sub, user.UserName),
+            new Claim(JwtRegisteredClaimNames.Jti, Guid.NewGuid().ToString()),
+            new Claim(ClaimTypes.Email, user.Email),
+            new Claim(ClaimTypes.Name, user.Fullname)
+        };
 
         var key = new SymmetricSecurityKey(Encoding.UTF8.GetBytes(_configuration["Jwt:Key"]));
         var creds = new SigningCredentials(key, SecurityAlgorithms.HmacSha256);
@@ -106,7 +100,7 @@ public class UsersServices
             issuer: _configuration["Jwt:Issuer"],
             audience: _configuration["Jwt:Audience"],
             claims: claims,
-            expires: DateTime.UtcNow.AddMinutes(30),
+            expires: DateTime.Now.AddMinutes(30),
             signingCredentials: creds);
 
         return new JwtSecurityTokenHandler().WriteToken(token);

survey-beta/bin/Debug/net8.0/survey-beta.deps.json

@@ -9,7 +9,6 @@
       "survey-beta/1.0.0": {
         "dependencies": {
           "AutoMapper.Extensions.Microsoft.DependencyInjection": "12.0.0",
-          "CsvHelper": "33.0.1",
           "EntityFramework": "6.5.1",
           "Microsoft.AspNet.Identity.EntityFramework": "2.2.4",
           "Microsoft.AspNetCore.Authentication.JwtBearer": "8.0.0",
@@ -49,14 +48,6 @@
           }
         }
       },
-      "CsvHelper/33.0.1": {
-        "runtime": {
-          "lib/net8.0/CsvHelper.dll": {
-            "assemblyVersion": "33.0.0.0",
-            "fileVersion": "33.0.1.24"
-          }
-        }
-      },
       "EntityFramework/6.5.1": {
         "dependencies": {
           "Microsoft.CSharp": "4.7.0",
@@ -1134,13 +1125,6 @@
       "path": "automapper.extensions.microsoft.dependencyinjection/12.0.0",
       "hashPath": "automapper.extensions.microsoft.dependencyinjection.12.0.0.nupkg.sha512"
     },
-    "CsvHelper/33.0.1": {
-      "type": "package",
-      "serviceable": true,
-      "sha512": "sha512-fev4lynklAU2A9GVMLtwarkwaanjSYB4wUqO2nOJX5hnzObORzUqVLe+bDYCUyIIRQM4o5Bsq3CcyJR89iMmEQ==",
-      "path": "csvhelper/33.0.1",
-      "hashPath": "csvhelper.33.0.1.nupkg.sha512"
-    },
     "EntityFramework/6.5.1": {
       "type": "package",
       "serviceable": true,

survey-beta/obj/Debug/net8.0/ApiEndpoints.json

@@ -1,8 +1,8 @@
 [
   {
     "ContainingType": "survey_beta.Controllers.AnalyticsController",
-    "Method": "ExportSurveyStatistics",
-    "RelativePath": "api/Analytics/{surveyId}/export",
+    "Method": "ExportSurveyData",
+    "RelativePath": "api/Analytics/export/{surveyId}",
     "HttpMethod": "GET",
     "IsController": true,
     "Order": 0,
@@ -17,8 +17,8 @@
   },
   {
     "ContainingType": "survey_beta.Controllers.AnalyticsController",
-    "Method": "GetSurveyStatistics",
-    "RelativePath": "api/Analytics/{surveyId}/statistics",
+    "Method": "GetSurveyAnalytics",
+    "RelativePath": "api/Analytics/survey/{surveyId}",
     "HttpMethod": "GET",
     "IsController": true,
     "Order": 0,
@@ -29,17 +29,7 @@
         "IsRequired": true
       }
     ],
-    "ReturnTypes": [
-      {
-        "Type": "survey_beta.DTOs.Update.SurveyStats",
-        "MediaTypes": [
-          "text/plain",
-          "application/json",
-          "text/json"
-        ],
-        "StatusCode": 200
-      }
-    ]
+    "ReturnTypes": []
   },
   {
     "ContainingType": "survey_beta.Controllers.ResponseController",

survey-beta/obj/Debug/net8.0/survey-beta.AssemblyInfo.cs

@@ -14,7 +14,7 @@ using System.Reflection;
 [assembly: System.Reflection.AssemblyCompanyAttribute("survey-beta")]
 [assembly: System.Reflection.AssemblyConfigurationAttribute("Debug")]
 [assembly: System.Reflection.AssemblyFileVersionAttribute("1.0.0.0")]
-[assembly: System.Reflection.AssemblyInformationalVersionAttribute("1.0.0+5c735417f244b7740bc440f2ef03d57b0374188b")]
+[assembly: System.Reflection.AssemblyInformationalVersionAttribute("1.0.0+b9036d8b7a60afe583ac07dee1c8d53204b07278")]
 [assembly: System.Reflection.AssemblyProductAttribute("survey-beta")]
 [assembly: System.Reflection.AssemblyTitleAttribute("survey-beta")]
 [assembly: System.Reflection.AssemblyVersionAttribute("1.0.0.0")]

survey-beta/obj/Debug/net8.0/survey-beta.AssemblyInfoInputs.cache

@@ -1 +1 @@
-7225f480dc7507f33e7c1dff9d1d2efef5e1b397dfeeaeb8ca8ea4374e21a387
+881624211deead8316978ca8e0f7ecf013b98335c33420e117fca69e5d45c51f

survey-beta/obj/Debug/net8.0/survey-beta.csproj.CoreCompileInputs.cache

@@ -1 +1 @@
-3879eb678b07b384c518e4050cdef22466b66afa8fee5cacbe2c485af19a45f3
+bf4cfa75951d95a308acd245137187f2446062728bd32c1ef69e8cdd119ba91a

survey-beta/obj/Debug/net8.0/survey-beta.csproj.FileListAbsolute.txt

@@ -334,4 +334,3 @@ C:\Users\alioa\Source\Repos\survey-beta2\survey-beta\obj\Debug\net8.0\refint\sur
 C:\Users\alioa\Source\Repos\survey-beta2\survey-beta\obj\Debug\net8.0\survey-beta.pdb
 C:\Users\alioa\Source\Repos\survey-beta2\survey-beta\obj\Debug\net8.0\survey-beta.genruntimeconfig.cache
 C:\Users\alioa\Source\Repos\survey-beta2\survey-beta\obj\Debug\net8.0\ref\survey-beta.dll
-C:\Users\alioa\Source\Repos\survey-beta2\survey-beta\bin\Debug\net8.0\CsvHelper.dll

survey-beta/obj/project.assets.json

@@ -31,19 +31,6 @@
           "lib/netstandard2.1/AutoMapper.Extensions.Microsoft.DependencyInjection.dll": {}
         }
       },
-      "CsvHelper/33.0.1": {
-        "type": "package",
-        "compile": {
-          "lib/net8.0/CsvHelper.dll": {
-            "related": ".xml"
-          }
-        },
-        "runtime": {
-          "lib/net8.0/CsvHelper.dll": {
-            "related": ".xml"
-          }
-        }
-      },
       "EntityFramework/6.5.1": {
         "type": "package",
         "dependencies": {
@@ -1810,34 +1797,6 @@
         "lib/netstandard2.1/AutoMapper.Extensions.Microsoft.DependencyInjection.dll"
       ]
     },
-    "CsvHelper/33.0.1": {
-      "sha512": "fev4lynklAU2A9GVMLtwarkwaanjSYB4wUqO2nOJX5hnzObORzUqVLe+bDYCUyIIRQM4o5Bsq3CcyJR89iMmEQ==",
-      "type": "package",
-      "path": "csvhelper/33.0.1",
-      "files": [
-        ".nupkg.metadata",
-        ".signature.p7s",
-        "Icon.png",
-        "csvhelper.33.0.1.nupkg.sha512",
-        "csvhelper.nuspec",
-        "lib/net462/CsvHelper.dll",
-        "lib/net462/CsvHelper.xml",
-        "lib/net47/CsvHelper.dll",
-        "lib/net47/CsvHelper.xml",
-        "lib/net48/CsvHelper.dll",
-        "lib/net48/CsvHelper.xml",
-        "lib/net6.0/CsvHelper.dll",
-        "lib/net6.0/CsvHelper.xml",
-        "lib/net7.0/CsvHelper.dll",
-        "lib/net7.0/CsvHelper.xml",
-        "lib/net8.0/CsvHelper.dll",
-        "lib/net8.0/CsvHelper.xml",
-        "lib/netstandard2.0/CsvHelper.dll",
-        "lib/netstandard2.0/CsvHelper.xml",
-        "lib/netstandard2.1/CsvHelper.dll",
-        "lib/netstandard2.1/CsvHelper.xml"
-      ]
-    },
     "EntityFramework/6.5.1": {
       "sha512": "sQRP2lWg1i3aAGWqdliAM8zrGx7LHMUk+9/MoxUjwfTZYGMXvZ2JYZTlyTm1PqDxvn3c9E3U76TWDON7Y5+CVA==",
       "type": "package",
@@ -5200,7 +5159,6 @@
   "projectFileDependencyGroups": {
     "net8.0": [
       "AutoMapper.Extensions.Microsoft.DependencyInjection >= 12.0.0",
-      "CsvHelper >= 33.0.1",
       "EntityFramework >= 6.5.1",
       "Microsoft.AspNet.Identity.EntityFramework >= 2.2.4",
       "Microsoft.AspNetCore.Authentication.JwtBearer >= 8.0.0",
@@ -5263,10 +5221,6 @@
             "target": "Package",
             "version": "[12.0.0, )"
           },
-          "CsvHelper": {
-            "target": "Package",
-            "version": "[33.0.1, )"
-          },
           "EntityFramework": {
             "target": "Package",
             "version": "[6.5.1, )"

survey-beta/obj/project.nuget.cache

@@ -1,12 +1,11 @@
 {
   "version": 2,
-  "dgSpecHash": "H2wNZ0h/QSI=",
+  "dgSpecHash": "Q9FP2JQ+UXs=",
   "success": true,
   "projectFilePath": "C:\\Users\\alioa\\Source\\Repos\\survey-beta2\\survey-beta\\survey-beta.csproj",
   "expectedPackageFiles": [
     "C:\\Users\\alioa\\.nuget\\packages\\automapper\\12.0.0\\automapper.12.0.0.nupkg.sha512",
     "C:\\Users\\alioa\\.nuget\\packages\\automapper.extensions.microsoft.dependencyinjection\\12.0.0\\automapper.extensions.microsoft.dependencyinjection.12.0.0.nupkg.sha512",
-    "C:\\Users\\alioa\\.nuget\\packages\\csvhelper\\33.0.1\\csvhelper.33.0.1.nupkg.sha512",
     "C:\\Users\\alioa\\.nuget\\packages\\entityframework\\6.5.1\\entityframework.6.5.1.nupkg.sha512",
     "C:\\Users\\alioa\\.nuget\\packages\\humanizer.core\\2.14.1\\humanizer.core.2.14.1.nupkg.sha512",
     "C:\\Users\\alioa\\.nuget\\packages\\microsoft.aspnet.identity.core\\2.2.4\\microsoft.aspnet.identity.core.2.2.4.nupkg.sha512",

survey-beta/obj/survey-beta.csproj.nuget.dgspec.json

@@ -50,10 +50,6 @@
               "target": "Package",
               "version": "[12.0.0, )"
             },
-            "CsvHelper": {
-              "target": "Package",
-              "version": "[33.0.1, )"
-            },
             "EntityFramework": {
               "target": "Package",
               "version": "[6.5.1, )"

survey-beta/survey-beta.csproj

@@ -9,7 +9,6 @@
 
   <ItemGroup>
     <PackageReference Include="AutoMapper.Extensions.Microsoft.DependencyInjection" Version="12.0.0" />
-    <PackageReference Include="CsvHelper" Version="33.0.1" />
     <PackageReference Include="EntityFramework" Version="6.5.1" />
     <PackageReference Include="Microsoft.AspNet.Identity.EntityFramework" Version="2.2.4" />
     <PackageReference Include="Microsoft.AspNetCore.Authentication.JwtBearer" Version="8.0.0" />
@@ -26,4 +25,9 @@
     <PackageReference Include="System.IdentityModel.Tokens.Jwt" Version="8.0.0" />
   </ItemGroup>
 
+  <ItemGroup>
+    <Folder Include="Controllers\" />
+    <Folder Include="Services\" />
+  </ItemGroup>
+
 </Project>