Skip to content

Maximum generations based on job - #1263

Open
Faengi wants to merge 2 commits into
DarkPack13:masterfrom
Faengi:joblimits
Open

Maximum generations based on job#1263
Faengi wants to merge 2 commits into
DarkPack13:masterfrom
Faengi:joblimits

Conversation

@Faengi

@Faengi Faengi commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

About The Pull Request

Jobs are generation-limited. Elders only in leadership, Ancillae only in 'faction' jobs (used loosely here; basically everything except Citizen/Janitor.) Exception made for Taxi Driver because I swear I have a sense of humor and/or a soul.

Why It's Good For The Game

Desired for downstream, made it a config. Infinite civilians stronger than the Hounds expected to keep them in check is bad for gameplay.

Changelog

🆑
config: Added a config setting to limit high generations to a limited selection of jobs
/:cl:

@Faengi
Faengi requested a review from a team as a code owner July 26, 2026 19:14
return JOB_UNAVAILABLE_KINDRED_AGE_MIN

if((!isnull(possible_job.maximum_immortal_age) && (player_client.prefs.read_preference(/datum/preference/numeric/immortal_age) > possible_job.maximum_immortal_age)))
if((CONFIG_GET(flag/max_gen_limits)) && (!isnull(possible_job.maximum_immortal_age) && (player_client.prefs.read_preference(/datum/preference/numeric/immortal_age) > possible_job.maximum_immortal_age)))

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this was not barred by config flag before and is not new behavior as per this PR - is the check for config flag necessary do you think?

@Faengi Faengi Jul 28, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I mostly did this because I added max ages to the jobs in the PR too (the code exists but no jobs had one before) and given they're also tied to power scaling (discipline dots) I figured I'd just lump elder ages + elder gens together for it.

@chazzyjazzy chazzyjazzy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see anything wrong? Perhaps there is utility in the future for defines regarding ancillae age or generation levels that denote certain 'ranks' of vamps or whatever - as I'm imagining in the future, perhaps some servers may not want to have such low gens.


return JOB_AVAILABLE

/datum/config_entry/flag/max_gen_limits //If you can be limited from a job for having too high a generation

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add a default value FALSE?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants