widelands-dev team mailing list archive

[Shevonar <infomh@xxxxxxxxxxx>]

Shevonar has proposed merging lp:~shevonar/widelands-website/bug1202301 into lp:widelands-website.

Requested reviews:
  Widelands Developers (widelands-dev)
Related bugs:
  Bug #1202301 in Widelands Website: "Some maps not accepted by Map upload"
  https://bugs.launchpad.net/widelands-website/+bug/1202301

For more details, see:
https://code.launchpad.net/~shevonar/widelands-website/bug1202301/+merge/175470

This should fix bug 1202301. However, I could not test it because I messed up my widelands-website installation when updating to current trunk. I tested the WidelandsConfigParser and it can now handle the % symbol in config files properly.
-- 
https://code.launchpad.net/~shevonar/widelands-website/bug1202301/+merge/175470
Your team Widelands Developers is requested to review the proposed merge of lp:~shevonar/widelands-website/bug1202301 into lp:widelands-website.

=== modified file 'widelandslib/conf.py'
--- widelandslib/conf.py	2010-11-01 17:52:47 +0000
+++ widelandslib/conf.py	2013-07-18 08:25:30 +0000
@@ -30,6 +30,8 @@
         except AttributeError:
             string = open(fn, "r").read()
 
+        string = string.replace('%', "%%")
+
         try:
             self.readfp(cStringIO.StringIO(string))
         except MissingSectionHeaderError: