opf/openproject

File storage.component.ts has 475 lines of code (exceeds 250 allowed). Consider refactoring.

Open

// -- copyright
// OpenProject is an open source project management software.
// Copyright (C) 2012-2024 the OpenProject GmbH
//
// This program is free software; you can redistribute it and/or

StorageComponent has 26 functions (exceeds 20 allowed). Consider refactoring.

Open

@Component({
  selector: 'op-storage',
  templateUrl: './storage.component.html',
  changeDetection: ChangeDetectionStrategy.OnPush,
  providers: [{ provide: OpUploadService, useClass: StorageUploadService }],

Function uploadAndCreateFileLink has 48 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  private uploadAndCreateFileLink(data:UploadData):void {
    let isUploadError = false;

    this.storage
      .pipe(

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

Open

  ngOnInit():void {
    this.storage = this.storagesResourceService.requireEntity(this.projectStorage._links.storage.href);

    this.fileLinks = this.collectionKey()
      .pipe(

Function handleUploadError has 28 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  private handleUploadError(error:HttpErrorResponse, fileName:string):void {
    if (error.status === 500 && (error.error as IHalErrorBase).errorIdentifier === v3ErrorIdentifierMissingEnterpriseToken) {
      this.toastService.addError(error);
      return;
    }

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

Open

  constructor(
    private readonly i18n:I18nService,
    private readonly cdRef:ChangeDetectorRef,
    private readonly toastService:ToastService,
    private readonly uploadService:OpUploadService,

• Read up
  Read up

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

Open

  ngOnDestroy():void {
    document.body.removeEventListener('dragenter', this.onGlobalDragEnter);
    document.body.removeEventListener('dragleave', this.onGlobalDragLeave);
    document.body.removeEventListener('dragend', this.onGlobalDragEnd);
    document.body.removeEventListener('drop', this.onGlobalDragEnd);

• Read up
  Read up

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

Open

  private onGlobalDragLeave:(_event:DragEvent) => void = (_event) => {
    this.dragging = Math.max(this.dragging - 1, 0);
    this.cdRef.detectChanges();
  };

• Read up
  Read up

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

Open

  private hasFileLinkViewErrors(fileLinks:IFileLink[]):boolean {
    return fileLinks.filter((fileLink) => fileLink._links.status?.href === fileLinkStatusError).length > 0;
  }

• Read up
  Read up

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

Open

  private onGlobalDragEnter:(_event:DragEvent) => void = (_event) => {
    // When the global drag and drop is active and the dragging happens over the DOM
    // elements, the dragenter and dragleave events are always fired in pairs.
    // On dragenter the this.dragging is set to 2 and on dragleave we deduct it to 1,
    // meaning the drag and drop remains active. When the drag and drop action is canceled

• Read up
  Read up

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

Open

    if (event.dataTransfer !== null && containsFiles(event.dataTransfer)) {
      // eslint-disable-next-line no-param-reassign
      event.dataTransfer.dropEffect = 'copy';
      this.draggingOverDropZone = true;
    }

• Read up
  Read up

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

Open

  private onGlobalDragEnd:(_event:DragEvent) => void = (_event) => {
    this.dragging = 0;
    this.cdRef.detectChanges();
  };

• Read up
  Read up

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

Open

      linkingAfterUploadFailed:
        (fileName:string, workPackageId:string):string => this.i18n.t(
          'js.storages.file_links.link_uploaded_file_error',
          {
            fileName,

• Read up
  Read up

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

Open

      uploadFailedSizeLimit:
        (fileName:string, storageType:string):string => this.i18n.t(
          'js.storages.file_links.upload_error.413',
          {
            fileName,

• Read up
  Read up