quote-props

Author: guyonroche

.eslintrc

@@ -15,12 +15,11 @@
     "no-multi-assign": ["off"],
     "one-var": ["off"],
     "one-var-declaration-per-line": ["off"],
+    "vars-on-top": ["off"],
 
     "no-var": ["off"],
-    "no-prototype-builtins": ["off"],
     "arrow-parens": ["off"],
-    "quote-props": ["off"],
-    "vars-on-top": ["off"],
+    "no-prototype-builtins": ["off"],
     "no-param-reassign": ["off"],
     "prefer-arrow-callback": ["off"],
     "comma-dangle": ["off"],

lib/xlsx/xform/core/app-xform.js

@@ -16,10 +16,10 @@ var AppTitleOfPartsXform = require('./app-titles-of-parts-xform');
 
 var AppXform = module.exports = function() {
   this.map = {
-    'Company': new StringXform({tag: 'Company'}),
-    'Manager': new StringXform({tag: 'Manager'}),
-    'HeadingPairs': new AppHeadingPairsXform(),
-    'TitleOfParts': new AppTitleOfPartsXform()
+    Company: new StringXform({tag: 'Company'}),
+    Manager: new StringXform({tag: 'Manager'}),
+    HeadingPairs: new AppHeadingPairsXform(),
+    TitleOfParts: new AppTitleOfPartsXform()
   };
 };
 
@@ -29,7 +29,7 @@ AppXform.DateFormat = function(dt) {
 AppXform.DateAttrs = {'xsi:type': 'dcterms:W3CDTF'};
 
 AppXform.PROPERTY_ATTRIBUTES = {
-  'xmlns': 'http://schemas.openxmlformats.org/officeDocument/2006/extended-properties',
+  xmlns: 'http://schemas.openxmlformats.org/officeDocument/2006/extended-properties',
   'xmlns:vt': 'http://schemas.openxmlformats.org/officeDocument/2006/docPropsVTypes'
 };
 

lib/xlsx/xform/style/styles-xform.js

@@ -57,7 +57,7 @@ var StylesXform = module.exports = function(initialise) {
 
 utils.inherits(StylesXform, BaseXform, {
   STYLESHEET_ATTRIBUTES: {
-    'xmlns': 'http://schemas.openxmlformats.org/spreadsheetml/2006/main',
+    xmlns: 'http://schemas.openxmlformats.org/spreadsheetml/2006/main',
     'xmlns:mc': 'http://schemas.openxmlformats.org/markup-compatibility/2006',
     'mc:Ignorable': 'x14ac x16r2',
     'xmlns:x14ac': 'http://schemas.microsoft.com/office/spreadsheetml/2009/9/ac',

spec/unit/doc/workbook.spec.js

@@ -86,15 +86,15 @@ describe('Workbook', function() {
     var wb = new Excel.Workbook();
     var ws = wb.addWorksheet('blort');
     ws.getCell('A1').value = {
-      'richText': [
-        {'font': {'size': 12, 'color': {'theme': 0}, 'name': 'Calibri', 'family': 2, 'scheme': 'minor'}, 'text': 'This is '},
-        {'font': {'italic': true, 'size': 12, 'color': {'theme': 0}, 'name': 'Calibri', 'scheme': 'minor'}, 'text': 'a'},
-        {'font': {'size': 12, 'color': {'theme': 1}, 'name': 'Calibri', 'family': 2, 'scheme': 'minor'}, 'text': ' '},
-        {'font': {'size': 12, 'color': {'argb': 'FFFF6600'}, 'name': 'Calibri', 'scheme': 'minor'}, 'text': 'colorful'},
-        {'font': {'size': 12, 'color': {'theme': 1}, 'name': 'Calibri', 'family': 2, 'scheme': 'minor'}, 'text': ' text '},
-        {'font': {'size': 12, 'color': {'argb': 'FFCCFFCC'}, 'name': 'Calibri', 'scheme': 'minor'}, 'text': 'with'},
-        {'font': {'size': 12, 'color': {'theme': 1}, 'name': 'Calibri', 'family': 2, 'scheme': 'minor'}, 'text': ' in-cell '},
-        {'font': {'bold': true, 'size': 12, 'color': {'theme': 1}, 'name': 'Calibri', 'family': 2, 'scheme': 'minor'}, 'text': 'format'}
+      richText: [
+        {font: {size: 12, color: {theme: 0}, name: 'Calibri', family: 2, scheme: 'minor'}, text: 'This is '},
+        {font: {italic: true, size: 12, color: {theme: 0}, name: 'Calibri', scheme: 'minor'}, text: 'a'},
+        {font: {size: 12, color: {theme: 1}, name: 'Calibri', family: 2, scheme: 'minor'}, text: ' '},
+        {font: {size: 12, color: {argb: 'FFFF6600'}, name: 'Calibri', scheme: 'minor'}, text: 'colorful'},
+        {font: {size: 12, color: {theme: 1}, name: 'Calibri', family: 2, scheme: 'minor'}, text: ' text '},
+        {font: {size: 12, color: {argb: 'FFCCFFCC'}, name: 'Calibri', scheme: 'minor'}, text: 'with'},
+        {font: {size: 12, color: {theme: 1}, name: 'Calibri', family: 2, scheme: 'minor'}, text: ' in-cell '},
+        {font: {bold: true, size: 12, color: {theme: 1}, name: 'Calibri', family: 2, scheme: 'minor'}, text: 'format'}
       ]
     };
 

spec/unit/xlsx/xform/strings/shared-string-xform.spec.js

@@ -15,9 +15,9 @@ var expectations = [
   {
     title: 'rich text',
     create: function() { return new SharedStringXform(); },
-    preparedModel: {richText: [{ 'font': { 'size': 11, 'bold': true, 'color': { 'theme': 1}, 'name': 'Calibri', 'family': 2, 'scheme': 'minor' }, 'text': 'Bold,' }]},
+    preparedModel: {richText: [{ font: { size: 11, bold: true, color: { theme: 1}, name: 'Calibri', family: 2, scheme: 'minor' }, text: 'Bold,' }]},
     xml: '<si><r><rPr><b/><color theme="1"/><family val="2"/><scheme val="minor"/><sz val="11"/><rFont val="Calibri"/></rPr><t>Bold,</t></r></si>',
-    parsedModel: {richText: [{ 'font': { 'size': 11, 'bold': true, 'color': { 'theme': 1}, 'name': 'Calibri', 'family': 2, 'scheme': 'minor' }, 'text': 'Bold,' }]},
+    parsedModel: {richText: [{ font: { size: 11, bold: true, color: { theme: 1}, name: 'Calibri', family: 2, scheme: 'minor' }, text: 'Bold,' }]},
     tests: ['render', 'renderIn', 'parse']
   },
   {