[rt-users] Callbacks not working

[Ruslan Zakirov]

On Wed, Jun 13, 2012 at 5:41 PM, Mark Collins <mark at westgateit.com> wrote:
> Yes - even after the mason cache has been cleared the callback isn't running.

Check permissions and logs.

> Mark Collins
> Westgate IT
> 0845 017 8852
>
>
> -----Original Message-----
> From: ruslan.zakirov at gmail.com [mailto:ruslan.zakirov at gmail.com] On Behalf Of Ruslan Zakirov
> Sent: 12 June 2012 18:54
> To: Mark Collins
> Cc: rt-users at lists.bestpractical.com
> Subject: Re: [rt-users] Callbacks not working
>
> Have you cleaned the mason cache?
>
> On Tue, Jun 12, 2012 at 6:45 PM, Mark Collins <mark at westgateit.com> wrote:
>> Ruslan,
>>
>> Thanks for the quick response - the file is in the correct path (that was a cut & paste error) with the correct name:
>>
>> loki:/opt/rt/local/html/Callbacks/Westgate/Tickets/Elements/ShowHistor
>> y # ls SkipTransaction
>>
>> When I look at the ticket history, all the content is still there, even if I change the <%init> block to skip everything.
>>
>> RT version is 4.0.5
>>
>> Mark Collins
>> Westgate IT
>> 0845 017 8852
>>
>>
>> -----Original Message-----
>> From: ruslan.zakirov at gmail.com [mailto:ruslan.zakirov at gmail.com] On
>> Behalf Of Ruslan Zakirov
>> Sent: 12 June 2012 15:40
>> To: Mark Collins
>> Cc: rt-users at lists.bestpractical.com
>> Subject: Re: [rt-users] Callbacks not working
>>
>> On Tue, Jun 12, 2012 at 6:29 PM, Mark Collins <mark at westgateit.com> wrote:
>>> Hi all,
>>>
>>> I'm trying to get a callback working that hides the system updates in a tickets history, but nothing is happening.
>>>
>>> I've placed the callback in /opt/rt/local/html/Callbacks/Westgate/Tickets/Elements (it's call ShowHistory), and the callback code just isn't running. The callback itself is as follows:
>>
>> It should be .../Westgate/Tickets/Elements/ShowHistory directory and file should be named after name of the callback you want to use. In this case it's SkipTransaction.
>>
>> Below see comments about your code.
>>
>>> ========================
>>> <%init>
>>> my $myskip=1;
>>>
>>> if ( $Transaction->Type =~ /^(Correspond|Create)$/ ) {
>>>        $myskip = 0;
>>> }
>>> $$skip=$myskip;
>>>
>>> </%init>
>>>
>>> <%args>
>>> $Transaction => undef
>>> $skip => undef
>>> </%args>
>>
>>
>> Logic of the code is slightly wrong. It will work, but it doesn't cope with multiple callbacks. You shouldn't set $$skip to 1. Use "return if $$skip;" as first line in the <%INIT> block. This way you can avoid conflicts with extensions that use this callback.
>>
>> Also, condition looks too agressive, you skip status changes, comments, custom field changes and many other things, but it's really up to you what to do here.
>>
>> Check that $m->request_path matches 'Display.html'. This way you can have short history on ticket's primary page and full when History tab is clicked.
>>
>>> ========================
>>>
>>> Any ideas on why this isn't working?
>>>
>>> Mark Collins
>>> Westgate IT
>>> Main
>>> 0845 017 8852
>>> Fax
>>> 0845 017 8854
>>> Web
>>> http://www.westgateit.com
>>>
>>>
>>>
>>>
>>> Fax to email
>>> 0845 017 8853
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>> Westgate IT Ltd. 12 Riverside Court, Lower Bristol Road, Bath, BA2
>>> 3DZ Company Registered in England and Wales, 3379298. Vat
>>> Registration
>>> Number: 692 0383 29
>>>
>>> This message (and any associated files) is intended only for the use of the individual or entity to which it is addressed and may contain information that is confidential, subject to copyright or constitutes a trade secret. If you are not the intended recipient you are hereby notified that any dissemination, copying or distribution of this message, or files associated with this message, is strictly prohibited. If you have received this message in error, please notify us immediately by replying to the message and deleting it from your computer. Messages sent to and from us may be monitored.
>>>
>>> Internet communications cannot be guaranteed to be secure or error-free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or contain viruses. Therefore, we do not accept responsibility for any errors or omissions that are present in this message, or any attachment, that have arisen as a result of e-mail transmission. If verification is required, please request a hard-copy version. Any views or opinions presented are solely those of the author and do not necessarily represent those of the company.
>>>
>>> Scanned by MailDefender - managed email security from intY -
>>> www.maildefender.net
>>
>>
>>
>> --
>> Best regards, Ruslan.
>>
>> Scanned by MailDefender - managed email security from intY -
>> www.maildefender.net
>
>
>
> --
> Best regards, Ruslan.
>
> Scanned by MailDefender - managed email security from intY - www.maildefender.net



-- 
Best regards, Ruslan.