From 0eeb4753ae85f1e4bbcc113c74b6acff203c4709 Mon Sep 17 00:00:00 2001
From: Robillard Sebastien <srobillard@wideagency.com>
Date: Mon, 31 Jan 2022 15:45:57 +0100
Subject: [PATCH 1/6] Fix issue #9 - Custom parameters added on
 data-call.params HTML attribute

---
 src/directives.js | 20 +++++++++++++++-----
 src/utils.js      |  4 ++--
 2 files changed, 17 insertions(+), 7 deletions(-)

diff --git a/src/directives.js b/src/directives.js
index a95cd15..d20104c 100644
--- a/src/directives.js
+++ b/src/directives.js
@@ -1,13 +1,16 @@
 import observe from '@wide/dom-observer'
 import { seek } from './index'
+import { parseDataCallParams } from './utils'
 
 const DEFAULT_TOGGLE_CLASS = '-active'
 
 
 /**
  * Run component's method from HTML
- * Ex: [data-call="modal#register.open"] -> modulus.seek('modal', '#register').open()
- * Ex with params: [data-call="modal#register.open"] [data-call.params="$el"] -> modulus.seek('modal', '#register').open(el)
+ * Ex: [data-call="modal#register.open"]
+ * Ex with params: [data-call="modal#register.open"] [data-call.params='[{ "myAttribute": "myValue" }]']
+ * 
+*  -> modulus.seek('modal', '#register').open({ el, e, data })
  */
 observe('[data-call]', {
   bind(el) {
@@ -17,10 +20,17 @@ observe('[data-call]', {
         const component = seek(name, id)
         if(component) {
           const params = el.dataset['call.params']
+          let data = null
+          
+          if (params) {
+            try {
+              data = JSON.parse(params)?.[0]
+            } catch(e) {
+              console.error('Invalid JSON format in `data-call.params`.', e)
+            }
+          }
 
-          if (params === '$el') component[method](el)
-          else if (params === '$event') component[method](e)
-          else component[method]()
+          component[method]({ el, e, data })
         } else console.error(`Unknown component "${str}"`)
       }
       else console.error(`Invalid call string "${str}"`)
diff --git a/src/utils.js b/src/utils.js
index 473a92b..8b0c40b 100644
--- a/src/utils.js
+++ b/src/utils.js
@@ -73,6 +73,6 @@ export function parseRefs(el, uid) {
  * @param {String} str 
  * @return {String}
  */
-export function camelize(str) {
+ export function camelize(str) {
   return str.replace(/[\s-]+(\w)/g, (m, c) => c.toUpperCase())
-}
\ No newline at end of file
+}

From 18513429d4a9da766b31de169add48e47000b8c9 Mon Sep 17 00:00:00 2001
From: Robillard Sebastien <srobillard@wideagency.com>
Date: Mon, 31 Jan 2022 16:13:31 +0100
Subject: [PATCH 2/6] Fix issue #9 - Readme updated

---
 readme.md | 38 ++++++++++++++++++++++++++++++--------
 1 file changed, 30 insertions(+), 8 deletions(-)

diff --git a/readme.md b/readme.md
index 46c83e7..447867b 100644
--- a/readme.md
+++ b/readme.md
@@ -103,22 +103,44 @@ Use the `[data-call]` helper with a formatted value `name#id.method`:
 
 will internally trigger:
 ```js
-modulus.seek('modal', '#register').open()
+modulus.seek('modal', '#register').open({ el, e, data })
 ```
 
-#### Parameters
-Use the `[data-call.params]` to pass some predefined values:
-
 | Value | Description |
-|---|---|---|---|
-| `$event` | `Event` object of the event listener method callback |
-| `$el` | Element object binded to the event |
+|---|---|
+| `el` | HTMLElement object binded to the event |
+| `e` | `Event` object of the event listener method callback |
+| `data` | Optional parameters defined in `[data-call.params]` |
+
+#### Parameters
+Use the `[data-call.params]` to pass custom values:
 
 ```html
-<button data-call="modal#register.open" data-call.params="$event">do something</button>
+<button data-call="modal#register.open" data-call.params='[{ "myAttr": "myValue" }]'>do something</button>
 ```
 
+> ⚠️ Note: `data-call.params` is waiting a JSON format only
+
+Exmple with the previous HTML code:
+```js
+modulus.component('modal', class extends Component {
+  run() {
+    // ...
+  }
 
+  /**
+   * Open modal and do some stuff
+   *
+   * @params {HTMLElement} el
+   * @params {Event} e
+   * @params {Object} [data]
+   */
+  open({ el, e, data }) {
+    // el: <button ...>
+    // e: Event{ ... }
+    // data: { ... } | null
+  }
+```
 
 ## Component class
 

From f5a543c423df572983f77d35f96b80481d18ffb9 Mon Sep 17 00:00:00 2001
From: Robillard Sebastien <srobillard@wideagency.com>
Date: Mon, 31 Jan 2022 16:29:35 +0100
Subject: [PATCH 3/6] Fix issue #9 - Readme updated

---
 readme.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/readme.md b/readme.md
index 447867b..79706a0 100644
--- a/readme.md
+++ b/readme.md
@@ -133,7 +133,7 @@ modulus.component('modal', class extends Component {
    *
    * @params {HTMLElement} el
    * @params {Event} e
-   * @params {Object} [data]
+   * @params {Object|null} [data]
    */
   open({ el, e, data }) {
     // el: <button ...>

From 2c27d6b5f3c22fa0487f30efbfa4ed919fb8cfd3 Mon Sep 17 00:00:00 2001
From: Aymeric <aymeric.assier@accenture.com>
Date: Tue, 8 Feb 2022 11:41:45 +0100
Subject: [PATCH 4/6] Add configurable log levels system

---
 readme.md         | 21 ++++++++++++++++
 src/directives.js |  5 ++--
 src/index.js      | 31 +++++++++++++++++++++---
 src/logger.js     | 62 +++++++++++++++++++++++++++++++++++++++++------
 4 files changed, 106 insertions(+), 13 deletions(-)

diff --git a/readme.md b/readme.md
index 79706a0..c8b6260 100644
--- a/readme.md
+++ b/readme.md
@@ -193,6 +193,27 @@ modulus.component('foo-bar', class extends Component {
 Every event listeners created using `this.on()` are automatically `off()`ed on component destruction.
 
 
+## Config
+
+### Log level
+
+To keep only `warn` and `error` logs (for production usage), set `silent`:
+```js
+import modulus from '@wide/modulus'
+
+modulus.config({ silent: true })
+```
+
+Or manually assign a log level:
+```js
+import modulus, { LOG_LEVELS } from '@wide/modulus'
+
+modulus.config({
+  logLevel: LOG_LEVELS.INFO // DEBUG (default), INFO, WARN, ERROR, NONE
+})
+```
+
+
 ## Authors
 
 - **Aymeric Assier** - [github.com/myeti](https://github.com/myeti)
diff --git a/src/directives.js b/src/directives.js
index d20104c..4730222 100644
--- a/src/directives.js
+++ b/src/directives.js
@@ -1,4 +1,5 @@
 import observe from '@wide/dom-observer'
+import { logger } from './logger'
 import { seek } from './index'
 import { parseDataCallParams } from './utils'
 
@@ -31,9 +32,9 @@ observe('[data-call]', {
           }
 
           component[method]({ el, e, data })
-        } else console.error(`Unknown component "${str}"`)
+        } else logger.error(`Unknown component "${str}"`)
       }
-      else console.error(`Invalid call string "${str}"`)
+      else logger.error(`Invalid call string "${str}"`)
     })
   }
 })
diff --git a/src/index.js b/src/index.js
index 0dc6ba7..e724c80 100644
--- a/src/index.js
+++ b/src/index.js
@@ -1,7 +1,20 @@
 import observe, { seek as _seek } from '@wide/dom-observer'
+import { LOG_LEVELS, Logger, logger } from './logger'
 import { bind, unbind } from './hooks'
 import './directives'
 
+export { LOG_LEVELS } from './logger'
+
+
+/**
+ * Default internal config values
+ * @type {Object}
+ */
+const DEFAULT_CONFIG = {
+  silent: false,
+  logLevel: LOG_LEVELS.DEBUG
+}
+
 
 /**
  * Register many regular and custom elements
@@ -39,7 +52,7 @@ export function registerMany(many) {
  * @param {Function} Klass 
  */
 export function registerComponent(name, Klass) {
-  console.debug(`# register component [is=${name}]`)
+  logger(`# register component [is=${name}]`)
   observe(`[is="${name}"]`, {
     bind: el => bind(el, name, Klass),
     unbind: el => unbind(el)
@@ -64,7 +77,7 @@ export function registerComponents(collection) {
  * @param {Function} Klass 
  */
 export function registerWebComponent(name, Klass) {
-  console.debug(`# register web component <${name}>`)
+  logger(`# register web component <${name}>`)
   try {
     window.customElements.define(name, class extends HTMLElement {
       connectedCallback() {
@@ -76,7 +89,7 @@ export function registerWebComponent(name, Klass) {
     })
   }
   catch(err) {
-    console.error(err)
+    logger.error(err)
   }
 }
 
@@ -131,6 +144,17 @@ export function seekAll(name, selector) {
 }
 
 
+/**
+ * Set internal config
+ * - assign log level
+ * @param {Object} values 
+ */
+export function setConfig(values = {}) {
+  const config = Object.assign({}, DEFAULT_CONFIG, values)
+  Logger.LEVEL = config.silent ? LOG_LEVELS.WARN : config.logLevel
+}
+
+
 /**
  * Extend modulus instance
  */
@@ -143,6 +167,7 @@ modulus.webComponents = registerWebComponents
 modulus.imports = registerImports
 modulus.seek = seek
 modulus.seekAll = seekAll
+modulus.config = setConfig
 
 
 /**
diff --git a/src/logger.js b/src/logger.js
index 5b3e99a..1e37ef0 100644
--- a/src/logger.js
+++ b/src/logger.js
@@ -1,16 +1,62 @@
+/**
+ * All available log levels
+ * @type {Object<String, Number>}
+ */
+export const LOG_LEVELS = {
+  DEBUG: 0,
+  INFO: 1,
+  WARN: 2,
+  ERROR: 3,
+  NONE: Infinity
+}
+
+
+/**
+ * Map levels to real console functions
+ * @type {Object<String, Function>}
+ */
+const LOG_METHODS = {
+  [LOG_LEVELS.DEBUG]: console.debug,
+  [LOG_LEVELS.INFO]: console.log,
+  [LOG_LEVELS.WARN]: console.warn,
+  [LOG_LEVELS.ERROR]: console.error
+}
+
+
+/**
+ * Write log depending on the level requested
+ * @param {Number} level 
+ * @param {String} prefix 
+ * @param {Array} args 
+ */
+function writeLog(level, prefix, args) {
+  const method = LOG_METHODS[level]
+  if(method && Logger.LEVEL <= level) method(prefix, ...args)
+}
+
+
 /**
  * Logger factory
  * @param {String} prefix 
  */
 export default function Logger(prefix = '') {
+  const logger = (...args) => writeLog(LOG_LEVELS.DEBUG, prefix, args)
+  logger.info = (...args) => writeLog(LOG_LEVELS.INFO, prefix, args)
+  logger.warn = (...args) => writeLog(LOG_LEVELS.WARN, prefix, args)
+  logger.error = (...args) => writeLog(LOG_LEVELS.ERROR, prefix, args)
+  return logger
+}
+
 
-  const logger = function(...args) {
-    console.debug(prefix, ...args)
-  }
+/**
+ * Logger global level
+ * @type {Number} 
+ */
+Logger.LEVEL = LOG_LEVELS.DEBUG
 
-  logger.info = (...args) => console.log(prefix, ...args)
-  logger.warn = (...args) => console.warn(prefix, ...args)
-  logger.error = (...args) => console.error(prefix, ...args)
 
-  return logger
-}
\ No newline at end of file
+/**
+ * Built-in instance
+ * @type {Logger}
+ */
+export const logger = new Logger()
\ No newline at end of file

From 16e9da0111e4d5a4ebf3701c5528a94e084f6662 Mon Sep 17 00:00:00 2001
From: Robillard Sebastien <srobillard@wideagency.com>
Date: Fri, 11 Feb 2022 16:19:35 +0100
Subject: [PATCH 5/6] Add enabled attribute

---
 package.json      |  4 +++-
 readme.md         | 28 +++++++++++++++++++++++++---
 src/directives.js |  3 +--
 src/index.js      | 19 ++++++++++++++++---
 4 files changed, 45 insertions(+), 9 deletions(-)

diff --git a/package.json b/package.json
index d548669..7897231 100644
--- a/package.json
+++ b/package.json
@@ -6,7 +6,9 @@
   "author": "Aymeric Assier (https://github.com/myeti)",
   "contributors": [
     "Aymeric Assier (https://github.com/myeti)",
-    "Julien Martins Da Costa (https://github.com/jdacosta)"
+    "Julien Martins Da Costa (https://github.com/jdacosta)",
+    "Sébastien Robillard (https://github.com/robiseb)",
+    "Kévin Poccard Soudard (https://github.com/kevpoccs)"
   ],
   "repository": {
     "type": "git",
diff --git a/readme.md b/readme.md
index c8b6260..12ea7ae 100644
--- a/readme.md
+++ b/readme.md
@@ -197,11 +197,11 @@ Every event listeners created using `this.on()` are automatically `off()`ed on c
 
 ### Log level
 
-To keep only `warn` and `error` logs (for production usage), set `silent`:
+To keep only `warn` and `error` logs (for production usage), set `production` to `true`:
 ```js
 import modulus from '@wide/modulus'
 
-modulus.config({ silent: true })
+modulus.config({ production: true })
 ```
 
 Or manually assign a log level:
@@ -209,16 +209,38 @@ Or manually assign a log level:
 import modulus, { LOG_LEVELS } from '@wide/modulus'
 
 modulus.config({
-  logLevel: LOG_LEVELS.INFO // DEBUG (default), INFO, WARN, ERROR, NONE
+  log: {
+    level: LOG_LEVELS.INFO // DEBUG (default), INFO, WARN, ERROR, NONE
+  }
 })
 ```
 
+> ⚠️ Note: assign a log level will override the `production` setting.
+
+To disable logs, set `enabled` to `false`:
+```js
+import modulus from '@wide/modulus'
+
+modulus.config({
+  log: {
+    enabled: false
+  }
+})
+```
+
+The default config is setted to show all kind of logs.
+
 
 ## Authors
 
 - **Aymeric Assier** - [github.com/myeti](https://github.com/myeti)
 - **Julien Martins Da Costa** - [github.com/jdacosta](https://github.com/jdacosta)
 
+### Contributors
+
+- **Sébastien Robillard** - [github.com/robiseb](https://github.com/robiseb)
+- **Kévin Poccard Soudard** - [github.com/kevpoccs](https://github.com/kevpoccs)
+
 
 ## License
 
diff --git a/src/directives.js b/src/directives.js
index 4730222..74ee1a7 100644
--- a/src/directives.js
+++ b/src/directives.js
@@ -1,7 +1,6 @@
 import observe from '@wide/dom-observer'
 import { logger } from './logger'
 import { seek } from './index'
-import { parseDataCallParams } from './utils'
 
 const DEFAULT_TOGGLE_CLASS = '-active'
 
@@ -27,7 +26,7 @@ observe('[data-call]', {
             try {
               data = JSON.parse(params)?.[0]
             } catch(e) {
-              console.error('Invalid JSON format in `data-call.params`.', e)
+              logger.error('Invalid JSON format in `data-call.params`.', e)
             }
           }
 
diff --git a/src/index.js b/src/index.js
index e724c80..4d6a0f0 100644
--- a/src/index.js
+++ b/src/index.js
@@ -11,8 +11,11 @@ export { LOG_LEVELS } from './logger'
  * @type {Object}
  */
 const DEFAULT_CONFIG = {
-  silent: false,
-  logLevel: LOG_LEVELS.DEBUG
+  production: false,
+  log: {
+    enabled: true,
+    level: null
+  }
 }
 
 
@@ -151,7 +154,17 @@ export function seekAll(name, selector) {
  */
 export function setConfig(values = {}) {
   const config = Object.assign({}, DEFAULT_CONFIG, values)
-  Logger.LEVEL = config.silent ? LOG_LEVELS.WARN : config.logLevel
+  const { enabled, level } = config.log
+
+  if (enabled) {
+    if (level) {
+      Logger.LEVEL = Object.values(LOG_LEVELS).includes(level) ? level : LOG_LEVELS.DEBUG
+    } else {
+      Logger.LEVEL = config.production ? LOG_LEVELS.WARN : LOG_LEVELS.DEBUG
+    }
+  } else {
+    Logger.LEVEL = LOG_LEVELS.NONE
+  }
 }
 
 

From e3b45fb44827d8c4477ed535ebabe674a101bd9f Mon Sep 17 00:00:00 2001
From: Robillard Sebastien <srobillard@wideagency.com>
Date: Tue, 15 Feb 2022 15:01:35 +0100
Subject: [PATCH 6/6] Fixes for issues #12 and #13

---
 package.json | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package.json b/package.json
index 7897231..418b1a4 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@wide/modulus",
-  "version": "2.1.2",
+  "version": "2.2.0",
   "description": "Robust Web Component interface",
   "license": "MIT",
   "author": "Aymeric Assier (https://github.com/myeti)",