tzispa 0.4.15 → 0.4.16

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1cb6d1509d41c50ba037726c1873f83f1a8b3ab2
-  data.tar.gz: c31027f97f9e020a1374fe41701fa765df5718b1
+  metadata.gz: 210b3ade3e73dabae1b524cb527eafe85b4d188d
+  data.tar.gz: f79ec50b9f89adc27c78cd6a8866eaa5dddc2966
 SHA512:
-  metadata.gz: 59623572733062ea7610fa746f39a5221112ee8eb09726a626cef6f349a90aa7172b0e7c354d4157d4febaf28978df6257e3ddffdb811e404d3713c5f36ec9c6
-  data.tar.gz: 4db3438a11d81744dcfcb2b322e44a245ae254e158160e175ba30c2376686cfa61f6a72fd57d5b6cfb08e73e7ea8b38eba589476d91bb8fc122fac02316f7848
+  metadata.gz: d466ff27221ebba0b9e7f39e890ab22240586f6008a18913366e1d14dcfb66e3ccb825cd9c3c2689b538f535a5b202021dafe86f690c764c9347e6391dd084de
+  data.tar.gz: fd36aaa6d64efe67dd8182449f2d2e508733017007584059e6a5c7d98f4eb502ee500d4755dddd995407b323900eff67f126b0f17a305021e2bffc943b74a365

data/CHANGELOG.md

@@ -2,11 +2,14 @@ Tzispa
 
 General purpose web framework
 
+## v0.4.16
+- check response_verb before send it to the handler in controller/api
+- move routes from app-class to app-instance
+
 ## v0.4.15
 - move context creation from app to middleware
 - remove controller/layout custom headers
 
-
 ## v0.4.14
 - remove moneta session in middleware: now use session goes in App class definition
 - remove repository dup in context because isn't required without the respository.use method

data/lib/tzispa/app.rb

@@ -28,8 +28,6 @@ module Tzispa
 
     class << self
 
-      attr_accessor :routes
-
       def inherited(base)
         super
         base.class_eval do
@@ -51,20 +49,19 @@ module Tzispa
         }
       end
 
-      def mount(mount_point, builder)
+      def mount(path, builder)
         self.new.tap { |app|
-          self.routes ||= Routes.new(mount_point)
-          yield(routes)
-          app.middleware.map mount_point, builder
+          app.routes ||= Routes.new(path)
+          yield(app.routes)
+          app.middleware.load_app path, builder
         }
       end
 
-      def router
-        self.routes&.router
-      end
 
     end
 
+    attr_accessor :routes
+
     def initialize(domain_name)
       @domain = Domain.new(domain_name)
       @config = Config::AppConfig.new(@domain).load!
@@ -75,22 +72,24 @@ module Tzispa
       middleware.call(env)
     end
 
+    def router
+      routes&.router
+    end
+
     def load!
-      unless @loaded
-        Mutex.new.synchronize {
-          load_locales
-          @repository = Data::Repository.new(@config.repository.to_h).load! if @config.respond_to? :repository
-          @engine = Rig::Engine.new(self, @config.template_cache.enabled, @config.template_cache.size)
-          @logger = Logger.new("logs/#{@domain.name}.log", 'weekly')
-          @logger.level = @config.respond_to?(:developing) && @config.developing ? Logger::DEBUG : Logger::INFO
-          @domain.require_dir 'helpers'
-          @domain.require_dir 'api'
-          @domain.require_dir 'middleware'
-          @middleware.load!
-          @loaded = true
-        }
-      end
-      self
+      Mutex.new.synchronize {
+        load_locales
+        @repository = Data::Repository.new(@config.repository.to_h).load! if @config.respond_to? :repository
+        @engine = Rig::Engine.new(self, @config.template_cache.enabled, @config.template_cache.size)
+        @logger = Logger.new("logs/#{@domain.name}.log", 'weekly')
+        @logger.level = @config.respond_to?(:developing) && @config.developing ? Logger::DEBUG : Logger::INFO
+        @domain.require_dir 'helpers'
+        @domain.require_dir 'api'
+        @domain.require_dir 'middleware'
+        @middleware.load!
+        @loaded = true
+      }
+    self
     end
 
     private

data/lib/tzispa/controller/api.rb

@@ -16,6 +16,8 @@ module Tzispa
       include Tzispa::Helpers::Security
       include Tzispa::Helpers::Response
 
+      attr_reader :hnd
+
       def dispatch!
         raise Error::InvalidSign.new unless sign?
         @handler, domain_name = context.router_params[:handler].split('.').reverse
@@ -24,7 +26,7 @@ module Tzispa
         @predicate = context.router_params[:predicate]
         @hnd = handler_class.new(context)
         @predicate ? hnd.send(@verb, @predicate) : hnd.send(@verb)
-        send hnd.response_verb
+        send hnd.response_verb if hnd.response_verb && hnd.respond_to?(hnd.response_verb)
         response.finish
       end
 
@@ -112,7 +114,6 @@ module Tzispa
 
       private
 
-      attr_reader :hnd
 
       def set_action_headers
         response['X-API'] = "#{context.router_params[:sign]}:#{context.router_params[:handler]}:#{context.router_params[:verb]}:#{context.router_params[:predicate]}"

data/lib/tzispa/http/context.rb

@@ -74,7 +74,7 @@ module Tzispa
       end
 
       def path(path_id, params={})
-        app.class.routes.path path_id, params
+        app.routes.path path_id, params
       end
 
       def canonical_url(path_id, params={})

data/lib/tzispa/middleware.rb

@@ -12,13 +12,13 @@ module Tzispa
       @builder = ::Rack::Builder.new
       load_default_stack
       @stack.each { |m, args, block| @builder.use load_middleware(m), *args, &block }
-      @builder.run @application.class.router
+      @builder.run @application.router
       self
     end
 
-    def map(mount_path, builder)
+    def load_app(path, builder)
       app = @application
-      builder.map mount_path do
+      builder.map path do
         run app.load!
       end
     end

data/lib/tzispa/version.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Tzispa
-  VERSION = '0.4.15'
+  VERSION = '0.4.16'
   FRAMEWORK_NAME = 'Tzispa'
   GEM_NAME = 'tzispa'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: tzispa
 version: !ruby/object:Gem::Version
-  version: 0.4.15
+  version: 0.4.16
 platform: ruby
 authors:
 - Juan Antonio Piñero
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-06-03 00:00:00.000000000 Z
+date: 2016-06-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack