]> source.dussan.org Git - redmine.git/commitdiff
Removed unneeded :foreign_key option on belongs_to associations.
authorJean-Philippe Lang <jp_lang@yahoo.fr>
Wed, 22 Oct 2014 18:41:03 +0000 (18:41 +0000)
committerJean-Philippe Lang <jp_lang@yahoo.fr>
Wed, 22 Oct 2014 18:41:03 +0000 (18:41 +0000)
git-svn-id: http://svn.redmine.org/redmine/trunk@13484 e93f8b46-1217-0410-a6f0-8f06a7374b81

12 files changed:
app/models/attachment.rb
app/models/board.rb
app/models/comment.rb
app/models/document.rb
app/models/issue.rb
app/models/issue_category.rb
app/models/issue_relation.rb
app/models/message.rb
app/models/news.rb
app/models/time_entry.rb
app/models/wiki_content.rb
app/models/workflow_rule.rb

index 9d7e0324760c73ef9c43714bfe61d2ac1181cf17..2af10663e6a36a328fcb2a02132600584a606f13 100644 (file)
@@ -20,7 +20,7 @@ require "fileutils"
 
 class Attachment < ActiveRecord::Base
   belongs_to :container, :polymorphic => true
-  belongs_to :author, :class_name => "User", :foreign_key => "author_id"
+  belongs_to :author, :class_name => "User"
 
   validates_presence_of :filename, :author
   validates_length_of :filename, :maximum => 255
index 2cbeb0a8a886660d8c85502d0fa060edf8deca99..4a9984a03f1da07508cbc8c9e7f2e4319c23b630 100644 (file)
@@ -20,7 +20,7 @@ class Board < ActiveRecord::Base
   belongs_to :project
   has_many :topics, lambda {where("#{Message.table_name}.parent_id IS NULL").order("#{Message.table_name}.created_on DESC")}, :class_name => 'Message'
   has_many :messages, lambda {order("#{Message.table_name}.created_on DESC")}, :dependent => :destroy
-  belongs_to :last_message, :class_name => 'Message', :foreign_key => :last_message_id
+  belongs_to :last_message, :class_name => 'Message'
   acts_as_tree :dependent => :nullify
   acts_as_list :scope => '(project_id = #{project_id} AND parent_id #{parent_id ? "= #{parent_id}" : "IS NULL"})'
   acts_as_watchable
index 9893ad664955fde24632a6e079e82f5426b7577c..abe7550188759b7e69cb29417025e1d275bb56b7 100644 (file)
@@ -18,7 +18,7 @@
 class Comment < ActiveRecord::Base
   include Redmine::SafeAttributes
   belongs_to :commented, :polymorphic => true, :counter_cache => true
-  belongs_to :author, :class_name => 'User', :foreign_key => 'author_id'
+  belongs_to :author, :class_name => 'User'
 
   validates_presence_of :commented, :author, :comments
   attr_protected :id
index c195f64cae18b9a990aadc2f5225ef25a6c3a70c..00530131aa86d38bda7681e45fb84598cc356de6 100644 (file)
@@ -18,7 +18,7 @@
 class Document < ActiveRecord::Base
   include Redmine::SafeAttributes
   belongs_to :project
-  belongs_to :category, :class_name => "DocumentCategory", :foreign_key => "category_id"
+  belongs_to :category, :class_name => "DocumentCategory"
   acts_as_attachable :delete_permission => :delete_documents
 
   acts_as_searchable :columns => ['title', "#{table_name}.description"],
index bb45a6f58e177195368667fdf6fb803ea8d13a53..34a49be0d9c1b9023f6f3a730f5f48a64631d5f0 100644 (file)
@@ -22,12 +22,12 @@ class Issue < ActiveRecord::Base
 
   belongs_to :project
   belongs_to :tracker
-  belongs_to :status, :class_name => 'IssueStatus', :foreign_key => 'status_id'
-  belongs_to :author, :class_name => 'User', :foreign_key => 'author_id'
-  belongs_to :assigned_to, :class_name => 'Principal', :foreign_key => 'assigned_to_id'
-  belongs_to :fixed_version, :class_name => 'Version', :foreign_key => 'fixed_version_id'
-  belongs_to :priority, :class_name => 'IssuePriority', :foreign_key => 'priority_id'
-  belongs_to :category, :class_name => 'IssueCategory', :foreign_key => 'category_id'
+  belongs_to :status, :class_name => 'IssueStatus'
+  belongs_to :author, :class_name => 'User'
+  belongs_to :assigned_to, :class_name => 'Principal'
+  belongs_to :fixed_version, :class_name => 'Version'
+  belongs_to :priority, :class_name => 'IssuePriority'
+  belongs_to :category, :class_name => 'IssueCategory'
 
   has_many :journals, :as => :journalized, :dependent => :destroy
   has_many :visible_journals,
index e5e508861c4884e8ae24bae552c80cc69b1dc550..4fda288b8476829bc888062e9df35c0e1f86fe57 100644 (file)
@@ -18,7 +18,7 @@
 class IssueCategory < ActiveRecord::Base
   include Redmine::SafeAttributes
   belongs_to :project
-  belongs_to :assigned_to, :class_name => 'Principal', :foreign_key => 'assigned_to_id'
+  belongs_to :assigned_to, :class_name => 'Principal'
   has_many :issues, :foreign_key => 'category_id', :dependent => :nullify
 
   validates_presence_of :name
index 1ba50edc13822642b6856b3c3c9d8031086fb9e8..2659bb2e72de6fd683f8fb28180173a4783a6d1d 100644 (file)
@@ -30,8 +30,8 @@ class IssueRelation < ActiveRecord::Base
     end
   end
 
-  belongs_to :issue_from, :class_name => 'Issue', :foreign_key => 'issue_from_id'
-  belongs_to :issue_to, :class_name => 'Issue', :foreign_key => 'issue_to_id'
+  belongs_to :issue_from, :class_name => 'Issue'
+  belongs_to :issue_to, :class_name => 'Issue'
 
   TYPE_RELATES      = "relates"
   TYPE_DUPLICATES   = "duplicates"
index 028a3eb551991e472c07672b51cb86812a0cab16..a5540d95354be0d227d9723aec6bf69d1a553d19 100644 (file)
 class Message < ActiveRecord::Base
   include Redmine::SafeAttributes
   belongs_to :board
-  belongs_to :author, :class_name => 'User', :foreign_key => 'author_id'
+  belongs_to :author, :class_name => 'User'
   acts_as_tree :counter_cache => :replies_count, :order => "#{Message.table_name}.created_on ASC"
   acts_as_attachable
-  belongs_to :last_reply, :class_name => 'Message', :foreign_key => 'last_reply_id'
+  belongs_to :last_reply, :class_name => 'Message'
   attr_protected :id
 
   acts_as_searchable :columns => ['subject', 'content'],
index baaa92fba377269a60cd6061289dd1ef646da8c9..59411bd44e47be03ce915c34b8fd84c284754e69 100644 (file)
@@ -18,7 +18,7 @@
 class News < ActiveRecord::Base
   include Redmine::SafeAttributes
   belongs_to :project
-  belongs_to :author, :class_name => 'User', :foreign_key => 'author_id'
+  belongs_to :author, :class_name => 'User'
   has_many :comments, lambda {order("created_on")}, :as => :commented, :dependent => :delete_all
 
   validates_presence_of :title, :description
index 8738668cc6fcdd7aeea08597531819c849bcb7b9..80112308a77b65560ccb15b1deeab00f14fa13d5 100644 (file)
@@ -22,7 +22,7 @@ class TimeEntry < ActiveRecord::Base
   belongs_to :project
   belongs_to :issue
   belongs_to :user
-  belongs_to :activity, :class_name => 'TimeEntryActivity', :foreign_key => 'activity_id'
+  belongs_to :activity, :class_name => 'TimeEntryActivity'
 
   attr_protected :user_id, :tyear, :tmonth, :tweek
 
index 3eac926888da063cd53935e7b3196e1c083a917d..9377ccfc2134fdd9996e8e51cb45d80e9ba308de 100644 (file)
@@ -19,8 +19,8 @@ require 'zlib'
 
 class WikiContent < ActiveRecord::Base
   self.locking_column = 'version'
-  belongs_to :page, :class_name => 'WikiPage', :foreign_key => 'page_id'
-  belongs_to :author, :class_name => 'User', :foreign_key => 'author_id'
+  belongs_to :page, :class_name => 'WikiPage'
+  belongs_to :author, :class_name => 'User'
   validates_presence_of :text
   validates_length_of :comments, :maximum => 255, :allow_nil => true
   attr_protected :id
@@ -54,8 +54,8 @@ class WikiContent < ActiveRecord::Base
   end
 
   class Version
-    belongs_to :page, :class_name => '::WikiPage', :foreign_key => 'page_id'
-    belongs_to :author, :class_name => '::User', :foreign_key => 'author_id'
+    belongs_to :page, :class_name => '::WikiPage'
+    belongs_to :author, :class_name => '::User'
     attr_protected :data
 
     acts_as_event :title => Proc.new {|o| "#{l(:label_wiki_edit)}: #{o.page.title} (##{o.version})"},
index 3d1b75bd09800db3a273b7fa388f673a760d5641..0cd837d7c9c52b148f0f58b61848b234aeb1ea99 100644 (file)
@@ -20,8 +20,8 @@ class WorkflowRule < ActiveRecord::Base
 
   belongs_to :role
   belongs_to :tracker
-  belongs_to :old_status, :class_name => 'IssueStatus', :foreign_key => 'old_status_id'
-  belongs_to :new_status, :class_name => 'IssueStatus', :foreign_key => 'new_status_id'
+  belongs_to :old_status, :class_name => 'IssueStatus'
+  belongs_to :new_status, :class_name => 'IssueStatus'
 
   validates_presence_of :role, :tracker, :old_status
   attr_protected :id