Merge pull request #69 from posthtml/refactor-tags

Author: cossssmin

lib/index.js

@@ -1,21 +1,22 @@
-'use strict'
-
-const isUrl = require('is-url-superb')
 const qs = require('query-string')
+const isUrl = require('is-url-superb')
+const matchHelper = require('posthtml-match-helper')
+
+module.exports = (config = {}) => tree => {
+  config.strict = typeof config.strict === 'boolean' ? config.strict : true
 
-module.exports = config => tree => {
   const process = node => {
     if (!config || !config.parameters) {
       return node
     }
 
     const tags = config && config.tags ? config.tags : ['a']
 
-    if (tags.includes(node.tag) && node.attrs && node.attrs.href) {
+    tree.match(matchHelper(tags.join(',')), node => {
       const url = node.attrs.href
-      const parsed = qs.parseUrl(url)
+      const parsed = qs.parseUrl(url, config.qs)
 
-      if (isUrl(parsed.url.trim()) === false) {
+      if (config.strict && !isUrl(parsed.url.trim())) {
         return node
       }
 
@@ -24,7 +25,9 @@ module.exports = config => tree => {
       })
 
       node.attrs.href = qs.stringifyUrl(parsed, config.qs)
-    }
+
+      return node
+    })
 
     return node
   }

package-lock.json

@@ -29,6 +29,7 @@
   "dependencies": {
     "is-url-superb": "^5.0.0",
     "posthtml": "^0.16.6",
+    "posthtml-match-helper": "^1.0.3",
     "query-string": "^7.1.1"
   },
   "devDependencies": {

package.json

@@ -59,7 +59,9 @@ require('posthtml-url-parameters')({
 
 Default: `[a]`
 
-Array of tag names to process. Only URLs inside `href=""` attributes of tags in this array will be processed.
+Array of tag names to process. 
+
+By default, only URLs inside `href=""` attributes of tags in this array will be processed.
 
 Example:
 
@@ -70,13 +72,48 @@ require('posthtml-url-parameters')({
 })
 ```
 
+You may use some CSS selectors when specifying tags:
+
+```js
+require('posthtml-url-parameters')({
+  tags: ['a.button', 'a[href*="example.com"]' 'link'],
+  // ...
+})
+```
+
+All [`posthtml-match-helper` selectors](https://github.com/posthtml/posthtml-match-helper) are supported.
+
+### `strict`
+
+Default: `false`
+
+By default, the plugin will append query parameters only to valid URLs.
+
+You may disable `strict` mode to append parameters to any string:
+
+```js
+const posthtml = require('posthtml')
+const urlParams = require('posthtml-url-parameters')
+
+posthtml([
+    urlParams({
+      parameters: { foo: 'bar' },
+      strict: false,
+    })
+  ])
+  .process('<a href="https://example.com/campaigns/{{ id }}">Details</div>')
+  .then(result => console.log(result.html)))
+
+  // <a href="https://example.com/campaigns/{{ id }}?foo=bar">Details</div>
+```
+
 ### `qs`
 
 Default: `undefined`
 
 Options to pass to `query-string` - see available options [here](https://github.com/sindresorhus/query-string#stringifyobject-options).
 
-For example, you can disable encoding:
+For example, you may disable encoding:
 
 ```js
 const posthtml = require('posthtml')