Enable logActions by default and not deepClone state for actions

vuejs · Apr 21, 2020 · 6542765 · 6542765
1 parent 3881bbd
commit 6542765
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git 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
@@ -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}`