Hi Ruslan,<div><br></div><div>w00t! First RT patch in some years. :)</div><div><br></div><div>What is keeping the patch from RT 3.8? For the new AT release I have to override RT::Ticket::LINKTYPEMAP(). :(</div><div><br></div>
<meta charset="utf-8"><div>I'll try to get some time to add tests. That shredding code is not the easiest to understand but I am getting there!</div><div><br></div><div>Should we relocate the link definitions to RT::Link and then add a method to RT::Record that returns valid link types for the object? <br>
<br><div class="gmail_quote">On Tue, Oct 26, 2010 at 8:37 PM, Ruslan Zakirov <span dir="ltr"><<a href="mailto:ruz@bestpractical.com">ruz@bestpractical.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Hi Todd,<br>
<br>
Looks good to me. It's not going into 3.8, but into 3.9-trunk branch.<br>
Any chance you can write some tests around shredding links?<br>
<div><div></div><div class="h5"><br>
On Mon, Oct 25, 2010 at 7:09 PM, Todd Chapman <<a href="mailto:todd@chaka.net">todd@chaka.net</a>> wrote:<br>
> The attached patch enables shredding of links that point to local objects<br>
> that define different link types than RT::Ticket. (Asset Tracker defines<br>
> it's own link types.)<br>
> BTW, shouldn't link definitions be stored somewhere other than RT::Ticket?<br>
> It seems the generalization of RT links has not been fully realized.<br>
> Jesse, please let me know if the attached patch isn't in the preferred<br>
> format. This is my first time generating a patch using git and I'm not sure<br>
> if I used the correct options for your preferred workflow.<br>
> -Todd<br>
><br>
</div></div>> _______________________________________________<br>
> List info: <a href="http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-devel" target="_blank">http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-devel</a><br>
><br>
><br>
<font color="#888888"><br>
<br>
<br>
--<br>
Best regards, Ruslan.<br>
</font></blockquote></div><br></div>