| Summary: | Attempting to view an invalid authority in the OPAC gives an error instead of 404 | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Jared Camins-Esakov <jcamins> |
| Component: | OPAC | Assignee: | Jared Camins-Esakov <jcamins> |
| Status: | CLOSED FIXED | QA Contact: | Jonathan Druart <jonathan.druart> |
| Severity: | major | ||
| Priority: | P5 - low | CC: | chris, jonathan.druart, paul.poulain |
| Version: | 3.8 | ||
| Hardware: | All | ||
| OS: | All | ||
| Change sponsored?: | --- | Patch complexity: | --- |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: | ||
| Circulation function: | |||
| Attachments: |
Bug 8572: invalid auth shouldn't cause error
Bug 8572: invalid auth shouldn't cause error |
||
|
Description
Jared Camins-Esakov
2012-08-04 17:39:49 UTC
Created attachment 11358 [details] [review] Bug 8572: invalid auth shouldn't cause error Prior to this patch, attempting to view an invalid authority in the OPAC gave a system error instead of a 404. With this patch, the user will just get a 404, as is the case with invalid bibs. To test: 1) Try to view an invalid authority by going to: /cgi-bin/koha/opac-authoritiesdetail.pl?authid=999999999999 (presuming that you don't have an authority with authid 999999999999 in your database) before and after applying this patch Created attachment 11371 [details] [review] Bug 8572: invalid auth shouldn't cause error Prior to this patch, attempting to view an invalid authority in the OPAC gave a system error instead of a 404. With this patch, the user will just get a 404, as is the case with invalid bibs. To test: 1) Try to view an invalid authority by going to: /cgi-bin/koha/opac-authoritiesdetail.pl?authid=999999999999 (presuming that you don't have an authority with authid 999999999999 in your database) before and after applying this patch Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> QA comments: It would have been more user-friendly to display an adapted error for this case. But Koha does not have a simply way to send messages (infos, warnings, errors) to the templates. So, I mark as Passed QA Pushed to 3.8.x, will be in 3.8.4 Patch pushed to master |