Summary: | The sub 'AnonymiseIssueHistory' fails quietly if 'AnonymousPatron' is not a valid entry | ||
---|---|---|---|
Product: | Koha | Reporter: | Christophe Croullebois <christophe.croullebois> |
Component: | OPAC | Assignee: | Christophe Croullebois <christophe.croullebois> |
Status: | RESOLVED DUPLICATE | QA Contact: | |
Severity: | normal | ||
Priority: | P5 - low | CC: | jonathan.druart, kyle |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Attachments: |
proposed patch
bug 9370: test case for splitting LC call number |
Description
Christophe Croullebois
2013-02-28 15:12:07 UTC
Created attachment 15778 [details] [review] proposed patch To test : Without patch leave 'AnonymousPatron' syspref blanck or with '0' value. Try in Opac/my privacy to use 'Immediate deletion' with a patron who have a reading history. The reading history is still present. With patch, the same context, the reading history is deleted. Thx for testing I believe it would be more proper to disable the "my privacy' tab if AnonymousPatron is not a valid patron. This would be more in line with how OPACPrivacy requires opacreadinghistory and the tab does not show if opacreadinghistory is not set to "Allow". Created attachment 18864 [details] bug 9370: test case for splitting LC call number QH541.15.C6 C25 2012 should be split as follows: QH 541.15 .C6 C25 2012 Signed-off-by: Galen Charlton <gmc@esilibrary.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> The content of attachment 18864 [details] has been deleted by Chris Cormack <chris@bigballofwax.co.nz> without providing any reason. The token used to delete this attachment was generated at 2013-06-12 21:47:50 NZST. I don't see comments about the failed QA status ? An error ? So I move in need ... oups sry Kyle, hum, you suggest to hide the tab in this case ... Ok I'll do a new patch in this way thx |