Adding a loose state (status_tag) to merge requests, adding state change to comments.

- Adding a comment to a merge request now potentially sets or changes the loose state
of the merge request (target)

This way, target repository owners can mark that a merge request  should change state
when adding comments. This will eventually replace the more final reason from merge requests,
replacing it with comments with an added context

1 files changed, 11 insertions, 0 deletions

diff --git a/app/models/merge_request.rb b/app/models/merge_request.rb
index 6dd5f29..a56a91f 100644
--- a/app/models/merge_request.rb
+++ b/app/models/merge_request.rb
@@ -387,6 +387,17 @@ class MergeRequest < ActiveRecord::Base
     save
   end
   
+  def update_status(options)
+    user = options[:user]
+    comment = comments.build(:body => options[:comment], :user => user)
+    if resolvable_by?(user) && new_tag = options[:tag]
+      comment.state_change = [status_tag, new_tag]
+      self.status_tag = new_tag
+    end
+    comment.save
+    save
+  end
+  
   def valid_oauth_credentials?
     response = access_token.get("/")
     return Net::HTTPSuccess === response