UoE/datashare: trim verbose code comments

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: milanmajchrak

src/app/core/data/item-data.service.ts

@@ -273,16 +273,14 @@ export abstract class BaseItemDataService extends IdentifiableDataService<Item>
    * Get the endpoint to move the item
    * @param itemId
    * @param inheritPolicies whether to inherit the destination collection's default policies
-   * @param keepEmbargoPolicies when inheriting, whether to keep an existing embargo (only the
-   *                            non-embargo access is inherited) instead of letting the inherited
-   *                            default read access lift it
+   * @param keepEmbargoPolicies when inheriting, keep an existing embargo instead of letting it be lifted
    */
   public getMoveItemEndpoint(itemId: string, inheritPolicies: boolean, keepEmbargoPolicies = true): Observable<string> {
     return this.halService.getEndpoint(this.linkPath).pipe(
       map((endpoint: string) => this.getIDHref(endpoint, itemId)),
       map((endpoint: string) => {
         let href = `${endpoint}/owningCollection?inheritPolicies=${inheritPolicies}`;
-        // keepEmbargoPolicies only affects the move when policies are inherited, so only send it then.
+        // Only relevant when inheriting.
         if (inheritPolicies) {
           href += `&keepEmbargoPolicies=${keepEmbargoPolicies}`;
         }

src/app/item-page/edit-item-page/item-move/item-move.component.ts

@@ -74,11 +74,7 @@ export class ItemMoveComponent implements OnInit {
 
   inheritPolicies = false;
 
-  /**
-   * When inheriting the destination collection's policies, whether to keep an existing embargo
-   * (only the non-embargo access is inherited) instead of letting the inherited default read
-   * access lift it. Only relevant while {@link inheritPolicies} is enabled.
-   */
+  /** When inheriting, keep an existing embargo instead of letting the inherited default read lift it. */
   keepEmbargoPolicies = true;
   itemRD$: Observable<RemoteData<Item>>;
   originalCollection: Collection;
@@ -152,8 +148,6 @@ export class ItemMoveComponent implements OnInit {
    */
   moveToCollection() {
     this.processing = true;
-    // keepEmbargoPolicies is only sent (and only matters) when inheriting; getMoveItemEndpoint omits
-    // it otherwise, so we can pass it straight through.
     const move$ = this.itemDataService.moveToCollection(
       this.item.id, this.selectedCollection, this.inheritPolicies, this.keepEmbargoPolicies,
     ).pipe(getFirstCompletedRemoteData());