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

from @art-lowel: update onSelectChange to work with ngModel

Browse Source 
Because we're working with ngModel, we're no longer getting a generic JS event in the onSelectChange, but simply the id of the selected option. I use that Id to find the option in the allOptions array that matches, and then use the data inside that matching option to navigate to the correct page.
This commit is contained in:
L. Henze
2019-10-08 12:40:57 -04:00
parent be19a2a961
commit c46480c97f
1 changed files with 4 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
src/app/shared/comcol-page-browse-by/comcol-page-browse-by.component.ts
View File

@@ -69,17 +69,9 @@ export class ComcolPageBrowseByComponent implements OnInit {
map((urlSegments: UrlSegment[]) => urlSegments[urlSegments.length - 1].path)
);
}
onSelectChange(target) {
const optionIndex = target.selectedIndex;
const selectedOptionElement = target.options[optionIndex];
const paramsAttribute = selectedOptionElement.getAttribute('data-params');
console.log('change value ' + target.value + ' paramsAttribute ' + paramsAttribute);
if (paramsAttribute) {
/* console.log('Yes paramsAttribute ' + paramsAttribute);*/
this.router.navigate([target.value], { queryParams: { scope: paramsAttribute } });
} else {
/* console.log('No paramsAttribute ');*/
this.router.navigate([target.value]);
}
onSelectChange(newId: string) {
const selectedOption = this.allOptions
.find((option: ComColPageNavOption) => option.id === newId);
this.router.navigate([selectedOption.routerLink], { queryParams: selectedOption.params });
}
}