routing-filter 0.4.0.1 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 93c77279d99c3ef9273f30e201021a45c1d9fecc
-  data.tar.gz: f6aea4562e3f3d7391e95ed7f2f144367923dbfa
+  metadata.gz: 155e4e4ff03c1905b4e940cd44a6882e44476251
+  data.tar.gz: 54c1d04f984a69ac8d6ca3cd68b8f02fe1c92730
 SHA512:
-  metadata.gz: 74dda3bad3c4ce07c421b404dfc976e67bdeacecad3897b62ce7a8db706ca87189e0d88a71e90734aa40c93f96db5183e7a340ed4683154a051603f6c850ad37
-  data.tar.gz: 4f2da211024088715d179c5275d113e3cacba82508f622984b6c1898a3c3f9ef6941cdfce9a4a067674ebefa30432b3cba36707f554c64869e492359ccdc66d5
+  metadata.gz: 293cb31a89bb6eb35013b2beac05cb4698ea072454bdf5afb95cf6b8817cc2855894e1967c20cfda8ca2c3991e5c3fdb324892a8961c89eb79f01879b2548c68
+  data.tar.gz: d37b6148b8e35ada00cc2b914f3a150abb7641cd9239fc28b49b553f5c884341fba7a74af0ae926881ee1b850a4c393fec82917656a2655b4064ab466ea72492

data/CHANGELOG.md

@@ -1,3 +1,11 @@
+## 0.5.0
+
+* Rails 4.2 only support
+
+## 0.4.0
+
+* Rails 4 support
+
 ## 0.3.1
 * Filters support exclude option.
 

data/README.markdown

@@ -19,48 +19,47 @@ implement custom ones. Maybe the most popular one is the Locale routing filter:
 * `Uuid` - prepends a uuid for authentication or other purposes (e.g. /d00fbbd1-82b6-4c1a-a57d-098d529d6854/products/1)
 * `Extension` - appends an extension (e.g. /products.html)
 
