padrino-helpers 0.16.0.pre4 → 0.16.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 05e9b76c66a9aaf28ad9829d6f9abbbb4c85cfd58b554d04cf17fd22fd49f085
-  data.tar.gz: 2492033eb92a224e723700667afede77d65070ec52dd5957f310549d51133925
+  metadata.gz: 5c443286ea7c2e047d727b3b844dcf35bfe75997fde2b2310485ab1c39003c4d
+  data.tar.gz: eb4829b196a0b7c9d2a32e8aeaad09b91d7479af9e23c26bf6e2adcf4fa266dc
 SHA512:
-  metadata.gz: 6c64dcd929e68215ac8e81b33e82e846dbeab8d14d6ede875a50b41505ec65e365ff153e49f07c9733e7f82691ea949c80526ee8e43937bf3eea0b6e6eb0ce8c
-  data.tar.gz: 8baf0b35927e0b0fd9fc5da5de3ee6360266c63ed313928e0282266370d74d596e42493263e41ede78793a03efbda2ca7b9e752e3f77cda993f2f61553fa1260
+  metadata.gz: 50ba74a40763da8b07ee23455b22366d7776be020b7b96694f666f17120f67fe503ac08dda6e328baa69d7a2d64c08baea0534333d05f299901b43fe91a6a826
+  data.tar.gz: 9a6177f039e74e912974025842326f7ac96268efdc441ecc46fe3be3795d0d8a4ad098f7ebacb850a360622be644d9fd19144b3d29255a715b71d3566edd5f71

data/README.rdoc

@@ -63,13 +63,13 @@ are three major functions for this category: <tt>tag</tt>, <tt>content_tag</tt>
 The tag and content_tag are for building arbitrary html tags with a name and specified options. If
 the tag contains 'content' within then <tt>content_tag</tt> is used. For example:
 
-  tag(:br, :style => 'clear:both') => <br style="clear:both" />
-  content_tag(:p, "demo", :class => 'light') => <p class="light">demo</p>
+  tag(:br, style: 'clear:both') => <br style="clear:both" />
+  content_tag(:p, "demo", class: 'light') => <p class="light">demo</p>
 
 The input_tag is used to build tags that are related to accepting input from the user:
 
-  input_tag :text, :class => "demo" => <input type='text' class='demo' />
-  input_tag :password, :value => "secret", :class => "demo"
+  input_tag :text, class: "demo" => <input type='text' class='demo' />
+  input_tag :password, value: "secret", class: "demo"
 
 Note that all of these accept html options and result in returning a string containing html tags.
 
@@ -90,9 +90,9 @@ simple view template:
   %body
     ...
     = flash_tag :notice
