fix(vuex): vuex 1.x compat, closes vuejs#916

Author: Guillaume Chau

src/backend/hook.js

@@ -103,25 +103,29 @@ export function installHook (target) {
       getters: store.getters
     }
     // Dynamic modules
-    hook.storeModules = []
-    const origRegister = store.registerModule.bind(store)
-    store.registerModule = (path, module, options) => {
-      if (typeof path === 'string') path = [path]
-      hook.storeModules.push({ path, module, options })
-      origRegister(path, module, options)
-    }
-    const origUnregister = store.unregisterModule.bind(store)
-    store.unregisterModule = (path) => {
-      if (typeof path === 'string') path = [path]
-      const key = path.join('/')
-      const index = hook.storeModules.findIndex(m => m.path.join('/') === key)
-      if (index !== -1) hook.storeModules.splice(0, 1)
-      origUnregister(path)
-    }
-    hook.flushStoreModules = () => {
-      store.registerModule = origRegister
-      store.unregisterModule = origUnregister
-      return hook.storeModules
+    if (store.registerModule) {
+      hook.storeModules = []
+      const origRegister = store.registerModule.bind(store)
+      store.registerModule = (path, module, options) => {
+        if (typeof path === 'string') path = [path]
+        hook.storeModules.push({ path, module, options })
+        origRegister(path, module, options)
+      }
+      const origUnregister = store.unregisterModule.bind(store)
+      store.unregisterModule = (path) => {
+        if (typeof path === 'string') path = [path]
+        const key = path.join('/')
+        const index = hook.storeModules.findIndex(m => m.path.join('/') === key)
+        if (index !== -1) hook.storeModules.splice(0, 1)
+        origUnregister(path)
+      }
+      hook.flushStoreModules = () => {
+        store.registerModule = origRegister
+        store.unregisterModule = origUnregister
+        return hook.storeModules
+      }
+    } else {
+      hook.flushStoreModules = () => []
     }
   })
 

src/backend/vuex.js

@@ -70,27 +70,33 @@ export function initVuexBackend (hook, bridge, isLegacy) {
     }
   }
 
-  const origRegisterModule = store.registerModule.bind(store)
-  store.registerModule = (path, module, options) => {
-    addModule(path, module, options)
-    origRegisterModule(path, module, options)
-  }
+  let origRegisterModule, origUnregisterModule
 
-  const origUnregisterModule = store.unregisterModule.bind(store)
-  store.unregisterModule = (path) => {
-    if (typeof path === 'string') path = [path]
+  if (store.registerModule) {
+    origRegisterModule = store.registerModule.bind(store)
+    store.registerModule = (path, module, options) => {
+      addModule(path, module, options)
+      origRegisterModule(path, module, options)
+    }
 
-    delete registeredModules[path.join('/')]
+    origUnregisterModule = store.unregisterModule.bind(store)
+    store.unregisterModule = (path) => {
+      if (typeof path === 'string') path = [path]
 
-    if (SharedData.recordVuex) {
-      addMutation(`Unregister module: ${path.join('/')}`, {
-        path
-      }, {
-        unregisterModule: true
-      })
-    }
+      delete registeredModules[path.join('/')]
 
-    origUnregisterModule(path)
+      if (SharedData.recordVuex) {
+        addMutation(`Unregister module: ${path.join('/')}`, {
+          path
+        }, {
+          unregisterModule: true
+        })
+      }
+
+      origUnregisterModule(path)
+    }
+  } else {
+    origRegisterModule = origUnregisterModule = () => {}
   }
 
   bridge.send('vuex:init')