miscellaneous changes

Author: PhilterPaper

Changes

@@ -1,17 +1,31 @@
 Change history for Perl extension PDF::Table.
+* = work in progress
 
-=== Version 1.008 (2025-03-25) UNRELEASED
+=== Version 1.008 (2025-05-14) UNRELEASED
 
 *   all Deprecated items updated
 
-*   consolidate A/B choice in lib/PDF/Table/ChooseDriver.pm
+*   consolidate A/B choice in lib/PDF/Table/ChooseDriver.pm and t/ and
+      examples/
+
+    MANIFEST, MANIFEST.SKIP (remove), SECURITY.md (new)
+     Address Kwalitee issues: unhappy with MANIFEST.SKIP (remove from MANIFEST,
+      erase MANIFEST.SKIP), add SECURITY.md with contact address per Builder.
+      Doesn't like .perl-version and .perlcritic, but won't say what's wrong.
+      At this time, unable to add "provides" per Builder at this time, as unlike
+      Builder, META.yml and META.json are auto-built and not manually provided.
+ 
+    t/lib/PDFAPI2Mock.pm
+     Add mediabox() to dummy stubs so that t/PDF-Table.t can still run
+
+    Change distribution (CPAN owner) to PMPERRY from OMEGA
 
 === Version 1.007 (2025-03-25)
 
     devtools/PDFversion.pl, devtools/TableBuild.pl, util/3_examples.pl
-     Updated to modern Perl standards to make 1_pc.pl PerlCritic happy.
+     Updated to modern Perl standards to make 1_pc.pl (PerlCritic) happy.
 
-    lib/PDF/Table.pm
+    lib/PDF/Table.pm, lib/PDF/Table.pod
      Permit 'trans' or 'transparent' background color (mostly for overriding
       a default bg color). Per #80 (reported by @az143).
 
@@ -20,6 +34,9 @@ Change history for Perl extension PDF::Table.
       code to show effect in colspan example. Per #78 (reported and fix 
       supplied by @BlueWlvrn).
 
+    LICENSE, version, INFO/Table.html, README.md
+     Update for new release and copyright year.
+
 === Version 1.006 (2024-03-21)
 
     devtools/*

MANIFEST

@@ -24,9 +24,9 @@ lib/PDF/Table/Settings.pm
 LICENSE
 Makefile.PL
 MANIFEST
-MANIFEST.SKIP
 PDFpref
 README.md
+SECURITY.md
 t/Basics.t
 t/Colspan.t
 t/lib/PDFAPI2Mock.pm

MANIFEST.SKIP

@@ -0,0 +1,23 @@
+# Security Policy for PDF::Table
+
+## Reporting security issues
+
+**Do not report security problems on public forums or in repository issues.**
+
+Privately report vulnerabilities to the maintainers listed at the end of this document. Include as many details as possible to reproduce the issue, including code samples or test cases. Check that your report does not expose any of your sensitive data, such as passwords, tokens, or other secrets.
+
+You do not need to have a solution or fix. Depending on the issue, CPANSec may be notified.
+
+You can also privately report issues to the CPAN Security Group (CPANSec) cpan-security@security.metacpan.org. This is especially important if you think a vulnerability is being actively exploited. CPANSec may report the issue to the relevant authorities. See [Report a Security Issue](https://security.metacpan.org/docs/report.html).
+
+## Response to reports
+
+The maintainers aim to respond to all reports within one day, but this may be affected by life and other things that happen to people who maintain open source code.
+
+A new release will be provided as soon as possible. In the meantime, a workaround or code patch may be provided, if deemed adequate for the purpose.
+
+## Maintainers
+
+    Phil M Perry, phil4597@catskilltech.com
+
+

SECURITY.md

@@ -1,8 +1,6 @@
 #!/usr/bin/perl
-    use strict;
-    use warnings;
-
-#my $ourLAST = 1;  # T: change my $LAST_UPDATE to our $LAST_UPDATE
+use strict;
+use warnings;
 
     my $src = $ARGV[0];  # directory root to work in for this call
     my $VERSION = $ARGV[1];  # version to use (e.g., 3.005)
@@ -70,9 +68,6 @@ sub update_version {
         while ($line = <$IN>) {
 	    # $line still has line-end \n
             $line =~ s/$pattern/$newVer/;
-	   #if ($ourLAST) {
-	   #    $line =~ s/^my \$LAST_UPDATE/our \$LAST_UPDATE/;
-	   #}
 	    print $OUT $line;
         }
 

devtools/PDFversion.pl

@@ -812,8 +812,11 @@ sub table {
                 unshift @$rows_height, $header_min_rh;
 
                 $first_row = 1; # Means YES
-                # Roll back the row_idx because a new header row added
-                $row_idx--; 
+                # Roll back the row_idx because a new header row added,
+                # unless it was the header itself that overflowed. NOTE that
+                # this will still produce a partial header on the first page,
+                # and a full header on the second page!
+                $row_idx-- if $row_idx > 0; 
             }
 
         }

lib/PDF/Table.pm

@@ -230,7 +230,7 @@ See the "Column.pl" example on PDF::Builder for some examples.
 Automatic cell sizing will not work with this, so you need to use the
 C<size> table parameter to specify relative cell widths.
 
-B<End of PDF::Builder ONLY:>
+B<End of PDF::Builder ONLY>
 
 For full C<%settings> description see section L</Table settings> below.
 

lib/PDF/Table.pod

@@ -72,8 +72,9 @@ BEGIN {
 
     package PDF::API2::Page; ## no critic
     our @ISA = 'Mock';
-    __PACKAGE__->factory( 'PDF::API2::Content',       'gfx' );
+    __PACKAGE__->factory( 'PDF::API2::Content',        'gfx' );
     __PACKAGE__->factory( 'PDF::API2::Content::Text', 'text' );
+    sub mediabox { return (0,0, 612,792) }
 
     package PDF::API2::Content; ## no critic
     our @ISA = 'Mock';