about summary refs log tree commit diff
path: root/app
diff options
context:
space:
mode:
authorZac <thezacattacks@gmail.com>2019-04-28 20:10:30 -0400
committerThibG <thib@sitedethib.com>2019-04-29 20:52:53 +0200
commit5616200ed415fabb8d31eb5257eec7525481b819 (patch)
tree75104dc8c50e572a00bf8764d1d1e2dd3f893a14 /app
parenta36affb9d6d556460e282d9acd206609825f3a78 (diff)
added documentation into .env.production.sample
moved max pin count into constant
Diffstat (limited to 'app')
-rw-r--r--app/validators/status_pin_validator.rb4
1 files changed, 3 insertions, 1 deletions
diff --git a/app/validators/status_pin_validator.rb b/app/validators/status_pin_validator.rb
index 01796f77b..c88a9e5d3 100644
--- a/app/validators/status_pin_validator.rb
+++ b/app/validators/status_pin_validator.rb
@@ -1,10 +1,12 @@
 # frozen_string_literal: true
 
 class StatusPinValidator < ActiveModel::Validator
+  MAX_PINNED = (ENV['MAX_PINNED_TOOTS'].to_i - 1 || 4)
+  
   def validate(pin)
     pin.errors.add(:base, I18n.t('statuses.pin_errors.reblog')) if pin.status.reblog?
     pin.errors.add(:base, I18n.t('statuses.pin_errors.ownership')) if pin.account_id != pin.status.account_id
     pin.errors.add(:base, I18n.t('statuses.pin_errors.private')) unless %w(public unlisted).include?(pin.status.visibility)
-    pin.errors.add(:base, I18n.t('statuses.pin_errors.limit')) if pin.account.status_pins.count >= (ENV['MAX_PINNED_TOOTS'].to_i || 5) && pin.account.local?
+    pin.errors.add(:base, I18n.t('statuses.pin_errors.limit')) if pin.account.status_pins.count > MAX_PINNED  && pin.account.local?
   end
 end