rest_framework 0.0.9 → 0.0.15

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ce317c05e27ab6421b18f37595dbb846a747c8616d19fc9a1927a9ecc30fefc7
-  data.tar.gz: b8f88631bfca835f395677af0e0c0079b40c079aa8d9a6cc8c8ee46dc5d589c2
+  metadata.gz: 70c2eb85612656cf6f1b3d34b97deb06ad01aa07459012b4254e543fc0bd5507
+  data.tar.gz: 76f44db8c396cfe5bb539faebb6f6031276d7133327b733cab4a645922176d95
 SHA512:
-  metadata.gz: abd9fc1e83d61c60af77d834d3a3e03a4a904863f3ad4bb851d8cf35a5a563a322eef508a273764ce727303e3e25f526cca03a643127957a7d9952e4988de743
-  data.tar.gz: 8aadf4ba5467c645da2404df61880bd741a491eea0c14a09b20e7056726022f7fc7bd4b8a3f90e9ac068b578e63870c85cfaa813296245cf8c4fddfb7d002a66
+  metadata.gz: 246bd5c3972c212c1d6fcf600da1c894fe05acaeda8596d432b6b360e265520b3e54f3b4842998c26868464c29bd79f3c164f51104145163c0a63ba9e94f06fb
+  data.tar.gz: 52af9985b986a167f70cd8def492a3ff2c7d61bafd48e120f49934cc3827f76a1e6dbc5c02651df6bba7fdd622be058f945534415d6cb74e3537c56f9c18c0e2

data/app/views/layouts/rest_framework.html.erb

@@ -19,7 +19,7 @@
     <div class="container py-3">
       <div class="container">
         <div class="row">
-          <h1><%= @title %></h1>
+          <h1><%= (@header_text if defined? @header_text) || @title %></h1>
         </div>
         <hr/>
         <% if @json_payload || @xml_payload %>
@@ -46,7 +46,7 @@
             <div class="tab-content w-100 pt-3">
               <div class="tab-pane fade show active" id="tab-json" role="tab">
                 <% if @json_payload %>
-                  <div><pre><code class="language-json"><%= JSON.pretty_generate(JSON.parse(@json_payload)) %></code></pre></div>
+                  <div><pre><code class="language-json"><%= JSON.pretty_generate(JSON.parse(@json_payload)) unless @json_payload == '' %></code></pre></div>
                 <% end %>
               </div>
               <div class="tab-pane fade" id="tab-xml" role="tab">

data/lib/rest_framework/VERSION_STAMP

@@ -1 +1 @@
-0.0.9
+0.0.15

data/lib/rest_framework/controller_mixins/base.rb

@@ -34,6 +34,11 @@ module RESTFramework
           :skip_actions,
           :paginator_class,
         ])
+
+        # skip csrf since this is an API
+        base.skip_before_action(:verify_authenticity_token) rescue nil
+
+        # handle some common exceptions
         base.rescue_from(ActiveRecord::RecordNotFound, with: :record_not_found)
         base.rescue_from(ActiveRecord::RecordInvalid, with: :record_invalid)
         base.rescue_from(ActiveRecord::RecordNotSaved, with: :record_not_saved)
@@ -69,40 +74,47 @@ module RESTFramework
       end
       return ActionDispatch::Routing::RoutesInspector.new(Rails.application.routes.routes).format(
         formatter.new
-      ).lines[1..].map { |r| r.split.last(3) }.map { |r|
+      ).lines.drop(1).map { |r| r.split.last(3) }.map { |r|
         {verb: r[0], path: r[1], action: r[2]}
       }.select { |r| r[:path].start_with?(request.path) }
     end
 
-    # Helper alias for `respond_to`/`render`, and replace nil responses with blank ones. `payload`
-    # must be already serialized to Ruby primitives.
+    # Helper alias for `respond_to`/`render`. `payload` should be already serialized to Ruby
+    # primitives.
     def api_response(payload, html_kwargs: nil, json_kwargs: nil, xml_kwargs: nil, **kwargs)
       html_kwargs ||= {}
       json_kwargs ||= {}
       xml_kwargs ||= {}
 
-      # make empty responses status 204 unless a status is already explicitly defined
-      if (payload.nil? || payload == '') && !kwargs.key?(:status)
-        kwargs[:status] = 204
-      end
+      # allow blank (no-content) responses
+      @blank = kwargs[:blank]
 
       respond_to do |format|
