Refactored list method in all controllers

Author: teetangh

app/controllers/api/v1/airlines_controller.rb

@@ -77,20 +77,7 @@ def destroy
       # GET /api/v1/airlines/list
       def index
         airlines = Airline.all
-        airlines.each do |foo|
-          puts foo
-        end
-        formatted_airlines = airlines.map do |airline|
-          {
-            id: airline.id,
-            name: airline.name,
-            iata: airline.iata,
-            icao: airline.icao,
-            callsign: airline.callsign,
-            country: airline.country
-          }
-        end
-        render json: formatted_airlines
+        render json: airlines.map { |airline| airline.attributes.except('id') }, status: :ok
       rescue StandardError => e
         render json: { error: e.message }, status: :internal_server_error
       end

app/controllers/api/v1/airports_controller.rb

@@ -76,23 +76,7 @@ def destroy
       # GET /api/v1/airports/list
       def index
         airports = Airport.all
-        formatted_airports = airports.map do |airport|
-          {
-            id: airport.id,
-            airportname: airport.airportname,
-            city: airport.city,
-            country: airport.country,
-            faa: airport.faa,
-            icao: airport.icao,
-            tz: airport.tz,
-            geo: {
-              lat: airport.geo&.lat,
-              lon: airport.geo&.lon,
-              alt: airport.geo&.alt
-            }
-          }
-        end
-        render json: formatted_airports
+        render json: airports.map { |airport| airport.attributes.except('id') }, status: :ok
       rescue StandardError => e
         render json: { error: e.message }, status: :internal_server_error
       end

config/routes.rb

@@ -13,13 +13,15 @@
       delete 'airlines/:id', to: 'airlines#destroy'
 
       # Airports resource routes
+      get 'airports/list', to: 'airports#index'
       get 'airports/direct-connections', to: 'airports#direct_connections'
       get 'airports/:id', to: 'airports#show'
       post 'airports/:id', to: 'airports#create'
       put 'airports/:id', to: 'airports#update'
       delete 'airports/:id', to: 'airports#destroy'
 
       # Routes resource routes
+      get 'routes/list', to: 'routes#index'
       get 'routes/:id', to: 'routes#show'
       post 'routes/:id', to: 'routes#create'
       put 'routes/:id', to: 'routes#update'