Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 319295 - review custom workflow repository settings UI
Summary: review custom workflow repository settings UI
Status: RESOLVED DUPLICATE of bug 341829
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Mylyn Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 259291
  Show dependency tree
 
Reported: 2010-07-08 14:12 EDT by Robert Elves CLA
Modified: 2011-05-21 14:25 EDT (History)
2 users (show)

See Also:


Attachments
Patch for 319295 (17.17 KB, patch)
2010-08-06 17:05 EDT, Charley Wang CLA
no flags Details | Diff
mylyn/context/zip (19.47 KB, application/octet-stream)
2010-08-06 17:05 EDT, Charley Wang CLA
no flags Details
UI Patch for 319295 (9.23 KB, text/plain)
2010-08-06 17:06 EDT, Charley Wang CLA
no flags Details
Tests for 319295 (1.03 KB, text/plain)
2010-08-06 17:06 EDT, Charley Wang CLA
no flags Details
Combined patch (19.70 KB, patch)
2010-08-13 14:17 EDT, Charley Wang CLA
no flags Details | Diff
mylyn/context/zip (2.78 KB, application/octet-stream)
2010-08-13 14:17 EDT, Charley Wang 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 2010-07-08 14:12:01 EDT
Current UI is difficult for users to understand, need to provide some pointer to docs.
Comment 1 Charley Wang CLA 2010-07-19 13:53:49 EDT
Hi,

I wrote up a very brief guide to the current UI at the top of my User talk page on Eclipse:

http://wiki.eclipse.org/User_talk:Charley.wang.gmail.com

I can migrate this into the Mylyn User Guide or another more appropriate location if it is all right. 

Comments and suggestions are welcome.

-Charley
Comment 2 Steffen Pingel CLA 2010-07-19 15:46:23 EDT
Thanks Charley! We'll add that to the user guide once we update it for 3.5. Currently it sill contains the 3.4 content.
Comment 3 Steffen Pingel CLA 2010-07-29 14:31:10 EDT
We discussed this on a call and determined that it might be best to simply remove the workflow settings and always default to auto-detection:

1. Check if XML-RPC is available
2. Check for a work flow properties file on the server, e.g. "https://bugs.eclipse.org/bugs/workflow.properties"
3. Fall-back to default workflow

In addition we could check for a system property, e.g. "org.eclipse.mylyn.bugzilla.workflow.bugs.eclipse.org", that could point to a local file to support advanced users.
Comment 4 Charley Wang CLA 2010-07-29 15:08:47 EDT
(In reply to comment #3)
> We discussed this on a call and determined that it might be best to simply
> remove the workflow settings and always default to auto-detection:
> 
> 1. Check if XML-RPC is available
> 2. Check for a work flow properties file on the server, e.g.
> "https://bugs.eclipse.org/bugs/workflow.properties"
> 3. Fall-back to default workflow
> 
> In addition we could check for a system property, e.g.
> "org.eclipse.mylyn.bugzilla.workflow.bugs.eclipse.org", that could point to a
> local file to support advanced users.

This definitely makes more sense as a UI flow -- having the file available on the Bugzilla server is much better than asking users to maintain their own. I would be happy to take a look at doing the changes this week or next, if nobody else has done so :)

-C
Comment 5 Charley Wang CLA 2010-08-06 17:05:13 EDT
Created attachment 176068 [details]
Patch for 319295

Includes the core component and a minor change to the UI. Changes the default Bugzilla behaviour but does not affect the frontend.
Comment 6 Charley Wang CLA 2010-08-06 17:05:16 EDT
Created attachment 176069 [details]
mylyn/context/zip
Comment 7 Charley Wang CLA 2010-08-06 17:06:09 EDT
Created attachment 176070 [details]
UI Patch for 319295

Removes unused portions of the Mylyn Bugzilla Repository Preference page that were used for Custom Workflow settings
Comment 8 Charley Wang CLA 2010-08-06 17:06:55 EDT
Created attachment 176071 [details]
Tests for 319295

Makes a slight change to the way Custom Repository tests are set up so that existing tests will not break with the new behaviour
Comment 9 Charley Wang CLA 2010-08-06 17:13:30 EDT
The current workflow goes:

1. Check file specified by system property org.eclipse.mylyn.bugzilla.desciptorFile 
2. Check file specified by the Bugzilla installation at getRepositoryUrl() + "/workflow.properties"
3. Check XMLRPC
4. Default workflow

I was thinking this would let an advanced user override all settings, allow a Bugzilla maintainer to override XMLRPC settings, and still allow XMLRPC to override default workflow. I'd be happy to change it use XMLRPC first, if you prefer :)

I need to regenerate the UI patch, currently one of the hunks is being ignored which results in 10 lines not being properly deleted near the end of BugzillaRepositorySettingsPage.
Comment 10 Steffen Pingel CLA 2010-08-06 17:17:00 EDT
Sounds reasonable, particularly since XML-RPC is still work in progress. Could you make a single patch that includes all changes?
Comment 11 Charley Wang CLA 2010-08-06 18:53:13 EDT
(In reply to comment #10)
> Sounds reasonable, particularly since XML-RPC is still work in progress. Could
> you make a single patch that includes all changes?

Sure, but it will be > 250 lines, is that all right?
Comment 12 Charley Wang CLA 2010-08-13 14:17:08 EDT
Created attachment 176577 [details]
Combined patch

Combines the three patches above, rebased to the latest Bugzilla from HEAD. Most of the changes just involve removing UI elements from the Bugzilla Repository Properties page.
Comment 13 Charley Wang CLA 2010-08-13 14:17:10 EDT
Created attachment 176578 [details]
mylyn/context/zip
Comment 14 Charley Wang CLA 2010-08-13 15:21:17 EDT
For the Aug 13-2010 2:17PM patch, the Bugzilla test suite does run correctly after the patch is applied.
Comment 15 Charley Wang CLA 2010-08-17 16:28:07 EDT
Please advise if there are any further steps I need to take for this patch or if I should change the workflow further.
Comment 16 Steffen Pingel CLA 2011-05-21 14:25:43 EDT
Let's track this on bug 341829.

*** This bug has been marked as a duplicate of bug 341829 ***