redmine_remotes 0.9.1 → 0.12.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 81fddbac08d334ddd2a052a196a055a5db15b5b5b64c2a33b3a5f9dd12f2b0f9
-  data.tar.gz: 5dcff16fa4012c50371997ceb89a0a3a7c957ddeaa3eeaa0e774d63489de84a3
+  metadata.gz: 70c4977d8154fa08f7d03093a35dc948c5bd78639a2a2ce42352e58c5d737050
+  data.tar.gz: 8c56794c3633be3b5e731800025a61a73a8df6e17c50776a87790659e0b3235d
 SHA512:
-  metadata.gz: 6811d6ff25bf95bb272097242eefd6faa8831e65d831c7200f3cdb48ef7704ecf3a67180e84a92c53852b86c3aceac6d1203ff5b8b4464bb9df24f601acea06f
-  data.tar.gz: b713b31ef70aad6ff5324af4bcc087ce663b99577896908d6c02f0441e519806f66ce0fe0a2aa57487d32f057e61ad75cd30f0c7531f44927102595c8c2c3ae1
+  metadata.gz: 546876674ce5856dacf18ed183ddca546dcbdbe2ae17ede908b2512e18af12512f9fd8449d829a82105c654994f62a6a01589c42a2b7e055f5e94a64a182cec2
+  data.tar.gz: 2e821818b0294b9e8b889612666a7a10c0eb494abe661743bc8f9122ca741285364aea93e679bb30068b73c9aec7f53aa25f1c1e96b95a0ba2d887199509c280

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

@@ -1,15 +1,15 @@
 # frozen_string_literal: true
 
-require 'eac/model'
+require 'eac_rails_utils/models/fetch_errors'
 require 'eac_ruby_utils/core_ext'
 
 module RedmineRemotes
   module Tableless
     module RemoteIssueFetch
-      class Base < ::EacRailsUtils::TablelessModel
+      class Base < ::EacRailsUtils::Models::Tableless
         enable_simple_cache
         require_sub __FILE__
-        include ::Eac::Model
+        include ::EacRailsUtils::Models::FetchErrors
 
         validates :local_tracker, presence: true
         validates :remote_tracker, presence: true

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

@@ -3,7 +3,7 @@
 module RedmineRemotes
   module Tableless
     module RemoteIssueFetch
-      class Base < ::EacRailsUtils::TablelessModel
+      class Base < ::EacRailsUtils::Models::Tableless
         private
 
         def save_local_issue
@@ -48,28 +48,24 @@ module RedmineRemotes
 
           ::RemoteUser.find_local_user(
             provider_issue.author_login,
-            remote_tracker
+            remote_tracker,
+            true
           )
         end
 
         def local_issue_status
-          local_issue_status_from_remote_tracker || local_issue_status_from_local_tracker
+          remote_tracker.try(:find_local_issue_status, local_tracker, provider_issue.status_name,
+                             true)
         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_description
+          <<~DESCRIPTION
+            #{provider_issue.description}
 
-        def local_issue_status_from_local_tracker
-          local_tracker.default_status
-        end
+            ----
 
-        def local_issue_description
-          provider_issue.description
+            _Demanda remota importada de #{provider_issue.human_view_url}._
+          DESCRIPTION
         end
       end
     end

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

@@ -3,7 +3,7 @@
 module RedmineRemotes
   module Tableless
     module RemoteIssueFetch
-      class Base < ::EacRailsUtils::TablelessModel
+      class Base < ::EacRailsUtils::Models::Tableless
         private
 
         def provider_issue_uncached

data/app/models/redmine_remotes/tableless/remote_issue_fetch/update.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require 'eac_ruby_utils/core_ext'
-require 'eac/model'
 
 module RedmineRemotes
   module Tableless

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/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/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

@@ -6,6 +6,10 @@ module RedmineRemotes
   module Esosti
     module Entities
       class Issue < ::Avmtrf1::Esosti::Entities::Issue
+        def human_view_url
+          instance.issue_human_view_url(uid)
+        end
+
         def author_login
           attributes.fetch('CREATEDBY').if_present { |v| v.gsub(/@[^@]+\z/, '') }
         end
@@ -23,7 +27,9 @@ module RedmineRemotes
         end
 
         def description
-          related_mbos.fetch('LONGDESCRIPTION').first.fetch('Attributes').fetch('LDTEXT')
+          ::RedmineRemotes::Esosti.remote_text_to_local(
+            related_mbos.fetch('LONGDESCRIPTION').first.fetch('Attributes').fetch('LDTEXT')
+          )
         end
 
         def related_mbos
@@ -37,6 +43,10 @@ module RedmineRemotes
         def subject
           attributes.fetch('DESCRIPTION')
         end
+
+        def uid
+          attributes.fetch('TICKETUID').fetch('content')
+        end
       end
     end
   end

data/lib/redmine_remotes/esosti/instance.rb

@@ -21,6 +21,10 @@ module RedmineRemotes
       def issue_get_url_suffix(provider_issue_id)
         "/os/MS_RMTICKET/?ticketid=#{provider_issue_id}"
       end
+
+      def issue_human_view_url(ticket_uid)
+        "#{root_url}/itsm/ui/maximo.jsp?event=loadapp&value=ms_viewsr&uniqueid=#{ticket_uid}"
+      end
     end
   end
 end

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

@@ -6,6 +6,10 @@ module RedmineRemotes
   module Jira
     module Entities
       class Issue < ::Avmtrf1::RestProvider::Entity
+        def human_view_url
+          instance.issue_human_view_url(code)
+        end
+
         def code
           data.fetch('key')
         end

data/lib/redmine_remotes/jira/instance.rb

@@ -29,6 +29,10 @@ module RedmineRemotes
       def issue_get_url_suffix(provider_issue_id)
         "/issue/#{provider_issue_id}"
       end
+
+      def issue_human_view_url(issue_code)
+        "#{root_url}/browse/#{issue_code.to_s.upcase}"
+      end
     end
   end
 end

data/lib/redmine_remotes/version.rb

@@ -3,7 +3,7 @@
 module RedmineRemotes
   SLUG = 'redmine_remotes'
   NAME = 'Redmine Remotes'
-  VERSION = '0.9.1'
+  VERSION = '0.12.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.9.1
+  version: 0.12.0
 platform: ruby
 authors:
 - Eduardo Henrique Bogoni
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-05-12 00:00:00.000000000 Z
+date: 2020-06-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: aranha-parsers
@@ -64,34 +64,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.5'
+        version: '0.11'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.5'
+        version: '0.11'
 - !ruby/object:Gem::Dependency
   name: eac_ruby_utils
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.10'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 0.10.1
+        version: '0.35'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.10'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 0.10.1
+        version: '0.35'
 - !ruby/object:Gem::Dependency
   name: validate_url
   requirement: !ruby/object:Gem::Requirement
@@ -139,9 +133,11 @@ 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
+- 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
@@ -164,6 +160,7 @@ 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