Browse Source

fix(client): prevent memory leak

pull/6685/head
Kenrick Tandrian 2 months ago
parent
commit
078977de66
  1. 35
      apps/client/src/app/components/user-account-access/create-or-update-access-dialog/create-or-update-access-dialog.component.ts

35
apps/client/src/app/components/user-account-access/create-or-update-access-dialog/create-or-update-access-dialog.component.ts

@ -83,22 +83,25 @@ export class GfCreateOrUpdateAccessDialogComponent implements OnInit {
]
});
this.accessForm.get('type')?.valueChanges.subscribe((accessType) => {
const granteeUserIdControl = this.accessForm.get('granteeUserId');
const permissionsControl = this.accessForm.get('permissions');
if (accessType === 'PRIVATE') {
granteeUserIdControl?.setValidators(Validators.required);
} else {
granteeUserIdControl?.clearValidators();
granteeUserIdControl?.setValue(null);
permissionsControl?.setValue(this.data.access.permissions[0]);
}
granteeUserIdControl?.updateValueAndValidity();
this.changeDetectorRef.markForCheck();
});
this.accessForm
.get('type')
?.valueChanges.pipe(takeUntilDestroyed(this.destroyRef))
.subscribe((accessType) => {
const granteeUserIdControl = this.accessForm.get('granteeUserId');
const permissionsControl = this.accessForm.get('permissions');
if (accessType === 'PRIVATE') {
granteeUserIdControl?.setValidators(Validators.required);
} else {
granteeUserIdControl?.clearValidators();
granteeUserIdControl?.setValue(null);
permissionsControl?.setValue(this.data.access.permissions[0]);
}
granteeUserIdControl?.updateValueAndValidity();
this.changeDetectorRef.markForCheck();
});
}
public onCancel() {

Loading…
Cancel
Save