Should be @categories[i], not cat
authorNguyễn Thái Ngọc Duy <[email protected]>
Thu, 17 Aug 2006 04:29:13 +0000 (17 11:29 +0700)
committerNguyễn Thái Ngọc Duy <[email protected]>
Thu, 17 Aug 2006 04:29:13 +0000 (17 11:29 +0700)
app/controllers/admin_controller.rb

index fd24c8a..d0f737e 100644 (file)
@@ -265,10 +265,11 @@ class AdminController < ApplicationController
     categories = Category.find(params[:cat].keys)
     for i  in 0...categories.length
       # TODO: validation checks
-      categories[i].cat_name = params[:cat][cat.id.to_s][:cat_name]
-      categories[i].disp_position = params[:cat][cat.id.to_s][:disp_position]
+      categories[i].cat_name = params[:cat][categories[i].id.to_s][:cat_name]
+      categories[i].disp_position = params[:cat][categories[i].id.to_s][:disp_position]
     end
-    categories.each { |cat| cat.save }
+    categories.each { |cat| cat.save or flash[:error] = cat.errors }
+    flash[:notice] = 'Categories updated' if flash[:error].nil?
   end
 
   def add_forum