@sumaris-net/ngx-components 2.4.2 → 2.4.3

package/fesm2020/sumaris-net.ngx-components.mjs

@@ -4641,7 +4641,7 @@ class AppFormArray extends UntypedFormArray {
      */
     setValue(values, options) {
         if (values === undefined)
-            throw new Error('\'undefined\' value not allowed in AppFormArray.setValue()');
+            throw new Error('\'undefined\' value not allowed in AppFormArray.setValue(). Use \'null\' or \'[]\' to clear the array');
         if (this.options.allowReuseControls === false) {
             const disabled = this.disabled;
             // Clean all
@@ -4664,14 +4664,14 @@ class AppFormArray extends UntypedFormArray {
         }
     }
     patchValue(values, options) {
-        // /!\ From the official Angular 'FormGroup.patchValue()' :
-        //     "Even though the `value` argument type doesn't allow `null` and `undefined` values, the
-        //     `patchValue` can be called recursively and inner data structures might have these values, so
-        //     we just ignore such cases when a field containing FormGroup instance receives `null` or
-        //     `undefined` as a value."
-        // BUT in AppFormArray, we only ignore the `undefined` value, so `null` value will reset the array (zero length).
-        if (values === undefined)
-            return;
+        // --- /!\ From official Angular doc of 'FormGroup.patchValue()' :
+        //   Even though the `value` argument type doesn't allow `null` and `undefined` values, the
+        //   `patchValue` can be called recursively and inner data structures might have these values, so
+        //   we just ignore such cases when a field containing FormGroup instance receives `null` or
+        //   `undefined` as a value.
+        // ---
+        if (values == null)
+            return; // Ignore
         if (this.options.allowReuseControls === false) {
             const disabled = this.disabled;
             // Clean all