Changeset 1358 for pykota/trunk/pykota/storage.py

Timestamp:

02/23/04 23:53:21 (20 years ago)

Author:

jalet

Message:

Don't retrieve data when it's not needed, to avoid database queries

Files:

• pykota/trunk/pykota/storage.py (modified) (6 diffs)

pykota/trunk/pykota/storage.py

@@ -22 +22 @@
 #
 # $Log$
+# Revision 1.42  2004/02/23 22:53:21  jalet
+# Don't retrieve data when it's not needed, to avoid database queries
+#
 # Revision 1.41  2004/02/04 17:12:33  jalet
 # Removing a printer from a printers group should work now.
@@ -257 +260 @@
         self.PricePerPage = None
         self.PricePerJob = None
-        self.LastJob = None
-        
+        
+    def __getattr__(self, name) :    
+        """Delays data retrieval until it's really needed."""
+        if name == "LastJob" : 
+            self.LastJob = self.parent.getPrinterLastJob(self)
+            return self.LastJob
+        else :
+            raise AttributeError, name
+            
     def addJobToHistory(self, jobid, user, pagecounter, action, jobsize=None, jobprice=None, filename=None, title=None, copies=None, options=None) :
         """Adds a job to the printer's history."""
@@ -309 +319 @@
         self.HardLimit = None
         self.DateLimit = None
-        self.ParentPrintersUserPQuota = (user.Exists and printer.Exists and parent.getParentPrintersUserPQuota(self)) or []
+        
+    def __getattr__(self, name) :    
+        """Delays data retrieval until it's really needed."""
+        if name == "ParentPrintersUserPQuota" : 
+            self.ParentPrintersUserPQuota = (self.User.Exists and self.Printer.Exists and self.parent.getParentPrintersUserPQuota(self)) or []
+            return self.ParentPrintersUserPQuota
+        else :
+            raise AttributeError, name
         
     def setDateLimit(self, datelimit) :    
@@ -383 +400 @@
     def __init__(self, parent) :
         StorageObject.__init__(self, parent)
-        self.User = None
-        self.Printer = None
+        self.UserName = None
+        self.PrinterName = None
         self.JobId = None
         self.PrinterPageCounter = None
@@ -396 +413 @@
         self.JobOptions = None
         
+    def __getattr__(self, name) :    
+        """Delays data retrieval until it's really needed."""
+        if name == "User" : 
+            self.User = self.parent.getUser(self.UserName)
+            return self.User
+        elif name == "Printer" :    
+            self.Printer = self.parent.getPrinter(self.PrinterName)
+            return self.Printer
+        else :
+            raise AttributeError, name
+        
 class StorageLastJob(StorageJob) :
     """Printer's Last Job class."""
@@ -401 +429 @@
         StorageJob.__init__(self, parent)
         self.Printer = printer
+        self.UserName = None
+        
+    def __getattr__(self, name) :    
+        """Delays data retrieval until it's really needed."""
+        if name == "User" : 
+            self.User = self.parent.getUser(self.UserName)
+            return self.User
+        else :    
+            raise AttributeError, name
         
     def setSize(self, jobsize) :