Merge pull request #93 from odaridavid/bug-fix-ioexception

odaridavid · web-flow · commit dfdf616f0f25 · 2020-05-31T10:37:19.000+03:00
Fix known exceptions
diff --git a/app/build.gradle b/app/build.gradle
@@ -42,18 +42,18 @@ android {
             minifyEnabled false
             proguardFiles getDefaultProguardFile('proguard-android-optimize.txt'), 'proguard-rules.pro'
             manifestPlaceholders = [
-                    crashlyticsEnabled: true,
-                    appIcon           : "@mipmap/ic_launcher",
-                    appIconRound      : "@mipmap/ic_launcher_round"
+                    crashlyticsEnabled     : true,
+                    appIcon                : "@mipmap/ic_launcher",
+                    appIconRound           : "@mipmap/ic_launcher_round"
             ]
         }
         debug {
             applicationIdSuffix ".debug"
             versionNameSuffix "-debug"
             manifestPlaceholders = [
-                    crashlyticsEnabled: false,
-                    appIcon           : "@mipmap/ic_debug_launcher",
-                    appIconRound      : "@mipmap/ic_debug_launcher_round"
+                    crashlyticsEnabled     : false,
+                    appIcon                : "@mipmap/ic_debug_launcher",
+                    appIconRound           : "@mipmap/ic_debug_launcher_round"
             ]
             testCoverageEnabled true
         }
diff --git a/app/src/debug/AndroidManifest.xml b/app/src/debug/AndroidManifest.xml
diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
@@ -9,6 +9,7 @@
         android:label="@string/app_name"
         android:roundIcon="${appIconRound}"
         android:supportsRtl="true"
+        android:networkSecurityConfig="@xml/network_security_config"
         android:theme="@style/AppTheme">
         <activity
             android:name=".features.settings.AboutActivity"
diff --git a/app/src/main/kotlin/com/k0d4black/theforce/features/character_details/CharacterDetailActivity.kt b/app/src/main/kotlin/com/k0d4black/theforce/features/character_details/CharacterDetailActivity.kt
@@ -118,18 +118,22 @@ class CharacterDetailActivity : AppCompatActivity() {
         onNetworkChange { isConnected ->
             characterDetailViewModel.detailViewState.value?.let { viewState ->
                 if (isConnected && viewState.error != null) {
-                    binding.filmsLayout.filmsErrorTextView.remove()
-                    binding.planetLayout.planetErrorTextView.remove()
-                    binding.specieLayout.specieErrorTextView.remove()
-                    binding.filmsLayout.filmsProgressBar.show()
-                    binding.planetLayout.planetProgressBar.show()
-                    binding.specieLayout.speciesProgressBar.show()
+                    resolveErrorViewState()
                     characterDetailViewModel.getCharacterDetails(characterUrl, isRetry = true)
                 }
             }
         }
     }
 
