proposed fix for jdorn#388 using math.js

* bower and grunt-bower-concat are now required for building json-editor
* math.js is now required for running json-editor, but is not checked by jshint
* grunt no longer runs jshint on the output dist file, as it may contain
  external libaries
* float comparisons for multipleOf, divisibleBy, maximum, minimum use
  math.js

Author: Adam Bloomston

.gitignore

@@ -1,2 +1,4 @@
 /node_modules/
 /.idea/
+/bower_components/
+/src/bower.js

Gruntfile.js

@@ -2,6 +2,14 @@
 
 module.exports = function(grunt) {
   grunt.initConfig({
+    bower_concat: {
+      all: {
+        dest: 'src/bower.js',
+        mainFiles: {
+          'mathjs': ['dist/math.js']
+	}
+      }
+    },
     concat_sourcemap: {
       options: {
         sourcesContent: true
@@ -12,6 +20,9 @@ module.exports = function(grunt) {
 
             // License & version info, start the containing closure
             'src/intro.js',
+
+            // bower dependencies
+            'src/bower.js',
 
             // Simple inheritance
             'src/class.js',
@@ -119,25 +130,18 @@ module.exports = function(grunt) {
 
         // Wrapper for $.fn style initialization
         'src/jquery.js'
-      ],
-      afterconcat: {
-        options: {
-          undef: true
-        },
-        files: {
-          src: ['dist/jsoneditor.js']
-        }
-      }
+      ]
     }
   });
 
   // These plugins provide necessary tasks.
   grunt.loadNpmTasks('grunt-contrib-uglify');
+  grunt.loadNpmTasks('grunt-bower-concat');
   grunt.loadNpmTasks('grunt-contrib-watch');
   grunt.loadNpmTasks('grunt-contrib-jshint');
   grunt.loadNpmTasks('grunt-concat-sourcemap');
 
   // Default task.
-  grunt.registerTask('default', ['jshint:beforeconcat','concat_sourcemap','jshint:afterconcat','uglify']);
+  grunt.registerTask('default', ['jshint:beforeconcat','bower_concat','concat_sourcemap','uglify']);
 
 };

README.md

@@ -16,7 +16,11 @@ Download the [production version][min] (22K when gzipped) or the [development ve
 Requirements
 -----------------
 
-JSON Editor has no required dependencies.  It only needs a modern browser (tested in Chrome and Firefox).
+JSON Schema has the following dependencies:
+
+* [math.js](http://mathjs.org/) for floating point math
+
+It needs a modern browser (tested in Chrome and Firefox).
 
 ### Optional Requirements
 

bower.json

@@ -23,5 +23,8 @@
     "package.json",
     "Gruntfile.js",
     "src"
-  ]
+  ],
+  "dependencies": {
+    "mathjs": "~1.7.0"
+  }
 }

package.json

@@ -27,14 +27,16 @@
     "node": ">= 0.8.0"
   },
   "devDependencies": {
+    "bower": "~1.4.1",
     "grunt": "~0.4.2",
+    "grunt-bower-concat": "~0.5.0",
     "grunt-concat-sourcemap": "^0.4.3",
     "grunt-contrib-jshint": "^0.10.0",
     "grunt-contrib-uglify": "~0.2.0",
     "grunt-contrib-watch": "~0.5.3"
   },
   "scripts": {
-    "build": "npm install && grunt",
+    "build": "npm install && node_modules/.bin/bower install --config.interactive=false && grunt",
     "start": "grunt watch",
     "test": "grunt"
   }

src/validator.js

@@ -208,26 +208,27 @@ JSONEditor.Validator = Class.extend({
     if(typeof value === "number") {
       // `multipleOf` and `divisibleBy`
       if(schema.multipleOf || schema.divisibleBy) {
-        valid = value / (schema.multipleOf || schema.divisibleBy);
-        if(valid !== Math.floor(valid)) {
+        var divisor = schema.multipleOf || schema.divisibleBy;
+        valid = math.mod(value, divisor);
+        if(!math.equal(valid, 0.0) && !math.equal(valid, divisor)) {
           errors.push({
             path: path,
             property: schema.multipleOf? 'multipleOf' : 'divisibleBy',
-            message: this.translate('error_multipleOf', [schema.multipleOf || schema.divisibleBy])
+            message: this.translate('error_multipleOf', [divisor])
           });
         }
       }
 
       // `maximum`
       if(schema.hasOwnProperty('maximum')) {
-        if(schema.exclusiveMaximum && value >= schema.maximum) {
+        if(schema.exclusiveMaximum && math.largerEq(value, schema.maximum)) {
           errors.push({
             path: path,
             property: 'maximum',
             message: this.translate('error_maximum_excl', [schema.maximum])
           });
         }
-        else if(!schema.exclusiveMaximum && value > schema.maximum) {
+        else if(!schema.exclusiveMaximum && math.larger(value, schema.maximum)) {
           errors.push({
             path: path,
             property: 'maximum',
@@ -238,14 +239,14 @@ JSONEditor.Validator = Class.extend({
 
       // `minimum`
       if(schema.hasOwnProperty('minimum')) {
-        if(schema.exclusiveMinimum && value <= schema.minimum) {
+        if(schema.exclusiveMinimum && math.smallerEq(value, schema.minimum)) {
           errors.push({
             path: path,
             property: 'minimum',
             message: this.translate('error_minimum_excl', [schema.minimum])
           });
         }
-        else if(!schema.exclusiveMinimum && value < schema.minimum) {
+        else if(!schema.exclusiveMinimum && math.smaller(value, schema.minimum)) {
           errors.push({
             path: path,
             property: 'minimum',