code cleanup: rubocop: fix Lint/IneffectiveAccessModifier in lib/redmine/helpers...

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

diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml
index 6e3fe213b68e74e6116bf9b3d83efa7b0cdc2bce..b08bd8eb9d1bcaf0ce557c525214ab35bd67d121 100644 (file)
--- a/.rubocop_todo.yml
+++ b/.rubocop_todo.yml
@@ -395,7 +395,6 @@ Lint/IneffectiveAccessModifier:
     - 'app/models/mail_handler.rb'
     - 'app/models/mailer.rb'
     - 'app/models/user.rb'
-    - 'lib/redmine/helpers/gantt.rb'
     - 'lib/redmine/scm/adapters/abstract_adapter.rb'
 
 Lint/InterpolationCheck:

diff --git a/lib/redmine/helpers/gantt.rb b/lib/redmine/helpers/gantt.rb
index 2258faf9c874854cd4f01305ad04551b15481c16..db0676ba7e7633974eb305465e2b52b54b941d4c 100644 (file)
--- a/lib/redmine/helpers/gantt.rb
+++ b/lib/redmine/helpers/gantt.rb
@@ -667,23 +667,26 @@ module Redmine
         start_date + (end_date - start_date + 1) * (progress / 100.0)
       end
 
-      def self.sort_issues!(issues)
-        issues.sort_by! {|issue| sort_issue_logic(issue)}
-      end
+      # Singleton class method is public
+      class << self
+        def sort_issues!(issues)
+          issues.sort_by! {|issue| sort_issue_logic(issue)}
+        end
 
-      def self.sort_issue_logic(issue)
-        julian_date = Date.new()
-        ancesters_start_date = []
-        current_issue = issue
-        begin
-          ancesters_start_date.unshift([current_issue.start_date || julian_date, current_issue.id])
-          current_issue = current_issue.parent
-        end while (current_issue)
-        ancesters_start_date
-      end
+        def sort_issue_logic(issue)
+          julian_date = Date.new()
+          ancesters_start_date = []
+          current_issue = issue
+          begin
+            ancesters_start_date.unshift([current_issue.start_date || julian_date, current_issue.id])
+            current_issue = current_issue.parent
+          end while (current_issue)
+          ancesters_start_date
+        end
 
-      def self.sort_versions!(versions)
-        versions.sort!
+        def sort_versions!(versions)
+          versions.sort!
+        end
       end
 
       def pdf_new_page?(options)