[rt-users] Ticket taken twice

[Jesse Vincent]

On Thu, Feb 26, 2009 at 02:48:19PM -0500, Jesse Vincent wrote:
> 
> 
> 
> On Thu, Feb 26, 2009 at 11:40:02AM -0800, Joachim Thuau wrote:
> > Sadly, we already tried that. We can change the owner every which way,
> > except for "tom" (who was the guy that worked on it). 
> > 
> 
> Can you try this patch? It's conceptually right but off the cuff. It's against 3.8
> but applies to recentish 3.6. It may need a tiny bit of massage to apply to 3.6.1
> 

Don't apply this patch. It's making the RT test suite explode. I have no
idea _what_ it would do to your DB.

> You'll need to stop and start apache after making the change. 
> 
> === lib/RT/Ticket_Overlay.pm
> ==================================================================
> --- lib/RT/Ticket_Overlay.pm	(revision 58932)
> +++ lib/RT/Ticket_Overlay.pm	(local)
> @@ -2782,7 +2782,12 @@
>      # Delete the owner in the owner group, then add a new one
>      # TODO: is this safe? it's not how we really want the API to work
>      # for most things, but it's fast.
> -    my ( $del_id, $del_msg ) = $self->OwnerGroup->MembersObj->First->Delete();
> +    my ( $del_id, $del_msg );
> +    for my $owner (@{$self->OwnerGroup->MembersObj->ItemsArrayRef}) {
> +        ($del_id, $del_msg) = $owner->Delete();
> +        last unless ($del_id);
> +    }
> +
>      unless ($del_id) {
>          $RT::Handle->Rollback();
>          return ( 0, $self->loc("Could not change owner: [_1]", $del_msg) );
> 
> 
> > Thanks,
> > Jok 
> > 
> > > -----Original Message-----
> > > From: Jesse Vincent [mailto:jesse at bestpractical.com] 
> > > Sent: Thursday, February 26, 2009 11:26 AM
> > > To: Joachim Thuau
> > > Cc: rt-users at lists.bestpractical.com
> > > Subject: Re: [rt-users] Ticket taken twice
> > > 
> > > 
> > > 
> > > 
> > > On Thu 26.Feb'09 at 11:04:08 -0800, Joachim Thuau wrote:
> > > > Will this cause issues with the upgrade process, if I 
> > > update to 3.8.2 now?
> > > > (ie: will it just "recover" or will the DB choke on it)
> > > > 
> > > > I guess I could just shred that ticket... But history is 
> > > important...
> > > > 
> > > 
> > > It's been a while, but I _believe_ that if someone "steals" 
> > > the ticket from here, RT will self-correct.
> > > 
> > 
> 
> -- 
> 

--