fix(radio): honour the "enable radio" switch when tuning

The switch added in 2.1.0 only governed the wait/play rows the editor
appends after the tune action; the tune itself went through
SaicRadio.tune, which called the vendor's srcPlayRadio every time, so the
radio took over the speakers with the switch off.

EVHardware 1.1.0 splits tuning from playback, and the executor now starts
playback only when Action.flag is set. The check lives in the executor
rather than only in ActionBundles because rules saved before 2.1.0 carry
the flag but no wait/play rows; playing twice for rules that do have them
is a no-op, the source is already the radio.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Author: malys

CHANGELOG.md

@@ -4,6 +4,19 @@ All notable changes to this project are documented here. Format follows
 [Keep a Changelog](https://keepachangelog.com/en/1.1.0/); versions follow
 [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [2.1.2] - 2026-08-22
+
+### Fixed
+
+- **"Tune and play radio" started the radio even with the switch turned off.** The switch
+  added in 2.1.0 only controlled the wait/play rows the editor appends after the tune
+  action; the tune itself went through `SaicRadio.tune`, which called the vendor's
+  `srcPlayRadio` on every tune and made the radio the current audio source regardless. A
+  rule meant to preset a station while music kept playing took over the speakers instead.
+  Tuning and playback are now separate vendor calls, and the action starts playback only
+  when the switch is on. Rules saved before 2.1.0, which carry no wait/play rows, keep
+  playing as they always did.
+
 ## [2.1.1] - 2026-08-21
 
 ### Fixed

EVHardware

@@ -10,8 +10,8 @@ android {
         applicationId = "com.evsuite.tasker"
         minSdk = 28
         targetSdk = 34
-        versionCode = 5
-        versionName = "2.1.1"
+        versionCode = 6
+        versionName = "2.1.2"
     }
 
     // EVTasker doit être signé avec la MÊME clé plateforme que EVProfile : c'est la seule

app/build.gradle.kts

@@ -181,11 +181,16 @@ class DirectExecutor(
      * A frequency the driver typed. Text that names no station is unsupported, not an error:
      * retrying "FM 250" three times with backoff will not make it a station, and the history
      * showing what was typed is what lets the user find the typo.
+     *
+     * Playback follows [Action.flag], the editor's "enable radio" switch, even though
+     * [com.evsuite.tasker.ui.ActionBundles] also appends a play tail when it is on: rules
+     * saved before that tail existed carry the flag but no tail, and would otherwise tune in
+     * silence. Playing twice is a no-op — the source is already the radio.
      */
     private fun tuneRadio(a: Action): ActionResult {
         val station = RadioFrequency.parse(a.text)
             ?: return ActionResult(a.type, false, BridgeContract.VERDICT_UNSUPPORTED, "not a frequency: ${a.text}")
-        val ok = SaicRadio.tune(station.band, station.frequencyKhz)
+        val ok = SaicRadio.tune(station.band, station.frequencyKhz) && (!a.flag || SaicRadio.play())
         return ActionResult(
             a.type,
             ok,