In article <[EMAIL PROTECTED]> you write:
> 
>         # Untrusted data *after* trusted data
>         v = self.form.get(key, _marker)
>         if v is not _marker:
>             other[key] = v                      # *boom*
>             return v
> 
> That magical promotion of the key & value to the other dictionary is
> what tripped me up.

Wouldn't
              other.setdefault(key, v)
be better? 
So a variable already existing in other wouldn't get clobbered.

Florent

-- 
Florent Guillaume, Nuxeo (Paris, France)
+33 1 40 33 79 87  http://nuxeo.com  mailto:[EMAIL PROTECTED]

_______________________________________________
Zope-Dev maillist  -  [EMAIL PROTECTED]
http://mail.zope.org/mailman/listinfo/zope-dev
**  No cross posts or HTML encoding!  **
(Related lists - 
 http://mail.zope.org/mailman/listinfo/zope-announce
 http://mail.zope.org/mailman/listinfo/zope )

Reply via email to