[#247] update validations

Conflicts:
	app/models/product_build_list.rb

app/models/authentication.rb

@@ -1,6 +1,6 @@
 class Authentication < ActiveRecord::Base
   belongs_to :user
 
-  validates :provider, :uid, :user_id, presence: true
+  validates :provider, :uid, :user, presence: true
   validates :uid, uniqueness: { scope: :provider, case_sensitive: false }
 end

app/models/build_list.rb

@@ -38,13 +38,13 @@ class BuildList < ActiveRecord::Base
     AUTO_PUBLISH_STATUS_TESTING = 'testing'
   ]
 
-  validates :project_id,
+  validates :project,
             :project_version,
             :arch,
             :include_repos,
-            :build_for_platform_id,
-            :save_to_platform_id,
-            :save_to_repository_id,
+            :build_for_platform,
+            :save_to_platform,
+            :save_to_repository,
             presence: true
   validates_numericality_of :priority, greater_than_or_equal_to: 0
   validates :external_nodes, inclusion: { in:  EXTERNAL_NODES }, allow_blank: true

app/models/build_list/package.rb

@@ -9,7 +9,7 @@ class BuildList::Package < ActiveRecord::Base
 
   attr_accessible :fullname, :name, :release, :version, :sha1, :epoch, :dependent_packages
 
-  validates :build_list_id, :project_id, :platform_id, :fullname,
+  validates :build_list, :project, :platform, :fullname,
             :package_type, :name, :release, :version,
             presence: true
   validates :package_type, inclusion: PACKAGE_TYPES

app/models/build_script.rb

@@ -9,8 +9,8 @@ class BuildScript < ActiveRecord::Base
 
   belongs_to :project
 
-  validates :treeish,     presence: true
-  validates :project_id,  presence: true, uniqueness: { scope: :treeish }
+  validates :treeish, presence: true
+  validates :project, presence: true, uniqueness: { scope: :treeish }
 
   scope :by_active,   -> { where(status: ACTIVE) }
   scope :by_treeish,  -> treeish { where(treeish: treeish) }

app/models/comment.rb

@@ -12,7 +12,7 @@ class Comment < ActiveRecord::Base
   belongs_to :project
   serialize :data
 
-  validates :body, :user_id, :commentable_id, :commentable_type, :project_id, presence: true
+  validates :body, :user, :commentable_id, :commentable_type, :project_id, presence: true
 
   scope :for_commit, ->(c) { where(commentable_id: c.id.hex, commentable_type: c.class) }
   default_scope { order(:created_at) }

app/models/hook.rb

@@ -6,7 +6,7 @@ class Hook < ActiveRecord::Base
   belongs_to :project
 
   before_validation :cleanup_data
-  validates :project_id, :data, presence: true
+  validates :project, :data, presence: true
   validates :name, presence: true, inclusion: {in: NAMES}
 
   attr_accessible :data, :name

app/models/issue.rb

@@ -35,7 +35,7 @@ class Issue < ActiveRecord::Base
 
   has_one :pull_request#, dependent: :destroy
 
-  validates :title, :body, :project_id, presence: true
+  validates :title, :body, :project, presence: true
 
   after_create :set_serial_id
   after_create :subscribe_users

app/models/key_pair.rb

@@ -7,7 +7,7 @@ class KeyPair < ActiveRecord::Base
   attr_accessible :public, :secret, :repository_id
   attr_encrypted :secret, key: APP_CONFIG['keys']['key_pair_secret_key']
 
-  validates :repository_id, :user_id, presence: true
+  validates :repository, :user, presence: true
   validates :secret, :public, presence: true, length: { maximum: 10000 }, on: :create
 
   validates :repository_id, uniqueness: { message: I18n.t("activerecord.errors.key_pair.repo_key_exists") }

app/models/platform_arch_setting.rb

@@ -7,7 +7,7 @@ class PlatformArchSetting < ActiveRecord::Base
   belongs_to :arch
   belongs_to :platform
 
