cleanup: rubocop: fix Style/MultilineIfModifier in lib/redmine/scm/adapters/subversio...

git-svn-id: http://svn.redmine.org/redmine/trunk@19183 e93f8b46-1217-0410-a6f0-8f06a7374b81

diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml
index 850e0595a7d64c4b93c248768536b7269f2a2740..a9152bb15f97d4383fd94b05409071a6d7710baa 100644 (file)
--- a/.rubocop_todo.yml
+++ b/.rubocop_todo.yml
@@ -1264,7 +1264,6 @@ Style/MultilineIfModifier:
     - 'lib/redmine/access_keys.rb'
     - 'lib/redmine/helpers/gantt.rb'
     - 'lib/redmine/scm/adapters/git_adapter.rb'
-    - 'lib/redmine/scm/adapters/subversion_adapter.rb'
     - 'lib/redmine/wiki_formatting.rb'
     - 'lib/redmine/wiki_formatting/textile/redcloth3.rb'
     - 'test/mocks/open_id_authentication_mock.rb'

diff --git a/lib/redmine/scm/adapters/subversion_adapter.rb b/lib/redmine/scm/adapters/subversion_adapter.rb
index f3b5a15341472dd6aa2b10fd8b2bd8407d5769b6..09b4aacb5b36b4cc6b3a930b4693cf68d0defd0f 100644 (file)
--- a/lib/redmine/scm/adapters/subversion_adapter.rb
+++ b/lib/redmine/scm/adapters/subversion_adapter.rb
@@ -172,13 +172,17 @@ module Redmine
               doc = parse_xml(output)
               each_xml_element(doc['log'], 'logentry') do |logentry|
                 paths = []
-                each_xml_element(logentry['paths'], 'path') do |path|
-                  paths << {:action => path['action'],
-                            :path => path['__content__'],
-                            :from_path => path['copyfrom-path'],
-                            :from_revision => path['copyfrom-rev']
-                            }
-                end if logentry['paths'] && logentry['paths']['path']
+                if logentry['paths'] && logentry['paths']['path']
+                  each_xml_element(logentry['paths'], 'path') do |path|
+                    paths <<
+                      {
+                        :action => path['action'],
+                        :path => path['__content__'],
+                        :from_path => path['copyfrom-path'],
+                        :from_revision => path['copyfrom-rev']
+                      }
+                  end
+                end
                 paths.sort_by! {|e| e[:path]}
 
                 revisions << Revision.new({:identifier => logentry['revision'],