Major changes

Author: PokestarFan

.gitignore

@@ -101,17 +101,14 @@ ENV/
 .mypy_cache/
 
 #my login
-login.py
-Utlities/login.py
+*login.py
 
 #pycharm
 .idea/
 
 #files used by program
 blockusers.txt
-posts_written_to.txt
-writes.txt
-replies.txt
 
 #logs
 *.log
+logs/

delete.py

@@ -1,15 +1,17 @@
 import praw
-from login import reddit
+from modules.logger import setup_logger
+from modules.login import reddit
+import logging
 
+logger = setup_logger('user_removed_comments')
 
-def main(limit, count=0):
-	for comment in r.redditor(str(user)).comments.new(limit=limit):
-		comment.delete()
-		count = count + 1
-		print('Finished comment #'+str(count))
-		
-if __name__ == '__main__':
-	limit = input('How many recent comments to delete? Type None for all comments')
-	count = 0
-	main(limit, count)
-	print('Complete')
+for item in reddit.inbox.stream():
+	logger.debug('On item {}'.format(str(item)))
+	try:
+		if item.body.lower() == 'delete':
+			item.parent.delete()
+			logging.info('Comment {} removed'.format(str(item.parent)))
+			item.reply('The top level post has been removed.')
+	except:
+		logging.debug('Item {} skipped'.format(str(item)))
+		pass

deleteallcomments.py

@@ -0,0 +1,19 @@
+import praw
+from modules.logger import setup_logger
+from modules.login import reddit
+import logging
+
+logger = setup_logger('remallcomments')
+
+
+def main(count=0):
+	for comment in r.redditor(str(r.user.me())).comments.new(limit=None):
+		comment.delete()
+		logging.info('Finished comment #'+str(count)+', id {}'.format(str(comment)))
+		
+if __name__ == '__main__':
+	while True:
+		try: 
+			main()
+		except:
+			logging.error('Error!', exc_info=True)

duplicate.py

@@ -1,70 +1,67 @@
-from datetime import datetime
+from modules.logger import setup_logger
 import logging
+from datetime import datetime
 import praw
 import prawcore
-import sys
-from login import reddit
-
-
-file_handler = logging.FileHandler(filename='duplicates_{}.log'.format(datetime.now().strftime('%m_%d_%y')))
-stdout_handler = logging.StreamHandler(sys.stdout)
-handlers = [file_handler, stdout_handler]
-
-logging.basicConfig(
-    level=logging.INFO, 
-    format='[%(asctime)s] {%(filename)s:%(lineno)d} %(levelname)s - %(message)s',
-    handlers=handlers
-)
+from modules.login import reddit
 
-logger = logging.getLogger(__name__)
+logger = setup_logger('duplicates')
 
 def action():
     for sub_id in reddit.subreddit('all').stream.submissions():
-        logging.debug('Starting submission {}'.format(sub_id))
-        blockeduser = 0
-        duplicates = []
-        submission = praw.models.Submission(reddit, id = sub_id)
-        with open('blockusers.txt','r') as newfile:
-            for line in newfile.readlines():
-                line = line.strip('\n')
-                if str(submission.author) == line or 'bot' in str(submission.author).lower():
-                    blockeduser = 1
-                    logger.debug('User {}\'s submission {} was blocked from posting'.format(str(submission.author),str(sub_id)))
-                else:
-                    pass
-        if blockeduser == 0:
-            for duplicate in submission.duplicates():   
-                dup_sub = praw.models.Submission(reddit, id = duplicate)
-                if 'imagesof' not in str(dup_sub.subreddit).lower() and 'auto' not in str(dup_sub.subreddit).lower() and 'bot' not in str(dup_sub.author).lower():
-                    time = dup_sub.created
-                    time = str(datetime.fromtimestamp(time))
-                    author = str(dup_sub.author)
-                    if str(submission.author) == author:
-                        author = author + '[author of both threads]'
-                    duplicates.append({'title':str(dup_sub.title), 'subreddit':str(dup_sub.subreddit), 'link':'https://www.reddit.com'+str(dup_sub.permalink), 'time':str(time), 'author':author, 'karma': str(dup_sub.score)})
-                    if len(duplicates) > 0:
-                        message = 'Here is a list of threads in other subreddits about the same content:\n'
-                        for dup in duplicates:
-                            message = str(message + '\n * [{}]({}) on /r/{} with {} karma (created at {} by {})').format(dup['title'], dup['link'], dup['subreddit'], dup['karma'],dup['time'], dup['author'])
-                        message = message + '\n\n ---- \n\n ^^I ^^am ^^a ^^bot  ^^[FAQ](https://www.reddit.com/r/DuplicatesBot/wiki/index)-[Code](https://github.com/PokestarFan/DuplicateBot)-[Bugs](https://www.reddit.com/r/DuplicatesBot/comments/6ypgmx/bugs_and_problems/)-[Suggestions](https://www.reddit.com/r/DuplicatesBot/comments/6ypg85/suggestion_for_duplicatesbot/)-[Block](https://www.reddit.com/r/DuplicatesBot/wiki/index#wiki_block_bot_from_tagging_on_your_posts)'
-            try:
-                submission.reply(message)
-                logger.info('Message posted on {}'.format(sub_id))
-                logger.debug('Message: {}'.format(message))
-                message = ''
-            except(praw.exceptions.APIException, UnboundLocalError)as e:
-                logger.info('Submission {} has been skipped due to missing text'.format(sub_id))
+        try:
+            logging.debug('Starting submission {}'.format(sub_id))
+            blockeduser = 0
+            duplicates = []
+            submission = praw.models.Submission(reddit, id = sub_id)
+            with open('blockusers.txt','r') as newfile:
+                for line in newfile.readlines():
+                    line = line.strip('\n')
+                    if str(submission.author) == line or 'bot' in str(submission.author).lower():
+                        blockeduser = 1
+                        logger.debug('User {}\'s submission {} was blocked from posting'.format(str(submission.author),str(sub_id)))
+                    else:
+                        pass
+            if blockeduser == 0:
+                for duplicate in submission.duplicates():   
+                    dup_sub = praw.models.Submission(reddit, id = duplicate)
+                    if 'imagesof' not in str(dup_sub.subreddit).lower() and 'auto' not in str(dup_sub.subreddit).lower() and 'bot' not in str(dup_sub.author).lower() and 'mistyfront' not in str(dup_sub.subreddit).lower() and 'unitedfederation' not in str(dup_sub.subreddit).lower():
+                        time = dup_sub.created
+                        time = str(datetime.fromtimestamp(time))
+                        author = str(dup_sub.author)
+                        if str(submission.author) == author:
+                            author = author + '[author of both threads]'
+                        duplicates.append({'title':str(dup_sub.title), 'subreddit':str(dup_sub.subreddit), 'link':'https://www.reddit.com'+str(dup_sub.permalink), 'time':str(time), 'author':author, 'karma': str(dup_sub.score)})
+                        if len(duplicates) > 0:
+                            message = 'Here is a list of threads in other subreddits about the same content:\n'
+                            for dup in duplicates:
+                                message = str(message + '\n * [{}]({}) on /r/{} with {} karma (created at {} by {})').format(dup['title'], dup['link'], dup['subreddit'], dup['karma'],dup['time'], dup['author'])
+                            message = message + '\n\n ---- \n\n ^^I ^^am ^^a ^^bot  ^^[FAQ](https://www.reddit.com/r/DuplicatesBot/wiki/index)-[Code](https://github.com/PokestarFan/DuplicateBot)-[Bugs](https://www.reddit.com/r/DuplicatesBot/comments/6ypgmx/bugs_and_problems/)-[Suggestions](https://www.reddit.com/r/DuplicatesBot/comments/6ypg85/suggestion_for_duplicatesbot/)-[Block](https://www.reddit.com/r/DuplicatesBot/wiki/index#wiki_block_bot_from_tagging_on_your_posts)'
                 try:
+                    submission.reply(message)
+                    logger.info('Message posted on {}'.format(sub_id))
                     logger.debug('Message: {}'.format(message))
+                    message = ''
+                except(praw.exceptions.APIException, UnboundLocalError)as e:
+                    logger.debug('Submission {} has been skipped due to missing text'.format(sub_id))
+                    message = ''
+                except(prawcore.exceptions.Forbidden):
+                    logger.debug('You are blocked on /r/{}'.format(str(submission.subreddit)))
+                    message = ''
+                except(AssertionError):
+                    logger.debug('Assertion Error occured! Printing message and traceback.')
+                    logger.debug(message + str(len(message)), exc_info=True)
+                    message = ''
+                except(KeyboardInterrupt):
+                    raise KeyboardInterrupt
                 except:
-                    logger.debug('Message output for exception ')
-                message = ''
-            except(prawcore.exceptions.Forbidden):
-                logger.info('You are blocked on /r/{}'.format(str(submission.subreddit)))
-                message = ''
-            except:
-                logger.error('Error occured!', exc_info=True)
-                message = ''
+                    logger.error('Error occured!', exc_info=True)
+                    message = ''
+        except(KeyboardInterrupt):
+            raise KeyboardInterrupt
+        except:
+            logger.error('Error on submission {} occured.'.format(str(sub_id)), exc_info=True)
+        
 
 
 if __name__ == '__main__':

goodbadbot.py

@@ -9,7 +9,11 @@
 
 import time
 import praw
-from login import reddit as r
+from modules.logger import setup_logger
+from modules.login import reddit as r
+import logging
+
+logger = setup_logger('low_score_remover')
 
 ###########################
 # definitions                     
@@ -23,51 +27,37 @@
 
 #defining the searching and removal
 def action():
-    global submission_titles #making global submission title list
     global comment_submissions #making glocal comment submission title list
     comment_submissions = []
     submission_titles = []
     submission_number = 0
     comment_number = 0
     current_submission_number = 0
     current_comment_number = 0
-    print('Searching...')
+    logging.info('Searching...')
     user = r.user.me()
-    for submission in r.redditor(str(user)).submissions.new(limit=None): #scanning all submissions by reddit user without limit
-        if submission.score < setvalue: #if statement checking the submission score < 0
-           submission.delete() #deleting the submission
-           submission_titles.append(submission.title) #adding the submission title to the list
     for comment in r.redditor(str(user)).comments.new(limit=None): #scanning all comments by reddit user without limit
+        logging.debug('On comment {}'.format(str(comment)))
         if comment.score < setvalue: #if statement checking the comment score < 0 
            comment.delete() #deleteing the comment if < 0
+           logging.info('Removed comment {}'.format(str(comment)))
            comment_submissions.append(comment.submission.title) #adding the comment's original submission title to the list
 
 #defining the outputs
 def print_results():
-    print('Search Complete')
-    print('--------------------------------------------------')
-    #if statement. True if there is 1 or more submissions removed. 
-    if len(submission_titles) > 0:
-        #printing true results
-        print('Removed ' + str(len(submission_titles)) + ' submission(s).')
-        print('Submission title(s) include: ')
-        print(*submission_titles, sep='\n') #printing submission titles with line breaks
-        print('--------------------------------------------------')
-    else:
-        #printing false results
-        print('No submissions removed.')
-        print('--------------------------------------------------')
+    logging.info('Search Complete')
+    logging.info('--------------------------------------------------')
     #if statement. True if there is 1 or more comments removed. 
     if len(comment_submissions) > 0:
-        #printing true results
-        print('Removed ' + str(len(comment_submissions)) + ' comment(s).')
-        print('Comments were under the following submissions: ')
-        print(*comment_submissions, sep='\n') #printing comment's submission's titles with line breaks
-        print('--------------------------------------------------')
+        #logging.infoing true results
+        logging.info('Removed ' + str(len(comment_submissions)) + ' comment(s).')
+        logging.info('Comments were under the following submissions: ')
+        logging.info(*comment_submissions, sep='\n') #logging.infoing comment's submission's titles with line breaks
+        logging.info('--------------------------------------------------')
     else:
-        #printing false results
-        print('No comments removed.')
-        print('--------------------------------------------------')
+        #logging.infoing false results
+        logging.info('No comments removed.')
+        logging.info('--------------------------------------------------')
 
 ###########################
 # code execution
@@ -77,7 +67,6 @@ def print_results():
 def main():
     action()
     print_results()
-	
+    
 while True:
-    main()
-    time.sleep(600)
+    main()

lowpostremover.py

@@ -0,0 +1,19 @@
+import logging
+from datetime import datetime
+import sys
+
+
+def setup_logger(name):
+	file_handler = logging.FileHandler(filename='logs/{}_{}.log'.format(name, datetime.now().strftime('%m_%d_%y')))
+	stdout_handler = logging.StreamHandler(sys.stdout)
+	handlers = [file_handler, stdout_handler]
+	
+	logging.basicConfig(
+		level=logging.INFO, 
+		format='[%(asctime)s] {%(filename)s:%(lineno)d} (%(funcName)s) %(levelname)s - %(message)s',
+		handlers=handlers
+	)
+
+	logger = logging.getLogger(__name__)
+	return logger
+	

modules/logger.py

@@ -0,0 +1,3 @@
+@echo off
+py duplicate.py
+run.bat

run.bat

@@ -0,0 +1,4 @@
+@echo off
+SET /P somevar= Username:
+echo %somevar%>>blockusers.txt
+updateblockedusers.bat