rest_framework 0.6.7 → 0.6.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4aef3d359b16de4e6980d055031a4eb1d1ebd3f9aafa39af6b9effec8e1319d2
-  data.tar.gz: 5df89b3b8f9e6879cb6422d629743471e7641cb3b06736fdd7ddc583e6a5d182
+  metadata.gz: fc722756068a8151e2b84e0305b5484cb831c555ecd9a648af883d70e0196036
+  data.tar.gz: 4c5681012eb56285f7cce8baf6e8b6afb275857c69eab317c8aedb742aee62a2
 SHA512:
-  metadata.gz: 01efb5108e191ad044fd64c61781cba2c2ea3e2a516f4cde1c7b3fec1fa7568ed29dda159a37440b1e59f59b6fbeb0754be4b4a37c86c0c4f94e10d81aa57ec4
-  data.tar.gz: c5302e76238dbd5fc610a9b2bea424c62646d8766f23a2a7cfe9d22ac5f4f40db99901c448696f315347e9f358756fd0d373b2003a32288d327e6ec08fa1cf75
+  metadata.gz: 0cc9f1280add8372ae481168786f1069ab8026d81971a75aab809f5614abcef656a16e0386875402a85bf780d3223e22af81b764bbfd996a8003efdf4fbb63e5
+  data.tar.gz: f8f2a20341a8a418c7dec9bd4a3fdea907ee4778bf7d6ac1557b9a430fe97dfba2e6b1f7bee94a30ad944e8bc3fb5ff77b4030c6d4a3a955f8e7f7a9ee0bd4b2

data/VERSION

@@ -1 +1 @@
-0.6.7
+0.6.8

data/lib/rest_framework/controller_mixins/base.rb

@@ -36,23 +36,28 @@ module RESTFramework::BaseControllerMixin
       {
         filter_pk_from_request_body: true,
         exclude_body_fields: [:created_at, :created_by, :updated_at, :updated_by],
-        accept_generic_params_as_body_params: true,
+        accept_generic_params_as_body_params: false,
+        show_backtrace: false,
         extra_actions: nil,
         extra_member_actions: nil,
         filter_backends: nil,
+        singleton_controller: nil,
+        skip_actions: nil,
+
+        # Options related to serialization.
+        rescue_unknown_format_with: :json,
+        serializer_class: nil,
+        serialize_to_json: true,
+        serialize_to_xml: true,
+
+        # Options related to pagination.
         paginator_class: nil,
         page_size: 20,
         page_query_param: "page",
         page_size_query_param: "page_size",
         max_page_size: nil,
-        rescue_unknown_format_with: :json,
-        serializer_class: nil,
-        serialize_to_json: true,
-        serialize_to_xml: true,
-        singleton_controller: nil,
-        skip_actions: nil,
 
-        # Helper to disable serializer adapters by default, mainly introduced because Active Model
+        # Option to disable serializer adapters by default, mainly introduced because Active Model
         # Serializers will do things like serialize `[]` into `{"":[]}`.
         disable_adapters_by_default: true,
       }.each do |a, default|
@@ -83,6 +88,7 @@ module RESTFramework::BaseControllerMixin
       base.rescue_from(ActiveRecord::RecordInvalid, with: :record_invalid)
       base.rescue_from(ActiveRecord::RecordNotSaved, with: :record_not_saved)
       base.rescue_from(ActiveRecord::RecordNotDestroyed, with: :record_not_destroyed)
+      base.rescue_from(ActiveModel::UnknownAttributeError, with: :unknown_attribute_error)
     end
   end
 
@@ -126,23 +132,46 @@ module RESTFramework::BaseControllerMixin
 
   def record_invalid(e)
     return api_response(
-      {message: "Record invalid.", exception: e, errors: e.record&.errors}, status: 400
+      {
+        message: "Record invalid.", errors: e.record&.errors
+      }.merge(self.class.show_backtrace ? {exception: e.full_message} : {}),
+      status: 400,
     )
   end
 
   def record_not_found(e)
-    return api_response({message: "Record not found.", exception: e}, status: 404)
+    return api_response(
+      {
+        message: "Record not found.",
+      }.merge(self.class.show_backtrace ? {exception: e.full_message} : {}),
+      status: 404,
+    )
   end
 
   def record_not_saved(e)
     return api_response(
-      {message: "Record not saved.", exception: e, errors: e.record&.errors}, status: 406
+      {
+        message: "Record not saved.", errors: e.record&.errors
+      }.merge(self.class.show_backtrace ? {exception: e.full_message} : {}),
+      status: 400,
     )
   end
 
   def record_not_destroyed(e)
     return api_response(
-      {message: "Record not destroyed.", exception: e, errors: e.record&.errors}, status: 406
+      {
+        message: "Record not destroyed.", errors: e.record&.errors
+      }.merge(self.class.show_backtrace ? {exception: e.full_message} : {}),
+      status: 400,
+    )
+  end
+
+  def unknown_attribute_error(e)
+    return api_response(
+      {
+        message: e.message.capitalize,
+      }.merge(self.class.show_backtrace ? {exception: e.full_message} : {}),
+      status: 400,
     )
   end
 

data/lib/rest_framework/controller_mixins/models.rb

@@ -125,17 +125,17 @@ module RESTFramework::BaseModelControllerMixin
   def get_body_params
     # Filter the request body and map to strings. Return all params if we cannot resolve a list of
     # allowed parameters or fields.
-    body_params = if allowed_params = self.get_allowed_parameters&.map(&:to_s)
+    allowed_params = self.get_allowed_parameters&.map(&:to_s)
+    body_params = if allowed_params
       request.request_parameters.select { |p| allowed_params.include?(p) }
     else
       request.request_parameters
     end
 
-    # Add query params in place of missing body params, if configured. If fields are not defined,
-    # fallback to using columns for this particular feature.
-    if self.class.accept_generic_params_as_body_params
-      (self.get_fields(fallback: true) - body_params.keys).each do |k|
-        if (value = params[k])
+    # Add query params in place of missing body params, if configured.
+    if self.class.accept_generic_params_as_body_params && allowed_params
+      (allowed_params - body_params.keys).each do |k|
+        if value = params[k].presence
           body_params[k] = value
         end
       end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rest_framework
 version: !ruby/object:Gem::Version
-  version: 0.6.7
+  version: 0.6.8
 platform: ruby
 authors:
 - Gregory N. Schmit
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-11-08 00:00:00.000000000 Z
+date: 2022-12-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails