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

Bug with the mod "Tanks and Other Vehicles mod" using the ID "turretframe" #39921

Closed
TheMrBakit opened this issue Apr 26, 2020 · 2 comments
Closed
Labels
(S4 - Invalid) wontfix / can't reproduce / notabug

Comments

@TheMrBakit
Copy link

Describe the bug

The issue is with the mod "Tank" there is some area that use the id "turretframe" and should be something else (my guess is "turret_mount").

Steps To Reproduce

Load a game with the tank mod (Tanks and Other Vehicles)

Expected behavior

Don't have the Error on loading the Tanks and Other Vehicles mod.

Versions and configuration

  • OS: Windows
    • OS Version: 10.0 1909
  • Game Version: 0.E-1220-g2876ece [64-bit]
  • Graphics Version: Tiles
  • Game Language: English [en]
  • Mods loaded: [
    Dark Days Ahead [dda],
    Craftable Gun Pack [craftgp],
    DeadLeaves' Fictional Guns [FIC_Weapons],
    Makeshift Items Mod [makeshift],
    Medieval and Historic Classes and Shields [Medieval_Stuff],
    More Survival Tools [More_Survival_Tools],
    Mythological Replicas [nw_pack],
    PKs Rebalancing [PK_REBALANCE],
    PRM++ Patch [noctpk],
    Cataclysm++ [Cata++],
    Beta National Guard Camp [national_guard_camp],
    More Locations [more_locations],
    Urban Development [Urban_Development],
    Folding Parts pack [deoxymod],
    Vehicle Additions Pack [blazemod],
    Tanks and Other Vehicles [Tanks],
    Necromancy [necromancy],
    No Fungal Monsters [No_Fungi],
    PK - No Doom Monsters [PK_No_Doom],
    PK - No Fungal Monsters [PK_No_Fungi],
    PK - No Triffids [PK_No_Triffids],
    Classes and Scenarios Mod [more_classes_scenarios],
    Classic Roguelike Classes [RL_Classes],
    Stats Through Skills [StatsThroughSkills],
    Manual Bionic Installation [manualbionicinstall],
    Disable NPC Needs [no_npc_food],
    Garden Pots [growable-pots],
    Hydroponics [hydroponics],
    Magiclysm [magiclysm],
    Aftershock [aftershock],
    Modular Turrets [modular_turrets],
    Salvaged Robots [Salvaged_Robots]
    ]

Additional context

DEBUG : Tried to get invalid vehicle part: turretframe

FUNCTION : const T& string_id::obj() const [with T = vpart_info]
FILE : src/veh_type.cpp
LINE : 150

@anothersimulacrum
Copy link
Member

This mod is no longer supported, and so this issue is invalid.
However, @Soup-de-Loop may be able to provide some sort of guidance for you on how to fix it, or otherwise.

@KorGgenT KorGgenT added the (S4 - Invalid) wontfix / can't reproduce / notabug label Apr 26, 2020
@Soup-de-Loop
Copy link
Contributor

Sorry about the late response: For context, known issue: it will not cause a segfault (unless you try to use the turretframe elsewhere) however, due to a smart cookie who added the vanilla turret mount to the tank vehicles at some point on the same slots as where the turretframes are. At most, it will throw an error about a deleted vehiclepart every time a military vehicle from Tankmod is spawned in worldgen- I will PR an already tried and true hotfix for this once I've finished modernising blazemod, it's low priority until I get to actually compartmentalising different features.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
(S4 - Invalid) wontfix / can't reproduce / notabug
Projects
None yet
Development

No branches or pull requests

4 participants