Rails4: db migrate: replace "Permission.find_by_controller_and_action"

git-svn-id: http://svn.redmine.org/redmine/trunk@12422 e93f8b46-1217-0410-a6f0-8f06a7374b81

diff --git a/db/migrate/018_set_doc_and_files_notifications.rb b/db/migrate/018_set_doc_and_files_notifications.rb
index 8c1d054c1adbdc28b8daab729ce43e4e1462a862..f260bebc635c19aa6c841e64b69b1efadde00a10 100644 (file)
--- a/db/migrate/018_set_doc_and_files_notifications.rb
+++ b/db/migrate/018_set_doc_and_files_notifications.rb
@@ -3,16 +3,16 @@ class SetDocAndFilesNotifications < ActiveRecord::Migration
   class Permission < ActiveRecord::Base; end
 
   def self.up
-    Permission.find_by_controller_and_action("projects", "add_file").update_attribute(:mail_option, true)
-    Permission.find_by_controller_and_action("projects", "add_document").update_attribute(:mail_option, true)
-    Permission.find_by_controller_and_action("documents", "add_attachment").update_attribute(:mail_option, true)
-    Permission.find_by_controller_and_action("issues", "add_attachment").update_attribute(:mail_option, true)
+    Permission.where(:controller => "projects", :action => "add_file").each {|p| p.update_attribute(:mail_option, true)}
+    Permission.where(:controller => "projects", :action => "add_document").each {|p| p.update_attribute(:mail_option, true)}
+    Permission.where(:controller => "documents", :action => "add_attachment").each {|p| p.update_attribute(:mail_option, true)}
+    Permission.where(:controller => "issues", :action => "add_attachment").each {|p| p.update_attribute(:mail_option, true)}
   end
 
   def self.down
-    Permission.find_by_controller_and_action("projects", "add_file").update_attribute(:mail_option, false)
-    Permission.find_by_controller_and_action("projects", "add_document").update_attribute(:mail_option, false)
-    Permission.find_by_controller_and_action("documents", "add_attachment").update_attribute(:mail_option, false)
-    Permission.find_by_controller_and_action("issues", "add_attachment").update_attribute(:mail_option, false)
+    Permission.where(:controller => "projects", :action => "add_file").each {|p| p.update_attribute(:mail_option, false)}
+    Permission.where(:controller => "projects", :action => "add_document").each {|p| p.update_attribute(:mail_option, false)}
+    Permission.where(:controller => "documents", :action => "add_attachment").each {|p| p.update_attribute(:mail_option, false)}
+    Permission.where(:controller => "issues", :action => "add_attachment").each {|p| p.update_attribute(:mail_option, false)}
   end
 end

diff --git a/db/migrate/025_add_search_permission.rb b/db/migrate/025_add_search_permission.rb
index a942b01b35d44b52fc37c61ce61871f574a72f46..7f1c5c6fc8233649fd1ca545d1b99773c626e27d 100644 (file)
--- a/db/migrate/025_add_search_permission.rb
+++ b/db/migrate/025_add_search_permission.rb
@@ -7,6 +7,6 @@ class AddSearchPermission < ActiveRecord::Migration
   end
 
   def self.down
-    Permission.find_by_controller_and_action('projects', 'search').destroy
+    Permission.where(:controller => "projects", :action => "search").each {|p| p.destroy}
   end
 end

diff --git a/db/migrate/030_add_projects_feeds_permissions.rb b/db/migrate/030_add_projects_feeds_permissions.rb
index 7f97035bf198340829ec459b9cde0f470e60bbe2..866cc39ae6503084c104143c789ff8178b36d0e5 100644 (file)
--- a/db/migrate/030_add_projects_feeds_permissions.rb
+++ b/db/migrate/030_add_projects_feeds_permissions.rb
@@ -7,6 +7,6 @@ class AddProjectsFeedsPermissions < ActiveRecord::Migration
   end
 
   def self.down
-    Permission.find_by_controller_and_action('projects', 'feeds').destroy
+    Permission.where(:controller => "projects", :action => "feeds").each {|p| p.destroy}
   end
 end

diff --git a/db/migrate/033_add_timelog_permissions.rb b/db/migrate/033_add_timelog_permissions.rb
index ab9c809e647653ee524d93f85dbb9540d993398b..58e2c436a12176f202754aaaaddc1e84b93802a1 100644 (file)
--- a/db/migrate/033_add_timelog_permissions.rb
+++ b/db/migrate/033_add_timelog_permissions.rb
@@ -7,6 +7,6 @@ class AddTimelogPermissions < ActiveRecord::Migration
   end
 
   def self.down
