Summary: | Exporting serial claims does not work if no notice defined | ||
---|---|---|---|
Product: | Koha | Reporter: | Harsha <bala.allmail> |
Component: | Serials | Assignee: | Jonathan Druart <jonathan.druart> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | normal | ||
Priority: | P5 - low | CC: | colin.campbell, fridolin.somers, gmcharlt, jonathan.druart, kyle |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Small patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Serials claims
No check boxes Bug 10837: Exporting serial claims does not work if no notice defined Bug 10837: Exporting serial claims does not work if no notice defined [PASSED QA] Bug 10837: Exporting serial claims does not work if no notice defined |
Description
Harsha
2013-09-07 01:24:37 UTC
Created attachment 20967 [details] Serials claims (In reply to Harsha from comment #0) > Check boxes are missing in Serial > Claims > 'Late Selections list'' . > > So I cannot do "Exported selected items"" in my installations(v 3.12.0 and > 3.12.3) > > In Koha Demo( 3.12v), I have observed same but Manual shows there are check > boxes in 3.10v for the same task > > I am new to KOHA and we, Open university, Sri Lanka testing the system > feasibility this is a drawback to us as we have significant amount to > Serials to manage. > > Thanks Hello Harsha, Could you detail the "Late Selections list" link? Did you get a javascript error (using Firebug)? Here my serial claims screen (on the master branch). It looks like that for me too - Harsha, could you upload a screenshot of your installation? Created attachment 23747 [details] No check boxes The check boxes are not availabe in 3.12 and 3.14 Path: http://YOURDOMAIN:PORT/cgi-bin/koha/serials/claims.pl ( )Staff client Please notice that the table and see no check boxes to select. Problem exist in both 3.12 and 3.14v Created attachment 23773 [details] [review] Bug 10837: Exporting serial claims does not work if no notice defined On the serial claims page, it is possible to export (using a csv profile) or claim 1+ serials. The checkboxes are not shown if the claiming notice is not defined. So it is not possible to export claims. Test plan: - delete your notice "claimissues" - go on the serial claims page - verify that you are able to export them as CSV (In reply to Harsha from comment #3) Hello Harsha, I suppose you don't have a claiming notice defined (claimissue). The patch I proposed will display checkboxes even if the notice is not defined. Created attachment 25953 [details] [review] Bug 10837: Exporting serial claims does not work if no notice defined On the serial claims page, it is possible to export (using a csv profile) or claim 1+ serials. The checkboxes are not shown if the claiming notice is not defined. So it is not possible to export claims. Test plan: - delete your notice "claimissues" - go on the serial claims page - verify that you are able to export them as CSV Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Created attachment 26119 [details] [review] [PASSED QA] Bug 10837: Exporting serial claims does not work if no notice defined On the serial claims page, it is possible to export (using a csv profile) or claim 1+ serials. The checkboxes are not shown if the claiming notice is not defined. So it is not possible to export claims. Test plan: - delete your notice "claimissues" - go on the serial claims page - verify that you are able to export them as CSV Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> This works as described and enables use of the export funtionality even if you are not using the email notifications. Exporting the serials will also set the claim date. Passes QA script and tests. There is one small thing I noticed - exporting the CSV file sets the claim counter, but you have to reload the page first for it to show up as downloading the CSV will not reload the page. Pushed to master. Thanks, Jonathan! Pushed to 3.14.x, will be in 3.14.07 Pushed to 3.12.x will be in 3.12.13 |