FIX: do not show private group flair on user avatars (#13872)

Meta ref: https://meta.discourse.org/t/visible-flair-for-invisible-groups-is-that-on-purpose/167674

app/models/group.rb

@@ -803,6 +803,8 @@ class Group < ActiveRecord::Base
   end
 
   def flair_url
+    if members_visibility_level == Group.visibility_levels[:public] &&
+      visibility_level == Group.visibility_levels[:public]
       case flair_type
       when :icon
         flair_icon
@@ -811,6 +813,9 @@ class Group < ActiveRecord::Base
       else
         nil
       end
+    else
+      nil
+    end
   end
 
   [:muted, :regular, :tracking, :watching, :watching_first_post].each do |level|

app/serializers/site_serializer.rb

@@ -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)
+        .select(:id, :name, :flair_icon, :flair_upload_id, :flair_bg_color, :flair_color, :visibility_level, :members_visibility_level)
         .map do |g|
           {
             id: g.id,
@@ -71,6 +71,8 @@ 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

lib/user_lookup.rb

@@ -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}
+    @group_lookup_columns ||= %i{id name flair_icon flair_upload_id flair_bg_color flair_color visibility_level members_visibility_level}
   end
 
   def initialize(user_ids = [])

spec/components/user_lookup_spec.rb

@@ -53,4 +53,32 @@ 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", 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")
+    end
+  end
 end

spec/models/group_spec.rb

@@ -1307,4 +1307,12 @@ 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", visibility_level: Group.visibility_levels[:public], members_visibility_level: Group.visibility_levels[:public])
+    private_group = Fabricate(:group, flair_icon: "icon", 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)
+  end
 end

spec/requests/groups_controller_spec.rb

@@ -725,7 +725,7 @@ describe GroupsController do
 
         expect(group.flair_bg_color).to eq('FFF')
         expect(group.flair_color).to eq('BBB')
-        expect(group.flair_url).to eq('fa-adjust')
+        expect(group.flair_url).to eq(nil)
         expect(group.bio_raw).to eq('testing')
         expect(group.full_name).to eq('awesome team')
         expect(group.public_admission).to eq(true)