Koha::Email has a typo for the Reply-To header!
Created attachment 118661 [details] [review] Bug 28023: Regression tests Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Created attachment 118662 [details] [review] Bug 28023: Fix Reply-To typo This patch fixes a typo making the header useless. To test: 1. Apply the regression tests patch 2. Run: $ kshell k$ prove t/Koha/Email.t => FAIL: The Reply-To header is not being set! 3. Apply this patch 4. Repeat 2 => SUCCESS: Tests pass! the header is set! 5. Sign off :-D
Created attachment 118666 [details] [review] Bug 28023: Regression tests Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Created attachment 118667 [details] [review] Bug 28023: Fix Reply-To typo This patch fixes a typo making the header useless. To test: 1. Apply the regression tests patch 2. Run: $ kshell k$ prove t/Koha/Email.t => FAIL: The Reply-To header is not being set! 3. Apply this patch 4. Repeat 2 => SUCCESS: Tests pass! the header is set! 5. Sign off :-D Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Trivial fix, going straight for QA on this one. Works as expected, obvious oversight in original implementation.. contains tests. Passing.
Pushed to master for 21.05, thanks to everybody involved!
Pushed to 20.11.x for 20.11.05
Doesn't apply to 20.05. Please rebase if needed