[CST-16309] update status handling

Author: FrancescoMolinaro

src/app/core/config/models/config-submission-access.model.ts

@@ -37,6 +37,12 @@ export class SubmissionAccessModel extends ConfigObject {
   @autoserialize
   singleAccessCondition: boolean;
 
+  /**
+   * Whether the field is mandatory
+   */
+  @autoserialize
+  required: boolean;
+
   /**
    * The links to all related resources returned by the rest api.
    */

src/app/submission/sections/accesses/section-accesses.component.ts

@@ -3,7 +3,7 @@ import { Component, Inject, ViewChild } from '@angular/core';
 import { UntypedFormControl } from '@angular/forms';
 
 import { filter, map, mergeMap, take } from 'rxjs/operators';
-import { combineLatest, Observable, of, Subscription } from 'rxjs';
+import { BehaviorSubject, combineLatest, Observable, Subscription } from 'rxjs';
 import { TranslateService } from '@ngx-translate/core';
 
 import { renderSectionFor } from '../sections-decorator';
@@ -42,7 +42,7 @@ import {
   FORM_ACCESS_CONDITION_TYPE_CONFIG,
   FORM_ACCESS_CONDITION_TYPE_LAYOUT
 } from './section-accesses.model';
-import { hasValue, isNotEmpty, isNotNull } from '../../../shared/empty.util';
+import { hasValue, isNotEmpty, isNotNull, isObjectEmpty } from '../../../shared/empty.util';
 import {
   WorkspaceitemSectionAccessesObject
 } from '../../../core/submission/models/workspaceitem-section-accesses.model';
@@ -124,6 +124,11 @@ export class SubmissionSectionAccessesComponent extends SectionModelComponent {
    */
   public canChangeDiscoverable: boolean;
 
+  /**
+   * Whether the section is required
+   */
+  public required$: BehaviorSubject<boolean> = new BehaviorSubject(false);
+
   /**
    * Initialize instance variables
    *
@@ -161,7 +166,7 @@ export class SubmissionSectionAccessesComponent extends SectionModelComponent {
    *    The form model
    */
   public initModelData(formModel: DynamicFormControlModel[]) {
-    this.accessesData.accessConditions.forEach((accessCondition, index) => {
+    this.accessesData.accessConditions?.forEach((accessCondition, index) => {
       Array.of('name', 'startDate', 'endDate')
         .filter((key) => accessCondition.hasOwnProperty(key) && isNotEmpty(accessCondition[key]))
         .forEach((key) => {
@@ -298,9 +303,8 @@ export class SubmissionSectionAccessesComponent extends SectionModelComponent {
       this.canChangeDiscoverable = !!config.canChangeDiscoverable;
       this.accessesData = accessData;
       this.formModel = this.buildFileEditForm();
+      this.required$.next(config.required);
     });
-
-
   }
 
   /**
@@ -310,7 +314,12 @@ export class SubmissionSectionAccessesComponent extends SectionModelComponent {
    *     the section status
    */
   protected getSectionStatus(): Observable<boolean> {
-    return of(true);
+    return combineLatest([
+      this.required$,
+      this.sectionService.getSectionErrors(this.submissionId,  this.sectionData.id)
+    ]).pipe(
+      map(([required, errors]) => (!required || (required  && isObjectEmpty(errors) && !isObjectEmpty(this.accessesData))))
+    );
   }
 
   /**