Index: KernelRecordsMVC.Application/ViewModels/CreateTrackViewModel.cs =================================================================== --- KernelRecordsMVC.Application/ViewModels/CreateTrackViewModel.cs (revision d89d25b3a7f785676877cf198c9cc706308ab4a3) +++ KernelRecordsMVC.Application/ViewModels/CreateTrackViewModel.cs (revision 595d2e58cf322a776ba1d3cedcb86e9bb2ee8093) @@ -5,15 +5,17 @@ public class CreateTrackViewModel { - [Required] + [Required(ErrorMessage = "Track name is required.")] [Display(Name = "Track Name")] public string SongName { get; set; } = string.Empty; - [Required] + [Required(ErrorMessage = "Track duration is required.")] + [RegularExpression( + @"^\d{1,2}:[0-5]\d$", + ErrorMessage = "Track duration must be in MM:SS format, for example 03:42.")] [Display(Name = "Duration")] public string SongDuration { get; set; } = string.Empty; - [Display(Name = "Track Artists")] public List ArtistIds { get; set; } = new(); } Index: KernelRecordsMVC.Application/obj/Debug/net8.0/KernelRecordsMVC.Application.AssemblyInfo.cs =================================================================== --- KernelRecordsMVC.Application/obj/Debug/net8.0/KernelRecordsMVC.Application.AssemblyInfo.cs (revision d89d25b3a7f785676877cf198c9cc706308ab4a3) +++ KernelRecordsMVC.Application/obj/Debug/net8.0/KernelRecordsMVC.Application.AssemblyInfo.cs (revision 595d2e58cf322a776ba1d3cedcb86e9bb2ee8093) @@ -14,5 +14,5 @@ [assembly: System.Reflection.AssemblyConfigurationAttribute("Debug")] [assembly: System.Reflection.AssemblyFileVersionAttribute("1.0.0.0")] -[assembly: System.Reflection.AssemblyInformationalVersionAttribute("1.0.0+d2eccb3e3d4db7d262fc57179b8ce6f7fb060d63")] +[assembly: System.Reflection.AssemblyInformationalVersionAttribute("1.0.0+d89d25b3a7f785676877cf198c9cc706308ab4a3")] [assembly: System.Reflection.AssemblyProductAttribute("KernelRecordsMVC.Application")] [assembly: System.Reflection.AssemblyTitleAttribute("KernelRecordsMVC.Application")] Index: KernelRecordsMVC.Application/obj/Debug/net8.0/KernelRecordsMVC.Application.AssemblyInfoInputs.cache =================================================================== --- KernelRecordsMVC.Application/obj/Debug/net8.0/KernelRecordsMVC.Application.AssemblyInfoInputs.cache (revision d89d25b3a7f785676877cf198c9cc706308ab4a3) +++ KernelRecordsMVC.Application/obj/Debug/net8.0/KernelRecordsMVC.Application.AssemblyInfoInputs.cache (revision 595d2e58cf322a776ba1d3cedcb86e9bb2ee8093) @@ -1,1 +1,1 @@ -0a806091492b419d863bcf31dc25ce545b03be1d3b3d188a337ab3ca6889ac3f +1de1e67b21e6555460bba54dec4600f29627ec11612fbabaf75aead1668384fb Index: KernelRecordsMVC.Application/obj/Debug/net8.0/KernelRecordsMVC.Application.sourcelink.json =================================================================== --- KernelRecordsMVC.Application/obj/Debug/net8.0/KernelRecordsMVC.Application.sourcelink.json (revision d89d25b3a7f785676877cf198c9cc706308ab4a3) +++ KernelRecordsMVC.Application/obj/Debug/net8.0/KernelRecordsMVC.Application.sourcelink.json (revision 595d2e58cf322a776ba1d3cedcb86e9bb2ee8093) @@ -1,1 +1,1 @@ -{"documents":{"C:\\Users\\Marko\\RiderProjects\\KernelRecordsMVC\\*":"https://raw.githubusercontent.com/mmilevski/kernelrecords/d2eccb3e3d4db7d262fc57179b8ce6f7fb060d63/*"}} +{"documents":{"C:\\Users\\Marko\\RiderProjects\\KernelRecordsMVC\\*":"https://raw.githubusercontent.com/mmilevski/kernelrecords/d89d25b3a7f785676877cf198c9cc706308ab4a3/*"}} Index: KernelRecordsMVC.Domain/obj/Debug/net8.0/KernelRecordsMVC.Domain.AssemblyInfo.cs =================================================================== --- KernelRecordsMVC.Domain/obj/Debug/net8.0/KernelRecordsMVC.Domain.AssemblyInfo.cs (revision d89d25b3a7f785676877cf198c9cc706308ab4a3) +++ KernelRecordsMVC.Domain/obj/Debug/net8.0/KernelRecordsMVC.Domain.AssemblyInfo.cs (revision 595d2e58cf322a776ba1d3cedcb86e9bb2ee8093) @@ -14,5 +14,5 @@ [assembly: System.Reflection.AssemblyConfigurationAttribute("Debug")] [assembly: System.Reflection.AssemblyFileVersionAttribute("1.0.0.0")] -[assembly: System.Reflection.AssemblyInformationalVersionAttribute("1.0.0+d2eccb3e3d4db7d262fc57179b8ce6f7fb060d63")] +[assembly: System.Reflection.AssemblyInformationalVersionAttribute("1.0.0+d89d25b3a7f785676877cf198c9cc706308ab4a3")] [assembly: System.Reflection.AssemblyProductAttribute("KernelRecordsMVC.Domain")] [assembly: System.Reflection.AssemblyTitleAttribute("KernelRecordsMVC.Domain")] Index: KernelRecordsMVC.Domain/obj/Debug/net8.0/KernelRecordsMVC.Domain.AssemblyInfoInputs.cache =================================================================== --- KernelRecordsMVC.Domain/obj/Debug/net8.0/KernelRecordsMVC.Domain.AssemblyInfoInputs.cache (revision d89d25b3a7f785676877cf198c9cc706308ab4a3) +++ KernelRecordsMVC.Domain/obj/Debug/net8.0/KernelRecordsMVC.Domain.AssemblyInfoInputs.cache (revision 595d2e58cf322a776ba1d3cedcb86e9bb2ee8093) @@ -1,1 +1,1 @@ -9ac54125bd905265e26e18e6c5ad395dc962c3187cbd749ef37d34bb164ca220 +2c5c6269aac41449bcb50d10a12b287cb457ef35a5d93683092ccb6ff4c72e0c Index: KernelRecordsMVC.Domain/obj/Debug/net8.0/KernelRecordsMVC.Domain.sourcelink.json =================================================================== --- KernelRecordsMVC.Domain/obj/Debug/net8.0/KernelRecordsMVC.Domain.sourcelink.json (revision d89d25b3a7f785676877cf198c9cc706308ab4a3) +++ KernelRecordsMVC.Domain/obj/Debug/net8.0/KernelRecordsMVC.Domain.sourcelink.json (revision 595d2e58cf322a776ba1d3cedcb86e9bb2ee8093) @@ -1,1 +1,1 @@ -{"documents":{"C:\\Users\\Marko\\RiderProjects\\KernelRecordsMVC\\*":"https://raw.githubusercontent.com/mmilevski/kernelrecords/d2eccb3e3d4db7d262fc57179b8ce6f7fb060d63/*"}} +{"documents":{"C:\\Users\\Marko\\RiderProjects\\KernelRecordsMVC\\*":"https://raw.githubusercontent.com/mmilevski/kernelrecords/d89d25b3a7f785676877cf198c9cc706308ab4a3/*"}} Index: KernelRecordsMVC.Infrastructure/obj/Debug/net8.0/KernelRecordsMVC.Infrastructure.AssemblyInfo.cs =================================================================== --- KernelRecordsMVC.Infrastructure/obj/Debug/net8.0/KernelRecordsMVC.Infrastructure.AssemblyInfo.cs (revision d89d25b3a7f785676877cf198c9cc706308ab4a3) +++ KernelRecordsMVC.Infrastructure/obj/Debug/net8.0/KernelRecordsMVC.Infrastructure.AssemblyInfo.cs (revision 595d2e58cf322a776ba1d3cedcb86e9bb2ee8093) @@ -14,5 +14,5 @@ [assembly: System.Reflection.AssemblyConfigurationAttribute("Debug")] [assembly: System.Reflection.AssemblyFileVersionAttribute("1.0.0.0")] -[assembly: System.Reflection.AssemblyInformationalVersionAttribute("1.0.0+d2eccb3e3d4db7d262fc57179b8ce6f7fb060d63")] +[assembly: System.Reflection.AssemblyInformationalVersionAttribute("1.0.0+d89d25b3a7f785676877cf198c9cc706308ab4a3")] [assembly: System.Reflection.AssemblyProductAttribute("KernelRecordsMVC.Infrastructure")] [assembly: System.Reflection.AssemblyTitleAttribute("KernelRecordsMVC.Infrastructure")] Index: KernelRecordsMVC.Infrastructure/obj/Debug/net8.0/KernelRecordsMVC.Infrastructure.AssemblyInfoInputs.cache =================================================================== --- KernelRecordsMVC.Infrastructure/obj/Debug/net8.0/KernelRecordsMVC.Infrastructure.AssemblyInfoInputs.cache (revision d89d25b3a7f785676877cf198c9cc706308ab4a3) +++ KernelRecordsMVC.Infrastructure/obj/Debug/net8.0/KernelRecordsMVC.Infrastructure.AssemblyInfoInputs.cache (revision 595d2e58cf322a776ba1d3cedcb86e9bb2ee8093) @@ -1,1 +1,1 @@ -0a244f365ce2ab97b7911a45cd06642ace0b6002cc3788b6ed779b9b4d4fb537 +5ec5e89a0b05a9fba0312a8736adff239c38327d3e73e894fad14b10b6985769 Index: KernelRecordsMVC.Infrastructure/obj/Debug/net8.0/KernelRecordsMVC.Infrastructure.sourcelink.json =================================================================== --- KernelRecordsMVC.Infrastructure/obj/Debug/net8.0/KernelRecordsMVC.Infrastructure.sourcelink.json (revision d89d25b3a7f785676877cf198c9cc706308ab4a3) +++ KernelRecordsMVC.Infrastructure/obj/Debug/net8.0/KernelRecordsMVC.Infrastructure.sourcelink.json (revision 595d2e58cf322a776ba1d3cedcb86e9bb2ee8093) @@ -1,1 +1,1 @@ -{"documents":{"C:\\Users\\Marko\\RiderProjects\\KernelRecordsMVC\\*":"https://raw.githubusercontent.com/mmilevski/kernelrecords/d2eccb3e3d4db7d262fc57179b8ce6f7fb060d63/*"}} +{"documents":{"C:\\Users\\Marko\\RiderProjects\\KernelRecordsMVC\\*":"https://raw.githubusercontent.com/mmilevski/kernelrecords/d89d25b3a7f785676877cf198c9cc706308ab4a3/*"}} Index: KernelRecordsMVC.Web/Controllers/AdminController.cs =================================================================== --- KernelRecordsMVC.Web/Controllers/AdminController.cs (revision d89d25b3a7f785676877cf198c9cc706308ab4a3) +++ KernelRecordsMVC.Web/Controllers/AdminController.cs (revision 595d2e58cf322a776ba1d3cedcb86e9bb2ee8093) @@ -497,7 +497,12 @@ - // ===================================================== - // RELEASE TYPE VALIDATION - // ===================================================== + // ========================================================= + // NORMALIZE RELEASE TYPE + // ========================================================= + + model.ReleaseType = + model.ReleaseType?.Trim().ToUpperInvariant() + ?? string.Empty; + if (model.ReleaseType != "ALBUM" && @@ -510,12 +515,11 @@ - // ===================================================== + // ========================================================= // MAIN ARTIST VALIDATION - // ===================================================== + // ========================================================= var mainArtistExists = _context.Artists.Any(a => - a.ArtistId == - model.MainArtistId); + a.ArtistId == model.MainArtistId); @@ -528,7 +532,7 @@ - // ===================================================== + // ========================================================= // FEATURED ARTISTS - // ===================================================== + // ========================================================= model.FeaturedArtistIds ??= @@ -539,14 +543,9 @@ model.FeaturedArtistIds .Distinct() + .Where(id => + id != model.MainArtistId) .ToList(); - // Do not allow the main artist - // to also be a featured artist. - model.FeaturedArtistIds.Remove( - model.MainArtistId); - - - // Validate that featured artists exist. if (model.FeaturedArtistIds.Count > 0) { @@ -567,12 +566,32 @@ - // ===================================================== + // ========================================================= // SINGLE VALIDATION - // ===================================================== + // ========================================================= if (model.ReleaseType == "SINGLE") { + // Album track values do not apply to a single. + model.Tracks = + new List(); + + + var trackModelStateKeys = + ModelState.Keys + .Where(key => + key.StartsWith( + "Tracks[", + StringComparison.OrdinalIgnoreCase)) + .ToList(); + + + foreach (var key in trackModelStateKeys) + { + ModelState.Remove(key); + } + + if (string.IsNullOrWhiteSpace( - model.SingleDuration)) + model.SingleDuration)) { ModelState.AddModelError( @@ -580,18 +599,51 @@ "Duration is required for a single."); } - } - - - // ===================================================== + else if (!System.Text.RegularExpressions.Regex.IsMatch( + model.SingleDuration.Trim(), + @"^\d{1,2}:[0-5]\d$")) + { + ModelState.AddModelError( + nameof(model.SingleDuration), + "Duration must be in MM:SS format, for example 03:42."); + } + } + + + // ========================================================= // ALBUM VALIDATION - // ===================================================== + // ========================================================= if (model.ReleaseType == "ALBUM") { + // Single duration does not apply to an album. + ModelState.Remove( + nameof(model.SingleDuration)); + + model.SingleDuration = + null; + + model.Tracks ??= new List(); - // Remove completely empty rows. + // The model binder may already have added validation + // errors for blank track rows. Remove those and validate + // the cleaned list ourselves. + var existingTrackKeys = + ModelState.Keys + .Where(key => + key.StartsWith( + "Tracks[", + StringComparison.OrdinalIgnoreCase)) + .ToList(); + + + foreach (var key in existingTrackKeys) + { + ModelState.Remove(key); + } + + model.Tracks = model.Tracks @@ -625,5 +677,5 @@ ModelState.AddModelError( $"Tracks[{i}].SongName", - "Track name is required."); + $"Track {i + 1}: name is required."); } @@ -634,5 +686,5 @@ ModelState.AddModelError( $"Tracks[{i}].SongDuration", - "Track duration is required."); + $"Track {i + 1}: duration is required."); } @@ -648,5 +700,4 @@ - // Validate selected track artists. if (track.ArtistIds.Count > 0) { @@ -662,5 +713,5 @@ ModelState.AddModelError( $"Tracks[{i}].ArtistIds", - "One or more selected track artists are invalid."); + $"Track {i + 1}: one or more artists are invalid."); } } @@ -669,4 +720,8 @@ + // ========================================================= + // VALIDATION FAILED + // ========================================================= + if (!ModelState.IsValid) { @@ -676,4 +731,8 @@ } + + // ========================================================= + // DATABASE TRANSACTION + // ========================================================= using var transaction = @@ -683,7 +742,7 @@ try { - // ================================================= - // CREATE RELEASE - // ================================================= + // ===================================================== + // RELEASE + // ===================================================== var release = @@ -719,9 +778,9 @@ - // ================================================= - // MAIN RELEASE ARTIST - // ================================================= - - var mainReleaseArtist = + // ===================================================== + // RELEASE ARTISTS + // ===================================================== + + _context.ReleaseArtists.Add( new ReleaseArtist { @@ -732,18 +791,11 @@ model.MainArtistId, - ReleaseOrdinal = 1, + ReleaseOrdinal = + 1, Type = ArtistReleaseType.MAIN - }; - - - _context.ReleaseArtists.Add( - mainReleaseArtist); - - - // ================================================= - // FEATURED RELEASE ARTISTS - // ================================================= + }); + long releaseOrdinal = 2; @@ -753,5 +805,5 @@ in model.FeaturedArtistIds) { - var featuredArtist = + _context.ReleaseArtists.Add( new ReleaseArtist { @@ -767,9 +819,5 @@ Type = ArtistReleaseType.FEATURE - }; - - - _context.ReleaseArtists.Add( - featuredArtist); + }); } @@ -778,7 +826,7 @@ - // ================================================= + // ===================================================== // ALBUM - // ================================================= + // ===================================================== if (model.ReleaseType == "ALBUM") @@ -798,7 +846,9 @@ - // ============================================= - // TRACKS - // ============================================= + // Get the next song ID once for this request. + // Each track then gets a unique incremented ID. + var nextSongId = + GetNextSongId(); + foreach (var trackModel @@ -809,15 +859,11 @@ { SongId = - GetNextSongId(), + nextSongId++, SongName = - trackModel - .SongName - .Trim(), + trackModel.SongName.Trim(), SongDuration = - trackModel - .SongDuration - .Trim() + trackModel.SongDuration.Trim() }; @@ -829,9 +875,5 @@ - // ========================================= - // LINK SONG TO ALBUM - // ========================================= - - var albumSong = + _context.AlbumSongs.Add( new AlbumSong { @@ -841,14 +883,6 @@ SongId = song.SongId - }; - - - _context.AlbumSongs.Add( - albumSong); - - - // ========================================= - // SONG ARTISTS - // ========================================= + }); + var trackArtistIds = @@ -858,7 +892,6 @@ - // If no track artists were selected, - // automatically use the release's - // main artist. + // Default to the main release artist when + // no track-specific artists were selected. if (trackArtistIds.Count == 0) { @@ -874,5 +907,5 @@ in trackArtistIds) { - var songArtist = + _context.SongArtists.Add( new SongArtist { @@ -885,9 +918,5 @@ SongOrdinal = songOrdinal++ - }; - - - _context.SongArtists.Add( - songArtist); + }); } @@ -898,11 +927,11 @@ - // ================================================= + // ===================================================== // SINGLE - // ================================================= + // ===================================================== else { - var single = + _context.SingleReleases.Add( new SingleRelease { @@ -911,11 +940,7 @@ Duration = - model.SingleDuration! - .Trim() - }; - - - _context.SingleReleases.Add( - single); + model.SingleDuration!.Trim() + }); + _context.SaveChanges(); @@ -931,11 +956,27 @@ return RedirectToAction( - nameof(Index)); - } - catch + nameof(Releases)); + } + catch (Exception ex) { transaction.Rollback(); - throw; + + Console.WriteLine( + "CREATE RELEASE FAILED:"); + + Console.WriteLine(ex); + + + ModelState.AddModelError( + string.Empty, + ex.InnerException?.Message + ?? ex.Message); + + + LoadArtists(); + + + return View(model); } } Index: KernelRecordsMVC.Web/Controllers/ReleaseController.cs =================================================================== --- KernelRecordsMVC.Web/Controllers/ReleaseController.cs (revision d89d25b3a7f785676877cf198c9cc706308ab4a3) +++ KernelRecordsMVC.Web/Controllers/ReleaseController.cs (revision 595d2e58cf322a776ba1d3cedcb86e9bb2ee8093) @@ -139,14 +139,16 @@ { var release = _context.Releases + .Include(r => r.ReleaseArtists) + .ThenInclude(ra => ra.Artist) + .Include(r => r.Products) - - .Include(r => r.ReleaseArtists) - .ThenInclude(ra => ra.Artist) + .ThenInclude(p => p.ModificationProducts) + .ThenInclude(mp => mp.Modification) .Include(r => r.Album) - .ThenInclude(a => a!.AlbumSongs) - .ThenInclude(x => x.Song) - .ThenInclude(s => s.SongArtists) - .ThenInclude(sa => sa.Artist) + .ThenInclude(a => a.AlbumSongs) + .ThenInclude(albumSong => albumSong.Song) + .ThenInclude(song => song.SongArtists) + .ThenInclude(songArtist => songArtist.Artist) .Include(r => r.SingleRelease) Index: KernelRecordsMVC.Web/Views/Admin/CreateRelease.cshtml =================================================================== --- KernelRecordsMVC.Web/Views/Admin/CreateRelease.cshtml (revision d89d25b3a7f785676877cf198c9cc706308ab4a3) +++ KernelRecordsMVC.Web/Views/Admin/CreateRelease.cshtml (revision 595d2e58cf322a776ba1d3cedcb86e9bb2ee8093) @@ -43,8 +43,7 @@ -
- - -
- -
- -
- - - ADMIN WORKFLOW - - -

