rest_framework 0.2.3 → 0.2.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '09c24135bd79001e0113aa92f21e63c5dcb67e93891db2a5df36b4d83d1c5da2'
-  data.tar.gz: ba3bd5829fdb9fd3c61a6050f5800c91a0077c76e78aedb049f850e581027f67
+  metadata.gz: 8d9ce4741f59e6f68bfa18bc2e6c11202c91fbe251eeaa5562f8f718e6e8afaa
+  data.tar.gz: 652ad42a8d0bad99e552af9117112df707e67d4c1e6dd2dd983e7925933fcab6
 SHA512:
-  metadata.gz: c89c8af64c2eeb36dba79747770df5497d8505f2a4ced03ed6e25e81804a3a8a8f656e87bb0739a31d26be8e1f0fdcdd9e3df25721766e2e5d4bb2a01d40746f
-  data.tar.gz: 4bd5bdf9f2735622a07795245f605137c0310d16b3596fc544e9ade2046744a499ea0fdf1e7f11c7f5fc49f137650cceefadcf805884e6a6df5361ed919211d4
+  metadata.gz: 26036468d40ccc77a203981ba442265ef82c7a991fee800ee3d7925d96d09a696f3db06930e8252c49cf63e86d2aee2fd121a735e5e06609585299ff984d6992
+  data.tar.gz: 3c53d8eed996e380ab3390acfde7a3cef262d38a1355d0016aa6d8129b8e329c99003c0181ec1602f5bb1ba8869355e6b1eb0ff366d8b814ff83f6bb3c9cbf4d

data/README.md

@@ -3,6 +3,7 @@
 [![Gem Version](https://badge.fury.io/rb/rest_framework.svg)](https://badge.fury.io/rb/rest_framework)
 [![Build Status](https://travis-ci.org/gregschmit/rails-rest-framework.svg?branch=master)](https://travis-ci.org/gregschmit/rails-rest-framework)
 [![Coverage Status](https://coveralls.io/repos/github/gregschmit/rails-rest-framework/badge.svg?branch=master)](https://coveralls.io/github/gregschmit/rails-rest-framework?branch=master)
+[![Maintainability](https://api.codeclimate.com/v1/badges/ba5df7706cb544d78555/maintainability)](https://codeclimate.com/github/gregschmit/rails-rest-framework/maintainability)
 
 A framework for DRY RESTful APIs in Ruby on Rails.
 

data/lib/rest_framework/VERSION_STAMP

@@ -1 +1 @@
-0.2.3
+0.2.4

data/lib/rest_framework/routers.rb

@@ -6,7 +6,7 @@ module ActionDispatch::Routing
     # Internal helper to take extra_actions hash and convert to a consistent format.
     protected def _parse_extra_actions(extra_actions)
       return (extra_actions || {}).map do |k,v|
-        kwargs = {}
+        kwargs = {action: k}
         path = k
 
         # Convert structure to path/methods/kwargs.
@@ -25,11 +25,6 @@ module ActionDispatch::Routing
             path = v.delete(:path)
           end
 
-          # Set the action to be the action key unless it's already defined.
-          if !kwargs[:action]
-            kwargs[:action] = k
-          end
-
           # Pass any further kwargs to the underlying Rails interface.
           kwargs = kwargs.merge(v)
         elsif v.is_a?(Symbol) || v.is_a?(String)
@@ -77,6 +72,16 @@ module ActionDispatch::Routing
       return controller
     end
 
+    # Interal interface for routing extra actions.
+    protected def _route_extra_actions(actions, &block)
+      actions.each do |action_config|
+        action_config[:methods].each do |m|
+          public_send(m, action_config[:path], **action_config[:kwargs])
+        end
+        yield if block_given?
+      end
+    end
+
     # Internal core implementation of the `rest_resource(s)` router, both singular and plural.
     # @param default_singular [Boolean] the default plurality of the resource if the plurality is
     #   not otherwise defined by the controller
@@ -105,28 +110,20 @@ module ActionDispatch::Routing
       end
       resource_method = singular ? :resource : :resources
 
-      # call either `resource` or `resources`, passing appropriate modifiers
+      # Call either `resource` or `resources`, passing appropriate modifiers.
       skip_undefined = kwargs.delete(:skip_undefined) || true
       skip = controller_class.get_skip_actions(skip_undefined: skip_undefined)
       public_send(resource_method, name, except: skip, **kwargs) do
         if controller_class.respond_to?(:extra_member_actions)
           member do
             actions = self._parse_extra_actions(controller_class.extra_member_actions)
-            actions.each do |action_config|
-              action_config[:methods].each do |m|
-                public_send(m, action_config[:path], **action_config[:kwargs])
-              end
-            end
+            _route_extra_actions(actions)
           end
         end
 
         collection do
           actions = self._parse_extra_actions(controller_class.extra_actions)
-          actions.each do |action_config|
-            action_config[:methods].each do |m|
-              public_send(m, action_config[:path], **action_config[:kwargs])
-            end
-          end
+          _route_extra_actions(actions)
         end
 
         yield if block_given?
@@ -150,6 +147,7 @@ module ActionDispatch::Routing
     # Route a controller without the default resourceful paths.
     def rest_route(name=nil, **kwargs, &block)
       controller = kwargs.delete(:controller) || name
+      route_root_to = kwargs.delete(:route_root_to)
       if controller.is_a?(Class)
         controller_class = controller
       else
@@ -162,42 +160,27 @@ module ActionDispatch::Routing
       # Route actions using the resourceful router, but skip all builtin actions.
       actions = self._parse_extra_actions(controller_class.extra_actions)
       public_send(:resource, name, only: [], **kwargs) do
-        actions.each do |action_config|
-          action_config[:methods].each do |m|
-            public_send(m, action_config[:path], **action_config[:kwargs])
-          end
-          yield if block_given?
+        # Route a root for this resource.
+        if route_root_to
+          get '', action: route_root_to
         end
+
+        _route_extra_actions(actions, &block)
       end
     end
 
     # Route a controller's `#root` to '/' in the current scope/namespace, along with other actions.
-    # @param name [Symbol] the snake_case name of the controller
     def rest_root(name=nil, **kwargs, &block)
       # By default, use RootController#root.
       root_action = kwargs.delete(:action) || :root
       controller = kwargs.delete(:controller) || name || :root
 
-      # Route the root.
-      get name.to_s, controller: controller, action: root_action
-
-      # Route any additional actions.
-      controller_class = self._get_controller_class(controller, pluralize: false)
-      actions = self._parse_extra_actions(controller_class.extra_actions)
-      actions.each do |action_config|
-        # Add :action unless kwargs defines it.
-        unless action_config[:kwargs].key?(:action)
-          action_config[:kwargs][:action] = action_config[:path]
-        end
+      # Remove path if name is nil (routing to the root of current namespace).
+      unless name
+        kwargs[:path] = ''
+      end
 
-        action_config[:methods].each do |m|
-          public_send(
-            m,
-            File.join(name.to_s, action_config[:path].to_s),
-            controller: controller,
-            **action_config[:kwargs],
-          )
-        end
+      return rest_route(controller, route_root_to: root_action, **kwargs) do
         yield if block_given?
       end
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rest_framework
 version: !ruby/object:Gem::Version
-  version: 0.2.3
+  version: 0.2.4
 platform: ruby
 authors:
 - Gregory N. Schmit