[teiid-designer-dev] [Teiid-Designer/teiid-designer] 7ecf98: TEIIDDES-1916: Moves the initialisation of passwor...
Branch: refs/heads/master
Home: https://github.com/Teiid-Designer/teiid-designer
Commit: 7ecf98e067a5f340ea1822f5713e8dc79ad0e935
https://github.com/Teiid-Designer/teiid-designer/commit/7ecf98e067a5f340e...
Author: Paul Richardson <p.g.richardson(a)redhat.com>
Date: 2013-11-05 (Tue, 05 Nov 2013)
Changed paths:
M
plugins/org.teiid.designer.dqp.ui/src/org/teiid/designer/runtime/ui/vdb/VdbDeployer.java
M plugins/org.teiid.designer.dqp/src/org/teiid/designer/runtime/DqpPlugin.java
M
plugins/org.teiid.designer.dqp/src/org/teiid/designer/runtime/TeiidDataSourceFactory.java
M
plugins/org.teiid.designer.dqp/src/org/teiid/designer/runtime/TeiidServerManager.java
M
plugins/org.teiid.designer.dqp/src/org/teiid/designer/runtime/preview/PreviewManager.java
M
tests/org.teiid.designer.dqp.test/src/org/teiid/designer/runtime/ServerManagerTest.java
Log Message:
-----------
TEIIDDES-1916: Moves the initialisation of password provider implementation
* TeiidDataSourceFactory
* The only class that requires the password provider, it should be
responsible for getting it.
* Getting the provider only when it is directly required, allows for the
implementing plugin to have been activated and available. Even it is not
then the data source will not be created and the user should add the
password to the profile. An altogether less catastrophic outcome!
* DqpPlugin
* TeiidServerManager
* PreviewManager
* Removes all responsibility for the password provider from these classes
Commit: 02a8dc2aae29c4720117f99df3bf8810989d66e0
https://github.com/Teiid-Designer/teiid-designer/commit/02a8dc2aae29c4720...
Author: Paul Richardson <p.g.richardson(a)redhat.com>
Date: 2013-11-05 (Tue, 05 Nov 2013)
Changed paths:
A plugins/org.teiid.designer.ui/icons/full/ctool16/refresh_model.gif
M plugins/org.teiid.designer.ui/plugin.xml
Log Message:
-----------
Avoids exception generated in logs when installed
* The action sets initialiser is unable to find the jdbc_import icon,
presumably due to the plugin not having been activated when designer is
newly started.
* Adds a copy of the icon to the designer.ui plugin to avoid a needless
dependency.
Compare:
https://github.com/Teiid-Designer/teiid-designer/compare/01fee62119b1...0...