Merge lp:~kolmis/wxbanker/windows-fixes into lp:wxbanker

[Karel Kolman]

[Karel Kolman (kolmis)]

A few bugfixes for Windows systems.

[Michael Rooney (mrooney)]

Thanks for fixing that %APPDATA% issue that I wasn't aware of, as well as using a FileConfig, that is probably a little cleaner. Also thanks for fixing 23 of the unit tests that were having IO problems :)

review: Approve

=== modified file 'controller.py'
--- controller.py	2009-09-19 07:36:09 +0000
+++ controller.py	2009-10-03 21:15:21 +0000
@@ -166,6 +166,11 @@
 Traceback (most recent call last):
   ...
 InvalidTransactionException: Transaction does not exist in account 'My Renamed Account'
+
+# cleanUp
+>>> controller.Close()
+>>> Publisher.unsubAll()
+>>> os.remove("test.db")
 """
 
 from persistentstore import PersistentStore
@@ -218,7 +223,7 @@
             self.MigrateIfFound(oldBankPath, fileservice.getDataFilePath(self.DB_NAME))
             
         # Okay, now our files are in happy locations, let's go!
-        config = wx.Config(localFilename=configPath)
+        config = wx.FileConfig(localFilename=configPath)
         wx.Config.Set(config)
         if not config.HasEntry("SIZE_X"):
             config.WriteInt("SIZE_X", 800)
@@ -271,7 +276,7 @@
         return model
 
     def Close(self, model=None):
-        if model is None: models = self.Models
+        if model is None: models = self.Models[:]
         else: models = [model]
 
         for model in models:
 
=== modified file 'fileservice.py'
--- fileservice.py	2009-09-18 07:27:13 +0000
+++ fileservice.py	2009-10-03 21:15:21 +0000
@@ -38,7 +38,11 @@
             pathdir = os.path.join(os.environ["HOME"], ".wxbanker")
         elif "APPDATA" in os.environ:
             # Windows!
-            pathdir = os.path.join(os.environ["APPDATA"], "wxbanker")
+            appdata = os.environ["APPDATA"]
+            if type(appdata) != unicode:
+              import locale
+              appdata = unicode(appdata, locale.getlocale()[1])
+            pathdir = os.path.join(appdata, "wxbanker")
         else:
             raise Exception("Unable to find a home for user data!")