fixing default POST routes and worker exits on errors

koopjs · Feb 10, 2015 · 6381fd3 · 6381fd3
1 parent 1fa4b25
commit 6381fd3
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 4 deletions.
diff --git a/index.js b/index.js
@@ -95,6 +95,7 @@ module.exports = function( config ) {
       if ( controller[ handler ] ){
         defaultRoutes[ handler ].forEach(function(route){
           app[ 'get' ]( '/'+ name + pattern + route, controller[ handler ]);
+          app[ 'post' ]( '/'+ name + pattern + route, controller[ handler ]);
         });
       }
     }

diff --git a/lib/ExportWorker.js b/lib/ExportWorker.js
@@ -192,6 +192,7 @@ function createFiles(job, done){
                 info.generating = { error: e };
                 koop.Cache.updateInfo( task.table, info, function(err, res){
                   done('failed to generate file ' + e);
+                  workerQ.kill();
                   cb();
                 });
               }

diff --git a/lib/GeoJSON.js b/lib/GeoJSON.js
@@ -135,8 +135,8 @@ exports.fromEsri = function( fields, json, callback ){
       } catch (e){
         if ( dateFields.length ){
           dateFields.forEach(function(d,i){
-            if ( newFeature.attributes[d] ){
-              feature.attributes[d] = new Date(feature.attributes[d]).toISOString();
+            if ( f.attributes[d] ){
+              f.attributes[d] = new Date(f.attributes[d]).toISOString();
             }
           });
         }
@@ -147,7 +147,7 @@ exports.fromEsri = function( fields, json, callback ){
             geometry: null
         };
         geojson.features.push( newFeature );
-        console.log('error parsing feature', e, f);
+        //console.log('error parsing feature', e, f);
       }
     });
     callback(null, geojson);

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "koop",
-  "version": "1.0.13",
+  "version": "1.0.14",
   "description": "A node module/express middleware for converting GeoJSON to Esri Feature Services.",
   "main": "index.js",
   "scripts": {