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 47406 - "Source Packages" node should not have properties
Summary: "Source Packages" node should not have properties
Status: CLOSED FIXED
Alias: None
Product: java
Classification: Unclassified
Component: Project (show other bugs)
Version: 4.x
Hardware: All All
: P3 blocker (vote)
Assignee: Petr Hrebejk
URL:
Keywords:
Depends on:
Blocks: 41537
  Show dependency tree
 
Reported: 2004-08-18 16:41 UTC by Jaromir Uhrik
Modified: 2006-03-24 10:19 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
Exception stack (4.20 KB, text/plain)
2004-08-18 16:42 UTC, Jaromir Uhrik
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jaromir Uhrik 2004-08-18 16:41:34 UTC
[200408180923; jdk1.5.0-rc-b63]
1.invoke properties of "Source Packages" (similar
problem is with "Test Packages")
2.change property "name"
3.NPE appears and the "Source Packages" will
vanish from the Projects view

The problem only in that the change of property is
not delegated in the projects.properties
Comment 1 Jaromir Uhrik 2004-08-18 16:42:21 UTC
Created attachment 16927 [details]
Exception stack
Comment 2 Jesse Glick 2004-08-18 20:13:56 UTC
This node should not have any properties and "Properties..." should
not be in the context menu; it is useless, an accidental byproduct of
the FolderNode-based implementation. Renaming ${src.dir} from the GUI
is not supported. Jano please update the spec.

Petr when fixed, please reassign to javacore as it clearly has some
kind of robustness bug here.
Comment 3 jrojcek 2004-08-19 16:52:55 UTC
I assume that also a package node should not have the Properties menu item.
Comment 4 Jesse Glick 2004-08-19 18:13:23 UTC
Indeed.

The "Sort Mode" property could in principle be useful in rare cases,
but probably not worth cluttering the GUI with. Anyway I'm not sure it
even works; and the "By Name (Packages First)" choice is nonsense in
the package view.
Comment 5 Petr Hrebejk 2004-08-23 13:04:37 UTC
Checking in
java/project/src/org/netbeans/spi/java/project/support/ui/PackageRootNode.java;
/cvs/java/project/src/org/netbeans/spi/java/project/support/ui/PackageRootNode.java,v
 <--  PackageRootNode.java
new revision: 1.9; previous revision: 1.8

Checking in
project/src/org/netbeans/spi/java/project/support/ui/PackageViewChildren.java;
/cvs/java/project/src/org/netbeans/spi/java/project/support/ui/PackageViewChildren.java,v
 <--  PackageViewChildren.javanew revision: 1.30; previous revision: 1.29
done

Comment 6 Jaromir Uhrik 2004-08-25 16:24:37 UTC
Verified in dev build #200408241800; jdk1.5.0-rc-b63 (no properties
available on the Source Packages and Test Packages node)