aboutsummaryrefslogtreecommitdiff
path: root/src/modules/users.js
diff options
context:
space:
mode:
authorRoger Braun <roger@rogerbraun.net>2017-03-08 17:59:12 +0100
committerRoger Braun <roger@rogerbraun.net>2017-03-08 17:59:12 +0100
commit480a1ba253d49d4df36fbc64192e1260fc51a181 (patch)
tree8f9810c9ceba416b86ad0060e75c38a8e14c67f1 /src/modules/users.js
parentbde12418433f3ae9d2ad3781343f334caf58cc95 (diff)
Use cache to quickly access users.
Diffstat (limited to 'src/modules/users.js')
-rw-r--r--src/modules/users.js10
1 files changed, 6 insertions, 4 deletions
diff --git a/src/modules/users.js b/src/modules/users.js
index 31731880..dc910c91 100644
--- a/src/modules/users.js
+++ b/src/modules/users.js
@@ -3,9 +3,9 @@ import { compact, map, each, find, merge } from 'lodash'
import { set } from 'vue'
// TODO: Unify with mergeOrAdd in statuses.js
-export const mergeOrAdd = (arr, item) => {
+export const mergeOrAdd = (arr, obj, item) => {
if (!item) { return false }
- const oldItem = find(arr, {id: item.id})
+ const oldItem = obj[item.id]
if (oldItem) {
// We already have this, so only merge the new info.
merge(oldItem, item)
@@ -13,6 +13,7 @@ export const mergeOrAdd = (arr, item) => {
} else {
// This is a new item, prepare it
arr.push(item)
+ obj[item.id] = item
return {item, new: true}
}
}
@@ -32,7 +33,7 @@ export const mutations = {
state.loggingIn = false
},
addNewUsers (state, users) {
- each(users, (user) => mergeOrAdd(state.users, user))
+ each(users, (user) => mergeOrAdd(state.users, state.usersObject, user))
},
setUserForStatus (state, status) {
status.user = find(state.users, status.user)
@@ -42,7 +43,8 @@ export const mutations = {
export const defaultState = {
currentUser: false,
loggingIn: false,
- users: []
+ users: [],
+ usersObject: {}
}
const users = {