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

[1846] bug - able to token any slot in CHI using C&WI private #895

Closed
benjaminxscott opened this issue Jun 27, 2020 · 3 comments · Fixed by #947
Closed

[1846] bug - able to token any slot in CHI using C&WI private #895

benjaminxscott opened this issue Jun 27, 2020 · 3 comments · Fixed by #947
Labels
1846 bug Prevents game from being played correctly

Comments

@benjaminxscott
Copy link
Collaborator

benjaminxscott commented Jun 27, 2020

repro, B&O owns CWI and is about to teleport to CHI

B&O is able to token any open slot in CHI, rather than being limited to the southeast slot

Related: I wasn't able to reproduce #891 , since I was able to teleport to CHI by clicking the city slot

@tobymao tobymao added 1846 bug Prevents game from being played correctly labels Jun 27, 2020
@michaeljb michaeljb changed the title [1846] bug - able to token any slot in CHI using CW&I private [1846] bug - able to token any slot in CHI using C&WI private Jun 27, 2020
@michaeljb
Copy link
Collaborator

This fix will probably require changing some tile definitions.

These are the current definitions:

edge => city

yellow
0 => 0
3 => 1
4 => 2
5 => 3

the others:
0 => 3
3 => 0
4 => 1
5 => 2

A city option will likely need to be added to the token schema, and it will need to know which city the token can go in whether it's the yellow or green tile.

@benjaminxscott
Copy link
Collaborator Author

It looks like the private ability data structure knows which city it belongs in

perhaps an optional data member in the token ability that includes the city it's limited to?

@michaeljb
Copy link
Collaborator

michaeljb commented Jun 27, 2020

Yep, that's what I meant by adding a city option to the token schema.

The main complication is that in yellow, the C&WI's city is 3, while in green the C&WI's city is 0. The tile definitions need to be made consistent so that both are city 3 (which will break games, but that's fine since it's still alpha and we have the pinning feature).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1846 bug Prevents game from being played correctly
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants