redmine_remotes 0.11.0 → 0.15.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7fe74fdd135a685b96165bd9ed58d166cfb9ba5b28535eca5bc25f12a730e656
-  data.tar.gz: ceb58f6cebd35a0db510929df86cac7465be85f1de0f79717e2fda174fe69b90
+  metadata.gz: 6f6710957b993d233b258cc399182801aaf348281e51df537cf4d10721309154
+  data.tar.gz: f9fc4342d569bf8359696a330eddb9c980293ee17f3e2a72be03c5fa6c0b5103
 SHA512:
-  metadata.gz: 77dbf10449d5d38021cf698ab65ce4225cf33377e6e5a3b4cf25ca52bba04f9216959c83a26b8cfacd25a703f887d0cb597fe0b4b28b56737c3f9baa2b89effb
-  data.tar.gz: 61dcddd64c24be40ed2dfacf389f535c1c56beab448b17cd817bce2e3164000b2ea190cdb195a7cfedd0f7bf59890ccdd5bc6f566870608c362a9c1e4962e633
+  metadata.gz: 27bb24d1b92d9c58e388cd0b14bf150e0ffa09d3a540baf41a5417eb6318ee707a4a18bbc59fc4fbd60d01e2924c0ce432741d8c9b630634852a8600101eec74
+  data.tar.gz: ea5eae31d4a6eff4e51d6b72428b798302bab4c49120ad2635c119313928234e4a82f0e43f658f79b80077449604f91efa92d1d5529747c5bf3757e80aa109ea

data/app/controllers/project_remote_issues_controller/create.rb

@@ -30,7 +30,7 @@ class ProjectRemoteIssuesController < ApplicationController
     end
 
     def create_model_class
-      ::RedmineRemotes::Tableless::RemoteIssueFetch::Create
+      ::RemoteIssueFetch::Create
     end
 
     def create_params

data/app/controllers/project_remote_issues_controller/update.rb

@@ -14,7 +14,7 @@ class ProjectRemoteIssuesController < ApplicationController
     private
 
     def update_model_class
-      ::RedmineRemotes::Tableless::RemoteIssueFetch::Update
+      ::RemoteIssueFetch::Update
     end
 
     def update_params

data/app/controllers/remote_trackers_controller.rb

@@ -14,7 +14,10 @@ class RemoteTrackersController < ApplicationController
     conf.columns[:password].form_ui = :password
     conf.columns[:profile].form_ui = :select
     conf.columns[:default_local_user].form_ui = :select
-    %w[list show].each { |action| conf.send(action).columns.exclude :password }
+    %w[list show].each do |action|
+      conf.send(action).columns.exclude :password
+      conf.send(action).columns << :fetched_at
+    end
   end
 
   private

data/app/models/remote_issue.rb

@@ -12,6 +12,17 @@ class RemoteIssue < ActiveRecord::Base
   validates :remote_tracker, presence: true
   validates :remote_code, presence: true, uniqueness: { scope: [:remote_tracker] }
 
+  scope :outdated, lambda {
+    t = arel_table
+    where(
+      t[:fetched_at].eq(nil)
+      .or(
+        t[:outdated_at].not_eq(nil)
+        .and(t[:fetched_at].lt(t[:outdated_at]))
+      )
+    )
+  }
+
   def to_s
     "#{remote_tracker}|#{remote_code}"
   end

data/app/models/remote_issue_fetch/base.rb

@@ -0,0 +1,45 @@
+# frozen_string_literal: true
+
+require 'eac_rails_utils/models/fetch_errors'
+require 'eac_ruby_utils/core_ext'
+
+module RemoteIssueFetch
+  class Base < ::EacRailsUtils::Models::Tableless
+    enable_simple_cache
+    require_sub __FILE__
+    include ::EacRailsUtils::Models::FetchErrors
+
+    validates :fetched_at, presence: true
+    validates :local_tracker, presence: true
+    validates :remote_tracker, presence: true
+    validates :issue_remote_code, presence: true
+
+    validate :author_present
+    validate :provider_issue_present
+
+    def author_present
+      return if provider_issue.blank?
+      return if local_issue_author.present?
+
+      errors.add(:remote_code,
+                 "Usuário local não encontrado para \"#{provider_issue.author_login}\"")
+    end
+
+    def provider_issue_present
+      return if issue_remote_code.blank?
+      return if provider_issue.present?
+
+      errors.add(:remote_code,
+                 "Demanda remota não encontrada com o código \"#{issue_remote_code}\"")
+    end
+
+    def save
+      ::Issue.transaction do
+        return false unless valid?
+        return false unless save_local_issue
+        return false unless save_remote_issue
+      end
+      true
+    end
+  end
+end

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

