fix: handle pipes in content correctly (#7)

Author: tjconcept

fixtures/inputs.js

@@ -183,5 +183,15 @@ module.exports = {
       '| Sarah | 22    | true  |',
       '| Lee   | 23    | true  |'
     ].join(os.EOL) + os.EOL
+  },
+  pipes: {
+    input: [
+      { content: 'yes | no' }
+    ],
+    expected: [
+      '| Content   |',
+      '| --------- |',
+      '| yes \\| no |'
+    ].join(os.EOL) + os.EOL
   }
 }

index.js

@@ -14,7 +14,7 @@ module.exports = (input, options) => {
   }
 
   options = Object.assign({
-    stringify: v => typeof v === 'undefined' ? '' : String(v)
+    stringify: toString
   }, options, {
     wrap: Object.assign({
       width: Infinity,
@@ -165,3 +165,9 @@ function padStart (what, target, start) {
 function padEnd (what, target, start) {
   return what + repeat(' ', target - what.length)
 }
+
+function toString (v) {
+  if (typeof v === 'undefined') return ''
+
+  return String(v).replace(/\|/g, '\\|')
+}

readme.md

@@ -79,13 +79,14 @@ tablemark(input, [options = {}])
 - `{Array<Object>} input`: the data to table-ify
 - `{Object} [options = {}]`
 
-| key            | type         | default    | description                                  |
-| :------------: | :----------: | :--------: | -------------------------------------------- |
-| `columns`      | `<Array>`    | -          | Array of column descriptors.                 |
-| `caseHeaders`  | `<Boolean>`  | `true`     | Sentence case headers derived from keys.     |
-| `stringify`    | `<Function>` | -          | Provide a custom "toString" function.        |
-| `wrap.width`   | `<Number>`   | `Infinity` | Wrap texts at this length.                   |
-| `wrap.gutters` | `<Boolean>`  | `false`    | Add sides (`| <content> |`) to wrapped rows. |
+| key            | type         | default    | description                                    |
+| :------------: | :----------: | :--------: | ---------------------------------------------- |
+| `columns`      | `<Array>`    | -          | Array of column descriptors.                   |
+| `caseHeaders`  | `<Boolean>`  | `true`     | Sentence case headers derived from keys.       |
+| `stringify`    | `<Function>` | -          | Provide a custom "toString" function.          |
+| `wrap.width`   | `<Number>`   | `Infinity` | Wrap texts at this length.                     |
+| `wrap.gutters` | `<Boolean>`  | `false`    | Add sides (`\| <content> \|`) to wrapped rows. |
+
 
 The `columns` array can either contain objects, in which case their
 `name` and `align` properties will be used to alter the display of

test.js

@@ -47,3 +47,8 @@ test('gutters', t => {
   const result = fn(cases.gutters.input, cases.gutters.options)
   t.is(result, cases.gutters.expected)
 })
+
+test('pipes in content', t => {
+  const result = fn(cases.pipes.input, cases.pipes.options)
+  t.is(result, cases.pipes.expected)
+})