search

alexames / DeltaBot

GNU General Public License v3.0
65 stars 18 forks source link

subreddit duplication #26

Closed PixelOrange closed 10 years ago

PixelOrange commented 10 years ago

There's two different variables for the subreddit name that are used throughout the code:

self.config.subreddit

self.subreddit

Is there a particular reason this is coded this way or can we clean the code up to only use one or the other?

Also, there appears to be various letter casing bugs that I haven't been able to narrow down. When I launched the program with all lowercase words for my subreddit/username/password, it wouldn't scan comments. If I mirrored what's in reddit, it works fine.

PixelOrange commented 10 years ago

Testing revealed that there are other ways to do this but they aren't any cleaner than this way.