WIP get user bookings

Paul Frank · Paul Frank · commit 3ab1c40da34c · 2023-11-12T19:30:46.000+01:00
diff --git a/TableBookingAPI/TableBooking.Logic/Converters/TableConverters/TableConverter.cs b/TableBookingAPI/TableBooking.Logic/Converters/TableConverters/TableConverter.cs
@@ -19,8 +19,8 @@ public TableDto TableToTableDto(Table table)
         {
             return new TableDto
             {
-                RestaurantId = table.Restaurant.Id,
-                NumberOfSeats = table.NumberOfSeats
+                RestaurantId = table.RestaurantId,
+                NumberOfSeats = table.NumberOfSeats,
             };
         }
     }
diff --git a/TableBookingAPI/TableBooking.Logic/Converters/TableConverters/TableToGetConverter.cs b/TableBookingAPI/TableBooking.Logic/Converters/TableConverters/TableToGetConverter.cs
@@ -20,7 +20,7 @@ public GetTablesDto TableToTableDto(Table table)
             return new GetTablesDto
             {
                 Id = table.Id,
-                RestaurantId = table.Restaurant.Id,
+                RestaurantId = table.RestaurantId,
                 NumberOfSeats = table.NumberOfSeats
             };
         }
diff --git a/TableBookingAPI/TableBooking.Logic/Interfaces/IBookingRepository.cs b/TableBookingAPI/TableBooking.Logic/Interfaces/IBookingRepository.cs
@@ -5,6 +5,6 @@ namespace TableBooking.Logic.Interfaces
     public interface IBookingRepository : IGenericRepository<Booking>
     {
         public Task<IEnumerable<Booking>> GetAllBookingsForSpecificUserAsync(Guid userId);
-        public Task<Booking> GetBookingByIdForSpecificUserAsync(Guid bookingId, Guid userId);
+        public Task<Booking?> GetBookingByIdForSpecificUserAsync(Guid bookingId, Guid userId);
     }
 }
diff --git a/TableBookingAPI/TableBooking.Logic/Repositories/BookingRepository.cs b/TableBookingAPI/TableBooking.Logic/Repositories/BookingRepository.cs
@@ -13,12 +13,12 @@ public BookingRepository(TableBookingContext context) : base(context)
 
         public async Task<IEnumerable<Booking>> GetAllBookingsForSpecificUserAsync(Guid userId)
         {
-            return await _objectSet.Where(x => x.User.Id.Equals(userId)).ToListAsync();
+            return await _objectSet.Where(x => x.Id.Equals(userId)).ToListAsync();
         }
 
-        public async Task<Booking> GetBookingByIdForSpecificUserAsync(Guid bookingId, Guid userId)
+        public async Task<Booking?> GetBookingByIdForSpecificUserAsync(Guid bookingId, Guid userId)
         {
-            return await _objectSet.FirstOrDefaultAsync(x => x.Id.Equals(bookingId) && x.User.Id.Equals(userId));
+            return await _objectSet.FirstOrDefaultAsync(x => x.Id.Equals(bookingId) && x.Id.Equals(userId));
         }
     }
 }
diff --git a/TableBookingAPI/TableBooking.Logic/Repositories/TableRepository.cs b/TableBookingAPI/TableBooking.Logic/Repositories/TableRepository.cs
@@ -13,8 +13,8 @@ public TableRepository(TableBookingContext context) : base(context)
         public async Task<IEnumerable<Table>> GetTablesByRestaurantIdAsync(Guid restaurantId)
         {
             return await _objectSet
-                .Include(x => x.Restaurant)
-                .Where(x => x.Restaurant.Id.Equals(restaurantId))
+                .Include(x => x.RestaurantId)
+                .Where(x => x.RestaurantId.Equals(restaurantId))
                 .ToListAsync();
         }
     }
