pg - npm Package Compare versions

Comparing version 0.6.10 to 0.6.11

lib/textParsers.js

@@ -0,1 +1,3 @@
+var arrayParser = require(__dirname + "/arrayParser.js");
+
 //parses PostgreSQL server formatted date strings into javascript date objects
@@ -54,20 +56,18 @@ var parseDate = function(isoDate) {
 var parseIntegerArray = function(val) {
-  return JSON.parse(val.replace("{","[").replace("}","]"));
+  if(!val) return null;
+  var p = arrayParser.create(val, function(entry){
+    if(entry != null)
+      entry = parseInt(entry);
+
+    return entry;
+  });
+  
+  return p.parse();
 };
 
 var parseStringArray = function(val) {
-  if (!val) return null;
-  if (val[0] !== '{' || val[val.length-1] !== '}')
-    throw "Not postgresql array! (" + arrStr + ")";
-
-  var x = val.substring(1, val.length - 1);
-  if (x === '') return [];
-  x = x.match(/(NULL|[^,]+|"((?:.|\n|\r)*?)(?!\\)"|\{((?:.|\n|\r)*?(?!\\)\}) (,|$))/mg);
-  if (x === null) throw "Not postgre array";
-  return x.map(function (el) {
-    if (el === 'NULL') return null;
-    if (el[0] === '{') return arguments.callee(el);
-    if (el[0] === '\"')  return el.substring(1, el.length - 1).replace(/\\(.)/g, '$1');
-    return el;
-  });
+  if(!val) return null;
+  
+  var p = arrayParser.create(val);
+  return p.parse();
 };
@@ -74,0 +74,0 @@

package.json

 { "name": "pg",
-  "version": "0.6.10",
+  "version": "0.6.11",
   "description": "PostgreSQL client - pure javascript & libpq with the same API",
@@ -4,0 +4,0 @@ "keywords" : ["postgres", "pg", "libpq", "postgre", "database", "rdbms"],

test/integration/client/array-tests.js

@@ -29,2 +29,97 @@ var helper = require(__dirname + "/test-helper");
     })
+    
+    test('empty array', function(){
+      client.query("SELECT '{}'::text[] as names", assert.success(function(result) {
+        var names = result.rows[0].names;
+        assert.lengthIs(names, 0);
+        pg.end();
+      }))
+    })
+
+    test('element containing comma', function(){
+      client.query("SELECT '{\"joe,bob\",jim}'::text[] as names", assert.success(function(result) {
+        var names = result.rows[0].names;
+        assert.lengthIs(names, 2);
+        assert.equal(names[0], 'joe,bob');
+        assert.equal(names[1], 'jim');
+        pg.end();
+      }))
+    })
+
+    test('bracket in quotes', function(){
+      client.query("SELECT '{\"{\",\"}\"}'::text[] as names", assert.success(function(result) {
+        var names = result.rows[0].names;
+        assert.lengthIs(names, 2);
+        assert.equal(names[0], '{');
+        assert.equal(names[1], '}');
+        pg.end();
+      }))
+    })
+
+    test('null value', function(){
+      client.query("SELECT '{joe,null,bob}'::text[] as names", assert.success(function(result) {
+        var names = result.rows[0].names;
+        assert.lengthIs(names, 3);
+        assert.equal(names[0], 'joe');
+        assert.equal(names[1], null);
+        assert.equal(names[2], 'bob');
+        pg.end();
+      }))
+    })
+
+    test('element containing quote char', function(){
+      client.query("SELECT '{\"joe''\",jim'',\"bob\\\\\"\"}'::text[] as names", assert.success(function(result) {
+        var names = result.rows[0].names;
+        assert.lengthIs(names, 3);
+        assert.equal(names[0], 'joe\'');
+        assert.equal(names[1], 'jim\'');
+        assert.equal(names[2], 'bob"');
+        pg.end();
+      }))
+    })
+
+    test('nested array', function(){
+      client.query("SELECT '{{1,joe},{2,bob}}'::text[] as names", assert.success(function(result) {
+        var names = result.rows[0].names;
+        assert.lengthIs(names, 2);
+
+        assert.lengthIs(names[0], 2);
+        assert.equal(names[0][0], '1');
+        assert.equal(names[0][1], 'joe');
+
+        assert.lengthIs(names[1], 2);
+        assert.equal(names[1][0], '2');
+        assert.equal(names[1][1], 'bob');
+
+        pg.end();
+      }))
+    })
+
+    test('integer array', function(){
+      client.query("SELECT '{1,2,3}'::integer[] as names", assert.success(function(result) {
+        var names = result.rows[0].names;
+        assert.lengthIs(names, 3);
+        assert.equal(names[0], 1);
+        assert.equal(names[1], 2);
+        assert.equal(names[2], 3);
+        pg.end();
+      }))
+    })
+
+    test('integer nested array', function(){
+      client.query("SELECT '{{1,100},{2,100},{3,100}}'::integer[] as names", assert.success(function(result) {
+        var names = result.rows[0].names;
+        assert.lengthIs(names, 3);
+        assert.equal(names[0][0], 1);
+        assert.equal(names[0][1], 100);
+
+        assert.equal(names[1][0], 2);
+        assert.equal(names[1][1], 100);
+
+        assert.equal(names[2][0], 3);
+        assert.equal(names[2][1], 100);
+        pg.end();
+      }))
+    })
+    
   }))
@@ -31,0 +126,0 @@ })

New alerts

Dynamic require

Supply chain risk

Dynamic require can indicate the package is performing dangerous or unsafe dynamic code execution.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

215594

increased by2.3%

Number of package files

88

increased by1.15%

Lines of code

5859

increased by2.97%

Worsened metrics

Number of low supply chain risk alerts

137

increased by0.74%

No dependency changes