player: add factor out redundant condition check

The addition of a "need_to_queue" method should reduce the amount of
cognitive overhead required to parse these conditions

1 files changed, 2 insertions, 4 deletions

diff --git a/lib/dtas/player/client_handler.rb b/lib/dtas/player/client_handler.rb
index 5c6653c..75e490d 100644
--- a/lib/dtas/player/client_handler.rb
+++ b/lib/dtas/player/client_handler.rb
@@ -552,7 +552,7 @@ module DTAS::Player::ClientHandler # :nodoc:
       _tl_skip if set_as_current
 
       # start playing if we're the only track
-      if @tl.size == 1 && !(@current || @queue[0] || @paused)
+      if @tl.size == 1 && ! need_to_queue
         next_source(_next)
       end
       io.emit("#{track_id}")
@@ -589,9 +589,7 @@ module DTAS::Player::ClientHandler # :nodoc:
       offset = msg.shift # may be nil
       if @tl.go_to(track_id.to_i, offset)
         _tl_skip
-        if !(@current || @queue[0] || @paused)
-          next_source(_next)
-        end
+        next_source(_next) unless need_to_queue
         io.emit("OK")
       else
         io.emit("MISSING")