diff --git a/TableBookingAPI/TableBooking.Model/Dtos/TableDtos/TableDto.cs b/TableBookingAPI/TableBooking.Model/Dtos/TableDtos/TableDto.cs
@@ -1,6 +1,8 @@
-﻿namespace TableBooking.Model.Dtos.TableDtos
+﻿using TableBooking.Model.Models;
+
+namespace TableBooking.Model.Dtos.TableDtos
 {
-    public class TableDto
+    public class TableDto : Entity
     {
         public int NumberOfSeats { get; set; }
         public Guid RestaurantId { get; set; }
diff --git a/TableBookingAPI/TableBooking.Model/Migrations/20231112181725_AppUserId.Designer.cs b/TableBookingAPI/TableBooking.Model/Migrations/20231112181725_AppUserId.Designer.cs
diff --git a/TableBookingAPI/TableBooking.Model/Migrations/20231112181725_AppUserId.cs b/TableBookingAPI/TableBooking.Model/Migrations/20231112181725_AppUserId.cs
@@ -5,7 +5,7 @@
 
 namespace TableBooking.Model.Migrations
 {
-    public partial class initialProperWithOldFixes : Migration
+    public partial class AppUserId : Migration
     {
         protected override void Up(MigrationBuilder migrationBuilder)
         {
@@ -113,7 +113,7 @@ protected override void Up(MigrationBuilder migrationBuilder)
                     Id = table.Column<Guid>(type: "uuid", nullable: false),
                     Date = table.Column<DateTime>(type: "timestamp with time zone", nullable: false),
                     DurationInMinutes = table.Column<int>(type: "integer", nullable: false),
-                    UserId = table.Column<Guid>(type: "uuid", nullable: false),
+                    AppUserId = table.Column<Guid>(type: "uuid", nullable: false),
                     TableId = table.Column<Guid>(type: "uuid", nullable: false)
                 },
                 constraints: table =>
@@ -126,22 +126,22 @@ protected override void Up(MigrationBuilder migrationBuilder)
                         principalColumn: "Id",
                         onDelete: ReferentialAction.Cascade);
                     table.ForeignKey(
-                        name: "FK_Bookings_Users_UserId",
-                        column: x => x.UserId,
+                        name: "FK_Bookings_Users_AppUserId",
+                        column: x => x.AppUserId,
                         principalTable: "Users",
                         principalColumn: "Id",
                         onDelete: ReferentialAction.Cascade);
                 });
 
             migrationBuilder.CreateIndex(
-                name: "IX_Bookings_TableId",
+                name: "IX_Bookings_AppUserId",
                 table: "Bookings",
-                column: "TableId");
+                column: "AppUserId");
 
             migrationBuilder.CreateIndex(
-                name: "IX_Bookings_UserId",
+                name: "IX_Bookings_TableId",
                 table: "Bookings",
-                column: "UserId");
+                column: "TableId");
 
             migrationBuilder.CreateIndex(
                 name: "IX_Ratings_AppUserId",
diff --git a/TableBookingAPI/TableBooking.Model/Migrations/TableBookingContextModelSnapshot.cs b/TableBookingAPI/TableBooking.Model/Migrations/TableBookingContextModelSnapshot.cs
@@ -87,6 +87,9 @@ protected override void BuildModel(ModelBuilder modelBuilder)
                         .ValueGeneratedOnAdd()
                         .HasColumnType("uuid");
 
+                    b.Property<Guid>("AppUserId")
+                        .HasColumnType("uuid");
+
                     b.Property<DateTime>("Date")
                         .HasColumnType("timestamp with time zone");
 
@@ -96,14 +99,11 @@ protected override void BuildModel(ModelBuilder modelBuilder)
                     b.Property<Guid>("TableId")
                         .HasColumnType("uuid");
 
-                    b.Property<Guid>("UserId")
-                        .HasColumnType("uuid");
-
                     b.HasKey("Id");
 
-                    b.HasIndex("TableId");
+                    b.HasIndex("AppUserId");
 
-                    b.HasIndex("UserId");
+                    b.HasIndex("TableId");
 
                     b.ToTable("Bookings");
                 });
