This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 56656

Summary:	Remove File -> Refresh All Files menu item
Product:	platform	Reporter:	Jesse Glick <jglick>
Component:	Window System	Assignee:	Jesse Glick <jglick>
Status:	CLOSED FIXED
Severity:	blocker	CC:	jrojcek, mkubec, mmatula, pkeegan, rmatous
Priority:	P3	Keywords:	SIMPLEFIX, UI
Version:	4.x
Hardware:	All
OS:	All
Issue Type:	DEFECT	Exception Reporter:

Description Jesse Glick 2005-03-18 16:38:35 UTC

Jano agrees that this menu item is not useful and would be better removed. I
would like to do so for E if possible, else F. Trivial to do, awaiting comment
from QA & Docs.

Comment 1 Patrick Keegan 2005-03-18 16:59:48 UTC

Seems reasonable to me. I didn't find a reference to it anywhere in the docs.
I'll forward this to other docs folks, but unless you get an objection by, say,
Tuesday, get rid of it.

Comment 2 Jesse Glick 2005-03-23 17:27:52 UTC

Working on it.

Comment 3 Jesse Glick 2005-03-23 17:31:55 UTC

Forgot to mention: I will also remove "Refresh" from the context menu on plain
folders.

Comment 4 Patrick Keegan 2005-03-23 17:47:53 UTC

What do you mean by "plain" folders. Don't people use Refresh e.g. to make sure
that VCS status is refreshed? I wouldn't be inclined to remove this just yet.
Maybe for 4.2.

Comment 5 jrojcek 2005-03-24 16:34:28 UTC

AFAIK, CVS (not sure about other VCS) is using a special "CVS > Refresh" contextual menu item on 
versioned folders. They are actually removing the original Refresh item from the contextual menu.

Comment 6 Jesse Glick 2005-03-24 16:56:15 UTC

Right, I should have clarified that the patch only affects folders which are
*not* on a VCS filesystem. The VCS support adds its own submenu which is not
affected by this.

committed 1.60  core/src/org/netbeans/core/DataSystem.java
committed 1.142 core/src/org/netbeans/core/actions/Bundle.properties
removed   1.6   core/src/org/netbeans/core/actions/RefreshAllFilesystemsAction.java
committed 1.90  core/ui/src/org/netbeans/core/ui/resources/layer.xml
committed 1.3   openide/masterfs/src/org/netbeans/modules/masterfs/Bundle.properties
committed 1.13 
openide/masterfs/src/org/netbeans/modules/masterfs/SpecialDelegates.java
committed 1.5  
openide/masterfs/src/org/netbeans/modules/masterfs/filebasedfs/Bundle.properties
committed 1.5  
openide/masterfs/src/org/netbeans/modules/masterfs/filebasedfs/FileBasedFileSystem.java
removed   1.2  
openide/masterfs/src/org/netbeans/modules/masterfs/filebasedfs/RefreshAction.java
committed 1.49 
projects/projectui/src/org/netbeans/modules/project/ui/resources/layer.xml

Comment 7 Jesse Glick 2005-04-06 19:25:57 UTC

Please note that a menu item called File | Refresh All Files was added to NB 4.1
after this change, but it is *different*. It it provided by the Java database
and scans Java packages for external changes. It has no effect on files outside
Java source roots.

Comment 8 Marian Mirilovic 2005-07-14 16:00:41 UTC

verified / closed