Skip to content

Commit f3cdae3

Browse files
committed
changing formatting and variable names.
1 parent 798af92 commit f3cdae3

File tree

6 files changed

+743
-531
lines changed

6 files changed

+743
-531
lines changed

.gitignore

+3
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,7 @@
11
auth/
22
logs/
3+
ENV/
4+
pylint_results/
5+
src/
36
__pycache__/
47
*.pyc

db_wrapper.py

+23-23
Original file line numberDiff line numberDiff line change
@@ -5,27 +5,27 @@
55

66
class DB_Wrapper:
77

8-
def __init__(self, user, passwd, host, db):
9-
self.user = user
10-
self.passwd = passwd
11-
self.host = host
12-
self.db = db
13-
return
8+
def __init__(self, user, passwd, host, db):
9+
self.user = user
10+
self.passwd = passwd
11+
self.host = host
12+
self.db = db
13+
return
1414

15-
async def execute(self, client, member, sql_command, notify):
16-
try:
17-
db_connection = MySQLdb.connect(user=self.user, passwd=self.passwd, host=self.host, db=self.db, charset="utf8mb4")
18-
cursor = db_connection.cursor()
19-
cursor.execute(sql_command)
20-
result = cursor.fetchall()
21-
db_connection.commit()
22-
cursor.close()
23-
db_connection.close()
24-
except Exception as e:
25-
time = str(datetime.now())
26-
print(time+": failed to properly execute DB command: "+sql_command)
27-
print("The exception that occurred was: "+str(e))
28-
if notify:
29-
await client.send_message(member, "I had some trouble with a database query. Please ask Chish#2578 to check the logs at time "+time)
30-
return
31-
return result
15+
async def execute(self, client, member, sql_command, notify):
16+
try:
17+
db_connection = MySQLdb.connect(user=self.user, passwd=self.passwd, host=self.host, db=self.db, charset="utf8mb4")
18+
cursor = db_connection.cursor()
19+
cursor.execute(sql_command)
20+
result = cursor.fetchall()
21+
db_connection.commit()
22+
cursor.close()
23+
db_connection.close()
24+
except Exception as e:
25+
time = str(datetime.now())
26+
print(time+": failed to properly execute DB command: "+sql_command)
27+
print("The exception that occurred was: "+str(e))
28+
if notify:
29+
await client.send_message(member, "I had some trouble with a database query. Please ask Chish#2578 to check the logs at time "+time)
30+
return
31+
return result

pylint_results/sfcjbot.txt

+164
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,164 @@
1+
************* Module sfcjbot
2+
W: 36, 0: TODO find a good pattern to clean all these commands all these up (including after the next comment) (fixme)
3+
W:433, 0: TODO hopefully this isn't the only record of that game in the table, huh? (fixme)
4+
W:446, 0: TODO this doesn't have a way of telling if anything actually happened yet. (fixme)
5+
W:454, 0: TODO First we make a list of all users that were queued for that game on that server (so we can notify them they were dequeued) (fixme)
6+
W:458, 0: TODO consider sending a message to the server or users about this. (fixme)
7+
W:468, 0: FIXME check that the second argument is actually a valid column. (fixme)
8+
W:558, 0: FIXME spaces should not be a separator here. (fixme)
9+
C: 23, 0: Line too long (142/100) (line-too-long)
10+
C: 26, 0: Line too long (143/100) (line-too-long)
11+
C: 28, 0: Line too long (242/100) (line-too-long)
12+
C: 30, 0: Line too long (150/100) (line-too-long)
13+
C: 33, 0: Line too long (160/100) (line-too-long)
14+
C: 36, 0: Line too long (110/100) (line-too-long)
15+
C: 40, 0: Line too long (148/100) (line-too-long)
16+
C: 43, 0: Line too long (167/100) (line-too-long)
17+
C: 51, 0: Line too long (147/100) (line-too-long)
18+
C: 55, 0: Line too long (156/100) (line-too-long)
19+
C: 62, 0: Line too long (190/100) (line-too-long)
20+
C: 66, 0: Line too long (226/100) (line-too-long)
21+
C: 69, 0: Line too long (118/100) (line-too-long)
22+
C: 73, 0: Line too long (118/100) (line-too-long)
23+
C: 77, 0: Line too long (109/100) (line-too-long)
24+
C: 85, 0: Line too long (114/100) (line-too-long)
25+
C:120, 0: Line too long (132/100) (line-too-long)
26+
C:144, 0: Line too long (185/100) (line-too-long)
27+
C:148, 0: Line too long (199/100) (line-too-long)
28+
C:149, 0: Line too long (115/100) (line-too-long)
29+
C:152, 0: Line too long (130/100) (line-too-long)
30+
C:155, 0: Line too long (233/100) (line-too-long)
31+
C:156, 0: Line too long (127/100) (line-too-long)
32+
C:159, 0: Line too long (174/100) (line-too-long)
33+
C:161, 0: Line too long (168/100) (line-too-long)
34+
C:162, 0: Line too long (152/100) (line-too-long)
35+
C:164, 0: Line too long (125/100) (line-too-long)
36+
C:173, 0: Line too long (128/100) (line-too-long)
37+
C:176, 0: Line too long (125/100) (line-too-long)
38+
C:178, 0: Line too long (106/100) (line-too-long)
39+
C:181, 0: Line too long (142/100) (line-too-long)
40+
C:185, 0: Line too long (123/100) (line-too-long)
41+
C:196, 0: Line too long (133/100) (line-too-long)
42+
C:197, 0: Line too long (108/100) (line-too-long)
43+
C:198, 0: Line too long (166/100) (line-too-long)
44+
C:199, 0: Line too long (116/100) (line-too-long)
45+
C:205, 0: Line too long (166/100) (line-too-long)
46+
C:206, 0: Line too long (120/100) (line-too-long)
47+
C:213, 0: Line too long (169/100) (line-too-long)
48+
C:216, 0: Line too long (186/100) (line-too-long)
49+
C:220, 0: Line too long (169/100) (line-too-long)
50+
C:223, 0: Line too long (158/100) (line-too-long)
51+
C:232, 0: Line too long (193/100) (line-too-long)
52+
C:239, 0: Line too long (102/100) (line-too-long)
53+
C:242, 0: Line too long (112/100) (line-too-long)
54+
C:243, 0: Line too long (108/100) (line-too-long)
55+
C:262, 0: Line too long (104/100) (line-too-long)
56+
C:265, 0: Line too long (178/100) (line-too-long)
57+
C:271, 0: Line too long (138/100) (line-too-long)
58+
C:272, 0: Line too long (114/100) (line-too-long)
59+
C:278, 0: Line too long (147/100) (line-too-long)
60+
C:280, 0: Line too long (126/100) (line-too-long)
61+
C:284, 0: Line too long (155/100) (line-too-long)
62+
C:286, 0: Line too long (122/100) (line-too-long)
63+
C:291, 0: Line too long (104/100) (line-too-long)
64+
C:292, 0: Line too long (145/100) (line-too-long)
65+
C:295, 0: Line too long (103/100) (line-too-long)
66+
C:297, 0: Line too long (103/100) (line-too-long)
67+
C:309, 0: Line too long (384/100) (line-too-long)
68+
C:311, 0: Line too long (137/100) (line-too-long)
69+
C:313, 0: Line too long (109/100) (line-too-long)
70+
C:324, 0: Line too long (109/100) (line-too-long)
71+
C:327, 0: Line too long (136/100) (line-too-long)
72+
C:329, 0: Line too long (112/100) (line-too-long)
73+
C:337, 0: Line too long (167/100) (line-too-long)
74+
C:344, 0: Line too long (260/100) (line-too-long)
75+
C:345, 0: Line too long (112/100) (line-too-long)
76+
C:360, 0: Line too long (106/100) (line-too-long)
77+
C:367, 0: Line too long (110/100) (line-too-long)
78+
C:368, 0: Line too long (101/100) (line-too-long)
79+
C:371, 0: Line too long (155/100) (line-too-long)
80+
C:372, 0: Line too long (155/100) (line-too-long)
81+
C:375, 0: Line too long (130/100) (line-too-long)
82+
C:376, 0: Line too long (142/100) (line-too-long)
83+
C:386, 0: Line too long (102/100) (line-too-long)
84+
C:387, 0: Line too long (110/100) (line-too-long)
85+
C:400, 0: Line too long (115/100) (line-too-long)
86+
C:401, 0: Line too long (115/100) (line-too-long)
87+
C:402, 0: Line too long (153/100) (line-too-long)
88+
C:414, 0: Line too long (211/100) (line-too-long)
89+
C:419, 0: Line too long (200/100) (line-too-long)
90+
C:420, 0: Line too long (103/100) (line-too-long)
91+
C:423, 0: Line too long (103/100) (line-too-long)
92+
C:427, 0: Line too long (149/100) (line-too-long)
93+
C:435, 0: Line too long (125/100) (line-too-long)
94+
C:437, 0: Line too long (121/100) (line-too-long)
95+
C:438, 0: Line too long (128/100) (line-too-long)
96+
C:440, 0: Line too long (128/100) (line-too-long)
97+
C:448, 0: Line too long (174/100) (line-too-long)
98+
C:450, 0: Line too long (110/100) (line-too-long)
99+
C:451, 0: Line too long (117/100) (line-too-long)
100+
C:454, 0: Line too long (137/100) (line-too-long)
101+
C:455, 0: Line too long (118/100) (line-too-long)
102+
C:457, 0: Line too long (127/100) (line-too-long)
103+
C:460, 0: Line too long (126/100) (line-too-long)
104+
C:471, 0: Line too long (134/100) (line-too-long)
105+
C:475, 0: Line too long (126/100) (line-too-long)
106+
C:476, 0: Line too long (155/100) (line-too-long)
107+
C:478, 0: Line too long (121/100) (line-too-long)
108+
C:479, 0: Line too long (114/100) (line-too-long)
109+
C:498, 0: Line too long (152/100) (line-too-long)
110+
C:501, 0: Line too long (102/100) (line-too-long)
111+
C:512, 0: Line too long (106/100) (line-too-long)
112+
C:513, 0: Line too long (123/100) (line-too-long)
113+
C:514, 0: Line too long (123/100) (line-too-long)
114+
C:519, 0: Line too long (154/100) (line-too-long)
115+
C:521, 0: Line too long (107/100) (line-too-long)
116+
C:528, 0: Line too long (155/100) (line-too-long)
117+
C:552, 0: Line too long (164/100) (line-too-long)
118+
C:562, 0: Line too long (231/100) (line-too-long)
119+
C:566, 0: Line too long (207/100) (line-too-long)
120+
C:567, 0: Line too long (117/100) (line-too-long)
121+
C:568, 0: Line too long (109/100) (line-too-long)
122+
C:570, 0: Line too long (106/100) (line-too-long)
123+
C:572, 0: Line too long (109/100) (line-too-long)
124+
C: 1, 0: Missing module docstring (missing-docstring)
125+
E: 5, 0: Unable to import 'MySQLdb' (import-error)
126+
E: 10, 0: Possible unbalanced tuple unpacking with sequence: left side has 4 label(s), right side has 0 value(s) (unbalanced-tuple-unpacking)
127+
C: 11, 0: Invalid constant name "client" (invalid-name)
128+
C: 12, 0: Invalid constant name "next_marvel_event_date" (invalid-name)
129+
R: 16, 0: Too many return statements (24/6) (too-many-return-statements)
130+
R: 16, 0: Too many branches (30/12) (too-many-branches)
131+
R: 16, 0: Too many statements (67/50) (too-many-statements)
132+
C:209, 6: Invalid function name "add_server_specific_tables_if_necessary" (invalid-name)
133+
W:217, 8: Unused variable 'table_created' (unused-variable)
134+
C:260, 6: Invalid function name "get_discord_and_secondary_using_challonge" (invalid-name)
135+
C:271, 4: Invalid variable name "getDiscordAndSecondaryQuery" (invalid-name)
136+
C:272, 4: Invalid variable name "discordAndSecondaryTuple" (invalid-name)
137+
C:273, 4: Invalid variable name "fallbackChallongeUsername" (invalid-name)
138+
C:278,16: Invalid variable name "newIDquery" (invalid-name)
139+
C:280,16: Invalid variable name "discordAndSecondaryTuple" (invalid-name)
140+
C:281,16: Invalid variable name "fallbackChallongeUsername" (invalid-name)
141+
C:344,12: Invalid variable name "get_players_marked_here_not_requester" (invalid-name)
142+
R:341, 4: Too many nested blocks (6/5) (too-many-nested-blocks)
143+
W:397,12: Redefining name 'match' from outer scope (line 301) (redefined-outer-name)
144+
W:436, 8: Unused variable 'result' (unused-variable)
145+
W:455, 8: Redefining name 'unqueue' from outer scope (line 556) (redefined-outer-name)
146+
W:448, 8: Redefining name 'removegame' from outer scope (line 444) (redefined-outer-name)
147+
W:449, 8: Unused variable 'result' (unused-variable)
148+
W:510,20: Redefining name 'match' from outer scope (line 301) (redefined-outer-name)
149+
C:577, 0: Missing function docstring (missing-docstring)
150+
C:582, 0: Invalid constant name "f" (invalid-name)
151+
C:583, 0: Invalid constant name "token" (invalid-name)
152+
C:585, 0: Invalid constant name "f" (invalid-name)
153+
C:589, 0: Invalid constant name "challonge" (invalid-name)
154+
E:589,12: Module 'challonge' has no 'Account' member (no-member)
155+
C:590, 0: Invalid constant name "f" (invalid-name)
156+
W: 1, 0: Unused import asyncio (unused-import)
157+
W: 5, 0: Unused import MySQLdb (unused-import)
158+
C: 3, 0: standard import "from datetime import datetime" should be placed before "import challonge" (wrong-import-order)
159+
C: 6, 0: standard import "import random" should be placed before "import challonge" (wrong-import-order)
160+
C: 7, 0: standard import "from sys import argv" should be placed before "import challonge" (wrong-import-order)
161+
162+
------------------------------------------------------------------
163+
Your code has been rated at 6.05/10 (previous run: 5.94/10, +0.12)
164+

0 commit comments

Comments
 (0)