Avoid unnecessary arrays
authormmd-osm <mmd.osm@gmail.com>
Thu, 27 Jun 2019 16:33:12 +0000 (18:33 +0200)
committermmd-osm <mmd.osm@gmail.com>
Thu, 27 Jun 2019 16:33:12 +0000 (18:33 +0200)
app/controllers/api/nodes_controller.rb
app/controllers/api/old_controller.rb
app/controllers/api/relations_controller.rb
app/controllers/api/ways_controller.rb
app/views/api/nodes/show.xml.builder
app/views/api/old_nodes/version.xml.builder
app/views/api/old_relations/version.xml.builder
app/views/api/old_ways/version.xml.builder
app/views/api/relations/show.xml.builder
app/views/api/ways/full.xml.builder
app/views/api/ways/show.xml.builder

index 630149a..891528a 100644 (file)
@@ -31,7 +31,7 @@ module Api
       response.last_modified = node.timestamp
 
       if node.visible
-        @nodes = [node]
+        @node = node
 
         # Render the result
         respond_to do |format|
index 39bb35c..bf3fcdb 100644 (file)
@@ -28,11 +28,7 @@ module Api
                            @elements.unredacted
                          end
 
-      @elems = []
-
-      visible_elements.each do |element|
-        @elems << element
-      end
+      @elems = visible_elements
 
       # Render the result
       respond_to do |format|
@@ -47,8 +43,6 @@ module Api
       else
         response.last_modified = @old_element.timestamp
 
-        @elems = [@old_element]
-
         # Render the result
         respond_to do |format|
           format.xml
index 9f125cf..40d919d 100644 (file)
@@ -25,7 +25,7 @@ module Api
       relation = Relation.find(params[:id])
       response.last_modified = relation.timestamp
       if relation.visible
-        @relations = [relation]
+        @relation = relation
 
         # Render the result
         respond_to do |format|
index b8ce101..04c823a 100644 (file)
@@ -27,7 +27,7 @@ module Api
       response.last_modified = way.timestamp
 
       if way.visible
-        @ways = [way]
+        @way = way
 
         # Render the result
         respond_to do |format|
@@ -80,7 +80,7 @@ module Api
           end
         end
 
-        @ways = [way]
+        @way = way
 
         # Render the result
         respond_to do |format|
index 1968de9..48f7e31 100644 (file)
@@ -1,5 +1,5 @@
 xml.instruct!
 
 xml.osm(OSM::API.new.xml_root_attributes) do |osm|
-  osm << (render(@nodes) || "")
+  osm << (render(@node) || "")
 end
index 905381c..c7b630d 100644 (file)
@@ -1,5 +1,5 @@
 xml.instruct!
 
 xml.osm(OSM::API.new.xml_root_attributes) do |osm|
-  osm << (render(@elems) || "")
+  osm << (render(@old_element) || "")
 end
index 905381c..c7b630d 100644 (file)
@@ -1,5 +1,5 @@
 xml.instruct!
 
 xml.osm(OSM::API.new.xml_root_attributes) do |osm|
-  osm << (render(@elems) || "")
+  osm << (render(@old_element) || "")
 end
index 905381c..c7b630d 100644 (file)
@@ -1,5 +1,5 @@
 xml.instruct!
 
 xml.osm(OSM::API.new.xml_root_attributes) do |osm|
-  osm << (render(@elems) || "")
+  osm << (render(@old_element) || "")
 end
index f39a20b..555eb4d 100644 (file)
@@ -1,5 +1,5 @@
 xml.instruct!
 
 xml.osm(OSM::API.new.xml_root_attributes) do |osm|
-  osm << (render(@relations) || "")
+  osm << (render(@relation) || "")
 end
index d8b3aea..0252916 100644 (file)
@@ -2,5 +2,5 @@ xml.instruct!
 
 xml.osm(OSM::API.new.xml_root_attributes) do |osm|
   osm << (render(@nodes) || "")
-  osm << (render(@ways) || "")
+  osm << (render(@way) || "")
 end
index bcb89cd..d520a08 100644 (file)
@@ -1,5 +1,5 @@
 xml.instruct!
 
 xml.osm(OSM::API.new.xml_root_attributes) do |osm|
-  osm << (render(@ways) || "")
+  osm << (render(@way) || "")
 end