http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9002 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff Assignee|koha-bugs@lists.koha-commun |dcook@prosentient.com.au |ity.org | --- Comment #1 from David Cook <dcook@prosentient.com.au> --- Created attachment 13159 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=13159&action=edit Bug 9002 - Remove Problematic Logic from Patron Messaging Preferences Form It's a super basic code removal. In most cases, its presence would probably not cause a problem. Currently, if SMSSendDriver isn't set, then there is no SMS column and no SMS checkbox. However, the logic says that if there is a message_transport_type, there should be a checkbox, and if there is not, there should be a hyphen. Since there is no heading, this could cause layout issues for people without the right message_transport_types. It makes sense to use SMSSendDriver as the condition rather than the message_transport_type... -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9002 Melia Meggs <melia@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |melia@bywatersolutions.com --- Comment #2 from Melia Meggs <melia@bywatersolutions.com> --- Hi David, I wanted to test your patch, but I am not sure I understand the bug. I am testing on master. I have SMSSendDriver blank in my system preferences. I have attached screenshots of what my patron messaging preferences look like in the staff client and in the OPAC. I think my columns look ok. Is there some other system preference that I need to change that will knock things out of alignment? Or is it maybe a browser specific issue? (I'm using Chrome.) Thanks, Melia -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9002 --- Comment #4 from Melia Meggs <melia@bywatersolutions.com> --- Created attachment 13302 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=13302&action=edit OPAC - patron messaging preferences -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9002 --- Comment #6 from David Cook <dcook@prosentient.com.au> --- (In reply to comment #5)

(In reply to comment #2)

...

Is there some other system preference that I need to change that will knock things out of alignment?

I don't see the problem in my installation either. From the description this sounds like a problem we had some time ago and fixed.

Thanks for your comments Melia and Owen. I should've mentioned that message_transport_type is set in the database, so you would have to delete the message_transport_type of "sms" from your database in order to see the layout problem. I've done some additional research, and I think the message_transport_type of "sms" was introduced back around version 3.1, so it "shouldn't" be missing from the database. I'm not sure why it was missing in the case I encountered. I think 99% of the time, the logic I outlined above wouldn't be a problem. This patch is for the other 1% and just to remove senseless code. To test it...I would suggest applying the patch and then try it with SMSSendDrive turns on and with it turned off. If it doesn't break anything, I would say that it's working. -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9002 --- Comment #8 from Sophie MEYNIEUX <sophie.meynieux@biblibre.com> --- Created attachment 16226 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=16226&action=edit Bug 9002 - Remove Problematic Logic from Patron Messaging Preferences Form If you do not have SMSSendDriver set and you do not have a message_transport_type of "sms", you will get an extra table column filled with a hyphen "-" and no corresponding table heading in the Patron Messaging Preferences table. This pushes the table out of alignment so the email checkbox goes under "Digests Only?", the digests checkbox goes under "Do not notify", and the Do not notify checkbox is pushed to the side with no heading. Signed-off-by: Sophie Meynieux <sophie.meynieux@biblibre.com> -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9002 Sophie MEYNIEUX <sophie.meynieux@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off CC| |sophie.meynieux@biblibre.co | |m --- Comment #9 from Sophie MEYNIEUX <sophie.meynieux@biblibre.com> --- Without the patch, if SMSSendDriver is blank, there is an extra column without heading in Messaging preferences table and column content is not under the correct heading. If SMSDriver is not blank, there is a SMS column and the table is correctly displaid. With the patch, if SMSSendDriver is not blanck, the table is correctly displaid with a SMS column, and if SMSSendDriver is blank, the table is correctly displaid without SMS column. For me, having or not 'sms' in message_transport_types has no consequences. Signed-off -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9002 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9002 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master --- Comment #11 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- This patch has been pushed to master. -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9002 --- Comment #13 from Sophie MEYNIEUX <sophie.meynieux@biblibre.com> --- As opac template does not use messaging-preference-form.inc, this problem still exists on OPAC -- You are receiving this mail because: You are watching all bug changes.