|Summary:||tools/upload-file.pl: scope $uploaded_file for plack|
|Product:||Koha||Reporter:||Dobrica Pavlinusic <dpavlin>|
|Component:||Tools||Assignee:||Dobrica Pavlinusic <dpavlin>|
|Status:||CLOSED FIXED||QA Contact:||Ian Walls <koha.sekjal>|
|Priority:||P5 - low||CC:||alex.arnaud, paul.poulain|
|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, 7773, 7790|
Bug 7775 - tools/upload-file.pl: scope $uploaded_file for plack
Bug 7775 - tools/upload-file.pl: scoping for plack
[Signed-Off] Bug 7775 - tools/upload-file.pl: scoping for plack
Description Dobrica Pavlinusic 2012-03-21 10:49:03 UTC Comment hidden (obsolete)
Created attachment 8398 [details] [review] proposed patch
Comment 1 Dobrica Pavlinusic 2012-03-21 14:52:59 UTC Comment hidden (obsolete)
Created attachment 8426 [details] [review] Bug 7775 - tools/upload-file.pl: scope $uploaded_file for plack
Comment 2 Alex Arnaud 2012-03-22 08:36:31 UTC
I've applied this patch and it seems to fix the problem described in title. Indeed, files upload works now but there's a (new?) problem: When the upload is done and clicking "Manage staged records" link on MARC Staging results page in order to view the details of the batch, i get an Internal Server Error. Also, in "Manage Staged MARC Records" page (list of batches) the file name is not shown on the line related to record(s) i've juste imported.
Comment 3 Dobrica Pavlinusic 2012-03-22 09:38:05 UTC
Alex, that's problem with tools/manage-marc-import.pl for which I submitted bug 7790, so you might as well sign-off this one ;-)
Comment 4 Alex Arnaud 2012-03-23 10:33:34 UTC
I have the same problem than in comment 2 when applying 7775 and 7790.
Comment 5 Dobrica Pavlinusic 2012-03-23 11:09:26 UTC Comment hidden (obsolete)
Created attachment 8584 [details] [review] Bug 7775 - tools/upload-file.pl: scoping for plack $uploaded_file is now scoped with our, and we set correct filename
Comment 6 Alex Arnaud 2012-03-23 11:16:07 UTC
Created attachment 8586 [details] [review] [Signed-Off] Bug 7775 - tools/upload-file.pl: scoping for plack $uploaded_file is now scoped with our, and we set correct filename Signed-off-by: Alex Arnaud <email@example.com>
Comment 7 Alex Arnaud 2012-03-23 11:16:55 UTC
This patch should be pushed with 7790.
Comment 8 Paul Poulain 2012-03-26 16:18:04 UTC
QA comment: same kind of patch as all the plack compliance patch. Passed QA
Comment 9 Jared Camins-Esakov 2012-12-31 00:48:45 UTC
There have been no further reports of problems so I am marking this bug resolved.