-        if payload.respond_to?(:to_json)
-          format.json {
-            kwargs = kwargs.merge(json_kwargs)
-            render(json: payload || '', **kwargs)
-          }
-        end
-        if payload.respond_to?(:to_xml)
-          format.xml {
-            kwargs = kwargs.merge(xml_kwargs)
-            render(xml: payload || '', **kwargs)
-          }
+        if @blank
+          format.json {head :no_content}
+          format.xml {head :no_content}
+        else
+          if payload.respond_to?(:to_json)
+            format.json {
+              kwargs = kwargs.merge(json_kwargs)
+              render(json: payload, layout: false, **kwargs)
+            }
+          end
+          if payload.respond_to?(:to_xml)
+            format.xml {
+              kwargs = kwargs.merge(xml_kwargs)
+              render(xml: payload, layout: false, **kwargs)
+            }
+          end
         end
         format.html {
           @payload = payload
-          @json_payload = payload.to_json
-          @xml_payload = payload.to_xml
+          @json_payload = ''
+          @xml_payload = ''
+          unless @blank
+            @json_payload = payload.to_json if payload.respond_to?(:to_json)
+            @xml_payload = payload.to_xml if payload.respond_to?(:to_xml)
+          end
           @template_logo_text ||= "Rails REST Framework"
           @title ||= self.controller_name.camelize
           @routes ||= self._get_routes

data/lib/rest_framework/controller_mixins/models.rb

@@ -20,6 +20,7 @@ module RESTFramework
           :allowed_action_parameters,
           :serializer_class,
           :extra_member_actions,
+          :disable_creation_from_recordset,
         ])
         base.alias_method(:extra_collection_actions=, :extra_actions=)
       end
@@ -31,49 +32,43 @@ module RESTFramework
       return self.class.serializer_class || NativeModelSerializer
     end
 
-    # Get a list of fields for an action (or the current action if none given).
-    def get_fields(action: nil)
+    # Get a list of fields for the current action.
+    def get_fields
       action_fields = self.class.action_fields || {}
-
-      # action will, by default, be the current action name
-      action = action_name.to_sym unless action
+      action = self.action_name.to_sym
 
       # index action should use :list fields if :index is not provided
       action = :list if action == :index && !action_fields.key?(:index)
 
-      return action_fields[action] || self.class.fields || []
+      return (action_fields[action] if action) || self.class.fields || []
     end
 
-    # Get a native serializer config for an action (or the current action if none given).
-    def get_native_serializer_config(action: nil)
-      native_serializer_action_config = self.class.native_serializer_action_config || {}
-
-      # action will, by default, be the current action name
-      action = action_name.to_sym unless action
+    # Get a native serializer config for the current action.
+    def get_native_serializer_config
+      action_serializer_config = self.class.native_serializer_action_config || {}
+      action = self.action_name.to_sym
 
       # index action should use :list serializer config if :index is not provided
-      action = :list if action == :index && !native_serializer_action_config.key?(:index)
+      action = :list if action == :index && !action_serializer_config.key?(:index)
 
-      return native_serializer_action_config[action] || self.class.native_serializer_config
+      return (action_serializer_config[action] if action) || self.class.native_serializer_config
     end
 
-    # Get a list of parameters allowed for an action (or the current action if none given).
-    def get_allowed_parameters(action: nil)
+    # Get a list of parameters allowed for the current action.
+    def get_allowed_parameters
       allowed_action_parameters = self.class.allowed_action_parameters || {}
-
-      # action will, by default, be the current action name
-      action = action_name.to_sym unless action
+      action = self.action_name.to_sym
 
       # index action should use :list allowed parameters if :index is not provided
       action = :list if action == :index && !allowed_action_parameters.key?(:index)
 
-      return allowed_action_parameters[action] || self.class.allowed_parameters
+      return (allowed_action_parameters[action] if action) || self.class.allowed_parameters
     end
 
     # Filter the request body for keys in current action's allowed_parameters/fields config.
     def _get_parameter_values_from_request_body
       fields = self.get_allowed_parameters || self.get_fields
