Skip to content
This repository has been archived by the owner on May 28, 2022. It is now read-only.

Config and DB initialized multiple times when running recordtool #406

Closed
zxiiro opened this issue Oct 24, 2013 · 3 comments
Closed

Config and DB initialized multiple times when running recordtool #406

zxiiro opened this issue Oct 24, 2013 · 3 comments

Comments

@zxiiro
Copy link
Member

zxiiro commented Oct 24, 2013

2013-10-23 21:35:21,857 (   DEBUG) freeseer.frontend.qtcommon.FreeseerApp  : Detected user's locale as en_US
2013-10-23 21:35:21,858 (    INFO) freeseer.framework.config               : Video directory exists.
2013-10-23 21:35:21,929 (    INFO) freeseer.framework.database             : Database upgrade failed.
2013-10-23 21:35:22,053 (   DEBUG) freeseer.framework.plugin               : Plugin manager initialized.
2013-10-23 21:35:22,117 (   DEBUG) freeseer.framework.multimedia           : Gstreamer initialized.
2013-10-23 21:35:22,193 (    INFO) freeseer.frontend.controller.Client     : Starting Client
2013-10-23 21:35:22,334 (   DEBUG) freeseer.frontend.qtcommon.FreeseerApp  : Detected user's locale as en_US
2013-10-23 21:35:22,403 (    INFO) freeseer.framework.config               : Video directory exists.
2013-10-23 21:35:22,507 (   DEBUG) freeseer.framework.plugin               : Plugin manager initialized.
2013-10-23 21:35:22,567 (    INFO) freeseer.frontend.qtcommon.FreeseerApp  : Switching language to: English
2013-10-23 21:35:22,744 (   DEBUG) freeseer.frontend.qtcommon.FreeseerApp  : Detected user's locale as en_US
2013-10-23 21:35:22,823 (    INFO) freeseer.framework.config               : Video directory exists.
QSqlDatabasePrivate::removeDatabase: connection 'qt_sql_default_connection' is still in use, all queries will cease to work.
QSqlDatabasePrivate::addDatabase: duplicate connection name 'qt_sql_default_connection', old connection removed.
2013-10-23 21:35:22,957 (    INFO) freeseer.framework.database             : Database upgrade failed.
2013-10-23 21:35:23,046 (    INFO) freeseer.frontend.qtcommon.FreeseerApp  : Switching language to: English
2013-10-23 21:35:23,134 (    INFO) freeseer.frontend.record.record         : Loading settings...
2013-10-23 21:35:23,221 (    INFO) freeseer.frontend.qtcommon.FreeseerApp  : Switching language to: English

@zxiiro
Copy link
Member Author

zxiiro commented Oct 24, 2013

RecordTool initializes a Config and DB instance
ConfigTool initializes a Config and PluginManager instance
TalkEditor initializes a Config and DB instance

We should make these instances optional parameters to pass to the initializers so that they don't duplicate.

zxiiro added a commit that referenced this issue Dec 13, 2013
@zxiiro
Copy link
Member Author

zxiiro commented Dec 13, 2013

Fixed in 3.0.x maintenance branch but will likely have to re-evaluate if the solution needs to be different in the master branch after merging all the UCOSP code from this term.

@zxiiro
Copy link
Member Author

zxiiro commented Dec 20, 2013

In the 3.1.x branch this is fixed via PR #373

@zxiiro zxiiro closed this as completed Dec 20, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant