[Koha-bugs] [Bug 13497] The pref AddPatronLists should be removed
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13497 Jonathan Druart changed: What|Removed |Added See Also||http://bugs.koha-community. ||org/bugzilla3/show_bug.cgi? ||id=13458 -- 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 13497] The pref AddPatronLists should be removed
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13497 Jonathan Druart changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|gmcha...@gmail.com |jonathan.dru...@biblibre.co ||m Severity|enhancement |normal -- 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 13497] The pref AddPatronLists should be removed
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13497 Jonathan Druart changed: What|Removed |Added Status|ASSIGNED|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 13497] The pref AddPatronLists should be removed
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13497 --- Comment #1 from Jonathan Druart --- Created attachment 34817 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=34817&action=edit Bug 13497: Remove the pref AddPatronsLists The pref AddPatronLists does not work as expected if 'generic' is selected. Indeed a patron should be linked to a valid patron category (see the constraint borrowers_ibfk_1 in the DB structure). Test plan: 0/ Confirm that the AddPatronLists does not work as expected if you choose the generic patron types. 1/ Delete all you patron categories and check that the interface invites you to create one. 2/ Confirm that you are able to create a patron and to link it to a patron category you have created. -- 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 13497] The pref AddPatronLists should be removed
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13497 Marc Véron changed: What|Removed |Added CC||ve...@veron.ch --- Comment #2 from Marc Véron --- Does not apply: CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/includes/patron-toolbar.inc Auto-merging installer/data/mysql/updatedatabase.pl CONFLICT (content): Merge conflict in installer/data/mysql/updatedatabase.pl -- 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 13497] The pref AddPatronLists should be removed
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13497 Jonathan Druart changed: What|Removed |Added Attachment #34817|0 |1 is obsolete|| --- Comment #3 from Jonathan Druart --- Created attachment 36021 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=36021&action=edit Bug 13497: Remove the pref AddPatronsLists The pref AddPatronLists does not work as expected if 'generic' is selected. Indeed a patron should be linked to a valid patron category (see the constraint borrowers_ibfk_1 in the DB structure). Test plan: 0/ Confirm that the AddPatronLists does not work as expected if you choose the generic patron types. 1/ Delete all you patron categories and check that the interface invites you to create one. 2/ Confirm that you are able to create a patron and to link it to a patron category you have created. -- 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 13497] The pref AddPatronLists should be removed
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13497 --- Comment #4 from Marc Véron --- Created attachment 36053 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=36053&action=edit [Signed-off] Bug 13497: Remove the pref AddPatronsLists The pref AddPatronLists does not work as expected if 'generic' is selected. Indeed a patron should be linked to a valid patron category (see the constraint borrowers_ibfk_1 in the DB structure). Test plan: 0/ Confirm that the AddPatronLists does not work as expected if you choose the generic patron types. 1/ Delete all you patron categories and check that the interface invites you to create one. 2/ Confirm that you are able to create a patron and to link it to a patron category you have created. I could not verify step 1/, because at least one Patron category can not be deleted (Staff, since I'm logged in as a Staff patron and you can not delete categories that are in use). Tested 0/ and 2/ Signed-off-by: Marc Véron -- 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 13497] The pref AddPatronLists should be removed
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13497 Marc Véron changed: What|Removed |Added Attachment #36021|0 |1 is obsolete|| -- 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 13497] The pref AddPatronLists should be removed
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13497 Marc Véron changed: What|Removed |Added Patch complexity|--- |Small patch 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 13497] The pref AddPatronLists should be removed
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13497 Katrin Fischer changed: What|Removed |Added Attachment #36053|0 |1 is obsolete|| --- Comment #5 from Katrin Fischer --- Created attachment 36360 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=36360&action=edit [PASSED QA] Bug 13497: Remove the pref AddPatronsLists The pref AddPatronLists does not work as expected if 'generic' is selected. Indeed a patron should be linked to a valid patron category (see the constraint borrowers_ibfk_1 in the DB structure). Test plan: 0/ Confirm that the AddPatronLists does not work as expected if you choose the generic patron types. 1/ Delete all you patron categories and check that the interface invites you to create one. 2/ Confirm that you are able to create a patron and to link it to a patron category you have created. I could not verify step 1/, because at least one Patron category can not be deleted (Staff, since I'm logged in as a Staff patron and you can not delete categories that are in use). Tested 0/ and 2/ Signed-off-by: Marc Véron Signed-off-by: Katrin Fischer Passes QA script and tests. Current patch-complexity: Small 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 13497] The pref AddPatronLists should be removed
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13497 Katrin Fischer changed: What|Removed |Added Status|Signed Off |In Discussion CC||katrin.fisc...@bsz-bw.de --- Comment #6 from Katrin Fischer --- Hi Jonathan, I am happy with the patch, but I am having some doubts if we understand how this was meant to work correctly. Would you mind sending a short email to the Koha mailing list asking people about removing it/a use case where this is needed for? -- 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 13497] The pref AddPatronLists should be removed
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13497 --- Comment #7 from Jonathan Druart --- (In reply to Katrin Fischer from comment #6) > Hi Jonathan, > > I am happy with the patch, but I am having some doubts if we understand how > this was meant to work correctly. Would you mind sending a short email to > the Koha mailing list asking people about removing it/a use case where this > is needed for? I am not sure it's necessary, the pref has been introduced by bug 1782, the default value is categorycode, and it does not work if category_type if defined. Actually I think this pref should have never been created, and it does not make sense since you need to set the borrowers.categorycode with a valid category code. -- 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 13497] The pref AddPatronLists should be removed
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13497 Katrin Fischer changed: What|Removed |Added Status|In Discussion |Passed QA --- Comment #8 from Katrin Fischer --- I guess I am just worried I misunderstood how it should work - looked for other related bugs, but had not much luck. -- 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 13497] The pref AddPatronLists should be removed
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13497 Jonathan Druart changed: What|Removed |Added See Also||http://bugs.koha-community. ||org/bugzilla3/show_bug.cgi? ||id=9314 -- 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 13497] The pref AddPatronLists should be removed
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13497 Jonathan Druart changed: What|Removed |Added Blocks||9314 -- 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 13497] The pref AddPatronLists should be removed
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13497 Jonathan Druart changed: What|Removed |Added Blocks||13970 -- 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 13497] The pref AddPatronLists should be removed
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13497 Jonathan Druart changed: What|Removed |Added Attachment #36360|0 |1 is obsolete|| --- Comment #9 from Jonathan Druart --- Created attachment 37755 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=37755&action=edit Bug 13497: Remove the pref AddPatronsLists The pref AddPatronLists does not work as expected if 'generic' is selected. Indeed a patron should be linked to a valid patron category (see the constraint borrowers_ibfk_1 in the DB structure). Test plan: 0/ Confirm that the AddPatronLists does not work as expected if you choose the generic patron types. 1/ Delete all you patron categories and check that the interface invites you to create one. 2/ Confirm that you are able to create a patron and to link it to a patron category you have created. I could not verify step 1/, because at least one Patron category can not be deleted (Staff, since I'm logged in as a Staff patron and you can not delete categories that are in use). Tested 0/ and 2/ Signed-off-by: Marc Véron Signed-off-by: Katrin Fischer Passes QA script and tests. -- 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 13497] The pref AddPatronLists should be removed
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13497 --- Comment #10 from Tomás Cohen Arazi --- Created attachment 37758 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=37758&action=edit Bug 13497: (Maniac followup) spell category in english Signed-off-by: Tomas Cohen Arazi -- 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 13497] The pref AddPatronLists should be removed
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13497 Tomás Cohen Arazi changed: What|Removed |Added Status|Passed QA |Pushed to Master CC||tomasco...@gmail.com --- Comment #11 from Tomás Cohen Arazi --- Patch pushed to master. Thanks Jonathan! -- 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/