added a copy method to C::R::Upload and updated Cookbook

[catagits/Catalyst-Runtime.git] / lib / Catalyst / Request / Upload.pm

diff --git a/lib/Catalyst/Request/Upload.pm b/lib/Catalyst/Request/Upload.pm
index 21646d7..389f810 100644 (file)
--- a/lib/Catalyst/Request/Upload.pm
+++ b/lib/Catalyst/Request/Upload.pm
@@ -3,7 +3,8 @@ package Catalyst::Request::Upload;
 use strict;
 use base 'Class::Accessor::Fast';
 
-use IO::File;
+use File::Copy ();
+use IO::File   ();
 
 __PACKAGE__->mk_accessors(qw/filename size tempname type/);
 
@@ -15,6 +16,7 @@ Catalyst::Request::Upload - Catalyst Request Upload Class
 
 =head1 SYNOPSIS
 
+    $upload->copy
     $upload->fh
     $upload->filename;
     $upload->link;
@@ -26,13 +28,24 @@ See also L<Catalyst>.
 
 =head1 DESCRIPTION
 
-This is the Catalyst Request Upload class, which provides a set of accessors to the
-upload data.
+This is the Catalyst Request Upload class, which provides a set of accessors 
+to the upload data.
 
 =head1 METHODS
 
 =over 4
 
+=item $upload->copy( $target [, $bufferlen ] )
+
+Copies tempname using C<File::Copy>. Returns true for success, false otherwise.
+
+=cut
+
+sub copy {
+    my ( $self, $target, $buffer ) = @_;
+    return File::Copy::copy( $self->tempname, $target, $buffer );
+}
+
 =item $upload->fh
 
 Opens tempname and returns a C<IO::File> handle.
@@ -42,7 +55,7 @@ Opens tempname and returns a C<IO::File> handle.
 sub fh {
     my $self = shift;
 
-    my $fh = IO::File->new( $self->tempname, O_RDWR )
+    my $fh = IO::File->new( $self->tempname, IO::File::O_RDONLY )
       or die( "Can't open ", $self->tempname, ": ", $! );
 
     return $fh;
@@ -62,10 +75,8 @@ success, false otherwise.
 =cut
 
 sub link {
-    my $self   = shift;
-    my $target = shift;
-
-    return link( $self->tempname, $target );
+    my ( $self, $target ) = @_;
+    return CORE::link( $self->tempname, $target );
 }
 
 =item $upload->size