<p>Hi.</p>
<p>Sure link maps should be expanded over all records. If you're going to work on this then fork rt on github and publish changes earlier, so we can review. This is especially important if you want to push it into core.</p>
<p>Regards, Ruslan. From phone.</p>
<div class="gmail_quote">2010 10 27 04:55 пользователь "Todd Chapman" <<a href="mailto:todd@chaka.net">todd@chaka.net</a>> написал:<br type="attribution">> Hi Ruslan,<br>> <br>> w00t! First RT patch in some years. :)<br>
> <br>> What is keeping the patch from RT 3.8? For the new AT release I have to<br>> override RT::Ticket::LINKTYPEMAP(). :(<br>> <br>> I'll try to get some time to add tests. That shredding code is not the<br>
> easiest to understand but I am getting there!<br>> <br>> Should we relocate the link definitions to RT::Link and then add a method to<br>> RT::Record that returns valid link types for the object?<br>> <br>
> On Tue, Oct 26, 2010 at 8:37 PM, Ruslan Zakirov <<a href="mailto:ruz@bestpractical.com">ruz@bestpractical.com</a>>wrote:<br>> <br>>> 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>>><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<br>>> 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<br>
>> sure<br>>> > if I used the correct options for your preferred workflow.<br>>> > -Todd<br>>> ><br>>> > _______________________________________________<br>>> > List info:<br>
>> <a href="http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-devel">http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-devel</a><br>>> ><br>>> ><br>>><br>>><br>>><br>
>> --<br>>> Best regards, Ruslan.<br>>><br></div>