jdorn / json-editor

JSON Schema Based Editor
MIT License
5.8k stars 1.08k forks source link

Bug in enum rendering with "watch" option #369

Open EduardOGAN opened 9 years ago

EduardOGAN commented 9 years ago

The problem arises when linked in "watch" section placed in schema after the enum with "watch" is caused. In this case value from enum default is not selected.

Example with probled: http://jeremydorn.com/json-editor/?schema=N4IgJgpgZghgrgGwC4gFygM4QHaQE5ogCMJRIANCAMoCyAKgApoDaoAFgPYYqrGkUgADhzw8ATAFZKcLHmwwAthEKkylQTAwYA7iLAr+6vNACWAD0IgAvuXZceIMU7EDhotJOmz5Sws5fqmjp6fs6uxlDmllYAujYgSCZICMq8DBB4GBzYAkgAnoKpIBwARgBWEADGKEYchaImEBhomDj4LQlJKYRUbQAECjAmCH2yAG4mlcrS2CYAjnAQAJJIEArNqEh4i5T5hYTceCbYAOYC2jBIlWwdGApIghsgeBwcSAB0tIzWlDhwClQOHA8FMWMwQAARaDwZAgGK2EBZYGg3h3B7NXZdIrAYBJNbvGQZHwQKxWARjGAIRaEHF4hQE7yKElk2LxL5MdCdZJFADCMiQHAUfXZwoyYwyGISBSKMDweBgeQEgzMKzWGyIAAZKFARIMHEgYCVujN5otVes0FsdkIjoM8HkANIQRWocGcbgCQlyJlwyh1RLZDagSocBAIGCPCD6VCwBBYShgEwYQ0pAD6IbDEawlu2EATSZTEFTsvleVTxj0GTQsaw8TpQal+14pQq1VyWMIAEEwGBhfQGKK8OKCH7BAHsA2M+HI9Ga3nwAWjUWp1nUlaSZQYD2kiZspSGC96okmtXKfGbXUMseG+6eKBEtyev2+reBJAMJUjmPdzleABRMxFEEFJUFGe5BHeE5XhOFJBmGd4QwUXJpQOLZjjObVdUuQhb2JJVjgAGRwE4kBuVAiHiNw70bIpsH+EoqwTJpPxMb9skIOg2CTPpuJOHAMkpBA8j6SQ+hEPoABYADYJHbR9eBFKi32hRBxCkEAFGOEwFH+NAKK8IkfU5PYikOdC5O6BTny9PCmI/L9x0IACgJAvoSlKd4wAAAROOCEAQwV8OwIjTlIvTKKCXQ8Gje8UNRNDTgEHU8D1QgNC0KL9ExeTqGfdLgmioKQpIsj9JtUwLGMjsFKqYEkkVOyWLY38QB5bJsCqcc+kECIohNBZllWC1NlzXY4sRBKMJAP4kNdEABC0BBUAAemW31wBU2E5rhUldqsIAA=&value=N4IgzgpgdgJhBOIBcIAiEBmBDArgGwBcQAaEAZQFkAVABWQG1QALAezCJQEZvOSQAHFvA4AmAKykckeFCwBbCMhA9epfljBgA7kJhKVffvEwBLAB5KQAX2LM2HECKcjDQ0RJBSEshUucu1DW1dP2dDYwxzSysAXSsgA=&theme=bootstrap2&iconlib=fontawesome4&object_layout=normal&show_errors=interaction

And without: http://jeremydorn.com/json-editor/?schema=N4IgJgpgZghgrgGwC4gFygM4QHaQE5ogCMJRIANCAMoCyAKgApoDaoAFgPYYqrGkUgADhzw8ATAFZKcLHmwwAthEKkylQTAwYA7iLAr+6vNACWAD0IgAvuXZceIMU7EDhotJOmz5Sws5fqmjp6fs6uxlDmllYAujYgSCZICMq8DBB4GBzYAkgAnoKpIBwARgBWEADGKEYchaImEBhooLSMLQlJKYQAwjJIHAoABG0MIxkAbhnNlPmFhDB4eDB5AgowZgCSSBAKzahEAAyUUCLrDkgwJd3S2CYAjnAQ27v7SHhPRibreHkA0hBVqhmCBONwBDIMj5lDFKHVEtl9qBKhwEAgYIIsPpULAEFhKGATBgrikAPootEYrBod6fcBEkkQUmLZZ5UnGPQZNC4rDxJKvDpzIqlCrVXJdIoAQTAYBG9DGVEmXLhggR2CRIAp6MxEGxPIgBIZ1yZWqpqVpEHiMBlSRM2RgCAYeDqGUSTW5DvxQmd9TdGrBPFAiWSRVGQwDAkgGEqeBMqrtOV4AFEzIpBClUEMMAokIIAHQAcw4HALKXWJgQeZRClyBSK3Fj2ALAlOeHOhAD0LWJmwABkcAWkGw0ER4m5Awk64RsHAFCVleAmjG42rCHQ2EShpuCzgMg6EHkhpIhiIhgAWABsEnFIcIYfHkeg8GQHikIAUPe+s5H8UhckUqRBlOvANj2zazBKd7ykMf5dgSS6xvG2SECmaYZkMJSlHmYAAAIFuWlbVt2fYDkOP6BFouh4NiQHzCB7xgS2ZwwA4GiUSEEG3rw95BFR+iUB+JFNmRBxjhEUToJ0XHUFUcCxvkj7Rohq68D02TYFUapDII4kWLcDxPC8ew0h8BqTnRICgU2Ag4N+wIgAIWgIKgAD0LkgLCi6wIgPAgh5VgBfEWC4FyknBt03E4LKBFZpMJiVMo+mPM8OzGagFqzMBlkMdZlDaCxlTDpJ2a5vsIDOhwSB5qM1iULZChUBwckJSwIIACJPj5Hm2JZTV4C1IE5piN4RSAwDAPyCh5rBAEBQIEwOk8hDjZN03eLNVjWHEAVAAAA&value=N4IgygsgKgCiBcBtUALA9gZwC4JARgLxABoQAHNAJx3gCYBWUgVwwFNKA7AQwFtXdCRUmS4YMAdyoATAYRLlKrAGYBLAB64QAX2KpMNELSO15FaggbM2nXv3iHjp0ROm5jJ4YtUb72gLqkbBxS7LIE2kAAA=&theme=bootstrap2&iconlib=fontawesome4&object_layout=normal&show_errors=interaction

jdorn commented 9 years ago

As a temporary workaround, you could use the propertyOrder keyword to order the object properties however you want, regardless of what order they are defined in the schema.

This bug is probably buried deep in the code. I'll try and take a look when I have some time.