-    %p= link_to 'Blog', '/blog', :class => 'example'
-    %p Mail me at #{mail_to 'fake@faker.com', "Fake Email Link", :cc => "test@demo.com"}
-    %p= image_tag 'padrino.png', :width => '35', :class => 'logo'
+    %p= link_to 'Blog', '/blog', class: 'example'
+    %p Mail me at #{mail_to 'fake@faker.com', "Fake Email Link", cc: "test@demo.com"}
+    %p= image_tag 'padrino.png', width: '35', class: 'logo'
 
 For more information on using asset helpers, check out the guide for
 {Padrino Helpers}[http://padrinorb.com/guides/application-helpers/asset-helpers/].
@@ -102,21 +102,21 @@ For more information on using asset helpers, check out the guide for
 Form helpers are the 'standard' form tag helpers you would come to expect when building forms. A simple
 example of constructing a non-object form would be:
 
-  = form_tag '/destroy', :class => 'destroy-form', :method => 'delete' do
+  = form_tag '/destroy', class: 'destroy-form', method: 'delete' do
     = flash_tag(:notice)
     = field_set_tag do
       %p
-        = label_tag :username, :class => 'first'
-        = text_field_tag :username, :value => params[:username]
+        = label_tag :username, class: 'first'
+        = text_field_tag :username, value: params[:username]
       %p
-        = label_tag :password, :class => 'first'
-        = password_field_tag :password, :value => params[:password]
+        = label_tag :password, class: 'first'
+        = password_field_tag :password, value: params[:password]
       %p
         = label_tag :strategy
-        = select_tag :strategy, :options => ['delete', 'destroy'], :selected => 'delete'
+        = select_tag :strategy, options: ['delete', 'destroy'], selected: 'delete'
       %p
         = check_box_tag :confirm_delete
-    = field_set_tag(:class => 'buttons') do
+    = field_set_tag(class: 'buttons') do
       = submit_tag "Remove"
 
 For more information on using form helpers, check out the guide for
@@ -129,10 +129,10 @@ using a simple, intuitive syntax.
 
 A form_for using these basic fields might look like:
 
-  = form_for @user, '/register', :id => 'register' do |f|
+  = form_for @user, '/register', id: 'register' do |f|
     = f.error_messages
     %p
-      = f.label :username, :caption => "Nickname"
+      = f.label :username, caption: "Nickname"
       = f.text_field :username
     %p
       = f.label :email
@@ -141,17 +141,17 @@ A form_for using these basic fields might look like:
       = f.label :password
       = f.password_field :password
     %p
-      = f.label :is_admin, :caption => "Admin User?"
+      = f.label :is_admin, caption: 'Admin User?'
       = f.check_box :is_admin
     %p
-      = f.label :color, :caption => "Favorite Color?"
-      = f.select :color, :options => ['red', 'black']
+      = f.label :color, caption: 'Favorite Color?'
+      = f.select :color, options: ['red', 'black']
     %p
       = fields_for @user.location do |location|
         = location.text_field :street
         = location.text_field :city
     %p
-      = f.submit "Create", :class => 'button'
+      = f.submit 'Create', class: 'button'
 
 Forms can also accept nested attributes using `fields_for` within the form builder in recent releases. Check out the guide for {Padrino Helpers}[http://padrinorb.com/guides/application-helpers/form-builders/] to learn more about nested forms.
 
@@ -159,14 +159,14 @@ There is also an additional StandardFormBuilder which builds on the abstract fie
 
 A form_for using these standard fields might be:
 
-  = form_for @user, '/register', :id => 'register' do |f|
+  = form_for @user, '/register', id: 'register' do |f|
     = f.error_messages
-    = f.text_field_block :name, :caption => "Full name"
+    = f.text_field_block :name, caption: "Full name"
     = f.text_field_block :email
     = f.check_box_block  :remember_me
-    = f.select_block     :fav_color, :options => ['red', 'blue']
+    = f.select_block     :fav_color, options: ['red', 'blue']
     = f.password_field_block :password
-    = f.submit_block "Create", :class => 'button'
+    = f.submit_block "Create", class: 'button'
 
 and would generate this html (with each input contained in a paragraph and containing a label):
 
@@ -200,9 +200,9 @@ Format helpers also includes a number of useful text manipulation functions such
 
   simple_format("hello\nworld") # => "<p>hello<br/>world</p>"
   pluralize(2, 'person') => '2 people'
-  word_wrap('Once upon a time', :line_width => 8) => "Once upon\na time"
-  truncate("Once upon a time in a world far far away", :length => 8) => "Once upon..."
-  truncate_words("Once upon a time in a world far far away", :length => 4) => "Once upon a time..."
+  word_wrap('Once upon a time', line_width: 8) => "Once upon\na time"
+  truncate("Once upon a time in a world far far away", length: 8) => "Once upon..."
+  truncate_words("Once upon a time in a world far far away", length: 4) => "Once upon a time..."
 
 These helpers can be invoked from any route or view within your application.
 
@@ -228,8 +228,8 @@ There is also a method which renders the first view matching the path and remove
 
 Finally, we have the all-important partials support for rendering mini-templates onto a page:
 
-  partial 'photo/_item', :object => @photo, :locals => { :foo => 'bar' }
-  partial 'photo/_item', :collection => @photos
+  partial 'photo/_item', object: @photo, locals: { foo: 'bar' }
+  partial 'photo/_item', collection: @photos
 
 For more information on using the render and partial helpers, check out the guide for
 {Padrino Helpers}[http://padrinorb.com/guides/application-helpers/render-helpers/].

data/Rakefile

@@ -1 +1 @@
-require File.expand_path(File.dirname(__FILE__) + '/../gem_rake_helper')
+require_relative '../gem_rake_helper'

data/lib/padrino/rendering/erb_template.rb

@@ -3,9 +3,9 @@ module Padrino
     class SafeERB < ::ERB
       class Compiler < ::ERB::Compiler
         def add_insert_cmd(out, content)
-          out.push("@__in_ruby_literal = true")
+          out.push('@__in_ruby_literal = true')
           super
-          out.push("@__in_ruby_literal = false")
+          out.push('@__in_ruby_literal = false')
         end
       end
 
@@ -25,7 +25,7 @@ module Padrino
       def render(*args)
         app       = args.first
         app_class = app.class
-        @is_padrino_app = (defined?(Padrino::Application) && app.kind_of?(Padrino::Application)) || 
+        @is_padrino_app = (defined?(Padrino::Application) && app.is_a?(Padrino::Application)) ||
                           (app_class.respond_to?(:erb) && app_class.erb[:safe_buffer])
         super
       end
@@ -33,14 +33,14 @@ module Padrino
       if ERB.instance_method(:initialize).parameters.assoc(:key) # Ruby 2.6+
         def prepare
           @outvar = options[:outvar] || self.class.default_output_variable
-          options[:trim] = '<>' if !(options[:trim] == false) && (options[:trim].nil? || options[:trim] == true)
+          options[:trim] = '<>' if options[:trim] != false && (options[:trim].nil? || options[:trim] == true)
 
           @engine = SafeERB.new(data, trim_mode: options[:trim], eoutvar: @outvar)
         end
       else
         def prepare
           @outvar = options[:outvar] || self.class.default_output_variable
-          options[:trim] = '<>' if !(options[:trim] == false) && (options[:trim].nil? || options[:trim] == true)
+          options[:trim] = '<>' if options[:trim] != false && (options[:trim].nil? || options[:trim] == true)
 
           @engine = SafeERB.new(data, options[:safe], options[:trim], @outvar)
         end
@@ -58,6 +58,6 @@ end
 Tilt.prefer(Padrino::Rendering::ERBTemplate, :erb)
 
 Padrino::Rendering.engine_configurations[:erb] = {
-  :safe_buffer => true,
-  :outvar => '@_out_buf',
+  safe_buffer: true,
+  outvar: '@_out_buf'
 }

data/lib/padrino/rendering/erubi_template.rb

@@ -7,7 +7,7 @@ module Padrino
         end
 
         def add_expression_result_escaped(code)
-          @src << " #{bufvar}.safe_concat (" << code << ");"
+          @src << " #{bufvar}.safe_concat (" << code << ');'
         end
 
         def add_text(text)
@@ -31,7 +31,7 @@ end
 Tilt.prefer(Padrino::Rendering::ErubiTemplate, :erb)
 
 Padrino::Rendering.engine_configurations[:erb] = {
-  :bufval => "SafeBuffer.new",
-  :bufvar => "@_out_buf",
-  :engine_class => Padrino::Rendering::SafeErubi
+  bufval: 'SafeBuffer.new',
+  bufvar: '@_out_buf',
+  engine_class: Padrino::Rendering::SafeErubi
 }

data/lib/padrino/rendering/erubis_template.rb

@@ -13,7 +13,7 @@ module Padrino
       def add_stmt(src, code)
         code = code.sub('end', 'nil;end') if code =~ /\A\s*end\s*\Z/
         src << code
-        src << ';' unless code[-1] == ?\n
+        src << ';' unless code[-1] == "\n"
       end
 
       def add_expr_escaped(src, code)
@@ -42,7 +42,7 @@ module Padrino
       def render(*args)
         app       = args.first
         app_class = app.class
-        @is_padrino_app = (defined?(Padrino::Application) && app.kind_of?(Padrino::Application)) || 
+        @is_padrino_app = (defined?(Padrino::Application) && app.is_a?(Padrino::Application)) ||
                           (app_class.respond_to?(:erb) && app_class.erb[:engine_class] == Padrino::Rendering::SafeEruby)
         super
       end
@@ -62,6 +62,6 @@ end
 Tilt.prefer(Padrino::Rendering::ErubisTemplate, :erb)
 
 Padrino::Rendering.engine_configurations[:erb] = {
-  :engine_class => Padrino::Rendering::SafeEruby,
-  :outvar => '@_out_buf',
+  engine_class: Padrino::Rendering::SafeEruby,
+  outvar: '@_out_buf'
 }

data/lib/padrino/rendering/haml_template.rb

@@ -22,5 +22,5 @@ end
 Tilt.prefer(Padrino::Rendering::HamlTemplate, :haml)
 
 Padrino::Rendering.engine_configurations[:haml] = {
-  :escape_html => true,
+  escape_html: true
 }

data/lib/padrino/rendering/hamlit_template.rb

@@ -1,7 +1,7 @@
 module Padrino
   module Rendering
     class HamlitOutputBuffer < Temple::Generators::StringBuffer
-      define_options :buffer_class => 'SafeBuffer'
+      define_options buffer_class: 'SafeBuffer'
 
       def call(exp)
         [preamble, compile(exp), postamble].flatten.compact.join('; '.freeze)
@@ -29,7 +29,7 @@ end
 Tilt.prefer(Padrino::Rendering::HamlitTemplate, :haml)
 
 Padrino::Rendering.engine_configurations[:haml] = {
-  :generator => Padrino::Rendering::HamlitOutputBuffer,
-  :buffer => "@_out_buf",
-  :use_html_safe => true,
+  generator: Padrino::Rendering::HamlitOutputBuffer,
+  buffer: '@_out_buf',
+  use_html_safe: true
 }

data/lib/padrino/rendering/slim_template.rb

@@ -1,7 +1,7 @@
 module Padrino
   module Rendering
     class SlimOutputBuffer < Temple::Generators::StringBuffer
-      define_options :buffer_class => 'SafeBuffer'
+      define_options buffer_class: 'SafeBuffer'
 
       def call(exp)
         [preamble, compile(exp), postamble].flatten.compact.join('; '.freeze)
@@ -29,8 +29,8 @@ end
 Tilt.prefer(Padrino::Rendering::SlimTemplate, :slim)
 
 Padrino::Rendering.engine_configurations[:slim] = {
-  :generator => Padrino::Rendering::SlimOutputBuffer,
-  :buffer => "@_out_buf",
-  :use_html_safe => true,
-  :disable_capture => true,
+  generator: Padrino::Rendering::SlimOutputBuffer,
+  buffer: '@_out_buf',
+  use_html_safe: true,
+  disable_capture: true
 }

data/lib/padrino/rendering.rb

@@ -31,19 +31,21 @@ module Padrino
     # @example
     #   Padrino::Rendering::IGNORE_FILE_PATTERN << /~$/
     #
-    IGNORE_FILE_PATTERN = [
-      /~$/ # This is for Gedit
-    ] unless defined?(IGNORE_FILE_PATTERN)
+    unless defined?(IGNORE_FILE_PATTERN)
+      IGNORE_FILE_PATTERN = [
+        /~$/ # This is for Gedit
+      ]
+    end
 
     ##
     # Defines common content-type alias mappings.
     #
-    CONTENT_TYPE_ALIASES = { :htm => :html }
+    CONTENT_TYPE_ALIASES = { htm: :html }
 
     ##
     # Default options used in the resolve_template-method.
     #
-    DEFAULT_RENDERING_OPTIONS = { :strict_format => false, :raise_exceptions => true } unless defined?(DEFAULT_RENDERING_OPTIONS)
+    DEFAULT_RENDERING_OPTIONS = { strict_format: false, raise_exceptions: true } unless defined?(DEFAULT_RENDERING_OPTIONS)
 
     class << self
       ##
@@ -58,7 +60,7 @@ module Padrino
       def registered(app)
         if defined?(Padrino::Application) && app == Padrino::Application
           # this fail can be removed later when jRuby is not bugged and MRI19 is dropped
-          fail 'Please, do not use `register` on Padrino::Application object, use `.dup` or subclassing'
+          raise 'Please, do not use `register` on Padrino::Application object, use `.dup` or subclassing'
         end
         included(app)
         engine_configurations.each do |engine, configs|
@@ -94,7 +96,7 @@ module Padrino
       #
       # @yield []
       #
-      def layout(name=:layout, &block)
+      def layout(name = :layout, &block)
         return super(name, &block) if block_given?
         @layout = name
       end
@@ -107,7 +109,7 @@ module Padrino
       # @param [String, nil] layouts_path
       #   The directory where the layouts are located. Defaults to #views.
       #
-      def fetch_layout_path(given_layout, layouts_path=views)
+      def fetch_layout_path(given_layout, layouts_path = views)
         layout_name = (given_layout || @layout || :application).to_s
         cache_layout_path(layout_name) do
           if Pathname.new(layout_name).absolute? && Dir["#{layout_name}.*"].any? || Dir["#{layouts_path}/#{layout_name}.*"].any?
@@ -120,7 +122,7 @@ module Padrino
 
       def cache_layout_path(name)
         @_cached_layout ||= {}
-        if !reload_templates? && path = @_cached_layout[name]
+        if !reload_templates? && (path = @_cached_layout[name])
           path
         else
           @_cached_layout[name] = yield(name)
@@ -131,7 +133,7 @@ module Padrino
         began_at = Time.now
         @_cached_templates ||= {}
         logging = defined?(settings) && settings.logging? && defined?(logger)
-        if !reload_templates? && path = @_cached_templates[options]
+        if !reload_templates? && (path = @_cached_templates[options])
           logger.debug :cached, began_at, path[0] if logging
         else
           path = @_cached_templates[options] = yield(name)
@@ -169,7 +171,7 @@ module Padrino
       #
       #   # => set directly the Content-Type to 'text/html'
       #
-      def content_type(type=nil, params={})
+      def content_type(type = nil, params = {})
         if type
           super(type, params)
           @_content_type = type
@@ -179,19 +181,19 @@ module Padrino
 
       private
 
-      def render_like_sinatra(engine, data, options={}, locals={}, &block)
+      def render_like_sinatra(engine, data, options = {}, locals = {}, &block)
         # merge app-level options
         engine_options = settings.respond_to?(engine) ? settings.send(engine) : {}
         options = engine_options.merge(options)
 
         # extract generic options
         locals          = options.delete(:locals) || locals         || {}
-        views           = options.delete(:views)  || settings.views || "./views"
+        views           = options.delete(:views)  || settings.views || './views'
         layout          = options[:layout]
         layout          = false if layout.nil? && options.include?(:layout)
         eat_errors      = layout.nil?
-        layout          = engine_options[:layout] if layout.nil? or (layout == true && engine_options[:layout] != false)
-        layout          = @default_layout         if layout.nil? or layout == true
+        layout          = engine_options[:layout] if layout.nil? || (layout == true && engine_options[:layout] != false)
+        layout          = @default_layout         if layout.nil? || (layout == true)
         layout_options  = options.delete(:layout_options) || {}
         content_type    = options.delete(:default_content_type)
         content_type    = options.delete(:content_type)   || content_type
@@ -215,7 +217,7 @@ module Padrino
         # render layout
         if layout
           layout_engine_options = settings.respond_to?(layout_engine) ? settings.send(layout_engine).dup : {}
-          options = layout_engine_options.update(:views => views, :layout => false, :eat_errors => eat_errors, :scope => scope).update(layout_options)
+          options = layout_engine_options.update(views: views, layout: false, eat_errors: eat_errors, scope: scope).update(layout_options)
           catch(:layout_missing) { return render_like_sinatra(layout_engine, layout, options, locals) { output } }
         end
 
@@ -230,16 +232,16 @@ module Padrino
       # * Use render 'path/to/my/template'   (without symbols)
       # * Use render 'path/to/my/template'   (with engine lookup)
       # * Use render 'path/to/template.haml' (with explicit engine lookup)
-      # * Use render 'path/to/template', :layout => false
-      # * Use render 'path/to/template', :layout => false, :engine => 'haml'
+      # * Use render 'path/to/template', layout: false
+      # * Use render 'path/to/template', layout: false, engine: 'haml'
       #
-      def render(engine, data=nil, options={}, locals={}, &block)
+      def render(engine, data = nil, options = {}, locals = {}, &block)
         # If engine is nil, ignore engine parameter and shift up all arguments
-        # render nil, "index", { :layout => true }, { :localvar => "foo" }
+        # render nil, 'index', { layout: true }, { localvar: "foo" }
         engine, data, options = data, options, locals if engine.nil? && data
 
         # Data is a hash of options when no engine isn't explicit
-        # render "index", { :layout => true }, { :localvar => "foo" }
+        # render 'index', { layout: true }, { localvar: "foo" }
         # Data is options, and options is locals in this case
         data, options, locals = nil, data, options if data.is_a?(Hash)
 
@@ -251,7 +253,7 @@ module Padrino
 
         # Cleanup the template.
         @current_engine, engine_was = engine, @current_engine
-        @_out_buf,  buf_was = SafeBuffer.new, @_out_buf
+        @_out_buf, buf_was = SafeBuffer.new, @_out_buf
 
         # Pass arguments to Sinatra render method.
         render_like_sinatra(engine, data, with_layout(options), locals, &block)
@@ -283,24 +285,24 @@ module Padrino
       #   The template could not be found.
       #
       # @example
-      #   get "/foo", :provides => [:html, :js] do; render 'path/to/foo'; end
+      #   get "/foo", provides: [:html, :js] do; render 'path/to/foo'; end
       #   # If you request "/foo.js" with I18n.locale == :ru => [:"/path/to/foo.ru.js", :erb]
       #   # If you request "/foo" with I18n.locale == :de => [:"/path/to/foo.de.haml", :haml]
       #
-      def resolve_template(template_path, options={})
+      def resolve_template(template_path, options = {})
         template_path = template_path.to_s
         controller_key = respond_to?(:request) && request.respond_to?(:controller) && request.controller
         rendering_options = [template_path, content_type || :html, locale]
 
         settings.cache_template_path(["#{controller_key}/#{template_path}", rendering_options[1], rendering_options[2]]) do
           options = DEFAULT_RENDERING_OPTIONS.merge(options)
-          view_path = options[:views] || settings.views || "./views"
+          view_path = options[:views] || settings.views || './views'
 
           template_candidates = glob_templates(view_path, template_path)
           selected_template = select_template(template_candidates, *rendering_options)
           selected_template ||= template_candidates.first unless options[:strict_format]
 
-          fail TemplateNotFound, "Template '#{template_path}' not found in '#{view_path}'"  if !selected_template && options[:raise_exceptions]
+          raise TemplateNotFound, "Template '#{template_path}' not found in '#{view_path}'" if !selected_template && options[:raise_exceptions]
           selected_template
         end
       end
@@ -312,8 +314,8 @@ module Padrino
         I18n.locale if defined?(I18n)
       end
 
-      def resolve_layout(layout, options={})
-        layouts_path = options[:layout_options] && options[:layout_options][:views] || options[:views] || settings.views || "./views"
+      def resolve_layout(layout, options = {})
+        layouts_path = options[:layout_options] && options[:layout_options][:views] || options[:views] || settings.views || './views'
         template_path = settings.fetch_layout_path(layout, layouts_path)
         rendering_options = [template_path, content_type || :html, locale]
 
@@ -321,7 +323,7 @@ module Padrino
           template_candidates = glob_templates(layouts_path, template_path)
           selected_template = select_template(template_candidates, *rendering_options)
 
-          fail TemplateNotFound, "Layout '#{template_path}' not found in '#{layouts_path}'" if !selected_template && layout
+          raise TemplateNotFound, "Layout '#{template_path}' not found in '#{layouts_path}'" if !selected_template && layout
           selected_template
         end
       end
@@ -332,15 +334,15 @@ module Padrino
         return options if layout == false
 
         layout = @layout if !layout || layout == true
-        return options if settings.templates.has_key?(:layout) && !layout
+        return options if settings.templates.key?(:layout) && !layout
 
-        if layout.kind_of?(String) && Pathname.new(layout).absolute?
+        if layout.is_a?(String) && Pathname.new(layout).absolute?
           layout_path, _, layout = layout.rpartition('/')
           options[:layout_options] ||= {}
           options[:layout_options][:views] ||= layout_path
         end
         layout, layout_engine = resolve_layout(layout, options)
-        options.update(:layout => layout, :layout_engine => layout_engine)
+        options.update(layout: layout, layout_engine: layout_engine)
       end
 
       def glob_templates(views_path, template_path)
@@ -349,28 +351,29 @@ module Padrino
         if respond_to?(:request) && request.respond_to?(:controller) && request.controller && Pathname.new(template_path).relative?
           parts << "{,#{request.controller}}"
         end
-        parts << template_path.chomp(File.extname(template_path)) + '.*'
-        Dir.glob(File.join(parts)).inject([]) do |all,file|
-          next all if IGNORE_FILE_PATTERN.any?{ |pattern| file.to_s =~ pattern }
+        parts << "#{template_path.chomp(File.extname(template_path))}.*"
+        Dir.glob(File.join(parts)).inject([]) do |all, file|
+          next all if IGNORE_FILE_PATTERN.any? { |pattern| file.to_s =~ pattern }
           all << path_and_engine(file, views_path)
         end
       end
 
       def select_template(templates, template_path, content_type, _locale)
         symbol = content_type_symbol(content_type)
-        simple_content_type = [:html, :plain].include?(symbol)
+        simple_content_type = %i[html plain].include?(symbol)
         target_path, target_engine = path_and_engine(template_path)
 
-        templates.find{ |file,_| file.to_s == "#{target_path}.#{locale}.#{symbol}" } ||
-        templates.find{ |file,_| file.to_s == "#{target_path}.#{locale}" && simple_content_type } ||
-        templates.find{ |file,engine| engine == target_engine || File.extname(file.to_s) == ".#{target_engine}" } ||
-        templates.find{ |file,_| file.to_s == "#{target_path}.#{symbol}" } ||
-        templates.find{ |file,_| file.to_s == "#{target_path}" && simple_content_type }
+        # FIXME: We can probable iterate just 1 time over the templates
+        templates.find { |file, _| file.to_s == "#{target_path}.#{locale}.#{symbol}" } ||
+          templates.find { |file, _| file.to_s == "#{target_path}.#{locale}" && simple_content_type } ||
+          templates.find { |file, engine| engine == target_engine || File.extname(file.to_s) == ".#{target_engine}" } ||
+          templates.find { |file, _| file.to_s == "#{target_path}.#{symbol}" } ||
+          templates.find { |file, _| file.to_s == target_path.to_s && simple_content_type }
       end
 
-      def path_and_engine(path, relative=nil)
+      def path_and_engine(path, relative = nil)
         extname = File.extname(path)
-        engine = (extname[1..-1]||'none').to_sym
+        engine = (extname[1..] || 'none').to_sym
         path = path.chomp(extname)
         path.insert(0, '/') unless Pathname.new(path).absolute?
         path = path.squeeze('/').sub(relative, '') if relative
@@ -382,15 +385,16 @@ module Padrino
         return nil unless engine == :erb
         require 'erb'
       rescue LoadError
+        # do nothing since erb is not available
       else
         require 'padrino/rendering/erb_template'
         settings.set :erb, Padrino::Rendering.engine_configurations[:erb]
       end
 
       def content_type_symbol(type)
-        if defined?(::Rack::Mime::MIME_TYPES) && type.kind_of?(String) &&
+        if defined?(::Rack::Mime::MIME_TYPES) && type.is_a?(String) &&
            ::Rack::Mime::MIME_TYPES.key(type)
-          type = ::Rack::Mime::MIME_TYPES.key(type).sub(/\./,'').to_sym
+          type = ::Rack::Mime::MIME_TYPES.key(type).sub(/\./, '').to_sym
         end
         CONTENT_TYPE_ALIASES[type] || type
       end
@@ -404,6 +408,7 @@ unless defined? Padrino::Rendering::HamlTemplate
     require 'haml/helpers/xss_mods'
     require 'haml/helpers/action_view_extensions'
   rescue LoadError
+    # do nothing since haml is not available
   else
     require 'padrino/rendering/haml_template'
   end
@@ -413,6 +418,7 @@ unless defined? Padrino::Rendering::HamlitTemplate
   begin
     require 'hamlit'
   rescue LoadError
+    # do nothing since hamlit is not available
   else
     require 'padrino/rendering/hamlit_template'
   end
@@ -422,6 +428,7 @@ unless defined? Padrino::Rendering::ErubisTemplate
   begin
     require 'erubis'
   rescue LoadError
+    # do nothing since erubis is not available
   else
     require 'padrino/rendering/erubis_template'
   end
@@ -431,6 +438,7 @@ unless defined? Padrino::Rendering::SlimTemplate
   begin
     require 'slim'
   rescue LoadError
+    # do nothing since slim is not available
   else
     require 'padrino/rendering/slim_template'
   end
@@ -440,6 +448,7 @@ unless defined? Padrino::Rendering::ErubiTemplate
   begin
     require 'erubi'
   rescue LoadError
+    # do nothing since erubi is not available
   else
     require 'padrino/rendering/erubi_template'
   end