Skip to content

Commit c86306a

Browse files
committed
fix: Common user folder permission
1 parent 3c7e5ff commit c86306a

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

apps/application/flow/step_node/variable_aggregation_node/impl/base_variable_aggregation_node.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,15 +25,15 @@ def get_first_non_null(self, variable_list):
2525
v = self.workflow_manage.get_reference_field(
2626
variable.get('variable')[0],
2727
variable.get('variable')[1:])
28-
if v is not None and not(isinstance(v, (str,list,dict)) and len(v) == 0) :
28+
if v is not None and not (isinstance(v, (str, list, dict)) and len(v) == 0):
2929
return v
3030
return None
3131

3232
def set_variable_to_json(self, variable_list):
3333

34-
return {variable.get('variable')[1:][0]: self.workflow_manage.get_reference_field(
34+
return [self.workflow_manage.get_reference_field(
3535
variable.get('variable')[0],
36-
variable.get('variable')[1:]) for variable in variable_list}
36+
variable.get('variable')[1:]) for variable in variable_list]
3737

3838
def reset_variable(self, variable):
3939
value = self.workflow_manage.get_reference_field(

apps/folders/views/folder.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ class FolderView(APIView):
4545
),
4646
lambda r, kwargs: ViewPermission([RoleConstants.USER.get_workspace_role()],
4747
[Permission(group=Group(f"{kwargs.get('source')}_FOLDER"),
48-
operate=Operate.SELF,
48+
operate=Operate.EDIT,
4949
resource_path=f"/WORKSPACE/{kwargs.get('workspace_id')}/{kwargs.get('source')}/{r.data.get('parent_id')}"
5050
)], CompareConstants.AND),
5151
RoleConstants.WORKSPACE_MANAGE.get_workspace_role()
@@ -107,7 +107,7 @@ class Operate(APIView):
107107
),
108108
lambda r, kwargs: ViewPermission([RoleConstants.USER.get_workspace_role()],
109109
[Permission(group=Group(f"{kwargs.get('source')}_FOLDER"),
110-
operate=Operate.SELF,
110+
operate=Operate.EDIT,
111111
resource_path=f"/WORKSPACE/{kwargs.get('workspace_id')}/{kwargs.get('source')}/{kwargs.get('folder_id')}"
112112
)], CompareConstants.AND),
113113
RoleConstants.WORKSPACE_MANAGE.get_workspace_role()
@@ -159,7 +159,7 @@ def get(self, request: Request, workspace_id: str, source: str, folder_id: str):
159159
),
160160
lambda r, kwargs: ViewPermission([RoleConstants.USER.get_workspace_role()],
161161
[Permission(group=Group(f"{kwargs.get('source')}_FOLDER"),
162-
operate=Operate.SELF,
162+
operate=Operate.EDIT,
163163
resource_path=f"/WORKSPACE/{kwargs.get('workspace_id')}/{kwargs.get('source')}/{kwargs.get('folder_id')}"
164164
)], CompareConstants.AND),
165165
RoleConstants.WORKSPACE_MANAGE.get_workspace_role()

0 commit comments

Comments
 (0)