Bug 14294 (Upload_umbrella)

Summary: Umbrella report for tracking improvements to File upload feature (see also 6874)
Product: Koha Reporter: Marcel de Rooy <m.de.rooy>
Component: ToolsAssignee: Marcel de Rooy <m.de.rooy>
Status: NEW --- QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: mirko, mtompset, nick, veron
Version: master   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14295
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6874
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14282
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14296
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14321
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14893
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14849
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14854
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14686
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6816
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17298
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17501
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17663
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17669
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17777
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19633
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20081
Change sponsored?: --- Patch complexity: ---
Who signed the patch off: Text to go in the release notes:
Version(s) released in:

Description Marcel de Rooy 2015-05-29 06:23:08 UTC

    
Comment 1 M. Tompsett 2015-05-29 13:00:01 UTC
I will try to address some of the points raised on bug 6874:
[1] The new dependency Test/CGI/Multipart.pm is marked as not required. But you do unconditionally use it in the unit test. Should it be required? 

No. You are not required to run t/db_dependent tests, and that is the only place it is used. Should the test fail nicely if not installed? I'd be willing to ponder a patch for that.
Comment 2 Marcel de Rooy 2015-05-31 12:35:50 UTC
(In reply to M. Tompsett from comment #1)
> No. You are not required to run t/db_dependent tests, and that is the only
> place it is used. Should the test fail nicely if not installed? I'd be
> willing to ponder a patch for that.

I agree that we do not really need that. Same case for some other modules and tests.
Comment 3 Marcel de Rooy 2015-08-17 13:27:05 UTC
Submitted bug 14321 now (larger patch, one module instead of two; and some additional benefits)
Comment 4 Marcel de Rooy 2015-09-29 09:44:17 UTC
(In reply to Marcel de Rooy from comment #3)
> Submitted bug 14321 now (larger patch, one module instead of two; and some
> additional benefits)

14321 got pushed now; we also need bug 14893 now.
Comment 5 Marcel de Rooy 2015-09-29 09:54:29 UTC
Bug 14686 should be the next point to address.
Comment 6 Marcel de Rooy 2016-01-18 14:14:03 UTC
(In reply to Marcel de Rooy from comment #5)
> Bug 14686 should be the next point to address.

I have been waiting quite a while now for bug 14893 now to move further. But it did not (yet). If you are interested in this development, please help by signing that one off ! Thanks in advance :)
Comment 7 Marcel de Rooy 2016-03-17 12:46:57 UTC
Bug 14686 has been submitted now..
Comment 8 Marcel de Rooy 2016-12-14 16:02:15 UTC
Three reports in NSO now: 17501 for Koha Object, 17669 for purging temp files and 17777 for a fix in koha-remove.