Performance: prioritize raw getter for AllowedMethods field (senaite#2149)

Co-authored-by: Ramon Bartl <[email protected]>

Author: xispa

CHANGES.rst

@@ -5,6 +5,7 @@ Changelog
 2.3.0 (unreleased)
 ------------------
 
+- #2150 Performance: prioritize raw getter for AllowedMethods field
 - #2148 Performance: prioritize raw getter for AllowedInstruments field
 - #2147 Remove stale function workflow.getReviewHistory
 - #2146 Fix "No object found for UID: <laboratory_uid>" in report preview

src/bika/lims/browser/analyses/view.py

@@ -1516,11 +1516,11 @@ def is_method_required(self, analysis):
         """
         # Always return true if the analysis has a method assigned
         obj = self.get_object(analysis)
-        method = obj.getMethod()
+        method = obj.getRawMethod()
         if method:
             return True
 
-        methods = obj.getAllowedMethods()
+        methods = obj.getRawAllowedMethods()
         return len(methods) > 0
 
     def is_instrument_required(self, analysis):

src/bika/lims/content/abstractanalysis.py

@@ -739,7 +739,7 @@ def isMethodAllowed(self, method):
         :rtype: bool
         """
         uid = api.get_uid(method)
-        return uid in map(api.get_uid, self.getAllowedMethods())
+        return uid in self.getRawAllowedMethods()
 
     @security.public
     def getAllowedMethods(self):
@@ -756,6 +756,15 @@ def getAllowedMethods(self):
         # get the available methods of the service
         return service.getMethods()
 
+    @security.public
+    def getRawAllowedMethods(self):
+        """Returns the UIDs of the allowed methods for this analysis
+        """
+        service = self.getAnalysisService()
+        if not service:
+            return []
+        return service.getRawMethods()
+
     @security.public
     def getAllowedInstruments(self):
         """Returns the allowed instruments from the service

src/bika/lims/content/analysisservice.py

@@ -315,17 +315,14 @@ def getMethods(self):
 
         :returns: List of method objects
         """
-        field = self.getField("Methods")
-        methods = field.get(self)
-        return methods
+        return self.getField("Methods").get(self)
 
     def getRawMethods(self):
         """Returns the assigned method UIDs
 
         :returns: List of method UIDs
         """
-        methods = self.getMethods()
-        return map(api.get_uid, methods)
+        return self.getField("Methods").getRaw(self)
 
     def getMethod(self):
         """Get the default method