141609 - "Version unspecified" error when changing product with product groups enabled

Status: RESOLVED FIXED

(Bugzilla :: Creating/Changing Bugs, defect)

Description

[Joel Peshkin]

In a bugzilla installation using the tip of CVS as of May 01, 2002, it can be 
impossible to change an existing bug to a differrent produst group. If this is 
attempted with bith usebuggroups and usebuggroupsentry enabled. A message 
indicating that "version" is not specified, this may be a bug in your browser is 
displayed by process_bug.cgi after the "Verify Bug Group" screen.

This problem appears to be caused by the following line in 
verify-new-product.html.tmpl version 1.2.   It appears that the contents of the 
version and component fields are excluded from propagating from one page to the 
next.

The following change (not excluding the version and component fields) seems to 
make the difference.

bash-2.03$ cvs diff verify-new-product.html.tmpl
Index: verify-new-product.html.tmpl
===================================================================
RCS file: /cvsroot/mozilla/webtools/bugzilla/template/en/default/bug/process/ver
ify-new-product.html.tmpl,v
retrieving revision 1.2
diff -r1.2 verify-new-product.html.tmpl
39c39
< [% PROCESS "global/hidden-fields.html.tmpl" exclude="^version|component|target
_milestone$" %]
---
> [% PROCESS "global/hidden-fields.html.tmpl" exclude="^target_milestone$" %]



This problem has been verified with Netscape 4.76 and ie5.
Client running on Win2000
Server running Solaris 2.7, Perl 5.6.1,  Tip of CVS as of 5-1-2001.

Comment 1

[Bradley Baetz (:bbaetz)]

gerv - you did that patch, didn't you? since we moved the files, I can't see
what the original bug # was.....

Comment 2

[Gervase Markham [:gerv]]

Myk did this one. The way you tell is because his name's at the top of the
template :-)

Gerv

Comment 3

[Bradley Baetz (:bbaetz)]

gerv: Yeah, but I thought this was a subsequent change. Oh well.

I haven't reproduced this, but looking at code, it seems that this could happen
if verify_fields isn't set, because you're moving between products where you
don't need to change the version, component, or target milestone (if any).
[e-mail], is that what you were doing?

The exclude probably needs to be conditional on verify_fields being set. myk,
does that sound right?

Comment 4

[Joel Peshkin]

That's exactly correct. I was changing ONLY the product.  This triggerred the
screen that asks if I want to change the group settings to match the new
product.  After that, the error occurred.

-Joel

Comment 5

[Bradley Baetz (:bbaetz)]

Right, but did the old product and the new product share common components and
versions? Ie were you asked to change the component/version/milestone as well as
being asked about the product groups?

Comment 6

[Joel Peshkin]

The components and versions were common to the two, so I did not change either 
field nor was I prompted to.

Comment 7

[Myk Melez [:myk] [@mykmelez]]

Attachment: patch v1: fixes problem

This patch should fix the problem.  Fixing this problem exposed a bug in
hidden-fields.html.tmpl when the value of its "exclude" parameter was the empty
string (it excluded everything when it should have excluded nothing), so I
fixed that bug as well.

Comment 8

[Joel Peshkin]

Tried it (http://bugzilla.mozilla.org/showattachment.cgi?attach_id=82091) -- 
previously broken case seems to work now.

Comment 9

[Gervase Markham [:gerv]]

Comment on attachment 82091 [details] [diff] [review]
patch v1: fixes problem

Works beautifully. 2xr=gerv.

Gerv

Comment 10

[Gervase Markham [:gerv]]

Fixed.

Checking in template/en/default/bug/process/verify-new-product.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/bug/process/verify-new-product.html.tmpl,v
 <--  verify-new-product.html.tmpl
new revision: 1.3; previous revision: 1.2
done
Checking in template/en/default/global/hidden-fields.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/global/hidden-fields.html.tmpl,v
 <--  hidden-fields.html.tmpl
new revision: 1.2; previous revision: 1.1
done

Gerv