rest_framework 0.8.7 → 0.8.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 057373b968952b2e2994e96e73617371a1ee8bb1f1169baa674df23de6009d77
-  data.tar.gz: d7773ccff492ddfcd89696502d82a041aafef09d4fbaca3188b73e4faeb00d5c
+  metadata.gz: 638d5d849cf0f6a3b456c7fc6d1906263df776333bc754f38553aa45162a7be3
+  data.tar.gz: 87bb5dcd4f65d3539d60bd80103be7d4dbafbf3f90c29200dadd144743edd19f
 SHA512:
-  metadata.gz: 2e8d2da4e111a9424a439dda5b3a997947981cf5895d56790d325d3c9b8afe70b53fd57a9f550fb5978081df3fec4ab7f047afacd5de52d93fe199a2b7ff3494
-  data.tar.gz: 19b9c83fd9846aa1f445eadc36943875b4fa96ec7648ec7ffbaa07d08ff7c5a9b370ca8c4ab3ccaab17ad83a611a1105ee3fefacc635c7fc3eba398d94fe4651
+  metadata.gz: e784ebdc889cf8c89e6fce1916e001d30c3a160b7093e5ff2de4903de1b86a577794dace867353d22d63d7caa4992e4e2ae95c1e4daf515395303eb032e84500
+  data.tar.gz: e1eb1f492d928d9c062676f72621b753fbdbf1fbe22ca3b04c920808ed51b82f1dd96859852fcce0c202f7d3276079ccc96524cbfcf76e897aff030563f16be9

data/VERSION

@@ -1 +1 @@
-0.8.7
+0.8.9

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

@@ -1,6 +1,6 @@
 <tr>
   <td>
-    <% if route[:route].name && route[:verb] == "GET" && route[:matches_params] %>
+    <% if route[:verb] == "GET" && route[:matches_params] %>
       <%= link_to route[:relative_path], @route_props[:with_path_args].call(route[:route]) %>
     <% else %>
       <%= route[:relative_path] %>

data/lib/rest_framework/controller_mixins/models.rb

@@ -71,7 +71,7 @@ module RESTFramework::BaseModelControllerMixin
     IGNORE_VALIDATORS_WITH_KEYS = [:if, :unless].freeze
 
     # Get the model for this controller.
-    def get_model(from_get_recordset: false)
+    def get_model
       return @model if @model
       return (@model = self.model) if self.model
 
@@ -81,19 +81,7 @@ module RESTFramework::BaseModelControllerMixin
       rescue NameError
       end
 
-      # Delegate to the recordset's model, if it's defined. This option prevents infinite recursion.
-      unless from_get_recordset
-        # Instantiate a new controller to get the recordset.
-        controller = self.new
-        controller.request = ActionController::TestRequest.new
-        controller.params = {}
-
-        if (recordset = controller.get_recordset)
-          return @model = recordset.klass
-        end
-      end
-
-      return nil
+      raise RESTFramework::UnknownModelError, self
     end
 
     # Override `get_label` to include ActiveRecord i18n-translated column names.
@@ -488,7 +476,7 @@ module RESTFramework::BaseModelControllerMixin
     return (@recordset = self.class.recordset) if self.class.recordset
 
     # If there is a model, return that model's default scope (all records by default).
-    if (model = self.class.get_model(from_get_recordset: true))
+    if (model = self.class.get_model)
       return @recordset = model.all
     end
 

data/lib/rest_framework/errors.rb

@@ -13,3 +13,19 @@ class RESTFramework::NilPassedToAPIResponseError < RESTFramework::Error
     MSG
   end
 end
+
+class RESTFramework::UnknownModelError < RESTFramework::Error
+  def initialize(controller_class)
+    super()
+    @controller_class = controller_class
+  end
+
+  def message
+    return <<~MSG.split("\n").join(" ")
+      The model class for `#{@controller_class}` could not be determined. Any controller that
+      includes `RESTFramework::BaseModelControllerMixin` (directly or indirectly) must either set
+      the `model` attribute on the controller, or the model must be deducible from the controller
+      name (e.g., `UsersController` could resolve to the `User` model).
+    MSG
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rest_framework
 version: !ruby/object:Gem::Version
-  version: 0.8.7
+  version: 0.8.9
 platform: ruby
 authors:
 - Gregory N. Schmit
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-02-14 00:00:00.000000000 Z
+date: 2023-02-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails