fix: alinear permisos, paywall y grabacion con lo que la app hace de verdad
Build & Deploy PluriWave / Análisis de código (push) Successful in 26s
Build & Deploy PluriWave / Build APK + AAB release (push) Successful in 3m13s

Revision previa al envio a produccion. Cada punto se verifico en el codigo
antes de tocarlo; lo que ya estaba bien se dejo como estaba.

Ubicacion: se declaraba precision fina sin usarla

El unico consumidor de ubicacion pide `LocationAccuracy.low` y se queda solo
con el codigo ISO del pais, asi que `ACCESS_FINE_LOCATION` no aportaba nada. Y
contradecia la declaracion de Seguridad de los datos ya aprobada en Play, que
dice ubicacion APROXIMADA: declarar una cosa y pedir otra es precisamente lo
que se penaliza en revision.

Verificado que los manifiestos de geolocator_android y geocoding_android no
declaran permisos propios, asi que el merge no lo reinyecta y no hace falta
`tools:node="remove"`. El plugin construye su peticion en tiempo de ejecucion a
partir de lo declarado, de modo que con COARSE pide COARSE. Sin cambio
funcional: la deteccion de pais sigue igual.

El paywall vendia Android Auto como exclusivo, y ya no lo es

La etiqueta era literalmente "Android Auto", a secas. Pero el tier gratuito
recibio una carpeta navegable con emisoras reproducibles cuando hubo que
cumplir las guias del coche, asi que esa frase dejo de ser cierta. Ahora dice
que PRO añade el catalogo completo, favoritos, mis emisoras y musica local, y
aclara que gratis tiene las destacadas. Un paywall que promete lo que el tier
gratuito ya tiene expone a reclamacion y a que se cite en revision.

Microfono: se pide al activar el visualizador, no antes

Con una explicacion previa en los 13 idiomas, en vez de aparecer sin contexto.

Grabacion: uso privado de verdad, no solo en el aviso

La pantalla de grabaciones entregaba el fichero a cualquier aplicacion con
`Share.shareXFiles`. La intencion era abrirlo en un reproductor del propio
telefono, no redistribuirlo, y una cosa es copia privada y la otra no. Ahora
usa el `openFile` que ya existia -- FileProvider + ACTION_VIEW -- y avisa
cuando ningun reproductor del dispositivo puede abrirla, en vez de fallar en
silencio. Se añade ademas el aviso de uso privado en esa pantalla.

`recordingActionShare` la usaban DOS botones con significados distintos: el de
grabaciones, que mandaba el audio, y el del reproductor, que comparte el nombre
y la url de la emisora. Una clave, dos sentidos, y esa ambiguedad basto para
que al leer el codigo pareciera que solo se compartian enlaces. Separadas en
`stationActionShare` y `recordingActionOpenIn`.

La grabacion sigue siendo PRO. Lo que reduce el riesgo es que la copia no salga
del dispositivo, no regalar la funcion: los anuncios tambien son monetizacion.

Suite completa: 1587 pasan, 2 omitidos. flutter analyze mantiene los 5 avisos
preexistentes.
This commit is contained in:
2026-09-18 17:06:59 +02:00
parent 69eea0f2a1
commit acf2ebb55f
52 changed files with 1631 additions and 265 deletions
@@ -0,0 +1,136 @@
import 'package:flutter/material.dart';
import 'package:provider/provider.dart';
import '../../estado/estado_visualizador.dart';
import '../../l10n/gen/app_localizations.dart';
import '../../widgets/pluri_glass_surface.dart';
import '../../widgets/pluri_layout.dart';
import '../../widgets/pluri_push_scaffold.dart';
/// AUDIO group · "Onda real del audio" — the point-of-intent opt-in for the
/// waveform visualizer's microphone permission.
///
/// Before this screen existed, `RECORD_AUDIO` was requested the moment
/// `VisualizadorAudio` subscribed to its native EventChannel, which the home
/// screen's "Escuchar" hero does on the user's FIRST play. A radio app that
/// pops "allow PluriWave to record audio?" the first time you press play is
/// asking for a sensitive permission with zero context, and Play expects
/// context.
///
/// Mirrors `PantallaAjustesSalidaAudio`'s point-of-intent shape (its
/// BLUETOOTH_CONNECT request sits behind the multi-device toggle the same
/// way), with one addition: the explanation is shown and accepted BEFORE the
/// flag flips, since flipping it is what triggers the system dialog.
class PantallaAjustesVisualizador extends StatelessWidget {
const PantallaAjustesVisualizador({super.key});
@override
Widget build(BuildContext context) {
final l10n = AppLocalizations.of(context);
return PluriPushScaffold(
title: l10n.visualizerRealWaveTitle,
body: ListView(
padding: PluriLayout.pageContentPadding,
children: const [_CuerpoVisualizador()],
),
);
}
}
class _CuerpoVisualizador extends StatelessWidget {
const _CuerpoVisualizador();
@override
Widget build(BuildContext context) {
final l10n = AppLocalizations.of(context);
final estado = context.watch<EstadoVisualizador>();
final habilitada = estado.ondaRealHabilitada;
return PluriGlassSurface(
child: Column(
crossAxisAlignment: CrossAxisAlignment.start,
children: [
// GestureDetector + custom row instead of SwitchListTile, for the
// same reason PantallaAjustesSalidaAudio does it: ListTile ink
// inside PluriGlassSurface's DecoratedBox trips a Material
// assertion.
Padding(
padding: const EdgeInsets.symmetric(vertical: 8),
child: Row(
children: [
Expanded(
child: GestureDetector(
behavior: HitTestBehavior.translucent,
onTap: () => _alternar(context, estado, !habilitada),
child: Column(
crossAxisAlignment: CrossAxisAlignment.start,
children: [
Text(
l10n.visualizerRealWaveTitle,
style: Theme.of(context).textTheme.bodyLarge,
),
const SizedBox(height: 2),
Text(
l10n.visualizerRealWaveSubtitle,
style: Theme.of(context).textTheme.bodySmall,
),
],
),
),
),
Switch.adaptive(
value: habilitada,
onChanged: (valor) => _alternar(context, estado, valor),
),
],
),
),
const SizedBox(height: 8),
// The same text the confirmation dialog shows, kept permanently on
// screen: a user who already granted the permission should be able
// to re-read what it is for without toggling anything.
Text(
l10n.visualizerRealWavePermissionExplanation,
style: Theme.of(context).textTheme.bodySmall,
),
],
),
);
}
/// Turning it OFF is immediate — withdrawing a permission must never be
/// harder than granting it. Turning it ON goes through the explanation
/// first, and only a deliberate confirmation flips the flag.
Future<void> _alternar(
BuildContext context,
EstadoVisualizador estado,
bool habilitada,
) async {
if (!habilitada) {
await estado.cambiarOndaReal(false);
return;
}
final l10n = AppLocalizations.of(context);
final confirmado = await showDialog<bool>(
context: context,
builder:
(ctx) => AlertDialog(
key: const ValueKey('visualizador-explicacion-permiso'),
title: Text(l10n.visualizerRealWaveTitle),
content: Text(l10n.visualizerRealWavePermissionExplanation),
actions: [
TextButton(
onPressed: () => Navigator.pop(ctx, false),
child: Text(l10n.cancelAction),
),
FilledButton(
onPressed: () => Navigator.pop(ctx, true),
child: Text(l10n.visualizerRealWaveEnableAction),
),
],
),
);
if (confirmado != true) return;
await estado.cambiarOndaReal(true);
}
}
+21
View File
@@ -4,6 +4,7 @@ import 'package:provider/provider.dart';
import '../estado/estado_ecualizador.dart';
import '../estado/estado_entitlement.dart';
import '../estado/estado_grabacion.dart';
import '../estado/estado_visualizador.dart';
import '../estado/estado_idioma.dart';
import '../estado/estado_radio.dart';
import '../l10n/display_names.dart';
@@ -27,6 +28,7 @@ import 'ajustes/pantalla_ajustes_musica_local.dart';
import 'ajustes/pantalla_ajustes_orden_listas.dart';
import 'ajustes/pantalla_ajustes_salida_audio.dart';
import 'ajustes/pantalla_ajustes_timer_sueno.dart';
import 'ajustes/pantalla_ajustes_visualizador.dart';
import 'ajustes/widgets/fila_ajuste.dart';
import 'pantalla_grabaciones.dart';
@@ -104,6 +106,9 @@ class _AjustesContent extends StatelessWidget {
final esPremium = context.select<EstadoEntitlement, bool>(
(e) => e.esPremium,
);
final ondaRealActiva = context.select<EstadoVisualizador, bool>(
(e) => e.ondaRealHabilitada,
);
return Column(
children: [
@@ -137,6 +142,22 @@ class _AjustesContent extends StatelessWidget {
(_) => const PantallaAjustesSalidaAudio(),
),
),
// Point-of-intent entry for the waveform visualizer's
// microphone opt-in. It is a nav row, not an inline switch,
// because the Settings root carries zero inline controls by
// design — the switch and its explanation live on the detail
// screen, which is also where the user reads what the
// permission is for before granting it.
FilaAjuste(
icon: Icons.graphic_eq_rounded,
titulo: l10n.visualizerRealWaveTitle,
valor: ondaRealActiva ? l10n.equalizerActive : null,
onTap:
() => PluriPushScaffold.push(
context,
(_) => const PantallaAjustesVisualizador(),
),
),
FilaAjuste(
icon: Icons.bedtime_rounded,
titulo: l10n.timerSectionTitle,
+53 -13
View File
@@ -3,7 +3,6 @@ import 'dart:async';
import 'package:flutter/material.dart';
import 'package:just_audio/just_audio.dart';
import 'package:provider/provider.dart';
import 'package:share_plus/share_plus.dart' show Share, XFile;
import '../estado/estado_grabacion.dart';
import '../l10n/gen/app_localizations.dart';
@@ -90,19 +89,24 @@ class _ReproductorGrabacionesJustAudio implements ReproductorGrabaciones {
/// WU15: the recordings library — storage usage, browsable rows with
/// inline playback, and a "⋮" menu constrained to exactly
/// Rename/Share/Delete (`recordings-library` spec). Distinct from
/// `PantallaAjustesGrabaciones` (WU3b), which is the folder/size-limit
/// Rename/Open-in-another-app/Delete (`recordings-library` spec). Distinct
/// from `PantallaAjustesGrabaciones` (WU3b), which is the folder/size-limit
/// SETTINGS screen, not this browsable file list.
class PantallaGrabaciones extends StatefulWidget {
const PantallaGrabaciones({
super.key,
ReproductorGrabaciones? reproductor,
Future<void> Function(String ruta)? compartir,
Future<bool> Function(String ruta)? abrirEnOtraApp,
}) : _reproductorInyectado = reproductor,
_compartirInyectado = compartir;
_abrirEnOtraAppInyectada = abrirEnOtraApp;
final ReproductorGrabaciones? _reproductorInyectado;
final Future<void> Function(String ruta)? _compartirInyectado;
/// Seam for the local-open action. Was `compartir`, which handed the audio
/// file to the system share sheet — redistribution of someone else's
/// broadcast. It now opens the file in a player already installed on THIS
/// device, and returns whether any app accepted it.
final Future<bool> Function(String ruta)? _abrirEnOtraAppInyectada;
@override
State<PantallaGrabaciones> createState() => _PantallaGrabacionesState();
@@ -111,8 +115,9 @@ class PantallaGrabaciones extends StatefulWidget {
class _PantallaGrabacionesState extends State<PantallaGrabaciones> {
late final ReproductorGrabaciones _reproductor =
widget._reproductorInyectado ?? _ReproductorGrabacionesJustAudio();
late final Future<void> Function(String ruta) _compartir =
widget._compartirInyectado ?? (ruta) => Share.shareXFiles([XFile(ruta)]);
late final Future<bool> Function(String ruta) _abrirEnOtraApp =
widget._abrirEnOtraAppInyectada ??
(ruta) => context.read<EstadoGrabacion>().abrirGrabacion(ruta);
late Future<List<ArchivoGrabacion>> _grabaciones;
final Map<String, Future<Duration?>> _duracionCache = {};
@@ -153,8 +158,8 @@ class _PantallaGrabacionesState extends State<PantallaGrabaciones> {
await _renombrar(archivo);
return;
}
if (accion == 'share') {
await _compartir(archivo.ruta);
if (accion == 'open') {
await _abrirLocalmente(archivo);
return;
}
if (accion == 'delete') {
@@ -162,6 +167,20 @@ class _PantallaGrabacionesState extends State<PantallaGrabaciones> {
}
}
/// Plays the user's own recording in another app on the same device. A
/// device with no audio viewer installed (and the native side's own
/// fallback to the containing folder failing too) returns `false` — the
/// action then says so instead of looking like a dead menu entry.
Future<void> _abrirLocalmente(ArchivoGrabacion archivo) async {
final messenger = ScaffoldMessenger.of(context);
final l10n = AppLocalizations.of(context);
final abierto = await _abrirEnOtraApp(archivo.ruta);
if (!mounted || abierto) return;
messenger.showSnackBar(
SnackBar(content: Text(l10n.recordingOpenNoAppError)),
);
}
Future<void> _renombrar(ArchivoGrabacion archivo) async {
final nuevoNombre = await showDialog<String>(
context: context,
@@ -286,6 +305,27 @@ class _PantallaGrabacionesState extends State<PantallaGrabaciones> {
),
],
),
// Production-readiness pass: recording a broadcast holds up as
// a private copy, and stops holding up the moment the product
// reads as a redistribution tool. The library had no such
// statement at all, while the manifest already publishes the
// recordings folder to the system file manager
// (RecordingsDocumentsProvider). Deliberately factual and
// low-key — a footnote, not a warning banner — and always
// visible, empty library included.
const SizedBox(height: 16),
Padding(
key: const ValueKey('grabaciones-aviso-uso-privado'),
padding: const EdgeInsets.symmetric(horizontal: 4),
child: Text(
l10n.recordingsPrivateUseNotice,
style: Theme.of(context).textTheme.bodySmall?.copyWith(
color: Theme.of(
context,
).textTheme.bodySmall?.color?.withValues(alpha: 0.7),
),
),
),
],
);
},
@@ -373,7 +413,7 @@ class _FilaGrabacion extends StatelessWidget {
// 44x44/radius-12 art placeholder (recordings carry no per-station
// favicon, so this is a themed fallback square, not invented artwork),
// name, meta line, a 24px play/pause affordance, and the SAME "-"
// menu (Rename/Share/Delete) as before, just restyled.
// menu (Rename/Open in another app/Delete) as before, just restyled.
return Padding(
padding: const EdgeInsets.symmetric(horizontal: 6, vertical: 10),
child: Row(
@@ -466,8 +506,8 @@ class _FilaGrabacion extends StatelessWidget {
child: Text(l10n.recordingActionRename),
),
PopupMenuItem(
value: 'share',
child: Text(l10n.recordingActionShare),
value: 'open',
child: Text(l10n.recordingActionOpenIn),
),
PopupMenuItem(
value: 'delete',
+9
View File
@@ -6,6 +6,7 @@ import 'package:shimmer/shimmer.dart' as shimmer;
import '../estado/estado_ecualizador.dart';
import '../estado/estado_navegacion.dart';
import '../estado/estado_radio.dart';
import '../estado/estado_visualizador.dart';
import '../l10n/display_names.dart';
import '../l10n/gen/app_localizations.dart';
import '../modelos/emisora.dart';
@@ -471,6 +472,14 @@ class _EscucharHero extends StatelessWidget {
// Audit 1.7 (t4 lines 66-68): 30 discrete
// bottom-anchored bars, not a continuous stroke.
barrasDiscretas: true,
// Sensitive-permission gate: subscribing to the
// native waveform channel is what makes Android ask
// for RECORD_AUDIO, so it happens only after the
// user opts in from Settings.
capturaRealHabilitada:
context
.watch<EstadoVisualizador>()
.ondaRealHabilitada,
),
],
),
+10 -1
View File
@@ -9,6 +9,7 @@ import 'package:shimmer/shimmer.dart';
import '../estado/estado_ecualizador.dart';
import '../estado/estado_grabacion.dart';
import '../estado/estado_radio.dart';
import '../estado/estado_visualizador.dart';
import '../l10n/gen/app_localizations.dart';
import '../modelos/emisora.dart';
import '../servicios/servicio_audio.dart';
@@ -185,6 +186,10 @@ class _PantallaReproductorState extends State<PantallaReproductor> {
color: tokens.warmCoral,
altura: 40,
barrasDiscretas: true,
// Sensitive-permission gate: see the same note on the
// Escuchar hero's visualizer in `pantalla_inicio.dart`.
capturaRealHabilitada:
context.watch<EstadoVisualizador>().ondaRealHabilitada,
gradienteFinAlpha: 0.45,
).pluriFadeIn(
context,
@@ -1133,7 +1138,11 @@ class _BandejaHerramientas extends StatelessWidget {
child: _TileHerramienta(
key: const Key('player-tool-share'),
icon: Icons.share_rounded,
label: l10n.recordingActionShare,
// Shares the STATION — its name and its stream url — never an
// audio file. This used to borrow `recordingActionShare`, the
// recordings library's own menu label, which made one key stand
// for two unrelated actions.
label: l10n.stationActionShare,
onTap: () => compartir('${emisora.nombre}\n${emisora.url}'),
),
),