REVERT "FIX: do not show private group flair on user avatars" (#13991)

This reverts commit fe3e18f981 and 0d8fd9ace6
This commit is contained in:
Arpit Jalan 2021-08-10 17:25:11 +05:30 committed by GitHub
parent 4ca34940d3
commit 3006de39d1
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
6 changed files with 8 additions and 59 deletions

View File

@ -803,22 +803,12 @@ class Group < ActiveRecord::Base
end
def flair_url
return if members_visibility_level != Group.visibility_levels[:public]
return if visibility_level != Group.visibility_levels[:public]
return flair_icon if flair_type == :icon
return upload_cdn_path(flair_upload.url) if flair_type == :image
nil
end
def flair_bg_color
return if members_visibility_level != Group.visibility_levels[:public]
return if visibility_level != Group.visibility_levels[:public]
read_attribute(:flair_bg_color)
end
[:muted, :regular, :tracking, :watching, :watching_first_post].each do |level|
define_method("#{level}_category_ids=") do |category_ids|
@category_notifications ||= {}

View File

@ -63,7 +63,7 @@ class SiteSerializer < ApplicationSerializer
def groups
cache_anon_fragment("group_names") do
object.groups.order(:name)
.select(:id, :name, :flair_icon, :flair_upload_id, :flair_bg_color, :flair_color, :visibility_level, :members_visibility_level)
.select(:id, :name, :flair_icon, :flair_upload_id, :flair_bg_color, :flair_color)
.map do |g|
{
id: g.id,
@ -71,8 +71,6 @@ class SiteSerializer < ApplicationSerializer
flair_url: g.flair_url,
flair_bg_color: g.flair_bg_color,
flair_color: g.flair_color,
visibility_level: g.visibility_level,
members_visibility_level: g.members_visibility_level,
}
end.as_json
end

View File

@ -6,7 +6,7 @@ class UserLookup
end
def self.group_lookup_columns
@group_lookup_columns ||= %i{id name flair_icon flair_upload_id flair_bg_color flair_color visibility_level members_visibility_level}
@group_lookup_columns ||= %i{id name flair_icon flair_upload_id flair_bg_color flair_color}
end
def initialize(user_ids = [])

View File

@ -53,33 +53,4 @@ describe UserLookup do
expect(user_lookup_group.name).to eq("testgroup")
end
end
describe '#flair_groups' do
fab!(:group) { Fabricate(:group, name: "flair_group", flair_icon: "icon", flair_bg_color: "40E0D0", visibility_level: Group.visibility_levels[:public], members_visibility_level: Group.visibility_levels[:public]) }
fab!(:user2) { Fabricate(:user, flair_group: group) }
before do
@user_lookup = UserLookup.new([user.id, user2.id, nil])
end
it 'returns nil if user_id does not exists' do
expect(@user_lookup.flair_groups[0]).to eq(nil)
end
it 'returns nil if user_id is nil' do
expect(@user_lookup.flair_groups[nil]).to eq(nil)
end
it 'returns nil if user has no flair group' do
expect(@user_lookup.flair_groups[user.id]).to eq(nil)
end
it 'returns group if user has flair group' do
user_lookup_group = @user_lookup.flair_groups[user2.id]
expect(user_lookup_group).to eq(group)
expect(user_lookup_group.name).to eq("flair_group")
expect(user_lookup_group.flair_url).to eq("icon")
expect(user_lookup_group.flair_bg_color).to eq("40E0D0")
end
end
end

View File

@ -1308,14 +1308,4 @@ describe Group do
expect(Group.find_by_email("nope@test.com")).to eq(nil)
end
end
it "fetches flair_url based on group visibility" do
public_group = Fabricate(:group, flair_icon: "icon", flair_bg_color: "40E0D0", visibility_level: Group.visibility_levels[:public], members_visibility_level: Group.visibility_levels[:public])
private_group = Fabricate(:group, flair_icon: "icon", flair_bg_color: "40E0D0", visibility_level: Group.visibility_levels[:logged_on_users], members_visibility_level: Group.visibility_levels[:public])
expect(public_group.flair_url).to eq("icon")
expect(private_group.flair_url).to eq(nil)
expect(public_group.flair_bg_color).to eq("40E0D0")
expect(private_group.flair_bg_color).to eq(nil)
end
end

View File

@ -723,9 +723,9 @@ describe GroupsController do
group.reload
expect(group.flair_bg_color).to eq(nil)
expect(group.flair_bg_color).to eq('FFF')
expect(group.flair_color).to eq('BBB')
expect(group.flair_url).to eq(nil)
expect(group.flair_url).to eq('fa-adjust')
expect(group.bio_raw).to eq('testing')
expect(group.full_name).to eq('awesome team')
expect(group.public_admission).to eq(true)
@ -834,10 +834,10 @@ describe GroupsController do
expect(response.status).to eq(200)
group.reload
expect(group.flair_bg_color).to eq(nil)
expect(group.flair_bg_color).to eq('FFF')
expect(group.flair_color).to eq('BBB')
expect(group.flair_icon).to eq('fa-adjust')
expect(group.flair_url).to eq(nil)
expect(group.flair_url).to eq('fa-adjust')
expect(group.name).to eq('admins')
expect(group.visibility_level).to eq(1)
expect(group.mentionable_level).to eq(1)
@ -1006,10 +1006,10 @@ describe GroupsController do
expect(response.status).to eq(200)
group.reload
expect(group.flair_bg_color).to eq(nil)
expect(group.flair_bg_color).to eq('FFF')
expect(group.flair_color).to eq('BBB')
expect(group.flair_icon).to eq('fa-adjust')
expect(group.flair_url).to eq(nil)
expect(group.flair_url).to eq('fa-adjust')
expect(group.name).to eq('trust_level_4')
expect(group.mentionable_level).to eq(1)
expect(group.messageable_level).to eq(1)