redmine_remotes 0.8.2 → 0.11.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bd846a3cc1a788360a4b120472ea5fe1d8f8d55835a95a3af3e842fe77e1d898
-  data.tar.gz: d4c1a544d79c5cedd5db4030df30d61cb021813a2b96457d039c77e1c35d5821
+  metadata.gz: 7fe74fdd135a685b96165bd9ed58d166cfb9ba5b28535eca5bc25f12a730e656
+  data.tar.gz: ceb58f6cebd35a0db510929df86cac7465be85f1de0f79717e2fda174fe69b90
 SHA512:
-  metadata.gz: 429c14c070e2c1d8fe18aa45d08eeb2561fc144397811e45c6f0b6571fc85e8fce06dd3c01798c47aa78b9b7ecef313e43d6352c2dd4615dd1d87e588085a2b0
-  data.tar.gz: 87dbdb0ddf2fb97b7db523512c5687e38c8982abf08cc1d4013c3bcd708162a9b240b93a2b39b002b078c9db883068367df158e646cf8bb5bfafecd2323b697f
+  metadata.gz: 77dbf10449d5d38021cf698ab65ce4225cf33377e6e5a3b4cf25ca52bba04f9216959c83a26b8cfacd25a703f887d0cb597fe0b4b28b56737c3f9baa2b89effb
+  data.tar.gz: 61dcddd64c24be40ed2dfacf389f535c1c56beab448b17cd817bce2e3164000b2ea190cdb195a7cfedd0f7bf59890ccdd5bc6f566870608c362a9c1e4962e633

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

@@ -48,24 +48,14 @@ module RedmineRemotes
 
           ::RemoteUser.find_local_user(
             provider_issue.author_login,
-            remote_tracker
-          )
-        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
+            true
           )
         end
 
-        def local_issue_status_from_local_tracker
-          local_tracker.default_status
+        def local_issue_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

@@ -6,14 +6,14 @@ require 'validate_url'
 
 class RemoteTracker < ActiveRecord::Base
   include ::EacRubyUtils::Listable
-  include ::RemoteTracker::Jira
+  include ::RemoteTracker::Remote
 
   has_many :issues, class_name: 'RemoteIssue', inverse_of: :remote_tracker,
                     dependent: :restrict_with_error
   has_many :settings, class_name: 'RemoteTrackerSetting', inverse_of: :remote_tracker,
                       dependent: :destroy
 
-  lists.add_string :profile, :jira
+  lists.add_string :profile, :esosti, :jira
 
   has_many :issue_statuses, class_name: 'RemoteIssueStatus', inverse_of: :remote_tracker,
                             dependent: :destroy
@@ -27,14 +27,13 @@ class RemoteTracker < ActiveRecord::Base
   validates :password, presence: true
   validates :default_local_user, presence: true
 
-  def to_s
-    "#{root_url} [#{profile_label}, #{username}]"
+  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 find_remote_issue(issue_id)
-    raise 'Profile blank' if profile.blank?
-
-    send("#{profile}_find_remote_issue", issue_id)
+  def to_s
+    "#{root_url} [#{profile_label}, #{username}]"
   end
 
   def setting_value(name)

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/app/models/remote_tracker/remote.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+class RemoteTracker < ActiveRecord::Base
+  module Remote
+    def find_remote_issue(issue_id)
+      remote_instance.find_issue(issue_id)
+    end
+
+    def remote_instance
+      remote_instance_class.new(self, root_url, username, password)
+    end
+
+    def remote_instance_class
+      raise 'Profile blank' if profile.blank?
+
+      ::RedmineRemotes.const_get(profile.capitalize).const_get('Instance')
+    end
+  end
+end

data/app/models/remote_user.rb

@@ -11,18 +11,7 @@ class RemoteUser < ActiveRecord::Base
   validates :remote_login, presence: true, uniqueness: { scope: [:remote_tracker] }
   validates :local_user, presence: true
 
