code cleanup: rubocop: fix Lint/IneffectiveAccessModifier in app/models/user.rb

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

.rubocop_todo.yml

@@ -393,7 +393,6 @@ Lint/IneffectiveAccessModifier:
     - 'app/models/attachment.rb'
     - 'app/models/issue.rb'
     - 'app/models/mail_handler.rb'
-    - 'app/models/user.rb'
 
 Lint/InterpolationCheck:
   Exclude:

app/models/user.rb

@@ -903,14 +903,17 @@ class User < Principal
     WikiContent::Version.where(['author_id = ?', id]).update_all(['author_id = ?', substitute.id])
   end
 
-  # Return password digest
-  def self.hash_password(clear_password)
-    Digest::SHA1.hexdigest(clear_password || "")
-  end
+  # Singleton class method is public
+  class << self
+    # Return password digest
+    def hash_password(clear_password)
+      Digest::SHA1.hexdigest(clear_password || "")
+    end
 
-  # Returns a 128bits random salt as a hex string (32 chars long)
-  def self.generate_salt
-    Redmine::Utils.random_hex(16)
+    # Returns a 128bits random salt as a hex string (32 chars long)
+    def generate_salt
+      Redmine::Utils.random_hex(16)
+    end
   end
 
   # Send a security notification to all admins if the user has gained/lost admin privileges