@@ -0,0 +1,77 @@
+# frozen_string_literal: true
+
+module RemoteIssueFetch
+  class Base < ::EacRailsUtils::Models::Tableless
+    private
+
+    DESCRIPTION_SECTION_SEPARATOR = "\n\n"
+
+    def save_local_issue
+      save_local_issue_except_create_on && save_local_issue_created_on
+    end
+
+    def save_local_issue_created_on
+      local_issue_provider_attributes[:created_on].if_present do |created_on|
+        if created_on.present? && local_issue.created_on != created_on
+          local_issue.update_column( # rubocop:disable Rails/SkipsModelValidations
+            :created_on, created_on
+          )
+        end
+      end
+
+      true
+    end
+
+    def save_local_issue_except_create_on
+      local_issue_provider_attributes.except(:created_on)
+                                     .each { |attr, value| local_issue.send("#{attr}=", value) }
+      return true if local_issue.save
+
+      fetch_record_errors(local_issue, default_column: default_error_column)
+      false
+    end
+
+    def local_issue_provider_attributes
+      {
+        subject: local_issue_subject, author: local_issue_author, status: local_issue_status,
+        description: local_issue_description,
+        created_on: provider_issue.created_on
+      }
+    end
+
+    def local_issue_subject
+      "[#{provider_issue.code}] #{provider_issue.subject}".truncate(100)
+    end
+
+    def local_issue_author_uncached
+      return nil if provider_issue.blank?
+
+      ::RemoteUser.find_local_user(
+        provider_issue.author_login,
+        remote_tracker,
+        true
+      )
+    end
+
+    def local_issue_status
+      remote_tracker.try(:find_local_issue_status, local_tracker, provider_issue.status_name, true)
+    end
+
+    def local_issue_description
+      [local_issue_extra_description(:before), provider_issue.description,
+       local_issue_extra_description(:after), local_issue_import_footer]
+        .reject(&:blank?).map(&:strip).join("\n\n----\n\n") + "\n"
+    end
+
+    def local_issue_extra_description(position)
+      extra = provider_issue.try("extra_description_#{position}").if_present('')
+      extra = extra.map { |k, v| "* *#{k}:* #{v}" } if extra.is_a?(::Hash)
+      extra = extra.map { |v| "#{v}\n" }.join if extra.is_a?(::Enumerable)
+      extra.to_s
+    end
+
+    def local_issue_import_footer
+      "_Demanda remota importada de #{provider_issue.human_view_url}._"
+    end
+  end
+end

data/app/models/remote_issue_fetch/base/_provider_issue.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+module RemoteIssueFetch
+  class Base < ::EacRailsUtils::Models::Tableless
+    private
+
+    def provider_issue_uncached
+      return nil if [remote_tracker, issue_remote_code].any?(&:blank?)
+
+      remote_tracker.find_remote_issue(issue_remote_code)
+    rescue ::Avmtrf1::RestProvider::Response => e
+      errors.add(default_error_column,
+                 "Requisição a \"#{e.url}\" retornou com status #{e.status}")
+      ::Rails.logger.warn(e.body)
+      nil
+    end
+  end
+end

data/app/models/remote_issue_fetch/base/_remote_issue.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+module RemoteIssueFetch
+  class Base < ::EacRailsUtils::Models::Tableless
+    private
+
+    def fetched_at_uncached
+      ::Time.zone.now
+    end
+
+    def save_remote_issue
+      remote_issue.fetched_at = fetched_at
+      return true if remote_issue.save
+
+      fetch_record_errors(remote_issue, default_column: default_error_column)
+      false
+    end
+  end
+end

data/app/models/remote_issue_fetch/create.rb

@@ -0,0 +1,44 @@
+# frozen_string_literal: true
+
+module RemoteIssueFetch
+  class Create < ::RemoteIssueFetch::Base
+    require_sub __FILE__
+
+    attribute :issue_remote_code, ::String
+    attribute :local_tracker_id, ::Integer
+    attribute :project_id, ::Integer
+    attribute :remote_tracker_id, ::Integer
+
+    belongs_to :local_tracker, class_name: 'Tracker'
+    belongs_to :project, class_name: 'Project'
+    belongs_to :remote_tracker, class_name: 'RemoteTracker'
+
+    validates :project, presence: true
+
+    validate :tracker_in_project
+
+    def default_error_column
+      :issue_remote_code
+    end
+
+    def save
+      remote_issue_by_remote_code.present? ? update : super
+    end
+
+    def tracker_in_project
+      return unless local_tracker.present? && project.present?
+      return if project.trackers.include?(local_tracker)
+
+      errors.add(:local_tracker, 'is not a tracker of project')
+    end
+
+    def update
+      update_record = ::RemoteIssueFetch::Update
+                      .new(remote_issue: remote_issue_by_remote_code)
+      return true if update_record.save
+
+      fetch_record_errors(update_record, default_column: default_error_column)
+      false
+    end
+  end
+end

