r13925@rob-kinyons-powerbook58: rob | 2006-06-02 14:29:05 -0400

[p5sagit/Excel-Template.git] / lib / Excel / Template / Element / Image.pm

diff --git a/lib/Excel/Template/Element/Image.pm b/lib/Excel/Template/Element/Image.pm
index 2817ad4..dd9b9dd 100644 (file)
--- a/lib/Excel/Template/Element/Image.pm
+++ b/lib/Excel/Template/Element/Image.pm
@@ -13,10 +13,9 @@ sub render {
     my $self = shift;
     my ($context) = @_;
 
-    my $path = $context->get( $self, 'PATH' );
-    my ($row, $col, $offset, $scale) = map {
+    my ($row, $col, $path, $offset, $scale) = map {
         $context->get($self, $_)
-    } qw( ROW COL OFFSET SCALE );
+    } qw( ROW COL PATH OFFSET SCALE );
 
     my @offsets = (0,0);
     if ( $offset =~ /^\s*([\d.]+)\s*,\s*([\d.]+)/ ) {
@@ -28,7 +27,7 @@ sub render {
         @scales = ($1,$2);
     }
 
-    $context->active_worksheet->insert_image(
+    $context->active_worksheet->insert_bitmap(
         $row, $col, $path, @offsets, @scales,
     );
 
@@ -54,12 +53,16 @@ To insert an image into the worksheet
 
 =head1 NODE NAME
 
-CELL
+IMAGE
 
 =head1 INHERITANCE
 
 L<ELEMENT|Excel::Template::Element>
 
+=head1 EFFECTS
+
+This will consume one column in the current row.
+
 =head1 DEPENDENCIES
 
 None
@@ -73,6 +76,8 @@ None
 
 Please see L<Spreadsheet::WriteExcel/> for more information about the offset and scaling options as well as any other restrictions that might be in place. This node does B<NOT> perform any sort of validation upon your parameters. You are assumed to know what you are doing.
 
+Note that the offset and scaling values are "X,Y". You I<must> provide both values, even if the Y value is 0. If you provide a 0 value for either scaling option, L<Spreadsheet::WriteExcel/> will default that to 1.
+
 =head1 AUTHOR
 
 Rob Kinyon (rob.kinyon@gmail.com)