further cosmetic refactoring

web/react/stores/post_store.jsx

@@ -6,7 +6,6 @@ var EventEmitter = require('events').EventEmitter;
 var assign = require('object-assign');
 
 var ChannelStore = require('../stores/channel_store.jsx');
-var UserStore = require('../stores/user_store.jsx');
 var BrowserStore = require('../stores/browser_store.jsx');
 
 var Constants = require('../utils/constants.jsx');
@@ -21,149 +20,148 @@ var ADD_MENTION_EVENT = 'add_mention';
 
 var PostStore = assign({}, EventEmitter.prototype, {
 
-    emitChange: function() {
+    emitChange: function emitChange() {
         this.emit(CHANGE_EVENT);
     },
 
-    addChangeListener: function(callback) {
+    addChangeListener: function addChangeListener(callback) {
         this.on(CHANGE_EVENT, callback);
     },
 
-    removeChangeListener: function(callback) {
+    removeChangeListener: function removeChangeListener(callback) {
         this.removeListener(CHANGE_EVENT, callback);
     },
 
-    emitSearchChange: function() {
+    emitSearchChange: function emitSearchChange() {
         this.emit(SEARCH_CHANGE_EVENT);
     },
 
-    addSearchChangeListener: function(callback) {
+    addSearchChangeListener: function addSearchChangeListener(callback) {
         this.on(SEARCH_CHANGE_EVENT, callback);
     },
 
-    removeSearchChangeListener: function(callback) {
+    removeSearchChangeListener: function removeSearchChangeListener(callback) {
         this.removeListener(SEARCH_CHANGE_EVENT, callback);
     },
 
-    emitSearchTermChange: function(doSearch, isMentionSearch) {
+    emitSearchTermChange: function emitSearchTermChange(doSearch, isMentionSearch) {
         this.emit(SEARCH_TERM_CHANGE_EVENT, doSearch, isMentionSearch);
     },
 
-    addSearchTermChangeListener: function(callback) {
+    addSearchTermChangeListener: function addSearchTermChangeListener(callback) {
         this.on(SEARCH_TERM_CHANGE_EVENT, callback);
     },
 
-    removeSearchTermChangeListener: function(callback) {
+    removeSearchTermChangeListener: function removeSearchTermChangeListener(callback) {
         this.removeListener(SEARCH_TERM_CHANGE_EVENT, callback);
     },
 
-    emitSelectedPostChange: function(from_search) {
+    emitSelectedPostChange: function emitSelectedPostChange(fromSearch) {
-        this.emit(SELECTED_POST_CHANGE_EVENT, from_search);
+        this.emit(SELECTED_POST_CHANGE_EVENT, fromSearch);
     },
 
-    addSelectedPostChangeListener: function(callback) {
+    addSelectedPostChangeListener: function addSelectedPostChangeListener(callback) {
         this.on(SELECTED_POST_CHANGE_EVENT, callback);
     },
 
-    removeSelectedPostChangeListener: function(callback) {
+    removeSelectedPostChangeListener: function removeSelectedPostChangeListener(callback) {
         this.removeListener(SELECTED_POST_CHANGE_EVENT, callback);
     },
 
-    emitMentionDataChange: function(id, mentionText) {
+    emitMentionDataChange: function emitMentionDataChange(id, mentionText) {
         this.emit(MENTION_DATA_CHANGE_EVENT, id, mentionText);
     },
 
-    addMentionDataChangeListener: function(callback) {
+    addMentionDataChangeListener: function addMentionDataChangeListener(callback) {
         this.on(MENTION_DATA_CHANGE_EVENT, callback);
     },
 
-    removeMentionDataChangeListener: function(callback) {
+    removeMentionDataChangeListener: function removeMentionDataChangeListener(callback) {
         this.removeListener(MENTION_DATA_CHANGE_EVENT, callback);
     },
 
-    emitAddMention: function(id, username) {
+    emitAddMention: function emitAddMention(id, username) {
         this.emit(ADD_MENTION_EVENT, id, username);
     },
 
-    addAddMentionListener: function(callback) {
+    addAddMentionListener: function addAddMentionListener(callback) {
         this.on(ADD_MENTION_EVENT, callback);
     },
 
-    removeAddMentionListener: function(callback) {
+    removeAddMentionListener: function removeAddMentionListener(callback) {
         this.removeListener(ADD_MENTION_EVENT, callback);
     },
 
-    getCurrentPosts: function() {
+    getCurrentPosts: function getCurrentPosts() {
         var currentId = ChannelStore.getCurrentId();
 
-        if (currentId != null)
+        if (currentId != null) {
             return this.getPosts(currentId);
-        else
+        }
-            return null;
+        return null;
     },
-    storePosts: function(channelId, posts) {
+    storePosts: function storePosts(channelId, posts) {
-        this._storePosts(channelId, posts);
+        this.pStorePosts(channelId, posts);
         this.emitChange();
     },
-    _storePosts: function(channelId, posts) {
+    pStorePosts: function pStorePosts(channelId, posts) {
-        BrowserStore.setItem("posts_" + channelId, posts);
+        BrowserStore.setItem('posts_' + channelId, posts);
     },
-    getPosts: function(channelId) {
+    getPosts: function getPosts(channelId) {
-        return BrowserStore.getItem("posts_" + channelId);
+        return BrowserStore.getItem('posts_' + channelId);
     },
-    storeSearchResults: function(results, is_mention_search) {
+    storeSearchResults: function storeSearchResults(results, isMentionSearch) {
-        BrowserStore.setItem("search_results", results);
+        BrowserStore.setItem('search_results', results);
-        is_mention_search = is_mention_search ? true : false; // force to bool
+        BrowserStore.setItem('is_mention_search', Boolean(isMentionSearch));
-        BrowserStore.setItem("is_mention_search", is_mention_search);
     },
-    getSearchResults: function() {
+    getSearchResults: function getSearchResults() {
-        return BrowserStore.getItem("search_results");
+        return BrowserStore.getItem('search_results');
     },
-    getIsMentionSearch: function() {
+    getIsMentionSearch: function getIsMentionSearch() {
-        return BrowserStore.getItem("is_mention_search");
+        return BrowserStore.getItem('is_mention_search');
     },
-    storeSelectedPost: function(post_list) {
+    storeSelectedPost: function storeSelectedPost(postList) {
-        BrowserStore.setItem("select_post", post_list);
+        BrowserStore.setItem('select_post', postList);
     },
-    getSelectedPost: function() {
+    getSelectedPost: function getSelectedPost() {
-        return BrowserStore.getItem("select_post");
+        return BrowserStore.getItem('select_post');
     },
-    storeSearchTerm: function(term) {
+    storeSearchTerm: function storeSearchTerm(term) {
-        BrowserStore.setItem("search_term", term);
+        BrowserStore.setItem('search_term', term);
     },
-    getSearchTerm: function() {
+    getSearchTerm: function getSearchTerm() {
-        return BrowserStore.getItem("search_term");
+        return BrowserStore.getItem('search_term');
     },
-    storeCurrentDraft: function(draft) {
+    storeCurrentDraft: function storeCurrentDraft(draft) {
-        var channel_id = ChannelStore.getCurrentId();
+        var channelId = ChannelStore.getCurrentId();
-        BrowserStore.setItem("draft_" + channel_id, draft);
+        BrowserStore.setItem('draft_' + channelId, draft);
     },
-    getCurrentDraft: function() {
+    getCurrentDraft: function getCurrentDraft() {
-        var channel_id = ChannelStore.getCurrentId();
+        var channelId = ChannelStore.getCurrentId();
-        return BrowserStore.getItem("draft_" + channel_id);
+        return BrowserStore.getItem('draft_' + channelId);
     },
-    storeDraft: function(channel_id, draft) {
+    storeDraft: function storeDraft(channelId, draft) {
-        BrowserStore.setItem("draft_" + channel_id, draft);
+        BrowserStore.setItem('draft_' + channelId, draft);
     },
-    getDraft: function(channel_id) {
+    getDraft: function getDraft(channelId) {
-        return BrowserStore.getItem("draft_" + channel_id);
+        return BrowserStore.getItem('draft_' + channelId);
     },
-    storeCommentDraft: function(parent_post_id, draft) {
+    storeCommentDraft: function storeCommentDraft(parentPostId, draft) {
-        BrowserStore.setItem("comment_draft_" + parent_post_id, draft);
+        BrowserStore.setItem('comment_draft_' + parentPostId, draft);
     },
-    getCommentDraft: function(parent_post_id) {
+    getCommentDraft: function getCommentDraft(parentPostId) {
-        return BrowserStore.getItem("comment_draft_" + parent_post_id);
+        return BrowserStore.getItem('comment_draft_' + parentPostId);
     },
-    clearDraftUploads: function() {
+    clearDraftUploads: function clearDraftUploads() {
-        BrowserStore.actionOnItemsWithPrefix("draft_", function (key, value) {
+        BrowserStore.actionOnItemsWithPrefix('draft_', function clearUploads(key, value) {
             if (value) {
                 value.uploadsInProgress = 0;
                 BrowserStore.setItem(key, value);
             }
         });
     },
-    clearCommentDraftUploads: function() {
+    clearCommentDraftUploads: function clearCommentDraftUploads() {
-        BrowserStore.actionOnItemsWithPrefix("comment_draft_", function (key, value) {
+        BrowserStore.actionOnItemsWithPrefix('comment_draft_', function clearUploads(key, value) {
             if (value) {
                 value.uploadsInProgress = 0;
                 BrowserStore.setItem(key, value);
@@ -172,12 +170,12 @@ var PostStore = assign({}, EventEmitter.prototype, {
     }
 });
 
-PostStore.dispatchToken = AppDispatcher.register(function(payload) {
+PostStore.dispatchToken = AppDispatcher.register(function registry(payload) {
     var action = payload.action;
 
-    switch(action.type) {
+    switch (action.type) {
         case ActionTypes.RECIEVED_POSTS:
-            PostStore._storePosts(action.id, action.post_list);
+            PostStore.pStorePosts(action.id, action.post_list);
             PostStore.emitChange();
             break;
         case ActionTypes.RECIEVED_SEARCH: