Closed
Bug 378681
Opened 18 years ago
Closed 15 years ago
Bug XML only contains group names, while process_bug only accepts group IDs
Categories
(Bugzilla :: Bug Import/Export & Moving, defect)
Bugzilla
Bug Import/Export & Moving
Tracking
()
RESOLVED
FIXED
Bugzilla 3.4
People
(Reporter: rob.elves, Assigned: LpSolit)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
(deleted),
patch
|
gregaryh
:
review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.3) Gecko/20070309 Firefox/2.0.0.3
Build Identifier: Firefox 2.0.0.3
Our client (eclipse.org/mylar) strictly makes use of the xml dump to make changes to existing bug reports. Unfortunately, groups are provided in the xml without any reference to the id/name used in the show_bug.cgi html such as "bit-16". There doesn't appear to be any guide to reproduce these ids client side when posting updates. Without this internal id/name mapping we are unable to post the groups information correctly resulting in loss of previously flagged groups on the bug report.
Secondly, the group options are not provided in the configuration rdf dump. This also makes working with groups outside of html difficult since we cannot display what groups are available to choose from.
Reproducible: Always
Steps to Reproduce:
1.View the xml dump of a bug report and notice that only group tag is provided without internal id/name mapping necessary to update the bug
2. View config.cgi rdf output is missing available groups
Reporter | ||
Comment 1•18 years ago
|
||
Old, potentially related bug report:
https://bugzilla.mozilla.org/show_bug.cgi?id=275386
Assignee | ||
Comment 2•18 years ago
|
||
You are reporting two bugs here, one about importxml.pl and another one about config.cgi. The one about importxml.pl is invalid as exporting the bug data to another DB means the group ID is probably different. What is relevant is the group name.
About groups being displayed in config.cgi, we should probably only restrict them to groups you belong to as some groups may be confidential. If you want the config.cgi part of your request to be addressed, please change the summary of this bug, else close it as INVALID.
Reporter | ||
Comment 3•18 years ago
|
||
(In reply to comment #2)
> You are reporting two bugs here, one about importxml.pl and another one about
> config.cgi. The one about importxml.pl is invalid as exporting the bug data to
> another DB means the group ID is probably different. What is relevant is the
> group name.
In our use case, we're trying to submit an update to a regular bug (via process_bug.cgi) back to the remote repository but don't have the id/name required for form submission (because all we have to work with is what we get in the xml).
Reporter | ||
Comment 4•18 years ago
|
||
To clarify, we're submitting back to the same Bugzilla repository the data came from, not another database.
Comment 6•17 years ago
|
||
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment 7•17 years ago
|
||
with mylyn and groupsecurity enabled this is IMO a showstopper, because each modification on the mylyn side (or any other client, that uses bugzilla xml) _resets_ the group access rights on the bug.
The only workaround is to disable the rich editor and use the browser option.
please add the groupsecurity information into the xml
Comment 8•17 years ago
|
||
As additional comment, deskzilla (a bugzilla rich desktop client) now _supports_ group visibility, but I assume that they somehow parse the html on the client side and do not rely on bugzilla s xml format.
http://almworks.com/deskzilla/
Updated•16 years ago
|
Blocks: bz-clients
Comment 9•16 years ago
|
||
Okay, so this is two bugs. One is that process_bug only understands ids for groups, not their names, while we only publish their names in the XML. This would of course be fixed in the XML-RPC interface when we have Bug.update(), but in the mean time we should do something about it. I'm morphing this bug to just address that fact.
The other bug is that config.cgi doesn't contain info about what groups are valid in which products. I've filed bug 449139 for that.
No longer blocks: bz-clients
Summary: Unable to update groups using information strictly provided in bug xml → Bug XML only contains group names, while process_bug only accepts group IDs
Updated•16 years ago
|
Blocks: bz-clients
Reporter | ||
Comment 10•16 years ago
|
||
Is this on the radar for 3.4? This combined with bug#449139 would make the group functionality in the Mylyn editor complete.
Assignee | ||
Updated•15 years ago
|
Assignee: create-and-change → LpSolit
Status: NEW → ASSIGNED
Component: Creating/Changing Bugs → Bug Import/Export & Moving
Target Milestone: --- → Bugzilla 3.4
Assignee | ||
Comment 11•15 years ago
|
||
Attachment #407008 -
Flags: review?(ghendricks)
Updated•15 years ago
|
Attachment #407008 -
Flags: review?(ghendricks) → review+
Assignee | ||
Updated•15 years ago
|
Flags: approval3.4+
Flags: approval+
Assignee | ||
Comment 12•15 years ago
|
||
tip:
Checking in template/en/default/bug/show.xml.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/bug/show.xml.tmpl,v <-- show.xml.tmpl
new revision: 1.32; previous revision: 1.31
done
3.4.2:
Checking in template/en/default/bug/show.xml.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/bug/show.xml.tmpl,v <-- show.xml.tmpl
new revision: 1.29.2.3; previous revision: 1.29.2.2
done
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Comment 13•15 years ago
|
||
The trouble with fixing it this way is that it makes clients keep track of both names and numbers. (In other words, a group becomes an object with a collection of properties rather than just a simple string.)
A cleaner interface would be for clients to use only the names, and have process_bug accept either names or numbers on input. Is there any possibility of that?
Gerv
Comment 14•15 years ago
|
||
Names can change; IDs can't. I think using IDs makes more sense for stability purposes.
Assignee | ||
Comment 15•15 years ago
|
||
I agree with reed. I think the way I did it is better than relying on names in process_bug.cgi.
Comment 16•15 years ago
|
||
OK; I will update the API to use group objects, containing both names and IDs.
Gerv
You need to log in
before you can comment on or make changes to this bug.
Description
•