nyny 3.2.1 → 3.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c200e1b2ab1e25cdb91af8dddd999ec3af0d254f
-  data.tar.gz: d42091221d9fab3e6f6a5999ac9b43cc2758fa95
+  metadata.gz: 64027ca973fda52e94edacef4d3570f8e78a6e8c
+  data.tar.gz: 47763de06fb5ffb12babdcc07d7ad0ab61ff86a0
 SHA512:
-  metadata.gz: be4f674778d164ec515061a2ef5f2ec2e855999fa908113b51d2b4a28e4478055764757c50b1ebf939dc1af8e38917e65cceefcacc9aedbad4054de9157a6c77
-  data.tar.gz: 3f9750100ec7671af67eebb2bf08ae834ddda3ab6853314a56f07e1db2fc4fc52685780b884cfaca0c0ce5a3a409f0927d7d2044de3e230b4dfe9f477f847b51
+  metadata.gz: 72a4aaad4c7473ec7b680ff1f52cf490084cf88d969a960f976bf7d79b854e2dc19ff63fadf968af9fd0a94d54046bedfb430e1b368e61c3621502bb29ec80eb
+  data.tar.gz: 3a81393aa445660e1105fecbc2002f8802c27c571d239621cb8a9bdc6041c06e5ad5037922a2a01232a4b6a83aea50de7ed410364bf04f189369e8e220822526

data/CHANGELOG

@@ -1,3 +1,9 @@
+3.2.2
+  - Use Journey directly from ActionDispatch, since rails/journey is obsolete
+  - Fix indifferent nested params
+  - Make helpers accessible inside namespaces
+  - Fix namespaces inheritance behaviour
+
 3.2.1
   - Fallback to Tilt 1.4.1 since 2.0.0 has conflicts with a lot
     of gems

data/lib/nyny/app.rb

@@ -66,8 +66,15 @@ module NYNY
       end
 
       def namespace url, &block
-        app = Class.new(NYNY::App, &block)
-        builder.map (url) { use app }
+        scope  = self.scope_class
+        parent = self.superclass
+
+        klass = Class.new parent do
+          self.scope_class = scope
+          class_eval(&block)
+        end
+
+        builder.map (url) { use klass }
       end
 
       def before &blk

data/lib/nyny/primitives.rb

@@ -1,5 +1,10 @@
+require 'active_support/hash_with_indifferent_access'
+
 module NYNY
   class Request < Rack::Request
+    def params
+      @params ||= ActiveSupport::HashWithIndifferentAccess.new(super)
+    end
   end
 
   class Response < Rack::Response

data/lib/nyny/request_scope.rb

@@ -6,11 +6,12 @@ module NYNY
     extend Forwardable
 
     attr_reader :request, :response
-    def_delegators :request, :params, :session
+    def_delegators :request, :session, :params
     def_delegators :response, :headers
 
-    def initialize request
-      @request  = request
+    def initialize env
+      @request  = Request.new(env)
+      @request.params.merge! env['nyny.params']
       @response = Response.new [], 200, {'Content-Type' => 'text/html'}
     end
 

data/lib/nyny/router.rb

@@ -1,4 +1,6 @@
-require 'journey'
+require 'active_support/concern'
+require 'action_dispatch/routing'
+require 'action_dispatch/journey'
 
 module NYNY
   class Router
@@ -25,12 +27,13 @@ module NYNY
     private
 
     def prepare_for_journey route_defs
-      @journey = Journey::Router.new(Journey::Routes.new, {
+      routes = ActionDispatch::Journey::Routes.new
+      @journey = ActionDispatch::Journey::Router.new(routes, {
         :parameters_key => 'nyny.params'
       })
 
       route_defs.each do |path, options, handler|
-        pat         = Journey::Path::Pattern.new(path)
+        pat         = ActionDispatch::Journey::Path::Pattern.new(path)
         constraints = options.fetch(:constraints, {})
         defaults    = options.fetch(:defaults, {})
 
@@ -40,13 +43,7 @@ module NYNY
 
     def compile handler
       Proc.new do |env|
-        request = Request.new(env)
-        request.params.merge! env["nyny.params"]
-        request.params.default_proc = lambda do |h, k|
-          h.fetch(k.to_s, nil) || h.fetch(k.to_sym, nil)
-        end
-
-        scope = scope_class.new(request)
+        scope = scope_class.new(env)
 
         response = catch (:halt) do
           before_hooks.each {|h| scope.instance_eval &h }

data/lib/nyny/version.rb

@@ -1,3 +1,3 @@
 module NYNY
-  VERSION = "3.2.1"
+  VERSION = "3.2.2"
 end

data/nyny.gemspec

@@ -4,8 +4,6 @@ $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'nyny/version'
 
 Gem::Specification.new do |spec|
-  spec.required_ruby_version = ">= 1.9.2"
-
   spec.name          = "nyny"
   spec.version       = NYNY::VERSION
   spec.authors       = ["Andrei Lisnic"]
@@ -20,11 +18,10 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ["lib"]
 
-  spec.add_dependency             "rack",           "~> 1.5.2"
   spec.add_dependency             "rack-contrib",   "~> 1.1.0"
   spec.add_dependency             "tilt",           "~> 1.4.1"
   spec.add_dependency             "better_errors",  "~> 1.1.0"
-  spec.add_dependency             "journey",        "~> 1.0.4"
+  spec.add_dependency             "actionpack",     "~> 4.0.2"
   spec.add_development_dependency "bundler", "~> 1.3"
   spec.add_development_dependency "rake"
   spec.add_development_dependency "rspec"

data/spec/app_spec.rb

@@ -27,8 +27,15 @@ describe App do
   describe 'namespace' do
     let (:app) do
       mock_app do
+        helpers do
+          def le_helper
+            :lulwut
+          end
+        end
+
         namespace '/foo' do
           get '/' do
+            le_helper.should == :lulwut
             'bar'
           end
         end
@@ -36,6 +43,7 @@ describe App do
         namespace '/nested' do
           namespace '/space' do
             get '/' do
+              le_helper.should == :lulwut
               'caramba'
             end
           end

data/spec/inheritance_spec.rb

@@ -37,6 +37,12 @@ describe NYNY::App do
         headers['child after'] = 'true'
       end
 
+      namespace '/nested' do
+        get '/' do
+          parent_helper.should == :parent
+        end
+      end
+
       get '/helpers' do
         parent_helper.should == :parent
         child_helper.should == :child
@@ -73,5 +79,10 @@ describe NYNY::App do
       parent.get('/helpers')
       child.get('/helpers')
     end
+
+    it 'works correctly for namespaces' do
+      child.get('/nested').headers['parent before'].should_not be_nil
+      child.get('/nested').headers['parent after'].should_not be_nil
+    end
   end
 end

data/spec/request_scope_spec.rb

@@ -2,8 +2,8 @@ require 'spec_helper'
 
 describe RequestScope do
   let (:env) { Rack::MockRequest.env_for '/', :params => {:some => 'param'} }
-  let (:dummy_request) { Rack::Request.new(env) }
-  let (:subject) { RequestScope.new dummy_request }
+  let (:subject) { RequestScope.new env.merge('nyny.params' => {}) }
+  let (:dummy_request) { subject.request }
   let (:handler) {
     Proc.new {"hello"}
   }
@@ -16,11 +16,11 @@ describe RequestScope do
     it 'params should have insensitive keys' do
       app = mock_app do
         get '/' do
-          params[:foo].should == params['foo']
+          params[:foo][:bar].should == params['foo']['bar']
         end
       end
 
-      app.get '/?foo=bar'
+      app.get '/', :params => {:foo => {:bar => 'baz'}}
     end
 
     describe 'cookies' do

metadata

@@ -1,29 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: nyny
 version: !ruby/object:Gem::Version
-  version: 3.2.1
+  version: 3.2.2
 platform: ruby
 authors:
 - Andrei Lisnic
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-01-24 00:00:00.000000000 Z
+date: 2014-01-28 00:00:00.000000000 Z
 dependencies:
-- !ruby/object:Gem::Dependency
-  name: rack
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 1.5.2
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 1.5.2
 - !ruby/object:Gem::Dependency
   name: rack-contrib
   requirement: !ruby/object:Gem::Requirement
@@ -67,19 +53,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 1.1.0
 - !ruby/object:Gem::Dependency
-  name: journey
+  name: actionpack
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.0.4
+        version: 4.0.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.0.4
+        version: 4.0.2
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -171,7 +157,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 1.9.2
+      version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="