open-learning-exchange/planet

File courses.component.ts has 425 lines of code (exceeds 250 allowed). Consider refactoring.

Open

import { Component, OnInit, AfterViewInit, ViewChild, OnDestroy, HostListener, Input, OnChanges } from '@angular/core';
import { CouchService } from '../shared/couchdb.service';
import { DialogsPromptComponent } from '../shared/dialogs/dialogs-prompt.component';
import { MatDialog, MatDialogRef } from '@angular/material/dialog';
import { MatPaginator, PageEvent } from '@angular/material/paginator';

CoursesComponent has 42 functions (exceeds 20 allowed). Consider refactoring.

Open

@Component({
  selector: 'planet-courses',
  templateUrl: './courses.component.html',
  styleUrls: [ './courses.scss' ]
})

Function ngOnInit has 30 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  ngOnInit() {
    this.titleSearch = '';
    this.getCourses();
    this.userShelf = this.userService.shelf;
    this.courses.filterPredicate = this.filterPredicate;

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  sendCourse() {
    return (selected: any) => {
      const coursesToSend = this.selection.selected.map(id => findByIdInArray(this.courses.data, id));
      this.syncService.createChildPullDoc(coursesToSend, 'courses', selected).subscribe(() => {
        const childType = {

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  openSendCourseDialog() {
    this.dialogsListService.getListAndColumns('communityregistrationrequests', { 'registrationRequest': 'accepted' })
    .pipe(takeUntil(this.onDestroy$))
    .subscribe((planet) => {
      const data = { okClick: this.sendCourse().bind(this),

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  masterToggle() {
    const start = this.paginator.pageIndex * this.paginator.pageSize;
    const end = start + this.paginator.pageSize;
    if (this.isAllSelected()) {
      this.selection.clear();

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Wontfix

  openDeleteDialog(okClick, amount, displayName = '', count) {
    this.deleteDialog = this.dialog.open(DialogsPromptComponent, {
      data: {
        okClick,
        amount,

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

  onSearchChange({ items, category }) {
    this.searchSelection[category] = items;
    this.searchSelection._empty = Object.entries(this.searchSelection).every(
      ([ field, val ]: any[]) => !Array.isArray(val) || val.length === 0
    );

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

  constructor(
    private couchService: CouchService,
    private coursesService: CoursesService,
    private dialog: MatDialog,
    private dialogsListService: DialogsListService,

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

  resetFilter() {
    this.tagFilter.setValue([]);
    this.tagFilterValue = [];
    Object.keys(this.searchSelection).forEach(key => this.searchSelection[key] = []);
    if (this.searchComponent) {

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  recordSearch(complete = false) {
    if (this.courses.filter !== '') {
      this.searchService.recordSearch({
        text: this._titleSearch,
        type: this.dbName,

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Wontfix

  ngAfterViewInit() {
    this.courses.sort = this.sort;
    this.courses.paginator = this.paginator;
    if (this.tagInputComponent) {
      this.tagInputComponent.addTags(this.route.snapshot.paramMap.get('collections'));

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  isAllSelected() {
    const itemsShown = Math.min(this.paginator.length - (this.paginator.pageIndex * this.paginator.pageSize), this.paginator.pageSize);
    return this.selection.selected.length === itemsShown;
  }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Wontfix

  set titleSearch(value: string) {
    // When setting the titleSearch, also set the courses filter
    this.courses.filter = value ? value : this.dropdownsFill();
    this._titleSearch = value;
    this.recordSearch();

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Wontfix

  removeFilteredFromSelection() {
    this.selection.deselect(...selectedOutOfFilter(this.courses.filteredData, this.selection, this.paginator));
    this.emptyData = this.courses.filteredData.length === 0;
  }

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

    return this.tagFilter.value.length > 0 ||
      Object.entries(this.searchSelection).findIndex(([ field, val ]: any[]) => val.length > 0) > -1 ||
      this.myCoursesFilter.value === 'on' ||

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

    this.tagFilter.valueChanges.subscribe((tags) => {
      this.tagFilterValue = tags;
      this.titleSearch = this.titleSearch;
      this.removeFilteredFromSelection();
    });

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

  openCourseViewDialog(courseId) {
    this.dialog.open(CoursesViewDetailDialogComponent, {
      data: { courseId },
      minWidth: '600px',
      maxWidth: '90vw',

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Wontfix

        this.courses.data = this.courses.data.filter((c: any) => data.id !== c._id);

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    )).subscribe(() => {
      this.planetMessageService.showMessage($localize`${courses.length} ${this.dbName} queued to ${msg}`);
    }, () => error => this.planetMessageService.showMessage(error));

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Wontfix

  ngOnDestroy() {
    this.onDestroy$.next();
    this.onDestroy$.complete();
    this.recordSearch(true);
  }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Wontfix

  updateCourse(course) {
    const { _id: courseId } = course;
    this.router.navigate([ '/courses/update/' + course._id ]);
  }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    this.selection.changed.subscribe(({ source }) => {
      this.countSelectNotEnrolled(source.selected);
    });

• Read up
  Read up