Config related cleanup #503

merged 3 commits into from Nov 15, 2012


None yet
2 participants

thezoggy commented Oct 6, 2012

No description provided.

thezoggy added some commits Oct 6, 2012

@thezoggy thezoggy Moved existing CheckSection to their perspective variable settings to…
… see what was missed/messed up. Corrected PLEX to Plex. Did PEP8 (whitespace/unusedvariable) fixes as well.
@thezoggy thezoggy Added missing CheckSection: BTN, NZBs, NZBsRUS, NZBMatrix, TVTORRENTS…
…, Womble, Notifo, Boxcar, Pushover, Libnotify, Trakt, Newznab, EZRSS, GUI

Moved the metadata related chunk of code up with the rest of the 'General' config settings. This way now people can just add at the bottom of the list rather than it being buried in the middle.
@thezoggy thezoggy Add UTF-8 encoding to the config.ini. This prevents people destroying…
… their config when setting a field with a unicode value. (easy way to test, go set your tvdownload dir to a unicode folder.. watch you end up with a 0byte and hung python on it crashing when writing.. this solves that) -- note that related logging messages do need to be updated to reflect this change.. but we can deal with that later.

@midgetspy midgetspy added a commit that referenced this pull request Nov 15, 2012

@midgetspy midgetspy Merge pull request #503 from thezoggy/dev-config_cleanup
Config related cleanup

@midgetspy midgetspy merged commit 7e89a6b into midgetspy:development Nov 15, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment