aboutsummaryrefslogtreecommitdiffstats
path: root/railties/lib/rails_generator
diff options
context:
space:
mode:
authorJoshua Peek <josh@joshpeek.com>2008-07-11 12:05:02 -0500
committerJoshua Peek <josh@joshpeek.com>2008-07-11 12:05:02 -0500
commitf522a89d6447da306778b67353adaf679c26bbd1 (patch)
treec55d228bf6c909bb91d8b24d27a9f1648bc970dd /railties/lib/rails_generator
parent04a87af5b7abd00e74b96c5b1bb789d4484da6d9 (diff)
downloadrails-f522a89d6447da306778b67353adaf679c26bbd1.tar.gz
rails-f522a89d6447da306778b67353adaf679c26bbd1.tar.bz2
rails-f522a89d6447da306778b67353adaf679c26bbd1.zip
Revert "Fixed generator collisions for nested controller modules."
This reverts commit 2d372d704987e05712ccd937e78d8dbd41242efe.
Diffstat (limited to 'railties/lib/rails_generator')
-rw-r--r--railties/lib/rails_generator/commands.rb25
1 files changed, 9 insertions, 16 deletions
diff --git a/railties/lib/rails_generator/commands.rb b/railties/lib/rails_generator/commands.rb
index fb62ba6940..d258aeaa0a 100644
--- a/railties/lib/rails_generator/commands.rb
+++ b/railties/lib/rails_generator/commands.rb
@@ -154,35 +154,28 @@ HELP
# Ruby or Rails. In the future, expand to check other namespaces
# such as the rest of the user's app.
def class_collisions(*class_names)
-
- # Initialize some check varibles
- last_class = Object
- current_class = nil
- name = nil
-
class_names.flatten.each do |class_name|
# Convert to string to allow symbol arguments.
class_name = class_name.to_s
# Skip empty strings.
- class_name.strip.empty? ? next : current_class = class_name
+ next if class_name.strip.empty?
# Split the class from its module nesting.
nesting = class_name.split('::')
name = nesting.pop
# Extract the last Module in the nesting.
- last = nesting.inject(last_class) { |last, nest|
- break unless last_class.const_defined?(nest)
- last_class = last_class.const_get(nest)
+ last = nesting.inject(Object) { |last, nest|
+ break unless last.const_defined?(nest)
+ last.const_get(nest)
}
- end
- # If the last Module exists, check whether the given
- # class exists and raise a collision if so.
-
- if last_class and last_class.const_defined?(name.camelize)
- raise_class_collision(current_class)
+ # If the last Module exists, check whether the given
+ # class exists and raise a collision if so.
+ if last and last.const_defined?(name.camelize)
+ raise_class_collision(class_name)
+ end
end
end