Merge branch 'master' of github.com:Motorola-Mobility/ninja

10 files changed, 130 insertions, 111 deletions

diff --git a/js/lib/NJUtils.js b/js/lib/NJUtils.js
index cfb7ef07..bca9b05b 100755
--- a/js/lib/NJUtils.js
+++ b/js/lib/NJUtils.js
@@ -241,6 +241,7 @@ exports.NJUtils = Montage.create(Component, {
     // 100px will return the following array: [100, px]
     getValueAndUnits: {
         value: function(input) {
+            if (!input) return [null, null];
             var numberValue = parseFloat(input);
 
             // Ignore all whitespace, digits, negative sign and "." when looking for units label
diff --git a/js/lib/drawing/world.js b/js/lib/drawing/world.js
index ca2f07cc..b9f9863f 100755
--- a/js/lib/drawing/world.js
+++ b/js/lib/drawing/world.js
@@ -439,6 +439,11 @@ var World = function GLWorld( canvas, use3D, preserveDrawingBuffer ) {
     // in the case of a procedurally built scene an init state is not needed for loading data
     this._canvas.rdgeid = this._canvas.getAttribute( "data-RDGE-id" );
     if (this._useWebGL) {
+        // make sure the id is unique
+        var id = this.uniqueifyID( this._canvas.rdgeid );
+        this._canvas.rdgeid = id;
+        this._canvas.setAttribute( "data-rdge-id", id );
+
         rdgeStarted = true;
         RDGE.globals.engine.unregisterCanvas( this._canvas );
         RDGE.globals.engine.registerCanvas(this._canvas, this);
@@ -673,13 +678,35 @@ World.prototype.updateMaterials = function( obj, time ) {
         var n = matArray.length;
         for (var i=0;  i<n;  i++) {
             matArray[i].update( time );
-    }
+        }
     }
 
     this.updateMaterials( obj.getNext(),  time );
     this.updateMaterials( obj.getChild(), time );
 };
 
+
+World.prototype.uniqueifyID = function( id )
+{
+    var ctx = RDGE.globals.engine.getContext( id );
+    while ( ctx )
+    {
+        var num = 0;
+        var index = id.indexOf( "_" );
+        if (index >= 0)
+        {
+            var subStr = id.substr( index+1);
+            num = Number( subStr ) + 1;
+            id = id.substr( 0,  index );
+        }
+
+        id = id + "_" + num;
+        ctx = RDGE.globals.engine.getContext( id );
+    }
+
+    return id;
+}
+
 // return the origin of the world in NDC
 World.prototype.getNDCOrigin = function() {
   var pt = MathUtils.transformPoint( [0,0,0], this.getCameraMatInverse() );
@@ -973,7 +1000,6 @@ World.prototype.importJSON = function (jObj)
         // start RDGE
         rdgeStarted = true;
         var id = this._canvas.getAttribute( "data-RDGE-id" );
-        this._canvas.rdgeid = id;
         RDGE.globals.engine.registerCanvas(this._canvas, this);
         RDGE.RDGEStart(this._canvas);
         this._canvas.task.stop()
diff --git a/js/lib/geom/brush-stroke.js b/js/lib/geom/brush-stroke.js
index 12f5c5a1..164d31ee 100755
--- a/js/lib/geom/brush-stroke.js
+++ b/js/lib/geom/brush-stroke.js
@@ -864,6 +864,17 @@ BrushStroke.prototype.drawToContext = function(ctx, drawStageWorldPts, stageWorl
 }; //this.drawToCanvas()
 
 
+
+BrushStroke.prototype._fixCoordPrecision = function(coord, precision){
+    var i=0;
+    var numPoints = coord.length;
+    for (i=0;i<numPoints;i++){
+        coord[i][0] = parseFloat((coord[i][0]).toFixed(precision));
+        coord[i][1] = parseFloat((coord[i][1]).toFixed(precision));
+        coord[i][2] = parseFloat((coord[i][2]).toFixed(precision));
+    }
+};
+
 BrushStroke.prototype.exportJSON = function(){
     var retObject= new Object();
     //the type of this object
@@ -873,9 +884,10 @@ BrushStroke.prototype.exportJSON = function(){
     //the geometry for this object
     retObject.localPoints = this._LocalPoints.slice(0);
     this._copyCoordinates3D(this._LocalPoints, retObject.localPoints); //todo is this necessary in addition to the slice(0) above?
+    this._fixCoordPrecision(retObject.localPoints, 4);
     retObject.origLocalPoints = this._OrigLocalPoints.slice(0);
     this._copyCoordinates3D(this._OrigLocalPoints, retObject.origLocalPoints); //todo <ditto>
-
+    this._fixCoordPrecision(retObject.origLocalPoints, 4);
     retObject.stageWorldCenter = [this._stageWorldCenter[0],this._stageWorldCenter[1],this._stageWorldCenter[2]];
     retObject.planeMat = this._planeMat;
     retObject.planeMatInv = this._planeMatInv;
diff --git a/js/lib/geom/circle.js b/js/lib/geom/circle.js
index ba47603b..4995c2cb 100755
--- a/js/lib/geom/circle.js
+++ b/js/lib/geom/circle.js
@@ -83,31 +83,13 @@ exports.Circle = Object.create(GeomObj, {
 
             if(strokeMaterial) {
                 this._strokeMaterial = strokeMaterial.dup();
-            } else {
-                this._strokeMaterial = MaterialsModel.getMaterial( MaterialsModel.getDefaultMaterialName() ).dup();
-            }
-
-            if(strokeColor) {
-                if(this._strokeMaterial.hasProperty("color")) {
-                    this._strokeMaterial.setProperty( "color",  this._strokeColor );
-                } else if (this._strokeMaterial && (this._strokeMaterial.gradientType === this._strokeColor.gradientMode)) {
-                    this._strokeMaterial.setGradientData(this._strokeColor.color);
-                }
             }
 
             if(fillMaterial) {
                 this._fillMaterial = fillMaterial.dup();
-            } else {
-                this._fillMaterial = MaterialsModel.getMaterial(  MaterialsModel.getDefaultMaterialName() ).dup();
             }
 
-            if(fillColor) {
-                if(this._fillMaterial.hasProperty("color")) {
-                    this._fillMaterial.setProperty( "color",  this._fillColor );
-                } else if (this._fillMaterial && (this._fillMaterial.gradientType === this._fillColor.gradientMode)) {
-                    this._fillMaterial.setGradientData(this._fillColor.color);
-        }
-            }
+            this.initColors();
         }
     },
 
@@ -770,8 +752,8 @@ exports.Circle = Object.create(GeomObj, {
                 'fillColor'     : this._fillColor,
                 'innerRadius'   : this._innerRadius,
                 'strokeStyle'   : this._strokeStyle,
-                'strokeMat'     : this._strokeMaterial ? this._strokeMaterial.getName() :  MaterialsModel.getDefaultMaterialName(),
-                'fillMat'       : this._fillMaterial ?  this._fillMaterial.getName() :  MaterialsModel.getDefaultMaterialName(),
+                'strokeMat'     : this._strokeMaterial ? this._strokeMaterial.getName() :  null,
+                'fillMat'       : this._fillMaterial ?  this._fillMaterial.getName() :  null,
                 'materials'     : this.exportMaterialsJSON()
             };
 
@@ -790,27 +772,26 @@ exports.Circle = Object.create(GeomObj, {
             this._fillColor         = jObj.fillColor;
             this._innerRadius       = jObj.innerRadius;
             this._strokeStyle       = jObj.strokeStyle;
-            var strokeMaterialName  = jObj.strokeMat;
-            var fillMaterialName    = jObj.fillMat;
 
-            var strokeMat = MaterialsModel.getMaterial( strokeMaterialName ).dup();
-            if (!strokeMat) {
-                console.log( "object material not found in library: " + strokeMaterialName );
-                strokeMat = MaterialsModel.getMaterial(  MaterialsModel.getDefaultMaterialName() ).dup();
+            if(jObj.strokeMat) {
+                var strokeMat = MaterialsModel.getMaterial(jObj.strokeMat).dup();
+                if (!strokeMat) {
+                    console.log("object material not found in library: " + jObj.strokeMat);
+                } else {
+                    this._strokeMaterial = strokeMat;
+                }
             }
-            this._strokeMaterial = strokeMat;
-            if (this._strokeMaterial.hasProperty( 'color' ))
-                this._strokeMaterial.setProperty( 'color', this._strokeColor );
-
-            var fillMat = MaterialsModel.getMaterial( fillMaterialName ).dup();
-            if (!fillMat) {
-                console.log( "object material not found in library: " + fillMaterialName );
-                fillMat = MaterialsModel.getMaterial(  MaterialsModel.getDefaultMaterialName() ).dup();
+
+            if(jObj.fillMat) {
+                var fillMat = MaterialsModel.getMaterial(jObj.fillMat).dup();
+                if (!fillMat) {
+                    console.log("object material not found in library: " + jObj.fillMat);
+                } else {
+                    this._fillMaterial = fillMat;
+                }
             }
-            this._fillMaterial = fillMat;
-            if (this._fillMaterial.hasProperty( 'color' ))
-                this._fillMaterial.setProperty( 'color', this._fillColor );
 
+            this.initColors();