Database cleanup #34

Merged
merged 4 commits into from Sep 7, 2012

Projects

None yet

2 participants

@fyfe
Contributor
fyfe commented Aug 15, 2012

I've started cleaning up the database code. These patches also fix a problem where you can't start EMM for the first time without a internet connection because it needs to download the SQL to create the database from the internet.

fyfe added some commits Aug 15, 2012
@fyfe fyfe EmberAPI: Store the SQL commands required to create the databases as …
…resources.

Signed-off-by: Andrew Fyfe <andrew@neptune-one.net>
36ccdca
@fyfe fyfe Clean up the database code.
- Remove wrapper functions for CreateCommand and BeginTransaction.

- Access the database connection through a read only property so
  only the Database class can change it.

- Store the SQL commands for initialising the database(s) in resource
  files so they are easier to maintain. This removes the dependency
  for an internet connection when opening the program as it no longer
  needs to download the SQL from http://pcjco.dommel.be

Signed-off-by: Andrew Fyfe <andrew@neptune-one.net>
6c1707f
@fyfe fyfe EmberAPI: Don't connect to the Jobs Log database.
Not sure what this database is used for. Lets see if anything
breaks without it.

Signed-off-by: Andrew Fyfe <andrew@neptune-one.net>
5cc6f78
@fyfe fyfe EmberAPI: Dispose of the connection after it's been closed.
Signed-off-by: Andrew Fyfe <andrew@neptune-one.net>
01fd50a
@bodrick bodrick merged commit 0b14db8 into bodrick:master Sep 7, 2012
@themagnifico themagnifico added a commit to themagnifico/Ember-MM that referenced this pull request Apr 3, 2013
@themagnifico themagnifico Update clsAdvancedSettings.vb
Added Denmark and Italy to ForceTitle available countries (Bug #34, http://ember.purplepig.net/issues/34)
eb9ceb9
@themagnifico themagnifico added a commit to themagnifico/Ember-MM that referenced this pull request Apr 4, 2013
@themagnifico themagnifico Added Denmark and Italy to Forced Title
Added Denmark and Italy to Forced Title dropdown list (Bug #34, http://ember.purplepig.net/issues/34)
bf73b25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment