Merge pull request #177 from gtt-project/ftr/support-include-layers-for-projects-api

Support 'include=layers' for projects index/show api (#172)

Author: sanak

app/views/projects/show.api.rsb

@@ -14,10 +14,6 @@ api.project do
     api.geojson ""
   end
 
-  if @project.gtt_tile_sources.present?
-    api.gttLayer @project.gtt_tile_sources
-  end
-
   render_api_custom_values @project.visible_custom_field_values, api
   render_api_includes(@project, api)
 

lib/redmine_gtt/patches/projects_helper_patch.rb

@@ -8,11 +8,21 @@ module Patches
     module ProjectsHelperPatch
 
       def self.apply
+        ProjectsHelper.prepend self unless ProjectsHelper < self
         ProjectsController.class_eval do
           helper SettingsMenuItem
         end
       end
 
+      def render_api_includes(project, api)
+        super
+        api.array :layers do
+          project.gtt_tile_sources.each do |gtt_tile_source|
+            api.layer(gtt_tile_source.attributes)
+          end
+        end if include_in_api_response?('layers')
+      end
+
       module SettingsMenuItem
         def project_settings_tabs
           super.tap do |tabs|

test/integration/projects_api_test.rb

@@ -164,6 +164,28 @@ class ProjectsApiTest < Redmine::IntegrationTest
     assert_equal 1, projects.xpath('geojson').size, projects.to_s
   end
 
+  test 'GET /projects.xml with include=layers should return layers' do
+    ts = RedmineGtt::Actions::CreateTileSource.(type: 'ol.source.OSM', name: 'default', default: true).tile_source
+    @project.gtt_tile_sources = [ts]
+    get '/projects.xml?include=layers'
+    assert_response :success
+    xml = xml_data
+    assert projects = xml.xpath('/projects/project')
+    assert layer = projects.xpath('layers/layer').first
+    assert_equal 'ol.source.OSM', layer['type']
+  end
+
+  test 'GET /projects/1.xml with include=layers should return layers' do
+    ts = RedmineGtt::Actions::CreateTileSource.(type: 'ol.source.OSM', name: 'default', default: true).tile_source
+    @project.gtt_tile_sources = [ts]
+    get '/projects/1.xml?include=layers'
+    assert_response :success
+    xml = xml_data
+    assert project = xml.xpath('/project')
+    assert layer = project.xpath('layers/layer').first
+    assert_equal 'ol.source.OSM', layer['type']
+  end
+
   def xml_data
     Nokogiri::XML(@response.body)
   end