From 8d34c71fb0f5f11e1f16ad11a83e715bc6af1dab Mon Sep 17 00:00:00 2001
From: Nathan Stocks <cleancut@github.com>
Date: Thu, 26 Nov 2020 14:28:58 -0700
Subject: [PATCH 1/3] Pause stops ticks. Consistent getter method names. Update
 tests.

---
 crates/bevy_core/src/time/timer.rs            | 66 +++++++++++--------
 .../src/print_diagnostics_plugin.rs           |  4 +-
 examples/2d/sprite_sheet.rs                   |  2 +-
 examples/app/plugin.rs                        |  2 +-
 examples/ecs/event.rs                         |  2 +-
 examples/ui/font_atlas_debug.rs               |  2 +-
 6 files changed, 46 insertions(+), 32 deletions(-)

diff --git a/crates/bevy_core/src/time/timer.rs b/crates/bevy_core/src/time/timer.rs
index efa2c3e41aec9..7deafeff0c8ac 100644
--- a/crates/bevy_core/src/time/timer.rs
+++ b/crates/bevy_core/src/time/timer.rs
@@ -41,12 +41,12 @@ impl Timer {
     }
 
     #[inline]
-    pub fn resume(&mut self) {
+    pub fn unpause(&mut self) {
         self.paused = false
     }
 
     #[inline]
-    pub fn is_paused(&self) -> bool {
+    pub fn paused(&self) -> bool {
         self.paused
     }
 
@@ -76,7 +76,7 @@ impl Timer {
     /// Non repeating timers will stop tracking and stay in the finished state until reset.
     /// Repeating timers will only be in the finished state on each tick `duration` is reached or exceeded, and can still be reset at any given point.
     #[inline]
-    pub fn is_finished(&self) -> bool {
+    pub fn finished(&self) -> bool {
         self.finished
     }
 
@@ -87,7 +87,7 @@ impl Timer {
     }
 
     #[inline]
-    pub fn is_repeating(&self) -> bool {
+    pub fn repeating(&self) -> bool {
         self.repeating
     }
 
@@ -98,6 +98,9 @@ impl Timer {
 
     /// Advances the timer by `delta` seconds.
     pub fn tick(&mut self, delta: f32) -> &Self {
+        if self.paused {
+            return self;
+        }
         let prev_finished = self.finished;
         self.elapsed += delta;
 
@@ -142,25 +145,36 @@ mod tests {
         let mut t = Timer::from_seconds(10.0, false);
         // Tick once, check all attributes
         t.tick(0.25);
-        assert_eq!(t.elapsed, 0.25);
-        assert_eq!(t.duration, 10.0);
-        assert_eq!(t.finished, false);
-        assert_eq!(t.just_finished, false);
-        assert_eq!(t.repeating, false);
+        assert_eq!(t.elapsed(), 0.25);
+        assert_eq!(t.duration(), 10.0);
+        assert_eq!(t.finished(), false);
+        assert_eq!(t.just_finished(), false);
+        assert_eq!(t.repeating(), false);
+        assert_eq!(t.percent(), 0.025);
+        assert_eq!(t.percent_left(), 0.975);
+        // Ticking while paused changes nothing
+        t.pause();
+        t.tick(500.0);
+        assert_eq!(t.elapsed(), 0.25);
+        assert_eq!(t.duration(), 10.0);
+        assert_eq!(t.finished(), false);
+        assert_eq!(t.just_finished(), false);
+        assert_eq!(t.repeating(), false);
         assert_eq!(t.percent(), 0.025);
         assert_eq!(t.percent_left(), 0.975);
         // Tick past the end and make sure elapsed doesn't go past 0.0 and other things update
+        t.unpause();
         t.tick(500.0);
-        assert_eq!(t.elapsed, 10.0);
-        assert_eq!(t.finished, true);
-        assert_eq!(t.just_finished, true);
+        assert_eq!(t.elapsed(), 10.0);
+        assert_eq!(t.finished(), true);
+        assert_eq!(t.just_finished(), true);
         assert_eq!(t.percent(), 1.0);
         assert_eq!(t.percent_left(), 0.0);
         // Continuing to tick when finished should only change just_finished
         t.tick(1.0);
-        assert_eq!(t.elapsed, 10.0);
-        assert_eq!(t.finished, true);
-        assert_eq!(t.just_finished, false);
+        assert_eq!(t.elapsed(), 10.0);
+        assert_eq!(t.finished(), true);
+        assert_eq!(t.just_finished(), false);
         assert_eq!(t.percent(), 1.0);
         assert_eq!(t.percent_left(), 0.0);
     }
@@ -170,25 +184,25 @@ mod tests {
         let mut t = Timer::from_seconds(2.0, true);
         // Tick once, check all attributes
         t.tick(0.75);
-        assert_eq!(t.elapsed, 0.75);
-        assert_eq!(t.duration, 2.0);
-        assert_eq!(t.finished, false);
-        assert_eq!(t.just_finished, false);
-        assert_eq!(t.repeating, true);
+        assert_eq!(t.elapsed(), 0.75);
+        assert_eq!(t.duration(), 2.0);
+        assert_eq!(t.finished(), false);
+        assert_eq!(t.just_finished(), false);
+        assert_eq!(t.repeating(), true);
         assert_eq!(t.percent(), 0.375);
         assert_eq!(t.percent_left(), 0.625);
         // Tick past the end and make sure elapsed wraps
         t.tick(1.5);
-        assert_eq!(t.elapsed, 0.25);
-        assert_eq!(t.finished, true);
-        assert_eq!(t.just_finished, true);
+        assert_eq!(t.elapsed(), 0.25);
+        assert_eq!(t.finished(), true);
+        assert_eq!(t.just_finished(), true);
         assert_eq!(t.percent(), 0.125);
         assert_eq!(t.percent_left(), 0.875);
         // Continuing to tick should turn off both finished & just_finished for repeating timers
         t.tick(1.0);
-        assert_eq!(t.elapsed, 1.25);
-        assert_eq!(t.finished, false);
-        assert_eq!(t.just_finished, false);
+        assert_eq!(t.elapsed(), 1.25);
+        assert_eq!(t.finished(), false);
+        assert_eq!(t.just_finished(), false);
         assert_eq!(t.percent(), 0.625);
         assert_eq!(t.percent_left(), 0.375);
     }
diff --git a/crates/bevy_diagnostic/src/print_diagnostics_plugin.rs b/crates/bevy_diagnostic/src/print_diagnostics_plugin.rs
index 0354bf581a0a8..2d9d077a78405 100644
--- a/crates/bevy_diagnostic/src/print_diagnostics_plugin.rs
+++ b/crates/bevy_diagnostic/src/print_diagnostics_plugin.rs
@@ -66,7 +66,7 @@ impl PrintDiagnosticsPlugin {
         time: Res<Time>,
         diagnostics: Res<Diagnostics>,
     ) {
-        if state.timer.tick(time.delta_seconds).is_finished() {
+        if state.timer.tick(time.delta_seconds).finished() {
             println!("Diagnostics:");
             println!("{}", "-".repeat(93));
             if let Some(ref filter) = state.filter {
@@ -86,7 +86,7 @@ impl PrintDiagnosticsPlugin {
         time: Res<Time>,
         diagnostics: Res<Diagnostics>,
     ) {
-        if state.timer.tick(time.delta_seconds).is_finished() {
+        if state.timer.tick(time.delta_seconds).finished() {
             println!("Diagnostics (Debug):");
             println!("{}", "-".repeat(93));
             if let Some(ref filter) = state.filter {
diff --git a/examples/2d/sprite_sheet.rs b/examples/2d/sprite_sheet.rs
index b2c241ea75841..05b0da5910ed7 100644
--- a/examples/2d/sprite_sheet.rs
+++ b/examples/2d/sprite_sheet.rs
@@ -13,7 +13,7 @@ fn animate_sprite_system(
     mut query: Query<(&mut Timer, &mut TextureAtlasSprite, &Handle<TextureAtlas>)>,
 ) {
     for (timer, mut sprite, texture_atlas_handle) in query.iter_mut() {
-        if timer.is_finished() {
+        if timer.finished() {
             let texture_atlas = texture_atlases.get(texture_atlas_handle).unwrap();
             sprite.index = ((sprite.index as usize + 1) % texture_atlas.textures.len()) as u32;
         }
diff --git a/examples/app/plugin.rs b/examples/app/plugin.rs
index 98c0d39469b16..d11b54f5d58a3 100644
--- a/examples/app/plugin.rs
+++ b/examples/app/plugin.rs
@@ -38,7 +38,7 @@ struct PrintMessageState {
 }
 
 fn print_message_system(mut state: ResMut<PrintMessageState>, time: Res<Time>) {
-    if state.timer.tick(time.delta_seconds).is_finished() {
+    if state.timer.tick(time.delta_seconds).finished() {
         println!("{}", state.message);
     }
 }
diff --git a/examples/ecs/event.rs b/examples/ecs/event.rs
index 44e63f2e2b878..c1ec889165dce 100644
--- a/examples/ecs/event.rs
+++ b/examples/ecs/event.rs
@@ -34,7 +34,7 @@ fn event_trigger_system(
     mut state: ResMut<EventTriggerState>,
     mut my_events: ResMut<Events<MyEvent>>,
 ) {
-    if state.event_timer.tick(time.delta_seconds).is_finished() {
+    if state.event_timer.tick(time.delta_seconds).finished() {
         my_events.send(MyEvent {
             message: "MyEvent just happened!".to_string(),
         });
diff --git a/examples/ui/font_atlas_debug.rs b/examples/ui/font_atlas_debug.rs
index faec75efdab76..d5694d1bcb129 100644
--- a/examples/ui/font_atlas_debug.rs
+++ b/examples/ui/font_atlas_debug.rs
@@ -62,7 +62,7 @@ fn atlas_render_system(
 }
 
 fn text_update_system(mut state: ResMut<State>, time: Res<Time>, mut query: Query<&mut Text>) {
-    if state.timer.tick(time.delta_seconds).is_finished() {
+    if state.timer.tick(time.delta_seconds).finished() {
         for mut text in query.iter_mut() {
             let c = rand::random::<u8>() as char;
             if !text.value.contains(c) {

From c8ebc21f141f0cfb9cf29588d611232f1c8b2a30 Mon Sep 17 00:00:00 2001
From: Amber Kowalski <amberkowalski03@gmail.com>
Date: Thu, 26 Nov 2020 23:56:36 -0600
Subject: [PATCH 2/3] Add timing example

---
 CHANGELOG.md                       |  3 +-
 Cargo.toml                         |  4 ++
 crates/bevy_core/src/time/timer.rs |  1 +
 examples/ecs/timers.rs             | 74 ++++++++++++++++++++++++++++++
 4 files changed, 81 insertions(+), 1 deletion(-)
 create mode 100644 examples/ecs/timers.rs

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 39da8e5e61283..05e7dbaaebd89 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -12,7 +12,8 @@ current changes on git with [previous release tags][git_tag_comparison].
 ### Added
 
 ### Changed
-
+- [Breaking changes to timer API][914]
+- [Removed timer auto-ticking system][931]
 ### Fixed
 
 
diff --git a/Cargo.toml b/Cargo.toml
index 26ff837bfc3e1..ee26789e927da 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -198,6 +198,10 @@ path = "examples/ecs/startup_system.rs"
 name = "system_chaining"
 path = "examples/ecs/system_chaining.rs"
 
+[[example]]
+name = "timers"
+path = "examples/ecs/timers.rs"
+
 [[example]]
 name = "ecs_guide"
 path = "examples/ecs/ecs_guide.rs"
diff --git a/crates/bevy_core/src/time/timer.rs b/crates/bevy_core/src/time/timer.rs
index 7deafeff0c8ac..b3f3fdd33edc6 100644
--- a/crates/bevy_core/src/time/timer.rs
+++ b/crates/bevy_core/src/time/timer.rs
@@ -106,6 +106,7 @@ impl Timer {
 
         self.finished = self.elapsed >= self.duration;
         self.just_finished = !prev_finished && self.finished;
+
         if self.finished {
             if self.repeating {
                 // Repeating timers wrap around
diff --git a/examples/ecs/timers.rs b/examples/ecs/timers.rs
new file mode 100644
index 0000000000000..e6b013337b4a5
--- /dev/null
+++ b/examples/ecs/timers.rs
@@ -0,0 +1,74 @@
+use bevy::log::info;
+use bevy::prelude::*;
+
+fn main() {
+    App::build()
+        .add_plugins(DefaultPlugins)
+        .add_resource(Countdown::default())
+        .add_startup_system(setup_system)
+        .add_system(countdown_system)
+        .add_system(timer_system)
+        .run();
+}
+
+pub struct Countdown {
+    pub percent_trigger: Timer,
+    pub main_timer: Timer,
+}
+
+impl Countdown {
+    pub fn new() -> Self {
+        Self {
+            percent_trigger: Timer::from_seconds(4.0, true),
+            main_timer: Timer::from_seconds(20.0, false),
+        }
+    }
+}
+
+impl Default for Countdown {
+    fn default() -> Self {
+        Self::new()
+    }
+}
+
+fn setup_system(commands: &mut Commands) {
+    // Add an entity to the world with a timer
+    commands.spawn((Timer::from_seconds(5.0, false),));
+}
+
+/// This system ticks all the `Timer` components on entities within the scene
+/// using bevy's `Time` resource to get the delta between each update.
+fn timer_system(time: Res<Time>, mut query: Query<&mut Timer>) {
+    for mut timer in query.iter_mut() {
+        if timer.tick(time.delta_seconds).just_finished() {
+            info!("Entity timer just finished")
+        }
+    }
+}
+
+/// This system controls ticking the timer within the countdown resource and
+/// handling its state.
+fn countdown_system(time: Res<Time>, mut countdown: ResMut<Countdown>) {
+    countdown.main_timer.tick(time.delta_seconds);
+
+    // The API encourages this kind of timer state checking (if you're only checking for one value)
+    // Additionally, `finished()` would accomplish the same thing as `just_finished` due to the timer
+    // being repeating, however this makes more sense visually.
+    if countdown
+        .percent_trigger
+        .tick(time.delta_seconds)
+        .just_finished()
+    {
+        if !countdown.main_timer.finished() {
+            // Print the percent complete the main timer is.
+            info!(
+                "Timer is {:0.0}% complete!",
+                countdown.main_timer.percent() * 100.0
+            );
+        } else {
+            // The timer has finished so we pause the percent output timer
+            countdown.percent_trigger.pause();
+            info!("Paused percent trigger timer")
+        }
+    }
+}

From 9160509dd127ff75100ab3251adf9ea3e1489693 Mon Sep 17 00:00:00 2001
From: Amber Kowalski <amberkowalski03@gmail.com>
Date: Fri, 27 Nov 2020 00:12:04 -0600
Subject: [PATCH 3/3] Format with the nightly formatter

---
 examples/ecs/timers.rs | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/examples/ecs/timers.rs b/examples/ecs/timers.rs
index e6b013337b4a5..22ffff1e6be4d 100644
--- a/examples/ecs/timers.rs
+++ b/examples/ecs/timers.rs
@@ -1,5 +1,4 @@
-use bevy::log::info;
-use bevy::prelude::*;
+use bevy::{log::info, prelude::*};
 
 fn main() {
     App::build()