Merge pull request #3460 from evanrolfe/refactor/active_job3

[api] Issue#fetch_issues uses ActiveJob
openSUSE · Aug 1, 2017 · 0f97947 · 0f97947
2 parents 038b12c + 02b8d48
commit 0f97947
Show file tree

Hide file tree

Showing 5 changed files with 530 additions and 2 deletions.
diff --git a/src/api/app/jobs/issue_tracker_fetch_issues_job.rb b/src/api/app/jobs/issue_tracker_fetch_issues_job.rb
@@ -0,0 +1,7 @@
+class IssueTrackerFetchIssuesJob < ApplicationJob
+  queue_as :issuetracking
+
+  def perform(issue_tracker_id)
+    IssueTracker.find(issue_tracker_id).fetch_issues
+  end
+end
diff --git a/src/api/app/models/issue.rb b/src/api/app/models/issue.rb
@@ -57,8 +57,7 @@ def self.valid_name?(tracker, name)
 
   after_create :fetch_issues
   def fetch_issues
-    # inject update jobs after issue got created
-    issue_tracker.delay(queue: 'issuetracking').fetch_issues
+    IssueTrackerFetchIssuesJob.perform_later(issue_tracker.id)
   end
 
   def fetch_updates