[Rt-commit] rt branch, 4.2/do-not-encode-mime-headers-in-forward-method, repushed

? sunnavy sunnavy at bestpractical.com
Tue Mar 10 14:33:19 EDT 2015


The branch 4.2/do-not-encode-mime-headers-in-forward-method was deleted and repushed:
       was bd70fea27144dd3230446d43eb566b525263e573
       now 9f9f5212d7fa1d8e4f54dc66fccfbf118775c313

1:  bd70fea ! 1:  9f9f521 it's not RT::Ticket::Forward's job to encode headers to mime
    @@ -17,17 +17,20 @@
         
         Fixes: I#29714
     
    -diff --git a/etc/upgrade/4.2.12/content b/etc/upgrade/4.2.12/content
    -new file mode 100644
    ---- /dev/null
    -+++ b/etc/upgrade/4.2.12/content
    +diff --git a/etc/upgrade/4.2.11/content b/etc/upgrade/4.2.11/content
    +--- a/etc/upgrade/4.2.11/content
    ++++ b/etc/upgrade/4.2.11/content
     @@
    -+use strict;
    -+use warnings;
    -+
    + use strict;
    + use warnings;
     +use Encode;
    -+
    -+our @Final = (
    + 
    + our @Initial = (
    +     sub {
    +@@
    +                AND Content IS NULL;
    + EOSQL
    +     },
     +    sub {
     +        my $txns = RT::Transactions->new(RT->SystemUser);
     +        $txns->Limit( FIELD => 'Type', VALUE => 'Forward Transaction' );
    @@ -46,7 +49,7 @@
     +            }
     +        }
     +    },
    -+);
    + );
     
     diff --git a/lib/RT/Ticket.pm b/lib/RT/Ticket.pm
     --- a/lib/RT/Ticket.pm
    @@ -67,3 +70,40 @@
                      Transaction => $args{Transaction},
                      Ticket      => $self,
                  )
    +
    +diff --git a/t/web/ticket_forward.t b/t/web/ticket_forward.t
    +--- a/t/web/ticket_forward.t
    ++++ b/t/web/ticket_forward.t
    +@@
    + print $att_fh "this is an attachment";
    + close $att_fh;
    + my $att_name = ( File::Spec->splitpath($att_file) )[-1];
    ++use Encode;
    + 
    + my ( $baseurl, $m ) = RT::Test->started_ok;
    + ok $m->login, 'logged in as root';
    +@@
    +     like($mail, qr/Subject: \[example.com #\d+\] OVERRIDING SUBJECT/);
    + }
    + 
    ++diag "Forward Transaction with non-ascii subject" if $ENV{TEST_VERBOSE};
    ++{
    ++    $m->follow_link_ok( { text => 'Forward', n => 2 }, 'follow 2nd Forward' );
    ++    my $subject = decode_utf8('test non-ascii äöü');
    ++    $m->submit_form(
    ++        form_name => 'ForwardMessage',
    ++        fields    => {
    ++            Subject => $subject,
    ++            To  => 'rt-to at example.com',
    ++        },
    ++        button => 'ForwardAndReturn'
    ++    );
    ++    my ($mail) = RT::Test->fetch_caught_mails;
    ++    if ( $mail =~ /Subject: (.+)/ ) {
    ++        like( decode_utf8(RT::I18N::DecodeMIMEWordsToUTF8( $1, 'Subject' )), qr/$subject/, 'non-ascii subject' );
    ++    }
    ++    $m->content_contains( $subject, 'non-ascii subject got displayed correctly' );
    ++}
    ++
    + undef $m;
    + done_testing;



More information about the rt-commit mailing list