-  class << self
-    def find_local_user(remote_login, remote_tracker = nil)
-      [remote_tracker, nil].each do |tracker|
-        remote_user = ::RemoteUser.where(
-          remote_tracker: tracker,
-          remote_login: remote_login_sanitize(remote_login)
-        ).first
-        return remote_user.local_user if remote_user
-      end
-      return remote_tracker.default_local_user if remote_tracker.present?
-
-      nil
-    end
+  def self.find_local_user(remote_login, remote_tracker = nil, register_not_found = false)
+    ::RemoteUser::FindLocalUser.new(remote_login, remote_tracker, register_not_found).result
   end
 end

data/app/models/remote_user/find_local_user.rb

@@ -0,0 +1,60 @@
+# frozen_string_literal: true
+
+require 'eac_ruby_utils/core_ext'
+
+class RemoteUser < ActiveRecord::Base
+  class FindLocalUser
+    common_constructor :remote_login, :remote_tracker, :register_not_found do
+      self.remote_login = ::RemoteUser.remote_login_sanitize(remote_login)
+    end
+
+    def result
+      result_by_local_user || result_by_remote_user || result_by_register_not_found ||
+        result_by_remote_tracker_default
+    end
+
+    private
+
+    def result_by_local_user
+      result_by_local_user_login || result_by_local_user_mail_address
+    end
+
+    def result_by_local_user_login
+      ::User.find_by(login: remote_login)
+    end
+
+    def result_by_local_user_mail_address
+      condition = ::EmailAddress.arel_table[:address].matches("#{remote_login}@%")
+      ::EmailAddress.where(condition).first.try(:user)
+    end
+
+    def result_by_register_not_found
+      return nil unless register_not_found
+      return nil if remote_tracker.blank?
+
+      ::RemoteUser.create!(remote_login: remote_login, remote_tracker: remote_tracker,
+                           local_user: remote_tracker.default_local_user).local_user
+    end
+
+    def result_by_remote_user
+      remote_trackers_to_search
+        .map { |tracker| remote_tracker_result(tracker) }
+        .find(&:present?)
+    end
+
+    def result_by_remote_tracker_default
+      remote_tracker.try(:default_local_user)
+    end
+
+    def remote_tracker_result(remote_tracker)
+      ::RemoteUser.where(
+        remote_tracker: remote_tracker,
+        remote_login: remote_login
+      ).first.try(:local_user)
+    end
+
+    def remote_trackers_to_search
+      remote_tracker.if_present([]) { |v| [v] } + [nil]
+    end
+  end
+end

data/config/locales/en.yml

@@ -3,6 +3,8 @@ en:
     listable:
       remote_tracker:
         profile:
+          esosti:
+            label: e-Sosti (Control Desk)
           jira:
             label: Jira
   remote_tracker_setting:

data/config/locales/pt-BR.yml

@@ -3,6 +3,8 @@ pt-BR:
     listable:
       remote_tracker:
         profile:
+          esosti:
+            label: e-Sosti (Control Desk)
           jira:
             label: Jira
       remote_tracker_setting:

data/lib/redmine_remotes/esosti.rb

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+module RedmineRemotes
+  module Esosti
+    class << self
+      RICH_TEXT_COMMENT = /<!--\s*RICH\s+TEXT\s-->/i.freeze
+
+      def remote_text_to_local(remote_text)
+        ::Redmine::WikiFormatting.html_parser.to_text(
+          remote_text.gsub(RICH_TEXT_COMMENT, '').each_line.map(&:strip).join(" \n")
+        ).each_line.map(&:strip).join("\n").gsub(/\n+/, "\n")
+      end
+    end
+  end
+end

data/lib/redmine_remotes/esosti/entities/issue.rb

