Comment 7 for bug 142731

Revision history for this message
Tres Seaver (tseaver) wrote :

Hmm, that assignment is by definition meaning free: the 'def' statement immediately following it overwrites the
binding of 'None' to the name "get_filepath"!

WRT the intent: 'get_filepath' should probably be
protected with the same permission ("View management
screens") which guards the other "accessor" methods (e.g.,
'body', 'params', 'document_src', etc.

We should also look at breaking up the monolithic
permission declarations, bringing them in line with the
style used by the rest of Zope (permission declarations
immediately prior to the actual method).