using .all instead of .find(:all)

ryanb · Nov 11, 2008 · 4daa66b · 4daa66b
1 parent 8aececb
commit 4daa66b
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/rails_generators/nifty_scaffold/templates/actions/index.rb b/rails_generators/nifty_scaffold/templates/actions/index.rb
@@ -1,3 +1,3 @@
   def index
-    @<%= plural_name %> = <%= class_name %>.find(:all)
+    @<%= plural_name %> = <%= class_name %>.all
   end
diff --git a/test/test_helper.rb b/test/test_helper.rb
@@ -96,9 +96,9 @@ class Column
       attr_accessor :name, :default, :type, :limit, :null, :sql_type, :precision, :scale
 
       def initialize(name, default, sql_type = nil)
-        @name=name
-        @default=default
-        @type=@sql_type=sql_type
+        @name = name
+        @default = default
+        @type = @sql_type = sql_type
       end
 
       def human_name

diff --git a/test/test_nifty_scaffold_generator.rb b/test/test_nifty_scaffold_generator.rb
@@ -100,7 +100,7 @@ class TestNiftyScaffoldGenerator < Test::Unit::TestCase
       should "generate controller with index action" do
         assert_generated_file "app/controllers/line_items_controller.rb" do |body|
           assert_match "def index", body
-          assert_match "@line_items = LineItem.find(:all)", body
+          assert_match "@line_items = LineItem.all", body
           assert_no_match(/    def index/, body)
         end
       end