Merge pull request #1193 from TechnologyEnhancedLearning/Develop/Fixes/TD-5501

TD-5501 Route fix for GetAllbyParent Bookmark dual endpoint

Author: OluwatobiAwe

OpenAPI/LearningHub.Nhs.OpenApi/Controllers/BookmarkController.cs

@@ -24,33 +24,28 @@ public BookmarkController(IBookmarkService bookmarkService)
             this.bookmarkService = bookmarkService;
         }
 
-        /// <summary>
-        /// <summary>
-        /// Gets all bookmarks by parent.
-        /// </summary>
-        /// <returns>Bookmarks.</returns>
-        [HttpGet]
-        [Route("GetAllByParent")]
-        public async Task<IEnumerable<UserBookmarkViewModel>> GetAllByParent()
-        {
-            return await this.bookmarkService.GetAllByParent(this.TokenWithoutBearer);
-        }
-
         /// <summary>
         /// The GetAllByParent.
         /// </summary>
         /// <param name="parentId">The parentId.</param>
         /// <param name="all">The all.</param>
         /// <returns>The <see cref="Task{IActionResult}"/>.</returns>
         [HttpGet]
+        [Route("GetAllByParent")]
         [Route("GetAllByParent/{parentId?}")]
         public async Task<IActionResult> GetAllByParent(int? parentId, bool? all = false)
         {
-            var bookmarks = await this.bookmarkService.GetAllByParent(this.CurrentUserId.GetValueOrDefault(), parentId, all);
-            return this.Ok(bookmarks);
+            if (this.CurrentUserId.GetValueOrDefault() != null)
+            {
+                var bookmarks = await this.bookmarkService.GetAllByParent(this.CurrentUserId.GetValueOrDefault(), parentId, all);
+                return this.Ok(bookmarks);
+            }
+            else
+            {
+                return this.Ok(await this.bookmarkService.GetAllByParent(this.TokenWithoutBearer));
+            }
         }
 
-
         /// <summary>
         /// The Create.
         /// </summary>