core: Fix removeItems for indices greater than 9 (#2181)

In mapDispatchToArrayControlProps, the removeItems method sorted indices as strings.
This broke removing items for indix combinations like [2, 11] because 11 was sorted before 2.
This is fixed with this commit by properly sorting as numbers.

Author: xou816

packages/core/src/util/renderer.ts

@@ -799,7 +799,7 @@ export const mapDispatchToArrayControlProps = (
     dispatch(
       update(path, (array) => {
         toDelete
-          .sort()
+          .sort((a, b) => a - b)
           .reverse()
           .forEach((s) => array.splice(s, 1));
         return array;

packages/core/test/util/renderer.test.ts

@@ -625,6 +625,31 @@ test('mapDispatchToArrayControlProps should remove items from array', (t) => {
   t.is(getCore().data[0], 'quux');
 });
 
+test('mapDispatchToArrayControlProps should remove items from array with more than 10 items', (t) => {
+  const data = ['0', '1', '2', '3', '4', '5', '6', '7', '8', '9', '10', '11'];
+  const schema: JsonSchema = {
+    type: 'array',
+    items: {
+      type: 'string',
+    },
+  };
+  const uischema: ControlElement = {
+    type: 'Control',
+    scope: '#',
+  };
+  const initCore: JsonFormsCore = {
+    uischema,
+    schema,
+    data,
+    errors: [] as ErrorObject[],
+  };
+  const [getCore, dispatch] = mockDispatch(initCore);
+  dispatch(init(data, schema, uischema));
+  const props = mapDispatchToArrayControlProps(dispatch);
+  props.removeItems('', [11, 2])();
+  t.is(getCore().data.length, 10);
+});
+
 test('mapStateToLayoutProps - visible via state with path from ownProps ', (t) => {
   const uischema = {
     type: 'VerticalLayout',