[rt-users] Problems in templates in Rt 4.2.1

[Guadagnino Cristiano]

-----Messaggio originale-----
Da: Cristiano Guadagnino <guadagnino.cristiano at creval.it>
Inviato: Fri Jan 03 2014 10:22:21 GMT+0100 (CET)
A: rt-users at lists.bestpractical.com
Oggetto: Re: [rt-users] Problems in templates in Rt 4.2.1

> -----Messaggio originale-----
> Da: Cristiano Guadagnino <guadagnino.cristiano at creval.it>
>
>> -----Messaggio originale-----
>> Da: Cristiano Guadagnino <guadagnino.cristiano at creval.it>
>>
>>> Alex Vandiver ha scritto:
>>>> On Thu, 2014-01-02 at 16:18 +0000, Guadagnino Cristiano wrote:
>>>>> This message is related to my previous thread titled "HTML templates
>>>>> in RT 4.2.1".
>>>> This is a known bug in 4.2.1, relating to how RT attempts to generate a
>>>> plain-text version of HTML templates.  A fix will be in 4.2.2 which
>>>> will
>>>> ensure that mail is always sent out, even if it lacks a text/plain part
>>>> due to failure of the HTML->text conversion.
>>>>   - Alex
>>> Thank you very much Alex!
>>> Do you know if a patch is available?
>>>
>>
>> Never mind Alex, I just read about it in another reply.
>>
>> Bye
>> Cris
>
> Hi Alex,
> while trying to apply the patch in
>
> https://github.com/bestpractical/rt/commit/8807f0d0.patch
>
> I get this:
>
> can't find file to patch at input line 26
>
> Looking at the patch file, I see it is trying to patch two files:
>
> /opt/rt4/lib/RT/Template.pm
> /opt/rt4/t/mail/html-outgoing.t
>
> The second file is non-existent here. Actually, I don't have any
> /opt/rt4/t directory nor any html-outgoing.t file under /opt/rt4.
>
> Can you help me?
>
> T.I.A.
> Bye
> Cris

Alex,
I figured that I needed to apply the patch to the source tree, so I did 
it successfully. Then I did a "make upgrade". All of this was done on a 
test server.

After that, I compared the new tree in /opt/rt4 with a backup of the old 
tree. Diff says only /opt/rt4/lib/RT/Template.pm is changed.

Is it safe to just copy the new Template.pm into our production server 
and clear the mason cache? Will it suffice to fix the bug?

Thank you in advance
Cris