If you have marked a report as public, one ought to be able to easily find the URL of the report to use from within the staff client. Possible locations for this are as another column (or link) in the table of saved reports, a link or text from the "Report created, what now?" page, or an automatic category of report that shows all public reports, with their OPAC links.
Created attachment 68286 [details] [review] Bug 16782 - disclose URL for public reports on the saved report page It has always annoyed me that you had to look at the help to divine the url for a public report. This patch works it out and displays it for you. Requirements: - OPACBaseURL must be set for the url to be complete. It's ok if it's not, but it's better if it is. - You must have a saved report that is public to your OPAC. To test: - create a public saved report. It can be anything. - view the saved reports page, note that under Public it says: Avaliable at (opacbaseurl)/cgi-bin/koha/svc/report?id=1 - clicking it should take you to the public results of your report. sponsored-by: Catalyst IT
Created attachment 68287 [details] [review] Bug 16782 - disclose URL for public reports on the saved report page It has always annoyed me that you had to look at the help to divine the url for a public report. This patch works it out and displays it for you. Requirements: - OPACBaseURL must be set for the url to be complete. It's ok if it's not, but it's better if it is. - You must have a saved report that is public to your OPAC. To test: - create a public saved report. It can be anything. - view the saved reports page, note that under Public it says: Avaliable at (opacbaseurl)/cgi-bin/koha/svc/report?id=1 - copying and pasting that into the address bar should take you to the public results of your report. sponsored-by: Catalyst IT
We have applied the patch we have found 1. It works as expected BUT if you modify the report, the new results doesn't appear until you clean cache and restart plack --> perhaps it is not related with this patch, but it fails. 2. we have seen that the info of the report if given in json format, it is nice for managin that, but is that the expected behaviour?
This patch does not touch the functionality of the public reports. Set the cache timer to a low value (not 0, 0 means it never refreshes) if you want to have instant-ish updates. Yes, JSON is the expected behaviour. Cheers, Liz
Created attachment 68446 [details] [review] Bug 16782 - disclose URL for public reports on the saved report page It has always annoyed me that you had to look at the help to divine the url for a public report. This patch works it out and displays it for you. Requirements: - OPACBaseURL must be set for the url to be complete. It's ok if it's not, but it's better if it is. - You must have a saved report that is public to your OPAC. To test: - create a public saved report. It can be anything. - view the saved reports page, note that under Public it says: Avaliable at (opacbaseurl)/cgi-bin/koha/svc/report?id=1 - copying and pasting that into the address bar should take you to the public results of your report. sponsored-by: Catalyst IT Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Created attachment 68447 [details] [review] Bug 16782: Fix spelling of 'Available' Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Doh thanks for catching that. :D
Created attachment 68619 [details] [review] Bug 16782 - disclose URL for public reports on the saved report page It has always annoyed me that you had to look at the help to divine the url for a public report. This patch works it out and displays it for you. Requirements: - OPACBaseURL must be set for the url to be complete. It's ok if it's not, but it's better if it is. - You must have a saved report that is public to your OPAC. To test: - create a public saved report. It can be anything. - view the saved reports page, note that under Public it says: Avaliable at (opacbaseurl)/cgi-bin/koha/svc/report?id=1 - copying and pasting that into the address bar should take you to the public results of your report. sponsored-by: Catalyst IT Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Created attachment 68620 [details] [review] Bug 16782: Fix spelling of 'Available' Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Created attachment 68621 [details] [review] Bug 16782: QA Followup - use a link instead of showing full URL
Display was a bit long, took the liberty of using a link instead of the URL, hope that's okay
Should we show the URL for non-public too? They are automatically available under the staff url, when set to not public.
Ha, I originally had it as a link, but decided not to in the end because it was awkward when OPACBaseURL wasn't set. Plus you never click on it, you only ever use it in code, so not much reason to be clickable. Cheers, Liz
Regarding showing the non-public, that's fine. I'll do a catchall followup that does it how I want it to be done. Cheers, Liz
Created attachment 68639 [details] [review] Bug 16782 - disclose JSON URL for reports on the saved report page It has always annoyed me that you had to look at the help to divine the url for a report. This patch works it out and displays it for you in a new column on the saved reports table. Requirements: - OPACBaseURL and staffClientBaseURL preferences must be set for the urls to be complete. It's ok if they are not, but it's better if they are. - You must have a saved report that is public to your OPAC, and one that is not public. To test: - create a public saved report and a private staff report. They can be anything. - view the saved reports page, note that under a new heading of JSON URL the link to the json feed for that report shows. - copying and pasting that link into the address bar, or clicking it, should take you to the results of the selected report. sponsored-by: Catalyst IT
Created attachment 68648 [details] [review] Bug 16782 - disclose JSON URL for reports on the saved report page It has always annoyed me that you had to look at the help to divine the url for a report. This patch works it out and displays it for you in a new column on the saved reports table. Requirements: - OPACBaseURL and staffClientBaseURL preferences must be set for the urls to be complete. It's ok if they are not, but it's better if they are. - You must have a saved report that is public to your OPAC, and one that is not public. To test: - create a public saved report and a private staff report. They can be anything. - view the saved reports page, note that under a new heading of JSON URL the link to the json feed for that report shows. - copying and pasting that link into the address bar, or clicking it, should take you to the results of the selected report. sponsored-by: Catalyst IT Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Created attachment 68649 [details] [review] Bug 16782 - missing </td>, fixed now
Thankfully this still applies. :)
Created attachment 70322 [details] [review] Bug 16782 - disclose JSON URL for reports on the saved report page It has always annoyed me that you had to look at the help to divine the url for a report. This patch works it out and displays it for you in a new column on the saved reports table. Requirements: - OPACBaseURL and staffClientBaseURL preferences must be set for the urls to be complete. It's ok if they are not, but it's better if they are. - You must have a saved report that is public to your OPAC, and one that is not public. To test: - create a public saved report and a private staff report. They can be anything. - view the saved reports page, note that under a new heading of JSON URL the link to the json feed for that report shows. - copying and pasting that link into the address bar, or clicking it, should take you to the results of the selected report. sponsored-by: Catalyst IT Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Created attachment 70323 [details] [review] Bug 16782 - missing </td>, fixed now
Created attachment 70324 [details] [review] Bug 16782: (QA Followup) Use Koha.Preference for staffClientBaseURL and provide classes
Created attachment 70395 [details] [review] Bug 16782: Use uri filter instead of html
You could have asked me to fix those things, I'd have been happy to. Thank you for the extra patches all the same. Liz
Pushed to master for 18.05, thanks to everybody involved!
Awesome work all, pushed to stable for 17.11.02
Hm, this'd be a feature not a bug. It's harmless but we should remember not to push features to stable branches. Cheers, Liz
Enhancement not pushed to 17.05.x