redmine_remotes 0.10.0 → 0.11.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2a0671e339c9e225c9413feb402fed23e0ac5e930a3e72027aa2b183749441d5
-  data.tar.gz: eb1245a9f15262922b95bec5574bdbb3dbc84f286d7fed371970b759657f6e2d
+  metadata.gz: 7fe74fdd135a685b96165bd9ed58d166cfb9ba5b28535eca5bc25f12a730e656
+  data.tar.gz: ceb58f6cebd35a0db510929df86cac7465be85f1de0f79717e2fda174fe69b90
 SHA512:
-  metadata.gz: 0af87885a31cc69c7ec0cc8f1bbd7201b189b8c5f260382d990189affb7ab642b5eba050231e3e7c0a78f5d65a134c21490148d5461858c61af9ad92c813e578
-  data.tar.gz: 322120abe8ba4dde501c3d6cccc3a3315933d18f585d4c7eb9a86b8c147ec797c0dcb2bff8b08ac29001466eb12e2824cb29a1ef7394b5db1d8f21cbe4c31778
+  metadata.gz: 77dbf10449d5d38021cf698ab65ce4225cf33377e6e5a3b4cf25ca52bba04f9216959c83a26b8cfacd25a703f887d0cb597fe0b4b28b56737c3f9baa2b89effb
+  data.tar.gz: 61dcddd64c24be40ed2dfacf389f535c1c56beab448b17cd817bce2e3164000b2ea190cdb195a7cfedd0f7bf59890ccdd5bc6f566870608c362a9c1e4962e633

data/app/models/redmine_remotes/tableless/remote_issue_fetch/base/_local_issue.rb

@@ -54,19 +54,8 @@ module RedmineRemotes
         end
 
         def local_issue_status
-          local_issue_status_from_remote_tracker || local_issue_status_from_local_tracker
-        end
-
-        def local_issue_status_from_remote_tracker
-          ::RemoteIssueStatus.find_local_issue_status(
-            provider_issue.status_name,
-            remote_tracker,
-            local_tracker
-          )
-        end
-
-        def local_issue_status_from_local_tracker
-          local_tracker.default_status
+          remote_tracker.try(:find_local_issue_status, local_tracker, provider_issue.status_name,
+                             true)
         end
 
         def local_issue_description

data/app/models/remote_issue_status.rb

@@ -13,15 +13,4 @@ class RemoteIssueStatus < ActiveRecord::Base
   validates :remote_tracker, presence: true
   validates :local_tracker, presence: true
   validates :local_issue_status, presence: true
-
-  class << self
-    def find_local_issue_status(remote_name, remote_tracker, local_tracker)
-      record = ::RemoteIssueStatus.where(
-        remote_name: remote_name_sanitize(remote_name),
-        remote_tracker: remote_tracker,
-        local_tracker: local_tracker
-      ).first
-      record ? record.local_issue_status : nil
-    end
-  end
 end

data/app/models/remote_tracker.rb

@@ -27,6 +27,11 @@ class RemoteTracker < ActiveRecord::Base
   validates :password, presence: true
   validates :default_local_user, presence: true
 
+  def find_local_issue_status(local_tracker, remote_status_name, register_not_found = false)
+    ::RemoteTracker::FindIssueLocalStatus.new(self, local_tracker, remote_status_name,
+                                              register_not_found).result
+  end
+
   def to_s
     "#{root_url} [#{profile_label}, #{username}]"
   end

data/app/models/remote_tracker/find_issue_local_status.rb

@@ -0,0 +1,40 @@
+# frozen_string_literal: true
+
+require 'eac_ruby_utils/core_ext'
+
+class RemoteTracker < ActiveRecord::Base
+  class FindIssueLocalStatus
+    common_constructor :remote_tracker, :local_tracker, :remote_status_name, :register_not_found
+
+    # @return [Tracker]
+    def result
+      result_by_remote_issue_status || result_by_register_not_found || result_by_local_issue_status
+    end
+
+    private
+
+    def result_by_register_not_found
+      return nil unless register_not_found
+
+      ::RemoteIssueStatus.create!(
+        remote_issue_status_find_attributes.merge(local_issue_status: result_by_local_issue_status)
+      ).local_issue_status
+    end
+
+    def result_by_remote_issue_status
+      ::RemoteIssueStatus.where(remote_issue_status_find_attributes).first.try(:local_issue_status)
+    end
+
+    def result_by_local_issue_status
+      local_tracker.default_status
+    end
+
+    def remote_issue_status_find_attributes
+      {
+        remote_name: ::RemoteIssueStatus.remote_name_sanitize(remote_status_name),
+        remote_tracker: remote_tracker,
+        local_tracker: local_tracker
+      }
+    end
+  end
+end

data/lib/redmine_remotes/version.rb

@@ -3,7 +3,7 @@
 module RedmineRemotes
   SLUG = 'redmine_remotes'
   NAME = 'Redmine Remotes'
-  VERSION = '0.10.0'
+  VERSION = '0.11.0'
   AUTHOR = 'Eduardo Henrique Bogoni'
   SUMMARY = 'Integração do Redmine com trackers diversos.'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: redmine_remotes
 version: !ruby/object:Gem::Version
-  version: 0.10.0
+  version: 0.11.0
 platform: ruby
 authors:
 - Eduardo Henrique Bogoni
@@ -139,6 +139,7 @@ files:
 - app/models/remote_issue.rb
 - app/models/remote_issue_status.rb
 - app/models/remote_tracker.rb
+- app/models/remote_tracker/find_issue_local_status.rb
 - app/models/remote_tracker/remote.rb
 - app/models/remote_tracker_setting.rb
 - app/models/remote_user.rb