Fractal Softworks Forum

Please login or register.

Login with username, password and session length

Author Topic: Scuttling ships with SYSTEM/no_drop/no_drop_salvage weapons  (Read 1208 times)

Vayra

  • Admiral
  • *****
  • Posts: 627
  • jangala delenda est
    • View Profile
Scuttling ships with SYSTEM/no_drop/no_drop_salvage weapons
« on: February 05, 2021, 05:48:55 PM »

If you scuttle a ship with weapons that have the SYSTEM hint and the no_drop and no_drop_salvage tags in normal weapon slots, the weapons still show up in your inventory, which seems like potentially unintended behavior.

I've heard reports that you can also get them from some form of salvage but I haven't confirmed those myself, might be worth checking if you do decide to change this though. I suppose this is technically a (modded) report since there aren't any ships that do this in vanilla. ;)
Logged
Kadur Remnant: http://fractalsoftworks.com/forum/index.php?topic=6649
Vayra's Sector: http://fractalsoftworks.com/forum/index.php?topic=16058
Vayra's Ship Pack: http://fractalsoftworks.com/forum/index.php?topic=16059

im gonna push jangala into the sun i swear to god im gonna do it

Alex

  • Administrator
  • Admiral
  • *****
  • Posts: 24114
    • View Profile
Re: Scuttling ships with SYSTEM/no_drop/no_drop_salvage weapons
« Reply #1 on: February 05, 2021, 06:31:26 PM »

Hmm - you would presumably also be able to get at these weapons by uninstalling them in the refit screen, no? The problem seems to be rooted in those weapons ending up in those slots in the first place, unless I'm missing something.
Logged

Vayra

  • Admiral
  • *****
  • Posts: 627
  • jangala delenda est
    • View Profile
Re: Scuttling ships with SYSTEM/no_drop/no_drop_salvage weapons
« Reply #2 on: February 06, 2021, 07:28:09 PM »

Hmm - you would presumably also be able to get at these weapons by uninstalling them in the refit screen, no? The problem seems to be rooted in those weapons ending up in those slots in the first place, unless I'm missing something.

Oh, true - I remove them from the inventory immediately with a hullmod script in that case. I'll find another way around it then, thanks for your time!
Logged
Kadur Remnant: http://fractalsoftworks.com/forum/index.php?topic=6649
Vayra's Sector: http://fractalsoftworks.com/forum/index.php?topic=16058
Vayra's Ship Pack: http://fractalsoftworks.com/forum/index.php?topic=16059

im gonna push jangala into the sun i swear to god im gonna do it

Tartiflette

  • Admiral
  • *****
  • Posts: 3529
  • MagicLab discord: https://discord.gg/EVQZaD3naU
    • View Profile
Re: Scuttling ships with SYSTEM/no_drop/no_drop_salvage weapons
« Reply #3 on: February 06, 2021, 11:53:33 PM »

You have the same issue with buit-in weapons if they are assigned in the variant instead of the hull file.

I have a ship with a hullmod allowing to cycle between 3 "built-in" loadouts in refit, and those built-in, SYSTEM, no_drop, no_drop_salvage weapons still show up in post battle loot.

Speaking of tags, no_drop_salvage, no_sell weapons can still show up in research stations salvage, which is inconvenient when making bosses with fancy weapons that should only be available by defeating them and looting their corpse.
Logged
 

Alex

  • Administrator
  • Admiral
  • *****
  • Posts: 24114
    • View Profile
Re: Scuttling ships with SYSTEM/no_drop/no_drop_salvage weapons
« Reply #4 on: February 07, 2021, 08:33:46 AM »

I have a ship with a hullmod allowing to cycle between 3 "built-in" loadouts in refit, and those built-in, SYSTEM, no_drop, no_drop_salvage weapons still show up in post battle loot.

Hmm - that *should* be fixed. Though, actually... I'm fairly sure the Tags.NO_DROP check for not dropping weapons is in the current release, so this is kind of confusing. Could you see if that's used in the latest version of FleetEncounterContext? The SYSTEM hint doesn't factor in here iirc, it's just the no_drop tag.


Speaking of tags, no_drop_salvage, no_sell weapons can still show up in research stations salvage, which is inconvenient when making bosses with fancy weapons that should only be available by defeating them and looting their corpse.

That should be fixed for the next release. Or, rather - it'll need a "no_drop" tag. There's also a new "restricted" tag that's more suitable for things like this which iirc also makes them not show up in mission refit.
Logged

Tartiflette

  • Admiral
  • *****
  • Posts: 3529
  • MagicLab discord: https://discord.gg/EVQZaD3naU
    • View Profile
Re: Scuttling ships with SYSTEM/no_drop/no_drop_salvage weapons
« Reply #5 on: February 07, 2021, 01:19:05 PM »

Mmmh, there seems to be a if (spec.hasTag(Tags.NO_DROP)) continue; in FleetEncouterContext in the current version yet the weapon drops. Small correction, the weapons I'm talking about are in a built-in slot (to prevent players from uninstalling them), so they shouldn't drop at all.

Now thinking about it, in the second case if I add a SYSTEM hint to the no_drop_salvage and no_sell of the boss weapons, they should drop in post battle salvage, but no longer appear in research stations I think.
Logged
 

Alex

  • Administrator
  • Admiral
  • *****
  • Posts: 24114
    • View Profile
Re: Scuttling ships with SYSTEM/no_drop/no_drop_salvage weapons
« Reply #6 on: February 07, 2021, 01:35:54 PM »

Yeah, they really shouldn't, hmm. I do remember making some fixes here, so it's possible that this will be resolved. If you're able to cook up a minimal test case, I'll be happy to give it a try to confirm the fix.
Logged

Tartiflette

  • Admiral
  • *****
  • Posts: 3529
  • MagicLab discord: https://discord.gg/EVQZaD3naU
    • View Profile
Re: Scuttling ships with SYSTEM/no_drop/no_drop_salvage weapons
« Reply #7 on: February 08, 2021, 12:59:08 AM »

Alright, I'll try to do that.
Logged