mfrasca / bauble.classic

desktop version of Bauble
http://bauble.readthedocs.io
GNU General Public License v2.0
2 stars 1 forks source link

replace prefs.py by standard ConfigParser #59

Closed mfrasca closed 8 years ago

mfrasca commented 9 years ago

Brett wrote: » Consider using ConfigObj since it does validation, type conversion and unicode automatically...the cons are that it adds another dependency and we would have to change the prefs interface throughout bauble«

what about, instead, removing prefs.py and just rely on ConfigParser?

this would not add type validation/conversion, but it would make the code base smaller, without adding any dependency (ConfigParser is part of Python 2.6 and 2.7 and we do not support earlier versions). I also guess it would have a positive impact on maintenance and readability.

we could later on add type conversion and data validation, by subclassing and without changing the way it's used.

mfrasca commented 8 years ago

This issue was moved to Bauble/bauble.classic#183