Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

use presence method instead of checking for blank #7637

Merged
merged 1 commit into from

3 participants

Nihad Abbasov Steve Klabnik Jeremy Kemper
Nihad Abbasov

No description provided.

Jeremy Kemper jeremy merged commit e1af54d into from
Steve Klabnik
Collaborator

We should try to keep better commit messages: I know I'm a bit new around here, but this barely explains anything about what's going on, and I don't want to make @drogus :cry:. :)

Nihad Abbasov

No worries, @steveklabnik . Thanks for making rails better.

Steve Klabnik
Collaborator

Thanks for actually writing patches!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Sep 14, 2012
  1. Nihad Abbasov
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  actionpack/lib/action_view/lookup_context.rb
2  actionpack/lib/action_view/lookup_context.rb
View
@@ -149,7 +149,7 @@ def detail_args_for(options)
# as well as incorrectly putting part of the path in the template
# name instead of the prefix.
def normalize_name(name, prefixes) #:nodoc:
- prefixes = nil if prefixes.blank?
+ prefixes = prefixes.presence
parts = name.to_s.split('/')
parts.shift if parts.first.empty?
name = parts.pop
Something went wrong with that request. Please try again.