diff --git a/Marechai/Areas/Admin/Views/InstructionSetExtensions/Edit.cshtml b/Marechai/Areas/Admin/Views/InstructionSetExtensions/Edit.cshtml deleted file mode 100644 index 17132367..00000000 --- a/Marechai/Areas/Admin/Views/InstructionSetExtensions/Edit.cshtml +++ /dev/null @@ -1,65 +0,0 @@ -@{ - /****************************************************************************** -// MARECHAI: Master repository of computing history artifacts information -// ---------------------------------------------------------------------------- -// -// Filename : Edit.cshtml -// Author(s) : Natalia Portillo -// -// --[ Description ] ---------------------------------------------------------- -// -// Admin view edit -// -// --[ License ] -------------------------------------------------------------- -// -// This program is free software: you can redistribute it and/or modify -// it under the terms of the GNU General Public License as -// published by the Free Software Foundation, either version 3 of the -// License, or (at your option) any later version. -// -// This program is distributed in the hope that it will be useful, -// but WITHOUT ANY WARRANTY; without even the implied warranty of -// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the -// GNU General Public License for more details. -// -// You should have received a copy of the GNU General Public License -// along with this program. If not, see . -// -// ---------------------------------------------------------------------------- -// Copyright © 2003-2020 Natalia Portillo -*******************************************************************************/ -} -@model Marechai.Database.Models.InstructionSetExtension - -@{ - ViewData["Title"] = "Edit"; -} -

Edit

-

Instruction set extension

-
-
-
-
-
-
- -
- - - - -
- -
-
-
- -@section Scripts { - @{ await Html.RenderPartialAsync("_ValidationScriptsPartial"); } -} \ No newline at end of file diff --git a/Marechai/Marechai.csproj b/Marechai/Marechai.csproj index b221c686..0304b93d 100644 --- a/Marechai/Marechai.csproj +++ b/Marechai/Marechai.csproj @@ -2,7 +2,7 @@ netcoreapp3.1 - 3.0.99.1227 + 3.0.99.1228 Canary Islands Computer Museum Copyright © 2003-2020 Natalia Portillo Canary Islands Computer Museum Website diff --git a/Marechai/Pages/Admin/Details/InstructionSetExtension.razor b/Marechai/Pages/Admin/Details/InstructionSetExtension.razor index 95ca663f..420417f0 100644 --- a/Marechai/Pages/Admin/Details/InstructionSetExtension.razor +++ b/Marechai/Pages/Admin/Details/InstructionSetExtension.razor @@ -31,9 +31,11 @@ } @page "/admin/instruction_set_extensions/details/{Id:int}" +@page "/admin/instruction_set_extensions/edit/{Id:int}" @inherits OwningComponentBase @inject IStringLocalizer L @attribute [Authorize(Roles = "UberAdmin, Admin")] +@inject NavigationManager NavigationManager

@L["Instruction set extension details"]


