Split the transaction used for ways so that we recover more cleanly

[rails.git] / app / models / node.rb

diff --git a/app/models/node.rb b/app/models/node.rb
index 230c83882cf12fee4ba8d23e5477cdc0e40729bc..7e78aaf592acfab37a89a9c20bbc084ea3b28ef5 100644 (file)
--- a/app/models/node.rb
+++ b/app/models/node.rb
@@ -68,18 +68,12 @@ class Node < ActiveRecord::Base
     return node
   end
 
     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
 
     end
   end