Catalog Process

- -
- -
- - -
- -
- - - 01 - - -
- - - Create Release - - -

- Add the album or single, - artists and track information. -

- -
- -
- - -
- - - 02 - - -
- - - Create Product - - -

- Add its vinyl, CD or cassette - store edition. -

- -
- -
- - -
- - - 03 - - -
- - - Manage Inventory - - -

- Change prices, stock, - descriptions or apply discounts. -

- -
- -
- -
- -
- Index: KernelRecordsMVC.Web/Views/Release/Details.cshtml =================================================================== --- KernelRecordsMVC.Web/Views/Release/Details.cshtml (revision d89d25b3a7f785676877cf198c9cc706308ab4a3) +++ KernelRecordsMVC.Web/Views/Release/Details.cshtml (revision 595d2e58cf322a776ba1d3cedcb86e9bb2ee8093) @@ -1,3 +1,4 @@ -@model KernelRecordsMVC.Models.Release +@using KernelRecordsMVC.Domain.Enums +@model KernelRecordsMVC.Models.Release @{ @@ -172,4 +173,37 @@ .OrderBy(x => x.Format)) { + var latestModification = + product.ModificationProducts? + .Select(mp => mp.Modification) + .OrderByDescending(m => m.DateModified) + .ThenByDescending(m => m.ModificationId) + .FirstOrDefault(); + + var isDiscounted = + latestModification != null && + latestModification.TypeOfModification == + ModificationType.DISCOUNT && + latestModification.Discount.HasValue && + latestModification.Discount.Value > 0 && + latestModification.Discount.Value < 100; + + decimal? originalPrice = null; + decimal? discountPercentage = null; + + if (isDiscounted) + { + discountPercentage = + latestModification!.Discount!.Value; + + originalPrice = + Math.Round( + product.Price / + (1m - + discountPercentage.Value / 100m), + 2, + MidpointRounding.AwayFromZero); + } + +
@@ -197,7 +231,36 @@ -
- @product.Price.ToString("C") -
+ + + @if (isDiscounted && + originalPrice.HasValue && + discountPercentage.HasValue) + { +
+ +
+ + + @originalPrice.Value.ToString("C") + + + + @product.Price.ToString("C") + + +
+ + + -@discountPercentage.Value.ToString("0.##")% + + +
+ } + else + { +
+ @product.Price.ToString("C") +
+ } @@ -249,6 +312,4 @@ {
- -
- @(lowestPrice?.ToString("C") ?? "N/A") + @(lowestPrice?.ToString("C") ?? "Coming soon!") Index: KernelRecordsMVC.Web/Views/Shared/_Layout.cshtml =================================================================== --- KernelRecordsMVC.Web/Views/Shared/_Layout.cshtml (revision d89d25b3a7f785676877cf198c9cc706308ab4a3) +++ KernelRecordsMVC.Web/Views/Shared/_Layout.cshtml (revision 595d2e58cf322a776ba1d3cedcb86e9bb2ee8093) @@ -1,6 +1,26 @@ @using Microsoft.AspNetCore.Http +@using Microsoft.EntityFrameworkCore +@using KernelRecordsMVC.Infrastructure.Data +@using KernelRecordsMVC.Domain.Enums +@inject KernelRecordsContext DbContext + +@{ + long cartCount = 0; + + var sessionUserId = Context.Session.GetInt32("UserId"); + + if (sessionUserId.HasValue) + { + var userId = (long)sessionUserId.Value; + + cartCount = await DbContext.OrderProducts + .Where(op => + op.Order.UserId == userId && + op.Order.Status == OrderStatusType.PENDING) + .SumAsync(op => (long?)op.Quantity) ?? 0; + } +} - @@ -8,10 +28,9 @@ - -@ViewData["Title"] - Kernel Records - - - - + @ViewData["Title"] - Kernel Records + + @@ -20,414 +39,293 @@
- - - - + + + + +
+ + + FREE SHIPPING ON ORDERS OVER $100 + +
- - - - - - -
- - - FREE SHIPPING ON ORDERS OVER €50 - - - - • - - - - NEW VINYL EVERY WEEK - - -
- +
+ - -
- -