cst1/openstreetmap-website
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Rubocop autofix: ambiguous operator precedence

Browse source 
This simply adds braces to clarify which mathematical operator comes
first.
This commit is contained in:
Andy Allan 2022-04-06 14:55:29 +01:00
parent e103620ff4
commit 561ee71129
9 changed files with 41 additions and 54 deletions
Download patch file Download diff file Expand all files Collapse all files

4
app/models/user.rb
View file

@ -370,7 +370,7 @@ class User < ApplicationRecord
account_age_in_hours = account_age_in_seconds / 3600
recent_messages = messages.where("sent_on >= ?", Time.now.utc - 3600).count
active_reports = issues.with_status(:open).sum(:reports_count)
max_messages = account_age_in_hours.ceil + recent_messages - active_reports * 10
max_messages = account_age_in_hours.ceil + recent_messages - (active_reports * 10)
max_messages.clamp(0, Settings.max_messages_per_hour)
end
@ -379,7 +379,7 @@ class User < ApplicationRecord
account_age_in_hours = account_age_in_seconds / 3600
recent_friends = Friendship.where(:befriendee => self).where("created_at >= ?", Time.now.utc - 3600).count
active_reports = issues.with_status(:open).sum(:reports_count)
max_friends = account_age_in_hours.ceil + recent_friends - active_reports * 10
max_friends = account_age_in_hours.ceil + recent_friends - (active_reports * 10)
max_friends.clamp(0, Settings.max_friends_per_hour)
end