+    private fun resolveErrorViewState() {
+        binding.filmsLayout.filmsErrorTextView.remove()
+        binding.planetLayout.planetErrorTextView.remove()
+        binding.specieLayout.specieErrorTextView.remove()
+        binding.filmsLayout.filmsProgressBar.show()
+        binding.planetLayout.planetProgressBar.show()
+        binding.specieLayout.speciesProgressBar.show()
+    }
+
     private fun onNetworkChange(block: (Boolean) -> Unit) {
         NetworkUtils.getNetworkStatus(this)
             .observe(this, Observer { isConnected ->
diff --git a/app/src/main/kotlin/com/k0d4black/theforce/features/character_details/CharacterDetailViewModel.kt b/app/src/main/kotlin/com/k0d4black/theforce/features/character_details/CharacterDetailViewModel.kt
@@ -1,7 +1,11 @@
 package com.k0d4black.theforce.features.character_details
 
+import android.util.Log
 import androidx.annotation.StringRes
-import androidx.lifecycle.*
+import androidx.lifecycle.LiveData
+import androidx.lifecycle.MutableLiveData
+import androidx.lifecycle.ViewModel
+import androidx.lifecycle.viewModelScope
 import com.k0d4black.theforce.commons.ExceptionHandler
 import com.k0d4black.theforce.domain.usecases.FilmsUseCase
 import com.k0d4black.theforce.domain.usecases.PlanetUseCase
@@ -27,6 +31,7 @@ internal class CharacterDetailViewModel(
     private var _detailViewState = MutableLiveData<CharacterDetailsViewState>()
 
     private val characterDetailExceptionHandler = CoroutineExceptionHandler { _, exception ->
+        Log.d("Character Detail VM", "$exception")
         val message = ExceptionHandler.parse(exception)
         _detailViewState.value = _detailViewState.value?.copy(error = Error(message))
     }
@@ -47,14 +52,10 @@ internal class CharacterDetailViewModel(
             _detailViewState.value = _detailViewState.value?.copy(error = null)
         }
         viewModelScope.launch(characterDetailExceptionHandler) {
-            val planetRequest = async { loadPlanet(characterUrl) }
-            val filmsRequest = async { loadFilms(characterUrl) }
-            val speciesRequest = async { loadSpecies(characterUrl) }
-            planetRequest.await()
-            filmsRequest.await()
-            speciesRequest.await()
+            async { loadPlanet(characterUrl) }.await()
+            async { loadFilms(characterUrl) }.await()
+            async { loadSpecies(characterUrl) }.await()
             _detailViewState.value = _detailViewState.value?.copy(isComplete = true)
-
         }
     }
 
diff --git a/app/src/main/kotlin/com/k0d4black/theforce/features/character_search/CharacterSearchViewModel.kt b/app/src/main/kotlin/com/k0d4black/theforce/features/character_search/CharacterSearchViewModel.kt
@@ -1,5 +1,6 @@
 package com.k0d4black.theforce.features.character_search
 
+import android.util.Log
 import androidx.annotation.StringRes
 import androidx.lifecycle.*
 import com.k0d4black.theforce.commons.ExceptionHandler
diff --git a/app/src/main/kotlin/com/k0d4black/theforce/mappers/DomainToPresentation.kt b/app/src/main/kotlin/com/k0d4black/theforce/mappers/DomainToPresentation.kt
@@ -22,7 +22,7 @@ fun Character.toPresentation(): CharacterPresentation {
 }
 
 fun Planet.toPresentation(): PlanetPresentation {
-    val populationAsLong = if (this.population.contains("Unknown")) 0L else this.population.toLong()
+    val populationAsLong = if (this.population.contains("unknown")) 0L else this.population.toLong()
     return PlanetPresentation(this.name, populationAsLong)
 }
 
diff --git a/app/src/main/res/xml/network_security_config.xml b/app/src/main/res/xml/network_security_config.xml
@@ -0,0 +1,21 @@
+<?xml version="1.0" encoding="utf-8"?><!--
+
+ Copyright  David Odari
+
+ Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except 
+ in compliance with the License. You may obtain a copy of the License at
+            http://www.apache.org/licenses/LICENSE-2.0
+ Unless required by applicable law or agreed to in writing, software distributed under the License 
+ is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express
+ or implied. See the License for the specific language governing permissions and limitations under
+ the License.
+
+-->
+<network-security-config>
+    <domain-config cleartextTrafficPermitted="true">
+        <domain includeSubdomains="true">swapi.dev</domain>
+    </domain-config>
+    <domain-config cleartextTrafficPermitted="true">
+        <domain includeSubdomains="true">localhost</domain>
+    </domain-config>
+</network-security-config>
diff --git a/dependencies.gradle b/dependencies.gradle
@@ -6,8 +6,8 @@ ext {
     buildToolsVersion = "29.0.2"
 
     //App Versioning
-    versionCodeMajor = 1
-    versionCodeMinor = 3
+    versionCodeMajor = 2
+    versionCodeMinor = 0
     versionCodePatch = 0
     versionName = "$versionCodeMajor.$versionCodeMinor.$versionCodePatch"
 

Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ fun Character.toPresentation(): CharacterPresentation {`
`22`	`22`	`}`
`23`	`23`
`24`	`24`	`fun Planet.toPresentation(): PlanetPresentation {`
`25`		`- val populationAsLong = if (this.population.contains("Unknown")) 0L else this.population.toLong()`
	`25`	`+ val populationAsLong = if (this.population.contains("unknown")) 0L else this.population.toLong()`
`26`	`26`	`return PlanetPresentation(this.name, populationAsLong)`
`27`	`27`	`}`
`28`	`28`