Topic catalogging - fix js error and enable the ability to uncatalog …

…a topic
TASVideos · Sep 7, 2024 · fecb04d · fecb04d
1 parent 7134edb
commit fecb04d
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 7 deletions.
diff --git a/TASVideos/Pages/Forum/Topics/Catalog.cshtml.cs b/TASVideos/Pages/Forum/Topics/Catalog.cshtml.cs
@@ -14,11 +14,9 @@ public class CatalogModel(ApplicationDbContext db) : BasePageModel
 	public string Title { get; set; } = "";
 
 	[BindProperty]
-	[Required]
 	public int? SystemId { get; set; }
 
 	[BindProperty]
-	[Required]
 	public int? GameId { get; set; }
 
 	public async Task<IActionResult> OnGet()
@@ -63,10 +61,13 @@ public async Task<IActionResult> OnPost()
 			return NotFound();
 		}
 
-		var gameExists = await db.Games.AnyAsync(g => g.Id == GameId);
-		if (!gameExists)
+		if (GameId.HasValue)
 		{
-			return BadRequest();
+			var gameExists = GameId.HasValue && await db.Games.AnyAsync(g => g.Id == GameId);
+			if (!gameExists)
+			{
+				return BadRequest();
+			}
 		}
 
 		topic.GameId = GameId;

diff --git a/TASVideos/wwwroot/js/catalog.js b/TASVideos/wwwroot/js/catalog.js
@@ -71,8 +71,10 @@ function enableCataloging() {
 		setSyncVerifiedCheckbox();
 	}
 
-	versionModel.onchange = function() {
-		setSyncVerifiedCheckbox();
+	if (versionModel) {
+		versionModel.onchange = function() {
+			setSyncVerifiedCheckbox();
+		}
 	}
 
 	document.getElementById('create-version')?.addEventListener('click', function () {