sugarcube-3 icon indicating copy to clipboard operation
sugarcube-3 copied to clipboard

History functions/methods should not consider the current moment

Open tmedwards opened this issue 4 years ago • 1 comments

Idea: The various history functions/methods should not consider the current moment.

Reasoning

Currently, the various history functions/methods—e.g., the visited family of functions and State.hasPlayed()—include the current moment in their tests/collations. This has the effect that testing for the existence of a moment based on the current passage is impossible with most of them, as it'll will be topmost moment that they check. This requires the use of visited() in many case where you simply want to know if the player had been there before, which is less than ideal as it looks at every single moment in the past+expired history.

tmedwards avatar Jun 05 '20 10:06 tmedwards