101623: Add missing docs + Fix lint issues

Author: nona-luypaert

src/app/browse-by/browse-by-routing.module.ts

@@ -4,8 +4,6 @@ import { BrowseByGuard } from './browse-by-guard';
 import { BrowseByDSOBreadcrumbResolver } from './browse-by-dso-breadcrumb.resolver';
 import { BrowseByI18nBreadcrumbResolver } from './browse-by-i18n-breadcrumb.resolver';
 import { ThemedBrowseBySwitcherComponent } from './browse-by-switcher/themed-browse-by-switcher.component';
-import { ThemedBrowseByTaxonomyPageComponent } from './browse-by-taxonomy-page/themed-browse-by-taxonomy-page.component';
-import { I18nBreadcrumbResolver } from '../core/breadcrumbs/i18n-breadcrumb.resolver';
 
 @NgModule({
   imports: [

src/app/core/data/browse-response-parsing.service.ts

@@ -15,7 +15,7 @@ import { ValueListBrowseDefinition } from '../shared/value-list-browse-definitio
 import { VALUE_LIST_BROWSE_DEFINITION } from '../shared/value-list-browse-definition.resource-type';
 
 /**
- * A ResponseParsingService used to parse RawRestResponse coming from the REST API to a BrowseDefinition object
+ * A ResponseParsingService used to parse a REST API response to a BrowseDefinition object
  */
 @Injectable({
   providedIn: 'root',

src/app/core/data/request.models.ts

@@ -119,6 +119,9 @@ export class PatchRequest extends DSpaceRestRequest {
   }
 }
 
+/**
+ * Class representing a BrowseDefinition HTTP Rest request object
+ */
 export class BrowseDefinitionRestRequest extends DSpaceRestRequest {
   getResponseParser(): GenericConstructor<ResponseParsingService> {
     return BrowseResponseParsingService;

src/app/core/shared/flat-browse-definition.model.ts

@@ -5,6 +5,9 @@ import { FLAT_BROWSE_DEFINITION } from './flat-browse-definition.resource-type';
 import { ResourceType } from './resource-type';
 import { NonHierarchicalBrowseDefinition } from './non-hierarchical-browse-definition';
 
+/**
+ * BrowseDefinition model for browses of type 'flatBrowse'
+ */
 @typedObject
 @inheritSerialization(NonHierarchicalBrowseDefinition)
 export class FlatBrowseDefinition extends NonHierarchicalBrowseDefinition {

src/app/core/shared/hierarchical-browse-definition.model.ts

@@ -6,6 +6,9 @@ import { HALLink } from './hal-link.model';
 import { ResourceType } from './resource-type';
 import { BrowseDefinition } from './browse-definition.model';
 
+/**
+ * BrowseDefinition model for browses of type 'hierarchicalBrowse'
+ */
 @typedObject
 @inheritSerialization(BrowseDefinition)
 export class HierarchicalBrowseDefinition extends BrowseDefinition {

src/app/core/shared/value-list-browse-definition.model.ts

@@ -5,6 +5,9 @@ import { VALUE_LIST_BROWSE_DEFINITION } from './value-list-browse-definition.res
 import { ResourceType } from './resource-type';
 import { NonHierarchicalBrowseDefinition } from './non-hierarchical-browse-definition';
 
+/**
+ * BrowseDefinition model for browses of type 'valueList'
+ */
 @typedObject
 @inheritSerialization(NonHierarchicalBrowseDefinition)
 export class ValueListBrowseDefinition extends NonHierarchicalBrowseDefinition {

src/app/shared/form/vocabulary-treeview/vocabulary-treeview.component.ts

@@ -248,7 +248,7 @@ export class VocabularyTreeviewComponent implements OnDestroy, OnInit {
   }
 
   /**
-   * Method called on entry select
+   * Method called on entry select/deselect
    */
   onSelect(item: VocabularyEntryDetail) {
     if (!this.selectedItems.includes(item.id)) {