Change issue status after merge request is merged.

Author: sandratatarevicova

README.md

@@ -41,6 +41,11 @@ One of the following environment variables need to be set:
 They must contain secrets which have to be configured in GitLab/GitHub to
 authenticate webhooks.
 
+Optionally, you can also set the following variables:
+
+* `REDMINE_MERGE_REQUEST_LINKS_AFTER_MERGE_STATUS` - Name of issue status which should be set after the merge request is merged.
+* `REDMINE_MERGE_REQUEST_LINKS_GITLAB_REDMINE_USER_ID` - ID of Redmine user who should change the status - used as journal author.
+
 Export the environment variable(s) in your bash or webserver config.
 Examples with Phusion Passenger webserver can be found here:
 https://www.phusionpassenger.com/library/indepth/environment_variables.html

app/models/merge_request.rb

@@ -10,7 +10,7 @@ class MergeRequest < ActiveRecord::Base
   # update the author name only once.
   attr_readonly :author_name
 
-  after_save :scan_description_for_issue_ids
+  after_save :scan_description_for_issue_ids, :update_mentioned_issues_status
 
   def self.find_all_by_issue(issue)
     includes(:issues).where(issues: { id: issue.id })
@@ -31,4 +31,22 @@ def mentioned_issue_ids
       (value || '').scan(ISSUE_ID_REGEXP)
     end.uniq
   end
+
+  def update_mentioned_issues_status
+    redmine_user_id = ENV['REDMINE_MERGE_REQUEST_LINKS_GITLAB_REDMINE_USER_ID']
+    after_merge_status = ENV['REDMINE_MERGE_REQUEST_LINKS_AFTER_MERGE_STATUS']
+
+    if state != 'merged' || redmine_user_id.blank? || after_merge_status.blank?
+      return
+    end
+
+    mentioned_issue_ids.map do |match|
+      issue = Issue.find_by_id(match[0])
+      if issue.present?
+        issue.init_journal(User.find(redmine_user_id))
+        issue.status = IssueStatus.find_by_name(after_merge_status)
+        issue.save!
+      end
+    end
+  end
 end