typos, backport comment tls-sprint
authorsylvain.thenault@logilab.fr
Wed, 04 Mar 2009 18:49:36 +0100
branchtls-sprint
changeset 1009 bcd0c3ae1935
parent 1008 f54b4309a3f5
child 1010 73591b2ffeb4
typos, backport comment
web/form.py
--- a/web/form.py	Wed Mar 04 17:27:21 2009 +0100
+++ b/web/form.py	Wed Mar 04 18:49:36 2009 +0100
@@ -932,6 +932,11 @@
             # XXX bw compat, default_<field name> on the entity
             warn('found %s_%s_vocabulary on %s, should be set on a specific form'
                  % (role, rtype, self.entity.id), DeprecationWarning)
+        # NOTE: it is the responsibility of `vocabfunc` to sort the result
+        #       (direclty through RQL or via a python sort). This is also
+        #       important because `vocabfunc` might return a list with
+        #       couples (label, None) which act as separators. In these
+        #       cases, it doesn't make sense to sort results afterwards.
         return vocabfunc(rtype)
 ## XXX BACKPORT ME
 ##         if self.sort:
@@ -986,8 +991,7 @@
                             limit=None, done=None):
         if done is None:
             done = set()
-        req = self.req
-        rset = entity.unrelated(rtype, targettype, role, limit)
+        rset = self.entity.unrelated(rtype, targettype, role, limit)
         res = []
         for entity in rset.entities():
             if entity.eid in done:
@@ -1051,7 +1055,7 @@
             enctype = 'multipart/form-data'
         else:
             enctype = 'application/x-www-form-urlencoded'
-        tag = ('<form action="%s" methody="post" id="%s" enctype="%s"' % (
+        tag = ('<form action="%s" method="post" id="%s" enctype="%s"' % (
             html_escape(form.action or '#'), form.domid, enctype))
         if form.onsubmit:
             tag += ' onsubmit="%s"' % html_escape(form.onsubmit)