rest_framework 0.8.13 → 0.8.15

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1e6db6727ad85819540fc9d5977205f553523fe2f1f4431c985193ffd47f27bc
-  data.tar.gz: cddc4ff0d75aa6d2856d1c6bde681d38f955675a9c11b3853eef617e1abfad8b
+  metadata.gz: 36036d097655858aedddc79945d798ae9e8c8dec48ee1e8c59e06e98b9bdf694
+  data.tar.gz: fc9ced23a919bd6c0528f1b91999e89ca959ce433c0ef99938a46abc46574ca9
 SHA512:
-  metadata.gz: 86384e43ffa97406feb6a9760cb9ca425cbc725cb41324fe21a688216cf075ccdf2e27b682231dd11d4d7507740a8645eb66c4076df165d69da4a56d87d11efd
-  data.tar.gz: 7b22d9730b039452065db1e27d580ee1db4105861f323bd2800906a8c5197ed4d8cf76bb532c193dd3ab3255d491faeb04cbf47a677acbb1dbb46bfc0b6f2fc3
+  metadata.gz: 3dea10ba8f4d5cc9cc4fe186b1101823eb7268fe3543ce4dfd850d33d477cb3fe7ba413138ab0b3ade2490e4a3b5bb1bc8880d8ad41296ada615e0e732618ea5
+  data.tar.gz: 3071fc01e29b932141f70301dae07de686c691416863906b09815d89f7cc6c50e1c4351582a94a5ec1fd67fecdd028808bf2f4efd3e57113790cc9f9f2baf8fc

data/VERSION

@@ -1 +1 @@
-0.8.13
+0.8.15

data/lib/rest_framework/controller_mixins/base.rb

@@ -60,7 +60,10 @@ module RESTFramework::BaseControllerMixin
 
     # Get a label from a field/column name, titleized and inflected.
     def get_label(s)
-      return RESTFramework::Utils.inflect(s.titleize(keep_id_suffix: true), self.inflect_acronyms)
+      return RESTFramework::Utils.inflect(
+        s.to_s.titleize(keep_id_suffix: true),
+        self.inflect_acronyms,
+      )
     end
 
     # Collect actions (including extra actions) metadata for this controller.
@@ -71,16 +74,20 @@ module RESTFramework::BaseControllerMixin
       RESTFramework::BUILTIN_ACTIONS.merge(
         RESTFramework::RRF_BUILTIN_ACTIONS,
       ).each do |action, methods|
-        if self.method_defined?(action)
-          actions[action] = {path: "", methods: methods, type: :builtin}
-        end
+        next unless self.method_defined?(action)
+
+        actions[action] = {
+          path: "", methods: methods, type: :builtin, metadata: {label: self.get_label(action)}
+        }
       end
 
       # Add builtin bulk actions.
       RESTFramework::RRF_BUILTIN_BULK_ACTIONS.each do |action, methods|
-        if self.method_defined?(action)
-          actions[action] = {path: "", methods: methods, type: :builtin}
-        end
+        next unless self.method_defined?(action)
+
+        actions[action] = {
+          path: "", methods: methods, type: :builtin, metadata: {label: self.get_label(action)}
+        }
       end
 
       # Add extra actions.
@@ -97,9 +104,11 @@ module RESTFramework::BaseControllerMixin
 
       # Start with builtin actions.
       RESTFramework::BUILTIN_MEMBER_ACTIONS.each do |action, methods|
-        if self.method_defined?(action)
-          actions[action] = {path: "", methods: methods, type: :builtin}
-        end
+        next unless self.method_defined?(action)
+
+        actions[action] = {
+          path: "", methods: methods, type: :builtin, metadata: {label: self.get_label(action)}
+        }
       end
 
       # Add extra actions.

data/lib/rest_framework/controller_mixins/models.rb

@@ -94,7 +94,7 @@ module RESTFramework::BaseModelControllerMixin
     # array. This should always return an array of strings, no symbols, and possibly `nil` (only if
     # `fallback` is false).
     def get_fields(input_fields: nil, fallback: true)
-      input_fields ||= self.fields&.map(&:to_s) if fallback
+      input_fields ||= self.fields if fallback
 
       # If fields is a hash, then parse it.
       if input_fields.is_a?(Hash)
@@ -107,6 +107,8 @@ module RESTFramework::BaseModelControllerMixin
         return model ? RESTFramework::Utils.fields_for(
           model, exclude_associations: self.exclude_associations
         ) : []
+      elsif input_fields
+        input_fields = input_fields.map(&:to_s)
       end
 
       return input_fields

data/lib/rest_framework/utils.rb

@@ -1,5 +1,5 @@
 module RESTFramework::Utils
-  HTTP_METHOD_ORDERING = %w(GET POST PUT PATCH DELETE OPTIONS HEAD)
+  HTTP_VERB_ORDERING = %w(GET POST PUT PATCH DELETE OPTIONS HEAD)
 
   # Convert `extra_actions` hash to a consistent format: `{path:, methods:, kwargs:}`, and
   # additional metadata fields.
@@ -8,6 +8,7 @@ module RESTFramework::Utils
   def self.parse_extra_actions(extra_actions, controller: nil)
     return (extra_actions || {}).map { |k, v|
       path = k
+      metadata = {}
 
       # Convert structure to path/methods/kwargs.
       if v.is_a?(Hash)  # Allow kwargs to be used to define path differently from the key.
@@ -39,14 +40,19 @@ module RESTFramework::Utils
         methods = v
       end
 
+      # Insert action label if it's not provided.
+      if controller
+        metadata[:label] ||= controller.get_label(k)
+      end
+
       next [
         k,
         {
           path: path,
           methods: methods,
           kwargs: kwargs,
-          metadata: metadata.presence,
           type: :extra,
+          metadata: metadata.presence,
         }.compact,
       ]
     }.to_h
@@ -119,9 +125,15 @@ module RESTFramework::Utils
         _levels: levels,
       }
     }.sort_by { |r|
-      # Sort by levels first, so the routes matching closely with current request show first, then
-      # by the path, and finally by the HTTP verb.
-      [r[:_levels], r[:path], HTTP_METHOD_ORDERING.index(r[:verb]) || 99]
+      [
+        # Sort by levels first, so routes matching closely with current request show first.
+        r[:_levels],
+        # Then match by path, but manually sort ':' to the end using knowledge that Ruby sorts the
+        # pipe character '|' after alphanumerics.
+        r[:path].tr(":", "|"),
+        # Finally, match by HTTP verb.
+        HTTP_VERB_ORDERING.index(r[:verb]) || 99,
+      ]
     }.group_by { |r| r[:controller] }.sort_by { |c, _r|
       # Sort the controller groups by current controller first, then alphanumerically.
       [request.params[:controller] == c ? 0 : 1, c]

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rest_framework
 version: !ruby/object:Gem::Version
-  version: 0.8.13
+  version: 0.8.15
 platform: ruby
 authors:
 - Gregory N. Schmit
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-03-02 00:00:00.000000000 Z
+date: 2023-03-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -62,7 +62,7 @@ licenses:
 metadata:
   homepage_uri: https://rails-rest-framework.com
   source_code_uri: https://github.com/gregschmit/rails-rest-framework
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -79,7 +79,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.2.33
-signing_key: 
+signing_key:
 specification_version: 4
 summary: A framework for DRY RESTful APIs in Ruby on Rails.
 test_files: []