RT-3.8.9 upgrade problem with Rules

I recently tried to update our RT server to 3.8.9, but had a problem
that I suspect may have something to do with this item from the changelog:

  • Include Rules with Scrips when previewing recipients

When using the web interface and clicking on the reply or comment links,
I just get a single error message:

RT::Transaction::Rules Unimplemented in HTML::Mason::Commands.
(/usr/share/rt3/html/Ticket/Elements/PreviewScrips line 88)

Note, line 88 of Ticket/Elements/PreviewScrips is the new section of
code added in 3.8.9 that starts with:

% if ( $Object and $Object->Rules ) {

I followed all the upgrade proceedures, checking rt-test-dependencies
and running vulnerable-passwords, shrink_transactions_table.pl and
rt-setup-database to run the update from the previous 3.8.8 version.

Does anyone know what might be wrong?

Thanks,
~Jason

smime.p7s (3.97 KB)

I recently tried to update our RT server to 3.8.9, but had a problem
that I suspect may have something to do with this item from the
changelog:

  • Include Rules with Scrips when previewing recipients

When using the web interface and clicking on the reply or comment
links, I just get a single error message:

RT::Transaction::Rules Unimplemented in HTML::Mason::Commands.
(/usr/share/rt3/html/Ticket/Elements/PreviewScrips line 88)

Do you get a full stack trace?

Did you clear out the mason cache?

What do you have in the way of local customizations?

Best,
Jesse

I recently tried to update our RT server to 3.8.9, but had a problem
that I suspect may have something to do with this item from the
changelog:

  • Include Rules with Scrips when previewing recipients

When using the web interface and clicking on the reply or comment
links, I just get a single error message:

RT::Transaction::Rules Unimplemented in HTML::Mason::Commands.
(/usr/share/rt3/html/Ticket/Elements/PreviewScrips line 88)

Do you get a full stack trace?

Nope, only that single line. I also checked the system and apache logs
and even tried turning on debugging, but didn’t see any additional info
related to this error.

Did you clear out the mason cache?

I thought restarting apache did this, but to be sure, I just tried again
and immediately after the restart I did verify that it was empty:

ls -l /var/cache/rt3/mason_data/obj/

total 0

What do you have in the way of local customizations?

It is mostly a stock RT installation, with several plugins installed:

Set(@Plugins, qw(RTx::BecomeUser RT::Extension::MergeUsers
RT::Extension::PriorityAsString RT::Extension::MandatorySubject
RT::Extension::CommandByMail RTx::EmailCompletion RTx::RightsMatrix
RTx::Calendar RT::Extension::ActivityReports RT::Extension::Timeline
RTx::From RTx::S3Invoker RT::Extension::MobileUI));

I just tried commenting out the above line in my RT_SiteConfig.pm and
got the same error. Other than that, we just have a few small patches
to do some minor things like adjust some of the form box sizes and such.

Thanks,
~Jason

smime.p7s (3.97 KB)

I recently tried to update our RT server to 3.8.9, but had a problem
that I suspect may have something to do with this item from the
changelog:

  • Include Rules with Scrips when previewing recipients

When using the web interface and clicking on the reply or comment
links, I just get a single error message:

RT::Transaction::Rules Unimplemented in HTML::Mason::Commands.
(/usr/share/rt3/html/Ticket/Elements/PreviewScrips line 88)

What do you have in the way of local customizations?

It is mostly a stock RT installation, with several plugins installed:

Set(@Plugins, qw(RTx::BecomeUser RT::Extension::MergeUsers
RT::Extension::PriorityAsString RT::Extension::MandatorySubject
RT::Extension::CommandByMail RTx::EmailCompletion RTx::RightsMatrix
RTx::Calendar RT::Extension::ActivityReports RT::Extension::Timeline
RTx::From RTx::S3Invoker RT::Extension::MobileUI));

I just tried commenting out the above line in my RT_SiteConfig.pm
and got the same error. Other than that, we just have a few small
patches to do some minor things like adjust some of the form box
sizes and such.

Can you try 3.8.9 without your small patches?

Can you try 3.8.9 without your small patches?

Okay, my fault. It wasn’t any of the patches, but one local
modification that I forgot to convert to a patch. I am using patches
when building my rpm so I can more easily keep our customizations
updated with changes to the upstream sources. It is a lot easier for me
to do it this this way, than to use the rt local directory, as this most
recent problem painfully points out.

I found a modified Transaction_Overlay.pm file (based on 3.8.8) in rt’s
local directory that I forgot to convert to a patch in my rpm, which was
causing my problem. The modification was reported a while ago:

http://issues.bestpractical.com/Ticket/Display.html?id=14624

which I haven’t received any feedback on yet by the way. Any comments
about this ticket? The patch makes our html templates work a lot better
when we get unwrapped user submitted content.

Thanks,
~Jason

smime.p7s (3.97 KB)

I found a modified Transaction_Overlay.pm file (based on 3.8.8) in
rt’s local directory that I forgot to convert to a patch in my rpm,
which was causing my problem. The modification was reported a while
ago:

If you’re going to put a file in local/lib, please use
Transaction_Local.pm to avoid future problems.
Using Foo_Overlay.pm is guaranteed to conflict with our changes.

-kevin

I found a modified Transaction_Overlay.pm file (based on 3.8.8) in
rt’s local directory that I forgot to convert to a patch in my rpm,
which was causing my problem. The modification was reported a while
ago:

If you’re going to put a file in local/lib, please use
Transaction_Local.pm to avoid future problems.
Using Foo_Overlay.pm is guaranteed to conflict with our changes.

I used a local copy of Transaction_Overlay.pm exactly because I was
testing out a few line patch to a function in the original file, as
stated in my bug report:

http://issues.bestpractical.com/Ticket/Display.html?id=14624

My only mistake was forgetting about this local copy after I was done
testing the patch and making it more permanent.

~Jason

smime.p7s (3.97 KB)