[ENG-10770] "See more" button shows error when only one admin contributor exists

src/app/features/collections/components/add-to-collection/project-contributors-step/project-contributors-step.component.html

@@ -42,6 +42,7 @@ <h3>{{ 'collections.addToCollection.projectContributors' | translate }}</h3>
           [(contributors)]="projectContributors"
           [tableParams]="tableParams()"
           [isLoading]="isContributorsLoading()"
+          [showLoadMore]="hasMoreContributors()"
           [isLoadingMore]="isLoadingMore()"
           (remove)="handleRemoveContributor($event)"
           (loadMore)="loadMoreContributors()"

src/app/features/collections/components/add-to-collection/project-contributors-step/project-contributors-step.component.ts

@@ -62,15 +62,16 @@ export class ProjectContributorsStepComponent {
   private readonly customConfirmationService = inject(CustomConfirmationService);
   private readonly router = inject(Router);
 
+  readonly currentUser = select(UserSelectors.getCurrentUser);
   readonly isContributorsLoading = select(ContributorsSelectors.isContributorsLoading);
   readonly contributorsTotalCount = select(ContributorsSelectors.getContributorsTotalCount);
   readonly selectedProject = select(ProjectsSelectors.getSelectedProject);
-  readonly currentUser = select(UserSelectors.getCurrentUser);
-  isLoadingMore = select(ContributorsSelectors.isContributorsLoadingMore);
+  readonly isLoadingMore = select(ContributorsSelectors.isContributorsLoadingMore);
+  readonly hasMoreContributors = select(ContributorsSelectors.hasMoreContributors);
+  readonly pageSize = select(ContributorsSelectors.getContributorsPageSize);
 
   private initialContributors = select(ContributorsSelectors.getContributors);
   readonly projectContributors = signal<ContributorModel[]>([]);
-  pageSize = select(ContributorsSelectors.getContributorsPageSize);
 
   readonly tableParams = computed<TableParameters>(() => ({
     ...DEFAULT_TABLE_PARAMS,

src/app/features/contributors/contributors.component.html

@@ -67,6 +67,7 @@ <h1 class="py-5 px-3 md:px-5 xl:px-4">{{ 'navigation.contributors' | translate }
     [tableParams]="tableParams()"
     [hasAdminAccess]="hasAdminAccess()"
     [currentUserId]="currentUser()?.id"
+    [showLoadMore]="hasMoreContributors()"
     [showCurator]="true"
     [showInfo]="true"
     [resourceType]="resourceType()"

src/app/features/contributors/contributors.component.ts

@@ -134,11 +134,12 @@ export class ContributorsComponent implements OnInit, OnDestroy {
   readonly isContributorsLoading = select(ContributorsSelectors.isContributorsLoading);
   readonly contributorsTotalCount = select(ContributorsSelectors.getContributorsTotalCount);
   readonly isViewOnlyLinksLoading = select(ViewOnlyLinkSelectors.isViewOnlyLinksLoading);
+  readonly isLoadingMore = select(ContributorsSelectors.isContributorsLoadingMore);
+  readonly pageSize = select(ContributorsSelectors.getContributorsPageSize);
+  readonly hasMoreContributors = select(ContributorsSelectors.hasMoreContributors);
   readonly hasAdminAccess = select(CurrentResourceSelectors.hasResourceAdminAccess);
   readonly resourceAccessRequestEnabled = select(CurrentResourceSelectors.resourceAccessRequestEnabled);
   readonly currentUser = select(UserSelectors.getCurrentUser);
-  pageSize = select(ContributorsSelectors.getContributorsPageSize);
-  isLoadingMore = select(ContributorsSelectors.isContributorsLoadingMore);
 
   readonly tableParams = computed<TableParameters>(() => ({
     ...DEFAULT_TABLE_PARAMS,

src/app/features/metadata/dialogs/contributors-dialog/contributors-dialog.component.html

@@ -20,6 +20,7 @@
     [tableParams]="tableParams()"
     [isLoading]="isLoading()"
     [isLoadingMore]="isLoadingMore()"
+    [showLoadMore]="hasMoreContributors()"
     [showEmployment]="false"
     [showEducation]="false"
     [hasAdminAccess]="hasAdminAccess()"

src/app/features/metadata/dialogs/contributors-dialog/contributors-dialog.component.ts

@@ -75,7 +75,9 @@ export class ContributorsDialogComponent implements OnInit {
   hasAdminAccess = select(MetadataSelectors.hasAdminAccess);
   contributors = signal<ContributorModel[]>([]);
   isLoadingMore = select(ContributorsSelectors.isContributorsLoadingMore);
+  hasMoreContributors = select(ContributorsSelectors.hasMoreContributors);
   pageSize = select(ContributorsSelectors.getContributorsPageSize);
+
   changesMade = signal<boolean>(false);
 
   currentUser = select(UserSelectors.getCurrentUser);

src/app/features/preprints/components/stepper/preprints-metadata-step/preprints-contributors/preprints-contributors.component.html

@@ -12,6 +12,7 @@ <h2>{{ 'common.labels.contributors' | translate }}</h2>
     [(contributors)]="contributors"
     [tableParams]="tableParams()"
     [isLoading]="isContributorsLoading()"
+    [showLoadMore]="hasMoreContributors()"
     [isLoadingMore]="isLoadingMore()"
     (remove)="removeContributor($event)"
     (loadMore)="loadMoreContributors()"

src/app/features/preprints/components/stepper/preprints-metadata-step/preprints-contributors/preprints-contributors.component.ts

@@ -64,6 +64,7 @@ export class PreprintsContributorsComponent implements OnInit {
   readonly contributors = signal<ContributorModel[]>([]);
   readonly contributorsTotalCount = select(ContributorsSelectors.getContributorsTotalCount);
   readonly isContributorsLoading = select(ContributorsSelectors.isContributorsLoading);
+  readonly hasMoreContributors = select(ContributorsSelectors.hasMoreContributors);
   readonly isLoadingMore = select(ContributorsSelectors.isContributorsLoadingMore);
   readonly pageSize = select(ContributorsSelectors.getContributorsPageSize);
 

src/app/features/registries/components/registries-metadata-step/registries-contributors/registries-contributors.component.html

@@ -6,6 +6,7 @@ <h2 class="mb-2">{{ 'common.labels.contributors' | translate }}</h2>
     [(contributors)]="contributors"
     [tableParams]="tableParams()"
     [isLoading]="isContributorsLoading()"
+    [showLoadMore]="hasMoreContributors()"
     [isLoadingMore]="isLoadingMore()"
     (remove)="removeContributor($event)"
     (loadMore)="loadMoreContributors()"

src/app/features/registries/components/registries-metadata-step/registries-contributors/registries-contributors.component.ts

@@ -68,6 +68,7 @@ export class RegistriesContributorsComponent implements OnInit, OnDestroy {
 
   isContributorsLoading = select(ContributorsSelectors.isContributorsLoading);
   contributorsTotalCount = select(ContributorsSelectors.getContributorsTotalCount);
+  hasMoreContributors = select(ContributorsSelectors.hasMoreContributors);
   isLoadingMore = select(ContributorsSelectors.isContributorsLoadingMore);
   pageSize = select(ContributorsSelectors.getContributorsPageSize);
 

src/app/shared/components/contributors/contributors-table/contributors-table.component.html

@@ -174,19 +174,19 @@
         </td>
       </tr>
     }
+  </ng-template>
 
-    @if (showLoadMore() && index === contributors().length - 1) {
+  <ng-template #footer>
+    @if (showLoadMore()) {
       <tr>
-        <td colspan="8" class="text-center">
-          <div class="flex justify-content-center">
-            <p-button
-              class="w-full"
-              styleClass="w-full"
-              (onClick)="loadMoreItems()"
-              [label]="'common.buttons.seeMore' | translate"
-              [loading]="isLoadingMore()"
-            />
-          </div>
+        <td colspan="8">
+          <p-button
+            class="w-full"
+            styleClass="w-full"
+            (onClick)="loadMoreItems()"
+            [label]="'common.buttons.seeMore' | translate"
+            [loading]="isLoadingMore()"
+          />
         </td>
       </tr>
     }

src/app/shared/components/contributors/contributors-table/contributors-table.component.ts

@@ -50,6 +50,7 @@ export class ContributorsTableComponent {
   showEducation = input(true);
   showEmployment = input(true);
   showInfo = input(false);
+  showLoadMore = input(false);
   resourceType = input(ResourceType.Project);
 
   currentUserId = input<string | undefined>(undefined);
@@ -69,12 +70,6 @@ export class ContributorsTableComponent {
 
   deactivatedContributors = computed(() => this.contributors().some((contributor) => contributor.deactivated));
 
-  showLoadMore = computed(() => {
-    const currentLoadedItems = this.contributors().length;
-    const totalRecords = this.tableParams().totalRecords;
-    return currentLoadedItems > 0 && currentLoadedItems < totalRecords;
-  });
-
   removeContributor(contributor: ContributorModel) {
     this.remove.emit(contributor);
   }