paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-11-04 14:03:30 +00:00
Code Issues Packages Projects Releases Wiki Activity

user_groups: Rename is_member_of function to is_direct_member_of.

Browse Source
This commit is contained in:
Sahil Batra
2022-05-04 17:04:25 +05:30
committed by Tim Abbott
parent a5b3510cb3
commit 6bca2453cf
6 changed files with 12 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
frontend_tests/node_tests/settings_user_groups.js
View File

@@ -55,7 +55,7 @@ test_ui("can_edit", () => {
assert.ok(!settings_user_groups.can_edit(1)); assert.ok(!settings_user_groups.can_edit(1));
settings_data.user_can_edit_user_groups = () => true; settings_data.user_can_edit_user_groups = () => true;
user_groups.is_member_of = (group_id, user_id) => { user_groups.is_direct_member_of = (group_id, user_id) => {
assert.equal(group_id, 1); assert.equal(group_id, 1);
assert.equal(user_id, undefined); assert.equal(user_id, undefined);
return false; return false;
@@ -71,7 +71,7 @@ test_ui("can_edit", () => {
page_params.is_admin = false; page_params.is_admin = false;
page_params.is_moderator = false; page_params.is_moderator = false;
user_groups.is_member_of = (group_id, user_id) => { user_groups.is_direct_member_of = (group_id, user_id) => {
assert.equal(group_id, 1); assert.equal(group_id, 1);
assert.equal(user_id, undefined); assert.equal(user_id, undefined);
return true; return true;

6
frontend_tests/node_tests/user_groups.js
View File

@@ -90,8 +90,8 @@ run_test("user_groups", () => {
const groups_of_users_nomatch = user_groups.get_user_groups_of_user(user_id_not_in_any_group); const groups_of_users_nomatch = user_groups.get_user_groups_of_user(user_id_not_in_any_group);
assert.equal(groups_of_users_nomatch.length, 0); assert.equal(groups_of_users_nomatch.length, 0);
assert.ok(!user_groups.is_member_of(admins.id, 4)); assert.ok(!user_groups.is_direct_member_of(admins.id, 4));
assert.ok(user_groups.is_member_of(admins.id, 3)); assert.ok(user_groups.is_direct_member_of(admins.id, 3));
user_groups.add_members(all.id, [5, 4]); user_groups.add_members(all.id, [5, 4]);
assert.deepEqual(user_groups.get_user_group_from_id(all.id).members, new Set([1, 2, 3, 5, 4])); assert.deepEqual(user_groups.get_user_group_from_id(all.id).members, new Set([1, 2, 3, 5, 4]));
@@ -119,7 +119,7 @@ run_test("user_groups", () => {
assert.equal(user_groups.get_realm_user_groups().length, 0); assert.equal(user_groups.get_realm_user_groups().length, 0);
blueslip.expect("error", "Could not find user group with ID -1"); blueslip.expect("error", "Could not find user group with ID -1");
assert.equal(user_groups.is_member_of(-1, 15), false); assert.equal(user_groups.is_direct_member_of(-1, 15), false);
blueslip.expect("error", "Could not find user group with ID -9999", 4); blueslip.expect("error", "Could not find user group with ID -9999", 4);
user_groups.add_members(-9999); user_groups.add_members(-9999);

2
static/js/markdown_config.js
View File

@@ -68,7 +68,7 @@ export const get_helpers = () => ({
// user groups // user groups
get_user_group_from_name: (name) => user_group(user_groups.get_user_group_from_name(name)), get_user_group_from_name: (name) => user_group(user_groups.get_user_group_from_name(name)),
is_member_of_user_group: user_groups.is_member_of, is_member_of_user_group: user_groups.is_direct_member_of,
// stream hashes // stream hashes
get_stream_by_name: (name) => stream(stream_data.get_sub(name)), get_stream_by_name: (name) => stream(stream_data.get_sub(name)),

2
static/js/rendered_markdown.js
View File

@@ -113,7 +113,7 @@ export const update_elements = ($content) => {
const my_user_id = people.my_current_user_id(); const my_user_id = people.my_current_user_id();
// Mark user group you're a member of. // Mark user group you're a member of.
if (user_groups.is_member_of(user_group_id, my_user_id)) { if (user_groups.is_direct_member_of(user_group_id, my_user_id)) {
$(this).addClass("user-mention-me"); $(this).addClass("user-mention-me");
} }

2
static/js/settings_user_groups.js
View File

@@ -47,7 +47,7 @@ export function can_edit(group_id) {
return true; return true;
} }
return user_groups.is_member_of(group_id, people.my_current_user_id()); return user_groups.is_direct_member_of(group_id, people.my_current_user_id());
} }
export function populate_user_groups() { export function populate_user_groups() {

6
static/js/user_groups.ts
View File

@@ -79,7 +79,7 @@ export function get_realm_user_groups(): UserGroup[] {
return user_groups.filter((group) => !group.is_system_group); return user_groups.filter((group) => !group.is_system_group);
} }
export function is_member_of(user_group_id: number, user_id: number): boolean { export function is_direct_member_of(user_group_id: number, user_id: number): boolean {
const user_group = user_group_by_id_dict.get(user_group_id); const user_group = user_group_by_id_dict.get(user_group_id);
if (user_group === undefined) { if (user_group === undefined) {
blueslip.error(`Could not find user group with ID ${user_group_id}`); blueslip.error(`Could not find user group with ID ${user_group_id}`);
@@ -148,6 +148,8 @@ export function is_user_group(item: User | UserGroup): item is UserGroup {
export function get_user_groups_of_user(user_id: number): UserGroup[] { export function get_user_groups_of_user(user_id: number): UserGroup[] {
const user_groups_realm = get_realm_user_groups(); const user_groups_realm = get_realm_user_groups();
const groups_of_user = user_groups_realm.filter((group) => is_member_of(group.id, user_id)); const groups_of_user = user_groups_realm.filter((group) =>
is_direct_member_of(group.id, user_id),
);
return groups_of_user; return groups_of_user;
} }