@@ -47,10 +49,24 @@
@L["Extension"] - + + + + @L["Please enter a valid extension name."] + + +
- @L["Edit"] - @L["Back to list"] + @if (!_editing) + { + + } + else + { + + + } + @L["Back to list"]
\ No newline at end of file diff --git a/Marechai/Pages/Admin/Details/InstructionSetExtension.razor.cs b/Marechai/Pages/Admin/Details/InstructionSetExtension.razor.cs index 714cf44e..09fc8ee5 100644 --- a/Marechai/Pages/Admin/Details/InstructionSetExtension.razor.cs +++ b/Marechai/Pages/Admin/Details/InstructionSetExtension.razor.cs @@ -1,11 +1,14 @@ +using System; using System.Threading.Tasks; +using Blazorise; +using Marechai.Shared; using Microsoft.AspNetCore.Components; namespace Marechai.Pages.Admin.Details { public partial class InstructionSetExtension { - bool _editable; + bool _editing; bool _loaded; Database.Models.InstructionSetExtension _model; [Parameter] @@ -23,7 +26,51 @@ namespace Marechai.Pages.Admin.Details _model = await Service.GetAsync(Id); + _editing = NavigationManager.ToBaseRelativePath(NavigationManager.Uri).ToLowerInvariant(). + StartsWith("admin/instruction_set_extensions/edit/", + StringComparison.InvariantCulture); + StateHasChanged(); } + + void OnEditClicked() + { + _editing = true; + StateHasChanged(); + } + + async void OnCancelClicked() + { + _editing = false; + _model = await Service.GetAsync(Id); + StateHasChanged(); + } + + async void OnSaveClicked() + { + if(string.IsNullOrWhiteSpace(_model.Extension) || + _model.Extension.Length > 45 || + !Service.VerifyUnique(_model.Extension)) + return; + + _editing = false; + await Service.UpdateAsync(_model); + _model = await Service.GetAsync(Id); + StateHasChanged(); + } + + void ValidateName(ValidatorEventArgs e) + { + Validators.ValidateString(e, L["Extension name cannot contain more than 45 characters."], 45); + + if(e.Status != ValidationStatus.Success) + return; + + if(Service.VerifyUnique(_model.Extension)) + return; + + e.Status = ValidationStatus.Error; + e.ErrorText = L["Extension name must be unique."]; + } } } \ No newline at end of file diff --git a/Marechai/Pages/Admin/InstructionSetExtensions.razor b/Marechai/Pages/Admin/InstructionSetExtensions.razor index bdd8fd71..8ec3a07c 100644 --- a/Marechai/Pages/Admin/InstructionSetExtensions.razor +++ b/Marechai/Pages/Admin/InstructionSetExtensions.razor @@ -64,9 +64,7 @@ @L["Details"] - - @L["Edit"] - + @L["Edit"] diff --git a/Marechai/Resources/Services/InstructionSetExtensionsService.es.resx b/Marechai/Resources/Services/InstructionSetExtensionsService.es.resx index f91aef53..9f18ef86 100644 --- a/Marechai/Resources/Services/InstructionSetExtensionsService.es.resx +++ b/Marechai/Resources/Services/InstructionSetExtensionsService.es.resx @@ -166,4 +166,20 @@ Volver a la lista Back to list + + Guardar + Save + + + Por favor introduce un nombre válido para la extensión. + Please enter a valid extension name. + + + El nombre de la extensión no puede contener más de 45 caracteres. + Extension name cannot contain more than 45 characters. + + + El nombre de la extensión debe ser único. + Extension name must be unique. + \ No newline at end of file diff --git a/Marechai/Services/InstructionSetExtensionsService.cs b/Marechai/Services/InstructionSetExtensionsService.cs index 553a9122..15710426 100644 --- a/Marechai/Services/InstructionSetExtensionsService.cs +++ b/Marechai/Services/InstructionSetExtensionsService.cs @@ -1,3 +1,4 @@ +using System; using System.Collections.Generic; using System.Linq; using System.Threading.Tasks; @@ -13,10 +14,28 @@ namespace Marechai.Services public InstructionSetExtensionsService(MarechaiContext context) => _context = context; public async Task> GetAsync() => - await _context.InstructionSetExtensions.OrderBy(e => e.Extension).ToListAsync(); + await _context.InstructionSetExtensions.OrderBy(e => e.Extension).Select(e => new InstructionSetExtension + { + Extension = e.Extension, Id = e.Id + }).ToListAsync(); public async Task GetAsync(int id) => - await _context.InstructionSetExtensions.FindAsync(id); + await _context.InstructionSetExtensions.Where(e => e.Id == id).Select(e => new InstructionSetExtension + { + Extension = e.Extension, Id = e.Id + }).FirstOrDefaultAsync(); + + public async Task UpdateAsync(InstructionSetExtension viewModel) + { + InstructionSetExtension model = await _context.InstructionSetExtensions.FindAsync(viewModel.Id); + + if(model is null) + return; + + model.Extension = viewModel.Extension; + + await _context.SaveChangesAsync(); + } public async Task DeleteAsync(int id) { @@ -29,5 +48,9 @@ namespace Marechai.Services await _context.SaveChangesAsync(); } + + public bool VerifyUnique(string extension) => + !_context.InstructionSetExtensions.Any(i => string.Equals(i.Extension, extension, + StringComparison.InvariantCultureIgnoreCase)); } } \ No newline at end of file