@@ -211,21 +211,19 @@ protected override void BuildModel(ModelBuilder modelBuilder)
 
             modelBuilder.Entity("TableBooking.Model.Models.Booking", b =>
                 {
-                    b.HasOne("TableBooking.Model.Models.Table", "Table")
+                    b.HasOne("TableBooking.Model.Models.AppUser", null)
                         .WithMany("Bookings")
-                        .HasForeignKey("TableId")
+                        .HasForeignKey("AppUserId")
                         .OnDelete(DeleteBehavior.Cascade)
                         .IsRequired();
 
-                    b.HasOne("TableBooking.Model.Models.AppUser", "User")
+                    b.HasOne("TableBooking.Model.Models.Table", "Table")
                         .WithMany("Bookings")
-                        .HasForeignKey("UserId")
+                        .HasForeignKey("TableId")
                         .OnDelete(DeleteBehavior.Cascade)
                         .IsRequired();
 
                     b.Navigation("Table");
-
-                    b.Navigation("User");
                 });
 
             modelBuilder.Entity("TableBooking.Model.Models.Rating", b =>
@@ -249,13 +247,11 @@ protected override void BuildModel(ModelBuilder modelBuilder)
 
             modelBuilder.Entity("TableBooking.Model.Models.Table", b =>
                 {
-                    b.HasOne("TableBooking.Model.Models.Restaurant", "Restaurant")
+                    b.HasOne("TableBooking.Model.Models.Restaurant", null)
                         .WithMany("Tables")
                         .HasForeignKey("RestaurantId")
                         .OnDelete(DeleteBehavior.Cascade)
                         .IsRequired();
-
-                    b.Navigation("Restaurant");
                 });
 
             modelBuilder.Entity("TableBooking.Model.Models.AppUser", b =>
diff --git a/TableBookingAPI/TableBooking.Model/Models/Booking.cs b/TableBookingAPI/TableBooking.Model/Models/Booking.cs
@@ -5,7 +5,7 @@ public class Booking : Entity
         public DateTime Date { get; set; }
         public int DurationInMinutes { get; set; }
         public Table Table { get; set; }
-        public AppUser User { get; set; }
+        public Guid AppUserId { get; set; }
         public Guid TableId { get; set; }
     }
 }
diff --git a/TableBookingAPI/TableBooking.Model/Models/Table.cs b/TableBookingAPI/TableBooking.Model/Models/Table.cs
@@ -3,7 +3,7 @@
     public class Table : Entity
     {
         public int NumberOfSeats { get; set; }
-        public Restaurant Restaurant { get; set; }
+        public Guid RestaurantId { get; set; }
         public IEnumerable<Booking> Bookings { get; set; }
     }
 }
diff --git a/TableBookingAPI/TableBooking.Model/TableBooking.Model.csproj b/TableBookingAPI/TableBooking.Model/TableBooking.Model.csproj
@@ -21,4 +21,8 @@
     <PackageReference Include="Npgsql.EntityFrameworkCore.PostgreSQL" Version="6.0.8" />
   </ItemGroup>
 
+  <ItemGroup>
+    <Folder Include="Migrations\" />
+  </ItemGroup>
+
 </Project>
diff --git a/TableBookingAPI/TableBooking/Controllers/BookingController.cs b/TableBookingAPI/TableBooking/Controllers/BookingController.cs
@@ -32,6 +32,8 @@ public BookingController(IBookingService bookingService, UserManager<AppUser> us
 
 
         [HttpGet("GetAllUserBookings")]
+        [ProducesResponseType(typeof(List<BookingDto>), StatusCodes.Status200OK)]
+        [ProducesResponseType(StatusCodes.Status400BadRequest)]
         public async Task<IActionResult> GetUserBookings()
         {
             var userId = Guid.Parse(User.FindFirstValue(ClaimTypes.NameIdentifier));
diff --git a/TableBookingAPI/TableBooking/Interfaces/ITableService.cs b/TableBookingAPI/TableBooking/Interfaces/ITableService.cs
@@ -1,5 +1,6 @@
 ﻿using Microsoft.AspNetCore.Mvc;
 using TableBooking.Model.Dtos.TableDtos;
+using TableBooking.Model.Models;
 
 namespace TableBooking.Api.Interfaces
 {
@@ -11,6 +12,6 @@ public interface ITableService
         public Task<IActionResult> CreateTableAsync(TableDto dto);
         public Task<IActionResult> UpdateTableAsync(TableDto dto);
         public Task<IActionResult> DeleteTableAsync(Guid tableId);
+        public Task<Table> GetTableObjectByIdAsync(Guid tableId);
     }
-
 }
diff --git a/TableBookingAPI/TableBooking/Program.cs b/TableBookingAPI/TableBooking/Program.cs
@@ -122,10 +122,6 @@
     options.User.RequireUniqueEmail = true;
     options.ClaimsIdentity.UserIdClaimType = ClaimTypes.NameIdentifier;
 });
