support for sparse fieldsets for providers and clients

app/controllers/application_controller.rb

@@ -14,6 +14,9 @@ class ApplicationController < ActionController::API
   # include helper module for pagination
   include Paginatable
 
+  # include helper module for sparse fieldsets
+  include Fieldable
+
   attr_accessor :current_user
 
   # pass ability into serializer

app/controllers/clients_controller.rb

@@ -54,15 +54,19 @@ def index
           "provider-id" => params[:provider_id],
           software: params[:software],
           year: params[:year],
-          fields: params[:fields],
           "page[number]" => page[:number] + 1,
           "page[size]" => page[:size],
           sort: params[:sort] }.compact.to_query
         }.compact
       options[:include] = @include
       options[:is_collection] = true
 
-      render json: ClientSerializer.new(@clients, options).serialized_json, status: :ok
+      fields = fields_from_params(params)
+      if fields
+        render json: ClientSerializer.new(@clients, options.merge(fields: fields)).serialized_json, status: :ok
+      else
+        render json: ClientSerializer.new(@clients, options).serialized_json, status: :ok
+      end
     rescue Elasticsearch::Transport::Transport::Errors::BadRequest => exception
       Raven.capture_exception(exception)
 

app/controllers/concerns/fieldable.rb

@@ -0,0 +1,13 @@
+module Fieldable
+  extend ActiveSupport::Concern
+
+  included do
+    def fields_from_params(params)
+      fields = params.to_unsafe_h.dig(:fields)
+      return nil unless fields.is_a?(Hash)
+
+      fields.each { |k, v| fields[k] = v.split(",") }
+      fields
+    end
+  end
+end

app/controllers/concerns/paginatable.rb

@@ -28,4 +28,4 @@ def page_from_params(params)
       page
     end
   end
-end
+end

app/controllers/dois_controller.rb

@@ -190,14 +190,10 @@ def index
           }
 
           bmr = Benchmark.ms {
-            logger.info params[:fields]
-            # support sparse fieldsets
-            if params[:fields]
-              f = params.to_unsafe_h.dig(:fields)
-              f = { dois: "doi" } unless f.is_a?(Hash)
-              f.each { |k, v| f[k] = v.split(",") }
-
-              render json: DoiSerializer.new(results, options.merge(fields: f)).serialized_json, status: :ok
+            # sparse fieldsets
+            fields = fields_from_params(params)
+            if fields              
+              render json: DoiSerializer.new(results, options.merge(fields: fields)).serialized_json, status: :ok
             else
               render json: DoiSerializer.new(results, options).serialized_json, status: :ok
             end

app/controllers/providers_controller.rb

@@ -61,7 +61,6 @@ def index
                 "member_type" => params[:member_type],
                 "organization_type" => params[:organization_type],
                 "focus-area" => params[:focus_area],
-                fields: params[:fields],
                 "page[number]" => page[:number] + 1,
                 "page[size]" => page[:size],
                 sort: sort }.compact.to_query
@@ -71,7 +70,13 @@ def index
             options[:params] = {
               :current_ability => current_ability,
             }
-            render json: ProviderSerializer.new(@providers, options).serialized_json, status: :ok
+
+            fields = fields_from_params(params)
+            if fields
+              render json: ProviderSerializer.new(@providers, options.merge(fields: fields)).serialized_json, status: :ok
+            else
+              render json: ProviderSerializer.new(@providers, options).serialized_json, status: :ok
+            end
         end
         header = %w(
           accountName