mirror of
https://github.com/EQEmu/Server.git
synced 2025-12-13 18:51:29 +00:00
[Cleanup] results variable is assigned but never used in SaveCharacterCreate() (#3180)
# Notes - This was unnecessary since `QueryDatabase()` runs regardless.
This commit is contained in:
Alex King
GitHub
parent
d1b7c675f9
commit
3386d13d2d
@ -449,7 +449,8 @@ bool Database::DeleteCharacter(char *character_name)
|
|||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
bool Database::SaveCharacterCreate(uint32 character_id, uint32 account_id, PlayerProfile_Struct* pp){
|
bool Database::SaveCharacterCreate(uint32 character_id, uint32 account_id, PlayerProfile_Struct *pp)
|
||||||
|
{
|
||||||
std::string query = StringFormat(
|
std::string query = StringFormat(
|
||||||
"REPLACE INTO `character_data` ("
|
"REPLACE INTO `character_data` ("
|
||||||
"id,"
|
"id,"
|
||||||
@ -634,101 +635,102 @@ bool Database::SaveCharacterCreate(uint32 character_id, uint32 account_id, Playe
|
|||||||
"%u," // guild_auto_consent
|
"%u," // guild_auto_consent
|
||||||
"%u" // RestTimer
|
"%u" // RestTimer
|
||||||
")",
|
")",
|
||||||
character_id, // " id, "
|
character_id, // " id, "
|
||||||
account_id, // " account_id, "
|
account_id, // " account_id, "
|
||||||
Strings::Escape(pp->name).c_str(), // " `name`, "
|
Strings::Escape(pp->name).c_str(), // " `name`, "
|
||||||
Strings::Escape(pp->last_name).c_str(), // " last_name, "
|
Strings::Escape(pp->last_name).c_str(), // " last_name, "
|
||||||
pp->gender, // " gender, "
|
pp->gender, // " gender, "
|
||||||
pp->race, // " race, "
|
pp->race, // " race, "
|
||||||
pp->class_, // " class, "
|
pp->class_, // " class, "
|
||||||
pp->level, // " `level`, "
|
pp->level, // " `level`, "
|
||||||
pp->deity, // " deity, "
|
pp->deity, // " deity, "
|
||||||
pp->birthday, // " birthday, "
|
pp->birthday, // " birthday, "
|
||||||
pp->lastlogin, // " last_login, "
|
pp->lastlogin, // " last_login, "
|
||||||
pp->timePlayedMin, // " time_played, "
|
pp->timePlayedMin, // " time_played, "
|
||||||
pp->pvp, // " pvp_status, "
|
pp->pvp, // " pvp_status, "
|
||||||
pp->level2, // " level2, "
|
pp->level2, // " level2, "
|
||||||
pp->anon, // " anon, "
|
pp->anon, // " anon, "
|
||||||
pp->gm, // " gm, "
|
pp->gm, // " gm, "
|
||||||
pp->intoxication, // " intoxication, "
|
pp->intoxication, // " intoxication, "
|
||||||
pp->haircolor, // " hair_color, "
|
pp->haircolor, // " hair_color, "
|
||||||
pp->beardcolor, // " beard_color, "
|
pp->beardcolor, // " beard_color, "
|
||||||
pp->eyecolor1, // " eye_color_1, "
|
pp->eyecolor1, // " eye_color_1, "
|
||||||
pp->eyecolor2, // " eye_color_2, "
|
pp->eyecolor2, // " eye_color_2, "
|
||||||
pp->hairstyle, // " hair_style, "
|
pp->hairstyle, // " hair_style, "
|
||||||
pp->beard, // " beard, "
|
pp->beard, // " beard, "
|
||||||
pp->ability_time_seconds, // " ability_time_seconds, "
|
pp->ability_time_seconds, // " ability_time_seconds, "
|
||||||
pp->ability_number, // " ability_number, "
|
pp->ability_number, // " ability_number, "
|
||||||
pp->ability_time_minutes, // " ability_time_minutes, "
|
pp->ability_time_minutes, // " ability_time_minutes, "
|
||||||
pp->ability_time_hours, // " ability_time_hours, "
|
pp->ability_time_hours, // " ability_time_hours, "
|
||||||
Strings::Escape(pp->title).c_str(), // " title, "
|
Strings::Escape(pp->title).c_str(), // " title, "
|
||||||
Strings::Escape(pp->suffix).c_str(), // " suffix, "
|
Strings::Escape(pp->suffix).c_str(), // " suffix, "
|
||||||
pp->exp, // " exp, "
|
pp->exp, // " exp, "
|
||||||
pp->points, // " points, "
|
pp->points, // " points, "
|
||||||
pp->mana, // " mana, "
|
pp->mana, // " mana, "
|
||||||
pp->cur_hp, // " cur_hp, "
|
pp->cur_hp, // " cur_hp, "
|
||||||
pp->STR, // " str, "
|
pp->STR, // " str, "
|
||||||
pp->STA, // " sta, "
|
pp->STA, // " sta, "
|
||||||
pp->CHA, // " cha, "
|
pp->CHA, // " cha, "
|
||||||
pp->DEX, // " dex, "
|
pp->DEX, // " dex, "
|
||||||
pp->INT, // " `int`, "
|
pp->INT, // " `int`, "
|
||||||
pp->AGI, // " agi, "
|
pp->AGI, // " agi, "
|
||||||
pp->WIS, // " wis, "
|
pp->WIS, // " wis, "
|
||||||
pp->face, // " face, "
|
pp->face, // " face, "
|
||||||
pp->y, // " y, "
|
pp->y, // " y, "
|
||||||
pp->x, // " x, "
|
pp->x, // " x, "
|
||||||
pp->z, // " z, "
|
pp->z, // " z, "
|
||||||
pp->heading, // " heading, "
|
pp->heading, // " heading, "
|
||||||
pp->pvp2, // " pvp2, "
|
pp->pvp2, // " pvp2, "
|
||||||
pp->pvptype, // " pvp_type, "
|
pp->pvptype, // " pvp_type, "
|
||||||
pp->autosplit, // " autosplit_enabled, "
|
pp->autosplit, // " autosplit_enabled, "
|
||||||
pp->zone_change_count, // " zone_change_count, "
|
pp->zone_change_count, // " zone_change_count, "
|
||||||
pp->drakkin_heritage, // " drakkin_heritage, "
|
pp->drakkin_heritage, // " drakkin_heritage, "
|
||||||
pp->drakkin_tattoo, // " drakkin_tattoo, "
|
pp->drakkin_tattoo, // " drakkin_tattoo, "
|
||||||
pp->drakkin_details, // " drakkin_details, "
|
pp->drakkin_details, // " drakkin_details, "
|
||||||
pp->toxicity, // " toxicity, "
|
pp->toxicity, // " toxicity, "
|
||||||
pp->hunger_level, // " hunger_level, "
|
pp->hunger_level, // " hunger_level, "
|
||||||
pp->thirst_level, // " thirst_level, "
|
pp->thirst_level, // " thirst_level, "
|
||||||
pp->ability_up, // " ability_up, "
|
pp->ability_up, // " ability_up, "
|
||||||
pp->zone_id, // " zone_id, "
|
pp->zone_id, // " zone_id, "
|
||||||
pp->zoneInstance, // " zone_instance, "
|
pp->zoneInstance, // " zone_instance, "
|
||||||
pp->leadAAActive, // " leadership_exp_on, "
|
pp->leadAAActive, // " leadership_exp_on, "
|
||||||
pp->ldon_points_guk, // " ldon_points_guk, "
|
pp->ldon_points_guk, // " ldon_points_guk, "
|
||||||
pp->ldon_points_mir, // " ldon_points_mir, "
|
pp->ldon_points_mir, // " ldon_points_mir, "
|
||||||
pp->ldon_points_mmc, // " ldon_points_mmc, "
|
pp->ldon_points_mmc, // " ldon_points_mmc, "
|
||||||
pp->ldon_points_ruj, // " ldon_points_ruj, "
|
pp->ldon_points_ruj, // " ldon_points_ruj, "
|
||||||
pp->ldon_points_tak, // " ldon_points_tak, "
|
pp->ldon_points_tak, // " ldon_points_tak, "
|
||||||
pp->ldon_points_available, // " ldon_points_available, "
|
pp->ldon_points_available, // " ldon_points_available, "
|
||||||
pp->tribute_time_remaining, // " tribute_time_remaining, "
|
pp->tribute_time_remaining, // " tribute_time_remaining, "
|
||||||
pp->showhelm, // " show_helm, "
|
pp->showhelm, // " show_helm, "
|
||||||
pp->career_tribute_points, // " career_tribute_points, "
|
pp->career_tribute_points, // " career_tribute_points, "
|
||||||
pp->tribute_points, // " tribute_points, "
|
pp->tribute_points, // " tribute_points, "
|
||||||
pp->tribute_active, // " tribute_active, "
|
pp->tribute_active, // " tribute_active, "
|
||||||
pp->endurance, // " endurance, "
|
pp->endurance, // " endurance, "
|
||||||
pp->group_leadership_exp, // " group_leadership_exp, "
|
pp->group_leadership_exp, // " group_leadership_exp, "
|
||||||
pp->raid_leadership_exp, // " raid_leadership_exp, "
|
pp->raid_leadership_exp, // " raid_leadership_exp, "
|
||||||
pp->group_leadership_points, // " group_leadership_points, "
|
pp->group_leadership_points, // " group_leadership_points, "
|
||||||
pp->raid_leadership_points, // " raid_leadership_points, "
|
pp->raid_leadership_points, // " raid_leadership_points, "
|
||||||
pp->air_remaining, // " air_remaining, "
|
pp->air_remaining, // " air_remaining, "
|
||||||
pp->PVPKills, // " pvp_kills, "
|
pp->PVPKills, // " pvp_kills, "
|
||||||
pp->PVPDeaths, // " pvp_deaths, "
|
pp->PVPDeaths, // " pvp_deaths, "
|
||||||
pp->PVPCurrentPoints, // " pvp_current_points, "
|
pp->PVPCurrentPoints, // " pvp_current_points, "
|
||||||
pp->PVPCareerPoints, // " pvp_career_points, "
|
pp->PVPCareerPoints, // " pvp_career_points, "
|
||||||
pp->PVPBestKillStreak, // " pvp_best_kill_streak, "
|
pp->PVPBestKillStreak, // " pvp_best_kill_streak, "
|
||||||
pp->PVPWorstDeathStreak, // " pvp_worst_death_streak, "
|
pp->PVPWorstDeathStreak, // " pvp_worst_death_streak, "
|
||||||
pp->PVPCurrentKillStreak, // " pvp_current_kill_streak, "
|
pp->PVPCurrentKillStreak, // " pvp_current_kill_streak, "
|
||||||
pp->aapoints_spent, // " aa_points_spent, "
|
pp->aapoints_spent, // " aa_points_spent, "
|
||||||
pp->expAA, // " aa_exp, "
|
pp->expAA, // " aa_exp, "
|
||||||
pp->aapoints, // " aa_points, "
|
pp->aapoints, // " aa_points, "
|
||||||
pp->groupAutoconsent, // " group_auto_consent, "
|
pp->groupAutoconsent, // " group_auto_consent, "
|
||||||
pp->raidAutoconsent, // " raid_auto_consent, "
|
pp->raidAutoconsent, // " raid_auto_consent, "
|
||||||
pp->guildAutoconsent, // " guild_auto_consent, "
|
pp->guildAutoconsent, // " guild_auto_consent, "
|
||||||
pp->RestTimer // " RestTimer) "
|
pp->RestTimer // " RestTimer) "
|
||||||
);
|
);
|
||||||
auto results = QueryDatabase(query);
|
QueryDatabase(query);
|
||||||
|
|
||||||
/* Save Bind Points */
|
/* Save Bind Points */
|
||||||
query = StringFormat("REPLACE INTO `character_bind` (id, zone_id, instance_id, x, y, z, heading, slot)"
|
query = StringFormat(
|
||||||
|
"REPLACE INTO `character_bind` (id, zone_id, instance_id, x, y, z, heading, slot)"
|
||||||
" VALUES (%u, %u, %u, %f, %f, %f, %f, %i), "
|
" VALUES (%u, %u, %u, %f, %f, %f, %f, %i), "
|
||||||
"(%u, %u, %u, %f, %f, %f, %f, %i), "
|
"(%u, %u, %u, %f, %f, %f, %f, %i), "
|
||||||
"(%u, %u, %u, %f, %f, %f, %f, %i), "
|
"(%u, %u, %u, %f, %f, %f, %f, %i), "
|
||||||
@ -739,44 +741,57 @@ bool Database::SaveCharacterCreate(uint32 character_id, uint32 account_id, Playe
|
|||||||
character_id, pp->binds[2].zone_id, 0, pp->binds[2].x, pp->binds[2].y, pp->binds[2].z, pp->binds[2].heading, 2,
|
character_id, pp->binds[2].zone_id, 0, pp->binds[2].x, pp->binds[2].y, pp->binds[2].z, pp->binds[2].heading, 2,
|
||||||
character_id, pp->binds[3].zone_id, 0, pp->binds[3].x, pp->binds[3].y, pp->binds[3].z, pp->binds[3].heading, 3,
|
character_id, pp->binds[3].zone_id, 0, pp->binds[3].x, pp->binds[3].y, pp->binds[3].z, pp->binds[3].heading, 3,
|
||||||
character_id, pp->binds[4].zone_id, 0, pp->binds[4].x, pp->binds[4].y, pp->binds[4].z, pp->binds[4].heading, 4
|
character_id, pp->binds[4].zone_id, 0, pp->binds[4].x, pp->binds[4].y, pp->binds[4].z, pp->binds[4].heading, 4
|
||||||
); results = QueryDatabase(query);
|
);
|
||||||
|
QueryDatabase(query);
|
||||||
|
|
||||||
/* HoTT Ability */
|
/* HoTT Ability */
|
||||||
if(RuleB(Character, GrantHoTTOnCreate))
|
if (RuleB(Character, GrantHoTTOnCreate)) {
|
||||||
{
|
query = StringFormat(
|
||||||
query = StringFormat("INSERT INTO `character_leadership_abilities` (id, slot, `rank`) VALUES (%u, %i, %i)", character_id, 14, 1);
|
"INSERT INTO `character_leadership_abilities` (id, slot, `rank`) VALUES (%u, %i, %i)",
|
||||||
results = QueryDatabase(query);
|
character_id,
|
||||||
}
|
14,
|
||||||
|
1
|
||||||
|
);
|
||||||
|
QueryDatabase(query);
|
||||||
|
}
|
||||||
|
|
||||||
/* Save Skills */
|
/* Save Skills */
|
||||||
int firstquery = 0;
|
int firstquery = 0;
|
||||||
for (int i = 0; i < MAX_PP_SKILL; i++){
|
for (int i = 0; i < MAX_PP_SKILL; i++) {
|
||||||
if (pp->skills[i] > 0){
|
if (pp->skills[i] > 0) {
|
||||||
if (firstquery != 1){
|
if (firstquery != 1) {
|
||||||
firstquery = 1;
|
firstquery = 1;
|
||||||
query = StringFormat("REPLACE INTO `character_skills` (id, skill_id, value) VALUES (%u, %u, %u)", character_id, i, pp->skills[i]);
|
query = StringFormat(
|
||||||
}
|
"REPLACE INTO `character_skills` (id, skill_id, value) VALUES (%u, %u, %u)",
|
||||||
else{
|
character_id,
|
||||||
|
i,
|
||||||
|
pp->skills[i]
|
||||||
|
);
|
||||||
|
} else {
|
||||||
query = query + StringFormat(", (%u, %u, %u)", character_id, i, pp->skills[i]);
|
query = query + StringFormat(", (%u, %u, %u)", character_id, i, pp->skills[i]);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
results = QueryDatabase(query);
|
QueryDatabase(query);
|
||||||
|
|
||||||
/* Save Language */
|
/* Save Language */
|
||||||
firstquery = 0;
|
firstquery = 0;
|
||||||
for (int i = 0; i < MAX_PP_LANGUAGE; i++){
|
for (int i = 0; i < MAX_PP_LANGUAGE; i++) {
|
||||||
if (pp->languages[i] > 0){
|
if (pp->languages[i] > 0) {
|
||||||
if (firstquery != 1){
|
if (firstquery != 1) {
|
||||||
firstquery = 1;
|
firstquery = 1;
|
||||||
query = StringFormat("REPLACE INTO `character_languages` (id, lang_id, value) VALUES (%u, %u, %u)", character_id, i, pp->languages[i]);
|
query = StringFormat(
|
||||||
}
|
"REPLACE INTO `character_languages` (id, lang_id, value) VALUES (%u, %u, %u)",
|
||||||
else{
|
character_id,
|
||||||
|
i,
|
||||||
|
pp->languages[i]
|
||||||
|
);
|
||||||
|
} else {
|
||||||
query = query + StringFormat(", (%u, %u, %u)", character_id, i, pp->languages[i]);
|
query = query + StringFormat(", (%u, %u, %u)", character_id, i, pp->languages[i]);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
results = QueryDatabase(query);
|
QueryDatabase(query);
|
||||||
|
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user