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

who broke spell_data? #70832

Closed
GuardianDll opened this issue Jan 10, 2024 · 3 comments
Closed

who broke spell_data? #70832

GuardianDll opened this issue Jan 10, 2024 · 3 comments
Labels
(S2 - Confirmed) Bug that's been confirmed to exist

Comments

@GuardianDll
Copy link
Member

Describe the bug

tr_portal_dungeon_next_level, trap, used to advance you to the next level in portal dungeon, doesn't work anymore

Attach save file

DEBUGDEBUG-trimmed.tar.gz

Steps to reproduce

Debug spawn portal storm
teleport to blue vortex
step on it
nothing happens

Expected behavior

you teleport to green vortex

Screenshots

GIF 10-01-2024 22-22-37

Versions and configuration

  • OS: Windows
    • OS Version: 10.0.19045.3803 (22H2)
  • Game Version: 2a67568 [64-bit]
  • Graphics Version: Tiles
  • Game Language: English [en]
  • Mods loaded: [
    Dark Days Ahead [dda],
    Disable NPC Needs [no_npc_food],
    Slowdown Fungal Growth [no_fungal_growth],
    Bionic Professions [package_bionic_professions],
    SpeedyDex [speedydex],
    Stats Through Kills [stats_through_kills],
    Stats Through Skills [StatsThroughSkills],
    Magiclysm [magiclysm],
    Xedra Evolved [xedra_evolved]
    ]

Additional context

No response

@GuardianDll GuardianDll added the (S1 - Need confirmation) Report waiting on confirmation of reproducibility label Jan 10, 2024
@andrei8l
Copy link
Contributor

/confirmed broken by #70479

This check is flipped

diff --git a/src/map.cpp b/src/map.cpp
index 34f4a70cd1..3d0d83a2ac 100644
--- a/src/map.cpp
+++ b/src/map.cpp
@@ -9715,7 +9715,7 @@ void map::creature_on_trap( Creature &c, const bool may_avoid ) const
     // gliding or boarded in a vehicle means the player is above the trap
     // like a flying monster and can never trigger the trap.
     const Character *const you = c.as_character();
-    if( you != nullptr && ( you->in_vehicle || !you->has_effect( effect_gliding ) ) ) {
+    if( you != nullptr && ( you->in_vehicle || you->has_effect( effect_gliding ) ) ) {
         return;
     }
 

In addition, tr_at is wrong (though unrelated)

diff --git a/src/map.cpp b/src/map.cpp
index 34f4a70cd1..437547d95c 100644
--- a/src/map.cpp
+++ b/src/map.cpp
@@ -6099,7 +6099,7 @@ const trap &map::tr_at( const tripoint &p ) const
 
 const trap &map::tr_at( const tripoint_abs_ms &p ) const
 {
-    return tr_at( p.raw() );
+    return tr_at( bub_from_abs( p ) );
 }
 
 const trap &map::tr_at( const tripoint_bub_ms &p ) const

@fairyarmadillo

@github-actions github-actions bot added (S2 - Confirmed) Bug that's been confirmed to exist and removed (S1 - Need confirmation) Report waiting on confirmation of reproducibility labels Jan 10, 2024
@Yobrocharlie
Copy link

Traveling through the green vortex doesn’t work either fun fact your just stuck there

@GuardianDll
Copy link
Member Author

fixed by #70836

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
(S2 - Confirmed) Bug that's been confirmed to exist
Projects
None yet
Development

No branches or pull requests

3 participants