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.
Please do it for org.netbeans.modules.properties.PropertiesDataLoader For details see issue #64153
This was done three months ago - see issue #59449.
Aha, the defaultActions return null. Is it necessary?
Yes, it is necessary because the API documentation does not specify the default value. The default value can be changed at any time (without breaking API compatibility) and I want it to be always null. Javadoc description of the method is at http://www.netbeans.org/download/dev/javadoc/LoadersAPI/org/openide/loaders/DataLoader.html#defaultActions()
Please cease to override defaultActions(). It does not matter what it returns if you are specifying a layer context, since it will not be called.
OK, I will do it. But bear in mind that it is not clear from the API description what is called first - defaultActions() or actionsContext(). Should I file an issue for this?
Go ahead and file a docs bug if you think it is needed.
Filed: http://www.netbeans.org/issues/show_bug.cgi?id=67712
What is the problem with overriding method that is not called? Functionality in org.openide.loaders shall not be affected by doing that...
Since my proposals of the API change were not accepted, I will keep the method.
"Since my proposals of the API change were not accepted, I will keep the method." - huh? The new Javadoc says that the old methods is not needed any more. Delete it, please, so that we do not have bogus matches in Find Usages etc.
Strictly speaking, neither the previous nor the new Javadoc description contains any obligation regarding the default value or use of the method. Compare it with description of actionsContext(): The default implementation returns null to indicate that no layer reading should be used (use {@link #defaultActions} instead).