#150 SortOptions config fixing

src/app/+community-page/sub-collection-list/community-page-sub-collection-list.component.html

@@ -2,10 +2,9 @@
   <div *ngIf="subCollectionsRD?.hasSucceeded " @fadeIn>
   <h2>{{'community.sub-collection-list.head' | translate}}</h2>
   <ul>
-      <ds-viewable-collection [objects]="subCollectionsRD?.payload">
+    <ds-viewable-collection [objects]="subCollectionsRD?.payload"
-      </ds-viewable-collection>
+    [sortConfig]="sortConfig"></ds-viewable-collection>
   </ul>
-  </div>
+</div><ds-error *ngIf="subCollectionsRD?.hasFailed" message="{{'error.sub-collections' | translate}}"></ds-error>
-  <ds-error *ngIf="subCollectionsRD?.hasFailed" message="{{'error.sub-collections' | translate}}"></ds-error>
   <ds-loading *ngIf="subCollectionsRD?.isLoading" message="{{'loading.sub-collections' | translate}}"></ds-loading>
 </ng-container>

src/app/+home-page/top-level-community-list/top-level-community-list.component.html

@@ -4,8 +4,8 @@
   <p class="lead">{{'home.top-level-communities.help' | translate}}</p>
   <ds-viewable-collection
     [config]="config"
+    [sortConfig]="sortConfig"
     [objects]="communitiesRD"
-
     (paginationChange)="updatePage($event)">
   </ds-viewable-collection>
 </div>

src/app/+search-page/search-page.component.html

@@ -30,7 +30,7 @@
               </button>
             </div>
             <ds-search-results [searchResults]="resultsRDObs | async"
-                               [searchConfig]="searchOptions"></ds-search-results>
+                               [searchConfig]="searchOptions" [sortConfig]="sortConfig"></ds-search-results>
           </div>
         </div>
       </div>

src/app/+search-page/search-page.component.ts

@@ -6,7 +6,7 @@ import { RemoteData } from '../core/data/remote-data';
 import { Community } from '../core/shared/community.model';
 import { DSpaceObject } from '../core/shared/dspace-object.model';
 import { isNotEmpty } from '../shared/empty.util';
-import { SearchOptions } from './search-options.model';
+import { SearchOptions,ViewMode } from './search-options.model';
 import { SearchResult } from './search-result.model';
 import { SearchService } from './search-service/search.service';
 import { pushInOut } from '../shared/animations/push';
