Using a reset point no longer marks statistics as invalid.
authorLMH <lmh.0013@gmail.com>
Mon, 2 Sep 2013 05:54:20 +0000 (19:54 -1000)
committerTobias Markus <tobbi@mozilla-uk.org>
Sat, 26 Oct 2013 16:45:20 +0000 (18:45 +0200)
commit2c598e1731002d4766821a5c4ba7c611077ae9bc
tree1d2d698fd8866aa0d2eae87077f591f942a28b08
parent49689000be2ee8066eaa159bcbad4d29e457168e
Using a reset point no longer marks statistics as invalid.
The rationale for this change is as follows: stats should only be marked as invalid in cases of cheating, not normal game mechanics.  When a player uses a reset point, they typically gain no advantage over starting from the begining of a level since all of the collection stats are reset while time continues to count up.  Not displaying/recording statistics once a reset point is used has traditionally confused players (the fine print in one infoblock in the first level is easily missed/forgotten).  Also in the case of custom levels with incredible difficulty, stats are essentially never recorded making the levels even less fun.

In addition to this change, the fine print has been removed from the first level infoblock, and additional conditions were added to Statistics::merge to eliminate any stat that wanders above the total possible for the level.  Thus any programming mistake that allows stats to record values higher than possible (e.g. the dispenser issue recently fixed) are essentially hidden from players (and unfortunately harder for developers to find).
data/levels/world1/01 - Welcome to Antarctica.stl
src/supertux/game_session.cpp
src/supertux/statistics.cpp