It is more correct to override setEnabled(Object) than isEnabled() in the command handlers. Or if setEnabled(Object) is not overriden, at least keep in mind that some events may be skipped (for example see JBIDE-7430).