Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Databasing whitelists #5402

Merged
merged 30 commits into from
Jan 22, 2024
Merged
Show file tree
Hide file tree
Changes from 13 commits
Commits
Show all changes
30 commits
Select commit Hold shift + click to select a range
5f89992
initial commit
harryob Nov 17, 2023
66ad03b
whitelist panel and changes
harryob Nov 25, 2023
a10c200
Merge remote-tracking branch 'cmss13-devs/master' into databasing/whi…
realforest2001 Jan 7, 2024
6fd3603
fixes
realforest2001 Jan 7, 2024
ff81084
Nuke Alien Whitelist
realforest2001 Jan 7, 2024
f11e045
TGUI
realforest2001 Jan 7, 2024
81652c3
opens
realforest2001 Jan 7, 2024
4322bcf
xxx
realforest2001 Jan 8, 2024
15d3b1b
functional?!
realforest2001 Jan 8, 2024
61f6f37
functional.
realforest2001 Jan 8, 2024
874748e
Fully functional
realforest2001 Jan 8, 2024
f39cffc
feedback message
realforest2001 Jan 8, 2024
364d9b3
edit lock
realforest2001 Jan 8, 2024
5150a81
changes
realforest2001 Jan 9, 2024
c703ce8
todo comment
realforest2001 Jan 9, 2024
2a8fe76
readable status
realforest2001 Jan 9, 2024
d521efc
ui user
realforest2001 Jan 9, 2024
5a9e225
runtime catch
realforest2001 Jan 9, 2024
e17609c
why was this here
harryob Jan 18, 2024
19a4bb5
Merge remote-tracking branch 'upstream/master' into pr/realforest2001…
harryob Jan 18, 2024
3a987cf
reverts changes to runtime
harryob Jan 18, 2024
051ea98
we do some accidental reverting as a treat
harryob Jan 18, 2024
7ce402c
multi-user
realforest2001 Jan 19, 2024
cd68235
search button
realforest2001 Jan 19, 2024
b3f2370
working joe fix
realforest2001 Jan 19, 2024
86faac5
Merge remote-tracking branch 'cmss13-devs/master' into databasing/whi…
realforest2001 Jan 19, 2024
88d0ecb
co lead fix
realforest2001 Jan 19, 2024
d2a74eb
fixes twerking joe
harryob Jan 19, 2024
6d70624
cleanup
harryob Jan 19, 2024
cd12902
very unhappy about that warning
harryob Jan 19, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
30 changes: 29 additions & 1 deletion code/__DEFINES/mode.dm
Original file line number Diff line number Diff line change
Expand Up @@ -156,6 +156,7 @@ GLOBAL_LIST_INIT(ROLES_UNASSIGNED, list(JOB_SQUAD_MARINE))
GLOBAL_LIST_INIT(whitelist_hierarchy, list(WHITELIST_NORMAL, WHITELIST_COUNCIL, WHITELIST_LEADER))

//=================================================

