Community
Participate
Working Groups
AbstractConnectionCreationTool#calculateCursor currently looks like that: protected Cursor calculateCursor() { if (isInState(STATE_INITIAL)) { if (getCurrentCommand() != null) return getDefaultCursor(); } return super.calculateCursor(); } It sets the cursor to Default even if the command is not actually executable. This can mislead User into thinking that a Connection is supported when it's not. A better implementation would be: protected Cursor calculateCursor() { if (isInState(STATE_INITIAL)) { Command command = getCurrentCommand(); if (command != null && command.isExecutable()) return getDefaultCursor(); } return super.calculateCursor(); } (BTW, super.calculateCursor() does call canExecute())
We assume that the command can not execute at that point because it contains only partial imformation. We need to add a new style setting for the connection creation tool that indicates that the command will answer canExecute() correctly, even though it is incomplete. You can then enable that style from the palette entry.
