Skip to content

Commit

Permalink
Merge pull request DIRACGrid#7200 from DIRACGridBot/cherry-pick-2-bf4…
Browse files Browse the repository at this point in the history
…30ff63-integration

[sweep:integration] Drop TS permission check for all read-only functions
  • Loading branch information
chrisburr authored Sep 14, 2023
2 parents df6a89f + 0534f9c commit 208bb79
Showing 1 changed file with 0 additions and 18 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -207,15 +207,11 @@ def export_getTransformation(self, transName, extraParams=False):
tfDetails = self.transformationDB.getTransformation(transName, extraParams=extraParams)
if not tfDetails["OK"]:
return tfDetails
if not (result := self.checkPermissions(transName))["OK"]:
return result
return tfDetails

types_getTransformationParameters = [[int, str], [str, list]]

def export_getTransformationParameters(self, transName, parameters):
if not (result := self.checkPermissions(transName))["OK"]:
return result
return self.transformationDB.getTransformationParameters(transName, parameters)

types_getTransformationWithStatus = [[str, list, tuple]]
Expand Down Expand Up @@ -275,15 +271,11 @@ def export_setFileStatusForTransformation(self, transName, dictOfNewFilesStatus)
types_getTransformationStats = [[int, str]]

def export_getTransformationStats(self, transName):
if not (result := self.checkPermissions(transName))["OK"]:
return result
return self.transformationDB.getTransformationStats(transName)

types_getTransformationFilesCount = [[int, str], str]

def export_getTransformationFilesCount(self, transName, field, selection={}):
if not (result := self.checkPermissions(transName))["OK"]:
return result
return self.transformationDB.getTransformationFilesCount(transName, field, selection=selection)

types_getTransformationFiles = []
Expand Down Expand Up @@ -360,8 +352,6 @@ def export_setTaskStatusAndWmsID(self, transName, taskID, status, taskWmsID):
types_getTransformationTaskStats = [[int, str]]

def export_getTransformationTaskStats(self, transName):
if not (result := self.checkPermissions(transName))["OK"]:
return result
return self.transformationDB.getTransformationTaskStats(transName)

types_deleteTasks = [[int, str], int, int]
Expand All @@ -386,8 +376,6 @@ def export_extendTransformation(self, transName, nTasks):

def export_getTasksToSubmit(self, transName, numTasks, site=""):
"""Get information necessary for submission for a given number of tasks for a given transformation"""
if not (result := self.checkPermissions(transName))["OK"]:
return result
res = self.transformationDB.getTransformation(transName)
if not res["OK"]:
return res
Expand Down Expand Up @@ -435,8 +423,6 @@ def export_deleteTransformationMetaQuery(self, transName, queryType):
types_getTransformationMetaQuery = [[int, str], str]

def export_getTransformationMetaQuery(self, transName, queryType):
if not (result := self.checkPermissions(transName))["OK"]:
return result
return self.transformationDB.getTransformationMetaQuery(transName, queryType)

####################################################################
Expand All @@ -447,8 +433,6 @@ def export_getTransformationMetaQuery(self, transName, queryType):
types_getTransformationLogging = [[int, str]]

def export_getTransformationLogging(self, transName):
if not (result := self.checkPermissions(transName))["OK"]:
return result
return self.transformationDB.getTransformationLogging(transName)

####################################################################
Expand All @@ -459,8 +443,6 @@ def export_getTransformationLogging(self, transName):
types_getAdditionalParameters = [[int, str]]

def export_getAdditionalParameters(self, transName):
if not (result := self.checkPermissions(transName))["OK"]:
return result
return self.transformationDB.getAdditionalParameters(transName)

####################################################################
Expand Down

0 comments on commit 208bb79

Please sign in to comment.