Skip to content

DATA-4392: Fix Aasimar Scion of Humanity feat prerequisites - #7722

Closed
cpstence wants to merge 0 commit into
PCGen:masterfrom
cpstence:DATA-4392-aasimar-humanity
Closed

DATA-4392: Fix Aasimar Scion of Humanity feat prerequisites#7722
cpstence wants to merge 0 commit into
PCGen:masterfrom
cpstence:DATA-4392-aasimar-humanity

Conversation

@cpstence

@cpstence cpstence commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Resolves Jira Ticket: DATA-4392

Issue:
Aasimars with the "Scion of Humanity" alternate racial trait were not qualifying for human-specific feats (such as Defiant Luck), despite the trait explicitly stating they count as human for feat prerequisites.

Fix:
Appended the RACESUBTYPE:Human tag to the Scion of Humanity entry in arg_abilities_race.lst. This ensures the PCGen engine properly flags the character with the human subtype, satisfying the PRE requirements for human-only feats and effects.

@cpstence
cpstence force-pushed the DATA-4392-aasimar-humanity branch from adb9ac4 to 865c4b8 Compare August 31, 2026 19:51
@Vest
Vest requested a review from LegacyKing August 31, 2026 21:25
@Vest Vest added the bug label Aug 31, 2026
@Vest

Vest commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

@cpstence your fix broke the test. Please verify it.

@Vest
Vest marked this pull request as draft September 1, 2026 08:35
@Vest
Vest marked this pull request as ready for review September 1, 2026 08:47
@cpstence cpstence closed this Sep 1, 2026
@cpstence
cpstence force-pushed the DATA-4392-aasimar-humanity branch from 3f31eed to a9bbc17 Compare September 1, 2026 14:53
@cpstence

cpstence commented Sep 1, 2026

Copy link
Copy Markdown
Contributor Author

The XSLT modifications from my other ticket accidentally hitched a ride onto this PR. I’ve just force-pushed a clean update to this branch. It removes the XSLT files entirely, removes the illegal RACESUBTYPE token that was breaking the test, and broadens the prerequisites in arg_feats.lst instead. The datatest is passing locally and this PR is now 100% isolated to just the Aasimar fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants