diff --git a/app/src/main/java/fr/free/nrw/commons/category/CategoriesModel.kt b/app/src/main/java/fr/free/nrw/commons/category/CategoriesModel.kt index ab11b9863..f2b1c3a4c 100644 --- a/app/src/main/java/fr/free/nrw/commons/category/CategoriesModel.kt +++ b/app/src/main/java/fr/free/nrw/commons/category/CategoriesModel.kt @@ -1,6 +1,5 @@ package fr.free.nrw.commons.category -import android.text.TextUtils import fr.free.nrw.commons.upload.GpsCategoryModel import fr.free.nrw.commons.upload.structure.depictions.DepictedItem import fr.free.nrw.commons.utils.StringSortingUtils @@ -82,7 +81,7 @@ class CategoriesModel @Inject constructor( imageTitleList: List, selectedDepictions: List ): Observable> { - return if (TextUtils.isEmpty(term)) + return if (term.isBlank()) Observable.combineLatest( categoriesFromDepiction(selectedDepictions), gpsCategoryModel.categoriesFromLocation, diff --git a/app/src/main/java/fr/free/nrw/commons/upload/categories/CategoriesPresenter.kt b/app/src/main/java/fr/free/nrw/commons/upload/categories/CategoriesPresenter.kt index 8ad0ccf69..c5aba757f 100644 --- a/app/src/main/java/fr/free/nrw/commons/upload/categories/CategoriesPresenter.kt +++ b/app/src/main/java/fr/free/nrw/commons/upload/categories/CategoriesPresenter.kt @@ -1,6 +1,5 @@ package fr.free.nrw.commons.upload.categories -import android.text.TextUtils import fr.free.nrw.commons.R import fr.free.nrw.commons.category.CategoryItem import fr.free.nrw.commons.di.CommonsApplicationModule @@ -84,7 +83,7 @@ class CategoriesPresenter @Inject constructor( private fun getImageTitleList(): List { return repository.uploads .map { it.uploadMediaDetails[0].captionText } - .filterNot { TextUtils.isEmpty(it) } + .filterNot { it.isBlank() } } /** diff --git a/app/src/test/kotlin/fr/free/nrw/commons/category/CategoryDaoTest.kt b/app/src/test/kotlin/fr/free/nrw/commons/category/CategoryDaoTest.kt index 89e597274..530b97a6d 100644 --- a/app/src/test/kotlin/fr/free/nrw/commons/category/CategoryDaoTest.kt +++ b/app/src/test/kotlin/fr/free/nrw/commons/category/CategoryDaoTest.kt @@ -7,7 +7,6 @@ import android.database.MatrixCursor import android.database.sqlite.SQLiteDatabase import android.os.RemoteException import com.nhaarman.mockitokotlin2.* -import fr.free.nrw.commons.BuildConfig import fr.free.nrw.commons.TestCommonsApplication import fr.free.nrw.commons.category.CategoryContentProvider.BASE_URI import fr.free.nrw.commons.category.CategoryContentProvider.uriForId @@ -151,9 +150,11 @@ class CategoryDaoTest { @Test(expected = RuntimeException::class) fun testSaveTranslatesRemoteExceptions() { whenever(client.insert(isA(), isA())).thenThrow(RemoteException("")) - testObject.save(Category()) + testObject.save(category()) } + private fun category() = Category(null,null,null,0) + @Test fun whenTheresNoDataFindReturnsNull_nullCursor() { whenever(client.query(any(), any(), any(), any(), any())).thenReturn(null) @@ -268,4 +269,4 @@ class CategoryDaoTest { } } -} \ No newline at end of file +}