search

eternaldensity / Sandcastle-Builder

xkcd: 1190: Time: The Game
Other
81 stars 66 forks source link

0 items of Infinite castle/NP tools #1316

Open Timeroot opened 9 years ago

Timeroot commented 9 years ago

See http://www.reddit.com/r/SandcastleBuilder/comments/32e784/why_am_i_getting_so_few_castles/ for a description of the problem. Beanie Builders were upgraded to Infinite castles/NP before they were even unlocked; since 0 were bought, they were producing NaN castles each ONG, which kept the game from progressing normally. This could theoretically have been fixed in the game by disabling Glass Ceiling 11, but this is very non-obvious -- tools that aren't even unlocked shouldn't be breaking the game. I suggest that Molpy.ONG only calls BuildPhase and DestroyPhase on the tools that are actually unlocked.

eternaldensity commented 9 years ago

The simple solution would be to check if 0 of a tool are owned, if so no need to calculate further.

LuminousLeopards commented 9 years ago

Very interesting that this never came up before. I put a pull request in that fixes it.

eternaldensity commented 9 years ago

Is good. Thanks!

My bad for not thinking of this back when I 'designed' it :P