[Koha-bugs] [Bug 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 Thibaud Guillot changed: What|Removed |Added Attachment #128400|1 |0 is patch|| Attachment #128400|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 https://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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 Thibaud Guillot changed: What|Removed |Added CC||thibaud.guil...@biblibre.co ||m --- Comment #19 from Thibaud Guillot --- Sorry for disturbing I miss my last bug attachement with git bz :/ -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 --- Comment #18 from Thibaud Guillot --- Created attachment 128400 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=128400=edit Add syspref to deal with overdues notification I took the same test plan as victor but I added the system preference to manage the case more easily, especially for users who do not have access to the koha server. Test plan 1. Check the size of the message queue With the following SQL query (using an SQL report if you want) SELECT COUNT(*) FROM message_queue; 2. Run misc/cronjobs/overdue_notices.pl 3. Check the size of the message queue To ensure that no other overdues will create noise in this test plan. Or you can take them into account. 4. Choose a patron with no email address 5. Create an overdue (checkout an item and unfold "Checkout settings" and set a date in the past which is compatible with what you find in staff:/cgi-bin/koha/tools/overduerules.pl 6. Run misc/cronjobs/overdue_notices.pl 7. Check that you have two new messages in the queue 8. Inspect these two messages SELECT * FROM message_queue ORDER BY time_queued DESC LIMIT 2 \G 1. One has the type "print" and the borrowernumber matching the patron. 2. The other has subject: Overdue Notices borrowernumber: NULL message_transport_type: email and contains "These messages were not sent directly to the patrons." This is the one we don't want anymore. Because it's now obsolete due to the first message. 9. Apply this patch 10. Change syspref 'EmailOverduesNoEmail' to "Don't send" 11. Delete data from message_queue (if you have access) for a cleaner view 12. Run again misc/cronjobs/overdue_notices.pl 13. Check that only the print message is now generated and not the "Overdue Notices" one. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 Jonathan Druartchanged: What|Removed |Added Keywords|RM_priority | -- 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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 Jonathan Druartchanged: What|Removed |Added Status|Passed QA |Pushed to Master --- Comment #17 from Jonathan Druart --- Pushed to master for 18.05, thanks to everybody involved! -- 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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 --- Comment #16 from Marcel de Rooy--- If you agree, Jonathan, take the last patch along coming from 20075. It fits better here.. -- 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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 --- Comment #15 from Marcel de Rooy--- Created attachment 72209 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72209=edit Bug 20074: (Follow-up) Replace hidden value -5 by 1 in data and interface [ Moved from 20075 to 20074 ] Since hidden is used as a boolean, it makes more sense to save the values 0 and 1 instead of 0 and -5. Test plan: Test toggling between Show all and Hide all in the auth frameworks. Run the db rev. Signed-off-by: Marcel de Rooy -- 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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 --- Comment #14 from Marcel de Rooy--- (In reply to Katrin Fischer from comment #13) > (In reply to Marcel de Rooy from comment #12) > > (In reply to Katrin Fischer from comment #11) > > > The database update will take care of the problem for existing > > > installations, but we will reintroduce the problem in every new > > > installation > > > as long as bug 20075 is not fixed. Maybe we should hold off the database > > > update here or perform it a second time with the fix for the frameworks? > > > > Thx Katrin. > > The problem we solve here, is not reintroduced since we improved the logic. > > I guess it is still possible to get 20075 also into 18.05, but we could > > repeat the db update on 20075 too altough is has only theoretical value for > > purists. > > Poor choice of words - "reintroduce unclean data" would have been better :) OK. I submitted a few patches on 20075 to obtain clean data. -- 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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 --- Comment #13 from Katrin Fischer--- (In reply to Marcel de Rooy from comment #12) > (In reply to Katrin Fischer from comment #11) > > The database update will take care of the problem for existing > > installations, but we will reintroduce the problem in every new installation > > as long as bug 20075 is not fixed. Maybe we should hold off the database > > update here or perform it a second time with the fix for the frameworks? > > Thx Katrin. > The problem we solve here, is not reintroduced since we improved the logic. > I guess it is still possible to get 20075 also into 18.05, but we could > repeat the db update on 20075 too altough is has only theoretical value for > purists. Poor choice of words - "reintroduce unclean data" would have been better :) -- 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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 --- Comment #12 from Marcel de Rooy--- (In reply to Katrin Fischer from comment #11) > The database update will take care of the problem for existing > installations, but we will reintroduce the problem in every new installation > as long as bug 20075 is not fixed. Maybe we should hold off the database > update here or perform it a second time with the fix for the frameworks? Thx Katrin. The problem we solve here, is not reintroduced since we improved the logic. I guess it is still possible to get 20075 also into 18.05, but we could repeat the db update on 20075 too altough is has only theoretical value for purists. -- 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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 Katrin Fischerchanged: 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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 Katrin Fischerchanged: What|Removed |Added CC||katrin.fisc...@bsz-bw.de Blocks||20075 --- Comment #11 from Katrin Fischer --- The database update will take care of the problem for existing installations, but we will reintroduce the problem in every new installation as long as bug 20075 is not fixed. Maybe we should hold off the database update here or perform it a second time with the fix for the frameworks? Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20075 [Bug 20075] Change authority hidden attribute in sql installer files -- 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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 Katrin Fischerchanged: What|Removed |Added Attachment #71928|0 |1 is obsolete|| --- Comment #10 from Katrin Fischer --- Created attachment 72193 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72193=edit Bug 20074: (Follow-up) Remove another remainder of biblio hidden logic Script authorities.pl still contains a module 2 calculation on the hidden attribute, although it is used as a boolean. Since -5 mod 2 == 1, it does no harm, but we better remove it. Test plan: Edit and save an authority. Verify that everything still works as expected. Signed-off-by: Josef Moravec Signed-off-by: Katrin Fischer -- 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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 Katrin Fischerchanged: What|Removed |Added Attachment #71927|0 |1 is obsolete|| --- Comment #9 from Katrin Fischer --- Created attachment 72192 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72192=edit Bug 20074: Database revision to convert hidden to -5 The hidden value is used as a boolean in authority frameworks (in contrast with biblio framework). It is consistent to apply the same value (-5) to all non-zero values. Test plan: Check if you a have 0, 1 and 8 value in hidden somewhere. Run the dbrev. Verify that 1 and 8 are converted to -5. Signed-off-by: Marcel de Rooy Signed-off-by: Josef Moravec Signed-off-by: Katrin Fischer -- 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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 Katrin Fischerchanged: What|Removed |Added Attachment #71926|0 |1 is obsolete|| --- Comment #8 from Katrin Fischer --- Created attachment 72191 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72191=edit Bug 20074: Fix hidden value in auth_subfields_structure The template included an else branch where a value not equal to 0 or -5 was interpreted as 0 (Show all). This effectively converted hidden values 1 and 8 to 0. This patch removes that else branch. Test plan: Toggle a authority field between Hide all and Show all. Check what happens. Signed-off-by: Marcel de Rooy Signed-off-by: Katrin Fischer -- 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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 --- Comment #6 from Josef Moravec--- Created attachment 71927 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=71927=edit Bug 20074: Database revision to convert hidden to -5 The hidden value is used as a boolean in authority frameworks (in contrast with biblio framework). It is consistent to apply the same value (-5) to all non-zero values. Test plan: Check if you a have 0, 1 and 8 value in hidden somewhere. Run the dbrev. Verify that 1 and 8 are converted to -5. Signed-off-by: Marcel de Rooy Signed-off-by: Josef Moravec -- 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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 Josef Moravecchanged: What|Removed |Added Attachment #70842|0 |1 is obsolete|| Attachment #70843|0 |1 is obsolete|| Attachment #70847|0 |1 is obsolete|| --- Comment #5 from Josef Moravec --- Created attachment 71926 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=71926=edit Bug 20074: Fix hidden value in auth_subfields_structure The template included an else branch where a value not equal to 0 or -5 was interpreted as 0 (Show all). This effectively converted hidden values 1 and 8 to 0. This patch removes that else branch. Test plan: Toggle a authority field between Hide all and Show all. Check what happens. Signed-off-by: Marcel de Rooy -- 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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 Josef Moravecchanged: 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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 --- Comment #7 from Josef Moravec--- Created attachment 71928 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=71928=edit Bug 20074: (Follow-up) Remove another remainder of biblio hidden logic Script authorities.pl still contains a module 2 calculation on the hidden attribute, although it is used as a boolean. Since -5 mod 2 == 1, it does no harm, but we better remove it. Test plan: Edit and save an authority. Verify that everything still works as expected. Signed-off-by: Josef Moravec -- 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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 --- Comment #4 from Marcel de Rooy--- Created attachment 70847 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70847=edit Bug 20074: (Follow-up) Remove another remainder of biblio hidden logic Script authorities.pl still contains a module 2 calculation on the hidden attribute, although it is used as a boolean. Since -5 mod 2 == 1, it does no harm, but we better remove it. Test plan: Edit and save an authority. Verify that everything still works as expected. -- 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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 Jonathan Druartchanged: What|Removed |Added Keywords||RM_priority CC||jonathan.dru...@bugs.koha-c ||ommunity.org -- 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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 --- Comment #3 from Marcel de Rooy--- Another improvement would be to correct the sql installer files. See bug 20075 -- 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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 --- Comment #1 from Marcel de Rooy--- Created attachment 70842 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70842=edit Bug 20074: Fix hidden value in auth_subfields_structure The template included an else branch where a value not equal to 0 or -5 was interpreted as 0 (Show all). This effectively converted hidden values 1 and 8 to 0. This patch removes that else branch. Test plan: Toggle a authority field between Hide all and Show all. Check what happens. Signed-off-by: Marcel de Rooy -- 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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 Marcel de Rooychanged: What|Removed |Added Patch complexity|--- |Small patch 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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 --- Comment #2 from Marcel de Rooy--- Created attachment 70843 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70843=edit Bug 20074: Database revision to convert hidden to -5 The hidden value is used as a boolean in authority frameworks (in contrast with biblio framework). It is consistent to apply the same value (-5) to all non-zero values. Test plan: Check if you a have 0, 1 and 8 value in hidden somewhere. Run the dbrev. Verify that 1 and 8 are converted to -5. Signed-off-by: Marcel de Rooy -- 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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 Marcel de Rooychanged: What|Removed |Added Depends on||18811 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18811 [Bug 18811] Visibility settings inconsistent between framework and authority editor -- 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 20074] Auth_subfield_structure changes hidden attribute
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20074 Marcel de Rooychanged: What|Removed |Added Assignee|koha-b...@lists.koha-commun |m.de.r...@rijksmuseum.nl |ity.org | Status|NEW |ASSIGNED -- You are receiving this mail because: You are the assignee for the bug. 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/