Improve documentation

This commit is contained in:
Hrafn Malmquist
2023-08-22 04:29:01 +01:00
parent 5f71de885b
commit d9b6e9d81f
3 changed files with 23 additions and 10 deletions

View File

@@ -25,7 +25,7 @@ import { ShowMoreFlatNode } from './show-more-flat-node.model';
import { FindListOptions } from '../core/data/find-list-options.model'; import { FindListOptions } from '../core/data/find-list-options.model';
import { AppConfig, APP_CONFIG } from 'src/config/app-config.interface'; import { AppConfig, APP_CONFIG } from 'src/config/app-config.interface';
// Helper method to combine an flatten an array of observables of flatNode arrays // Helper method to combine and flatten an array of observables of flatNode arrays
export const combineAndFlatten = (obsList: Observable<FlatNode[]>[]): Observable<FlatNode[]> => export const combineAndFlatten = (obsList: Observable<FlatNode[]>[]): Observable<FlatNode[]> =>
observableCombineLatest([...obsList]).pipe( observableCombineLatest([...obsList]).pipe(
map((matrix: any[][]) => [].concat(...matrix)), map((matrix: any[][]) => [].concat(...matrix)),
@@ -199,7 +199,7 @@ export class CommunityListService {
* Transforms a community in a list of FlatNodes containing firstly a flatnode of the community itself, * Transforms a community in a list of FlatNodes containing firstly a flatnode of the community itself,
* followed by flatNodes of its possible subcommunities and collection * followed by flatNodes of its possible subcommunities and collection
* It gets called recursively for each subcommunity to add its subcommunities and collections to the list * It gets called recursively for each subcommunity to add its subcommunities and collections to the list
* Number of subcommunities and collections added, is dependant on the current page the parent is at for respectively subcommunities and collections. * Number of subcommunities and collections added, is dependent on the current page the parent is at for respectively subcommunities and collections.
* @param community Community being transformed * @param community Community being transformed
* @param level Depth of the community in the list, subcommunities and collections go one level deeper * @param level Depth of the community in the list, subcommunities and collections go one level deeper
* @param parent Flatnode of the parent community * @param parent Flatnode of the parent community
@@ -275,7 +275,7 @@ export class CommunityListService {
/** /**
* Checks if a community has subcommunities or collections by querying the respective services with a pageSize = 0 * Checks if a community has subcommunities or collections by querying the respective services with a pageSize = 0
* Returns an observable that combines the result.payload.totalElements fo the observables that the * Returns an observable that combines the result.payload.totalElements of the observables that the
* respective services return when queried * respective services return when queried
* @param community Community being checked whether it is expandable (if it has subcommunities or collections) * @param community Community being checked whether it is expandable (if it has subcommunities or collections)
*/ */

View File

@@ -59,18 +59,28 @@ export class CommunityListComponent implements OnInit, OnDestroy {
this.communityListService.saveCommunityListStateToStore(this.expandedNodes, this.loadingNode); this.communityListService.saveCommunityListStateToStore(this.expandedNodes, this.loadingNode);
} }
// whether or not this node has children (subcommunities or collections) /**
* Whether this node has children (subcommunities or collections)
* @param _
* @param node
*/
hasChild(_: number, node: FlatNode) { hasChild(_: number, node: FlatNode) {
return node.isExpandable$; return node.isExpandable$;
} }
// whether or not it is a show more node (contains no data, but is indication that there are more topcoms, subcoms or collections /**
* Whether this is a show more node that contains no data, but indicates that there is
* one or more community or collection.
* @param _
* @param node
*/
isShowMore(_: number, node: FlatNode) { isShowMore(_: number, node: FlatNode) {
return node.isShowMoreNode; return node.isShowMoreNode;
} }
/** /**
* Toggles the expanded variable of a node, adds it to the expanded nodes list and reloads the tree so this node is expanded * Toggles the expanded variable of a node, adds it to the expanded nodes list and reloads the tree
* so this node is expanded
* @param node Node we want to expand * @param node Node we want to expand
*/ */
toggleExpanded(node: FlatNode) { toggleExpanded(node: FlatNode) {
@@ -93,9 +103,12 @@ export class CommunityListComponent implements OnInit, OnDestroy {
/** /**
* Makes sure the next page of a node is added to the tree (top community, sub community of collection) * Makes sure the next page of a node is added to the tree (top community, sub community of collection)
* > Finds its parent (if not top community) and increases its corresponding collection/subcommunity currentPage * > Finds its parent (if not top community) and increases its corresponding collection/subcommunity
* > Reloads tree with new page added to corresponding top community lis, sub community list or collection list * currentPage
* @param node The show more node indicating whether it's an increase in top communities, sub communities or collections * > Reloads tree with new page added to corresponding top community lis, sub community list or
* collection list
* @param node The show more node indicating whether it's an increase in top communities, sub communities
* or collections
*/ */
getNextPage(node: FlatNode): void { getNextPage(node: FlatNode): void {
this.loadingNode = node; this.loadingNode = node;

View File

@@ -1,6 +1,6 @@
/** /**
* The show more links in the community tree are also represented by a flatNode so we know where in * The show more links in the community tree are also represented by a flatNode so we know where in
* the tree it should be rendered an who its parent is (needed for the action resulting in clicking this link) * the tree it should be rendered and who its parent is (needed for the action resulting in clicking this link)
*/ */
export class ShowMoreFlatNode { export class ShowMoreFlatNode {
} }