From e3e1cf2df730f2dc12882e1f33b2a7ba44647de9 Mon Sep 17 00:00:00 2001 From: Sky Leite Date: Sat, 4 Feb 2023 00:14:28 -0300 Subject: Fix F811 linter rule get_user_group was defined twice, so I merged both definitions --- backend/helpers.py | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) (limited to 'backend') diff --git a/backend/helpers.py b/backend/helpers.py index 0c1ba6b1..e2385506 100644 --- a/backend/helpers.py +++ b/backend/helpers.py @@ -96,11 +96,6 @@ def get_user_owner(file_path) -> str: return pwd.getpwuid(os.stat(file_path).st_uid).pw_name -# Get the user group of the given file path. -def get_user_group(file_path) -> str: - return grp.getgrgid(os.stat(file_path).st_gid).gr_name - - # Deprecated def set_user_group() -> str: return get_user_group() @@ -112,8 +107,11 @@ def get_user_group_id() -> int: # Get the group of the user hosting the plugin loader -def get_user_group() -> str: - return grp.getgrgid(get_user_group_id()).gr_name +def get_user_group(file_path) -> str: + if file_path: + return grp.getgrgid(os.stat(file_path).st_gid).gr_name + else: + return grp.getgrgid(get_user_group_id()).gr_name # Get the default home path unless a user is specified -- cgit v1.2.3