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] Create item via macro #126

Closed
highrockgames opened this issue Apr 18, 2024 · 7 comments
Closed

[BUG] Create item via macro #126

highrockgames opened this issue Apr 18, 2024 · 7 comments
Assignees
Labels
bug Something isn't working module compatibility It's about other module compatibility

Comments

@highrockgames
Copy link

v11.315
dnd5e 3.1.2
magicitems 4.1.4

There are two errors so far in my tests.

  1. Using item piles to add items to an actor.
    image

  2. Create item on actor using macro.
    image

Tested with only magicitems and item piles and #1 and #2 produced.
Tested with only magicitems and #2 produced.

Tested after applying magicitem field migration suggested in 4.1.4 release notes. No change.

This seems like a fault with the dnd5e system and magicitems than item piles and magicitems. There doesn't seem to be any loss in functionality. However, these errors are massive spam.

@highrockgames highrockgames added the bug Something isn't working label Apr 18, 2024
@PwQt PwQt self-assigned this Apr 18, 2024
@PwQt PwQt added the module compatibility It's about other module compatibility label Apr 18, 2024
@PwQt PwQt added this to the 4.1.5 milestone Apr 18, 2024
@PwQt PwQt added this to Magic Items Apr 18, 2024
@PwQt PwQt moved this to Task in Magic Items Apr 18, 2024
@PwQt PwQt modified the milestones: 4.1.5, 4.1.6 May 23, 2024
@PwQt
Copy link
Owner

PwQt commented Jun 23, 2024

@highrockgames do you have an example macro that would create an item?

@PwQt PwQt modified the milestones: 4.1.7, 4.1.8 Jun 25, 2024
@hollowphoton
Copy link

I ran into this too, here is a very short example macro you can use to test:

async function testAddItem() {
  //generate item
  var item = { 
    _id: 'NzDxIISxSXSyQltk',
    name: 'Thing',
    type: 'loot'
  };
  //run the function for all selected tokens
  canvas.tokens.controlled.forEach(function(token){
    //Update the token's inventory
    token.actor.createEmbeddedDocuments('Item', [item]);
  });
}

testAddItem();

@hollowphoton
Copy link

Can confirm this is an issue whether adding items via the default create function - or with itempiles.

@PwQt
Copy link
Owner

PwQt commented Jul 10, 2024

Thanks, will check that for testing

@PwQt
Copy link
Owner

PwQt commented Jul 25, 2024

@hollowphoton it seems like the macro you've provided seems to be working perfectly fine for me at FVTT v12.330 and D&D 3.3.1 with newest version, could you please check?

@hollowphoton
Copy link

I'm not ready to upgrade to v12 just yet, but appreciate the patch!

@PwQt
Copy link
Owner

PwQt commented Jul 26, 2024

Okay, then I'm closing the ticket, but not locking it - should it still occur on the latest versions, please write here again 😃 !

@PwQt PwQt closed this as not planned Won't fix, can't repro, duplicate, stale Jul 26, 2024
@github-project-automation github-project-automation bot moved this from Task to Done in Magic Items Jul 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working module compatibility It's about other module compatibility
Projects
Archived in project
Development

No branches or pull requests

3 participants