Fixes array without items for custom fields (rjsf-team#590)

Author: vierno

src/utils.js

@@ -278,12 +278,16 @@ export function orderProperties(properties, order) {
 }
 
 export function isMultiSelect(schema) {
- return Array.isArray(schema.items.enum) && schema.uniqueItems;
+ return schema.items
+ ? Array.isArray(schema.items.enum) && schema.uniqueItems
+ : false;
 }
 
 export function isFilesArray(schema, uiSchema) {
  return (
- (schema.items.type === "string" && schema.items.format === "data-url") ||
+ (schema.items &&
+ schema.items.type === "string" &&
+ schema.items.format === "data-url") ||
  uiSchema["ui:widget"] === "files"
  );
 }

test/utils_test.js

@@ -5,6 +5,7 @@ import {
  dataURItoBlob,
  deepEquals,
  getDefaultFormState,
+ isFilesArray,
  isMultiSelect,
  mergeObjects,
  pad,
@@ -274,6 +275,11 @@ describe("utils", () => {
  expect(isMultiSelect(schema)).to.be.true;
  });
 
+ it("should be false if items is undefined", () => {
+ const schema = {};
+ expect(isMultiSelect(schema)).to.be.false;
+ });
+
  it("should be false if uniqueItems is false", () => {
  const schema = { items: { enum: ["foo", "bar"] }, uniqueItems: false };
  expect(isMultiSelect(schema)).to.be.false;
@@ -285,6 +291,20 @@ describe("utils", () => {
  });
  });
 
+ describe("isFilesArray()", () => {
+ it("should be true if items have data-url format", () => {
+ const schema = { items: { type: "string", format: "data-url" } };
+ const uiSchema = {};
+ expect(isFilesArray(schema, uiSchema)).to.be.true;
+ });
+
+ it("should be false if items is undefined", () => {
+ const schema = {};
+ const uiSchema = {};
+ expect(isFilesArray(schema, uiSchema)).to.be.false;
+ });
+ });
+
  describe("mergeObjects()", () => {
  it("should't mutate the provided objects", () => {
  const obj1 = { a: 1 };