[Catalyst] Catalyst::Request::Upload - uploadtmp

[Tobias Kremer]

> On Jun 8, 2007, at 10:09 AM, Tobias Kremer wrote:
> > The manpage of Catalyst::Request::Upload suggests the following:
> > --snip--
> > To specify where Catalyst should put the temporary files, set the
> > 'uploadtmp'
> > option in the Catalyst config. If unset, Catalyst will use the
> > system temp dir.
> > __PACKAGE__->config( uploadtmp =3D> '/path/to/tmpdir' );
> > --snip--
> > Unfortunately this doesn't work for me (all file uploads are placed
> > in the
> > system default location, e.g. /tmp).
> > Now, there's this in Catalyst::Engine:
> > #321: $c->request->{_body}->{tmpdir} =3D $c->config->{uploadtmp}
> > But HTTP::Body::MultiPart doesn't consider $self->{tmpdir} in its
> > call to
> > File::Temp. Looks like tmpdir isn't used at all and really should
> > be added
> > as an object method to avoid poking in the HTTP::Body object.
> > Have I misunderstood the purpose of uploadtmp? If not, I'd be happy to
> > provide patches for HTTP::Body::MultiPart and Catalyst::Engine.
>
> This sounds like a bug in HTTP::Body, a patch would be great, thanks!
>
> -Andy

Patch attached. Hope it's okay.

--Tobias
-------------- next part --------------
A non-text attachment was scrubbed...
Name: HTTP_Body_MultiPart.patch
Type: text/x-patch
Size: 381 bytes
Desc: not available
Url : http://lists.scsys.co.uk/pipermail/catalyst/attachments/20070608/b7bc=
d245/HTTP_Body_MultiPart.bin