On Wed, Jun 13, 2012 at 5:41 PM, Mark Collins <m...@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.zaki...@gmail.com [mailto:ruslan.zaki...@gmail.com] On Behalf Of 
> Ruslan Zakirov
> Sent: 12 June 2012 18:54
> To: Mark Collins
> Cc: rt-users@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 <m...@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.zaki...@gmail.com [mailto:ruslan.zaki...@gmail.com] On
>> Behalf Of Ruslan Zakirov
>> Sent: 12 June 2012 15:40
>> To: Mark Collins
>> Cc: rt-users@lists.bestpractical.com
>> Subject: Re: [rt-users] Callbacks not working
>>
>> On Tue, Jun 12, 2012 at 6:29 PM, Mark Collins <m...@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.

Reply via email to