#define WHITELIST_YAUTJA (1<<0)
///Old holders of YAUTJA_ELDER
#define WHITELIST_YAUTJA_LEGACY (1<<1)
Expand Down Expand Up @@ -186,7 +187,34 @@ GLOBAL_LIST_INIT(whitelist_hierarchy, list(WHITELIST_NORMAL, WHITELIST_COUNCIL,

#define WHITELIST_EVERYTHING (WHITELISTS_GENERAL|WHITELISTS_COUNCIL|WHITELISTS_LEADER)

#define isCouncil(A) (GLOB.RoleAuthority.roles_whitelist[A.ckey] & WHITELIST_YAUTJA_COUNCIL) || (GLOB.RoleAuthority.roles_whitelist[A.ckey] & WHITELIST_SYNTHETIC_COUNCIL) || (GLOB.RoleAuthority.roles_whitelist[A.ckey] & WHITELIST_COMMANDER_COUNCIL)
#define COUNCIL_LIST list(WHITELIST_COMMANDER_COUNCIL, WHITELIST_SYNTHETIC_COUNCIL, WHITELIST_YAUTJA_COUNCIL)
#define SENATOR_LIST list(WHITELIST_COMMANDER_LEADER, WHITELIST_SYNTHETIC_LEADER, WHITELIST_YAUTJA_LEADER)
#define isCouncil(A) (A.check_whitelist_status_list(COUNCIL_LIST))
#define isSenator(A) (A.check_whitelist_status_list(SENATOR_LIST))

GLOBAL_LIST_INIT_TYPED(whitelist_permissions, /list, list(
WHITELIST_SYNTHETIC_LEADER = list(WHITELIST_SYNTHETIC_COUNCIL, WHITELIST_SYNTHETIC_COUNCIL_LEGACY, WHITELIST_SYNTHETIC),
WHITELIST_YAUTJA_LEADER = list(WHITELIST_YAUTJA_COUNCIL, WHITELIST_YAUTJA_COUNCIL_LEGACY, WHITELIST_YAUTJA),
WHITELIST_COMMANDER_LEADER = list(WHITELIST_COMMANDER_COUNCIL, WHITELIST_COMMANDER_COUNCIL_LEGACY, WHITELIST_COMMANDER),
realforest2001 marked this conversation as resolved.
Show resolved Hide resolved
))

DEFINE_BITFIELD(whitelist_status, list(
"WHITELIST_YAUTJA" = WHITELIST_YAUTJA,
"WHITELIST_YAUTJA_LEGACY" = WHITELIST_YAUTJA_LEGACY,
"WHITELIST_YAUTJA_COUNCIL" = WHITELIST_YAUTJA_COUNCIL,
"WHITELIST_YAUTJA_COUNCIL_LEGACY" = WHITELIST_YAUTJA_COUNCIL_LEGACY,
"WHITELIST_YAUTJA_LEADER" = WHITELIST_YAUTJA_LEADER,
"WHITELIST_COMMANDER" = WHITELIST_COMMANDER,
"WHITELIST_COMMANDER_COUNCIL" = WHITELIST_COMMANDER_COUNCIL,
"WHITELIST_COMMANDER_COUNCIL_LEGACY" = WHITELIST_COMMANDER_COUNCIL_LEGACY,
"WHITELIST_COMMANDER_LEADER" = WHITELIST_COMMANDER_LEADER,
"WHITELIST_JOE" = WHITELIST_JOE,
"WHITELIST_SYNTHETIC" = WHITELIST_SYNTHETIC,
"WHITELIST_SYNTHETIC_COUNCIL" = WHITELIST_SYNTHETIC_COUNCIL,
"WHITELIST_SYNTHETIC_COUNCIL_LEGACY" = WHITELIST_SYNTHETIC_COUNCIL_LEGACY,
"WHITELIST_SYNTHETIC_LEADER" = WHITELIST_SYNTHETIC_LEADER,
"WHITELIST_MENTOR" = WHITELIST_MENTOR,
))

//=================================================

Expand Down
2 changes: 1 addition & 1 deletion code/datums/emergency_calls/clf.dm
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@
leader = H
to_chat(H, SPAN_ROLE_HEADER("You are a Cell Leader of the local resistance group, the Colonial Liberation Front!"))
arm_equipment(H, /datum/equipment_preset/clf/leader, TRUE, TRUE)
else if(synths < max_synths && HAS_FLAG(H.client.prefs.toggles_ert, PLAY_SYNTH) && GLOB.RoleAuthority.roles_whitelist[H.ckey] & WHITELIST_SYNTHETIC)
else if(synths < max_synths && HAS_FLAG(H.client.prefs.toggles_ert, PLAY_SYNTH) && H.client.check_whitelist_status(WHITELIST_SYNTHETIC))
synths++
to_chat(H, SPAN_ROLE_HEADER("You are a Multi-Purpose Synthetic for the local resistance group, the Colonial Liberation Front!"))
arm_equipment(H, /datum/equipment_preset/clf/synth, TRUE, TRUE)
Expand Down
2 changes: 1 addition & 1 deletion code/datums/emergency_calls/cmb.dm
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@
leader = mob
to_chat(mob, SPAN_ROLE_HEADER("You are the Colonial Marshal!"))
arm_equipment(mob, /datum/equipment_preset/cmb/leader, TRUE, TRUE)
else if(synths < max_synths && HAS_FLAG(mob?.client.prefs.toggles_ert, PLAY_SYNTH) && GLOB.RoleAuthority.roles_whitelist[mob.ckey] & WHITELIST_SYNTHETIC)
else if(synths < max_synths && HAS_FLAG(mob?.client.prefs.toggles_ert, PLAY_SYNTH) && mob.client.check_whitelist_status(WHITELIST_SYNTHETIC))
synths++
to_chat(mob, SPAN_ROLE_HEADER("You are a CMB Investigative Synthetic!"))
arm_equipment(mob, /datum/equipment_preset/cmb/synth, TRUE, TRUE)
Expand Down
4 changes: 2 additions & 2 deletions code/datums/emergency_calls/contractor.dm
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@
leader = mob
to_chat(mob, SPAN_ROLE_HEADER("You are a Contractor Team Leader of Vanguard's Arrow Incorporated!"))
arm_equipment(mob, /datum/equipment_preset/contractor/duty/leader, TRUE, TRUE)
else if(synths < max_synths && HAS_FLAG(mob.client.prefs.toggles_ert, PLAY_SYNTH) && GLOB.RoleAuthority.roles_whitelist[mob.ckey] & WHITELIST_SYNTHETIC)
else if(synths < max_synths && HAS_FLAG(mob.client.prefs.toggles_ert, PLAY_SYNTH) && mob.client.check_whitelist_status(WHITELIST_SYNTHETIC))
synths++
to_chat(mob, SPAN_ROLE_HEADER("You are a Contractor Support Synthetic of Vanguard's Arrow Incorporated!"))
arm_equipment(mob, /datum/equipment_preset/contractor/duty/synth, TRUE, TRUE)
Expand Down Expand Up @@ -123,7 +123,7 @@
leader = H
to_chat(H, SPAN_ROLE_HEADER("You are a Covert Contractor Team Leader of Vanguard's Arrow Incorporated!"))
arm_equipment(H, /datum/equipment_preset/contractor/covert/leader, TRUE, TRUE)
else if(synths < max_synths && HAS_FLAG(H.client.prefs.toggles_ert, PLAY_SYNTH) && GLOB.RoleAuthority.roles_whitelist[H.ckey] & WHITELIST_SYNTHETIC)
else if(synths < max_synths && HAS_FLAG(H.client.prefs.toggles_ert, PLAY_SYNTH) && H.client.check_whitelist_status(WHITELIST_SYNTHETIC))
synths++
to_chat(H, SPAN_ROLE_HEADER("You are a Contractor Support Synthetic of Vanguard's Arrow Incorporated!"))
arm_equipment(H, /datum/equipment_preset/contractor/covert/synth, TRUE, TRUE)
Expand Down
2 changes: 1 addition & 1 deletion code/datums/emergency_calls/inspection.dm
Original file line number Diff line number Diff line change
Expand Up @@ -202,7 +202,7 @@
leader = mob
to_chat(mob, SPAN_ROLE_HEADER("You are the Colonial Marshal!"))
arm_equipment(mob, /datum/equipment_preset/cmb/leader, TRUE, TRUE)
else if(synths < max_synths && HAS_FLAG(mob?.client.prefs.toggles_ert, PLAY_SYNTH) && GLOB.RoleAuthority.roles_whitelist[mob.ckey] & WHITELIST_SYNTHETIC)
else if(synths < max_synths && HAS_FLAG(mob?.client.prefs.toggles_ert, PLAY_SYNTH) && mob.client.check_whitelist_status(WHITELIST_SYNTHETIC))
synths++
to_chat(mob, SPAN_ROLE_HEADER("You are a CMB Investigative Synthetic!"))
arm_equipment(mob, /datum/equipment_preset/cmb/synth, TRUE, TRUE)
Expand Down
2 changes: 1 addition & 1 deletion code/datums/emergency_calls/pmc.dm
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@
leader = mob
to_chat(mob, SPAN_ROLE_HEADER("You are a Weyland-Yutani PMC Squad Leader!"))
arm_equipment(mob, /datum/equipment_preset/pmc/pmc_leader, TRUE, TRUE)
else if(synths < max_synths && HAS_FLAG(mob.client.prefs.toggles_ert, PLAY_SYNTH) && GLOB.RoleAuthority.roles_whitelist[mob.ckey] & WHITELIST_SYNTHETIC)
else if(synths < max_synths && HAS_FLAG(mob.client.prefs.toggles_ert, PLAY_SYNTH) && mob.client.check_whitelist_status(WHITELIST_SYNTHETIC))
synths++
to_chat(mob, SPAN_ROLE_HEADER("You are a Weyland-Yutani PMC Support Synthetic!"))
arm_equipment(mob, /datum/equipment_preset/pmc/synth, TRUE, TRUE)
Expand Down
2 changes: 1 addition & 1 deletion code/datums/emergency_calls/upp.dm
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@
leader = H
arm_equipment(H, /datum/equipment_preset/upp/leader, TRUE, TRUE)
to_chat(H, SPAN_ROLE_HEADER("You are an Officer of the Union of Progressive People, a powerful socialist state that rivals the United Americas!"))
else if(synths < max_synths && HAS_FLAG(H.client.prefs.toggles_ert, PLAY_SYNTH) && GLOB.RoleAuthority.roles_whitelist[H.ckey] & WHITELIST_SYNTHETIC)
else if(synths < max_synths && HAS_FLAG(H.client.prefs.toggles_ert, PLAY_SYNTH) && H.client.check_whitelist_status(WHITELIST_SYNTHETIC))
synths++
to_chat(H, SPAN_ROLE_HEADER("You are a Combat Synthetic of the Union of Progressive People, a powerful socialist state that rivals the United Americas!"))
arm_equipment(H, /datum/equipment_preset/upp/synth, TRUE, TRUE)
Expand Down
2 changes: 1 addition & 1 deletion code/datums/emergency_calls/xeno_cultists.dm
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@
leader = H
to_chat(H, SPAN_ROLE_HEADER("You are the leader of this xeno cult! Bring glory to Queen Mother!"))
arm_equipment(H, /datum/equipment_preset/other/xeno_cultist/leader, TRUE, TRUE)
else if(synths < max_synths && HAS_FLAG(H.client.prefs.toggles_ert, PLAY_SYNTH) && GLOB.RoleAuthority.roles_whitelist[H.ckey] & WHITELIST_SYNTHETIC)
else if(synths < max_synths && HAS_FLAG(H.client.prefs.toggles_ert, PLAY_SYNTH) && H.client.check_whitelist_status(WHITELIST_SYNTHETIC))
synths++
to_chat(H, SPAN_ROLE_HEADER("You are the xeno cult's synthetic! Tend to the Hive and the captured hosts, make sure the Hive grows!"))
arm_equipment(H, /datum/equipment_preset/synth/survivor/cultist_synth, TRUE, TRUE)
Expand Down
42 changes: 28 additions & 14 deletions code/datums/entities/player.dm
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,9 @@
var/last_known_ip
var/last_known_cid

var/whitelist_status
var/whitelist_flags

var/discord_link_id

var/last_login
Expand Down Expand Up @@ -63,6 +66,7 @@ BSQL_PROTECT_DATUM(/datum/entity/player)
"is_permabanned" = DB_FIELDTYPE_INT,
"permaban_reason" = DB_FIELDTYPE_STRING_MAX,
"permaban_date" = DB_FIELDTYPE_STRING_LARGE,
"whitelist_status" = DB_FIELDTYPE_STRING_MAX,
"discord_link_id" = DB_FIELDTYPE_BIGINT,
"permaban_admin_id" = DB_FIELDTYPE_BIGINT,
"is_time_banned" = DB_FIELDTYPE_INT,
Expand Down Expand Up @@ -321,20 +325,6 @@ BSQL_PROTECT_DATUM(/datum/entity/player)
value.delete()
job_bans -= value

/datum/entity/player/proc/load_refs()
if(refs_loaded)
return
while(!notes_loaded || !jobbans_loaded)
stoplag()
for(var/key in job_bans)
var/datum/entity/player_job_ban/value = job_bans[key]
if(istype(value))
value.load_refs()
for(var/datum/entity/player_note/note in notes)
if(istype(note))
note.load_refs()
refs_loaded = TRUE

/datum/entity_meta/player/on_read(datum/entity/player/player)
player.job_bans = list()
player.notes = list()
Expand Down Expand Up @@ -391,7 +381,12 @@ BSQL_PROTECT_DATUM(/datum/entity/player)
if(discord_link_id)
discord_link = DB_ENTITY(/datum/entity/discord_link, discord_link_id)

if(whitelist_status)
var/list/whitelists = splittext(whitelist_status, "|")

for(var/whitelist in whitelists)
if(whitelist in GLOB.bitfields["whitelist_status"])
whitelist_flags |= GLOB.bitfields["whitelist_status"]["[whitelist]"]

/datum/entity/player/proc/on_read_notes(list/datum/entity/player_note/_notes)
notes_loaded = TRUE
Expand Down Expand Up @@ -657,6 +652,23 @@ BSQL_PROTECT_DATUM(/datum/entity/player)
stat.stat_number += num
stat.save()

/datum/entity/player/proc/check_whitelist_status(flag_to_check)
if(whitelist_flags & flag_to_check)
return TRUE

return FALSE

/datum/entity/player/proc/set_whitelist_status(field_to_set)
whitelist_flags = field_to_set

var/list/output = list()
for(var/bitfield in GLOB.bitfields["whitelist_status"])
if(field_to_set & GLOB.bitfields["whitelist_status"]["[bitfield]"])
output += bitfield
whitelist_status = output.Join("|")

save()

/datum/entity_link/player_to_banning_admin
parent_entity = /datum/entity/player
child_entity = /datum/entity/player
Expand Down Expand Up @@ -685,6 +697,7 @@ BSQL_PROTECT_DATUM(/datum/entity/player)
var/last_known_cid
var/last_known_ip
var/discord_link_id
var/whitelist_status

/datum/entity_view_meta/players
root_record_type = /datum/entity/player
Expand All @@ -702,4 +715,5 @@ BSQL_PROTECT_DATUM(/datum/entity/player)
"last_known_ip",
"last_known_cid",
"discord_link_id",
"whitelist_status"
)
12 changes: 6 additions & 6 deletions code/game/gamemodes/cm_initialize.dm
Original file line number Diff line number Diff line change
Expand Up @@ -155,7 +155,7 @@ Additional game mode variables.
else
if(!istype(player,/mob/dead)) continue //Otherwise we just want to grab the ghosts.

if(GLOB.RoleAuthority.roles_whitelist[player.ckey] & WHITELIST_PREDATOR) //Are they whitelisted?
if(player?.client.check_whitelist_status(WHITELIST_PREDATOR)) //Are they whitelisted?
if(!player.client.prefs)
player.client.prefs = new /datum/preferences(player.client) //Somehow they don't have one.

Expand Down Expand Up @@ -188,7 +188,7 @@ Additional game mode variables.
if(show_warning) to_chat(pred_candidate, SPAN_WARNING("Something went wrong!"))
return

if(!(GLOB.RoleAuthority.roles_whitelist[pred_candidate.ckey] & WHITELIST_PREDATOR))
if(!(pred_candidate?.client.check_whitelist_status(WHITELIST_PREDATOR)))
if(show_warning) to_chat(pred_candidate, SPAN_WARNING("You are not whitelisted! You may apply on the forums to be whitelisted as a predator."))
return

Expand All @@ -201,9 +201,9 @@ Additional game mode variables.
to_chat(pred_candidate, SPAN_WARNING("You already were a Yautja! Give someone else a chance."))
return

if(show_warning && tgui_alert(pred_candidate, "Confirm joining the hunt. You will join as \a [lowertext(J.get_whitelist_status(GLOB.RoleAuthority.roles_whitelist, pred_candidate.client))] predator", "Confirmation", list("Yes", "No"), 10 SECONDS) != "Yes")
if(show_warning && tgui_alert(pred_candidate, "Confirm joining the hunt. You will join as \a [lowertext(J.get_whitelist_status(pred_candidate.client))] predator", "Confirmation", list("Yes", "No"), 10 SECONDS) != "Yes")
return
if(J.get_whitelist_status(GLOB.RoleAuthority.roles_whitelist, pred_candidate.client) == WHITELIST_NORMAL)
if(J.get_whitelist_status(pred_candidate.client) == WHITELIST_NORMAL)
var/pred_max = calculate_pred_max
if(pred_current_num >= pred_max)
if(show_warning) to_chat(pred_candidate, SPAN_WARNING("Only [pred_max] predators may spawn this round, but Councillors and Ancients do not count."))
Expand Down Expand Up @@ -1001,7 +1001,7 @@ Additional game mode variables.
to_chat(joe_candidate, SPAN_WARNING("Something went wrong!"))
return

if(!(GLOB.RoleAuthority.roles_whitelist[joe_candidate.ckey] & WHITELIST_JOE))
if(!(joe_candidate.client.check_whitelist_status(WHITELIST_JOE)))
if(show_warning)
to_chat(joe_candidate, SPAN_WARNING("You are not whitelisted! You may apply on the forums to be whitelisted as a synth."))
return
Expand All @@ -1012,7 +1012,7 @@ Additional game mode variables.
return

// council doesn't count towards this conditional.
if(joe_job.get_whitelist_status(GLOB.RoleAuthority.roles_whitelist, joe_candidate.client) == WHITELIST_NORMAL)
if(joe_job.get_whitelist_status(joe_candidate.client) == WHITELIST_NORMAL)
var/joe_max = joe_job.total_positions
if((joe_job.current_positions >= joe_max) && !MODE_HAS_TOGGLEABLE_FLAG(MODE_BYPASS_JOE))
if(show_warning)
Expand Down
2 changes: 1 addition & 1 deletion code/game/gamemodes/extended/infection.dm
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@
possible_synth_survivors -= A
continue

if(GLOB.RoleAuthority.roles_whitelist[ckey(A.key)] & WHITELIST_SYNTHETIC)
if(A.current.client?.check_whitelist_status(WHITELIST_SYNTHETIC))
if(A in possible_survivors)
continue //they are already applying to be a survivor
else
Expand Down
11 changes: 2 additions & 9 deletions code/game/jobs/job/antag/other/pred.dm
Original file line number Diff line number Diff line change
Expand Up @@ -31,11 +31,7 @@

SSticker.mode.attempt_to_join_as_predator(player)

/datum/job/antag/predator/get_whitelist_status(list/roles_whitelist, client/player) // Might be a problem waiting here, but we've got no choice
. = ..()
if(!.)
return

/datum/job/antag/predator/get_whitelist_status(client/player) // Might be a problem waiting here, but we've got no choice
if(!player.clan_info)
return CLAN_RANK_BLOODED

Expand All @@ -49,10 +45,7 @@
if(!("[JOB_PREDATOR][rank]" in gear_preset_whitelist))
return CLAN_RANK_BLOODED

if(\
(roles_whitelist[player.ckey] & (WHITELIST_YAUTJA_LEADER|WHITELIST_YAUTJA_COUNCIL|WHITELIST_YAUTJA_COUNCIL_LEGACY)) &&\
get_desired_status(player.prefs.yautja_status, WHITELIST_COUNCIL) == WHITELIST_NORMAL\
)
if(player.check_whitelist_status(WHITELIST_YAUTJA_LEADER|WHITELIST_YAUTJA_COUNCIL|WHITELIST_YAUTJA_COUNCIL_LEGACY) && get_desired_status(player.prefs.yautja_status, WHITELIST_COUNCIL) == WHITELIST_NORMAL)
return CLAN_RANK_BLOODED

return rank
Expand Down
8 changes: 4 additions & 4 deletions code/game/jobs/job/civilians/support/synthetic.dm
Original file line number Diff line number Diff line change
Expand Up @@ -19,16 +19,16 @@
"[JOB_SYNTH][WHITELIST_LEADER]" = /datum/equipment_preset/synth/uscm/councillor
)

