From 6542765cd88f8e619cbcf7077548f4fb35294629 Mon Sep 17 00:00:00 2001 From: Daniel Almaguer Date: Fri, 13 Oct 2017 22:24:36 -0500 Subject: [PATCH] Enable logActions by default and not deepClone state for actions --- docs/guide/plugins.md | 2 +- src/plugins/logger.js | 6 ++---- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/docs/guide/plugins.md b/docs/guide/plugins.md index bb13ed555..c89c1bfb3 100644 --- a/docs/guide/plugins.md +++ b/docs/guide/plugins.md @@ -128,7 +128,7 @@ const logger = createLogger({ // Same as mutationTransformer but for actions return action.type }, - logActions: false, // Log Actions + logActions: true, // Log Actions logMutations: true, // Log mutations logger: console, // implementation of the `console` API, default `console` }) diff --git a/src/plugins/logger.js b/src/plugins/logger.js index 496dc43b7..ccf2b6537 100644 --- a/src/plugins/logger.js +++ b/src/plugins/logger.js @@ -9,7 +9,7 @@ export default function createLogger ({ mutationTransformer = mut => mut, actionFilter = (action, state) => true, actionTransformer = act => act, - logActions = false, + logActions = true, logMutations = true, logger = console } = {}) { @@ -42,9 +42,7 @@ export default function createLogger ({ if (logActions) { store.subscribeAction((action, state) => { - const currentState = deepCopy(state) - - if (actionFilter(action, currentState)) { + if (actionFilter(action, state)) { const formattedTime = getFormattedTime() const formattedAction = actionTransformer(action) const message = `action ${action.type}${formattedTime}`