Apache Allura / Chat is hosted on FreeNode IRC channel #allura
ASFBot: tell brondsem report looks good :)
I'll pass that along, Humbedooh!
brondsem: 5 hours and 8 minutes ago, Humbedooh said; report looks good :)
it works fine, but...
yeah, those don't sound the best to me either. Let me take a cursory look at the code
not much code :)
yeah, that was a nice feature of this ticket :)
better impl might be a lot more though, we'll see
yeah, using a custom field is also going to be really slow for lookups on a large db
true, i hadn't even thought of that
i had been thinking more along the lines of having a new artifact for MovedTicket which would have a URL to the new location
then those could be chained and the client gets multiple redirects
yeah, that way you could just follow the chain
yeah, what you said :P
it'd also have the flexibility to redirect to an external URL. which I suspect we may want to do when we start moving our own tickets from SF allura to ASF allura
at least backend support for it
so if a ticket is missing, the code does a lookup on the MovedTicket collection?
we don't allow mount point changes, so storing a URL path should be fine. The current impl is a little more flexible in that regard, storing a app_config_id, but not too big a deal I guess
well we could store whatever we want right, any reason to not store the app config id?
I guess only if we wanted to redirect to an arbitrary url instead
that's not necessarily a requirement at this point though. but seems like it might be nice
yeah, i'll tell them to just store the url that the ticket is moving to
another thing - the ticket API should be updated too. Probably just return a 301 instead of 404
ah, right, good catch
MovedArtifact instead? with a moved_to_url field
hm, i guess you'd end up with MovedTicket extending that, since you'd need to store ticket_nu
yeah, a generic class might be useful elsewhere. We'd still need a way to do the lookup by a specific key (ticket_num) so that might necessitate ….. yes
dbrondsema:  bump wiki importer versions