Allow using field names containing a dot character (rjsf-team#397)

Author: yang2007chun

package.json

@@ -30,6 +30,7 @@
  },
  "dependencies": {
  "jsonschema": "^1.0.2",
+ "lodash.topath": "^4.5.2",
  "setimmediate": "^1.0.5"
  },
  "devDependencies": {
@@ -49,7 +50,6 @@
  "chai": "^3.3.0",
  "cross-env": "^2.0.1",
  "css-loader": "^0.23.1",
- "eslint": "^2.9.0",
  "eslint-plugin-react": "^4.2.3",
  "estraverse": "^4.2.0",
  "estraverse-fb": "^1.3.1",

src/validate.js

@@ -1,26 +1,8 @@
+import toPath from 'lodash.topath';
 import {validate as jsonValidate} from "jsonschema";
 
 import {isObject, mergeObjects} from "./utils";
 
-
-const RE_ERROR_ARRAY_PATH = /\[\d+]/g;
-
-function errorPropertyToPath(property) {
- // Parse array indices, eg. "instance.level1.level2[2].level3"
- // => ["instance", "level1", "level2", 2, "level3"]
- return property.split(".").reduce((path, node) => {
- const match = node.match(RE_ERROR_ARRAY_PATH);
- if (match) {
- const nodeName = node.slice(0, node.indexOf("["));
- const indices = match.map(str => parseInt(str.slice(1, -1), 10));
- path = path.concat(nodeName, indices);
- } else {
- path.push(node);
- }
- return path;
- }, []);
-}
-
 function toErrorSchema(errors) {
  // Transforms a jsonschema validation errors list:
  // [
@@ -42,7 +24,7 @@ function toErrorSchema(errors) {
  }
  return errors.reduce((errorSchema, error) => {
  const {property, message} = error;
- const path = errorPropertyToPath(property);
+ const path = toPath(property);
  let parent = errorSchema;
  for (const segment of path.slice(1)) {
  if (!(segment in parent)) {

test/validate_test.js

@@ -9,29 +9,31 @@ import {createFormComponent} from "./test_utils";
 describe("Validation", () => {
  describe("validate.validateFormData()", () => {
  describe("No custom validate function", () => {
+ const illFormedKey = "bar.'\"[]()=+*&^%$#@!";
  const schema = {
  type: "object",
- properties: {
- foo: {type: "string"}
- }
+ properties: { foo: {type: "string"}, [illFormedKey]: {type: "string"} }
  };
 
  let errors, errorSchema;
 
  beforeEach(() => {
- const result = validateFormData({foo: 42}, schema);
+ const result = validateFormData({ foo: 42, [illFormedKey]: 41 }, schema);
  errors = result.errors;
  errorSchema = result.errorSchema;
  });
 
  it("should return an error list", () => {
- expect(errors).to.have.length.of(1);
+ expect(errors).to.have.length.of(2);
  expect(errors[0].message).eql("is not of a type(s) string");
+ expect(errors[1].message).eql("is not of a type(s) string");
  });
 
  it("should return an errorSchema", () => {
  expect(errorSchema.foo.__errors).to.have.length.of(1);
  expect(errorSchema.foo.__errors[0]).eql("is not of a type(s) string");
+ expect(errorSchema[illFormedKey].__errors).to.have.length.of(1);
+ expect(errorSchema[illFormedKey].__errors[0]).eql("is not of a type(s) string");
  });
  });