pk/atst
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Rename confusing variable

Browse Source
This commit is contained in:
Montana 2018-09-21 09:33:49 -04:00
parent a8b777acda
commit a071ff625f
1 changed files with 7 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
js/components/forms/edit_workspace_member.js
View File

@ -23,29 +23,30 @@ export default {
data: function () { data: function () {
return { return {
value: this.initialData, new_role: this.initialData,
} }
}, },
methods: { methods: {
change: function (e) { change: function (e) {
this.value = e.target.value e.preventDefault()
this.new_role = e.target.value
}, },
cancel: function (current_role, selected_role) { cancel: function (current_role, selected_role) {
if (current_role != selected_role) { if (current_role != selected_role) {
this.value = current_role this.new_role = current_role
} }
} },
}, },
computed: { computed: {
displayName: function () { displayName: function () {
for (var arr in this.choices) { for (var arr in this.choices) {
if (this.choices[arr][0] == this.value) { if (this.choices[arr][0] == this.new_role) {
return this.choices[arr][1].name return this.choices[arr][1].name
} }
} }
return this.value ? this.value : "no access" return this.new_role ? this.new_role : "no access"
}, },
label_class: function () { label_class: function () {
return this.displayName === "no access" ? return this.displayName === "no access" ?