CODEWITHSUNDEEP
javascriptvue.jsvuejs2vuex
Mateusz Urbański
Mateusz Urbański

Reputation: 7862

vuex: unknown getter: user

I'm experimenting with vuex and I was looking for best way to organize my vuex files I finished with something like this:

/src/store/user/state.js:

export default {
  state: {
    user: null
  }
}

/src/store/user/getters.js:

export default {
  getters: {
    user (state) {
      return state.user
    }
  }
}

/src/store/user/mutations.js:

export default {
  mutations: {
    'SET_USER' (state, user) {
      state.user = user
    }
  }
}

/src/store/user/actions.js

export default {
  actions: {
    loginUser ({ commit }, params) {
       commit('SET_USER', {id: 1})
    }
  }
}

/src/store/user/index.js

import state from './state'
import getters from './getters'
import actions from './actions'
import mutations from './mutations'

export default {
  state,
  getters,
  actions,
  mutations
}

/src/store/index.js:

import Vue from 'vue'
import Vuex from 'vuex'
import user from './user'

Vue.use(Vuex)

export default new Vuex.Store({
  modules: {
    user
  }
})

When I load my code it returns in console following error: 

vuex: unknown getter: user

Upvotes: 10

Views: 29300

Answers (1)

thanksd
thanksd

Reputation: 55664

Each of your user-related files are using export default, which means when you import those files, you are naming the entire object being exported state, getters, etc.

So, in the scope of index, the state variable has a property named state, the getters variable has a property named getters, etc. And this is throwing things off.

You should export a const for each of these files instead:

export const state = {
  user: null,
}

And then when importing grab the named const like so:

import { state } from './state'

Alternatively, you could just remove the properties for state, getters, etc. from each file:

// state.js
export default {
  user: null,
}

And then import like you're doing already:

import state from './state'

Upvotes: 16

Related Questions