/datum/job/civilian/synthetic/get_whitelist_status(list/roles_whitelist, client/player)
/datum/job/civilian/synthetic/get_whitelist_status(client/player)
. = ..()
if(!.)
return

if(roles_whitelist[player.ckey] & WHITELIST_SYNTHETIC_LEADER)
if(player.check_whitelist_status(WHITELIST_SYNTHETIC_LEADER))
return get_desired_status(player.prefs.synth_status, WHITELIST_LEADER)
else if(roles_whitelist[player.ckey] & (WHITELIST_SYNTHETIC_COUNCIL|WHITELIST_SYNTHETIC_COUNCIL_LEGACY))
if(player.check_whitelist_status(WHITELIST_SYNTHETIC_COUNCIL|WHITELIST_SYNTHETIC_COUNCIL_LEGACY))
return get_desired_status(player.prefs.synth_status, WHITELIST_COUNCIL)
else if(roles_whitelist[player.ckey] & WHITELIST_SYNTHETIC)
if(player.check_whitelist_status(WHITELIST_SYNTHETIC))
return get_desired_status(player.prefs.synth_status, WHITELIST_NORMAL)

/datum/job/civilian/synthetic/set_spawn_positions(count)
Expand Down
8 changes: 4 additions & 4 deletions code/game/jobs/job/command/cic/captain.dm
Original file line number Diff line number Diff line change
Expand Up @@ -19,16 +19,16 @@
entry_message_body = "<a href='[generate_wiki_link()]'>You are the Commanding Officer of the [MAIN_SHIP_NAME] as well as the operation.</a> Your goal is to lead the Marines on their mission as well as protect and command the ship and her crew. Your job involves heavy roleplay and requires you to behave like a high-ranking officer and to stay in character at all times. As the Commanding Officer your only superior is High Command itself. You must abide by the <a href='[CONFIG_GET(string/wikiarticleurl)]/[URL_WIKI_CO_RULES]'>Commanding Officer Code of Conduct</a>. Failure to do so may result in punitive action against you. Godspeed."
return ..()

