mirror of
				https://github.com/zulip/zulip.git
				synced 2025-11-04 05:53:43 +00:00 
			
		
		
		
	events: Include all active streams in initial state data for admins.
Realm administrators already get creation and deletion events for all streams, including private streams. So these should be reflected in the initial state data. Signed-off-by: Anders Kaseorg <anders@zulip.com>
This commit is contained in:
		
				
					committed by
					
						
						Tim Abbott
					
				
			
			
				
	
			
			
			
						parent
						
							1de9444242
						
					
				
				
					commit
					8bd0ec2fca
				
			@@ -401,7 +401,9 @@ def fetch_initial_state_data(
 | 
				
			|||||||
            # it uses data from state["subscriptions"] and other
 | 
					            # it uses data from state["subscriptions"] and other
 | 
				
			||||||
            # places.
 | 
					            # places.
 | 
				
			||||||
            if user_profile is not None:
 | 
					            if user_profile is not None:
 | 
				
			||||||
                state["streams"] = do_get_streams(user_profile)
 | 
					                state["streams"] = do_get_streams(
 | 
				
			||||||
 | 
					                    user_profile, include_all_active=user_profile.is_realm_admin
 | 
				
			||||||
 | 
					                )
 | 
				
			||||||
            else:
 | 
					            else:
 | 
				
			||||||
                # TODO: This line isn't used by the webapp because it
 | 
					                # TODO: This line isn't used by the webapp because it
 | 
				
			||||||
                # gets these data via the `subscriptions` key; it will
 | 
					                # gets these data via the `subscriptions` key; it will
 | 
				
			||||||
@@ -602,7 +604,9 @@ def apply_event(
 | 
				
			|||||||
                        state["never_subscribed"] = sub_info.never_subscribed
 | 
					                        state["never_subscribed"] = sub_info.never_subscribed
 | 
				
			||||||
 | 
					
 | 
				
			||||||
                    if "streams" in state:
 | 
					                    if "streams" in state:
 | 
				
			||||||
                        state["streams"] = do_get_streams(user_profile)
 | 
					                        state["streams"] = do_get_streams(
 | 
				
			||||||
 | 
					                            user_profile, include_all_active=user_profile.is_realm_admin
 | 
				
			||||||
 | 
					                        )
 | 
				
			||||||
 | 
					
 | 
				
			||||||
                for field in ["delivery_email", "email", "full_name"]:
 | 
					                for field in ["delivery_email", "email", "full_name"]:
 | 
				
			||||||
                    if field in person and field in state:
 | 
					                    if field in person and field in state:
 | 
				
			||||||
 
 | 
				
			|||||||
		Reference in New Issue
	
	Block a user