[Rt-devel] BUG: Search Builder link searching

[Todd Chapman]

On Tue, Dec 06, 2005 at 03:20:40PM -0500, Jesse Vincent wrote:
> 
> 
> 
> On Tue, Dec 06, 2005 at 03:53:54PM -0500, Todd Chapman wrote:
> > On Tue, Dec 06, 2005 at 02:58:58PM -0500, Jesse Vincent wrote:
> > > 
> > > 
> > > 
> > > On Tue, Dec 06, 2005 at 03:22:21PM -0500, Todd Chapman wrote:
> > > > > That looks like a bug.
> > > > 
> > > > I really don't understand the need for LocalTarget/LocalBase. Is
> > > > it just an optimization? Target/Base should be all we need
> > > > for searching...
> > > 
> > > Ok. Show me how to join the tickets table to the links table.
> > > 
> > 
> > I think I see the problem. Does that mean you can't join
> > rtfm articles to the links table?
> 
> Nope. it means we need to store the Article's URI in the Articles table
> (which is what we do). It was something I wasn't comfortable doing with
> the "primary" object in RT, but for future changes, benchmarking can
> sway me.  (But joining on ints will alaways be faster than joining on a
> long string)

So at this time you want a patch for RT::Link::Create() that
makes sure LocalBase and LocalTarget are only set for Tickets
and NULL otherwise?

What about the old records? Should we have an upgrade script to
fixup the Links?

Damn, now I have to go add a URI field to Assets...

-Todd