Review tweaks

This commit is contained in:
AffectedArc07
2021-09-07 12:52:28 +01:00
parent 70fd70f2eb
commit e73866cbf5
14 changed files with 45 additions and 46 deletions
+5 -6
View File
@@ -286,18 +286,18 @@
log_client_to_db(tdata) // Make sure our client exists in the DB
// This is where stuff happens on the client
// This is where we load all of the clients stuff from the DB
if(SSdbcore.IsConnected())
// Load in all our client data from the DB
var/list/datum/db_query/login_queries = list() // List of queries to run for login processing
for(var/datum/client_login_processor/CLP in GLOB.client_login_processors)
for(var/datum/client_login_processor/CLP as anything in GLOB.client_login_processors)
login_queries[CLP.type] = CLP.get_query(src)
SSdbcore.MassExecute(login_queries, TRUE, FALSE, TRUE, FALSE) // Warn, no qdel, assoc, no log
// Now do fancy things with the results
for(var/datum/client_login_processor/CLP in GLOB.client_login_processors)
for(var/datum/client_login_processor/CLP as anything in GLOB.client_login_processors)
CLP.process_result(login_queries[CLP.type], src)
QDEL_LIST_ASSOC_VAL(login_queries) // Clear out the used queries
@@ -309,7 +309,7 @@
prefs.character_saves = list()
// Random character
prefs.character_saves.Add(new /datum/character_save)
prefs.character_saves += new /datum/character_save
prefs.active_character = prefs.character_saves[1]
// ToS accepted
@@ -318,8 +318,7 @@
prefs.last_ip = address //these are gonna be used for banning
prefs.last_id = computer_id //these are gonna be used for banning
if(world.byond_version >= 511 && byond_version >= 511 && prefs.clientfps)
fps = prefs.clientfps
fps = prefs.clientfps
// Log alts
if(length(related_accounts_ip))
@@ -2,8 +2,8 @@
priority = 36
/datum/client_login_processor/watchlist/get_query(client/C)
var/datum/db_query/query = SSdbcore.NewQuery("SELECT reason FROM watch WHERE ckey=:target_ckey", list(
"target_ckey" = C.ckey
var/datum/db_query/query = SSdbcore.NewQuery("SELECT reason FROM watch WHERE ckey=:ckey", list(
"ckey" = C.ckey
))
return query
@@ -11,4 +11,4 @@
while(Q.NextRow())
if("[Q.item[1]]" == C.ckey)
continue
C.related_accounts_ip.Add("[Q.item[1]]")
C.related_accounts_ip += "[Q.item[1]]"
@@ -11,4 +11,4 @@
while(Q.NextRow())
if("[Q.item[1]]" == C.ckey)
continue
C.related_accounts_cid.Add("[Q.item[1]]")
C.related_accounts_cid += "[Q.item[1]]"
+23 -24
View File
@@ -3,18 +3,18 @@
/datum/character_save
var/real_name //our character's name
var/be_random_name = 0 //whether we are a random name every round
var/gender = MALE //gender of character (well duh)
var/age = 30 //age of character
var/spawnpoint = "Arrivals Shuttle" //where this character will spawn (0-2).
var/b_type = "A+" //blood type (not-chooseable)
var/real_name //our character's name
var/be_random_name = FALSE //whether we are a random name every round
var/gender = MALE //gender of character (well duh)
var/age = 30 //age of character
var/spawnpoint = "Arrivals Shuttle" //where this character will spawn (0-2).
var/b_type = "A+" //blood type (not-chooseable)
var/underwear = "Nude" //underwear type
var/undershirt = "Nude" //undershirt type
var/socks = "Nude" //socks type
var/backbag = GBACKPACK //backpack type
var/ha_style = "None" //Head accessory style
var/hacc_colour = "#000000" //Head accessory colour
var/socks = "Nude" //socks type
var/backbag = GBACKPACK //backpack type
var/ha_style = "None" //Head accessory style
var/hacc_colour = "#000000" //Head accessory colour. If this line looks badly indented in vscode, its because of the shitty colour square
var/list/m_styles = list(
"head" = "None",
"body" = "None",
@@ -41,14 +41,14 @@
var/body_accessory = null
var/speciesprefs = 0//I hate having to do this, I really do (Using this for oldvox code, making names universal I guess
var/speciesprefs = 0 //I hate having to do this, I really do (Using this for oldvox code, making names universal I guess
//Mob preview
//Mob preview
var/icon/preview_icon = null
var/icon/preview_icon_front = null
var/icon/preview_icon_side = null
//Jobs, uses bitflags
//Jobs, uses bitflags
var/job_support_high = 0
var/job_support_med = 0
var/job_support_low = 0
@@ -348,7 +348,7 @@
"med_record" = med_record,
"sec_record" = sec_record,
"gen_record" = gen_record,
"playertitlelist" = (playertitlelist ? playertitlelist : ""), // This it intentnional. It wont work without it!
"playertitlelist" = (playertitlelist ? playertitlelist : ""), // This it intentional. It wont work without it!
"disabilities" = disabilities,
"organlist" = (organlist ? organlist : ""),
"rlimblist" = (rlimblist ? rlimblist : ""),
@@ -443,8 +443,8 @@
//Sanitize
var/datum/species/SP = GLOB.all_species[species]
metadata = sanitize_text(metadata, initial(metadata))
real_name = reject_bad_name(real_name, 1)
metadata = sanitize_text(metadata, initial(metadata))
real_name = reject_bad_name(real_name, TRUE)
if(isnull(species))
species = "Human"
@@ -459,7 +459,7 @@
speciesprefs = initial(speciesprefs)
if(!real_name)
real_name = random_name(gender,species)
real_name = random_name(gender, species)
be_random_name = sanitize_integer(be_random_name, 0, 1, initial(be_random_name))
gender = sanitize_gender(gender, FALSE, !SP.has_gender)
@@ -1771,7 +1771,7 @@
var/datum/species/S = GLOB.all_species[species]
character.set_species(S.type) // Yell at me if this causes everything to melt
if(be_random_name)
real_name = random_name(gender,species)
real_name = random_name(gender, species)
character.add_language(language)
@@ -1850,7 +1850,7 @@
character.m_styles = m_styles
if(body_accessory)
character.body_accessory = GLOB.body_accessory_by_name["[body_accessory]"]
character.body_accessory = GLOB.body_accessory_by_name[body_accessory]
character.backbag = backbag
@@ -1918,8 +1918,8 @@
character.dna.UpdateSE()
domutcheck(character, MUTCHK_FORCED) //'Activates' all the above disabilities.
character.dna.ready_dna(character, flatten_SE = 0)
character.sync_organ_dna(assimilate=1)
character.dna.ready_dna(character, flatten_SE = FALSE)
character.sync_organ_dna(assimilate = TRUE)
character.UpdateAppearance()
// Do the initial caching of the player's body icons.
@@ -1998,7 +1998,7 @@
continue
var/available_in_playtime = job.available_in_playtime(user.client)
if(available_in_playtime)
html += "<del class='dark'>[rank]</del></td><td class='bad'><b> \[" + get_exp_format(available_in_playtime) + " as " + job.get_exp_req_type() + "\]</b></td></tr>"
html += "<del class='dark'>[rank]</del></td><td class='bad'><b> \[[get_exp_format(available_in_playtime)] as [job.get_exp_req_type()]\]</b></td></tr>"
continue
if(job.barred_by_disability(user.client))
html += "<del class='dark'>[rank]</del></td><td class='bad'><b> \[DISABILITY\]</b></td></tr>"
@@ -2093,8 +2093,7 @@
popup.set_window_options("can_close=0")
var/html_string = html.Join()
popup.set_content(html_string)
popup.open(0)
return
popup.open(FALSE)
/datum/character_save/proc/clear_character_slot(client/C)
@@ -1,6 +1,6 @@
/datum/preferences/proc/process_link(mob/user, list/href_list)
if(!user) return
if(!user)
return
var/datum/species/S = GLOB.all_species[active_character.species]
if(href_list["preference"] == "job")
@@ -41,7 +41,6 @@
active_character.SetChoices(user)
return 1
else if(href_list["preference"] == "disabilities")
switch(href_list["task"])
if("close")
user << browse(null, "window=disabil")
@@ -64,6 +63,7 @@
return
else
user << browse(null, "window=records")
if(href_list["task"] == "med_record")
var/medmsg = input(usr,"Set your medical notes here.","Medical Records",html_decode(active_character.med_record)) as message
@@ -83,6 +83,7 @@
active_character.sec_record = secmsg
active_character.SetRecords(user)
if(href_list["task"] == "gen_record")
var/genmsg = input(usr,"Set your employment notes here.","Employment Records",html_decode(active_character.gen_record)) as message
@@ -420,8 +420,8 @@ GLOBAL_LIST_INIT(special_role_times, list( //minimum age (in days) for accounts
if(TAB_GEAR)
var/total_cost = 0
var/list/type_blacklist = list()
if(active_character.loadout_gear && length(active_character.loadout_gear))
for(var/i = 1, i <= length(active_character.loadout_gear), i++)
if(length(active_character.loadout_gear))
for(var/i in 1 to length(active_character.loadout_gear))
var/datum/gear/G = GLOB.gear_datums[active_character.loadout_gear[i]]
if(G)
if(!G.subtype_cost_overlap)
@@ -103,7 +103,7 @@
var/list/datum/character_save/valid_slots = list()
for(var/datum/character_save/CS in character_saves)
if(CS.valid_save)
valid_slots.Add(CS)
valid_slots += CS
if(!length(valid_slots))
// They have no valid saves. Lets just randomise #1
@@ -115,7 +115,6 @@
var/datum/character_save/CS = pick(valid_slots)
C.prefs.active_character = CS
return
/**
* Saves [/datum/preferences/proc/volume_mixer] for the current client.