Bug Fixes for Canvas & WebGL File IO

4 files changed, 17 insertions, 52 deletions

diff --git a/js/lib/geom/circle.js b/js/lib/geom/circle.js
index fec62308..d48bf98b 100755
--- a/js/lib/geom/circle.js
+++ b/js/lib/geom/circle.js
@@ -634,24 +634,6 @@ var Circle = function GLCircle() {
                 var strokeMaterialName  = jObj.strokeMat;
                 var fillMaterialName    = jObj.fillMat;
                 this.importMaterialsJSON( jObj.materials );
-
-                var strokeMat = MaterialsModel.getMaterial( strokeMaterialName );
-                if (!strokeMat) {
-                        console.log( "object material not found in library: " + strokeMaterialName );
-                        strokeMat = MaterialsModel.getMaterial(  MaterialsModel.getDefaultMaterialName() );
-                }
-                else
-                        strokeMat = strokeMat.dup();
-                this._strokeMaterial = strokeMat;
-
-                var fillMat = MaterialsModel.getMaterial( fillMaterialName );
-                if (!fillMat) {
-                        console.log( "object material not found in library: " + fillMaterialName );
-                        fillMat = MaterialsModel.getMaterial(  MaterialsModel.getDefaultMaterialName() );
-                }
-                else
-                        fillMat = fillMat.dup();
-                this._fillMaterial = fillMat;
         };
 
 
diff --git a/js/lib/geom/geom-obj.js b/js/lib/geom/geom-obj.js
index 63f71955..75b7808b 100755
--- a/js/lib/geom/geom-obj.js
+++ b/js/lib/geom/geom-obj.js
@@ -279,6 +279,11 @@ var GeomObj = function GLGeomObj() {
 
         this.importMaterialsJSON = function( jObj )
         {
+                this._materialArray = [];
+                this._materialTypeArray = [];
+
+                if (!jObj)  return;
+
                 var nMaterials = jObj.nMaterials;
                 var matArray = jObj.materials;
                 for (var i=0;  i<nMaterials;  i++)
@@ -318,7 +323,14 @@ var GeomObj = function GLGeomObj() {
                         }
 
                         if (mat)
+                        {
                                 mat.importJSON( matObj );
+                                this._materialArray.push( mat );
+                                this._materialTypeArray.push( matObj.type );
+                                var type = matArray[i].type;
+                                if (type == "fill")  this._fillMaterial = mat;
+                                else  this._strokeMaterial = mat;
+                        }
                 }
         }
 
diff --git a/js/lib/geom/line.js b/js/lib/geom/line.js
index a706eb93..e839e229 100755
--- a/js/lib/geom/line.js
+++ b/js/lib/geom/line.js
@@ -137,16 +137,7 @@ var Line = function GLLine( world, xOffset, yOffset, width, height, slope, strok
                 this._strokeStyle               = jObj.strokeStyle;
                 this._strokeColor               = jObj.strokeColor;
                 var strokeMaterialName  = jObj.strokeMat;
-
-                var strokeMat = MaterialsModel.getMaterial( strokeMaterialName );
-                if (!strokeMat) {
-                        console.log( "object material not found in library: " + strokeMaterialName );
-                        strokeMat = MaterialsModel.getMaterial(  MaterialsModel.getDefaultMaterialName() );
-                }
-                else
-                        strokeMat = strokeMat.dup();
-                this._strokeMaterial = strokeMat;
-
+                this.importMaterialsJSON( jObj.materials );
         };
 
         this.export = function() {
diff --git a/js/lib/geom/rectangle.js b/js/lib/geom/rectangle.js
index c6bc2d88..38e428bd 100755
--- a/js/lib/geom/rectangle.js
+++ b/js/lib/geom/rectangle.js
@@ -78,8 +78,6 @@ var Rectangle = function GLRectangle() {
         } else {
                         this._fillMaterial = MaterialsModel.getMaterial( MaterialsModel.getDefaultMaterialName() );
         }
-
-                this.exportMaterials();
         };
 
         ///////////////////////////////////////////////////////////////////////
@@ -291,24 +289,6 @@ var Rectangle = function GLRectangle() {
                 var strokeMaterialName  = jObj.strokeMat;
                 var fillMaterialName    = jObj.fillMat;
                 this.importMaterialsJSON( jObj.materials );
-
-                var strokeMat = MaterialsModel.getMaterial( strokeMaterialName );
-                if (!strokeMat) {
-                        console.log( "object material not found in library: " + strokeMaterialName );
-                        strokeMat = MaterialsModel.getMaterial(  MaterialsModel.getDefaultMaterialName() );
-                }
-                else
-                        strokeMat = strokeMat.dup();
-                this._strokeMaterial = strokeMat;
-
-                var fillMat = MaterialsModel.getMaterial( fillMaterialName );
-                if (!fillMat) {
-                        console.log( "object material not found in library: " + fillMaterialName );
-                        fillMat = MaterialsModel.getMaterial(  MaterialsModel.getDefaultMaterialName() );
-                }
-                else
-                        fillMat = fillMat.dup();
-                this._fillMaterial = fillMat;
         };
 
         this.import = function( importStr ) {
@@ -430,10 +410,10 @@ var Rectangle = function GLRectangle() {
                 brRadius = -z*(r-l)/(2.0*zn)*brRadiusNDC;
 
                 // stroke
-//              var strokeMaterial = this.makeStrokeMaterial();
-//              var strokePrim = this.createStroke([x,y],  2*xFill,  2*yFill,  strokeSize,  tlRadius, blRadius, brRadius, trRadius, strokeMaterial);
-//        this._primArray.push( strokePrim );
-//              this._materialNodeArray.push( strokeMaterial.getMaterialNode() );
+                var strokeMaterial = this.makeStrokeMaterial();
+                var strokePrim = this.createStroke([x,y],  2*xFill,  2*yFill,  strokeSize,  tlRadius, blRadius, brRadius, trRadius, strokeMaterial);
+        this._primArray.push( strokePrim );
+                this._materialNodeArray.push( strokeMaterial.getMaterialNode() );
 
                 // fill
                 tlRadius -= strokeSize;  if (tlRadius < 0)  tlRadius = 0.0;