[ADF-1677] fix issues with paging (#2446)

* fix issues with paging

* update unit test
This commit is contained in:
Denys Vuika
2017-10-06 17:17:34 +01:00
committed by Eugenio Romano
parent 48f0ccfc4f
commit 7894d687ab
2 changed files with 8 additions and 4 deletions

View File

@@ -975,8 +975,10 @@ describe('DocumentList', () => {
it('should reset pagination when switching sources', () => {
spyOn(documentList, 'resetPagination').and.callThrough();
documentList.loadFolderByNodeId('-trashcan-');
documentList.loadFolderByNodeId('-sites-');
documentList.ngOnChanges({currentFolderId: new SimpleChange(null, '-trashcan-', false)});
documentList.ngOnChanges({currentFolderId: new SimpleChange(null, '-sites-', false)});
expect(documentList.resetPagination).toHaveBeenCalledTimes(2);
});
});

View File

@@ -242,6 +242,9 @@ export class DocumentListComponent implements OnInit, OnChanges, AfterContentIni
if (changes.folderNode && changes.folderNode.currentValue) {
this.loadFolder();
} else if (changes.currentFolderId && changes.currentFolderId.currentValue) {
if (changes.currentFolderId.previousValue !== changes.currentFolderId.currentValue) {
this.resetPagination();
}
if (!this.hasCustomLayout) {
this.setupDefaultColumns(changes.currentFolderId.currentValue);
}
@@ -404,9 +407,7 @@ export class DocumentListComponent implements OnInit, OnChanges, AfterContentIni
// gets folder node and its content
loadFolderByNodeId(nodeId: string) {
this.loading = true;
this.resetSelection();
this.resetPagination();
if (nodeId === '-trashcan-') {
this.loadTrashcan();
@@ -423,6 +424,7 @@ export class DocumentListComponent implements OnInit, OnChanges, AfterContentIni
.getFolderNode(nodeId).then(node => {
this.folderNode = node;
this.currentFolderId = node.id;
this.skipCount = 0;
this.currentNodeAllowableOperations = node['allowableOperations'] ? node['allowableOperations'] : [];
this.loadFolderNodesByFolderNodeId(node.id, this.pageSize, this.skipCount).catch(err => this.error.emit(err));
})