Jonathan Druart
86f999812e
To test: 1. Enable multi-languages 2. Set the preference 'TranslateNotices' on 'Allow' 3. Go to: tools==>Notices & slips==>Edit, make sure it has multilingual email templates. 4. Set the preference TranslateNotices on 'Don't allow'. 5. Go to: tools==>Notices & slips==>Edit, the template shows several tab for the same transport type. 6. Apply the patch. 7. Repeat the steps 4 and 5 8. Success. It only shows the default template when TranslateNotices is 'Dont allow'. Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
136 lines
5.5 KiB
Perl
136 lines
5.5 KiB
Perl
use Modern::Perl;
|
|
use Test::More tests => 8;
|
|
|
|
use C4::Context;
|
|
use C4::Letters qw( GetLetterTemplates );
|
|
use Koha::Database;
|
|
use t::lib::Mocks;
|
|
|
|
my $schema = Koha::Database->new->schema;
|
|
$schema->storage->txn_begin;
|
|
my $dbh = C4::Context->dbh;
|
|
|
|
$dbh->do(q|DELETE FROM letter|);
|
|
|
|
my $letters = [
|
|
{
|
|
module => 'circulation',
|
|
code => 'code1',
|
|
branchcode => '',
|
|
name => 'B default name for code1 circ',
|
|
is_html => 0,
|
|
title => 'default title for code1 email',
|
|
content => 'default content for code1 email',
|
|
message_transport_type => 'email',
|
|
lang => 'default',
|
|
},
|
|
{
|
|
module => 'circulation',
|
|
code => 'code1',
|
|
branchcode => '',
|
|
name => 'B default name for code1 circ',
|
|
is_html => 0,
|
|
title => 'default title for code1 sms',
|
|
content => 'default content for code1 sms',
|
|
message_transport_type => 'sms',
|
|
lang => 'es-ES',
|
|
},
|
|
{
|
|
module => 'circulation',
|
|
code => 'code2',
|
|
branchcode => '',
|
|
name => 'A default name for code2 circ',
|
|
is_html => 0,
|
|
title => 'default title for code2 email',
|
|
content => 'default content for code2 email',
|
|
message_transport_type => 'email',
|
|
lang => 'default',
|
|
},
|
|
{
|
|
module => 'circulation',
|
|
code => 'code3',
|
|
branchcode => '',
|
|
name => 'C default name for code3 circ',
|
|
is_html => 0,
|
|
title => 'default title for code3 email',
|
|
content => 'default content for code3 email',
|
|
message_transport_type => 'email',
|
|
lang => 'default',
|
|
},
|
|
|
|
{
|
|
module => 'cataloguing',
|
|
code => 'code1',
|
|
branchcode => '',
|
|
name => 'default name for code1 cat',
|
|
is_html => 0,
|
|
title => 'default title for code1 cat email',
|
|
content => 'default content for code1 cat email',
|
|
message_transport_type => 'email',
|
|
lang => 'default',
|
|
},
|
|
|
|
{
|
|
module => 'circulation',
|
|
code => 'code1',
|
|
branchcode => 'CPL',
|
|
name => 'B CPL name for code1 circ',
|
|
is_html => 0,
|
|
title => 'CPL title for code1 email',
|
|
content => 'CPL content for code1 email',
|
|
message_transport_type => 'email',
|
|
lang => 'default',
|
|
},
|
|
{
|
|
module => 'circulation',
|
|
code => 'code2',
|
|
branchcode => 'CPL',
|
|
name => 'A CPL name for code1 circ',
|
|
is_html => 0,
|
|
title => 'CPL title for code1 sms',
|
|
content => 'CPL content for code1 sms',
|
|
message_transport_type => 'sms',
|
|
lang => 'default',
|
|
},
|
|
];
|
|
|
|
my $sth = $dbh->prepare(
|
|
q|INSERT INTO letter(module, code, branchcode, name, title, content, message_transport_type, lang) VALUES (?, ?, ?, ?, ?, ?, ?, ?)|
|
|
);
|
|
for my $l (@$letters) {
|
|
$sth->execute( $l->{module}, $l->{code}, $l->{branchcode}, $l->{name},
|
|
$l->{title}, $l->{content}, $l->{message_transport_type}, $l->{lang} );
|
|
}
|
|
|
|
t::lib::Mocks::mock_preference('TranslateNotices', 1);
|
|
my $letter_templates;
|
|
$letter_templates = C4::Letters::GetLetterTemplates;
|
|
is_deeply( $letter_templates, [],
|
|
'GetLetterTemplates should not return templates if not param is given' );
|
|
|
|
$letter_templates = C4::Letters::GetLetterTemplates(
|
|
{ module => 'circulation', code => 'code1', branchcode => '' } );
|
|
is( scalar( @$letter_templates ),
|
|
2, '2 default templates should exist for circulation code1' );
|
|
my $has_email = grep { $_->{message_transport_type} eq 'email' } @$letter_templates;
|
|
is( $has_email, 1, 'The mtt email should exist for circulation code1' );
|
|
my $has_sms = grep { $_->{message_transport_type} eq 'sms' } @$letter_templates;
|
|
is( $has_sms, 1, 'The mtt sms should exist for circulation code1' );
|
|
|
|
$letter_templates = C4::Letters::GetLetterTemplates(
|
|
{ module => 'circulation', code => 'code1', branchcode => 'CPL' } );
|
|
is( scalar( @$letter_templates ),
|
|
1, '1 template should exist for circulation CPL code1' );
|
|
$has_email = grep { $_->{message_transport_type} eq 'email' } @$letter_templates;
|
|
is( $has_email, 1, 'The mtt should be email for circulation CPL code1' );
|
|
|
|
$letter_templates = C4::Letters::GetLetterTemplates(
|
|
{ module => 'circulation', code => 'code1' } );
|
|
is( scalar( @$letter_templates ),
|
|
2, '2 default templates should exist for circulation code1 (even if branchcode is not given)' );
|
|
|
|
t::lib::Mocks::mock_preference('TranslateNotices', 0);
|
|
$letter_templates = C4::Letters::GetLetterTemplates(
|
|
{ module => 'circulation', code => 'code1' } );
|
|
is( scalar( @$letter_templates ),
|
|
1, 'There should exist only 1 template circulation code1 for default language' );
|