hazza/dspace-angular
1
0
Fork 1
mirror of https://github.com/DSpace/dspace-angular.git synced 2025-10-07 10:04:11 +00:00
Code Issues Packages Projects Releases Wiki Activity

Resolve conflict occurred when two or more pagination components are present in the same page

Browse Source
This commit is contained in:
Giuseppe Digilio
2019-12-23 15:10:08 +01:00
parent 4b67dbf10f
commit ce004c2e58
1 changed files with 28 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
src/app/shared/pagination/pagination.component.ts
View File

@@ -225,10 +225,14 @@ export class PaginationComponent implements OnDestroy, OnInit {
} }
/** /**
* @param cdRef
* ChangeDetectorRef is a singleton service provided by Angular.
* @param route * @param route
* Route is a singleton service provided by Angular. * Route is a singleton service provided by Angular.
* @param router * @param router
* Router is a singleton service provided by Angular. * Router is a singleton service provided by Angular.
* @param hostWindowService
* the HostWindowService singleton.
*/ */
constructor(private cdRef: ChangeDetectorRef, constructor(private cdRef: ChangeDetectorRef,
private route: ActivatedRoute, private route: ActivatedRoute,
@@ -243,7 +247,7 @@ export class PaginationComponent implements OnDestroy, OnInit {
* The page being navigated to. * The page being navigated to.
*/ */
public doPageChange(page: number) { public doPageChange(page: number) {
this.updateRoute({ page: page.toString() }); this.updateRoute({ pageId: this.id, page: page.toString() });
} }
/** /**
@@ -253,7 +257,7 @@ export class PaginationComponent implements OnDestroy, OnInit {
* The page size being navigated to. * The page size being navigated to.
*/ */
public doPageSizeChange(pageSize: number) { public doPageSizeChange(pageSize: number) {
this.updateRoute({ page: 1, pageSize: pageSize }); this.updateRoute({ pageId: this.id, page: 1, pageSize: pageSize });
} }
/** /**
@@ -263,7 +267,7 @@ export class PaginationComponent implements OnDestroy, OnInit {
* The sort direction being navigated to. * The sort direction being navigated to.
*/ */
public doSortDirectionChange(sortDirection: SortDirection) { public doSortDirectionChange(sortDirection: SortDirection) {
this.updateRoute({ page: 1, sortDirection: sortDirection }); this.updateRoute({ pageId: this.id, page: 1, sortDirection: sortDirection });
} }
/** /**
@@ -273,7 +277,7 @@ export class PaginationComponent implements OnDestroy, OnInit {
* The sort field being navigated to. * The sort field being navigated to.
*/ */
public doSortFieldChange(field: string) { public doSortFieldChange(field: string) {
this.updateRoute({ page: 1, sortField: field }); this.updateRoute({ pageId: this.id, page: 1, sortField: field });
} }
/** /**
@@ -413,6 +417,8 @@ export class PaginationComponent implements OnDestroy, OnInit {
* Method to update all pagination variables to the current query parameters * Method to update all pagination variables to the current query parameters
*/ */
private setFields() { private setFields() {
// set fields only when page id is the one configured for this pagination instance
if (this.currentQueryParams.pageId === this.id) {
// (+) converts string to a number // (+) converts string to a number
const page = this.currentQueryParams.page; const page = this.currentQueryParams.page;
if (this.currentPage !== +page) { if (this.currentPage !== +page) {
@@ -435,6 +441,7 @@ export class PaginationComponent implements OnDestroy, OnInit {
} }
this.cdRef.detectChanges(); this.cdRef.detectChanges();
} }
}
/** /**
* Method to validate the current page value * Method to validate the current page value