Bug 14294 (Upload_umbrella) - Umbrella report for tracking improvements to File upload feature (see also 6874)
Summary: Umbrella report for tracking improvements to File upload feature (see also 6874)
Status: NEW
Alias: Upload_umbrella
Product: Koha
Classification: Unclassified
Component: Tools (show other bugs)
Version: master
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Marcel de Rooy
QA Contact: Testopia
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-05-29 06:23 UTC by Marcel de Rooy
Modified: 2018-01-23 20:26 UTC (History)
4 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Bot Control: ---
When did the bot last check this:
Who signed the patch off:
Text to go in the release notes:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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.