rest_framework 0.0.3 → 0.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 573511c8fd193f39ad79529ae7af61b890536bc1ba8d033a1b7e1d9ee8825ed7
-  data.tar.gz: a46c43aaf66552a5fe5ff4f50d214dda32ce7777e1994fbe5d0fdcf8018b279f
+  metadata.gz: 6d0230ba5202225c71130921355680792e1907f593fa5c664d1d1bc862076d18
+  data.tar.gz: 65fe4ade26bdeadc825261e568de4947ef6440e07a2783241eecaec9147dcf16
 SHA512:
-  metadata.gz: 626f03645ce25dbd70fd51c19750edd846e864ac16037ba07f9e93d9525c672f14a1c5b2b8a49db24242117e538d87ce33363a7e0649a9dc3e49eedd4dfd1f69
-  data.tar.gz: ed442d6cf44fff1e36e211c69279372efd5eebf4a74d1ab4721ef0bc6f10bb5a81a7252b857de92224c547848e9236aa9b56e28b33d6589748ce938f4b143e47
+  metadata.gz: d7b880340dd0b3b784e366abae39286632d3d36e784a792f76e74181e6a39a364cbcdb6d206893ae759f249f3cec559fe741c413b95ed741f359700ffdc4652a
+  data.tar.gz: 904b8f2cd4390e7d912b63795e09c5cdbbfba7972fdcf6131c680e73a3d214eb88569ade00ce87da4ef9fa5a5b505877a5edd1c35c26f7431cb69c504381a269

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

@@ -2,7 +2,6 @@
 <meta name="viewport" content="width=device-width, initial-scale=1, shrink-to-fit=no">
 <%= csrf_meta_tags %>
 <%= csp_meta_tag rescue nil %>
-<%= favicon_link_tag 'rest_framework_favicon.ico' rescue nil %>
 
 <link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/4.0.0/css/bootstrap.min.css" integrity="sha384-Gn5384xqQ1aoWXA+058RXPxPg6fy4IWvTNh0E263XmFcJlSAwiGgFAW/dAiS6JXm" crossorigin="anonymous">
 <style>

data/lib/rest_framework/VERSION_STAMP

@@ -1 +1 @@
-0.0.3
+0.0.5

data/lib/rest_framework/controllers/base.rb

@@ -2,7 +2,6 @@ module RESTFramework
 
   # This module provides helpers for mixin `ClassMethods` submodules.
   module ClassMethodHelpers
-
     # This helper assists in providing reader interfaces for mixin properties.
     def _restframework_attr_reader(property, default: nil)
       method = <<~RUBY
@@ -31,26 +30,22 @@ module RESTFramework
     module ClassMethods
       extend ClassMethodHelpers
 
-      # Interface for getting class-level instance/class variables.
+      # Interface for getting class-level instance/class variables. Note: we check if they are
+      # defined first rather than rescuing NameError to prevent uninitialized variable warnings.
       private def _restframework_try_class_level_variable_get(name, default: nil)
-        begin
-          v = instance_variable_get("@#{name}")
+        instance_variable = "@#{name}"
+        if self.instance_variable_defined?(instance_variable)
+          v = self.instance_variable_get(instance_variable)
           return v unless v.nil?
-        rescue NameError
         end
-        begin
-          v = class_variable_get("@@#{name}")
+        class_variable = "@@#{name}"
+        if self.class_variable_defined?(class_variable)
+          v = self.class_variable_get(class_variable)
           return v unless v.nil?
-        rescue NameError
         end
         return default
       end
 
-      # Interface for registering exceptions handlers.
-      # private def _restframework_register_exception_handlers
-      #   rescue_from
-      # end
-
       _restframework_attr_reader(:singleton_controller)
       _restframework_attr_reader(:extra_actions, default: {})
       _restframework_attr_reader(:template_logo_text, default: 'Rails REST Framework')
@@ -94,7 +89,7 @@ module RESTFramework
 
       # serialize
       if self.respond_to?(:get_model_serializer_config, true)
-        serialized_payload = payload.to_json(**self.get_model_serializer_config)
+        serialized_payload = payload.to_json(self.get_model_serializer_config)
       else
         serialized_payload = payload.to_json
       end

data/lib/rest_framework/controllers/models.rb

@@ -50,7 +50,7 @@ module RESTFramework
 
     # Get a list of fields for the current action.
     def get_fields
-      return @fields if @fields
+      return @fields if instance_variable_defined?(:@fields) && @fields
 
       # index action should use list_fields
       name = (action_name == 'index') ? 'list' : action_name
@@ -59,9 +59,8 @@ module RESTFramework
         @fields = self.class.send("#{name}_fields")
       rescue NameError
       end
-      @fields ||= self.class.fields || []
 
-      return @fields
+      return @fields ||= self.class.fields || []
     end
 
     # Get a configuration passable to `as_json` for the model.
@@ -112,7 +111,7 @@ module RESTFramework
 
     # Internal interface for get_model, protecting against infinite recursion with get_recordset.
     def _get_model(from_internal_get_recordset: false)
-      return @model if @model
+      return @model if instance_variable_defined?(:@model) && @model
       return self.class.model if self.class.model
       unless from_internal_get_recordset  # prevent infinite recursion
         recordset = self._get_recordset(from_internal_get_model: true)
@@ -127,7 +126,7 @@ module RESTFramework
 
     # Internal interface for get_recordset, protecting against infinite recursion with get_model.
     def _get_recordset(from_internal_get_model: false)
-      return @recordset if @recordset
+      return @recordset if instance_variable_defined?(:@recordset) && @recordset
       return self.class.recordset if self.class.recordset
       unless from_internal_get_model  # prevent infinite recursion
         model = self._get_model(from_internal_get_recordset: true)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rest_framework
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.5
 platform: ruby
 authors:
 - Gregory N. Schmit
@@ -33,8 +33,6 @@ extra_rdoc_files: []
 files:
 - LICENSE
 - README.md
-- app/assets/images/rest_framework_favicon.ico
-- app/assets/stylesheets/rest_framework.css
 - app/views/layouts/rest_framework.html.erb
 - app/views/rest_framework/_head.html.erb
 - app/views/rest_framework/_routes.html.erb

data/app/assets/stylesheets/rest_framework.css

@@ -1 +0,0 @@
-h1, h2, h3, h4, h5, h6 { width: 100%; }