-------- Forwarded Message --------
From: Scott McNally <[EMAIL PROTECTED]>
To: 'Kristis Makris' <[EMAIL PROTECTED]>
Subject: RE: Bugzilla 3.0 "Delete Tag" - found the issue not sure how to
solve
Date: Tue, 11 Sep 2007 13:45:19 -0500

The problem lies in the fact that you are passing the  ProductID and not the 
VersionID.  
Hmm.. I checked the api as well and it may very well be that they have a bug 
with that.
IN the internal bugzilla tables there is

Versions:
Value,product_id,id

The issue is the constructor looks at id  when it does the select rather than 
product id.
So when pulling the information back it is highly likely to grab an incorrect 
version, or a non-existent one.


-----Original Message-----
From: Kristis Makris [mailto:[EMAIL PROTECTED] 
Sent: Tuesday, September 11, 2007 1:42 PM
To: Scott McNally
Cc: [email protected]
Subject: Re: Bugzilla 3.0 "Delete Tag" - found the issue not sure how to solve

Scott thanks for looking into this.

The source seems ok as written so far:

my $product = new Bugzilla::Product( { name => $product_name } ); my $version = 
new Bugzilla::Version( $product->id, $tag_name ); $version->remove_from_db;

I looked up the Version and Product interfaces at:

http://www.bugzilla.org/docs/tip/html/api/

And I'm not sure how else it could be written.

Dmitri ??

On Mon, 2007-09-10 at 18:08 -0500, Scott McNally wrote:
> Doing some changing of our svn /scmbug setup. The issue with delete 
> tag is that it passes the Product ID instead of the version ID into
> the constructor of Bugzilla::Version.   Some form of sql call needs to
> be made to grab the correct version id.
> 
>  
> 
> “SELECT id FROM versions WHERE product_id =  ? AND value = ? “
> 
>  
> 
>  
> 
> -Scott McNally
> 
> Development Manager
> 
> P2P Engineering
> 
> 850-607-6161
> 
>  
> 
> 


Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
scmbug-users mailing list
[email protected]
http://lists.mkgnu.net/cgi-bin/mailman/listinfo/scmbug-users

Reply via email to