foundryvtt-lancer icon indicating copy to clipboard operation
foundryvtt-lancer copied to clipboard

[BUG] Integrated Weapons Can't be Modified

Open msprijatelj opened this issue 3 years ago • 4 comments

Describe the bug Integrated weapons on mechs that have a new frame assigned to them can't be modified (e.g. change Loaded, edit stats, etc.), as if it were directly referencing the compendium item.

To Reproduce Steps to reproduce the behavior:

  1. Create a new mech
  2. Assign a Frame with an integrated weapon (e.g. Barbarossa, Raleigh)
  3. Attempt to modify the integrated weapon (e.g. Mark as Destroyed, even use the Full Repair macro)
  4. Foundry throws an error: You may not modify the world.mech_weapon Compendium which is currently locked.
  5. Attempt to Edit the integrated weapon
  6. Observe the options to change are greyed out and locked (and "Import" is in the top right corner)

Expected behavior Adding a frame to a mech creates an instance of the integrated weapon, not a reference to the Compendium item (if the Compendium item is subsequently unlocked somehow, modifying it from a Mech sheet also seems undesirable).

Screenshots Attempting to Edit the M35 Mjolnir: image

Platform (please complete the following information):

  • OS: Windows
  • Foundry version 0.8.9
  • Lancer system version 1.0.3

Additional context This seems to also cause the Full Repair macro fail, as it's probably trying to modify the integrated weapon.

EDIT: This error doesn't occur when drag-and-dropping a weapon from the compendium; in that case, it instantiates the weapon as expected.

Discord contact info Valkyrion#6196

msprijatelj avatar Sep 23 '21 14:09 msprijatelj

This appears to be fixed now.

Eranziel avatar Jan 16 '22 06:01 Eranziel

@Eranziel After the update to Lancer v1.1.0, I'm seeing that this error still happens when directly dragging the Frame from the Compendium. An instantiated weapon still appears in the mech inventory, with its own state, but the auto-equipped integrated weapon is still compendium-locked.

The good news is that it's equipping correctly when importing mechs directly from CompCon; I imported a Raleigh without this issue on its Mjolnir.

EDIT: For more info, this is happening AFTER I cleared and re-installed the Core Lancer Compendium; the duplicate Tags issue has disappeared, indicating I'm using the newest compilation of the compendium from Machine-Mind.

msprijatelj avatar Jan 26 '22 16:01 msprijatelj

Okay, I'll reopen. I believe I tested by importing a pilot with a relevant frame (Caliban), but I didn't try dragging the frame directly.

Eranziel avatar Jan 26 '22 16:01 Eranziel

I just tested this by dragging the frame directly onto the sheet. It's fixed there too.

OS: Windows 10 Browser: Chrome Foundry version: 11.315 Lancer system version: 2.0.0

Davis-K-Johnson avatar Mar 11 '24 15:03 Davis-K-Johnson