Summary: | Cannot view detail bib record because missing perl module WWW::YouTube::Download | ||
---|---|---|---|
Product: | Koha | Reporter: | Héctor Eduardo Castro Avalos <hector.hecaxmmx> |
Component: | Cataloging | Assignee: | Josef Moravec <josef.moravec> |
Status: | RESOLVED DUPLICATE | QA Contact: | Testopia <testopia> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | josef.moravec, m.de.rooy, mirko, nengard |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14168 | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Attachments: | [SIGNED-OFF]Bug 15964: Followup: make module WWW::YouTube::Download required |
Description
Héctor Eduardo Castro Avalos
2016-03-02 19:52:40 UTC
Created attachment 48562 [details] [review] [SIGNED-OFF]Bug 15964: Followup: make module WWW::YouTube::Download required Module C4::HTML5Media is used in detail.pl, so if you don't have installled module for YouTube, you get an internal server error. This patch make module WWW::YouTube::Download required and adds it to package dependencies Test plan: 1. Don't have installed module WWW::YouTube:Download, neither package libwww-youtube-download-perl 2. Try to access biblio detail in staff client (detail.pl) - you'll be presented withh internal server error 3. Check the About page -> Perl modules, the WWW::YouTube::Download should be reported as missing optional 4. Apply the patch 5. Check the About page -> Perl modules, the WWW::YouTube::Download should now be reported as missing required 6. Create and install debian package, the libwww-youtube-download-perl should be the dependency and so installed with koha-common package (use apt-get install -f, if neccessery) 7. Check the About page -> Perl modules, the WWW::YouTube::Download should now be reported as Module current 8. Try to access biblio detail in staff client (detail.pl) - it should be display, no error now Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com> Works as advertised Hi Josef Moravec I move your patch to this new bug Regard Thanks Héctor, also for sign-off ;) Hi Héctor and Josef, we talked about this on IRC today and the plan is to fix it in a different way. The module is not available in Wheezy and it is not a core function, only a few people will actually need it. We will change the "use WWW::YouTube:Download" to a 'require' with the condition of the youtube-syspref being enabled, so people on Wheezy don't get bothered by this. I plan to add a patch tomorrow. I should be more often on IRC ;) Thanks Mirko for clarification, I'm looking forward to your patch ;) |