Merge pull request 'Fix TypeError: Cannot read properties of undefined...

Merge pull request 'Fix TypeError: Cannot read properties of undefined (reading 'ngModule')' (#485) from fix-dialog-fixed into master

Reviewed-on: https://git.ozg-sh.de/ozgcloud-app/alfa/pulls/485


Reviewed-by: OZGCloud <ozgcloud@mgm-tp.com>
Reviewed-by: OZGCloud <ozgcloud@mgm-tp.com>

alfa-client/libs/postfach-shared/src/lib/+state/postfach.facade.ts

@@ -21,7 +21,6 @@
  * Die sprachspezifischen Genehmigungen und Beschränkungen
  * unter der Lizenz sind dem Lizenztext zu entnehmen.
  */
-import { Injectable } from '@angular/core';
 import {
   BinaryFileListResource,
   DownloadBinaryFileAsPdfAction,
@@ -33,6 +32,7 @@ import {
   formatFullDateWithoutSeperator,
 } from '@alfa-client/tech-shared';
 import { VorgangHeaderLinkRel, VorgangWithEingangResource } from '@alfa-client/vorgang-shared';
+import { Injectable } from '@angular/core';
 import { Store } from '@ngrx/store';
 import { TypedAction } from '@ngrx/store/src/models';
 import { getUrl } from '@ngxp/rest';

alfa-client/libs/postfach-shared/src/lib/postfach-shared.module.ts

@@ -21,14 +21,15 @@
  * Die sprachspezifischen Genehmigungen und Beschränkungen
  * unter der Lizenz sind dem Lizenztext zu entnehmen.
  */
+import { CommandSharedModule } from '@alfa-client/command-shared';
 import { CommonModule } from '@angular/common';
 import { NgModule } from '@angular/core';
-import { CommandSharedModule } from '@alfa-client/command-shared';
 import { EffectsModule } from '@ngrx/effects';
 import { StoreModule } from '@ngrx/store';
 import { PostfachEffects } from './+state/postfach.effects';
 import { PostfachFacade } from './+state/postfach.facade';
 import * as fromPostfach from './+state/postfach.reducer';
+import { PostfachService } from './postfach.service';
 
 @NgModule({
   imports: [
@@ -37,6 +38,6 @@ import * as fromPostfach from './+state/postfach.reducer';
     StoreModule.forFeature(fromPostfach.POSTFACH_FEATURE_KEY, fromPostfach.postfachReducer),
     EffectsModule.forFeature([PostfachEffects]),
   ],
-  providers: [PostfachFacade],
+  providers: [PostfachFacade, PostfachService],
 })
 export class PostfachSharedModule {}

alfa-client/libs/postfach-shared/src/lib/postfach.service.ts

@@ -21,9 +21,6 @@
  * Die sprachspezifischen Genehmigungen und Beschränkungen
  * unter der Lizenz sind dem Lizenztext zu entnehmen.
  */
-import { Injectable } from '@angular/core';
-import { MatDialog } from '@angular/material/dialog';
-import { Params } from '@angular/router';
 import {
   BinaryFileListResource,
   BinaryFileResource,
@@ -49,6 +46,9 @@ import {
 } from '@alfa-client/tech-shared';
 import { SnackBarService } from '@alfa-client/ui';
 import { VorgangResource, VorgangService } from '@alfa-client/vorgang-shared';
+import { Injectable } from '@angular/core';
+import { MatDialog } from '@angular/material/dialog';
+import { Params } from '@angular/router';
 import { Resource, hasLink } from '@ngxp/rest';
 import { isNil, isNull } from 'lodash-es';
 import { BehaviorSubject, Observable, Subscription, combineLatest } from 'rxjs';
@@ -66,7 +66,7 @@ import {
 import { PostfachRepository } from './postfach.repository';
 import { createResendPostfachMailCommand, createSendPostfachMailCommand } from './postfach.util';
 
-@Injectable({ providedIn: 'root' })
+@Injectable()
 export class PostfachService {
   private readonly isPollSendPostachMail: BehaviorSubject<boolean> = new BehaviorSubject<boolean>(
     false,

alfa-client/libs/postfach/src/lib/postfach.module.ts

@@ -21,15 +21,14 @@
  * Die sprachspezifischen Genehmigungen und Beschränkungen
  * unter der Lizenz sind dem Lizenztext zu entnehmen.
  */
-import { CommonModule } from '@angular/common';
-import { NgModule } from '@angular/core';
-import { RouterModule, Routes } from '@angular/router';
 import { BinaryFileModule } from '@alfa-client/binary-file';
 import { ON_PAGE, PostfachSharedModule } from '@alfa-client/postfach-shared';
 import { UiModule } from '@alfa-client/ui';
 import { UserProfileModule } from '@alfa-client/user-profile';
-import { VorgangSharedModule } from '@alfa-client/vorgang-shared';
 import { VorgangSharedUiModule } from '@alfa-client/vorgang-shared-ui';
+import { CommonModule } from '@angular/common';
+import { NgModule } from '@angular/core';
+import { RouterModule, Routes } from '@angular/router';
 import { PostfachMailButtonContainerComponent } from './postfach-mail-button-container/postfach-mail-button-container.component';
 import { PostfachMailButtonComponent } from './postfach-mail-button-container/postfach-mail-button/postfach-mail-button.component';
 import { PostfachMailFormComponent } from './postfach-mail-form/postfach-mail-form.component';
@@ -66,7 +65,6 @@ const routes: Routes = [
     UiModule,
     PostfachSharedModule,
     UserProfileModule,
-    VorgangSharedModule,
     VorgangSharedUiModule,
     BinaryFileModule,
   ],

alfa-client/libs/ui/src/lib/ui/fixed-dialog/fixed-dialog-data.model.ts

@@ -22,10 +22,8 @@
  * unter der Lizenz sind dem Lizenztext zu entnehmen.
  */
 import { ComponentType } from '@angular/cdk/overlay';
-import { VorgangWithEingangResource } from '@alfa-client/vorgang-shared';
 
 export interface FixedDialogData {
   title: string;
   component: ComponentType<any>;
-  vorgang?: VorgangWithEingangResource;
 }

alfa-client/libs/ui/src/lib/ui/fixed-dialog/fixed-dialog.component.ts

@@ -21,9 +21,8 @@
  * Die sprachspezifischen Genehmigungen und Beschränkungen
  * unter der Lizenz sind dem Lizenztext zu entnehmen.
  */
-import { Component, Inject, Injectable, Injector } from '@angular/core';
+import { Component, Inject, Injector } from '@angular/core';
 import { MAT_DIALOG_DATA } from '@angular/material/dialog';
-import { VorgangWithEingangResource } from '@alfa-client/vorgang-shared';
 import { FixedDialogData } from './fixed-dialog-data.model';
 
 @Component({
@@ -40,12 +39,7 @@ export class FixedDialogComponent {
     injector: Injector,
   ) {
     this.componentInjector = Injector.create({
-      providers: [
-        {
-          provide: InjectedVorgangInDialog,
-          useValue: new InjectedVorgangInDialog(this.data.vorgang),
-        },
-      ],
+      providers: [],
       parent: injector,
     });
   }
@@ -54,8 +48,3 @@ export class FixedDialogComponent {
     this.isMinimized = !this.isMinimized;
   }
 }
-
-@Injectable()
-export class InjectedVorgangInDialog {
-  constructor(public vorgang: VorgangWithEingangResource) {}
-}

alfa-client/libs/vorgang-shared/src/lib/vorgang-shared.module.ts

@@ -21,14 +21,13 @@
  * Die sprachspezifischen Genehmigungen und Beschränkungen
  * unter der Lizenz sind dem Lizenztext zu entnehmen.
  */
-import { CommonModule } from '@angular/common';
-import { NgModule } from '@angular/core';
-import { RouterModule } from '@angular/router';
 import { BinaryFileSharedModule } from '@alfa-client/binary-file-shared';
 import { CommandSharedModule } from '@alfa-client/command-shared';
 import { NavigationSharedModule } from '@alfa-client/navigation-shared';
-import { PostfachSharedModule } from '@alfa-client/postfach-shared';
 import { TechSharedModule } from '@alfa-client/tech-shared';
+import { CommonModule } from '@angular/common';
+import { NgModule } from '@angular/core';
+import { RouterModule } from '@angular/router';
 import { EffectsModule } from '@ngrx/effects';
 import { StoreModule } from '@ngrx/store';
 import { RestModule } from '@ngxp/rest';
@@ -45,7 +44,6 @@ import * as fromVorgang from './+state/vorgang.reducer';
     TechSharedModule,
     CommandSharedModule,
     BinaryFileSharedModule,
-    PostfachSharedModule,
     RouterModule,
     StoreModule.forFeature(fromVorgang.VORGANG_FEATURE_KEY, fromVorgang.reducer),
     EffectsModule.forFeature([VorgangEffects]),