redmine_remotes 0.8.1 → 0.8.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4b65843d810df106f89cfab8c6da6beef03d5421263f21a7b7730e9dab08ce13
-  data.tar.gz: ec23913f8340a42f5c69f28e0f24f72baacf3b59b38adb9d8f32dadeca9fc06c
+  metadata.gz: bd846a3cc1a788360a4b120472ea5fe1d8f8d55835a95a3af3e842fe77e1d898
+  data.tar.gz: d4c1a544d79c5cedd5db4030df30d61cb021813a2b96457d039c77e1c35d5821
 SHA512:
-  metadata.gz: 886781f236327ed6e0d30d26f3950e80a07629c3c2905c8bf98a50cb8c9ed10daa9be495c804059d2ac195816894c77ee2af43e87d3c253ccb20a568bcd0bb5d
-  data.tar.gz: 41578f7e2bfa7a6c97bdad8cc9bff55c1a8e26f8099d5ba824c0b0e420ed746ee46f8058b6d9cfb8069de6fc6dda8c4b16b5ed566bce325abada1043e1b268dd
+  metadata.gz: 429c14c070e2c1d8fe18aa45d08eeb2561fc144397811e45c6f0b6571fc85e8fce06dd3c01798c47aa78b9b7ecef313e43d6352c2dd4615dd1d87e588085a2b0
+  data.tar.gz: 87dbdb0ddf2fb97b7db523512c5687e38c8982abf08cc1d4013c3bcd708162a9b240b93a2b39b002b078c9db883068367df158e646cf8bb5bfafecd2323b697f

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

@@ -6,11 +6,36 @@ module RedmineRemotes
       class Base < ::EacRailsUtils::TablelessModel
         private
 
+        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, updated_on: provider_issue.updated_on
+            created_on: provider_issue.created_on
           }
         end
 

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

@@ -22,7 +22,7 @@ module RedmineRemotes
         def create
           ::Issue.transaction do
             return false unless valid?
-            return false unless create_local_issue
+            return false unless save_local_issue
             return false unless create_remote_issue
           end
           true

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

@@ -6,20 +6,14 @@ module RedmineRemotes
       class Create < ::RedmineRemotes::Tableless::RemoteIssueFetch::Base
         private
 
-        def create_local_issue
-          return true if local_issue.save
-
-          fetch_record_errors(local_issue, default_column: default_error_column)
-          false
+        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 { |v| return v.local_issue }
-          ::Issue.new(
-            local_issue_provider_attributes.merge(
-              project: project, tracker: local_tracker, priority: local_issue_priority
-            )
-          )
+          remote_issue_by_remote_code.if_present(&:local_issue) || ::Issue.new
         end
 
         def local_issue_priority

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

@@ -20,7 +20,7 @@ module RedmineRemotes
 
         def save
           return false unless valid?
-          return false unless update_local_issue_with_remote_data
+          return false unless save_local_issue
 
           true
         end

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

@@ -13,13 +13,6 @@ module RedmineRemotes
         def local_tracker
           local_issue.if_present(&:tracker)
         end
-
-        def update_local_issue_with_remote_data
-          return true if local_issue.update(local_issue_provider_attributes)
-
-          fetch_record_errors(local_issue, default_column: default_error_column)
-          false
-        end
       end
     end
   end

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

@@ -30,10 +30,6 @@ module RedmineRemotes
           fields.fetch('status').fetch('name')
         end
 
-        def updated_on
-          fields.fetch('updated')
-        end
-
         def author_login
           fields.try(:fetch, 'reporter').try(:fetch, 'key')
         end

data/lib/redmine_remotes/version.rb

@@ -3,7 +3,7 @@
 module RedmineRemotes
   SLUG = 'redmine_remotes'
   NAME = 'Redmine Remotes'
-  VERSION = '0.8.1'
+  VERSION = '0.8.2'
   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.8.1
+  version: 0.8.2
 platform: ruby
 authors:
 - Eduardo Henrique Bogoni