Addressed observations

1 files changed, 6 insertions, 6 deletions

diff --git a/app/assets/javascripts/protected_branches/protected_branch_create.js b/app/assets/javascripts/protected_branches/protected_branch_create.js
index 007846d732a..efc612a3dc7 100644
--- a/app/assets/javascripts/protected_branches/protected_branch_create.js
+++ b/app/assets/javascripts/protected_branches/protected_branch_create.js
@@ -50,13 +50,13 @@ export default class ProtectedBranchCreate {
     const $branchInput = this.$form.find('input[name="protected_branch[name]"]');
     const $allowedToMergeInput = this.$form.find('input[name="protected_branch[merge_access_levels_attributes][0][access_level]"]');
     const $allowedToPushInput = this.$form.find('input[name="protected_branch[push_access_levels_attributes][0][access_level]"]');
-    const completedForm = !($branchInput.val() && $allowedToMergeInput.length
-                          && $allowedToPushInput.length);
-    // Save previously used settings
-    if (!completedForm) {
-      this.savePreviousSelection($allowedToMergeInput.val(), $allowedToPushInput.val());
-    }
+    const completedForm = !(
+                            $branchInput.val() &&
+                            $allowedToMergeInput.length &&
+                            $allowedToPushInput.length
+                          );
 
+    this.savePreviousSelection($allowedToMergeInput.val(), $allowedToPushInput.val());
     this.$form.find('input[type="submit"]').attr('disabled', completedForm);
   }