@@ -36,6 +36,7 @@ export class SearchPageComponent implements OnInit, OnDestroy {
   resultsRDObs: Observable<RemoteData<Array<SearchResult<DSpaceObject>>>>;
   currentParams = {};
   searchOptions: SearchOptions;
+  sortConfig: SortOptions;
   scopeListRDObs: Observable<RemoteData<Community[]>>;
   isMobileView: Observable<boolean>;
 
@@ -58,15 +59,15 @@ export class SearchPageComponent implements OnInit, OnDestroy {
 
     // TODO Update to accommodate view switcher
     this.route.queryParams.map((params) => {
-      if (isNotEmpty(params.view) && params.view == 'grid') {
+      if (isNotEmpty(params.view) && params.view == ViewMode.Grid) {
         pagination.pageSize = 12;
       }
     });
 
 
     const sort: SortOptions = new SortOptions();
+    this.sortConfig=sort;
     this.searchOptions = this.service.searchOptions;
-
   }
 
   ngOnInit(): void {

src/app/+search-page/search-results/search-results.component.ts

@@ -3,6 +3,7 @@ import { RemoteData } from '../../core/data/remote-data';
 import { DSpaceObject } from '../../core/shared/dspace-object.model';
 import { fadeIn, fadeInOut } from '../../shared/animations/fade';
 import { SearchOptions } from '../search-options.model';
+import { SortOptions } from '../../core/cache/models/sort-options.model';
 import { SearchResult } from '../search-result.model';
 
 /**
@@ -21,4 +22,5 @@ import { SearchResult } from '../search-result.model';
 export class SearchResultsComponent {
   @Input() searchResults: RemoteData<Array<SearchResult<DSpaceObject>>>;
   @Input() searchConfig: SearchOptions;
+  @Input() sortConfig: SortOptions;
 }

src/app/object-collection/object-collection.component.html

@@ -1,13 +1,13 @@
 <ds-object-list [config]="config"
                 [sortConfig]="sortConfig"
                 [objects]="objects"
-                *ngIf="getViewMode()=='list'">
+                *ngIf="getViewMode()===viewModeEnum.List">
 </ds-object-list>
 
 <ds-object-grid [config]="config"
                 [sortConfig]="sortConfig"
                 [objects]="objects"
-                *ngIf="getViewMode()=='grid'">
+                *ngIf="getViewMode()===viewModeEnum.Grid">
 </ds-object-grid>
 
 

src/app/object-collection/object-collection.component.ts

@@ -11,10 +11,12 @@ import { PageInfo } from '../core/shared/page-info.model';
 
 import { PaginationComponentOptions } from '../shared/pagination/pagination-component-options.model';
 
-import {  SortDirection } from '../core/cache/models/sort-options.model';
+import { SortDirection, SortOptions } from '../core/cache/models/sort-options.model';
 
 import { ListableObject } from './shared/listable-object.model';
+import { ViewMode } from '../+search-page/search-options.model';
 import { hasValue, isNotEmpty } from '../shared/empty.util';
+
 @Component({
   selector: 'ds-viewable-collection',
   styleUrls: ['./object-collection.component.scss'],
@@ -24,6 +26,7 @@ export class ObjectCollectionComponent implements OnChanges, OnInit {
 
   @Input() objects: RemoteData<ListableObject[]>;
   @Input() config?: PaginationComponentOptions;
+  @Input() sortConfig: SortOptions;
   pageInfo: Observable<PageInfo>;
 
   /**
@@ -52,7 +55,8 @@ export class ObjectCollectionComponent implements OnChanges, OnInit {
    */
   @Output() sortFieldChange: EventEmitter<string> = new EventEmitter<string>();
   data: any = {};
-  defaultViewMode: string ='list';
+  currentMode: ViewMode = ViewMode.List;
+  viewModeEnum = ViewMode;
 
   ngOnChanges(changes: SimpleChanges) {
     if (changes.objects && !changes.objects.isFirstChange()) {
@@ -60,6 +64,7 @@ export class ObjectCollectionComponent implements OnChanges, OnInit {
     }
   }
 
+
   ngOnInit(): void {
     this.pageInfo = this.objects.pageInfo;
   }
@@ -74,22 +79,15 @@ export class ObjectCollectionComponent implements OnChanges, OnInit {
               private router: Router) {
   }
 
-  getViewMode(): string {
+  getViewMode(): ViewMode {
-    // TODO Update to accommodate view switcher
-
     this.route.queryParams.map((params) => {
       if (isNotEmpty(params.view) && hasValue(params.view)) {
-        return params.view;
+        this.currentMode= params.view;
-      } else {
-        return this.defaultViewMode;
       }
     });
-    return this.defaultViewMode;
+    return this.currentMode;
   }
 
-  setViewMode(viewMode: string) {
-    this.defaultViewMode = viewMode;
-  }
   onPageChange(event) {
     this.pageChange.emit(event);
   }

src/app/object-grid/object-grid.component.ts

@@ -82,5 +82,24 @@ export class ObjectGridComponent implements OnChanges, OnInit  {
   constructor(private cdRef: ChangeDetectorRef) {
   }
 
+  onPageChange(event) {
+    this.pageChange.emit(event);
+  }
+
+  onPageSizeChange(event) {
+    this.pageSizeChange.emit(event);
+  }
+
+  onSortDirectionChange(event) {
+    this.sortDirectionChange.emit(event);
+  }
+
+  onSortFieldChange(event) {
+    this.sortFieldChange.emit(event);
+  }
+
+  onPaginationChange(event) {
+    this.paginationChange.emit(event);
+  }
 
 }

src/app/object-grid/search-result-grid-element/item-search-result/item-search-result-grid-element.component.html

@@ -20,8 +20,8 @@
     <p *ngIf="dso.findMetadata('dc.description.abstract')" class="item-abstract card-text">
       {{dso.findMetadata("dc.description.abstract") | dsTruncate:[200] }}</p>
 
-  </div>
     <a [routerLink]="['/items/' + dso.id]" class="lead btn btn-primary viewButton">View</a>
+  </div>
 
 </div>
 

src/app/object-list/object-list.component.ts

@@ -103,4 +103,23 @@ export class ObjectListComponent {
     this.paginationChange.emit(event);
   }
 
+  onPageChange(event) {
+    this.pageChange.emit(event);
+  }
+
+  onPageSizeChange(event) {
+    this.pageSizeChange.emit(event);
+  }
+
+  onSortDirectionChange(event) {
+    this.sortDirectionChange.emit(event);
+  }
+
+  onSortFieldChange(event) {
+    this.sortFieldChange.emit(event);
+  }
+
+  onPaginationChange(event) {
+    this.paginationChange.emit(event);
+  }
 }