data/app/models/remote_issue_fetch/create/_local_issue.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+module RemoteIssueFetch
+  class Create < ::RemoteIssueFetch::Base
+    private
+
+    def local_issue_provider_attributes
+      super.merge(
+        project: project, tracker: local_tracker, priority: local_issue_priority
+      )
+    end
+
+    def local_issue_uncached
+      remote_issue_by_remote_code.if_present(&:local_issue) || ::Issue.new
+    end
+
+    def local_issue_priority
+      ::IssuePriority.default
+    end
+  end
+end

data/app/models/remote_issue_fetch/create/_remote_issue.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+module RemoteIssueFetch
+  class Create < ::RemoteIssueFetch::Base
+    private
+
+    def remote_issue_uncached
+      remote_issue_by_remote_code.if_blank do
+        ::RemoteIssue.new(remote_tracker: remote_tracker, remote_code: issue_remote_code,
+                          local_issue: local_issue)
+      end
+    end
+
+    def remote_issue_by_remote_code_uncached
+      ::RemoteIssue.find_by(
+        remote_tracker: remote_tracker,
+        remote_code: ::RemoteIssue.remote_code_sanitize(issue_remote_code)
+      )
+    end
+  end
+end

data/app/models/remote_issue_fetch/update.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+require 'eac_ruby_utils/core_ext'
+
+module RemoteIssueFetch
+  class Update < ::RemoteIssueFetch::Base
+    require_sub __FILE__
+
+    attribute :remote_issue_id, ::Integer
+    belongs_to :remote_issue
+
+    validates :remote_issue, presence: true
+
+    def default_error_column
+      :remote_issue_id
+    end
+  end
+end

data/app/models/remote_issue_fetch/update/_local_issue.rb

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+module RemoteIssueFetch
+  class Update < ::RemoteIssueFetch::Base
+    private
+
+    def local_issue
+      remote_issue.local_issue
+    end
+
+    def local_tracker
+      local_issue.if_present(&:tracker)
+    end
+  end
+end

data/app/models/remote_issue_fetch/update/_remote_issue.rb

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+module RemoteIssueFetch
+  class Update < ::RemoteIssueFetch::Base
+    private
+
+    def issue_remote_code
+      remote_issue.remote_code
+    end
+
+    def remote_tracker
+      remote_issue.remote_tracker
+    end
+  end
+end

data/app/models/remote_tracker.rb

@@ -27,6 +27,12 @@ class RemoteTracker < ActiveRecord::Base
   validates :password, presence: true
   validates :default_local_user, presence: true
 
+  scope :fetchable, -> { where(profile: fetchable_profiles) }
+
+  def fetchable?
+    self.class.fetchable_profiles.include?(profile)
+  end
+
   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
@@ -37,6 +43,6 @@ class RemoteTracker < ActiveRecord::Base
   end
 
   def setting_value(name)
-    ::RemoteTrackerSetting.where(remote_tracker: self, name: name).first.if_present(&:value)
+    ::RemoteTrackerSetting.find_by(remote_tracker: self, name: name).if_present(&:value)
   end
 end

data/app/models/remote_tracker/find_issue_local_status.rb

@@ -22,7 +22,7 @@ class RemoteTracker < ActiveRecord::Base
     end
 
     def result_by_remote_issue_status
-      ::RemoteIssueStatus.where(remote_issue_status_find_attributes).first.try(:local_issue_status)
+      ::RemoteIssueStatus.find_by(remote_issue_status_find_attributes).try(:local_issue_status)
     end
 
     def result_by_local_issue_status

data/app/models/remote_tracker/remote.rb

@@ -1,19 +1,43 @@
 # frozen_string_literal: true
 
+require 'eac_ruby_utils/core_ext'
+
 class RemoteTracker < ActiveRecord::Base
   module Remote
-    def find_remote_issue(issue_id)
-      remote_instance.find_issue(issue_id)
-    end
+    common_concern
+
+    module ClassMethods
+      def fetchable_profiles
+        @fetchable_profiles ||= lists.profile.values.select do |profile|
+          profile_fetchable?(profile)
+        end
+      end
 
-    def remote_instance
-      remote_instance_class.new(self, root_url, username, password)
+      def profile_fetchable?(profile)
+        provider_instance_class(profile).method_defined?(:fetch_issues_changed)
+      end
+
+      def provider_instance_class(profile)
+        lists.profile.value_validate!(profile)
+
+        ::RedmineRemotes.const_get(profile.capitalize).const_get('Instance')
+      end
     end
 
-    def remote_instance_class
-      raise 'Profile blank' if profile.blank?
+    module InstanceMethods
+      def find_remote_issue(issue_id)
+        remote_instance.find_issue(issue_id)
+      end
+
+      def remote_instance
+        r = remote_instance_class.new(root_url, username, password)
+        r.send('remote_tracker=', self)
+        r
+      end
 
-      ::RedmineRemotes.const_get(profile.capitalize).const_get('Instance')
+      def remote_instance_class
+        self.class.provider_instance_class(profile)
+      end
     end
   end
 end