Skip to content

Reimplements Visceratika#739

Open
chazzyjazzy wants to merge 7 commits intoDarkPack13:masterfrom
chazzyjazzy:garg
Open

Reimplements Visceratika#739
chazzyjazzy wants to merge 7 commits intoDarkPack13:masterfrom
chazzyjazzy:garg

Conversation

@chazzyjazzy
Copy link
Contributor

About The Pull Request

No flying yet thats a diferent PR since im trying to atomize

Why It's Good For The Game

Gar of the Goyle

Changelog

🆑
add: Reimplements Visceratika

/:cl:

if(isliving(mover))
var/mob/living/moving_mob = mover
if(HAS_TRAIT(moving_mob, TRAIT_PASS_THROUGH_WALLS) && (get_area(moving_mob) == get_area(src)))
if(HAS_TRAIT(moving_mob, TRAIT_PASS_THROUGH_WALLS)/* && (get_area(moving_mob) == get_area(src))*/)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

not sure if the area check (gargoyles passing thru walls can only travel thru walls within a particular area, not go from area to area, for ex, they cant go from the sewers into the chantry) is a lore thing or a balance thing so might need a ruling on that...

Copy link
Member

Choose a reason for hiding this comment

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

Balance. It also prevents people from walking out into space/unintended parts of the map. Its kinda required unless we map better.

@chazzyjazzy chazzyjazzy marked this pull request as ready for review March 2, 2026 19:15
@chazzyjazzy chazzyjazzy requested a review from a team as a code owner March 2, 2026 19:15
@github-actions
Copy link
Contributor

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~7 days. Please address any outstanding review items and ensure your PR is finished - if both are true, and you have auto-staled anyway, you need to actively ask maintainers (by pinging them in the /tg/station Discord) to (re)review or merge your PR. If no maintainer responds to your request, you may wish to close this PR yourself while you seek maintainer comment, as you will later be able to reopen the PR yourself.

Comment on lines +22 to +29
if(level >= 4)
owner.physiology.brute_mod *= 0.8
owner.physiology.heat_mod *= 0.5
//owner.physiology.clone_mod *= 0.9
//ADD_TRAIT(owner, TRAIT_IGNOREDAMAGESLOWDOWN, TRAIT_GENERIC)
ADD_TRAIT(owner, TRAIT_NOSOFTCRIT, DISCIPLINE_TRAIT)
if(!(owner.is_clan(/datum/subsplat/vampire_clan/gargoyle)))
ADD_TRAIT(owner, TRAIT_MASQUERADE_VIOLATING_FACE, DISCIPLINE_TRAIT)
Copy link
Member

Choose a reason for hiding this comment

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

In most code rn this is done on the gain of the disc power instead...

if(level >= 4)
owner.physiology.brute_mod *= 0.8
owner.physiology.heat_mod *= 0.5
//owner.physiology.clone_mod *= 0.9
Copy link
Member

Choose a reason for hiding this comment

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

agg_mod?

Comment on lines +28 to +29
if(!(owner.is_clan(/datum/subsplat/vampire_clan/gargoyle)))
ADD_TRAIT(owner, TRAIT_MASQUERADE_VIOLATING_FACE, DISCIPLINE_TRAIT)
Copy link
Member

Choose a reason for hiding this comment

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

I dont see a reason to have a check for gargy here. The point of trait sources is so we can apply them with overlap like this.

Comment on lines +90 to +96
for(var/mob/living/player in GLOB.player_list)
if(get_area(player) == get_area(owner))
var/their_name = player.name
if(ishuman(player))
var/mob/living/carbon/human/human_player = player
their_name = human_player.real_name
to_chat(owner, "- [their_name]")
Copy link
Member

Choose a reason for hiding this comment

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

This seems a little unneeded and is represented fine with thermal vision. its also based on area and shows real name for little reason..

if(isliving(mover))
var/mob/living/moving_mob = mover
if(HAS_TRAIT(moving_mob, TRAIT_PASS_THROUGH_WALLS) && (get_area(moving_mob) == get_area(src)))
if(HAS_TRAIT(moving_mob, TRAIT_PASS_THROUGH_WALLS)/* && (get_area(moving_mob) == get_area(src))*/)
Copy link
Member

Choose a reason for hiding this comment

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

Balance. It also prevents people from walking out into space/unintended parts of the map. Its kinda required unless we map better.

ADD_TRAIT(owner, TRAIT_MUTE, STATUE_MUTE)
ADD_TRAIT(owner, TRAIT_IMMOBILIZED, MAGIC_TRAIT)
ADD_TRAIT(owner, TRAIT_HANDS_BLOCKED, MAGIC_TRAIT)
to_chat(owner, span_danger("This is a passive ability. The effects are already active!"))
Copy link
Member

Choose a reason for hiding this comment

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

might be better as just a toggle. otherwise we really need code to just not add true passives to the hud like this.

to_chat(owner, span_warning("Your bond with the nearby stone is interrupted!"))
exit_turf = null
return FALSE
owner.forceMove(stone_turf)
Copy link
Member

Choose a reason for hiding this comment

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

if code works u could step after u add the trait.

Comment on lines +125 to +126
var/turf/exit_turf
var/turf/stone_turf
Copy link
Member

Choose a reason for hiding this comment

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

with how these are used. they could both be weakrefed pretty easy

Comment on lines +74 to +77
/datum/storyteller_roll/scry_the_hearthstone
bumper_text = "scry the hearthstone"
difficulty = 6
applicable_stats = list(STAT_PERCEPTION, STAT_AWARENESS)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
/datum/storyteller_roll/scry_the_hearthstone
bumper_text = "scry the hearthstone"
difficulty = 6
applicable_stats = list(STAT_PERCEPTION, STAT_AWARENESS)
/datum/storyteller_roll/scry_the_hearthstone
bumper_text = "scry the hearthstone"
applicable_stats = list(STAT_PERCEPTION, STAT_AWARENESS)

"TRAIT_BLOODY_SUCKER" = TRAIT_BLOODY_SUCKER, // DARKPACK EDIT ADD
"TRAIT_BLOOD_DRINKER" = TRAIT_BLOOD_DRINKER, // DARKPACK EDIT ADD
"TRAIT_BLUSH_OF_HEALTH" = TRAIT_BLUSH_OF_HEALTH, // DARKPACK EDIT ADD
"TRAIT_BOND_WITHIN_THE_MOUNTAIN" = TRAIT_BOND_WITHIN_THE_MOUNTAIN, // DARKPACK EDIT ADD - Visceratika
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"TRAIT_BOND_WITHIN_THE_MOUNTAIN" = TRAIT_BOND_WITHIN_THE_MOUNTAIN, // DARKPACK EDIT ADD - Visceratika
"TRAIT_BOND_WITHIN_THE_MOUNTAIN" = TRAIT_BOND_WITHIN_THE_MOUNTAIN, // DARKPACK EDIT ADD - POWERS - (Visceratika)

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