Release 1.2.3.2

SimoneTinella · Jun 30, 2020 · d08a5e2 · d08a5e2
1 parent ed8d83f
commit d08a5e2
Show file tree

Hide file tree

Showing 6 changed files with 22 additions and 16 deletions.
diff --git a/StatoCovid19Italia.apk b/StatoCovid19Italia.apk
diff --git a/app/build.gradle b/app/build.gradle
@@ -8,8 +8,8 @@ android {
         applicationId "org.twistedappdeveloper.statocovid19italia"
         minSdkVersion 14
         targetSdkVersion 29
-        versionCode 24
-        versionName "1.2.3.1"
+        versionCode 25
+        versionName "1.2.3.2"
     }
     buildTypes {
         release {

diff --git a/app/src/main/java/org/twistedappdeveloper/statocovid19italia/datastorage/DataStorage.java b/app/src/main/java/org/twistedappdeveloper/statocovid19italia/datastorage/DataStorage.java
@@ -246,7 +246,7 @@ public void setDataArrayJson(JSONArray dataArrayJson) {
                     }
                     for (String nota : split) {
                         Avviso avviso = getAvvisoByKey(nota);
-                        if(avviso != null){
+                        if (avviso != null) {
                             avvisi.add(getAvvisoByKey(nota));
                         }
                     }
@@ -300,8 +300,6 @@ public void setSubLvlDataArrayJson(JSONArray subLevelDataJSONArray, String key,
                 JSONObject jsonObject = subLevelDataJSONArray.getJSONObject(i);
 
                 String regione = jsonObject.getString(key);
-
-                //FIXME da gestire i casi separatamente
                 regione = TrendUtils.getFixedProvinciaDen(regione);
 
                 JSONArray regionalJSONArray;
@@ -311,7 +309,18 @@ public void setSubLvlDataArrayJson(JSONArray subLevelDataJSONArray, String key,
                     regionalJSONArray = new JSONArray();
                     regionalJsonArrayMap.put(regione, regionalJSONArray);
                 }
-                regionalJSONArray.put(jsonObject);
+                // unisco i valori nel caso di province unite. Es In fase di definizione o Fuori Regione etc..
+                boolean found = false;
+                for (int j = 0; j < regionalJSONArray.length(); j++) {
+                    JSONObject jsonObjectTmp = regionalJSONArray.getJSONObject(j);
+                    if (jsonObjectTmp.get(DataStorage.DATA_KEY).equals(jsonObject.get(DataStorage.DATA_KEY))) {
+                        jsonObjectTmp.put(DataStorage.TOTALE_CASI_KEY, jsonObjectTmp.getInt(DataStorage.TOTALE_CASI_KEY) + jsonObject.getInt(DataStorage.TOTALE_CASI_KEY));
+                        found = true;
+                    }
+                }
+                if (!found) {
+                    regionalJSONArray.put(jsonObject);
+                }
             }
 
             for (String dataContext : regionalJsonArrayMap.keySet()) {

diff --git a/...ain/java/org/twistedappdeveloper/statocovid19italia/fragments/DataVisualizerFragment.java b/...ain/java/org/twistedappdeveloper/statocovid19italia/fragments/DataVisualizerFragment.java
@@ -144,7 +144,6 @@ private void displayData() {
                 List<String> province = dataStorage.getSubLevelDataKeys();
                 Collections.sort(province);
                 for (String provincia : province) {
-                    //FIXME da gestire i casi separatamente
                     provincia = TrendUtils.getFixedProvinciaDen(provincia);
                     TrendInfo totaleCasiProvincia = dataStorage.getDataStorageByDataContext(provincia).getTrendByKey(trendInfo.getKey());
                     if (totaleCasiProvincia != null) {

diff --git a/app/src/main/java/org/twistedappdeveloper/statocovid19italia/utils/TrendUtils.java b/app/src/main/java/org/twistedappdeveloper/statocovid19italia/utils/TrendUtils.java
@@ -173,14 +173,8 @@ public static String getTrendDescriptionByTrendKey(Context context, String key)
         }
     }
 
-    //FIXME da gestire i casi separatamente e rimuovere questo metodo
     public static String getFixedProvinciaDen(String den) {
-        if (
-                den.compareToIgnoreCase("In fase di definizione") == 0 ||
-                        den.compareToIgnoreCase("In fase di definizione/aggiornamento") == 0 ||
-                        den.compareToIgnoreCase("Fuori Regione / Provincia Autonoma") == 0 ||
-                        den.compareToIgnoreCase(denProvinciaFixed) == 0
-        ) {
+        if (discardProvincia(den)) {
             return denProvinciaFixed;
         } else {
             return den;
@@ -190,7 +184,7 @@ public static String getFixedProvinciaDen(String den) {
     public static boolean discardProvincia(String den) {
         return den.compareToIgnoreCase("In fase di definizione") == 0 ||
                 den.compareToIgnoreCase("In fase di definizione/aggiornamento") == 0 ||
-                den.compareToIgnoreCase("fuori Regione/P.A.") == 0 ||
+                den.compareToIgnoreCase("Fuori Regione / Provincia Autonoma") == 0 ||
                 den.compareToIgnoreCase(denProvinciaFixed) == 0;
     }
 

diff --git a/notification.json b/notification.json
@@ -1,6 +1,10 @@
 {
-  "latest_app_version": 24,
+  "latest_app_version": 25,
   "changelog": [
+    {
+      "ver": "1.2.3.2",
+      "description": "-Sistemato calcolo per dati provinciali In fase di definizione o Fuori Regione/P.A"
+    },
     {
       "ver": "1.2.3.1",
       "description": "-Hotfix: Adattamento alla nuova struttura dati rilasciata dalla Protezione Civile in data 29/06/2020. Sono state aggiunte due nuove misure: Casi da Sospetto Diagnostico e Casi da Screening. \n\nNote: \n1) Le due nuove misure sono valorizzate dal 25/06/2020 in poi. \n2)Al momento gli avvisi sui dati sono stati disattivati per assenza degli stessi."