Skip Menu |

This queue is for tickets about the Net-Google-Spreadsheets CPAN distribution.

Report information
The Basics
Id: 57540
Status: resolved
Priority: 0/
Queue: Net-Google-Spreadsheets

People
Owner: Nobody in particular
Requestors: yan [...] pixie.org.uk
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Creating new table: num_rows not documented and ignored
Date: Mon, 17 May 2010 00:35:17 +0100
To: bug-Net-Google-Spreadsheets [...] rt.cpan.org
From: Yan Wong <yan [...] pixie.org.uk>
Net-Google-Spreadsheets-0.11 (perl v5.10.0 built for darwin-thread-multi-2level, OS Darwin Kernel Version 10.3.0) When using tables, there is a parameter num_rows which is not documented. A quick browse of the code in Table.pm shows that it is referred to in the code (line 64), and used when reading frmo an atom feed (line 79), but not when writing to an atom feed. However, there is a potentially duplicate reference to setting the startRow attribute. I suspect this is a mistype, such that line 99 of Table.pm should read: $data->setAttribute(numRows => $self->num_rows) if $self->num_rows; rather than $data->setAttribute(startRow => $self->start_row) if $self->start_rows; This should also be documented, I guess. Cheers Yan
Thank you for the rt. I fixed the problem and released version 0.12. -- nd