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

@NARKOZ

No description provided.

@jeremy jeremy merged commit e1af54d into from
@steveklabnik
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:. :)

@NARKOZ

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

@steveklabnik
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. @NARKOZ
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
View
2  actionpack/lib/action_view/lookup_context.rb
@@ -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.