summaryrefslogtreecommitdiff
path: root/app/assets/javascripts/notes/stores/getters.js
diff options
context:
space:
mode:
authorFilipa Lacerda <filipa@gitlab.com>2017-07-28 19:59:31 +0100
committerFilipa Lacerda <filipa@gitlab.com>2017-07-28 19:59:42 +0100
commitf5a21c596b79c5f86c6b8034e1d0bf1d7078c593 (patch)
tree0ed02b93f530141b13c1ef04bde63c38cc627ffc /app/assets/javascripts/notes/stores/getters.js
parent9b87e680ca9653f40897ab8fa916d44fcfd1f4d5 (diff)
downloadgitlab-ce-f5a21c596b79c5f86c6b8034e1d0bf1d7078c593.tar.gz
[ci skip] Fix shortcuts for preview
Diffstat (limited to 'app/assets/javascripts/notes/stores/getters.js')
-rw-r--r--app/assets/javascripts/notes/stores/getters.js16
1 files changed, 5 insertions, 11 deletions
diff --git a/app/assets/javascripts/notes/stores/getters.js b/app/assets/javascripts/notes/stores/getters.js
index 8dc24cd745e..fab2252eeb8 100644
--- a/app/assets/javascripts/notes/stores/getters.js
+++ b/app/assets/javascripts/notes/stores/getters.js
@@ -10,17 +10,10 @@ export const getIssueDataByProp = state => prop => state.issueData[prop];
export const getUserData = state => state.userData;
export const getUserDataByProp = state => prop => state.notesData[prop];
-export const notesById = (state) => {
- const notesByIdObject = {};
- // TODO: FILIPA: TRANSFORM INTO A REDUCE
- state.notes.forEach((note) => {
- note.notes.forEach((n) => {
- notesByIdObject[n.id] = n;
- });
- });
-
- return notesByIdObject;
-};
+export const notesById = state => state.notes.reduce((acc, note) => {
+ note.notes.every(n => Object.assign(acc, { [n.id]: n }));
+ return acc;
+}, {});
const reverseNotes = array => array.slice(0).reverse();
const isLastNote = (note, userId) => !note.system && note.author.id === userId;
@@ -31,6 +24,7 @@ export const getCurrentUserLastNote = state => userId => reverseNotes(state.note
return acc;
}, []).filter(el => el !== undefined)[0];
+// eslint-disable-next-line no-unused-vars
export const getDiscussionLastNote = state => (discussion, userId) => reverseNotes(discussion.notes)
.find(el => isLastNote(el, userId));