Bug 246260 - ensure support for forthcoming Bugzilla 3.2
Summary: ensure support for forthcoming Bugzilla 3.2
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal with 2 votes (vote)
Target Milestone: 3.1   Edit
Assignee: Robert Elves CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 226851 247220 252297
Blocks:
  Show dependency tree
 
Reported: 2008-09-04 14:14 EDT by Robert Elves CLA
Modified: 2009-01-15 20:23 EST (History)
9 users (show)

See Also:


Attachments
patch (1.63 KB, patch)
2008-09-13 15:03 EDT, Frank Becker CLA
no flags Details | Diff
mylyn/context/zip (4.21 KB, application/octet-stream)
2008-09-13 15:03 EDT, Frank Becker CLA
no flags Details
new version (19.77 KB, patch)
2008-09-20 15:27 EDT, Frank Becker CLA
no flags Details | Diff
mylyn/context/zip (5.17 KB, application/octet-stream)
2008-09-20 15:27 EDT, Frank Becker CLA
no flags Details
Platform and Importance (3.91 KB, image/png)
2008-10-04 16:10 EDT, Robert Elves CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Robert Elves CLA 2008-09-04 14:14:48 EDT
Bugzilla 3.2 is on the horizon and so we need to do the testing necessary to ensure Mylyn is ready.
Comment 1 Robert Elves CLA 2008-09-04 14:20:28 EDT
Frank, do you want to start some testing on your end and I'll get the rc installed on the vserver?
Comment 3 Frank Becker CLA 2008-09-06 14:28:09 EDT
(In reply to comment #1)
> Frank, do you want to start some testing on your end and I'll get the rc
> installed on the vserver?

Yes, I can start with testing. On my Machine that version is allready installed.

Should we start a new JUnit for this version and include all existing tests?
Comment 4 Robert Elves CLA 2008-09-08 17:29:17 EDT
I think we should create tests for any failures, not necessarily a whole new suite. You could comment in the test method that it is 3.2 specific.
Comment 5 Frank Becker CLA 2008-09-10 17:38:59 EDT
Rob,

the support for the new custom field types in in bug#226851
Comment 6 Frank Becker CLA 2008-09-13 15:03:07 EDT
Created attachment 112501 [details]
patch
Comment 7 Frank Becker CLA 2008-09-13 15:03:10 EDT
Created attachment 112502 [details]
mylyn/context/zip
Comment 8 Frank Becker CLA 2008-09-14 16:44:54 EDT
Bugzilla 3.2 use the following new way.

There are two places where an attribute has no lables.

1) Platform has select for rep_platform and op_sys
2) Importance has select for priority and bug_severity

Should we provide an generic fix or should we do two specific implementation for the described two cases. 
Comment 9 Frank Becker CLA 2008-09-20 15:27:25 EDT
Created attachment 113068 [details]
new version

