[Koha-bugs] [Bug 26290] Allow setting default SMTP parameters in koha-conf.xml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26290 Jonathan Druart changed: What|Removed |Added Status|Passed QA |Pushed to master Version(s)||20.11.00 released in|| -- 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 26290] Allow setting default SMTP parameters in koha-conf.xml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26290 --- Comment #16 from Jonathan Druart --- Pushed to master for 20.11, 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 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 26290] Allow setting default SMTP parameters in koha-conf.xml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26290 --- Comment #15 from Tomás Cohen Arazi --- (In reply to Jonathan Druart from comment #14) > Created attachment 41 [details] [review] > Bug 26290: Display the correct default values on the admin page > > The "Default configuration" section at the top of the SMTP servers admin page > contains the hardcoded values in Koha::SMTP::Servers->default_setting. > > It should be the values from the config (and default to hardcoded values > unless defined) Thanks! I really missed it! -- 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 26290] Allow setting default SMTP parameters in koha-conf.xml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26290 --- Comment #14 from Jonathan Druart --- Created attachment 41 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=41=edit Bug 26290: Display the correct default values on the admin page The "Default configuration" section at the top of the SMTP servers admin page contains the hardcoded values in Koha::SMTP::Servers->default_setting. It should be the values from the config (and default to hardcoded values unless defined) -- 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 26290] Allow setting default SMTP parameters in koha-conf.xml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26290 Martin Renvoize changed: What|Removed |Added Attachment #109949|0 |1 is obsolete|| --- Comment #12 from Martin Renvoize --- Created attachment 110760 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=110760=edit Bug 26290: Make Makefile.PL aware of SMTP configs Signed-off-by: Martin Renvoize -- 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 26290] Allow setting default SMTP parameters in koha-conf.xml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26290 Martin Renvoize changed: What|Removed |Added Attachment #109618|0 |1 is obsolete|| --- Comment #11 from Martin Renvoize --- Created attachment 110759 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=110759=edit Bug 26290: Make Koha::SMTP::Servers->get_default use koha-conf.xml This patch makes the mentioned method use the configured entry in koha-conf.xml if present, instead of the hardcoded one, which is kept for people that carries an old koha-conf.xml Signed-off-by: Kyle M Hall Signed-off-by: Martin Renvoize -- 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 26290] Allow setting default SMTP parameters in koha-conf.xml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26290 Martin Renvoize changed: What|Removed |Added Attachment #109617|0 |1 is obsolete|| --- Comment #10 from Martin Renvoize --- Created attachment 110758 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=110758=edit Bug 26290: Add option forsetting default SMTP parameters in koha-conf.xml This patch adds an entry to koha-conf.xml for setting a default SMTP server config. 'koha-create' gets option switches for all options. To test: 1. Use the tweaked script to crete a new instance 2. Notice it contains entries for smtp_server and they respect your options. 3. If you don't choose any option, it sets the current default. Signed-off-by: Kyle M Hall Signed-off-by: Martin Renvoize -- 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 26290] Allow setting default SMTP parameters in koha-conf.xml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26290 Martin Renvoize changed: What|Removed |Added CC||martin.renvoize@ptfs-europe ||.com Status|Signed Off |Passed QA --- Comment #13 from Martin Renvoize --- All works well for me and tests are passing. I worked through a dev install, standard install and a package build.. all good. Passing QA -- 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 26290] Allow setting default SMTP parameters in koha-conf.xml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26290 --- Comment #9 from Tomás Cohen Arazi --- Created attachment 109949 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=109949=edit Bug 26290: Make Makefile.PL aware of SMTP configs -- 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 26290] Allow setting default SMTP parameters in koha-conf.xml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26290 --- Comment #8 from Tomás Cohen Arazi --- (In reply to Julian Maurice from comment #7) > (In reply to Tomás Cohen Arazi from comment #6) > > I wasn't sure it was useful. Do you think so? > > I guess that if it's useful to debian package users, it should be useful to > non-debian package users too. > That can be added in a follow-up bug, but I'm a bit worried about this > pattern of adding features to debian packages and not in Koha "core". In the > long term it could mean that it would be more and more difficult to install > Koha on non-debian systems. Hold on. Its been a while since I touched the Makefile.PL, and I'm testing my follow-up. -- 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 26290] Allow setting default SMTP parameters in koha-conf.xml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26290 --- Comment #7 from Julian Maurice --- (In reply to Tomás Cohen Arazi from comment #6) > I wasn't sure it was useful. Do you think so? I guess that if it's useful to debian package users, it should be useful to non-debian package users too. That can be added in a follow-up bug, but I'm a bit worried about this pattern of adding features to debian packages and not in Koha "core". In the long term it could mean that it would be more and more difficult to install Koha on non-debian systems. -- 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 26290] Allow setting default SMTP parameters in koha-conf.xml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26290 --- Comment #6 from Tomás Cohen Arazi --- (In reply to Julian Maurice from comment #5) > The SMTP server section can be entirely configured by a debian script, but > the default etc/koha-conf.xml used by the "perl Makefile.PL" method cannot > be configured at all. > Is there a specific reason for that ? I wasn't sure it was useful. Do you think so? -- 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 26290] Allow setting default SMTP parameters in koha-conf.xml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26290 Julian Maurice changed: What|Removed |Added CC||julian.maur...@biblibre.com --- Comment #5 from Julian Maurice --- The SMTP server section can be entirely configured by a debian script, but the default etc/koha-conf.xml used by the "perl Makefile.PL" method cannot be configured at all. Is there a specific reason for that ? -- 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 26290] Allow setting default SMTP parameters in koha-conf.xml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26290 --- Comment #4 from Kyle M Hall --- Created attachment 109618 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=109618=edit Bug 26290: Make Koha::SMTP::Servers->get_default use koha-conf.xml This patch makes the mentioned method use the configured entry in koha-conf.xml if present, instead of the hardcoded one, which is kept for people that carries an old koha-conf.xml 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 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 26290] Allow setting default SMTP parameters in koha-conf.xml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26290 Kyle M Hall changed: What|Removed |Added Attachment #109035|0 |1 is obsolete|| Attachment #109036|0 |1 is obsolete|| --- Comment #3 from Kyle M Hall --- Created attachment 109617 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=109617=edit Bug 26290: Add option forsetting default SMTP parameters in koha-conf.xml This patch adds an entry to koha-conf.xml for setting a default SMTP server config. 'koha-create' gets option switches for all options. To test: 1. Use the tweaked script to crete a new instance 2. Notice it contains entries for smtp_server and they respect your options. 3. If you don't choose any option, it sets the current default. 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 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 26290] Allow setting default SMTP parameters in koha-conf.xml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26290 Kyle M Hall 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 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 26290] Allow setting default SMTP parameters in koha-conf.xml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26290 --- Comment #2 from Tomás Cohen Arazi --- Created attachment 109036 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=109036=edit Bug 26290: Make Koha::SMTP::Servers->get_default use koha-conf.xml This patch makes the mentioned method use the configured entry in koha-conf.xml if present, instead of the hardcoded one, which is kept for people that carries an old koha-conf.xml -- 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 26290] Allow setting default SMTP parameters in koha-conf.xml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26290 --- Comment #1 from Tomás Cohen Arazi --- Created attachment 109035 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=109035=edit Bug 26290: Add option forsetting default SMTP parameters in koha-conf.xml This patch adds an entry to koha-conf.xml for setting a default SMTP server config. 'koha-create' gets option switches for all options. To test: 1. Use the tweaked script to crete a new instance 2. Notice it contains entries for smtp_server and they respect your options. 3. If you don't choose any option, it sets the current default. -- 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 26290] Allow setting default SMTP parameters in koha-conf.xml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26290 Tomás Cohen Arazi changed: 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 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 26290] Allow setting default SMTP parameters in koha-conf.xml
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26290 Tomás Cohen Arazi changed: What|Removed |Added Assignee|koha-b...@lists.koha-commun |tomasco...@gmail.com |ity.org | Depends on||22343 Status|NEW |ASSIGNED Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22343 [Bug 22343] Add SMTP configuration options to Administration -- 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 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/
