rest_framework 0.9.11 → 0.9.14

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 04ef01197efeca7c6276edbf2ed4c9ef3f50e4fc4d1df927c3d530953d5aab2a
-  data.tar.gz: 71bd781956d163f33cdc7fd6aa3950ce93d4f86a0c03325e2bebcd2ba1dfc74b
+  metadata.gz: f0a39ff15a5a6714fb7dc52f11e7db3cebe918805a44b0a8486567debebcefac
+  data.tar.gz: 5e2e530f249b567c64bb0d527875b291180052b07d61919bbbf76ae3f5ad7b1b
 SHA512:
-  metadata.gz: 96cbf98cec8f47c54309fa5b2b46c90c60249e7da1fa7142ef85c041bfee1154b9fd60c5d420d145a2dfb852df4d8af906e3577cb2f5f70d0bc6fd2da9beeb97
-  data.tar.gz: 4b0ab5e4d529f54e4890c8e0da2763ee8915a393ce1260da5c7ce70d6ef940c64e36f15d5a5b70cca255ec881c53b4c48bbcd5f95f2e7d83cc88e233b623b327
+  metadata.gz: c15b1d0904dc34e2cbc095628b5d642213c0d33b6387e52b38660cf78b6552bc32818dca5ecbc5de710502139c7308fffb67a0a2aa3049749833c5e64a6757b5
+  data.tar.gz: 5656754554aa3bb4acc7ee333b72f21ea848d3e3cc8c06acf9e16d3529b427acfec9e00b3bac3ae67536b39608640146bcaab1459db77a7c6c46e6545e72dbae

data/README.md

@@ -149,4 +149,4 @@ end
 After you clone the repository, cd'ing into the directory should create a new gemset if you are using RVM.
 Then run `bin/setup` to install the appropriate gems and set things up.
 
-The top-level `bin/rails` proxies all Rails commands to the test project, so you can operate it via the usual commands (e.g., `rails test`, `rails server` and `rails console`).
+The top-level `bin/rails` proxies all Rails commands to the test project, so you can operate it via the usual commands (e.g., `rails test`, `rails server` and `rails console`). For development, use `foreman start` to run the web server and the job queue.

data/VERSION

@@ -1 +1 @@
-0.9.11
+0.9.14

data/app/views/rest_framework/routes_and_forms/_html_form.html.erb

@@ -10,13 +10,14 @@
     </label>
   </div>
 
-  <%= form_with(
+  <%= form_with(**{
     model: @record,
     url: "",
     method: :patch,
     id: "htmlForm",
     scope: "",
-  ) do |form| %>
+    local: true,
+  }.compact) do |form| %>
     <% controller.get_fields.map(&:to_s).each do |f| %>
       <%
         # Don't provide form fields for associations or primary keys.

data/app/views/rest_framework/routes_and_forms/_raw_form.html.erb

@@ -32,12 +32,13 @@
   <% if @is_model_controller && model = controller.class.get_model %>
     <% if attachment_reflections = model.attachment_reflections.presence %>
       <div class="mb-2" style="display: none" id="rawFilesFormWrapper">
-        <%= form_with(
+        <%= form_with(**{
           model: @record,
           url: "",
           id: "rawFilesForm",
           scope: "",
-        ) do |form| %>
+          local: true,
+        }.compact) do |form| %>
           <% attachment_reflections.each do |field, ref| %>
             <label class="form-label w-100"><%= controller.class.get_label(field) %>
               <%= form.file_field field, multiple: ref.macro == :has_many_attached %>

data/lib/rest_framework/filters/model_ordering_filter.rb

@@ -16,7 +16,8 @@ class RESTFramework::Filters::ModelOrderingFilter < RESTFramework::Filters::Base
     if order_string.present?
       ordering = {}.with_indifferent_access
 
-      order_string.split(",").each do |field|
+      order_string = order_string.join(",") if order_string.is_a?(Array)
+      order_string.split(",").map(&:strip).each do |field|
         if field[0] == "-"
           column = field[1..-1]
           direction = :desc

data/lib/rest_framework/mixins/model_controller_mixin.rb

