ballerina-core 1.0.224 → 1.0.226

package/package.json

@@ -2,7 +2,7 @@
   "name": "ballerina-core",
   "author": "Dr. Giuseppe Maggiore",
   "private": false,
-  "version": "1.0.224",
+  "version": "1.0.226",
   "main": "main.ts",
   "scripts": {
     "prettier": "prettier --write ."

package/src/forms/domains/dispatched-forms/built-ins/state.ts

@@ -1877,10 +1877,37 @@ export const dispatchToAPIRawValue =
           );
         }
 
+        const filteredRawValues = raw.fields.filter((value) => {
+          if (!PredicateValue.Operations.IsRecord(value)) {
+            console.warn(
+              "Received a non-record value in a multi selection, ignoring: ",
+              JSON.stringify(value),
+            );
+            return false;
+          }
+
+          const fieldsObject = value.fields.toJS();
+
+          if (
+            !CollectionReference.Operations.IsCollectionReference(
+              fieldsObject,
+            ) &&
+            !EnumReference.Operations.IsEnumReference(fieldsObject)
+          ) {
+            console.warn(
+              "Received a non-collection or enum reference value in a multi selection, ignoring: ",
+              JSON.stringify(value),
+            );
+            return false;
+          }
+          return true;
+        });
+
         const rawValue: Map<
           string,
           ValueOrErrors<CollectionReference | EnumReference, string>
