Fix: FormArrays were not getting cleared and were basically fucking the whole thing up.

Fix: EmployeeAttendance was borking because the inital employee was null.

overlord/src/app/attendance/attendance.component.ts

@@ -57,7 +57,7 @@ export class AttendanceComponent implements OnInit {
       this.info = data.info;
       this.attendanceTypes = data.attendanceTypes;
       this.form.controls.date.setValue(moment(this.info.date, 'DD-MMM-YYYY').toDate());
-      this.form.controls.attendances.reset();
+      this.form.controls.attendances.clear();
       this.info.body.forEach((x) =>
         this.form.controls.attendances.push(
           new FormGroup({

overlord/src/app/closing-stock/closing-stock.component.ts

@@ -81,6 +81,7 @@ export class ClosingStockComponent implements OnInit {
         date: moment(this.info.date, 'DD-MMM-YYYY').toDate(),
         costCentre: this.info.costCentre.id,
       });
+      this.form.controls.stocks.clear();
       this.info.items.forEach((x) =>
         this.form.controls.stocks.push(
           new FormGroup({

overlord/src/app/employee-attendance/employee-attendance.component.ts

@@ -79,8 +79,8 @@ export class EmployeeAttendanceComponent implements OnInit, AfterViewInit {
       this.attendanceTypes = data.attendanceTypes;
       this.form.controls.startDate.setValue(moment(this.info.startDate, 'DD-MMM-YYYY').toDate());
       this.form.controls.finishDate.setValue(moment(this.info.finishDate, 'DD-MMM-YYYY').toDate());
-      this.form.controls.employee.setValue(this.info.employee.name);
-      this.form.controls.attendances.reset();
+      this.form.controls.employee.setValue(this.info.employee?.name ?? '');
+      this.form.controls.attendances.clear();
       this.info.body.forEach((x) =>
         this.form.controls.attendances.push(
           new FormGroup({
@@ -120,12 +120,14 @@ export class EmployeeAttendanceComponent implements OnInit, AfterViewInit {
     const formValue = this.form.value;
     this.info.startDate = moment(formValue.startDate).format('DD-MMM-YYYY');
     this.info.finishDate = moment(formValue.finishDate).format('DD-MMM-YYYY');
-    this.router.navigate(['/employee-attendance', this.info.employee.id], {
-      queryParams: {
-        startDate: this.info.startDate,
-        finishDate: this.info.finishDate,
-      },
-    });
+    if (this.info.employee) {
+      this.router.navigate(['/employee-attendance', this.info.employee.id], {
+        queryParams: {
+          startDate: this.info.startDate,
+          finishDate: this.info.finishDate,
+        },
+      });
+    }
   }
 
   save() {

overlord/src/app/employee-attendance/employee-attendance.service.ts

@@ -1,6 +1,6 @@
 import { HttpClient, HttpParams } from '@angular/common/http';
 import { Injectable } from '@angular/core';
-import { Observable } from 'rxjs/internal/Observable';
+import { Observable, of as observableOf } from 'rxjs';
 import { catchError } from 'rxjs/operators';
 
 import { ErrorLoggerService } from '../core/error-logger.service';
@@ -35,6 +35,9 @@ export class EmployeeAttendanceService {
   }
 
   save(employeeAttendance: EmployeeAttendance): Observable<EmployeeAttendance> {
+    if (!employeeAttendance.employee) {
+      return observableOf(new EmployeeAttendance());
+    }
     const { id } = employeeAttendance.employee;
     return this.http
       .post<EmployeeAttendance>(`${url}/${id}`, employeeAttendance)

overlord/src/app/employee-attendance/employee-attendance.ts

@@ -5,13 +5,12 @@ import { EmployeeAttendanceItem } from './employee-attendance-item';
 export class EmployeeAttendance {
   startDate: string;
   finishDate: string;
-  employee: Employee;
+  employee?: Employee;
   body: EmployeeAttendanceItem[];
 
   public constructor(init?: Partial<EmployeeAttendance>) {
     this.startDate = '';
     this.finishDate = '';
-    this.employee = new Employee();
     this.body = [];
     Object.assign(this, init);
   }

overlord/src/app/incentive/incentive.component.ts

@@ -77,6 +77,7 @@ export class IncentiveComponent implements OnInit {
   loadVoucher(voucher: Voucher) {
     this.voucher = voucher;
     this.form.controls.date.setValue(moment(this.voucher.date, 'DD-MMM-YYYY').toDate());
+    this.form.controls.incentives.clear();
     this.voucher.incentives.forEach((x) =>
       this.form.controls.incentives.push(
         new FormGroup({

overlord/src/app/role/role-detail/role-detail.component.ts

@@ -45,7 +45,7 @@ export class RoleDetailComponent implements OnInit, AfterViewInit {
       this.item = data.item;
 
       this.form.controls.name.setValue(this.item.name);
-      this.form.controls.permissions.reset();
+      this.form.controls.permissions.clear();
       this.item.permissions.forEach((x) =>
         this.form.controls.permissions.push(
           new FormGroup({

overlord/src/app/settings/settings.component.ts

@@ -111,7 +111,7 @@ export class SettingsComponent implements OnInit {
 
   showLockInformation(info: LockInfo[]) {
     this.lockInformation = info;
-    this.lockInfoForm.controls.accountTypes.reset();
+    this.lockInfoForm.controls.accountTypes.clear();
     this.accountTypes.forEach((x) =>
       this.lockInfoForm.controls.accountTypes.push(
         new FormGroup({
@@ -119,7 +119,7 @@ export class SettingsComponent implements OnInit {
         }),
       ),
     );
-    this.lockInfoForm.controls.voucherTypes.reset();
+    this.lockInfoForm.controls.voucherTypes.clear();
     this.voucherTypes.forEach((x) =>
       this.lockInfoForm.controls.voucherTypes.push(
         new FormGroup({

overlord/src/app/user/user-detail/user-detail.component.ts

@@ -58,6 +58,7 @@ export class UserDetailComponent implements OnInit, AfterViewInit {
     this.form.controls.name.setValue(item.name);
     this.form.controls.password.setValue(null);
     this.form.controls.lockedOut.setValue(item.lockedOut);
+    this.form.controls.roles.clear();
     this.item.roles.forEach((x) =>
       this.form.controls.roles.push(
         new FormGroup({