-Please note that Rails 3's routing system is much more powerful and flexible
-than Rails 2 was. There are many usecases that now can be covered with just
-Rails 3 default routing features that weren't doable in Rails 2. For an example
-of a quite complex and flexible route see this [gist by Andrew White](http://gist.github.com/653543)
 
 ## Requirements
 
-routing-filter currently only works with Rails. It should not be all too hard
+Latest routing-filter (~> 0.5.0) currently only works with Rails 4.2. It should not be all too hard
 to get it working with plain Rack::Mount but I haven't had that usecase, yet.
 
+For older Rails use `0-4-stable` branch.
+
 ## Installation
 
 Just install the Gem:
 
     $ gem install routing-filter
 
-The Gem should work out of the box for Rails 3 after specifying it in your
+The Gem should work out of the box with Rails 4.2 after specifying it in your
 application's Gemfile.
 
-    # Gemfile
-    gem 'routing-filter'
-
-In order to use it with Rails 2.x you could specify it in your environment.rb
-
-    # config/environment.rb
-    gem 'routing-filter'
+```ruby
+# Gemfile
+gem 'routing-filter'
+```
 
 ## Usage
 
 Once the Gem has loaded you can setup the filters in your routes file like this:
 
-    # in config/routes.rb
-    Rails.application.routes.draw do
-      filter :pagination, :uuid
-    end
+```ruby
+# in config/routes.rb
+Rails.application.routes.draw do
+  filter :pagination, :uuid
+end
+```
 
 Filters can also accept options:
 
-    Rails.application.routes.draw do
-      filter :extension, :exclude => %r(^admin/)
-      filter :locale,    :exclude => /^\/admin/
-    end
+```ruby
+Rails.application.routes.draw do
+  filter :extension, :exclude => %r(^admin/)
+  filter :locale,    :exclude => /^\/admin/
+end
+```
 
 ### Testing
 
@@ -71,23 +70,14 @@ gets executed for these testcases.
 
 To disable RoutingFilter in your test suite add the following to your test_helper.rb / spec_helper.rb:
 
-    RoutingFilter.active = false
+```ruby
+RoutingFilter.active = false
+```
 
 ## Running the tests
 
-To run the tests against different dependencies [appraisal](https://github.com/thoughtbot/appraisal) is used.
-
-Running the tests for all supported Rails versions:
-
     $ bundle install
-    $ bundle exec rake all
-
-Running the tests for a single version, for example Rails 3.1:
-
-    $ bundle install
-    $ bundle exec rake appraisal:rails-3.1 test
-
-Valid appraisal targets include rails-2.3, rails-3.0, rails-3.1 and rails-3.2
+    $ bundle exec rake test
 
 ## Filter order
 
@@ -109,36 +99,40 @@ For example implementations have a look at the existing filters in
 
 The following would be a sceleton of an empty filter:
 
-    module RoutingFilter
-      class Awesomeness < Filter
-        def around_recognize(path, env, &block)
-          # Alter the path here before it gets recognized.
-          # Make sure to yield (calls the next around filter if present and
-          # eventually `recognize_path` on the routeset):
-          yield.tap do |params|
-            # You can additionally modify the params here before they get passed
-            # to the controller.
-          end
-        end
-
-        def around_generate(params, &block)
-          # Alter arguments here before they are passed to `url_for`.
-          # Make sure to yield (calls the next around filter if present and
-          # eventually `url_for` on the controller):
-          yield.tap do |result|
-            # You can change the generated url_or_path here. Make sure to use
-            # one of the "in-place" modifying String methods though (like sub!
-            # and friends).
-          end
-        end
+```ruby
+module RoutingFilter
+  class Awesomeness < Filter
+    def around_recognize(path, env, &block)
+      # Alter the path here before it gets recognized.
+      # Make sure to yield (calls the next around filter if present and
+      # eventually `recognize_path` on the routeset):
+      yield.tap do |params|
+        # You can additionally modify the params here before they get passed
+        # to the controller.
       end
     end
 
+    def around_generate(params, &block)
+      # Alter arguments here before they are passed to `url_for`.
+      # Make sure to yield (calls the next around filter if present and
+      # eventually `url_for` on the controller):
+      yield.tap do |result|
+        # You can change the generated url_or_path here. Make sure to use
+        # one of the "in-place" modifying String methods though (like sub!
+        # and friends).
+      end
+    end
+  end
+end
+```
+
 You can specify the filter explicitely in your routes.rb:
 
-    Rails.application.routes.draw do
-      filter :awesomeness
-    end
+```ruby
+Rails.application.routes.draw do
+  filter :awesomeness
+end
+```
 
 (I am not sure if it makes sense to provide more technical information than
 this because the usage of this plugin definitely requires some advanced

data/lib/routing_filter/adapters/{rails_4.rb → rails.rb}

@@ -22,9 +22,13 @@ ActionDispatch::Routing::RouteSet.class_eval do
     names.each { |name| filters.unshift(RoutingFilter.build(name, options)) }
   end
 
-  def generate_with_filtering(options, recall = {})
+  def generate_with_filtering(route_key, options, recall = {})
+    options = options.symbolize_keys
+
     # `around_generate` is destructive method and it breaks url. To avoid this, `dup` is required.
-    filters.run(:around_generate, options, &lambda{ generate_without_filtering(options, recall).map(&:dup) })
+    filters.run(:around_generate, options, &lambda{
+      generate_without_filtering(route_key, options, recall).map(&:dup)
+    })
   end
   alias_method_chain :generate, :filtering
 

data/lib/routing_filter/adapters/routers/journey.rb

@@ -1,20 +1,13 @@
-if defined?(ActionDispatch::Journey) # rails 4
-  journey = ActionDispatch::Journey
-else # rails 3.2
-  require 'journey/routes'
-  require 'journey/router'
-  journey = Journey
-end
-
-journey::Routes.class_eval do
+ActionDispatch::Journey::Routes.class_eval do
   def filters
     @filters || RoutingFilter::Chain.new.tap { |f| @filters = f unless frozen? }
   end
 end
 
-journey::Router.class_eval do
-  def find_routes_with_filtering env
-    path, filter_parameters = env['PATH_INFO'], {}
+ActionDispatch::Journey::Router.class_eval do
+  def find_routes_with_filtering(env)
+    path = env.is_a?(Hash) ? env['PATH_INFO'] : env.path_info
+    filter_parameters = {}
     original_path = path.dup
 
     @routes.filters.run(:around_recognize, path, env) do

data/lib/routing_filter/version.rb

@@ -1,3 +1,3 @@
 module RoutingFilter
-  VERSION = '0.4.0.1'
+  VERSION = '0.5.0'
 end

data/lib/routing_filter.rb

@@ -24,4 +24,4 @@ module RoutingFilter
   end
 end
 
-require "routing_filter/adapters/rails_#{ActionPack::VERSION::MAJOR}"
+require "routing_filter/adapters/rails"

metadata

@@ -1,43 +1,43 @@
 --- !ruby/object:Gem::Specification
 name: routing-filter
 version: !ruby/object:Gem::Version
-  version: 0.4.0.1
+  version: 0.5.0
 platform: ruby
 authors:
 - Sven Fuchs
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-02-17 00:00:00.000000000 Z
+date: 2015-02-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionpack
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "<"
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '4.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "<"
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '4.2'
 - !ruby/object:Gem::Dependency
-  name: appraisal
+  name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
+        version: '4.2'
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '4.2'
 - !ruby/object:Gem::Dependency
   name: i18n
   requirement: !ruby/object:Gem::Requirement
@@ -66,6 +66,34 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rack-test
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.6.2
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.6.2
+- !ruby/object:Gem::Dependency
+  name: rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '4.2'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '4.2'
 description: Routing filters wraps around the complex beast that the Rails routing
   system is, allowing for unseen flexibility and power in Rails URL recognition and
   generation.
@@ -80,9 +108,7 @@ files:
 - lib/routing-filter.rb
 - lib/routing/filter.rb
 - lib/routing_filter.rb
-- lib/routing_filter/adapters/rails_2.rb
-- lib/routing_filter/adapters/rails_3.rb
-- lib/routing_filter/adapters/rails_4.rb
+- lib/routing_filter/adapters/rails.rb
 - lib/routing_filter/adapters/routers/journey.rb
 - lib/routing_filter/adapters/routers/rack_mount.rb
 - lib/routing_filter/chain.rb

data/lib/routing_filter/adapters/rails_2.rb

@@ -1,69 +0,0 @@
-require 'action_controller'
-
-# allows to install a filter to the route set by calling: map.filter 'locale'
-ActionController::Routing::RouteSet::Mapper.class_eval do
-  def filter(*args)
-    @set.add_filters(*args)
-  end
-end
-
-# same here for the optimized url generation in named routes
-ActionController::Routing::RouteSet::NamedRouteCollection.class_eval do
-  # gosh. monkey engineering optimization code
-  def generate_optimisation_block_with_filtering(*args)
-    code = generate_optimisation_block_without_filtering(*args)
-    if match = code.match(%r(^return (.*) if (.*)))
-      # returned string must not contain newlines, or we'll spill out of inline code comments in
-      # ActionController::Routing::RouteSet::NamedRouteCollection#define_url_helper
-      "#{match[1]}.tap { |result|" +
-      "  ActionController::Routing::Routes.filters.run(:around_generate, *args, &lambda{ result }) " +
-      "} if #{match[2]}"
-    end
-  end
-  alias_method_chain :generate_optimisation_block, :filtering
-end
-
-ActionController::Routing::RouteSet.class_eval do
-  attr_writer :filters
-
-  def filters
-    @filters ||= RoutingFilter::Chain.new
-  end
-
-  def add_filters(*names)
-    options = names.extract_options!
-    names.each { |name| filters.unshift(RoutingFilter.build(name, options)) }
-  end
-
-  def recognize_path_with_filtering(path, env = {})
-    path = ::URI.unescape(path.dup) # string is frozen due to memoize
-    filters.run(:around_recognize, path, env, &lambda{ recognize_path_without_filtering(path, env) })
-  end
-  alias_method_chain :recognize_path, :filtering
-
-  def generate_with_filtering(*args)
-    filters.run(:around_generate, args.first, &lambda{ generate_without_filtering(*args) })
-  end
-  alias_method_chain :generate, :filtering
-
-  def clear_with_filtering!
-    @filters.clear if @filters
-    clear_without_filtering!
-  end
-  alias_method_chain :clear!, :filtering
-
-  # add some useful information to the request environment
-  # right, this is from jamis buck's excellent article about routes internals
-  # http://weblog.jamisbuck.org/2006/10/26/monkey-patching-rails-extending-routes-2
-  # TODO move this ... where?
-  alias_method :extract_request_environment_without_host, :extract_request_environment unless method_defined? :extract_request_environment_without_host
-  def extract_request_environment(request)
-    extract_request_environment_without_host(request).tap do |env|
-      env.merge! :host => request.host,
-                 :port => request.port,
-                 :host_with_port => request.host_with_port,
-                 :domain => request.domain,
-                 :subdomain => request.subdomains.first
-    end
-  end
-end

data/lib/routing_filter/adapters/rails_3.rb

@@ -1,47 +0,0 @@
-require 'action_dispatch'
-require 'active_support/core_ext/module/aliasing'
-require 'active_support/core_ext/hash/reverse_merge'
-
-mappers = [ActionDispatch::Routing::Mapper]
-mappers << ActionDispatch::Routing::DeprecatedMapper if defined?(ActionDispatch::Routing::DeprecatedMapper)
-mappers.each do |mapper|
-  mapper.class_eval do
-    def filter(*args)
-      @set.add_filters(*args)
-    end
-  end
-end
-
-ActionDispatch::Routing::RouteSet.class_eval do
-  def filters
-    @set.filters if @set
-  end
-
-  def add_filters(*names)
-    options = names.extract_options!
-    names.each { |name| filters.unshift(RoutingFilter.build(name, options)) }
-  end
-
-  # def recognize_path_with_filtering(path, env = {})
-  #   @set.filters.run(:around_recognize, path.dup, env, &lambda{ recognize_path_without_filtering(path.dup, env) })
-  # end
-  # alias_method_chain :recognize_path, :filtering
-
-  def generate_with_filtering(options, recall = {}, extras = false)
-    filters.run(:around_generate, options, &lambda{ generate_without_filtering(options, recall, extras) })
-  end
-  alias_method_chain :generate, :filtering
-
-  def clear_with_filtering!
-    filters.clear if filters
-    clear_without_filtering!
-  end
-  alias_method_chain :clear!, :filtering
-end
-
-case ActionPack::VERSION::MINOR
-when 2
-  require 'routing_filter/adapters/routers/journey'
-when 0,1
-  require 'routing_filter/adapters/routers/rack_mount'
-end