another fix to indexargs

Author: Richard Jones

CHANGES.txt

@@ -59,6 +59,7 @@ Fixed:
 - removed safeget() from the API (sf bug 994750)
 - demo tracker is always set up on localhost (sf bug 1049101)
 
+
 2004-10-06 0.7.11
 Fixed:
 - index args URL generation broken in .10 (sf bug 1096027)

roundup/cgi/templating.py

@@ -2019,7 +2019,7 @@ def indexargs_form(self, columns=1, sort=1, group=1, filter=1,
             l.append(s%(sc+'group', val))
         if filter and self.filter:
             l.append(s%(sc+'filter', ','.join(self.filter)))
-        if filterspec:
+        if self.classname and filterspec:
             props = self.client.db.getclass(self.classname).getprops()
             for k,v in self.filterspec.items():
                 if type(v) == type([]):
@@ -2072,17 +2072,18 @@ def indexargs_url(self, url, args):
             l.append(sc+'startwith=%s'%self.startwith)
 
         # finally, the remainder of the filter args in the request
-        props = self.client.db.getclass(self.classname).getprops()
-        q = urllib.quote
-        for k,v in self.filterspec.items():
-            if not args.has_key(k):
-                if type(v) == type([]):
-                    if isinstance(props[k], hyperdb.String):
-                        l.append('%s=%s'%(k, '%20'.join([q(i) for i in v])))
+        if self.classname and self.filterspec:
+            props = self.client.db.getclass(self.classname).getprops()
+            q = urllib.quote
+            for k,v in self.filterspec.items():
+                if not args.has_key(k):
+                    if type(v) == type([]):
+                        if isinstance(props[k], hyperdb.String):
+                            l.append('%s=%s'%(k, '%20'.join([q(i) for i in v])))
+                        else:
+                            l.append('%s=%s'%(k, ','.join([q(i) for i in v])))
                     else:
-                        l.append('%s=%s'%(k, ','.join([q(i) for i in v])))
-                else:
-                    l.append('%s=%s'%(k, q(v)))
+                        l.append('%s=%s'%(k, q(v)))
         return '%s?%s'%(url, '&'.join(l))
     indexargs_href = indexargs_url