Community
Participate
Working Groups
Support for read/update/delete of the See Also field is missing from the Bugzilla connector. Adding new entries to the See Also field is working atm, although the necessary enums and constants are missing from BugzillaAttribute.java and Messages.java. Reading the contents of a bug's see also field is not possible, BugzillaClient.getTaskData() does not return such an attribute. Removing separate See Also entries is also not possible. For adding read support the SaxMultiBugReportContentHandler.endElement() method can be extended with a new case block, similar to eg.: DEPENDSON Support for adding new entries only needs new constant and enum definitions in BugzilllaAttribute.java and Messages.java/messages.properties Support for deleting See Also entries needs also new constant and enum defs and some modification in BugzillaClient.getPairsForExisting(). The handling of See Also deletion is done through a new attribute called remove_see_also. In one operation several see also entries can be removed.
Created attachment 163529 [details] Support for adding, reading and deleting See Also entries Manul tests have been performed.
Created attachment 163530 [details] mylyn/context/zip
Attila, I did a review and found the following * UI only works when you have one value in see_also. More then one is not correctly supported. * there is no way to add a value if you not already have one. * Better UI like Add CC: and CC: Can you create a new patch or should I resolve this?
Frank, I will take a look at it. I must admit I haven't checked how the UI behaves with this patch, only tested if the direct API calls are working.
For a full support we need a fix for https://bugzilla.mozilla.org/show_bug.cgi?id=557183 because the usage of this field can be off in the administation of bugzilla.
Is there any concern with applying the patch meanwhile?
(In reply to comment #6) > Is there any concern with applying the patch meanwhile? I think we need the fix for https://bugzilla.mozilla.org/show_bug.cgi?id=557183
Frank, it would be useful to support the See Also field now that eclipse.org is going to use it to link to Gerrit reviews and commits (see bug 434841). Would you be interested in working on this?
create review https://git.eclipse.org/r/40894
Created attachment 250437 [details] mylyn/context/zip
If you do not see the "See Also" Editor please check if in the Task Repository Properties under "Additional Setting" the "See Also" is not checked (group "Task Editor Attributes to Suppress").
Nothing left to do!