-        > = raw.fields.map((value) => {
+        > = filteredRawValues.map((value) => {
+          // should never happen due to the filter above but is a type check
           if (!PredicateValue.Operations.IsRecord(value)) {
             return ValueOrErrors.Default.throwOne(
               `Record expected but got ${JSON.stringify(value)}`,
@@ -1900,6 +1927,7 @@ export const dispatchToAPIRawValue =
               )}`,
             );
           }
+
           return ValueOrErrors.Default.return(fieldsObject);
         });
 

package/src/forms/domains/dispatched-forms/runner/domains/abstract-renderers/list/state.ts

@@ -95,7 +95,9 @@ export type ListAbstractRendererView<
       ListAbstractRendererState,
       ListAbstractRendererForeignMutationsExpected<Flags>
     >;
-    embeddedPlaceholderElementTemplate: () => (
+    embeddedPlaceholderElementTemplate: (
+      elementIndex: number,
+    ) => (
       flags: Flags | undefined,
     ) => Template<
       ListAbstractRendererReadonlyContext<
@@ -104,7 +106,7 @@ export type ListAbstractRendererView<
       > &
         ListAbstractRendererState,
       ListAbstractRendererState,
-      CommonAbstractRendererForeignMutationsExpected<Flags>
+      ListAbstractRendererForeignMutationsExpected<Flags>
     >;
   }
 >;

package/src/forms/domains/dispatched-forms/runner/domains/abstract-renderers/list/template.tsx

@@ -143,37 +143,105 @@ export const ListAbstractRenderer = <
           },
         }));
 
-  const embeddedPlaceholderElementTemplate = () => (flags: Flags | undefined) =>
-    elementTemplate
-      .mapContext(
-        (
-          _: ListAbstractRendererReadonlyContext<
-            CustomPresentationContext,
-            ExtraContext
-          > &
-            ListAbstractRendererState,
-        ) => ({
-          disabled: _.disabled,
-          locked: _.locked,
-          value: PredicateValue.Operations.IsUnit(_.value)
-            ? _.value
-            : GetDefaultElementValue(),
-          ...GetDefaultElementState(),
-          bindings: _.bindings,
-          extraContext: _.extraContext,
-          type: _.type.args[0],
-          customPresentationContext: _.customPresentationContext,
-          remoteEntityVersionIdentifier: _.remoteEntityVersionIdentifier,
-          domNodeAncestorPath: _.domNodeAncestorPath + `[list][placeholder]`,
-          typeAncestors: [_.type as DispatchParsedType<T>].concat(
-            _.typeAncestors,
-          ),
-          lookupTypeAncestorNames: _.lookupTypeAncestorNames,
-        }),
-      )
-      .mapState((_) =>
-        ListAbstractRendererState.Updaters.Core.elementFormStates(id),
-      );
+  const embeddedPlaceholderElementTemplate =
+    (elementIndex: number) => (flags: Flags | undefined) =>
+      elementTemplate
+        .mapContext(
+          (
+            _: ListAbstractRendererReadonlyContext<
+              CustomPresentationContext,
+              ExtraContext
+            > &
+              ListAbstractRendererState,
+          ) => ({
+            disabled: _.disabled,
+            locked: _.locked,
+            value: PredicateValue.Operations.IsUnit(_.value)
+              ? _.value
+              : _.value.values.get(elementIndex) || GetDefaultElementValue(),
+            ...(_.elementFormStates?.get(elementIndex) ||
+              GetDefaultElementState()),
+            bindings: _.bindings,
+            extraContext: _.extraContext,
+            type: _.type.args[0],
+            customPresentationContext: _.customPresentationContext,
+            remoteEntityVersionIdentifier: _.remoteEntityVersionIdentifier,
+            domNodeAncestorPath:
+              _.domNodeAncestorPath + `[list][${elementIndex}]`,
+            typeAncestors: [_.type as DispatchParsedType<T>].concat(
+              _.typeAncestors,
+            ),
+            lookupTypeAncestorNames: _.lookupTypeAncestorNames,
+          }),
+        )
+        .mapState((_) =>
+          ListAbstractRendererState.Updaters.Core.elementFormStates(id),
+        )
+        .mapForeignMutationsFromProps<
+          ListAbstractRendererForeignMutationsExpected<Flags>
+        >((props) => ({
+          onChange: (elementUpdater, nestedDelta) => {
+            const value =
+              props.context.value.kind == "tuple"
+                ? props.context.value.values.get(elementIndex) ||
+                  GetDefaultElementValue()
+                : GetDefaultElementValue();
+            const delta: DispatchDelta<Flags> = {
+              kind: "ArrayAdd",
+              value:
+                elementUpdater.kind == "r"
+                  ? elementUpdater.value(value)
+                  : GetDefaultElementValue(),
+              state: {
+                commonFormState: props.context.commonFormState,
+                elementFormStates: props.context.elementFormStates,
+              },
+              type: props.context.type.args[0],
+              flags,
+              sourceAncestorLookupTypeNames:
+                nestedDelta.sourceAncestorLookupTypeNames,
+            };
+            props.foreignMutations.onChange(
+              elementUpdater.kind == "l"
+                ? Option.Default.none()
+                : Option.Default.some(
+                    Updater((list) =>
+                      list.values.has(elementIndex)
+                        ? PredicateValue.Default.tuple(
+                            list.values.update(
+                              elementIndex,
+                              PredicateValue.Default.unit(),
+                              elementUpdater.value,
+                            ),
+                          )
+                        : list,
+                    ),
+                  ),
+              delta,
+            );
+            props.setState(
+              ListAbstractRendererState.Updaters.Core.commonFormState(
+                DispatchCommonFormState.Updaters.modifiedByUser(
+                  replaceWith(true),
+                ),
+              ).then(
+                ListAbstractRendererState.Updaters.Core.elementFormStates(
+                  MapRepo.Updaters.upsert(
+                    elementIndex,
+                    () => GetDefaultElementState(),
+                    (_) => ({
+                      ..._,
+                      commonFormState:
+                        DispatchCommonFormState.Updaters.modifiedByUser(
+                          replaceWith(true),
+                        )(_.commonFormState),
+                    }),
+                  ),
+                ),
+              ),
+            );
+          },
+        }));
 
   return Template.Default<
     ListAbstractRendererReadonlyContext<