Only update once

3e281f95 · Robert Schilling · e552b4af · 3e281f95
Commit 3e281f95 authored Jul 19, 2016 by Robert Schilling
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 2 deletions

lib/api/branches.rb lib/api/branches.rb +3 -2

No files found.
--- a/lib/api/branches.rb
+++ b/lib/api/branches.rb
@@ -51,8 +51,9 @@ module API
        developers_can_merge = to_boolean(params[:developers_can_merge])

        if protected_branch
-          protected_branch.update(developers_can_push: developers_can_push) unless developers_can_push.nil?
-          protected_branch.update(developers_can_merge: developers_can_merge) unless developers_can_merge.nil?
+          protected_branch.developers_can_push = developers_can_push unless developers_can_push.nil?
+          protected_branch.developers_can_merge = developers_can_merge unless developers_can_merge.nil?
+          protected_branch.save
        else
          user_project.protected_branches.create(name: @branch.name,
                                                 developers_can_push: developers_can_push || false,