New Editor for two Combo Attribute is included in this patch
Comment 10 Frank Becker CLA 2008-09-20 15:27:29 EDT
Created attachment 113069 [details]
mylyn/context/zip
Comment 11 Robert Elves CLA 2008-09-23 12:22:35 EDT
Great stuff Frank. As for our ability to automate testing, due to security precautions, upgrading perl modules isn't feasible on the vServer and this new release requires a few modules not currently installed.  I've email Denis about this so will likely have a server running this week.
Comment 12 Kevin Benton CLA 2008-09-23 12:27:05 EDT
If this patch will support custom fields, can we also add support for flags at the same time?
Comment 13 Frank Becker CLA 2008-09-23 22:56:27 EDT
(In reply to comment #11)
> Great stuff Frank. As for our ability to automate testing, due to security
> precautions, upgrading perl modules isn't feasible on the vServer and this new
> release requires a few modules not currently installed.

Have you seen http://www.bugzilla.org/releases/3.2/release-notes.html#v32_feat_install

You don't need administrator-level access to your machine for the ney installation

Comment 14 Robert Elves CLA 2008-09-24 01:18:10 EDT
(In reply to comment #13)
> (In reply to comment #11)
> > Great stuff Frank. As for our ability to automate testing, due to security
> > precautions, upgrading perl modules isn't feasible on the vServer and this new
> > release requires a few modules not currently installed.
> 
> Have you seen
> http://www.bugzilla.org/releases/3.2/release-notes.html#v32_feat_install
> 
> You don't need administrator-level access to your machine for the ney
> installation
Hey that is really handy, thanks for pointing that out Frank. Unfortunately, as for our vServer, it is completely blocked from opening outbound network connections (for security reasons).  The way we've installed in the past is to scp a bugzilla.tar up there then run the install as usual. All past installs have been fine and didn't require any more perl modules.  The 3.2 stream seems to require a 3-4 modules (+ dependencies) that aren't already installed, so we're stuck. There are work-arounds which I've attempted but they become too time consuming. We're looking into replacing the vServer entirely and if this happens, will likely have the modules we need (since the current vServer is quite dated).
Comment 15 Robert Elves CLA 2008-09-24 01:27:43 EDT
regarding comment#14, see bug#248363
Comment 16 Robert Elves CLA 2008-10-04 16:10:11 EDT
Created attachment 114266 [details]
Platform and Importance


  (In reply to comment #12)
> If this patch will support custom fields, can we also add support for flags at
> the same time?
Kevin, I see you've already found the related bug but for others interested, flag support is being tracked on bug#186265.  Also of note is that progress is being made on 3.2 realted custom attribute improvements on bug#186265.

Frank, your patch updates the rich editor to reflect the new Bugzilla ui very well (with minor nits we can work out). Although the direction Bugzilla has taken with the Platform and Importance fields decreases the footprint slightly, in my opinion the lack of labels is somewhat confusing.  Lets get Miks, take on these UI changes before we proceed.  

Mik, attached is a screenshot showing how platform and os have been mergedinto a double dropdown simply labled Platform. Similarly, priority and severity have been merged into Improtance. This has the potential to decrease the overall size of the attributes section but may be at the cost of clarity. Thoughts?
Comment 17 Mik Kersten CLA 2008-10-07 00:36:31 EDT
I like what they've done with merging these fields.  But I think we should wait on changing the attributes section, since we need to consider which part of this belongs in the task editor header.  For example, we already have a graphical way of indicating "Importance" with the task and priority icon, and could make that more prominent and editable.
Comment 18 Kevin Benton CLA 2008-10-07 11:33:04 EDT
(In reply to comment #17)
> I like what they've done with merging these fields.  But I think we should wait
> on changing the attributes section, since we need to consider which part of
> this belongs in the task editor header.  For example, we already have a
> graphical way of indicating "Importance" with the task and priority icon, and
> could make that more prominent and editable.
> 

Please be aware that using names other than what is indicated in the Bugzilla may cause conflicts with custom field attributes.  For example, the image I saw used "Importance" for the combination of Priority and Severity.  Many companies actually have service level agreements defined that dictate who is allowed to set Priority and who is allowed to set Severity.  If I were recoding that section, I would probably change Importance to Pri./Sev. because a) it doesn't further overload meaning in these fields, and b) it actually takes even less space than Importance. :-)
Comment 19 Frank Becker CLA 2008-10-30 18:34:40 EDT
(In reply to comment #18)
> (In reply to comment #17)
> > I like what they've done with merging these fields.  But I think we should wait
> > on changing the attributes section, since we need to consider which part of
> > this belongs in the task editor header.  For example, we already have a
> > graphical way of indicating "Importance" with the task and priority icon, and
> > could make that more prominent and editable.
> > 
> 
> Please be aware that using names other than what is indicated in the Bugzilla
> may cause conflicts with custom field attributes.  For example, the image I saw
> used "Importance" for the combination of Priority and Severity. 

This is the name used in Bugzilla 3.2rc1 so if this is a problem for you please report this to Bugzilla.
 > Many companies
> actually have service level agreements defined that dictate who is allowed to
> set Priority and who is allowed to set Severity.  If I were recoding that
> section, I would probably change Importance to Pri./Sev. because a) it doesn't
> further overload meaning in these fields, and b) it actually takes even less
> space than Importance. :-)
> 

Comment 20 Kevin Benton CLA 2008-10-31 11:46:34 EDT
(In reply to comment #19)
> (In reply to comment #18)
> > Please be aware that using names other than what is indicated in the Bugzilla
> > may cause conflicts with custom field attributes.  For example, the image I saw
> > used "Importance" for the combination of Priority and Severity. 
> 
> This is the name used in Bugzilla 3.2rc1 so if this is a problem for you please
> report this to Bugzilla.

That's true in the bug update screen, however, it's not true in the bug entry screen.  I'm still lobbying with the BZ community to change it in such a way that it allows administrators to determine which label is displayed.

>  > Many companies
> > actually have service level agreements defined that dictate who is allowed to
> > set Priority and who is allowed to set Severity.  If I were recoding that
> > section, I would probably change Importance to Pri./Sev. because a) it doesn't
> > further overload meaning in these fields, and b) it actually takes even less
> > space than Importance. :-)

Of course, this is the "why"... :-)
Comment 22 Robert Elves CLA 2009-01-15 20:23:35 EST
The major bugs to support 3.2 have been resolved. Frank, I've moved your patch for editor nits to bug#13941394.