Use getattr to load config settings so we can easily set a default

demonrik / hdhr2plex

HDHomeRun DVR to Plex Scripts

The Unlicense

14 stars 6 forks source link

Use getattr to load config settings so we can easily set a default #1

Closed natewalck closed 8 years ago

natewalck commented 8 years ago

Hey demonrik! I saw this script and liked the idea, so I thought I'd do a few PRs and try to help out a bit.

This change makes renamedir, renamefile and forceupdate optional in the config and defaults them to False if not present.

natewalck commented 8 years ago

Logic is a bit wonky. It is doing False, even if set. I'll update this.

demonrik commented 8 years ago

Hi Natewalck Just noticed this now - was on the road last week and didn't get any notification. Sorry for the delay :( Great to see some more input.. I'll keep an eye on the project more now :) TIA

natewalck commented 8 years ago

@demonrik No worries! I realized I had to do some more refactoring for this to work, so I abandoned this specific PR. I will try to circle back around, but I'm not sure when exactly.