/datum/job/command/commander/get_whitelist_status(list/roles_whitelist, client/player)
/datum/job/command/commander/get_whitelist_status(client/player)
. = ..()
if(!.)
return

if(roles_whitelist[player.ckey] & WHITELIST_COMMANDER_LEADER)
if(player.check_whitelist_status(WHITELIST_COMMANDER_LEADER))
return get_desired_status(player.prefs.commander_status, WHITELIST_LEADER)
else if(roles_whitelist[player.ckey] & (WHITELIST_COMMANDER_COUNCIL|WHITELIST_COMMANDER_COUNCIL_LEGACY))
if(player.check_whitelist_status(WHITELIST_COMMANDER_COUNCIL|WHITELIST_COMMANDER_COUNCIL_LEGACY))
return get_desired_status(player.prefs.commander_status, WHITELIST_COUNCIL)
else if(roles_whitelist[player.ckey] & WHITELIST_COMMANDER)
if(player.check_whitelist_status(WHITELIST_COMMANDER))
return get_desired_status(player.prefs.commander_status, WHITELIST_NORMAL)

/datum/job/command/commander/announce_entry_message(mob/living/carbon/human/H)
Expand Down
7 changes: 2 additions & 5 deletions code/game/jobs/job/job.dm
Original file line number Diff line number Diff line change
Expand Up @@ -66,10 +66,7 @@
return ""
return "[CONFIG_GET(string/wikiarticleurl)]/[replacetext(title, " ", "_")]"

/datum/job/proc/get_whitelist_status(list/roles_whitelist, client/player)
if(!roles_whitelist)
return FALSE

/datum/job/proc/get_whitelist_status(client/player)
return WHITELIST_NORMAL

/datum/timelock
Expand Down Expand Up @@ -252,7 +249,7 @@
var/mob/living/carbon/human/human = M

var/job_whitelist = title
var/whitelist_status = get_whitelist_status(GLOB.RoleAuthority.roles_whitelist, human.client)
var/whitelist_status = get_whitelist_status(human.client)

if(whitelist_status)
job_whitelist = "[title][whitelist_status]"
Expand Down
Loading
Loading