-  validates :arch_id, :platform_id, presence: true
+  validates :arch, :platform, presence: true
   validates :platform_id, uniqueness: { scope: :arch_id }
   validate lambda {
     errors.add(:platform, I18n.t('flash.platform_arch_settings.wrong_platform')) unless platform.main?

app/models/product.rb

@@ -8,7 +8,7 @@ class Product < ActiveRecord::Base
   has_many :product_build_lists, dependent: :destroy
 
   validates :name, presence: true, uniqueness: { scope: :platform_id }
-  validates :project_id, presence: true
+  validates :project, presence: true
   validates :main_script, :params, length: { maximum: 255 }
 
   scope :recent, -> { order(:name) }

app/models/product_build_list.rb

@@ -45,11 +45,11 @@ class ProductBuildList < ActiveRecord::Base
   before_validation -> { self.arch_id = Arch.find_by(name: 'x86_64').id }, on: :create
   # field "not_delete" can be changed only if build has been completed
   before_validation -> { self.not_delete = false unless build_completed?; true }
-  validates :product_id,
+  validates :product,
             :status,
-            :project_id,
+            :project,
             :main_script,
-            :arch_id, presence: true
+            :arch, presence: true
   validates :status, inclusion: { in: STATUSES }
   validates :main_script, :params, length: { maximum: 255 }
 

app/models/project.rb

@@ -41,9 +41,9 @@ class Project < ActiveRecord::Base
                    presence: true,
                    format: { with: /\A#{NAME_REGEXP.source}\z/,
                              message: I18n.t("activerecord.errors.project.uname") }
-  validates :maintainer_id, presence: true, unless: :new_record?
+  validates :maintainer, presence: true, unless: :new_record?
   validates :url, presence: true, format: { with: /\Ahttps?:\/\/[\S]+\z/ }, if: :mass_import
-  validates :add_to_repository_id, presence: true, if: :mass_import
+  validates :add_to_repository, presence: true, if: :mass_import
   validates :visibility, presence: true, inclusion: { in: VISIBILITIES }
   validate { errors.add(:base, :can_have_less_or_equal, count: MAX_OWN_PROJECTS) if owner.projects.size >= MAX_OWN_PROJECTS }
   validate :check_default_branch

app/models/project_import.rb

@@ -3,7 +3,7 @@ class ProjectImport < ActiveRecord::Base
   belongs_to :platform
 
   validates :name, uniqueness: { scope: :platform_id, case_sensitive: false }
-  validates :name, :platform_id, :version, presence: true
+  validates :name, :platform, :version, presence: true
 
   scope :by_name, ->(name) { where("#{table_name}.name ILIKE ?", name) }
 

app/models/project_statistic.rb

@@ -3,7 +3,7 @@ class ProjectStatistic < ActiveRecord::Base
   belongs_to :arch
   belongs_to :project
 
-  validates :arch_id, :project_id, :average_build_time, :build_count, presence: true
+  validates :arch, :project, :average_build_time, :build_count, presence: true
   validates :project_id, uniqueness: { scope: :arch_id }
 
   attr_accessible :average_build_time, :build_count

app/models/project_tag.rb

@@ -8,7 +8,7 @@ class ProjectTag < ActiveRecord::Base
 
   belongs_to :project
 
-  validates :project_id, :commit_id, :sha1, :tag_name, :format_id, presence: true
+  validates :project, :commit, :sha1, :tag_name, :format_id, presence: true
   validates :project_id, uniqueness: { scope: [:tag_name, :format_id] }
 
   attr_accessible :project_id, :commit_id, :sha1, :tag_name, :format_id

app/models/repository_status.rb

@@ -28,7 +28,7 @@ class RepositoryStatus < ActiveRecord::Base
   belongs_to :platform
   belongs_to :repository
 
-  validates :repository_id, :platform_id, presence: true
+  validates :repository, :platform, presence: true
   validates :repository_id, uniqueness: { scope: :platform_id }
 
   attr_accessible :platform_id, :repository_id

app/models/settings_notifier.rb

@@ -1,7 +1,7 @@
 class SettingsNotifier < ActiveRecord::Base
   belongs_to :user
 
-  validates :user_id, presence: true
+  validates :user, presence: true
 
   attr_accessible :can_notify,
                   :update_code,
@@ -9,7 +9,7 @@ class SettingsNotifier < ActiveRecord::Base
                   :new_comment_commit_repo_owner,
                   :new_comment_commit_commentor,
                   :new_comment,
-                  :new_comment_reply, 
+                  :new_comment_reply,
                   :new_issue,
                   :issue_assign,
                   :new_build,