Bug 210568 - [Import/Export] Refresh button does not update list of projects
Summary: [Import/Export] Refresh button does not update list of projects
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: IDE (show other bugs)
Version: 3.3.1   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.4 M6   Edit
Assignee: Tod Creasey CLA
QA Contact:
URL:
Whiteboard: hasPatch
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2007-11-21 13:56 EST by Igor Kozlov CLA
Modified: 2020-11-02 06:26 EST (History)
2 users (show)

See Also:

Attachments
Patch to use the modification timestamp of the file or folder instead of its name to determine whether a refresh is needed or not. (2.09 KB, patch)
2007-11-22 23:35 EST, Remy Suen CLA 		no flags Details | Diff
New patch to check both the string and the modification timestamp. (2.21 KB, patch)
2007-11-23 09:58 EST, Remy Suen CLA 		no flags Details | Diff
New patch to reset the path if the path is null or its length is zero. (4.86 KB, patch)
2007-11-23 12:10 EST, Remy Suen CLA 		no flags Details | Diff
Updated patch for latest CVS HEAD. (5.01 KB, patch)
2007-12-04 14:10 EST, Remy Suen CLA 		no flags Details | Diff
Show Obsolete (3) 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 Igor Kozlov CLA 2007-11-21 13:56:31 EST 
Refresh button does not update list of projects 
Steps to reproduce
1. Invoke Import Existing Projects wizard
2. Select any folder as the root directory [List of project will be shown]
3. Copy another project into that folder 
4. Press Refresh

The new project is not shown.
I would expect that the Refresh button rescans the directory for new projects.
Comment 1 Remy Suen CLA 2007-11-22 23:35:38 EST 
Created attachment 83596 [details]
Patch to use the modification timestamp of the file or folder instead of its name to determine whether a refresh is needed or not.
Comment 2 Remy Suen CLA 2007-11-23 07:21:50 EST 
Comment on attachment 83596 [details]
Patch to use the modification timestamp of the file or folder instead of its name to determine whether a refresh is needed or not.

Scratch that. The comparison should check both the string and the modification timestamp as the user may select another file or folder that has the same timestamp as what was previously selected.
Comment 3 Remy Suen CLA 2007-11-23 09:58:12 EST 
Created attachment 83641 [details]
New patch to check both the string and the modification timestamp.
Comment 4 Remy Suen CLA 2007-11-23 12:10:15 EST 
Created attachment 83662 [details]
New patch to reset the path if the path is null or its length is zero.

The second patch didn't reset the path so if you entered a valid folder, refreshed, delete the path, refreshed (now there are no projects), then used the same path and refreshed again, nothing would happen as the stored path was the same as the new and identical path that had been entered previously.

This patch will store the reset to ensure that a refresh will happen if a path is entered after an attempt at refreshing nothing.
Comment 5 Tod Creasey CLA 2007-12-04 13:05:25 EST 
Remy this patch is out of date - would you mind updating it?
Comment 6 Remy Suen CLA 2007-12-04 14:10:03 EST 
Created attachment 84448 [details]
Updated patch for latest CVS HEAD.

(In reply to comment #5)
> Remy this patch is out of date - would you mind updating it?

Your wish is my command.
Comment 7 Tod Creasey CLA 2008-02-28 16:15:48 EST 
Patch released for build >20080228
Comment 8 Tod Creasey CLA 2008-03-25 09:06:23 EDT 
Verified in I20080325-0100