rest_framework 0.6.6 → 0.6.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 88e17bbb8855d22fd7f73d637059d35bd12cc3fc9d982fbde73e11097b66af17
-  data.tar.gz: 8fe97e8557a9c957b3a31b51b89c4a7623e97c2fbc75c9c8b80bb23b54dc8429
+  metadata.gz: fc722756068a8151e2b84e0305b5484cb831c555ecd9a648af883d70e0196036
+  data.tar.gz: 4c5681012eb56285f7cce8baf6e8b6afb275857c69eab317c8aedb742aee62a2
 SHA512:
-  metadata.gz: 5362740a21ef0ef6248c36783da74ff5ad0b142ea9d2de7e228d5052d9badc7af3f78e259dba1015112c04379fd17c16265b0843c26cc2654e89a1742f53e051
-  data.tar.gz: 61d6ac5dca5113be086b9e3232e1e7c2e3fa26d45634764993c474cffcc62403f5285b9da0ebdcfd69b4dbe0b9fedaaa5f9d30d2366286a71c458fdbc9fe4ef7
+  metadata.gz: 0cc9f1280add8372ae481168786f1069ab8026d81971a75aab809f5614abcef656a16e0386875402a85bf780d3223e22af81b764bbfd996a8003efdf4fbb63e5
+  data.tar.gz: f8f2a20341a8a418c7dec9bd4a3fdea907ee4778bf7d6ac1557b9a430fe97dfba2e6b1f7bee94a30ad944e8bc3fb5ff77b4030c6d4a3a955f8e7f7a9ee0bd4b2

data/VERSION

@@ -1 +1 @@
-0.6.6
+0.6.8

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

@@ -73,7 +73,7 @@
         </div>
         <% if @json_payload.present? || @xml_payload.present? %>
           <div class="row">
-            <div class="w-100">
+            <div>
               <ul class="nav nav-tabs">
                 <% if @json_payload.present? %>
                   <li class="nav-item">
@@ -91,7 +91,7 @@
                 <% end %>
               </ul>
             </div>
-            <div class="tab-content w-100 pt-3">
+            <div class="tab-content pt-2">
               <div class="tab-pane fade show active" id="tab-json" role="tab">
                 <% if @json_payload.present? %>
                   <div>
@@ -109,7 +109,7 @@
         <% end %>
         <% if @route_groups.present? %>
           <div class="row">
-            <div class="w-100">
+            <div>
               <ul class="nav nav-tabs">
                 <li class="nav-item">
                   <a class="nav-link active" href="#tab-routes" data-bs-toggle="tab" role="tab">
@@ -128,7 +128,7 @@
                 <% end %>
               </ul>
             </div>
-            <div class="tab-content w-100 pt-3">
+            <div class="tab-content pt-2">
               <div class="tab-pane fade show active" id="tab-routes" role="tab">
                 <%= render partial: 'rest_framework/routes' %>
               </div>

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

@@ -18,7 +18,7 @@ h6 { font-size: 1rem; }
 /* Make code and code blocks a little nicer looking. */
 code {
   padding: 0 .35em;
-  background-color: #eee !important;
+  background-color: #f3f3f3 !important;
   border: 1px solid #aaa;
   border-radius: 3px;
 }

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

data/lib/rest_framework/filters.rb

@@ -86,7 +86,7 @@ class RESTFramework::ModelSearchFilter < RESTFramework::BaseFilter
     unless search.blank?
       return data.where(
         fields.map { |f|
-          "CAST(#{f} AS CHAR) #{@controller.class.search_ilike ? "ILIKE" : "LIKE"} ?"
+          "CAST(#{f} AS VARCHAR) #{@controller.class.search_ilike ? "ILIKE" : "LIKE"} ?"
         }.join(" OR "),
         *(["%#{search}%"] * fields.length),
       )

metadata

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