-    Permission.find_by_controller_and_action('timelog', 'edit').destroy
+    Permission.where(:controller => "timelog", :action => "edit").each {|p| p.destroy}
   end
 end

diff --git a/db/migrate/043_add_relations_permissions.rb b/db/migrate/043_add_relations_permissions.rb
index 32d464a585ea4d507a34307e35ade33e5cb0f221..3c86d7e60fbdedd72c54ee638a76cd057498b61d 100644 (file)
--- a/db/migrate/043_add_relations_permissions.rb
+++ b/db/migrate/043_add_relations_permissions.rb
@@ -8,7 +8,7 @@ class AddRelationsPermissions < ActiveRecord::Migration
   end
 
   def self.down
-    Permission.find_by_controller_and_action("issue_relations", "new").destroy
-    Permission.find_by_controller_and_action("issue_relations", "destroy").destroy
+    Permission.where(:controller => "issue_relations", :action => "new").each {|p| p.destroy}
+    Permission.where(:controller => "issue_relations", :action => "destroy").each {|p| p.destroy}
   end
 end

diff --git a/db/migrate/047_add_boards_permissions.rb b/db/migrate/047_add_boards_permissions.rb
index 5b1f6f779ce9e186bf17cb964def06f007c7f80b..1a9f095f7b721214224df0ba75a29537eac44ea7 100644 (file)
--- a/db/migrate/047_add_boards_permissions.rb
+++ b/db/migrate/047_add_boards_permissions.rb
@@ -9,8 +9,8 @@ class AddBoardsPermissions < ActiveRecord::Migration
   end
 
   def self.down
-    Permission.find_by_controller_and_action("boards", "new").destroy
-    Permission.find_by_controller_and_action("boards", "edit").destroy
-    Permission.find_by_controller_and_action("boards", "destroy").destroy
+    Permission.where(:controller => "boards", :action => "new").each {|p| p.destroy}
+    Permission.where(:controller => "boards", :action => "edit").each {|p| p.destroy}
+    Permission.where(:controller => "boards", :action => "destroy").each {|p| p.destroy}
   end
 end

diff --git a/db/migrate/049_add_wiki_destroy_page_permission.rb b/db/migrate/049_add_wiki_destroy_page_permission.rb
index c82152388352b0761dd6022565fe8947499419b1..803d357a1ca853c89ccb613d1cb6499a74d02dd7 100644 (file)
--- a/db/migrate/049_add_wiki_destroy_page_permission.rb
+++ b/db/migrate/049_add_wiki_destroy_page_permission.rb
@@ -7,6 +7,6 @@ class AddWikiDestroyPagePermission < ActiveRecord::Migration
   end
 
   def self.down
-    Permission.find_by_controller_and_action('wiki', 'destroy').destroy
+    Permission.where(:controller => "wiki", :action => "destroy").each {|p| p.destroy}
   end
 end

diff --git a/db/migrate/050_add_wiki_attachments_permissions.rb b/db/migrate/050_add_wiki_attachments_permissions.rb
index c0697be9c9b026652f4143f3866a136b8b173b57..e87a46bb332f07e6185e975aa3cbf3066b8aa8e4 100644 (file)
--- a/db/migrate/050_add_wiki_attachments_permissions.rb
+++ b/db/migrate/050_add_wiki_attachments_permissions.rb
@@ -8,7 +8,7 @@ class AddWikiAttachmentsPermissions < ActiveRecord::Migration
   end
 
   def self.down
-    Permission.find_by_controller_and_action('wiki', 'add_attachment').destroy
-    Permission.find_by_controller_and_action('wiki', 'destroy_attachment').destroy
+    Permission.where(:controller => "wiki", :action => "add_attachment").each {|p| p.destroy}
+    Permission.where(:controller => "wiki", :action => "destroy_attachment").each {|p| p.destroy}
   end
 end

diff --git a/db/migrate/056_add_repositories_changes_permission.rb b/db/migrate/056_add_repositories_changes_permission.rb
index 0d9b13b59dddc3cead7b143b85abaea93f46b166..00252db431c44fa72c1b1d99cea40db5475fcbca 100644 (file)
--- a/db/migrate/056_add_repositories_changes_permission.rb
+++ b/db/migrate/056_add_repositories_changes_permission.rb
@@ -7,6 +7,6 @@ class AddRepositoriesChangesPermission < ActiveRecord::Migration
   end
 
   def self.down
-    Permission.find_by_controller_and_action('repositories', 'changes').destroy
+    Permission.where(:controller => "repositories", :action => "changes").each {|p| p.destroy}
   end
 end