Bug 79445 - [Wizards] Share Project dialog lists repositories in random order
Summary: [Wizards] Share Project dialog lists repositories in random order
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: CVS (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows 2000
: P5 normal (vote)
Target Milestone: 3.4 M1   Edit
Assignee: Tomasz Zarna CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
: 66521 114985 (view as bug list)
Depends on:
Blocks:
 
Reported: 2004-11-24 16:05 EST by Nick Edgar CLA
Modified: 2007-08-07 12:07 EDT (History)
3 users (show)

See Also:

Attachments
Screen shot (47.50 KB, image/jpeg)
2004-11-24 16:06 EST, Nick Edgar CLA 		no flags Details
Fix (1.37 KB, patch)
2007-07-17 07:45 EDT, Tomasz Zarna CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Nick Edgar CLA 2004-11-24 16:05:59 EST 
build I20041124

- share a new project when you have many repo locations defined
- the order in the Share Project dialog appears to be random
Comment 1 Nick Edgar CLA 2004-11-24 16:06:18 EST 
Created attachment 16111 [details]
Screen shot
Comment 2 Michael Valenta CLA 2005-11-04 10:57:52 EST 
*** Bug 114985 has been marked as a duplicate of this bug. ***
Comment 3 Michael Valenta CLA 2006-08-17 11:14:47 EDT 
There is currently no plan to address this issue.
Comment 4 Tomasz Zarna CLA 2007-07-13 11:32:41 EDT 
IMO this bug is related to bug 66521. We will need to add the same ordering mechanism to the import/export wizard.
Comment 5 Michael Valenta CLA 2007-07-16 10:08:26 EDT 
This should be straight forward.
Comment 6 Tomasz Zarna CLA 2007-07-17 07:45:48 EDT 
Created attachment 73937 [details]
Fix

I used the same comparator as in the CVS Repositories view. This patch also fixes bug 66521 as both Check out Wizard and Share Project Wizard use the same Page (i.e.RepositorySelectionPage) which has been just adjusted.
Comment 7 Tomasz Zarna CLA 2007-07-17 07:48:34 EDT 
*** Bug 66521 has been marked as a duplicate of this bug. ***
Comment 8 Michael Valenta CLA 2007-07-17 11:07:15 EDT 
Patch released to HEAD
Comment 9 Tomasz Zarna CLA 2007-07-25 09:04:00 EDT 
Nick could you verify it? It looks good to me in N20070723-0010.
Comment 10 Michael Valenta CLA 2007-08-07 12:07:26 EDT 
Verified in build I20070807-0010