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

[V12] Group Initiative Plain Doesn't Work #526

Closed
1 task done
NekohimeMusou opened this issue Jul 9, 2024 · 1 comment · Fixed by #534
Closed
1 task done

[V12] Group Initiative Plain Doesn't Work #526

NekohimeMusou opened this issue Jul 9, 2024 · 1 comment · Fixed by #534
Assignees
Labels
bug Something isn't working triage This bug still needs to be validated and weighted

Comments

@NekohimeMusou
Copy link
Contributor

What happened?

In short, grouped initiative is completely broken in v12 to the point where you can't begin a combat.

I don't know anyone actively using v12 to run an OSE campaign, so I could be missing something important or there could be workarounds for this and the individual init glitch (issue pending) I don't know of. If not, the combination of both problems is a bad enough show stopper that I think it's inaccurate to claim the system is v12 compatible.

This happens both on a copy of my current campaign world and in a completely fresh OSE world with no mods enabled.

Reproduction steps:

  1. Set Initiative to Grouped. This behavior occurs regardless of the Initiative Persistence setting. Mine was set to Keep while I did this.
  2. Create a new combat and add some PCs and monsters. Set their groups appropriately.
  3. Click the Begin Combat button.
  4. Initiative is (re)rolled automatically, regardless of whether it was already rolled.
  5. An error notification (pictured) appears, and combat does not begin.
    group-error-notif
  6. Repeat steps 3 through 5 as many times as you like; initiative is rerolled each time, but combat never begins

What is the expected behaviour?

When Begin Combat is clicked, combat should begin and the first combatant's turn should come up. Also, initiative should probably not be rerolled if it was already rolled before combat began, especially with Initiative Persistence set to Keep.

Relevant Errors and Warnings

Cannot assign to read only property '_id' of object '#<Combatant>'

TypeError: Cannot assign to read only property '_id' of object '#<Combatant>'
    at SchemaField._cleanType (foundry-esm.js:7847:20)
    at SchemaField.clean (foundry-esm.js:7342:19)
    at foundry-esm.js:8838:41
    at Array.map (<anonymous>)
    at EmbeddedCollectionField._cleanType (foundry-esm.js:8838:20)
    at EmbeddedCollectionField.clean (foundry-esm.js:7342:19)
    at SchemaField._cleanType (foundry-esm.js:7847:28)
    at SchemaField.clean (foundry-esm.js:7342:19)
    at Combat.cleanData (foundry-esm.js:10295:26)
    at Combat.validate (foundry-esm.js:10407:48)

Additional Support Details

Foundry Virtual Tabletop: Version 12 Stable, 12.328
Game System: ose, 1.10.3
Active Modules: 0
Performance Mode: 1

OS: Windows 10.0.0 x86 (64-bit)
Client: Not/A)Brand/8.0.0.0
GPU: ANGLE (Intel, Intel(R) UHD Graphics 630 (0x00003E9B) Direct3D11 vs_5_0 ps_5_0, D3D11)
Max Texture Size: 16384

Scene: 3840 x 1920 | Grid Size: 100 | Padding Percentage: 0
Walls: 0 | Ambient Lights: 5 | Ambient Sounds: 0 | Tiles: 1 | Tokens: 5
Largest Texture: 3840 x 1920

Actors: 4 | Items: 0 | Journal Entries: 0 | Rollable Tables: 0 |
Playlists: 0 | Compendium Packs: 0 | Chat Messages: 4

World Scripts:

Foundry VTT Core Version

None

Old-School Essentials Core Version

No response

Please update your browser regularly. Which (updated) browsers are you seeing the problem on?

Firefox, Chrome

Code of Conduct

  • I agree to follow this project's Code of Conduct
@NekohimeMusou NekohimeMusou added bug Something isn't working triage This bug still needs to be validated and weighted labels Jul 9, 2024
@PXR5
Copy link

PXR5 commented Aug 28, 2024

I confirm this is still an issue and is blocking me from upgrading to V12.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working triage This bug still needs to be validated and weighted
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants