Bug #118494 for Spreadsheet-Read: Capability to add multiple CSV files as "sheets"

Mon Oct 24 17:06:12 2016 mlx [...] cpan.org - Ticket created

Subject:

Capability to add multiple CSV files as "sheets"

It would be useful when building tests for tools built from S::R to be able to provide multi-sheet structures based on CSV data. I have a few modules used in-house to process multi-sheet forms. I like to include testing for my form processing. I don't like including a binary file in my source. All I care about in the test is that the processing of the S::R data structure is handled correctly, not that S::R parses an Excel, LibreOffice, or some other raw file content. By using CSV for single-sheet workbooks, I am able to include the data in a much-reduced form in the test itself. For multi-sheet workbooks, I need to include an XLS test file in my distribution. I may also just be missing some mechanism in the current code to allow for this. Thanks, MLX

Sun Nov 27 14:27:55 2016 HMBRAND [...] cpan.org - Correspondence added

On Mon Oct 24 17:06:12 2016, midlifexis@wightmanfam.org wrote: Show quoted text

> It would be useful when building tests for tools built from S::R to be > able to provide multi-sheet structures based on CSV data. I have a > few modules used in-house to process multi-sheet forms. I like to > include testing for my form processing. I don't like including a > binary file in my source. > > All I care about in the test is that the processing of the S::R data > structure is handled correctly, not that S::R parses an Excel, > LibreOffice, or some other raw file content. By using CSV for single- > sheet workbooks, I am able to include the data in a much-reduced form > in the test itself. For multi-sheet workbooks, I need to include an > XLS test file in my distribution. > > I may also just be missing some mechanism in the current code to allow > for this.

I've just committed https://github.com/Tux/Spreadsheet-Read/commit/2c90d21ef54f8698028b13cce6fa26a3c4b50148 Is this meeting your needs? add my $book = ReadData ("file.csv"); Spreadsheet::Read::add ($book, "file.xlsx"); my $book = Spreadsheet::Read->new ("file.csv"); $book->add ("file.xlsx"); # OO Feedback welcom

Sun Nov 27 14:27:55 2016 The RT System itself - Status changed from 'new' to 'open'

Mon Nov 28 10:13:13 2016 HMBRAND [...] cpan.org - Status changed from 'open' to 'patched'

Tue Nov 29 10:08:12 2016 HMBRAND [...] cpan.org - Fixed in 0.68 added

Tue Nov 29 10:08:12 2016 HMBRAND [...] cpan.org - Status changed from 'patched' to 'resolved'

Thu Dec 01 10:58:57 2016 mlx [...] cpan.org - Correspondence added

Subject:	Re: [rt.cpan.org #118494] Resolved: Capability to add multiple CSV files as "sheets"
Date:	Thu, 1 Dec 2016 09:58:28 -0600
To:	bug-Spreadsheet-Read [...] rt.cpan.org
From:	Brian Wightman <mlx [...] cpan.org>

It looks like it should, but I have not (yet) had the cycles to test it out. Thanks. On Tue, Nov 29, 2016 at 9:08 AM, H.Merijn Brand via RT < bug-Spreadsheet-Read@rt.cpan.org> wrote: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=118494 > > > According to our records, your request has been resolved. If you have any > further questions or concerns, please respond to this message. >