Skip to content

[Corpses] Add corpse entity variables to DB#4911

Merged
Akkadius merged 3 commits intomasterfrom
trust/corpse_entity_variable_database
Jun 25, 2025
Merged

[Corpses] Add corpse entity variables to DB#4911
Akkadius merged 3 commits intomasterfrom
trust/corpse_entity_variable_database

Conversation

@fryguy503
Copy link
Copy Markdown
Contributor

Description

  • When corpses have entity variables add/remove/modified it will update the database accordingly.
  • Corpse loading will pull the database values if they exist.

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

Testing

image
image

Clients tested: RoF2

Checklist

  • I have tested my changes
  • I have performed a self-review of my code. Ensuring variables, functions and methods are named in a human-readable way, comments are added only where naming of variables, functions and methods can't give enough context.
  • I own the changes of my code and take responsibility for the potential issues that occur
  • If my changes make database schema changes, I have tested the changes on a local database (attach image). Updated version.h CURRENT_BINARY_DATABASE_VERSION to the new version. (Delete this if not applicable)

- When corpses have entity variables add/remove/modified it will update the database accordingly.
- Corpse loading will pull the database values if they exist.
@Akkadius Akkadius force-pushed the trust/corpse_entity_variable_database branch from 04f5686 to 68b5a6f Compare June 25, 2025 20:19
@Akkadius Akkadius merged commit 50ae0f8 into master Jun 25, 2025
0 of 2 checks passed
@Akkadius Akkadius deleted the trust/corpse_entity_variable_database branch June 25, 2025 20:21
@Akkadius Akkadius mentioned this pull request Jun 25, 2025
joligario added a commit to ProjectEQ/peqphpeditor that referenced this pull request Jun 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants