Compare commits

...
5 changed files with 817 additions and 180 deletions
+10 -5
View File
@@ -28,11 +28,16 @@ class SubmissionListener(commands.Cog, name="Message Listener"):
if proof_message.channel.id == highscores_data["submission_channel_id"]: if proof_message.channel.id == highscores_data["submission_channel_id"]:
embed = submission_message.embeds[0] embed = submission_message.embeds[0]
embed.add_field(name="Score", value=proof_message.content, inline=False) if proof_message.content:
await submission_message.edit( embed.add_field(name="Score", value=proof_message.content, inline=False)
files=[await attch.to_file() for attch in proof_message.attachments], await submission_message.edit(
embed=embed, embed=embed,
) )
if len(proof_message.attachments) > 0:
await submission_message.edit(
files=[await attch.to_file() for attch in proof_message.attachments],
)
await proof_message.delete() await proof_message.delete()
+4 -2
View File
@@ -37,8 +37,10 @@ class PostHighscores(commands.Cog, name="Post Highscores"):
Register with "?register" command.' Register with "?register" command.'
assert channel is not None, await ctx.send(error_response) assert channel is not None, await ctx.send(error_response)
for boss in highscores_config["bosses"]: for boss, categories in highscores_config[ # pylint: disable=unused-variable
message: nextcord.Message = await send_highscore_message(channel, boss["boss"]) "highscore_table"
].items():
message: nextcord.Message = await send_highscore_message(channel, boss)
await message.edit(view=SubmissionButton(self.bot)) await message.edit(view=SubmissionButton(self.bot))
save_highscores_data(highscores_data) save_highscores_data(highscores_data)
+793 -155
View File
@@ -1,205 +1,843 @@
{ {
"bosses": [ "highscore_table": {
{ "The Ambassador": {
"boss": "The Ambassador", "Kill Count": {
"hardmode": true "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Group Kill Count": {
"ascending": false,
"is_time_record": false
},
"Group Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Araxxi": {
"boss": "Araxxi", "Kill Count": {
"hardmode": false "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Arch-Glacor": {
"boss": "Arch-Glacor", "Kill Count": {
"hardmode": true "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Hardmode Kill Count": {
"ascending": false,
"is_time_record": false
},
"Hardmode Kill Time": {
"ascending": true,
"is_time_record": true
},
"Enrage": {
"ascending": false,
"is_time_record": false
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Astellarn": {
"boss": "Astellarn", "Kill Count": {
"hardmode": true "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Group Kill Count": {
"ascending": false,
"is_time_record": false
},
"Group Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "The Barrows Brothers": {
"boss": "The Barrows Brothers", "Kill Count": {
"hardmode": false "ascending": false,
"is_time_record": false
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "The Barrows: Rise of the Six": {
"boss": "The Barrows: Rise of the Six", "Kill Count": {
"hardmode": false "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Beastmaster Durzag": {
"boss": "Beastmaster Durzag", "Kill Count": {
"hardmode": false "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Black Stone Dragon": {
"boss": "Black Stone Dragon", "Kill Count": {
"hardmode": true "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Group Kill Count": {
"ascending": false,
"is_time_record": false
},
"Group Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Chaos Elemental": {
"boss": "Chaos Elemental", "Kill Count": {
"hardmode": false "ascending": false,
"is_time_record": false
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Commander Zilyana": {
"boss": "Commander Zilyana", "Kill Count": {
"hardmode": true "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Hardmode Kill Count": {
"ascending": false,
"is_time_record": false
},
"Hardmode Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Corporeal Beast": {
"boss": "Corporeal Beast", "Kill Count": {
"hardmode": false "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Crassian Leviathan": {
"boss": "Crassian Leviathan", "Kill Count": {
"hardmode": true "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Group Kill Count": {
"ascending": false,
"is_time_record": false
},
"Group Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Croesus": {
"boss": "Croesus", "Kill Count": {
"hardmode": false "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Dagannoth Kings": {
"boss": "Dagannoth Kings", "Kill Count": {
"hardmode": false "ascending": false,
"is_time_record": false
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "General Graardor": {
"boss": "General Graardor", "Kill Count": {
"hardmode": true "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Hardmode Kill Count": {
"ascending": false,
"is_time_record": false
},
"Hardmode Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Giant Mole": {
"boss": "Giant Mole", "Kill Count": {
"hardmode": true "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Hardmode Kill Count": {
"ascending": false,
"is_time_record": false
},
"Hardmode Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Gregorovic": {
"boss": "Gregorovic", "Kill Count": {
"hardmode": true "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Hardmode Kill Count": {
"ascending": false,
"is_time_record": false
},
"Hardmode Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Har-Aken": {
"boss": "Har-Aken", "Kill Count": {
"hardmode": false "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Helwyr": {
"boss": "Helwyr", "Kill Count": {
"hardmode": true "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Hardmode Kill Count": {
"ascending": false,
"is_time_record": false
},
"Hardmode Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Kalphite King": {
"boss": "Kalphite King", "Kill Count": {
"hardmode": false "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Kalphite Queen": {
"boss": "Kalphite Queen", "Kill Count": {
"hardmode": false "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Kerapac": {
"boss": "Kerapac", "Kill Count": {
"hardmode": true "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Hardmode Kill Count": {
"ascending": false,
"is_time_record": false
},
"Hardmode Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "King Black Dragon": {
"boss": "King Black Dragon", "Kill Count": {
"hardmode": false "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Kree'arra": {
"boss": "Kree'arra", "Kill Count": {
"hardmode": true "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Hardmode Kill Count": {
"ascending": false,
"is_time_record": false
},
"Hardmode Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "K'ril Tsutsaroth": {
"boss": "K'ril Tsutsaroth", "Kill Count": {
"hardmode": true "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Hardmode Kill Count": {
"ascending": false,
"is_time_record": false
},
"Hardmode Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Legiones": {
"boss": "Legiones", "Kill Count": {
"hardmode": false "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "The Magister": {
"boss": "The Magister", "Kill Count": {
"hardmode": false "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Masuta the Ascended": {
"boss": "Masuta the Ascended", "Kill Count": {
"hardmode": true "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Group Kill Count": {
"ascending": false,
"is_time_record": false
},
"Group Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Nex": {
"boss": "Nex", "Kill Count": {
"hardmode": false "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Nex - Angel of Death": {
"boss": "Nex - Angel of Death", "Kill Count": {
"hardmode": false "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Queen Black Dragon": {
"boss": "Queen Black Dragon", "Kill Count": {
"hardmode": false "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Raksha": {
"boss": "Raksha", "Solo Kill Count": {
"hardmode": true "ascending": false,
"is_time_record": false
},
"Solo Kill Time": {
"ascending": true,
"is_time_record": true
},
"Duo Kill Count": {
"ascending": false,
"is_time_record": false
},
"Duo Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Rex Matriarchs": {
"boss": "Rex Matriarchs", "Kill Count": {
"hardmode": false "ascending": false,
"is_time_record": false
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "The Sanctum Guardian": {
"boss": "The Sanctum Guardian", "Kill Count": {
"hardmode": true "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Group Kill Count": {
"ascending": false,
"is_time_record": false
},
"Group Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Seiryu the Azure Serpent": {
"boss": "Seiryu the Azure Serpent", "Kill Count": {
"hardmode": true "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Group Kill Count": {
"ascending": false,
"is_time_record": false
},
"Group Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Solak": {
"boss": "Solak", "Kill Count": {
"hardmode": false "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Taraket the Necromancer": {
"boss": "Taraket the Necromancer", "Kill Count": {
"hardmode": true "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Group Kill Count": {
"ascending": false,
"is_time_record": false
},
"Group Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Telos": {
"boss": "Telos", "Kill Count": {
"hardmode": false "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Enrage": {
"ascending": false,
"is_time_record": false
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "The Twin Furies": {
"boss": "The Twin Furies", "Kill Count": {
"hardmode": true "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Hardmode Kill Count": {
"ascending": false,
"is_time_record": false
},
"Hardmode Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "TzKal-Zuk": {
"boss": "TzKal-Zuk", "Kill Count": {
"hardmode": true "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Hardmode Kill Count": {
"ascending": false,
"is_time_record": false
},
"Hardmode Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "TzTok-Jad": {
"boss": "TzTok-Jad", "Kill Count": {
"hardmode": false "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Verak Lith": {
"boss": "Verak Lith", "Kill Count": {
"hardmode": true "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Group Kill Count": {
"ascending": false,
"is_time_record": false
},
"Group Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Vindicta & Gorvek": {
"boss": "Vindicta & Gorvek", "Kill Count": {
"hardmode": true "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Hardmode Kill Count": {
"ascending": false,
"is_time_record": false
},
"Hardmode Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Vorago": {
"boss": "Vorago", "Kill Count": {
"hardmode": true "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Hardmode Kill Count": {
"ascending": false,
"is_time_record": false
},
"Hardmode Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Yakamaru": {
"boss": "Yakamaru", "Kill Count": {
"hardmode": false "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
}, },
{ "Zamorak, Lord of Chaos (solo)": {
"boss": "Zamorak, Lord of Chaos", "Kill Count": {
"hardmode": true "ascending": false,
"is_time_record": false
},
"Kill Time": {
"ascending": true,
"is_time_record": true
},
"Enrage": {
"ascending": false,
"is_time_record": false
},
"Spooned Log": {
"ascending": true,
"is_time_record": false
}
} }
], ,
"categories": { "Zamorak, Lord of Chaos (group)": {
"Kill Count": { "Kill Count": {
"hardmode": true, "ascending": false,
"ascending": false, "is_time_record": false
"is_time_record": false },
}, "Kill Time": {
"Kill Time": { "ascending": true,
"hardmode": true, "is_time_record": true
"ascending": true, },
"is_time_record": true "Enrage": {
}, "ascending": false,
"Spooned Log": { "is_time_record": false
"hardmode": false, }
"ascending": true,
"is_time_record": false
} }
}, },
"highscore_size": 3 "highscore_size": 3
+6 -9
View File
@@ -29,17 +29,14 @@ def open_highscores_data():
else: else:
# create the json object # create the json object
data = {"highscore_channel_id": -1, "verification_channel_id": -1} data = {"highscore_channel_id": -1, "verification_channel_id": -1}
config = open_highscores_config() config: dict = open_highscores_config()
for boss in config["bosses"]: for boss, categories in config["highscore_table"].items():
data[boss["boss"]] = {"message_id": 0} data[boss] = {"message_id": 0}
boss_categories = {} boss_categories = {}
for key, value in config["categories"].items(): for category, category_config in categories.items(): # pylint: disable=unused-variable
rank_list = [] rank_list = []
boss_categories[key] = rank_list boss_categories[category] = rank_list
if boss["hardmode"] and value["hardmode"]: data[boss]["categories"] = boss_categories
rank_list = []
boss_categories["Hardmode " + key] = rank_list
data[boss["boss"]]["categories"] = boss_categories
return data return data
+4 -9
View File
@@ -74,13 +74,8 @@ async def submit_score(
@param user: user who submitted score @param user: user who submitted score
@param score: score to add @param score: score to add
""" """
# TODO update user submission to not have dupe users
# hardmode category, must have different category variable for config access. boss_category_config = highscores_config["highscore_table"][boss_name][category]
if category.find("Hardmode ") != -1:
config_category = category.replace("Hardmode ", "")
else:
config_category = category
# capping to 12 characters because runescape names are 12 characters long. # capping to 12 characters because runescape names are 12 characters long.
# Splitting on pipe. My discord has multiple names on account split by pipe. # Splitting on pipe. My discord has multiple names on account split by pipe.
@@ -91,7 +86,7 @@ async def submit_score(
# create score tuple for either time or int # create score tuple for either time or int
# define sort index # define sort index
if highscores_config["categories"][config_category]["is_time_record"]: if boss_category_config["is_time_record"]:
if score.find(":") == -1: if score.find(":") == -1:
score_seconds = float(score) score_seconds = float(score)
else: else:
@@ -117,7 +112,7 @@ async def submit_score(
# sort users_scores # sort users_scores
users_scores.sort( users_scores.sort(
key=lambda x: x[sort_index], key=lambda x: x[sort_index],
reverse=not highscores_config["categories"][config_category]["ascending"], reverse=not boss_category_config["ascending"],
) )
for index, tup in enumerate(users_scores): for index, tup in enumerate(users_scores):
@@ -133,7 +128,7 @@ async def submit_score(
# sort scores. # sort scores.
scores.sort( scores.sort(
key=lambda x: x[sort_index], key=lambda x: x[sort_index],
reverse=not highscores_config["categories"][config_category]["ascending"], reverse=not boss_category_config["ascending"],
) )
# enforce highscore size # enforce highscore size