Roundup Tracker - Issues

Issue 565996

classification
The "Attach a File to this Issue" fails
Type: Severity: normal
Components: Web interface Versions:
process
Status: closed fixed
:
: richard : dman13, gmcm, richard
Priority: normal :

Created on 2002-06-07 21:37 by gmcm, last changed 2002-06-11 22:18 by richard.

Messages
msg258 Author: [hidden] (gmcm) Date: 2002-06-07 21:37
The file is uploaded, but it is not attached to the issue.

Using the file widget on the issue form does work. This is
using Mozilla.
msg259 Author: [hidden] (gmcm) Date: 2002-06-10 21:03
Logged In: YES 
user_id=4923

Found it.
cgi_client.Client._post_edit_node, in the ":multilink" branch,
the line:
  value = link.get(nodeid, property)
should be
  value = link.get(nodeid, property)[:]

Otherwise node.set will see it "already" set.
msg260 Author: [hidden] (dman13) Date: 2002-06-11 01:27
Logged In: YES 
user_id=555791

Ok, Goron's patch works if I use the standard CGI interface.

It still fails through Zope.  I did some playing around with
it -- in 2 locations, the form has the empty string as the
key when it should be :multilink.  The current code bombs
(IndexError) when the empty string is found.  Any ideas on
why the form gets hosed by zope/zroundup?
msg261 Author: [hidden] (richard) Date: 2002-06-11 06:36
Logged In: YES 
user_id=6405

Patch applied to CVS, thanks.   
   
Sorry, but I don't think that'll ever work in Zope. You can 
make it work by forcing the form to be a GET form, but by the 
time the form contents get to ZRoundup in a POST, the ':foo'   
parameters are swallowed whole by Zope's ':type' argument   
magic handling.   
 
msg262 Author: [hidden] (gmcm) Date: 2002-06-11 15:23
Logged In: YES 
user_id=4923

Found it.
cgi_client.Client._post_edit_node, in the ":multilink" branch,
the line:
  value = link.get(nodeid, property)
should be
  value = link.get(nodeid, property)[:]

Otherwise node.set will see it "already" set.
msg263 Author: [hidden] (richard) Date: 2002-06-11 22:18
Logged In: YES 
user_id=6405

I've applied that patch already. 
History
Date User Action Args
2002-06-07 21:37:33gmcmcreate