-      return @_get_field_values_from_request_body ||= (request.request_parameters.select { |p|
+      return @_get_parameter_values_from_request_body ||= (request.request_parameters.select { |p|
         fields.include?(p.to_sym) || fields.include?(p.to_s)
       })
     end
@@ -83,7 +78,7 @@ module RESTFramework
     # Filter params for keys allowed by the current action's filterset_fields/fields config.
     def _get_filterset_values_from_params
       fields = self.filterset_fields || self.get_fields
-      return @_get_field_values_from_params ||= request.query_parameters.select { |p|
+      return @_get_filterset_values_from_params ||= request.query_parameters.select { |p|
         fields.include?(p.to_sym) || fields.include?(p.to_s)
       }
     end
@@ -171,7 +166,13 @@ module RESTFramework
 
   module CreateModelMixin
     def create
-      @record = self.get_model.create!(self.get_create_params)
+      if self.get_recordset.respond_to?(:create!) && !self.disable_creation_from_recordset
+        # Create with any properties inherited from the recordset (like associations).
+        @record = self.get_recordset.create!(self.get_create_params)
+      else
+        # Otherwise, perform a "bare" create.
+        @record = self.get_model.create!(self.get_create_params)
+      end
       @serialized_record = self.get_serializer_class.new(
         object: @record, controller: self
       ).serialize

data/lib/rest_framework/serializers.rb

@@ -7,29 +7,59 @@ module RESTFramework
       @data = data
       @controller = controller
     end
-
-    def is_valid
-      return true
-    end
   end
 
   # This serializer uses `.as_json` to serialize objects. Despite the name, `.as_json` is a Rails
   # method which converts objects to Ruby primitives (with the top-level being either an array or a
   # hash).
   class NativeModelSerializer < BaseSerializer
-    def initialize(model: nil, **kwargs)
+    class_attribute :config
+    class_attribute :singular_config
+    class_attribute :plural_config
+    class_attribute :action_config
+
+    def initialize(model: nil, many: nil, **kwargs)
       super(**kwargs)
-      @model = model || @controller.send(:get_model)
+      @many = many
+      @model = model || (@controller ? @controller.send(:get_model) : nil)
+    end
+
+    # Get controller action, if possible.
+    def get_action
+      return @controller&.action_name&.to_sym
+    end
+
+    # Get a locally defined configuration, if one is defined.
+    def get_local_serializer_config
+      action = self.get_action
+
+      if action && self.action_config
+        # index action should use :list serializer config if :index is not provided
+        action = :list if action == :index && !self.action_config.key?(:index)
+
+        return self.action_config[action] if self.action_config[action]
+      end
+
+      # no action_config, so try singular/plural config
+      return self.plural_config if @many && self.plural_config
+      return self.singular_config if !@many && self.singular_config
+
+      # lastly, try the default config
+      return self.config
     end
 
     # Get a configuration passable to `as_json` for the model.
-    def get_native_serializer_config
+    def get_serializer_config
+      # return a locally defined serializer config if one is defined
+      local_config = self.get_local_serializer_config
+      return local_config if local_config
+
       # return a serializer config if one is defined
       serializer_config = @controller.send(:get_native_serializer_config)
       return serializer_config if serializer_config
 
-      # build serializer config from fields
-      fields = @controller.send(:get_fields)
+      # otherwise, build a serializer config from fields
+      fields = @controller.send(:get_fields) if @controller
       unless fields.blank?
         columns, methods = fields.partition { |f| f.to_s.in?(@model.column_names) }
         return {only: columns, methods: methods}
@@ -41,9 +71,38 @@ module RESTFramework
     # Convert the object(s) to Ruby primitives.
     def serialize
       if @object
-        return @object.as_json(self.get_native_serializer_config)
+        @many = @object.respond_to?(:each) if @many.nil?
+        return @object.as_json(self.get_serializer_config)
       end
       return nil
     end
+
+    # Allow a serializer instance to be used as a hash directly in a nested serializer config.
+    def [](key)
+      unless instance_variable_defined?(:@_nested_config)
+        @_nested_config = self.get_serializer_config
+      end
+      return @_nested_config[key]
+    end
+    def []=(key, value)
+      unless instance_variable_defined?(:@_nested_config)
+        @_nested_config = self.get_serializer_config
+      end
+      return @_nested_config[key] = value
+    end
+
+    # Allow a serializer class to be used as a hash directly in a nested serializer config.
+    def self.[](key)
+      unless instance_variable_defined?(:@_nested_config)
+        @_nested_config = self.new.get_serializer_config
+      end
+      return @_nested_config[key]
+    end
+    def self.[]=(key, value)
+      unless instance_variable_defined?(:@_nested_config)
+        @_nested_config = self.new.get_serializer_config
+      end
+      return @_nested_config[key] = value
+    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rest_framework
 version: !ruby/object:Gem::Version
-  version: 0.0.9
+  version: 0.0.15
 platform: ruby
 authors:
 - Gregory N. Schmit
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-10-06 00:00:00.000000000 Z
+date: 2020-12-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails