~rgrjr/Spreadsheet-Table-Extract: master

aacf7210 — Bob Rogers 1 year, 10 months ago master

* lib/Spreadsheet/Table/Extract.pm:  Fix make_disjoint doc typo.

c9af2974 — Bob Rogers 1 year, 10 months ago

Make explain_overlap smarter

* lib/Spreadsheet/Table/Extract.pm:
   + (explain_overlap):  Check whether tables are actually disjoint
     before trying to report on overlap.
* t/01-basic.t:
   + Add a test for same.

46ed3ff6 — Bob Rogers 1 year, 10 months ago

Make find_tables and extract return lists

* lib/Spreadsheet/Table/Extract.pm:
   + (find_tables):  Return a list rather than an arrayref.
   + (extract):  Likewise.
* t/01-basic.t:
   + Update the tests.

6fbaa31f — Bob Rogers 1 year, 10 months ago

Have find_tables default the table name

* lib/Spreadsheet/Table/Extract.pm:
   + (find_tables $make_table_descriptor):  Default the table name if
     given a base_name.
* t/01-basic.t:
   + Test same.

842a4808 — Bob Rogers 1 year, 10 months ago

Make the find_tables minimum_slots param a keyword

* lib/Spreadsheet/Table/Extract.pm:
   + (find_tables):  Make minimum_slots a keyword instead of a
     positional parameter, adjust doc accordingly.
* t/01-basic.t:
   + Update tests.

577212c8 — Bob Rogers 1 year, 10 months ago

Support a better validate_object default

* lib/Spreadsheet/Table/Extract.pm:
   + (extract):  Support a reasonable default for validate_object, that
     just rejects blank rows.  Also, wrap eval around instance creation.
* t/01-basic.t:
   + Test same.

d1fce71f — Bob Rogers 1 year, 10 months ago

* lib/Spreadsheet/Table/Extract.pm:  Add a description.

f4de604d — Bob Rogers 1 year, 10 months ago

Use the first column definition if there are duplicates

* lib/Spreadsheet/Table/Extract.pm:
   + (find_tables):  Switch to ignoring second and subsequent column
     redefintions instead of taking the last one.

4b58df5c — Bob Rogers 1 year, 10 months ago

Instantiate tables of a specified class

* lib/Spreadsheet/Table/Extract.pm:
   + (find_tables):  Support instantiating tables of a different class
     from the invocant.
* t/01-basic.t:
   + Test same.

84b15a5d — Bob Rogers 1 year, 10 months ago

Remove redundant find_tables code

* lib/Spreadsheet/Table/Extract.pm:
   + (find_tables):  Remove a redundant "while ($row <= $maxrow) ..."
     loop, move the $row variable accordingly.

3f97199c — Bob Rogers 1 year, 10 months ago

Clean up column name whitespace

* lib/Spreadsheet/Table/Extract.pm:
   + (find_tables):  Clean up leading & trailing whitespace from
     potential column names before comparing, & update the doc to match.

fe96c948 — Bob Rogers 1 year, 10 months ago

Recognize side-by-side tables

* lib/Spreadsheet/Table/Extract.pm:
   + (find_tables $have_enough_p):  Helper sub.
   + (find_tables $make_table_descriptor):  Another helper.
   + (find_tables):  If we find a duplicate column and we have all
     required columns, then spin off a table with what we have, and
     start a new one.
* t/01-basic.t:
   + Add a test for side-by-side table definitions.
* t/data/side-by-side.gnumeric (added):
   + Test data for same.

295dd2d1 — Bob Rogers 1 year, 10 months ago

Provide a default for the $min_slots arg to find_tables

* lib/Spreadsheet/Table/Extract.pm:
   + (find_tables):  Default $min_slots to mean require everything.
* t/01-basic.t:
   + Test same.

dc843fc7 — Bob Rogers 1 year, 10 months ago

Fix side-by-side table bug

* lib/Spreadsheet/Table/Extract.pm:
   + (find_tables):  Bug fix:  Don't adjust last_row on $last_table if
     we find another; the tables might be side-by-side, and that would
     be up to make_disjoint to figure out.

85595aef — Bob Rogers 1 year, 10 months ago

* lib/Spreadsheet/Table/Extract.pm:  Document

81c93f34 — Bob Rogers 1 year, 10 months ago

Update metadata per cpants_lint.pl

* dist.ini:
   + Add MetaJSON and Prereqs, per cpants_lint.pl recommendations.
   + Also, tweak the abstract.

5a87f3a3 — Bob Rogers 1 year, 11 months ago

Fix tests for a Spreadsheet::ReadGnumeric bug

* t/01-basic.t, t/data/basic-test-output.pm:
   + Update the multiple.gnumeric case for the Spreadsheet::ReadGnumeric
     maxcol & maxrow origin bug.

ad9477fb — Bob Rogers 1 year, 11 months ago

Make tables disjoint

* lib/Spreadsheet/Table/Extract.pm:
   + (explain_overlap):  Diagnostic tool (with perhaps TMI).
   + (overlaps):  Test for overlapping tables.
   + (make_disjoint):  Adjust last_row values to remove overlaps.
* t/01-basic.t:
   + Add a test case for same.

4dd84679 — Bob Rogers 1 year, 11 months ago

Allow naming of extracted tables

* lib/Spreadsheet/Table/Extract.pm:
   + Add a "name" slot.
* t/01-basic.t:
   + Add names to the multiple.gnumeric tests.
* t/data/basic-test-output.pm:
   + Changed test output.

bd7ce764 — Bob Rogers 1 year, 11 months ago

Extract the overlapping multiple.gnumeric check table

* t/01-basic.t:
   + Add a test that extracts the multiple.gnumeric checkbook table,
     which gets four more rows than it should because it doesn't know
     that this runs into the other tables.