-
-//builder.Services.AddHttpContextAccessor();
-builder.Services.AddAuthorization();
-
 builder.Services.AddScoped<IUnitOfWork, UnitOfWork>();
 
 builder.Services.AddTransient<ITableConverter, TableConverter>(); // doczytaj debilu
diff --git a/TableBookingAPI/TableBooking/Services/BookingService.cs b/TableBookingAPI/TableBooking/Services/BookingService.cs
@@ -11,24 +11,28 @@ public class BookingService : IBookingService
     {
         public IUnitOfWork _unitOfWork;
         private readonly ITableConverter _tableConverter;
+        private readonly ITableService _tableService;
 
-        public BookingService(IUnitOfWork unitOfWork, ITableConverter tableConverter)
+
+        public BookingService(IUnitOfWork unitOfWork, ITableConverter tableConverter, ITableService tableService)
         {
             _unitOfWork = unitOfWork;
             _tableConverter = tableConverter;
+            _tableService = tableService;
         }
         public async Task<IActionResult> CreateBookingAsync(CreateBookingDto request, Guid userId)
         {
-            //string userId = User.FindFirstValue(ClaimTypes.NameIdentifier); // Retrieve the authenticated user's ID
-
+            var table = await _tableService.GetTableObjectByIdAsync(request.TableId);
+            
             var newBooking = new Booking
             {
                 Date = request.Date,
                 DurationInMinutes = request.DurationInMinutes,
-                TableId = request.TableId
+                TableId = request.TableId,
+                AppUserId = userId,
+                Table = table
             };
 
-
             await _unitOfWork.BookingRepository.InsertAsync(newBooking);
             await _unitOfWork.SaveChangesAsync();
 
@@ -68,7 +72,7 @@ public async Task<IActionResult> GetBookingByIdAsync(Guid bookingId, Guid userId
                     Date = booking.Date,
                     DurationInMinutes = booking.DurationInMinutes,
                     TableDto = _tableConverter.TableToTableDto(booking.Table),
-                    UserId = booking.User.Id
+                    UserId = userId
                 };
                 return new OkObjectResult(bookingDto);
             }
diff --git a/TableBookingAPI/TableBooking/Services/TableService.cs b/TableBookingAPI/TableBooking/Services/TableService.cs
@@ -53,6 +53,14 @@ public async Task<IActionResult> GetTableByIdAsync(Guid tableId)
                 return new BadRequestObjectResult($"Can't find table with {tableId}");
             return new OkObjectResult(table);
         }