@@ -22,8 +22,6 @@ module RESTFramework::Mixins::BaseModelControllerMixin
 
     # Options for what should be included/excluded from default fields.
     exclude_associations: false,
-    include_active_storage: false,
-    include_action_text: false,
   }
   RRF_BASE_MODEL_INSTANCE_CONFIG = {
     # Attributes for finding records.

data/lib/rest_framework/serializers/native_serializer.rb

@@ -221,6 +221,14 @@ class RESTFramework::Serializers::NativeSerializer < RESTFramework::Serializers:
             self.define_singleton_method(f) do |record|
               next record.send(f).limit(limit).as_json(**sub_config)
             end
+
+            # Disable this for now, as it's not clear if this improves performance of count.
+            #
+            # # Even though we use a serializer method, if the count will later be added, then put
+            # # this field into the includes_map.
+            # if @controller.native_serializer_include_associations_count
+            #   includes_map[f] = f.to_sym
+            # end
           else
             includes[f] = sub_config
             includes_map[f] = f.to_sym

data/lib/rest_framework/utils.rb

@@ -15,13 +15,18 @@ module RESTFramework::Utils
         # Symbolize keys (which also makes a copy so we don't mutate the original).
         v = v.symbolize_keys
         methods = v.delete(:methods)
+        if v.key?(:method)
+          methods = v.delete(:method)
+        end
 
         # First, remove the route metadata.
         metadata = v.delete(:metadata) || {}
 
         # Add label to fields.
         if controller && metadata[:fields]
-          metadata[:fields] = metadata[:fields].map { |f| [f, {}] }.to_h if f.is_a?(Array)
+          metadata[:fields] = metadata[:fields].map { |f|
+            [f, {}]
+          }.to_h if metadata[:fields].is_a?(Array)
           metadata[:fields]&.each do |field, cfg|
             cfg[:label] = controller.get_label(field) unless cfg[:label]
           end
@@ -99,12 +104,10 @@ module RESTFramework::Utils
     return route_props, application_routes.routes.select { |r|
       # We `select` first to avoid unnecessarily calculating metadata for routes we don't even want
       # to show.
-      (
-        (r.defaults[:subdomain].blank? || r.defaults[:subdomain] == request.subdomain) &&
+      (r.defaults[:subdomain].blank? || r.defaults[:subdomain] == request.subdomain) &&
         current_comparable_path.match?(self.comparable_path(r.path.spec.to_s)) &&
         r.defaults[:controller].present? &&
         r.defaults[:action].present?
-      )
     }.map { |r|
       path = r.path.spec.to_s.gsub("(.:format)", "")
       levels = path.count("/")
@@ -171,15 +174,12 @@ module RESTFramework::Utils
   # strings, not symbols.
   def self.fields_for(model, exclude_associations: nil)
     foreign_keys = model.reflect_on_all_associations(:belongs_to).map(&:foreign_key)
+    base_fields = model.column_names.reject { |c| c.in?(foreign_keys) }
 
-    if exclude_associations
-      return model.column_names.reject { |c| c.in?(foreign_keys) }
-    end
+    return base_fields if exclude_associations
 
     # Add associations in addition to normal columns.
-    return model.column_names.reject { |c|
-      c.in?(foreign_keys)
-    } + model.reflections.map { |association, ref|
+    return base_fields + model.reflections.map { |association, ref|
       # Ignore associations for which we have custom integrations.
       if ref.class_name.in?(%w(ActiveStorage::Attachment ActiveStorage::Blob ActionText::RichText))
         next nil

data/lib/rest_framework.rb

@@ -149,7 +149,7 @@ module RESTFramework
     # Freeze configuration attributes during finalization to prevent accidental mutation.
     attr_accessor :freeze_config
 
-    # Specify reverse association tables that are typically very large, andd therefore should not be
+    # Specify reverse association tables that are typically very large, and therefore should not be
     # added to fields by default.
     attr_accessor :large_reverse_association_tables
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rest_framework
 version: !ruby/object:Gem::Version
-  version: 0.9.11
+  version: 0.9.14
 platform: ruby
 authors:
 - Gregory N. Schmit
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-08-16 00:00:00.000000000 Z
+date: 2024-09-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails