ProcessTicketLinks

Hi,

Apologies for the long signature , its not in my control.
I have created an unmerge function that worked well in 3.8 , however now I have moved to 4.0.10 it stopped working as expected.
The problem I have is ProcessTicketLinks not being called from my local/lib/RT/Interface/Web_Local.pm and instead its being called from the main lib/RT/Interface/Web.pm

I have other functions in Web_Local.pm (example ShowRequestedPage) which are being called correctly .
Have I missed anything special about ProcessTicketLinks ?? Any pointers will be greatly appreciated

Regards;
Roy

[cid:image939c62.PNG@9288549f.47a7187a]

This email (and any attachments) contains information, which may be privileged and/or confidential. It’s meant only for the individual(s) or entity named above. If you’re not the intended recipient, please note that disclosing, copying, distributing or using this information is prohibited. If you’ve received this email in error, please let me know immediately by emailing me at the email address above and then delete all traces of this email from your system. We monitor our email system, and may record your emails.

Computer viruses can be transmitted by email. You should check this email and any attachments for the presence of viruses. We accept no liability for any damage caused by any virus transmitted by this email or any attachments.

Daisy Communications Ltd
Registered office: Daisy House, Lindred Road Business Park, Nelson, Lancashire, BB9 5SR
Registered in England and Wales with company number: 4145329

Apologies for the long signature , its not in my control.
I have created an unmerge function that worked well in 3.8 , however now I have moved to 4.0.10 it stopped working as expected.
The problem I have is ProcessTicketLinks not being called from my local/lib/RT/Interface/Web_Local.pm and instead its being called from the main lib/RT/Interface/Web.pm

I have other functions in Web_Local.pm (example ShowRequestedPage) which are being called correctly .
Have I missed anything special about ProcessTicketLinks ?? Any pointers will be greatly appreciated

You didn’t include your code, so this is just a guess, but since
ShowRequestedPage is in the RT::Interface::Web package while
ProcessTicketLinks is in the HTML::Mason::Commands (to make it
available easily to Mason components) I’m guessing that you don’t have
correct package lines.

-kevin

Thanks Kevin, that was the issue … fixed now …
By the way on a related note, have you folks put any thought into a clean unmerge function ?? I know its been debated over the past few years and wondering why you have n’t done it … is there a reason that I should be aware off ?? my unmerge function:
Sets the effective id back to == id
Resets the status back to open
Removes any requestors from $self-> that exist in the unmerged ticket unless these existed in $self before the MergeInto transaction time …

Roy

Visit our website today www.daisygroupplc.com

Registered Office: Daisy House, Lindred Road Business Park, Nelson, Lancashire BB9 5SR
Company Registration Number: 4145329 | VAT Number: 722471355
Daisy Communications Limited is a company registered in England and Wales.
DISCLAIMER

This email (including any attachments) is strictly confidential and may also be legally privileged. If the recipient has received this email in error please notify the sender and do not read, print, re-transmit, store or act in reliance on the email or its attachments and immediately delete this email and its attachments from the recipient’s system. Daisy Communications Limited cannot accept liability for any breaches of confidence arising through use of email. Employees of Daisy Communications Limited are expressly required not to make any defamatory statements and not to infringe or authorise any infringement of copyright or any other legal right by email communications. Any such communication is contrary to the company’s policy and outside the scope of the employment of the individual concerned. Daisy Communications Limited will not accept any liability in respect of such a communication, and the employee responsible will be personally liable for any damages or other liability arising.

If you are the intended recipient of this email please ensure that neither the email nor any attachments are copied to third parties outside your organisation or saved without the written permission of the sender. In the event of any unauthorised copying or forwarding, the recipient will be required to indemnify Daisy Communications Limited against any claim for loss or damage caused by any viruses or otherwise.

WARNING: Computer viruses can be transmitted by email. The recipient should check this email and any attachments for the presence of viruses. Daisy Communications Limited accepts no liability for any damage caused by any virus transmitted by this email or any attachments.
NOTICE TO CUSTOMERS
If you have ordered a telephone number from Daisy Communications Limited (non-geographic or new line installation) please do NOT arrange for any form of advertising until the number is live and tested.-----Original Message-----
From: rt-devel-bounces@lists.bestpractical.com [mailto:rt-devel-bounces@lists.bestpractical.com] On Behalf Of Kevin Falcone
Sent: 06 June 2013 18:49
To: rt-devel@lists.bestpractical.com
Subject: Re: [rt-devel] ProcessTicketLinks

On Wed, Jun 05, 2013 at 04:27:10PM +0000, Raed El-Hames wrote:

Apologies for the long signature , its not in my control.
I have created an unmerge function that worked well in 3.8 , however now I have moved to 4.0.10 it stopped working as expected.
The problem I have is ProcessTicketLinks not being called from my
local/lib/RT/Interface/Web_Local.pm and instead its being called from
the main lib/RT/Interface/Web.pm

I have other functions in Web_Local.pm (example ShowRequestedPage) which are being called correctly .
Have I missed anything special about ProcessTicketLinks ?? Any
pointers will be greatly appreciated

You didn’t include your code, so this is just a guess, but since ShowRequestedPage is in the RT::Interface::Web package while ProcessTicketLinks is in the HTML::Mason::Commands (to make it available easily to Mason components) I’m guessing that you don’t have correct package lines.

-kevin

Thanks Kevin, that was the issue … fixed now …
By the way on a related note, have you folks put any thought into a
clean unmerge function ?? I know its been debated over the past few
years and wondering why you have n’t done it … is there a reason that
I should be aware off ?? my unmerge function:
Sets the effective id back to == id
Resets the status back to open
Removes any requestors from $self-> that exist in the unmerged ticket unless these existed in $self before the MergeInto transaction time …

That’s only some of the things done in a merge. Check the _MergeInto
method on RT::Ticket for more things that need to be unwound.

It’s actually annoyingly difficult to untangle correctly, especially
if you merge ticket 1 into 2 and then 2 into 3 and then want to
unmerge ticket 1.

There is a branch in review for 4.2 to make it more obvious when
you’re about to merge the wrong tickets.

-kevin

Thanks Kevin, that was the issue … fixed now …
By the way on a related note, have you folks put any thought into a
clean unmerge function ?? I know its been debated over the past few
years and wondering why you have n’t done it … is there a reason that
I should be aware off ?? my unmerge function:
Sets the effective id back to == id
Resets the status back to open
Removes any requestors from $self-> that exist in the unmerged ticket unless these existed in $self before the MergeInto transaction time …

That’s only some of the things done in a merge. Check the _MergeInto
method on RT::Ticket for more things that need to be unwound.

It’s actually annoyingly difficult to untangle correctly, especially
if you merge ticket 1 into 2 and then 2 into 3 and then want to
unmerge ticket 1.

There is a branch in review for 4.2 to make it more obvious when
you’re about to merge the wrong tickets.

-kevin

I always thought it would be nice if merging produced an SQL dump like shredding does.

-Brad