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

Legio specific entries (wargear and princep traits) appearing across maniples on multiple maniple lists #216

Open
zopha opened this issue Jan 22, 2022 · 1 comment
Labels
bug Something isn't working

Comments

@zopha
Copy link
Collaborator

zopha commented Jan 22, 2022

If I have two or more maniples with different Legios selected then princep traits and legio specific wargear appear in both maniples.

Showing two legio princep traits on one princep
image

Showing Mortis wargear on a Fortidus titan
image

@zopha zopha added the bug Something isn't working label Jan 22, 2022
@the-koetjeboe
Copy link
Collaborator

I think this is a similar issue I encountered with the Legio traits. Currently the Personal Traits are made visible like this:

image

The first condition doesn't actually work, Legio selection is not an ancestor to the Princeps (it is a descendant from a sibling that shares the ancestor specific maniple entry). The 2nd condition then causes the issue because that just means if any maniple in the battlegroup has the matching Legio, make it visible.
Afaik there is no way in the data to describe the actual selection we want to do, i.e. based on the status of a selection of a child of another descendant entry of an specific ancestor (you can only select the 'category' maniple, not the specific instance maniple i.e. arcus/firmus maniple). This is very closely related to the issue of # of princeps selection when you have multiple maniples which has a similar cause.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants