[Koha-bugs] [Bug 15096] Export today's checked in barcodes: Display warning if reading history is set to "never"
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15096 Brendan Gallagher changed: What|Removed |Added Status|Passed QA |Pushed to Master CC||bren...@bywatersolutions.co ||m --- Comment #14 from Brendan Gallagher --- Pushed to Master - Should be in the May 2016 release. Thanks! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15096] Export today's checked in barcodes: Display warning if reading history is set to "never"
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15096 Kyle M Hall changed: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15096] Export today's checked in barcodes: Display warning if reading history is set to "never"
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15096 Kyle M Hall changed: What|Removed |Added Attachment #46628|0 |1 is obsolete|| --- Comment #13 from Kyle M Hall --- Created attachment 47212 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=47212&action=edit Bug 15096: Export today's checked in barcodes: Display warning if reading history is set to "never" If a patron's privacy settings are set to "Never" for keeping a reading history, "Export today's checked in barcodes" returns an empty file. This patch does not allow to export in such case. To test: - Apply patch - Check out / check in in some items -Test toolbar: - On user's detail page, go to More->Export today's checked in barcodes - Verify that the menu item does not appear if syspref 'intranetreadinghistory' is set to 'Don't allow' - Verify that the menu item appears if syspref 'intranetreadinghistory' is set to allow and - that the menu item is grayed out with a tooltip if the user has set privacy settings to never keep a reading history - that the menu item works as before if user's privacy settings allow reading history. - Test left tab "Circulation history": - Verify that the left tab "Circulation history" does not appear if syspref 'intranetreadinghistory' is set to 'Don't allow' - Verify that the tab appars if syspref 'intranetreadinghistory' is set to 'Allow' and - that a message appears if user's privacy settings do not allow to keep the reading history - that the export works as before if user's privacy settings allow to keep the reading history (Amended and changed test plan for comment #9) Signed-off-by: Aleisha Signed-off-by: Kyle M Hall -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15096] Export today's checked in barcodes: Display warning if reading history is set to "never"
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15096 Aleisha Amohia changed: What|Removed |Added Attachment #46591|0 |1 is obsolete|| --- Comment #12 from Aleisha Amohia --- Created attachment 46628 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46628&action=edit Bug 15096: Export today's checked in barcodes: Display warning if reading history is set to "never" If a patron's privacy settings are set to "Never" for keeping a reading history, "Export today's checked in barcodes" returns an empty file. This patch does not allow to export in such case. To test: - Apply patch - Check out / check in in some items -Test toolbar: - On user's detail page, go to More->Export today's checked in barcodes - Verify that the menu item does not appear if syspref 'intranetreadinghistory' is set to 'Don't allow' - Verify that the menu item appears if syspref 'intranetreadinghistory' is set to allow and - that the menu item is grayed out with a tooltip if the user has set privacy settings to never keep a reading history - that the menu item works as before if user's privacy settings allow reading history. - Test left tab "Circulation history": - Verify that the left tab "Circulation history" does not appear if syspref 'intranetreadinghistory' is set to 'Don't allow' - Verify that the tab appars if syspref 'intranetreadinghistory' is set to 'Allow' and - that a message appears if user's privacy settings do not allow to keep the reading history - that the export works as before if user's privacy settings allow to keep the reading history (Amended and changed test plan for comment #9) Signed-off-by: Aleisha -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15096] Export today's checked in barcodes: Display warning if reading history is set to "never"
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15096 Aleisha Amohia changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15096] Export today's checked in barcodes: Display warning if reading history is set to "never"
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15096 Marc Véron changed: What|Removed |Added Status|Signed Off |Needs Signoff --- Comment #11 from Marc Véron --- This patch needs a new sing-off. For new behaviour see new test plan (on patch). -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15096] Export today's checked in barcodes: Display warning if reading history is set to "never"
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15096 Marc Véron changed: What|Removed |Added Attachment #46501|0 |1 is obsolete|| --- Comment #10 from Marc Véron --- Created attachment 46591 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46591&action=edit Bug 15096: Export today's checked in barcodes: Display warning if reading history is set to "never" If a patron's privacy settings are set to "Never" for keeping a reading history, "Export today's checked in barcodes" returns an empty file. This patch does not allow to export in such case. To test: - Apply patch - Check out / check in in some items -Test toolbar: - On user's detail page, go to More->Export today's checked in barcodes - Verify that the menu item does not appear if syspref 'intranetreadinghistory' is set to 'Don't allow' - Verify that the menu item appears if syspref 'intranetreadinghistory' is set to allow and - that the menu item is grayed out with a tooltip if the user has set privacy settings to never keep a reading history - that the menu item works as before if user's privacy settings allow reading history. - Test left tab "Circulation history": - Verify that the left tab "Circulation history" does not appear if syspref 'intranetreadinghistory' is set to 'Don't allow' - Verify that the tab appars if syspref 'intranetreadinghistory' is set to 'Allow' and - that a message appears if user's privacy settings do not allow to keep the reading history - that the export works as before if user's privacy settings allow to keep the reading history (Amended and changed test plan for comment #9) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15096] Export today's checked in barcodes: Display warning if reading history is set to "never"
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15096
--- Comment #9 from Jonathan Druart
---
Comment on attachment 46501
--> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46501
Bug 15096: Export today's checked in barcodes: Display warning if reading
history is set to "never"
Review of attachment 46501:
-->
(http://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=15096&attachment=46501)
-
::: koha-tmpl/intranet-tmpl/prog/en/includes/members-toolbar.inc
@@ +118,5 @@
> }
> function export_barcodes() {
> +[% IF (
> + ( ! Koha.Preference('intranetreadinghistory') ) ||
> + ( Koha.Preference('AnonymousPatron') == ( borrowernumber) ) ||
Could you detail this use case? If you are logged in with the anonymous patron
why do you want to block the export?
@@ +122,5 @@
> + ( Koha.Preference('AnonymousPatron') == ( borrowernumber) ) ||
> + ( privacy == 2 )
> + )
> +%]
> +window.location =
> '/cgi-bin/koha/members/readingrec.pl?borrowernumber=[% borrowernumber %] ' ;
I'd prefer to disable the button on the same way as we do for the "update child
to adult patron" button.
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15096] Export today's checked in barcodes: Display warning if reading history is set to "never"
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15096 Marc Véron changed: What|Removed |Added See Also||http://bugs.koha-community. ||org/bugzilla3/show_bug.cgi? ||id=15552 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15096] Export today's checked in barcodes: Display warning if reading history is set to "never"
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15096
--- Comment #8 from Aleisha Amohia ---
(In reply to Marc Véron from comment #6)
> However, the explanation is not precise ('it is stored regardless'), I think
> it should be something like ('it is stored regardless if the patron has
> allowed to keep the reading history'). - That would be something for a
> separate bug.
See Bug 15552.
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15096] Export today's checked in barcodes: Display warning if reading history is set to "never"
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15096 Aleisha Amohia changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15096] Export today's checked in barcodes: Display warning if reading history is set to "never"
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15096 Aleisha Amohia changed: What|Removed |Added Attachment #45508|0 |1 is obsolete|| --- Comment #7 from Aleisha Amohia --- Created attachment 46501 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46501&action=edit Bug 15096: Export today's checked in barcodes: Display warning if reading history is set to "never" If a patron's privacy settings are set to "Never" for keeping a reading history, "Export today's checked in barcodes" returns an empty file. This patch skips the export if such privacy rule is set and dsiplays a message. To test: - Apply patch - Search for a user who has privacy rules set to never keeping a circulation history - Check out / check in in some items - On users detail page, go to More->Export today's checked in barcodes - Verify that a message appears about user's privacy settings (in same window) - Click left tab "Circulation history" - Verify that the same message appearsar - Verify the same with the other conditions that prevent the display of the user's circulation history: - Syspref 'intranetreadinghistory' is 'Don't allow' - Patron is 'Anonymous patron' (syspref 'Anonymous patron') - Search for a user who allows circulation history - repeat steps above, verify that circulation history appears (if allowed by syspref 'intranetreadinghistory' and that today's checkouts are exported. (Amended for comment #3) Signed-off-by: Aleisha -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15096] Export today's checked in barcodes: Display warning if reading history is set to "never"
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15096
--- Comment #6 from Marc Véron ---
(In reply to Aleisha Amohia from comment #5)
> When 'intranetreadinghistory' is set to "don't allow', the circulation
> history tab disappears - was this the intention? if yes, happy to sign off.
Just tested without patch, it is the same behaviour.
The explanation of 'intranetreadinghistory' is
"[Allow / don't] allow staff to access a patron's checkout history (it is
stored regardless)."
So it makes sense that the tab disappears when 'intranetreadinghistory' is set
to "don't allow".
However, the explanation is not precise ('it is stored regardless'), I think it
should be something like ('it is stored regardless if the patron has allowed to
keep the reading history'). - That would be something for a separate bug.
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15096] Export today's checked in barcodes: Display warning if reading history is set to "never"
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15096 --- Comment #5 from Aleisha Amohia --- When 'intranetreadinghistory' is set to "don't allow', the circulation history tab disappears - was this the intention? if yes, happy to sign off. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15096] Export today's checked in barcodes: Display warning if reading history is set to "never"
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15096 Marc Véron changed: What|Removed |Added Attachment #44269|0 |1 is obsolete|| --- Comment #4 from Marc Véron --- Created attachment 45508 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=45508&action=edit Bug 15096: Export today's checked in barcodes: Display warning if reading history is set to "never" If a patron's privacy settings are set to "Never" for keeping a reading history, "Export today's checked in barcodes" returns an empty file. This patch skips the export if such privacy rule is set and dsiplays a message. To test: - Apply patch - Search for a user who has privacy rules set to never keeping a circulation history - Check out / check in in some items - On users detail page, go to More->Export today's checked in barcodes - Verify that a message appears about user's privacy settings (in same window) - Click left tab "Circulation history" - Verify that the same message appearsar - Verify the same with the other conditions that prevent the display of the user's circulation history: - Syspref 'intranetreadinghistory' is 'Don't allow' - Patron is 'Anonymous patron' (syspref 'Anonymous patron') - Search for a user who allows circulation history - repeat steps above, verify that circulation history appears (if allowed by syspref 'intranetreadinghistory' and that today's checkouts are exported. (Amended for comment #3) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15096] Export today's checked in barcodes: Display warning if reading history is set to "never"
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15096 Aleisha Amohia changed: What|Removed |Added CC||aleishaamo...@hotmail.com --- Comment #3 from Aleisha Amohia --- This patch works as expected! My only concern/problem is that going to More->Export today's checked in barcodes for a patron with settings set to "never" for keeping a reading history opens in a new tab. It would be good to stay on the same tab instead of opening another one as this makes it confusing for the user and things get a bit messy! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15096] Export today's checked in barcodes: Display warning if reading history is set to "never"
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15096 --- Comment #2 from Marc Véron --- Still applies. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15096] Export today's checked in barcodes: Display warning if reading history is set to "never"
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15096 Marc Véron changed: What|Removed |Added See Also||http://bugs.koha-community. ||org/bugzilla3/show_bug.cgi? ||id=14482 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15096] Export today's checked in barcodes: Display warning if reading history is set to "never"
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15096 Marc Véron changed: What|Removed |Added Patch complexity|--- |Trivial patch Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15096] Export today's checked in barcodes: Display warning if reading history is set to "never"
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15096 --- Comment #1 from Marc Véron --- Created attachment 44269 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=44269&action=edit Bug 15096: Export today's checked in barcodes: Display warning if reading history is set to "never" If a patron's privacy settings are set to "Never" for keeping a reading history, "Export today's checked in barcodes" returns an empty file. This patch skips the export if such privacy rule is set and dsiplays a message. To test: - Apply patch - Search for a user who has privacy rules set to never keeping a circulation history - Check out / check in in some items - On users detail page, go to More->Export today's checked in barcodes - Verify that a message appears about user's privacy settings - Click left tab "Circulation hsitory" - Verify that the same message appears - Seach for a user who allows circulation hsitory - repeat steps above, verify that circulation hsitory appears and that today's checkouts are exported. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15096] Export today's checked in barcodes: Display warning if reading history is set to "never"
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15096 Marc Véron changed: What|Removed |Added Assignee|koha-b...@lists.koha-commun |ve...@veron.ch |ity.org | -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
