multi-ini - npm Package Compare versions

Comparing version 1.0.0 to 1.0.1

lib/serializer.js

@@ -13,3 +13,4 @@ 'use strict';
 var defaults = {
-    line_breaks: 'unix'
+    line_breaks: 'unix',
+    keep_quotes: false
 };
@@ -29,2 +30,7 @@
         value: function needToBeQuoted(value) {
+            if (this.options.keep_quotes) {
+                return false;
+            }
+
+            // wrapped with qoutes
             if (value.match(/^"[\s\S]*?"$/g)) {
@@ -34,2 +40,3 @@ return false;
 
+            // escaped quote at the end
             if (value.match(/^[\s\S]*?\\"$/g)) {
@@ -39,2 +46,3 @@ return true;
 
+            // ends or starts with a quote
             if (value.match(/^[\s\S]*?"$/g) || value.match(/^"[\s\S]*?$/g)) {
@@ -41,0 +49,0 @@ return false;

package.json

 {
   "name": "multi-ini",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "An ini-file parser which supports multi line, multiple levels and arrays to get a maximum of compatibility with Zend config files.",
@@ -5,0 +5,0 @@ "main": "lib/index.js",

README.md

@@ -50,5 +50,22 @@ # multi-ini [![Build Status](https://travis-ci.org/evangelion1204/multi-ini.png?branch=master)](https://travis-ci.org/evangelion1204/multi-ini) [![Coverage Status](https://coveralls.io/repos/evangelion1204/multi-ini/badge.svg?branch=master)](https://coveralls.io/r/evangelion1204/multi-ini?branch=master)
 ini = require('multi-ini');
-content = ini.read(file, {keep_quotes: false});
+content = ini.read(file, {keep_quotes: true});
 ```
 
+This will also affect the Serializer and serialized values. Using it will not quote anything automatically.
+```js
+{
+    production: {
+        quoted: '"quoted"',
+        not_quoted: 'not_quoted'
+    }
+}
+```
+
+Will result in a ini like
+```ini
+[production]
+quoted="quoted"
+not_quotes=not_quoted
+```
+
 #### filters
@@ -124,2 +141,5 @@
 
+### 1.0.1
+* Fixed bug with `keep_quotes` ignored when writing files
+
 ### 1.0.0
@@ -126,0 +146,0 @@ * First full release keeping backwards compatibility

src/multi-ini-class.js

@@ -46,2 +46,2 @@ 'use strict';
 
-module.exports = MultiIni
+module.exports = MultiIni;

src/serializer.js

@@ -8,2 +8,3 @@ 'use strict';
     line_breaks: 'unix',
+    keep_quotes: false,
 };
@@ -18,2 +19,7 @@
     needToBeQuoted(value) {
+        if (this.options.keep_quotes) {
+            return false;
+        }
+
+        // wrapped with qoutes
         if (value.match(/^"[\s\S]*?"$/g)) {
@@ -23,2 +29,3 @@ return false;
 
+        // escaped quote at the end
         if (value.match(/^[\s\S]*?\\"$/g)) {
@@ -28,2 +35,3 @@ return true;
 
+        // ends or starts with a quote
         if (value.match(/^[\s\S]*?"$/g) || value.match(/^"[\s\S]*?$/g)) {
@@ -30,0 +38,0 @@ return false;

test/backward_compatibility_spec.js

 'use strict';
 
-const chai = require('chai')
-const sinon = require('sinon')
-const sinonChai = require('sinon-chai')
-const expect = chai.expect
+const chai = require('chai');
+const sinon = require('sinon');
+const sinonChai = require('sinon-chai');
+const expect = chai.expect;
 
-chai.use(sinonChai)
+chai.use(sinonChai);
 
@@ -234,3 +234,3 @@ describe("Basic testing with backward compatibility to pre 0.3.x", function () {
 
-        MultiIni.write('test/out/combined_keep_quotes.ini', data);
+        MultiIni.write('test/out/combined_keep_quotes.ini', data, {keep_quotes: true});
 
@@ -237,0 +237,0 @@ var content = fs.readFileSync('test/out/combined_keep_quotes.ini', {encoding: 'utf8'});

test/basic_spec.js

 'use strict';
 
-const chai = require('chai')
-const sinon = require('sinon')
-const sinonChai = require('sinon-chai')
-const expect = chai.expect
+const chai = require('chai');
+const sinon = require('sinon');
+const sinonChai = require('sinon-chai');
+const expect = chai.expect;
 
-chai.use(sinonChai)
+chai.use(sinonChai);
 
@@ -329,3 +329,3 @@ describe("Basic testing includes reading of different files", function () {
 
-        MultiIni.write('test/out/combined_keep_quotes.ini', data);
+        ini.write('test/out/combined_keep_quotes.ini', data);
 
@@ -352,3 +352,3 @@ var content = fs.readFileSync('test/out/combined_keep_quotes.ini', {encoding: 'utf8'});
 
-        MultiIni.write('test/out/constant.ini', data);
+        ini.write('test/out/constant.ini', data);
     });
@@ -370,3 +370,3 @@
 
-        MultiIni.write('test/out/constant_keep.ini', data);
+        ini.write('test/out/constant_keep.ini', data);
     });
@@ -378,3 +378,3 @@
 
-        MultiIni.write('test/out/escaped_quotes.ini', data);
+        ini.write('test/out/escaped_quotes.ini', data);
 
@@ -391,2 +391,3 @@ var content = fs.readFileSync('test/out/escaped_quotes.ini', {encoding: 'utf8'});
         });
+
         var data = ini.read('test/data/single.ini');
@@ -427,3 +428,3 @@
     it("Read multi line ini of issue #12 with windows line breaks", function () {
-        var instance = new MultiIni.Class({line_breaks: 'windows'})
+        var instance = new MultiIni.Class({line_breaks: 'windows'});
         var data = instance.read('test/data/issue_12.ini');
@@ -430,0 +431,0 @@

test/constants_replacements_spec.js

 'use strict';
 
-const chai = require('chai')
-const sinon = require('sinon')
-const sinonChai = require('sinon-chai')
-const expect = chai.expect
+const chai = require('chai');
+const sinon = require('sinon');
+const sinonChai = require('sinon-chai');
+const expect = chai.expect;
 
-chai.use(sinonChai)
+chai.use(sinonChai);
 
@@ -10,0 +10,0 @@ describe("Constants replacement in ini files", function () {

test/filter_spec.js

 'use strict';
 
-const chai = require('chai')
-const sinon = require('sinon')
-const sinonChai = require('sinon-chai')
-const expect = chai.expect
+const chai = require('chai');
+const sinon = require('sinon');
+const sinonChai = require('sinon-chai');
+const expect = chai.expect;
 
-chai.use(sinonChai)
+chai.use(sinonChai);
 
@@ -10,0 +10,0 @@ describe("Filters", function () {

test/parser_spec.js

 'use strict';
 
-const chai = require('chai')
-const sinon = require('sinon')
-const sinonChai = require('sinon-chai')
-const expect = chai.expect
+const chai = require('chai');
+const sinon = require('sinon');
+const sinonChai = require('sinon-chai');
+const expect = chai.expect;
 
-chai.use(sinonChai)
+chai.use(sinonChai);
 
@@ -10,0 +10,0 @@ describe("Testing parser", function () {

Fixed alerts

Major refactor

Supply chain risk

Package has recently undergone a major refactor. It may be unstable or indicate significant internal changes. Use caution when updating to versions that include significant changes.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

77000

increased by2.81%

Number of package files

40

increased by2.56%

Lines of code

1600

increased by3.09%

Number of lines in readme file

171

increased by13.25%

Number of medium supply chain risk alerts

0

decreased by-100%

No dependency changes