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.
To reproduce create properties file from template (Other|Properties File). It is automatically opened and the following is logged into console: java.lang.IllegalStateException: This must happen in the event thread! It seems it is caused by calling findAction.setActionPerformer(...) as was identified in issue 35953.
Created attachment 11776 [details] Full stack trace
Yes, indeed. I will make a change so that the method is called directly (from the current thread), instead of posting it as a task for a request processor.
Better to get rid of the call to setActionPerformer and the override of componentActivated altogether. This style is deprecated as of NB 3.5. Instead, make FindPerformer impl Action (not just ActionPerformer), and in the constructor for PropertiesCloneableTopComponent, get your ActionMap and bind "org.openide.actions.FindAction" to the desired FindPerformer instance. Simpler and safer.
I have tried it for the Search module. It didn't work. I've also looked whether any other module uses the recommended solution and found that all modules using the FindAction use the deprecated setActionPerformer(). I'am starting a discussion about this issue on mailing list nbdev@netbeans.org.
So you tried it and it didn't work - either you didn't do it quite right, or there is a core bug (P2 probably), please file. I know modules are not yet using the proper technique - that's why you have to start!
*** Issue 36783 has been marked as a duplicate of this issue. ***
*** Issue 39170 has been marked as a duplicate of this issue. ***
It seems this issue has critical impact on automated test run. It blocks xtest on windows machines. It throws these messages: Warning: use of system property netbeans.debug.exceptions in org.netbeans.modules.properties.PropertiesTableModel$TablePropertyBundleListener has been obsoleted in favor of ErrorManager The following property was fired by Node org.netbeans.modules.properties.PropertiesLocaleNode@16103c7[Name=, displayName=Default]: cookie. This should be fired only to Node listeners, not general property change listeners
Emanuel: the two warnings just added to this issue are indeed (minor) bugs in the properties module, but have nothing to do with this issue whatsoever.
Well. I add them because they are usually the last messages which are logged before deadlock of automated test.
Just a coincidence.
The call stack originates from method PropertiesOpen.PropertiesCloneableTopComponent.componentActivated() (its beginning is currently at line 840)
Fixed in the trunk.
Verified in build 20040224-0536. It should override getPersistenceType() to get rid of the following note: Note - org.netbeans.modules.properties.PropertiesOpen$PropertiesCloneableTopComp onent ought to override getPersistenceType() rather than using the client proper ty or accepting the default.