[Zope3-checkins] Re: SVN: Zope3/trunk/src/zope/app/http/ Conform to RFC2616 for PUT requests by returning a Location-header.

[Philipp von Weitershausen]

Wolfgang Schnerring wrote:
> Log message for revision 70412:
>   Conform to RFC2616 for PUT requests by returning a Location-header.
>   
>   This is a merge from branches/ctheune-issue-125, rev. 70404-70411.
>   
> 
> Changed:
>   A   Zope3/trunk/src/zope/app/http/ftests/
>   U   Zope3/trunk/src/zope/app/http/put.py
>   U   Zope3/trunk/src/zope/app/http/tests/test_put.py
> 
> -=-
...
>  
>          # Get a "directory" surrogate for the container
> -        dir = IWriteDirectory(container, None)
> +        # XXX Argh. Why don't we have a unioned Interface for that?!?
> +        dir_write = IWriteDirectory(container)
> +        dir_read = IReadDirectory(container)

What's the purpose of this XXX comment? Usually we only mark things with 
XXX that need immediate fixing. XXX comments are show stoppers for 
releases. If you just want to note a general task, use a TODO comment. I 
think in this case the XXX can simply be removed.