X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=PLC%2FAddresses.py;h=36c0d1dca655087a729f3d18e7c5eac1e6d64936;hb=refs%2Fheads%2Fplanetlab-4_0-branch;hp=030821b1a79dfd2cd8e458e99e8532ad53bf9477;hpb=b0102261e4247755c0cd6bb253f190452941e834;p=plcapi.git

diff --git a/PLC/Addresses.py b/PLC/Addresses.py
index 030821b..36c0d1d 100644
--- a/PLC/Addresses.py
+++ b/PLC/Addresses.py
@@ -1,6 +1,7 @@
 from PLC.Faults import *
 from PLC.Parameter import Parameter
 from PLC.Table import Row, Table
+from PLC.Filter import Filter
 from PLC.AddressTypes import AddressType, AddressTypes
 
 class Address(Row):
@@ -14,15 +15,15 @@ class Address(Row):
     join_tables = ['address_address_type', 'site_address']
     fields = {
         'address_id': Parameter(int, "Address identifier"),
-        'line1': Parameter(str, "Address line 1", max = 254, optional = False),
-        'line2': Parameter(str, "Address line 2", max = 254),
-        'line3': Parameter(str, "Address line 3", max = 254),
-        'city': Parameter(str, "City", max = 254, optional = False),
-        'state': Parameter(str, "State or province", max = 254, optional = False),
-        'postalcode': Parameter(str, "Postal code", max = 64, optional = False),
-        'country': Parameter(str, "Country", max = 128, optional = False),
-        'address_type_ids': Parameter([int], "Address type identifiers", ro = True),
-        'address_types': Parameter([str], "Address types", ro = True),
+        'line1': Parameter(str, "Address line 1", max = 254),
+        'line2': Parameter(str, "Address line 2", max = 254, nullok = True),
+        'line3': Parameter(str, "Address line 3", max = 254, nullok = True),
+        'city': Parameter(str, "City", max = 254),
+        'state': Parameter(str, "State or province", max = 254),
+        'postalcode': Parameter(str, "Postal code", max = 64),
+        'country': Parameter(str, "Country", max = 128),
+        'address_type_ids': Parameter([int], "Address type identifiers"),
+        'address_types': Parameter([str], "Address types"),
         }
 
     def add_address_type(self, address_type, commit = True):
@@ -82,25 +83,17 @@ class Addresses(Table):
     database.
     """
 
-    def __init__(self, api, address_id_list = None):
-	self.api = api
+    def __init__(self, api, address_filter = None, columns = None):
+	Table.__init__(self, api, Address, columns)
 
-        sql = "SELECT %s FROM view_addresses" % \
-              ", ".join(Address.fields)
+        sql = "SELECT %s FROM view_addresses WHERE True" % \
+              ", ".join(self.columns)
 
-        if address_id_list:
-            sql += " WHERE address_id IN (%s)" % ", ".join(map(str, address_id_list))
+        if address_filter is not None:
+            if isinstance(address_filter, (list, tuple, set)):
+                address_filter = Filter(Address.fields, {'address_id': address_filter})
+            elif isinstance(address_filter, dict):
+                address_filter = Filter(Address.fields, address_filter)
+            sql += " AND (%s) %s" % address_filter.sql(api)
 
-        rows = self.api.db.selectall(sql)
-
-        for row in rows:
-            self[row['address_id']] = address = Address(api, row)
-            for aggregate in 'address_type_ids', 'address_types':
-                if not address.has_key(aggregate) or address[aggregate] is None:
-                    address[aggregate] = []
-                else:
-                    elements = address[aggregate].split(',')
-                    try:
-                        address[aggregate] = map(int, elements)
-                    except ValueError:
-                        address[aggregate] = elements
+        self.selectall(sql)