X-Git-Url: https://git.openstreetmap.org/rails.git/blobdiff_plain/c11d961f624cb610c4b8184d24e522b2c093d1ea..e15916ad5b081f94c433f1e0bb708be242dbe5ab:/app/models/node.rb

diff --git a/app/models/node.rb b/app/models/node.rb
index 8b09ffef7..7e78aaf59 100644
--- a/app/models/node.rb
+++ b/app/models/node.rb
@@ -68,18 +68,12 @@ class Node < ActiveRecord::Base
     return node
   end
 
-  def save_with_history
-    begin
-      Node.transaction do
-        self.timestamp = Time.now
-        self.save!
-        old_node = OldNode.from_node(self)
-        old_node.save!
-      end
-
-      return true
-    rescue
-      return nil
+  def save_with_history!
+    Node.transaction do
+      self.timestamp = Time.now
+      self.save!
+      old_node = OldNode.from_node(self)
+      old_node.save!
     end
   end
 
@@ -97,7 +91,7 @@ class Node < ActiveRecord::Base
 
     user_display_name_cache = {} if user_display_name_cache.nil?
 
-    if user_display_name_cache and user_display_name_cache[self.user_id]
+    if user_display_name_cache and user_display_name_cache.key?(self.user_id)
       # use the cache if available
     elsif self.user.data_public?
       user_display_name_cache[self.user_id] = self.user.display_name