[Koha-bugs] [Bug 14206] notices using non email templates can't be deleted from the staff client

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14206

--- Comment #9 from Marc Véron <veron at veron.ch> ---
Created attachment 39321
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=39321&action=edit
[Signed-off] Bug 14206 Adds two tests to t/db_dependent/Letters.t

C4::Letters::getletter is called in tools/letter.pl by the function
delete_confirm() to display the selected notice for deletion. Due to
current implementation of getletter(), a notice that does not use
the 'email' template (but uses any/all of the other templates - sms,
print or phone) can't be deleted from the staff client.

This patch adds 2 tests to t/db_dependent/Letters.t to test the bug.

Test plan
=========

1/ Apply the patch and run prove -v t/db_dependent/Letters.t
2/ Test 38 should pass demonstrating unpatched getletter() behaviour
   Test 39 should FAIL, highlighting the problem

     #   Failed test 'Bug 14206 - returns actual message_transport_type'
     #   at t/db_dependent/Letters.t line 181.
     #          got: undef
     #     expected: 'print'

(Test plan continues in the second commit)

Followed test plan, works as expected
Signed-off-by: Marc Véron <veron at veron.ch>

-- 
You are receiving this mail because:
You are watching all bug changes.