Bug #52034 for RT-Extension-RT2toRT3: Migrating links pointing to URLs

Tue Nov 24 12:22:01 2009 dom [...] cpan.org - Ticket created

Subject:

Migrating links pointing to URLs

In our RT2 install we have some tickets with links (refers to, usually) pointing to not other ticket IDs, but external URLS. RT2 linkifies them for us which suggests that it's a deliberate feature. RT3 also supports links to URLs. However dumpfile-to-rt3.0 barfs on them: Couldn't create link from fsck.com-rt:/ticket/1402681 to https://[elided]/rt/Ticket/Display.html?id=57490 because Internal Error: Couldn't execute the query 'INSERT INTO Links (Type, LastUpdated, Target, Base, LocalTarget, Creator, LastUpdatedBy, Created, LocalBase) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?)'ERROR: invalid input syntax for integer: "https://[elided]/rt/Ticket/Display.html?id=57490" I will probably just manually restore the links afterwards, since the number of tickets affected is relatively low, but I thought it would be worth filing as a bug for the record.

Tue Nov 24 12:40:31 2009 kevin [...] jibsheet.com - Correspondence added

Subject:	Re: [rt.cpan.org #52034] Migrating links pointing to URLs
Date:	Tue, 24 Nov 2009 12:40:10 -0500
To:	Dominic Hargreaves via RT <bug-RT-Extension-RT2toRT3 [...] rt.cpan.org>
From:	Kevin Falcone <kevin [...] jibsheet.com>

On Tue, Nov 24, 2009 at 12:22:02PM -0500, Dominic Hargreaves via RT wrote: Show quoted text

> In our RT2 install we have some tickets with links (refers to, usually) > pointing to not other ticket IDs, but external URLS. RT2 linkifies them > for us which suggests that it's a deliberate feature. > > RT3 also supports links to URLs. > > However dumpfile-to-rt3.0 barfs on them: > > Couldn't create link from fsck.com-rt:/ticket/1402681 to > https://[elided]/rt/Ticket/Display.html?id=57490 because Internal Error: > Couldn't execute the query 'INSERT INTO Links (Type, LastUpdated, > Target, Base, LocalTarget, Creator, LastUpdatedBy, Created, LocalBase) > VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?)'ERROR: invalid input syntax for > integer: "https://[elided]/rt/Ticket/Display.html?id=57490" > > I will probably just manually restore the links afterwards, since the > number of tickets affected is relatively low, but I thought it would be > worth filing as a bug for the record.

Judging from this error, I'd guess that the converter's attempt to trim a URL (which is what RT2 used) back into the ticket ID for use in the LocalBase/LocalTarget failed. That probably need to be coerced to 0 if LocalTarget/LocalBase matches \D at all. I know that code has been rewritten a few times -kevin

Tue Nov 24 12:40:32 2009 The RT System itself - Status changed from 'new' to 'open'

Thu Dec 31 14:10:09 2009 dom [...] earth.li - Correspondence added

Subject:	Re: [rt.cpan.org #52034] Migrating links pointing to URLs
Date:	Thu, 31 Dec 2009 19:09:49 +0000
To:	Kevin Falcone via RT <bug-RT-Extension-RT2toRT3 [...] rt.cpan.org>
From:	Dominic Hargreaves <dom [...] earth.li>

On Tue, Nov 24, 2009 at 12:40:32PM -0500, Kevin Falcone via RT wrote: Show quoted text

> Judging from this error, I'd guess that the converter's attempt to > trim a URL (which is what RT2 used) back into the ticket ID for use in > the LocalBase/LocalTarget failed. > > That probably need to be coerced to 0 if LocalTarget/LocalBase matches > \D at all. > > I know that code has been rewritten a few times

Thanks for the input. Just to let you know that we have decided that this isn't important enough to fix in our installation (the number of things it hits is small and the data is still available in transaction text), so feel free to keep this bug open for reference but I won't be persuing a fix. Dominic. -- Dominic Hargreaves | http://www.larted.org.uk/~dom/ PGP key 5178E2A5 from the.earth.li (keyserver,web,email)