Say which way is too long.

diff --git a/app/models/way.rb b/app/models/way.rb
index 8788bd671b1e2ffdd9b1c9bd82c8f371edabc45a..e2641873284d5e84de69b47a220ed4c7cfbd6655 100644 (file)
--- a/app/models/way.rb
+++ b/app/models/way.rb
@@ -234,7 +234,7 @@ class Way < ActiveRecord::Base
   def preconditions_ok?(old_nodes = [])
     return false if self.nds.empty?
     if self.nds.length > APP_CONFIG['max_number_of_way_nodes']
-      raise OSM::APITooManyWayNodesError.new(self.nds.length, APP_CONFIG['max_number_of_way_nodes'])
+      raise OSM::APITooManyWayNodesError.new(self.id, self.nds.length, APP_CONFIG['max_number_of_way_nodes'])
     end
 
     # check only the new nodes, for efficiency - old nodes having been checked last time and can't

diff --git a/lib/osm.rb b/lib/osm.rb
index 789a5f4398b72dadb1a186e39f352155766638eb..f36deb0a80906fa185e3c62764727c29604fb436 100644 (file)
--- a/lib/osm.rb
+++ b/lib/osm.rb
@@ -187,18 +187,18 @@ module OSM
   # Raised when a way has more than the configured number of way nodes.
   # This prevents ways from being to long and difficult to work with
   class APITooManyWayNodesError < APIError
-    def initialize(provided, max)
-      @provided, @max = provided, max
+    def initialize(id, provided, max)
+      @id, @provided, @max = id, provided, max
     end
     
-    attr_reader :provided, :max
+    attr_reader :id, :provided, :max
 
     def status
       :bad_request
     end
     
     def to_s
-      "You tried to add #{provided} nodes to the way, however only #{max} are allowed"
+      "You tried to add #{provided} nodes to way #{id}, however only #{max} are allowed"
     end
   end