@@ -0,0 +1,45 @@
+# frozen_string_literal: true
+
+require 'avmtrf1/esosti/entities/issue'
+
+module RedmineRemotes
+  module Esosti
+    module Entities
+      class Issue < ::Avmtrf1::Esosti::Entities::Issue
+        def author_login
+          attributes.fetch('CREATEDBY').if_present { |v| v.gsub(/@[^@]+\z/, '') }
+        end
+
+        def attributes
+          data.first.fetch('Attributes')
+        end
+
+        def code
+          attributes.fetch('TICKETID')
+        end
+
+        def created_on
+          attributes.fetch('CREATIONDATE')
+        end
+
+        def description
+          ::RedmineRemotes::Esosti.remote_text_to_local(
+            related_mbos.fetch('LONGDESCRIPTION').first.fetch('Attributes').fetch('LDTEXT')
+          )
+        end
+
+        def related_mbos
+          data.first.fetch('RelatedMbos')
+        end
+
+        def status_name
+          attributes.fetch('STATUS')
+        end
+
+        def subject
+          attributes.fetch('DESCRIPTION')
+        end
+      end
+    end
+  end
+end

data/lib/redmine_remotes/esosti/instance.rb

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+module RedmineRemotes
+  module Esosti
+    class Instance < ::RedmineRemotes::RestProvider::Instance
+      ISSUE_ID_PATTERN = /\A(?:ss|in)[0-9]+\z/i.freeze
+
+      class << self
+        def parse_issue_id(global_issue_id)
+          m = ISSUE_ID_PATTERN.match(global_issue_id)
+          return nil unless m
+
+          ::OpenStruct.new(provider_issue_id: m[0], project_id: nil, project_issue_id: nil)
+        end
+      end
+
+      def build_service_url(service_url_suffix)
+        "#{root_url}/maxrest/rest#{service_url_suffix}"
+      end
+
+      def issue_get_url_suffix(provider_issue_id)
+        "/os/MS_RMTICKET/?ticketid=#{provider_issue_id}"
+      end
+    end
+  end
+end

data/lib/redmine_remotes/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: redmine_remotes
 version: !ruby/object:Gem::Version
-  version: 0.8.2
+  version: 0.11.0
 platform: ruby
 authors:
 - Eduardo Henrique Bogoni
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-05-07 00:00:00.000000000 Z
+date: 2020-05-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: aranha-parsers
@@ -30,20 +30,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.27'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 0.27.1
+        version: '0.28'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.27'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 0.27.1
+        version: '0.28'
 - !ruby/object:Gem::Dependency
   name: curb
   requirement: !ruby/object:Gem::Requirement
@@ -145,9 +139,11 @@ files:
 - app/models/remote_issue.rb
 - app/models/remote_issue_status.rb
 - app/models/remote_tracker.rb
-- app/models/remote_tracker/jira.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
+- app/models/remote_user/find_local_user.rb
 - app/views/project_remote_issues/_issues.html.erb
 - app/views/project_remote_issues/_trackers.html.erb
 - app/views/project_remote_issues/index.html.erb
@@ -170,6 +166,9 @@ files:
 - db/migrate/20200505161002_add_timestamps_to_remote_users.rb
 - init.rb
 - lib/redmine_remotes.rb
+- lib/redmine_remotes/esosti.rb
+- lib/redmine_remotes/esosti/entities/issue.rb
+- lib/redmine_remotes/esosti/instance.rb
 - lib/redmine_remotes/jira/entities/issue.rb
 - lib/redmine_remotes/jira/instance.rb
 - lib/redmine_remotes/patches/issue.rb

data/app/models/remote_tracker/jira.rb

@@ -1,13 +0,0 @@
-# frozen_string_literal: true
-
-class RemoteTracker < ActiveRecord::Base
-  module Jira
-    def jira_find_remote_issue(issue_id)
-      jira_create_instance.find_issue(issue_id)
-    end
-
-    def jira_create_instance
-      ::RedmineRemotes::Jira::Instance.new(self, root_url, username, password)
-    end
-  end
-end