+        
+        public async Task<Table> GetTableObjectByIdAsync(Guid tableId)
+        {
+            var table = await _unitOfWork.TableRepository.GetByIdAsync(tableId);
+            if (table == null)
+                throw new BadHttpRequestException($"Table id: {tableId} doesn't exist.");
+            return table;
+        }
 
         public async Task<IActionResult> GetTableByRestaurantAsync(Guid restaurantId)
         {

Original file line number	Diff line number	Diff line change
`@@ -19,8 +19,8 @@ public TableDto TableToTableDto(Table table)`
`19`	`19`	`{`
`20`	`20`	`return new TableDto`
`21`	`21`	`{`
`22`		`- RestaurantId = table.Restaurant.Id,`
`23`		`- NumberOfSeats = table.NumberOfSeats`
	`22`	`+ RestaurantId = table.RestaurantId,`
	`23`	`+ NumberOfSeats = table.NumberOfSeats,`
`24`	`24`	`};`
`25`	`25`	`}`
`26`	`26`	`}`
Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,7 @@ public GetTablesDto TableToTableDto(Table table)`
`20`	`20`	`return new GetTablesDto`
`21`	`21`	`{`
`22`	`22`	`Id = table.Id,`
`23`		`- RestaurantId = table.Restaurant.Id,`
	`23`	`+ RestaurantId = table.RestaurantId,`
`24`	`24`	`NumberOfSeats = table.NumberOfSeats`
`25`	`25`	`};`
`26`	`26`	`}`
Original file line number	Diff line number	Diff line change
`@@ -5,6 +5,6 @@ namespace TableBooking.Logic.Interfaces`
`5`	`5`	`public interface IBookingRepository : IGenericRepository<Booking>`
`6`	`6`	`{`
`7`	`7`	`public Task<IEnumerable<Booking>> GetAllBookingsForSpecificUserAsync(Guid userId);`
`8`		`- public Task<Booking> GetBookingByIdForSpecificUserAsync(Guid bookingId, Guid userId);`
	`8`	`+ public Task<Booking?> GetBookingByIdForSpecificUserAsync(Guid bookingId, Guid userId);`
`9`	`9`	`}`
`10`	`10`	`}`
Original file line number	Diff line number	Diff line change
`@@ -13,12 +13,12 @@ public BookingRepository(TableBookingContext context) : base(context)`
`13`	`13`
`14`	`14`	`public async Task<IEnumerable<Booking>> GetAllBookingsForSpecificUserAsync(Guid userId)`
`15`	`15`	`{`
`16`		`- return await _objectSet.Where(x => x.User.Id.Equals(userId)).ToListAsync();`
	`16`	`+ return await _objectSet.Where(x => x.Id.Equals(userId)).ToListAsync();`
`17`	`17`	`}`
`18`	`18`
`19`		`- public async Task<Booking> GetBookingByIdForSpecificUserAsync(Guid bookingId, Guid userId)`
	`19`	`+ public async Task<Booking?> GetBookingByIdForSpecificUserAsync(Guid bookingId, Guid userId)`
`20`	`20`	`{`
`21`		`- return await _objectSet.FirstOrDefaultAsync(x => x.Id.Equals(bookingId) && x.User.Id.Equals(userId));`
	`21`	`+ return await _objectSet.FirstOrDefaultAsync(x => x.Id.Equals(bookingId) && x.Id.Equals(userId));`
`22`	`22`	`}`
`23`	`23`	`}`
`24`	`24`	`}`
Original file line number	Diff line number	Diff line change
`@@ -13,8 +13,8 @@ public TableRepository(TableBookingContext context) : base(context)`
`13`	`13`	`public async Task<IEnumerable<Table>> GetTablesByRestaurantIdAsync(Guid restaurantId)`
`14`	`14`	`{`
`15`	`15`	`return await _objectSet`
`16`		`- .Include(x => x.Restaurant)`
`17`		`- .Where(x => x.Restaurant.Id.Equals(restaurantId))`
	`16`	`+ .Include(x => x.RestaurantId)`
	`17`	`+ .Where(x => x.RestaurantId.Equals(restaurantId))`
`18`	`18`	`.ToListAsync();`
`19`	`19`	`}`
`20`	`20`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,8 @@`
`1`		`-namespace TableBooking.Model.Dtos.TableDtos`
	`1`	`+using TableBooking.Model.Models;`
	`2`	`+`
	`3`	`+namespace TableBooking.Model.Dtos.TableDtos`
`2`	`4`	`{`
`3`		`- public class TableDto`
	`5`	`+ public class TableDto : Entity`
`4`	`6`	`{`
`5`	`7`	`public int NumberOfSeats { get; set; }`
`6`	`8`	`public Guid RestaurantId { get; set; }`
Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,7 @@ public class Booking : Entity`
`5`	`5`	`public DateTime Date { get; set; }`
`6`	`6`	`public int DurationInMinutes { get; set; }`
`7`	`7`	`public Table Table { get; set; }`
`8`		`- public AppUser User { get; set; }`
	`8`	`+ public Guid AppUserId { get; set; }`
`9`	`9`	`public Guid TableId { get; set; }`
`10`	`10`	`}`
`11`	`11`	`}`