diff options
author | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2014-01-30 14:29:28 +0200 |
---|---|---|
committer | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2014-01-30 14:29:28 +0200 |
commit | 52b60a1742ff9207d6d715f37176c81e73d73099 (patch) | |
tree | f49449ac9dcc24c53529886059c657850cb108ab /app/models | |
parent | adb5d0ee198a3f4047e07cd5e5d7d9b45944865c (diff) | |
download | gitlab-ci-52b60a1742ff9207d6d715f37176c81e73d73099.tar.gz |
Fix public scope, use new project field name
Diffstat (limited to 'app/models')
-rw-r--r-- | app/models/build.rb | 2 | ||||
-rw-r--r-- | app/models/project.rb | 8 |
2 files changed, 5 insertions, 5 deletions
diff --git a/app/models/build.rb b/app/models/build.rb index 6977a2a..bf8b388 100644 --- a/app/models/build.rb +++ b/app/models/build.rb @@ -80,7 +80,7 @@ class Build < ActiveRecord::Base project = build.project if project.email_notification? - if build.status.to_sym == :failed || !project.email_all_broken_builds + if build.status.to_sym == :failed || !project.email_only_broken_builds NotificationService.new.build_ended(build) end end diff --git a/app/models/project.rb b/app/models/project.rb index 80a5b8a..b916c9f 100644 --- a/app/models/project.rb +++ b/app/models/project.rb @@ -19,14 +19,14 @@ # allow_git_fetch :boolean default(TRUE), not null # email_recipients :string(255) default(""), not null # email_add_committer :boolean default(TRUE), not null -# email_all_broken_builds :boolean default(TRUE), not null +# email_only_broken_builds :boolean default(TRUE), not null # class Project < ActiveRecord::Base attr_accessible :name, :path, :scripts, :timeout, :token, :default_ref, :gitlab_url, :always_build, :polling_interval, :public, :ssh_url_to_repo, :gitlab_id, :allow_git_fetch, - :email_recipients, :email_add_committer, :email_all_broken_builds + :email_recipients, :email_add_committer, :email_only_broken_builds has_many :builds, dependent: :destroy has_many :runner_projects, dependent: :destroy @@ -44,7 +44,7 @@ class Project < ActiveRecord::Base if: ->(project) { project.always_build.present? } - scope :public, ->(_) { where(public: true) } + scope :public_only, ->() { where(public: true) } before_validation :set_default_values @@ -60,7 +60,7 @@ class Project < ActiveRecord::Base default_ref: project.default_branch || 'master', ssh_url_to_repo: project.ssh_url_to_repo, email_add_committer: GitlabCi.config.gitlab_ci.add_committer, - email_all_broken_builds: GitlabCi.config.gitlab_ci.all_broken_builds, + email_only_broken_builds: GitlabCi.config.gitlab_ci.all_broken_builds, } Project.new(params) |