Remove state modifications from boards components
This is part of refactoring issue boards to Vuex (&1278 (closed)) and needs to happen after https://gitlab.com/gitlab-org/gitlab-ce/issues/61622.
Mutation of a store state should only happen in the store itself, so the following places need to be wrapped in store methods:
$ grep -Eir 'store\.(.*) = ' {ee/,}app/assets/javascripts/boards/components
ee/app/assets/javascripts/boards/components/boards_list_selector/index.js: this.store.state[this.listType] = data;
ee/app/assets/javascripts/boards/components/boards_list_selector/index.js: this.store.state.lists = _.sortBy(this.store.state.lists, 'position');
ee/app/assets/javascripts/boards/components/boards_selector.vue: boardsStore.state.assignees = [];
ee/app/assets/javascripts/boards/components/boards_selector.vue: boardsStore.state.milestones = [];
ee/app/assets/javascripts/boards/components/board_form.vue: boardsStore.state.currentPage = '';
app/assets/javascripts/boards/components/sidebar/remove_issue.vue: boardsStore.detail.issue = {};
app/assets/javascripts/boards/components/board_blank_state.vue: boardsStore.state.lists = _.sortBy(boardsStore.state.lists, 'position');
app/assets/javascripts/boards/components/board_card.vue: boardsStore.detail.list = this.list;
app/assets/javascripts/boards/components/board_list.vue: boardsStore.moving.list = card.list;
app/assets/javascripts/boards/components/board_list.vue: boardsStore.moving.issue = boardsStore.moving.list.findIssue(+e.item.dataset.issueId);
app/assets/javascripts/boards/components/new_list_dropdown.js: boardsStore.state.lists = _.sortBy(boardsStore.state.lists, 'position');
app/assets/javascripts/boards/components/board_new_issue.vue: boardsStore.detail.issue = issue;
app/assets/javascripts/boards/components/board_new_issue.vue: boardsStore.detail.list = this.list;
app/assets/javascripts/boards/components/issue_card_inner.vue: boardsStore.filter.path = filterPath.join('&');
app/assets/javascripts/boards/components/modal/filters.js: this.store.path = '';
-
ee/app/assets/javascripts/boards/components/boards_list_selector/index.js -
ee/app/assets/javascripts/boards/components/boards_selector.vue -
ee/app/assets/javascripts/boards/components/board_form.vue -
app/assets/javascripts/boards/components/sidebar/remove_issue.vue -
app/assets/javascripts/boards/components/board_blank_state.vue -
app/assets/javascripts/boards/components/board_card.vue -
app/assets/javascripts/boards/components/board_list.vue -
app/assets/javascripts/boards/components/new_list_dropdown.js -
app/assets/javascripts/boards/components/board_new_issue.vue -
app/assets/javascripts/boards/components/issue_card_inner.vue -
app/assets/javascripts/boards/components/modal/filters.js - This uses a different store (
app/assets/javascripts/boards/stores/modal_store.js), so I would consider it out of scope here.
- This uses a different store (
Example
- boardsStore.detail.issue = {};
+ boardsStore.clearIssue();
Edited by Inactive Account