From 18f617e08aedd5d9920171e0076fe376ee508457 Mon Sep 17 00:00:00 2001
From: r888888888 <r888888888@gmail.com>
Date: Thu, 24 Aug 2017 15:39:06 -0700
Subject: [PATCH] potential fix for #3189

---
 app/models/user.rb | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/app/models/user.rb b/app/models/user.rb
index 00a32acec..b37ca7ea6 100644
--- a/app/models/user.rb
+++ b/app/models/user.rb
@@ -138,8 +138,13 @@ class User < ApplicationRecord
 
     module ClassMethods
       def name_to_id(name)
-        Cache.get("uni:#{Cache.sanitize(name)}", 4.hours) do
-          select_value_sql("SELECT id FROM users WHERE lower(name) = ?", name.mb_chars.downcase.tr(" ", "_").to_s)
+        Cache.get("uni:#{Cache.hash(name)}", 4.hours) do
+          val = select_value_sql("SELECT id FROM users WHERE lower(name) = ?", name.mb_chars.downcase.tr(" ", "_").to_s)
+          if val.present?
+            val.to_i
+          else
+            nil
+          end
         end
       end
 
@@ -168,7 +173,7 @@ class User < ApplicationRecord
 
     def update_cache
       Cache.put("uin:#{id}", name)
-      Cache.put("uni:#{Cache.sanitize(name)}", id)
+      Cache.put("uni:#{Cache.hash(name)}", id)
     end
 
     def update_remote_cache