Bug 7952

Summary: PDF::Reuse under plack writes to console STDOUT instead to browser
Product: Koha Reporter: Dobrica Pavlinusic <dpavlin>
Component: Label/patron card printingAssignee: Dobrica Pavlinusic <dpavlin>
Status: CLOSED FIXED QA Contact: Ian Walls <koha.sekjal>
Severity: normal    
Priority: P5 - low CC: alex.arnaud, chris, paul.poulain
Version: 3.8   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on:    
Bug Blocks: 7172    
Attachments: Bug 7952 - PDF::Reuse under plack writes to console STDOUT instead to browser
Bug 7952 - PDF::Reuse under plack writes to console STDOUT instead to browser
[Signed-off] Bug 7952 - PDF::Reuse under plack writes to console STDOUT instead to browser
Bug 7952 - PDF::Reuse under plack writes to console STDOUT instead to browser
Bug 7952 - PDF::Reuse under plack writes to console STDOUT instead to browser
Bug 7952 - followup to fix t/Creators.t

Description Dobrica Pavlinusic 2012-04-12 21:33:28 UTC

    
Comment 1 Dobrica Pavlinusic 2012-04-12 22:21:49 UTC Comment hidden (obsolete)
Comment 2 Paul Poulain 2012-06-01 17:45:52 UTC
QA comment:
 * there is an unconditional warn added:
+    warn "## Name [$opts{Name}] $fh";

please remove it of write $ENV{DEBUG} and warn "## Name [$opts{Name}] $fh";
Comment 3 Dobrica Pavlinusic 2012-06-01 19:24:49 UTC Comment hidden (obsolete)
Comment 4 Alex Arnaud 2012-06-09 15:32:59 UTC Comment hidden (obsolete)
Comment 5 Paul Poulain 2012-06-10 11:04:01 UTC
QA comment:
 * sorry about that, but there's an unconditionnal warn:
+    warn "## Name [$opts{Name}] $fh";

so, failed QA until you've removed it (or $ENV{DEBUG} &&...)

Other than this, tiny patch, will pass QA
Comment 6 Dobrica Pavlinusic 2012-06-10 11:29:06 UTC Comment hidden (obsolete)
Comment 7 Dobrica Pavlinusic 2012-06-10 11:33:37 UTC
Sorry about that. It seems that I wanted to add $ENV{DEBUG} around that warn (and even included it in commit message) but never changed code. On second look, I think it's not really useful to anyone expect me while development so it's removed -- less cruft, more kittens!
Comment 8 Chris Cormack 2012-06-10 15:08:26 UTC Comment hidden (obsolete)
Comment 9 Paul Poulain 2012-06-11 16:22:40 UTC
QA comment: this time it's OK, nothing to object !
Comment 10 Paul Poulain 2012-06-11 16:24:51 UTC
Patch pushed (I haven't tested it, I have no patron cards on y test database)
Comment 11 Chris Cormack 2012-06-12 06:32:08 UTC
This now needs a follow up to correct the behaviour in t/Creators.t
Comment 12 Dobrica Pavlinusic 2012-06-14 12:02:05 UTC
Created attachment 10278 [details] [review]
Bug 7952 - followup to fix t/Creators.t

Since we are now using temporary file, we are creating test.pdf
at end of test and checking if it exists and have non-zero size.
Comment 13 Paul Poulain 2012-06-14 13:54:49 UTC
Trivial follow-up, make test working, will signoff/pass QA/push in one go
Comment 14 Chris Cormack 2012-06-15 19:02:36 UTC
Pushed both the original and follow up to 3.8.x, will be in 3.8.2