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 9698 - "New" wizard should hide read-only filesystems from you
Summary: "New" wizard should hide read-only filesystems from you
Status: RESOLVED FIXED
Alias: None
Product: platform
Classification: Unclassified
Component: Dialogs&Wizards (show other bugs)
Version: 3.x
Hardware: PC Linux
: P2 blocker (vote)
Assignee: Jiri Rechtacek
URL:
Keywords: UI
Depends on:
Blocks:
 
Reported: 2001-02-20 10:26 UTC by Jesse Glick
Modified: 2008-12-22 20:51 UTC (History)
2 users (show)

See Also:
Issue Type: ENHANCEMENT
Exception Reporter:


Attachments
patch (1.13 KB, patch)
2002-05-02 14:57 UTC, mslama
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jesse Glick 2001-02-20 10:26:40 UTC
Not sure if it already disables the Next button when you select a read-only
folder on panel #2, but anyway read-only filesystems should be excluded entirely
from this panel of the wizard since there is no way you can create a new file in
them.
Comment 1 Jan Chalupa 2001-05-06 08:14:15 UTC
Target milestone -> 3.3
Comment 2 Jan Chalupa 2001-11-27 13:04:10 UTC
Target milestone -> 3.3.1.
Comment 3 mslama 2002-04-29 14:18:49 UTC
Please Jesse how to filter read only filesystems from view? I found
org.openide.loaders.TemplateWizard2.createPackageNode().
Comment 4 Jesse Glick 2002-04-30 02:07:55 UTC
Looks like ExCh.createNodes is where to look.... it could check for
((Node)key).getCookie(DataFolder.class) being the root folder of a r/o
FS, and if so, return new Node[0]. Or maybe TW2.acceptDataObject could
return false for r/o FSs, that would probably be easiest.
Comment 5 mslama 2002-05-02 14:53:13 UTC
I tried both and both work. The only difference is that
acceptDataObject() is called for all nodes as tree is expanded and
ExCh.createNodes() is called just for root and its children.

Solution in acceptDataObject() is simpler because it means just adding
one additional check because filesystem is alreadu used there.
I will attach diff for this solution.
Comment 6 mslama 2002-05-02 14:57:31 UTC
Created attachment 5624 [details]
patch
Comment 7 Jesse Glick 2002-05-02 17:45:16 UTC
Looks right to me...
Comment 8 mslama 2002-05-03 11:28:13 UTC
Patch is integrated to main trunk.

Modified:
 org/openide/loaders/TemplateWizard2.java r.1.41
Comment 9 Marian Mirilovic 2003-07-14 16:54:39 UTC
Marek, I have tried it on [nb_dev](20030714) and it isn't fixed.
Comment 10 mslama 2003-12-01 09:23:02 UTC
Reassigning to new owner of wizards.
Comment 11 Jiri Rechtacek 2004-04-20 08:34:49 UTC
I think it's already solved, an user is notified be message on
read-only filesystem and Next/Finish buttons are disabled. Also fixed
issue 32191 make sure that New actions are disabled in popup menu.
Comment 12 Jesse Glick 2004-04-21 19:45:56 UTC
Anyway for D the issue makes no sense - there are no user-visible
filesystems.