Merge pull request #2 from isunican/feature/506089-FiltrarGasolineras…

…PorMunicipios PullRequest - Feature/506089 filtrar gasolineras por municipios
isunican · Nov 7, 2024 · 56955a5 · 56955a5
2 parents df8316d + 6dabadc
commit 56955a5
Show file tree

Hide file tree

Showing 7 changed files with 79 additions and 25 deletions.
diff --git a/...ject/app/src/main/java/es/unican/gasolineras/activities/main/GasolinerasArrayAdapter.java b/...ject/app/src/main/java/es/unican/gasolineras/activities/main/GasolinerasArrayAdapter.java
@@ -84,7 +84,7 @@ public View getView(int position, @Nullable View convertView, @NonNull ViewGroup
             addressTextView.setTextColor(Color.RED);               // Cambiar color a rojo
             addressTextView.setGravity(Gravity.CENTER);            // Centrar el texto
             addressTextView.setTextAlignment(View.TEXT_ALIGNMENT_CENTER);
-            addressTextView.setTextSize(20);
+            addressTextView.setTextSize(18);
             addressTextView.setMaxLines(2);                         // Limitar a dos líneas
             addressTextView.setEllipsize(TextUtils.TruncateAt.END); // Elipsis si el texto es muy largo
             addressTextView.setLineSpacing(0, 1.2f);

diff --git a/AndroidProject/app/src/main/java/es/unican/gasolineras/activities/main/MainPresenter.java b/AndroidProject/app/src/main/java/es/unican/gasolineras/activities/main/MainPresenter.java
@@ -72,10 +72,12 @@ public void onMenuConsultarClicked() {
      */
     @Override
     public void onMenuDescuentoClicked()  { view.showDescuentoActivity();}
+
     /**
      * @see IMainContract.Presenter#onBtnFiltrarClicked(String)
      * @param municipio el municipio a aplicar como filtro
      */
+    @Override
     public void onBtnFiltrarClicked(String municipio) {
 
 
@@ -89,7 +91,7 @@ public void onBtnFiltrarClicked(String municipio) {
 
         if (listaFiltrada.isEmpty()) {
 
-            view.mostrarErrorNoGasolinerasEnMunicipio("Error: No exiten gasolineras con \n el filtro aplicado");
+            view.mostrarErrorNoGasolinerasEnMunicipio("Error: No exiten gasolineras \n con el filtro aplicado");
             return;
         }
 
@@ -139,12 +141,20 @@ public void onFailure(Throwable e) {
         repository.requestGasolineras(callBack, IDCCAAs.CANTABRIA.id);
     }
 
+    /**
+     * Activa el filtro sobre un municipio dado
+     * @param municipio municipio sobre el que activar el filtro
+     * @return el municipio sobre el que se ha filtrado
+     */
     public String activarFiltro(String municipio) {
-
         filtroActivado = true;
         return municipio;
     }
 
+    /**
+     * Comprueba si hay un filtro activado
+     * @return el filtro que hay activado o null si no hay ninguno
+     */
     public String hayFiltroActivado() {
         if (filtroActivado) {
             return filtroActual;

diff --git a/AndroidProject/app/src/main/java/es/unican/gasolineras/activities/main/MainView.java b/AndroidProject/app/src/main/java/es/unican/gasolineras/activities/main/MainView.java
@@ -262,7 +262,7 @@ public void showDescuentoActivity() {
      */
     @Override
     public void mostrarErrorNoGasolinerasEnMunicipio(String mensajeError) {
-        DescuentoDAO descuentoDAO = null;
+        DescuentoDAO descuentoDAO = db.descuentoDao();
         // Crear una lista temporal que contenga solo el mensaje de error
         List<Gasolinera> emptyMessage = new ArrayList<>();
         Gasolinera gasolinera = new Gasolinera();

diff --git a/AndroidProject/app/src/main/res/layout/activity_filtrar.xml b/AndroidProject/app/src/main/res/layout/activity_filtrar.xml
@@ -1,4 +1,9 @@
 <?xml version="1.0" encoding="utf-8"?>
+<ScrollView xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:app="http://schemas.android.com/apk/res-auto"
+    xmlns:tools="http://schemas.android.com/tools"
+    android:layout_width="match_parent"
+    android:layout_height="match_parent">
 <LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
     android:orientation="vertical"
     android:padding="40dp"
@@ -37,3 +42,4 @@
         android:layout_marginTop="20dp" />
 
 </LinearLayout>
+</ScrollView>
diff --git a/AndroidProject/app/src/main/res/values/strings.xml b/AndroidProject/app/src/main/res/values/strings.xml
@@ -28,69 +28,107 @@
     <string-array name="municipiosArray">
         <item>Mostrar todos</item>
         <item>Alfoz de Lloredo</item>
-        <item>Alto de la Pasa</item>
         <item>Ampuero</item>
         <item>Anievas</item>
+        <item>Arenas de Iguña</item>
         <item>Argoños</item>
         <item>Arnuero</item>
+        <item>Arredondo</item>
         <item>Bárcena de Cicero</item>
         <item>Bárcena de Pie de Concha</item>
-        <item>Barros</item>
         <item>Bareyo</item>
+        <item>Cabezon de la Sal</item>
+        <item>Cabezón de Liebana</item>
+        <item>Cabuerniga</item>
+        <item>Camaleño</item>
+        <item>Camargo</item>
+        <item>Campoo de Enmedio</item>
+        <item>Campoo de Yuso</item>
         <item>Cartes</item>
         <item>Castañeda</item>
+        <item>Castro Urdiales</item>
         <item>Cieza</item>
+        <item>Cillorigo de Liebana</item>
+        <item>Colindres</item>
         <item>Comillas</item>
         <item>Corvera de Toranzo</item>
-        <item>Cotorredondo</item>
+        <item>El Astillero</item>
         <item>Entrambasaguas</item>
         <item>Escalante</item>
-        <item>Escobedo</item>
-        <item>Gajano</item>
+        <item>Guriezo</item>
         <item>Hazas de Cesto</item>
-        <item>Hoznayo</item>
-        <item>Liérganes</item>
+        <item>Hermandad de Campoo de Suso</item>
+        <item>Herrerias</item>
+        <item>Lamason</item>
+        <item>Laredo</item>
+        <item>Las Rozas de Valdearroyo</item>
         <item>Liendo</item>
+        <item>Lierganes</item>
         <item>Limpias</item>
-        <item>Liendo</item>
-        <item>Mares</item>
-        <item>Miengo</item>
+        <item>Los Corrales de Buelna</item>
+        <item>Los Tojos</item>
+        <item>Luena</item>
+        <item>Marina de Cudeyo</item>
         <item>Mazcuerras</item>
+        <item>Medio Cudeyo</item>
         <item>Meruelo</item>
+        <item>Miengo</item>
         <item>Miera</item>
-        <item>Minería</item>
-        <item>Mogro</item>
         <item>Molledo</item>
-        <item>Peñagos</item>
+        <item>Noja</item>
+        <item>Peñarubia</item>
+        <item>Penagos</item>
+        <item>Pesaguero</item>
         <item>Pesquera</item>
         <item>Piélagos</item>
+        <item>Polaciones</item>
+        <item>Polanco</item>
         <item>Potes</item>
+        <item>Puente Viesgo</item>
+        <item>Ramales de la Victoria</item>
+        <item>Rasines</item>
         <item>Reinosa</item>
-        <item>Riaño</item>
+        <item>Reocin</item>
         <item>Ribamontán al Mar</item>
         <item>Ribamontán al Monte</item>
-        <item>Riva de Coto</item>
+        <item>Rionansa</item>
+        <item>Riotuerto</item>
+        <item>Ruente</item>
         <item>Ruesga</item>
+        <item>Ruiloba</item>
         <item>San Felices de Buelna</item>
+        <item>San Miguel de Aguayo</item>
+        <item>San Pedro del Romeral</item>
+        <item>San Roque de Riomiera</item>
         <item>San Vicente de la Barquera</item>
+        <item>Santa Cruz de Bezana</item>
         <item>Santa María de Cayón</item>
         <item>Santander</item>
+        <item>Santillana del Mar</item>
+        <item>Santiurde de Reinosa</item>
         <item>Santiurde de Toranzo</item>
+        <item>Santoña</item>
+        <item>Saro</item>
         <item>Selaya</item>
         <item>Soba</item>
         <item>Solórzano</item>
+        <item>Suances</item>
         <item>Torrelavega</item>
+        <item>Tresviso</item>
+        <item>Tudanca</item>
+        <item>Udias</item>
         <item>Val de San Vicente</item>
         <item>Valdáliga</item>
         <item>Valdeolea</item>
         <item>Valdeprado del Río</item>
+        <item>Valderredible</item>
+        <item>Valle de Villaverde</item>
         <item>Vega de Liébana</item>
-        <item>Voto</item>
-        <item>Valles Pasiegos</item>
+        <item>Vega de Pas</item>
         <item>Villacarriedo</item>
+        <item>Villaescusa</item>
         <item>Villafufre</item>
-        <item>Villaverde de Pontones</item>
-        <item>Val de Villaverde</item>
+        <item>Voto</item>
     </string-array>
 
 

diff --git a/...idProject/app/src/test/java/es/unican/gasolineras/activities/main/MainPresenterITest.java b/...idProject/app/src/test/java/es/unican/gasolineras/activities/main/MainPresenterITest.java
@@ -158,7 +158,7 @@ public void ITestFiltrarPorMunicipioSinGasolineras() {
         sut.onBtnFiltrarClicked("Bareyo");
 
         verify(mockMainView).getGasolinerasRepository();
-        verify(mockMainView).mostrarErrorNoGasolinerasEnMunicipio("Error: No exiten gasolineras con \n el filtro aplicado");
+        verify(mockMainView).mostrarErrorNoGasolinerasEnMunicipio("Error: No exiten gasolineras \n con el filtro aplicado");
         assertFalse(sut.filtroActivado);
     }
 

diff --git a/...oidProject/app/src/test/java/es/unican/gasolineras/activities/main/MainPresenterTest.java b/...oidProject/app/src/test/java/es/unican/gasolineras/activities/main/MainPresenterTest.java
@@ -131,7 +131,7 @@ public void testFiltrarPorMunicipioSinGasolineras() {
 
         verify(mockRepository).requestGasolineras(any(), any());
         assertFalse(sut.filtroActivado);
-        verify(mockView).mostrarErrorNoGasolinerasEnMunicipio("Error: No exiten gasolineras con \n el filtro aplicado");
+        verify(mockView).mostrarErrorNoGasolinerasEnMunicipio("Error: No exiten gasolineras \n con el filtro aplicado");
 
     }