Fixing sonar code quality recommendations

This commit is contained in:
Andrew Song 2020-12-25 15:08:40 -06:00
parent 4f93fef661
commit 2d066ea36c
5 changed files with 7 additions and 18 deletions

View File

@ -61,20 +61,7 @@ namespace API.Controllers
{
return Ok(await _libraryRepository.GetLibrariesAsync());
}
// Do I need this method?
// [HttpGet("library/{username}")]
// public async Task<ActionResult<IEnumerable<LibraryDto>>> GetLibrariesForUser(string username)
// {
// _logger.LogDebug("Method hit");
// var user = await _userRepository.GetUserByUsernameAsync(User.GetUsername());
//
// if (user == null) return BadRequest("Could not validate user");
//
// return Ok(await _libraryRepository.GetLibrariesForUserAsync(user));
// }
[Authorize(Policy = "RequireAdminRole")]
[HttpPut("update-for")]
public async Task<ActionResult<MemberDto>> UpdateLibrary(UpdateLibraryDto updateLibraryDto)

View File

@ -30,7 +30,6 @@ namespace API.Controllers
{
// NOTE: I think we should move this into library controller because it gets added to all admins
//_logger.Log(LogLevel.Debug, "Creating a new " + createLibraryDto.Type + " library");
var user = await _userRepository.GetUserByUsernameAsync(User.GetUsername());
if (user == null) return BadRequest("Could not validate user");

View File

@ -6,7 +6,7 @@ using Microsoft.AspNetCore.Identity;
namespace API.Data
{
public class Seed
public static class Seed
{
public static async Task SeedRoles(RoleManager<AppRole> roleManager)
{

View File

@ -13,6 +13,10 @@ namespace API
{
public class Program
{
protected Program()
{
}
public static async Task Main(string[] args)
{
var host = CreateHostBuilder(args).Build();
@ -37,7 +41,7 @@ namespace API
await host.RunAsync();
}
public static IHostBuilder CreateHostBuilder(string[] args) =>
private static IHostBuilder CreateHostBuilder(string[] args) =>
Host.CreateDefaultBuilder(args)
.ConfigureWebHostDefaults(webBuilder =>
{

View File

@ -39,7 +39,6 @@ namespace API
if (env.IsDevelopment())
{
//app.UseDeveloperExceptionPage();
app.UseSwagger();
app.UseSwaggerUI(c => c.SwaggerEndpoint("/swagger/v1/swagger.json", "API v1"));
}