Use && and || consistently so that there's less chance of copy paste errors in the future
This commit is contained in:
parent
22e3db703e
commit
a9a585f66a
|
@ -169,7 +169,7 @@ class AdminDashboardData
|
||||||
|
|
||||||
def sidekiq_check
|
def sidekiq_check
|
||||||
last_job_performed_at = Jobs.last_job_performed_at
|
last_job_performed_at = Jobs.last_job_performed_at
|
||||||
I18n.t('dashboard.sidekiq_warning') if Jobs.queued > 0 and (last_job_performed_at.nil? or last_job_performed_at < 2.minutes.ago)
|
I18n.t('dashboard.sidekiq_warning') if Jobs.queued > 0 && (last_job_performed_at.nil? || last_job_performed_at < 2.minutes.ago)
|
||||||
end
|
end
|
||||||
|
|
||||||
def queue_size_check
|
def queue_size_check
|
||||||
|
@ -178,7 +178,7 @@ class AdminDashboardData
|
||||||
end
|
end
|
||||||
|
|
||||||
def ram_check
|
def ram_check
|
||||||
I18n.t('dashboard.memory_warning') if MemInfo.new.mem_total and MemInfo.new.mem_total < 1_000_000
|
I18n.t('dashboard.memory_warning') if MemInfo.new.mem_total && MemInfo.new.mem_total < 1_000_000
|
||||||
end
|
end
|
||||||
|
|
||||||
def google_oauth2_config_check
|
def google_oauth2_config_check
|
||||||
|
@ -190,23 +190,23 @@ class AdminDashboardData
|
||||||
end
|
end
|
||||||
|
|
||||||
def twitter_config_check
|
def twitter_config_check
|
||||||
I18n.t('dashboard.twitter_config_warning') if SiteSetting.enable_twitter_logins and (SiteSetting.twitter_consumer_key.blank? or SiteSetting.twitter_consumer_secret.blank?)
|
I18n.t('dashboard.twitter_config_warning') if SiteSetting.enable_twitter_logins && (SiteSetting.twitter_consumer_key.blank? || SiteSetting.twitter_consumer_secret.blank?)
|
||||||
end
|
end
|
||||||
|
|
||||||
def github_config_check
|
def github_config_check
|
||||||
I18n.t('dashboard.github_config_warning') if SiteSetting.enable_github_logins and (SiteSetting.github_client_id.blank? or SiteSetting.github_client_secret.blank?)
|
I18n.t('dashboard.github_config_warning') if SiteSetting.enable_github_logins && (SiteSetting.github_client_id.blank? || SiteSetting.github_client_secret.blank?)
|
||||||
end
|
end
|
||||||
|
|
||||||
def s3_config_check
|
def s3_config_check
|
||||||
bad_keys = (SiteSetting.s3_access_key_id.blank? || SiteSetting.s3_secret_access_key.blank?) && !SiteSetting.s3_use_iam_profile
|
bad_keys = (SiteSetting.s3_access_key_id.blank? || SiteSetting.s3_secret_access_key.blank?) && !SiteSetting.s3_use_iam_profile
|
||||||
|
|
||||||
return I18n.t('dashboard.s3_config_warning') if SiteSetting.enable_s3_uploads and (bad_keys or SiteSetting.s3_upload_bucket.blank?)
|
return I18n.t('dashboard.s3_config_warning') if SiteSetting.enable_s3_uploads && (bad_keys || SiteSetting.s3_upload_bucket.blank?)
|
||||||
return I18n.t('dashboard.s3_backup_config_warning') if SiteSetting.enable_s3_backups and (bad_keys or SiteSetting.s3_backup_bucket.blank?)
|
return I18n.t('dashboard.s3_backup_config_warning') if SiteSetting.enable_s3_backups && (bad_keys || SiteSetting.s3_backup_bucket.blank?)
|
||||||
nil
|
nil
|
||||||
end
|
end
|
||||||
|
|
||||||
def image_magick_check
|
def image_magick_check
|
||||||
I18n.t('dashboard.image_magick_warning') if SiteSetting.create_thumbnails and !system("command -v convert >/dev/null;")
|
I18n.t('dashboard.image_magick_warning') if SiteSetting.create_thumbnails && !system("command -v convert >/dev/null;")
|
||||||
end
|
end
|
||||||
|
|
||||||
def failing_emails_check
|
def failing_emails_check
|
||||||
|
|
Loading…
Reference in New Issue