fix: corregir ecualizador desincronizado, musica local en Android Auto y bloqueo del paywall
Tres fallos reportados en uso real, con sus causas raiz verificadas en codigo.
1. Ecualizador: el estado no tenia dueño unico
El handler arrancaba con `_ecualizadorActivo = true` a fuego. El valor
persistido solo llegaba por EstadoEcualizador.cargarPersistido(), alcanzable
unicamente desde el arbol de widgets, que un arranque headless de Android Auto
nunca construye. Resultado: el coche reproducia con el EQ forzado a ON mientras
disco e interfaz decian OFF.
Ahora registrarHandler siembra el flag desde disco en todos los motores y
setEcualizadorActivo persiste por su cuenta, asi que un toggle desde el coche o
la notificacion sobrevive sin EstadoEcualizador. _resincronizarConHandler pasa
a ser adopcion pura de interfaz.
Ademas mapearGananciaNativa enviaba 0 dB al punto MEDIO del rango nativo. Con
un getBandLevelRange() asimetrico, un preset plano metia varios dB de boost
real: la causa del "suena muy alto con el boton apagado". Reescrito para
escalar cada lado contra su propio extremo, de modo que 0 dB es siempre 0.
El dispatch de customAction no tenia ningun test. Se extrae decidirToggleEq y
se cubre contra el handler real. El efecto nativo se re-asierta al reactivarse
el reproductor, porque AudioEffect.setEnabled de just_audio es un no-op
mientras la plataforma esta desacoplada.
2. Musica Local no aparecia en el arbol de Android Auto
hayCarpetaConfigurada() consultaba MethodChannel('pluriwave/file_actions'),
registrado solo en MainActivity.configureFlutterEngine. Sin Activity no hay
handler, invokeMethod lanza MissingPluginException y el catch la confundia con
"permiso revocado", omitiendo el nodo. No dependia del entitlement.
La logica SAF sale a packages/pluriwave_file_actions, un paquete plugin local.
El motor headless que crea audio_service ejecuta GeneratedPluginRegistrant en
su constructor, asi que el canal queda registrado en ambos motores. Repuntar el
manifest a una subclase de AudioService no era viable: AudioServicePlugin
enlaza por ComponentName explicito y la app perderia el audio.
EstadoCarpetaLocal de tres valores separa "sin carpeta" de "canal no
disponible"; la raiz decide por la URI persistida y el subarbol muestra un item
explicativo en vez de una carpeta vacia. La invalidacion del arbol cacheado se
dispara al reanudar con el coche ya suscrito; el guardia anterior miraba
View.maybeOf, que bajo runApp siempre existe, por lo que se gastaba en el
arranque headless y no volvia a dispararse.
3. El paywall bloqueaba las compras
restorePurchases() de in_app_purchase_android emite siempre, y con lista vacia
si no hay nada que restaurar. El `if (compras.isEmpty) return;` se la tragaba,
noEncontrada nunca se emitia y la rama que limpia _compraEnCurso estaba muerta
en produccion. Como comprar y restaurar comparten ese flag, un usuario sin
compras que pulsaba restaurar se quedaba sin poder comprar.
Suite completa: 1366 pasan, 2 omitidos. 17 tests nuevos, todos nacidos rojos y
verificados por mutacion. flutter analyze mantiene los 5 avisos preexistentes.
This commit is contained in:
@@ -1,5 +1,6 @@
|
||||
import 'dart:collection';
|
||||
|
||||
import 'package:flutter/foundation.dart';
|
||||
import 'package:flutter/services.dart';
|
||||
import 'package:shared_preferences/shared_preferences.dart';
|
||||
|
||||
@@ -71,6 +72,24 @@ String nombreCarpetaDesdeUri(String treeUri, {required String nombreGenerico}) {
|
||||
return recortado.isEmpty ? nombreGenerico : recortado;
|
||||
}
|
||||
|
||||
/// Three-valued answer to «¿hay música local usable?»
|
||||
/// (fix/android-auto-musica-local).
|
||||
///
|
||||
/// Sustituye al `bool` anterior, que colapsaba dos causas MUY distintas en
|
||||
/// el mismo `false`:
|
||||
///
|
||||
/// * [noConfigurada] — no hay URI persistida, o el nativo respondió que el
|
||||
/// permiso ya no es válido (el usuario nunca eligió carpeta, o la
|
||||
/// revocó). Es la única respuesta que justifica ocultar el nodo.
|
||||
/// * [configurada] — hay URI persistida y el nativo confirma el permiso.
|
||||
/// * [canalNoDisponible] — hay URI persistida pero el canal
|
||||
/// `pluriwave/file_actions` NO tiene handler nativo, así que no se puede
|
||||
/// saber nada del permiso. Es lo que ocurre en el motor Flutter headless
|
||||
/// que `audio_service` levanta cuando Android Auto arranca la app sin
|
||||
/// Activity: `MainActivity.configureFlutterEngine` (único sitio donde se
|
||||
/// registra ese canal) nunca corre. NO significa «no hay carpeta».
|
||||
enum EstadoCarpetaLocal { noConfigurada, configurada, canalNoDisponible }
|
||||
|
||||
/// Browse-source abstraction for the local-music branch of the Android Auto
|
||||
/// tree (Design "Interfaces / Contracts"), mirroring [FuenteEmisorasAuto]'s
|
||||
/// (`navegacion_auto.dart`) cold-start-safe, never-throws contract. Kept as
|
||||
@@ -78,9 +97,11 @@ String nombreCarpetaDesdeUri(String treeUri, {required String nombreGenerico}) {
|
||||
/// browse domain, not a station source.
|
||||
abstract class FuenteMusicaLocalAuto {
|
||||
/// Whether a local-music root folder is picked AND its permission is
|
||||
/// still valid. Never throws — a revoked/never-granted permission
|
||||
/// degrades to `false` (Spec "Permission revoked or never granted").
|
||||
Future<bool> hayCarpetaConfigurada();
|
||||
/// still valid — o si esa pregunta no se puede contestar porque el canal
|
||||
/// nativo no existe en este motor. Never throws: cualquier fallo degrada
|
||||
/// a un valor de [EstadoCarpetaLocal], nunca a una excepción (Spec
|
||||
/// "Permission revoked or never granted").
|
||||
Future<EstadoCarpetaLocal> estadoCarpeta();
|
||||
|
||||
/// Immediate children of [documentId] (`''` = the tree root itself), one
|
||||
/// SAF level deep (Design "Lazy per-folder enumeration, never an eager
|
||||
@@ -197,18 +218,56 @@ class FuenteMusicaLocalAutoImpl implements FuenteMusicaLocalAuto {
|
||||
}
|
||||
|
||||
@override
|
||||
Future<bool> hayCarpetaConfigurada() async {
|
||||
Future<EstadoCarpetaLocal> estadoCarpeta() async {
|
||||
// Its OWN try, deliberately not merged with the channel one below.
|
||||
//
|
||||
// Never-throws restoration: the three-valued refactor moved this read
|
||||
// outside the try, and the only caller (`getChildren`'s root branch)
|
||||
// awaits it inline — so a prefs failure took the whole browse root down
|
||||
// and emptied the car, against this method's own interface doc.
|
||||
//
|
||||
// Kept SEPARATE because a prefs failure and a channel failure both
|
||||
// surface as `MissingPluginException`: one shared `on
|
||||
// MissingPluginException` clause would answer `canalNoDisponible` —
|
||||
// «hay carpeta pero no puedo comprobar el permiso» — for a store that
|
||||
// never told us whether a folder exists at all. That would put an
|
||||
// unreachable «Música Local» node in the car explaining a channel
|
||||
// problem that is not happening, which is precisely the collapse the
|
||||
// three-valued [EstadoCarpetaLocal] exists to prevent.
|
||||
//
|
||||
// `noConfigurada` is the honest answer here (the app cannot prove a
|
||||
// folder was ever picked) and is what this path returned before the
|
||||
// refactor, when the read still sat inside the catch-all below.
|
||||
final String? uri;
|
||||
try {
|
||||
uri = await _uriPersistida();
|
||||
} catch (e) {
|
||||
debugPrint('[PluriWave][musica_local] no se pudo leer la URI local: $e');
|
||||
return EstadoCarpetaLocal.noConfigurada;
|
||||
}
|
||||
if (uri == null || uri.isEmpty) return EstadoCarpetaLocal.noConfigurada;
|
||||
try {
|
||||
final uri = await _uriPersistida();
|
||||
if (uri == null || uri.isEmpty) return false;
|
||||
final valido = await _canal.invokeMethod<bool>('hasPersistedPermission', {
|
||||
'treeUri': uri,
|
||||
});
|
||||
return valido ?? false;
|
||||
} catch (_) {
|
||||
return valido == true
|
||||
? EstadoCarpetaLocal.configurada
|
||||
: EstadoCarpetaLocal.noConfigurada;
|
||||
} on MissingPluginException catch (e) {
|
||||
// El canal no tiene handler en ESTE motor. Antes esto caía en el
|
||||
// mismo `catch (_)` que un permiso revocado y devolvía `false`, que
|
||||
// es exactamente por lo que «Música Local» desaparecía del árbol de
|
||||
// Android Auto cuando el coche arrancaba la app sin Activity.
|
||||
debugPrint(
|
||||
'[PluriWave][musica_local] hasPersistedPermission sin handler '
|
||||
'nativo (motor sin Activity): $e',
|
||||
);
|
||||
return EstadoCarpetaLocal.canalNoDisponible;
|
||||
} catch (e) {
|
||||
// Cold-start / revoked-permission safety (Spec "Permission revoked or
|
||||
// never granted"): never throw, degrade to "not configured".
|
||||
return false;
|
||||
debugPrint('[PluriWave][musica_local] hasPersistedPermission ERROR $e');
|
||||
return EstadoCarpetaLocal.noConfigurada;
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user