move home property list fragments, ViewModels to sub folders

SmartToolFactory · SmartToolFactory · commit ddf46b874b64 · 2020-09-09T08:17:41.000+03:00
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
@@ -1,5 +1,5 @@
 <resources>
-    <string name="app_name">PropertyFindAR</string>
+    <string name="app_name">Property FindAR</string>
     <string name="title_home">Home</string>
     <string name="title_favorites">Favorites</string>
     <string name="title_notification">Notification</string>
diff --git a/features/home/src/main/java/com/smarttoolfactory/home/di/HomeComponent.kt b/features/home/src/main/java/com/smarttoolfactory/home/di/HomeComponent.kt
@@ -2,9 +2,9 @@ package com.smarttoolfactory.home.di
 
 import androidx.fragment.app.Fragment
 import com.smarttoolfactory.core.di.CoreModuleDependencies
-import com.smarttoolfactory.home.propertylist.PagedPropertyListFragment
-import com.smarttoolfactory.home.propertylist.PropertyListFlowFragment
-import com.smarttoolfactory.home.propertylist.PropertyListRxjava3Fragment
+import com.smarttoolfactory.home.propertylist.flow.PropertyListFlowFragment
+import com.smarttoolfactory.home.propertylist.paged.PagedPropertyListFragment
+import com.smarttoolfactory.home.propertylist.rxjava.PropertyListRxjava3Fragment
 import dagger.BindsInstance
 import dagger.Component
 
diff --git a/features/home/src/main/java/com/smarttoolfactory/home/di/HomeModule.kt b/features/home/src/main/java/com/smarttoolfactory/home/di/HomeModule.kt
@@ -2,12 +2,12 @@ package com.smarttoolfactory.home.di
 
 import androidx.fragment.app.Fragment
 import androidx.lifecycle.ViewModelProvider
-import com.smarttoolfactory.home.viewmodel.PagedPropertyListViewModel
+import com.smarttoolfactory.home.propertylist.flow.PropertyListViewModelFlow
+import com.smarttoolfactory.home.propertylist.paged.PagedPropertyListViewModel
+import com.smarttoolfactory.home.propertylist.rxjava.PropertyListViewModelRxJava3
 import com.smarttoolfactory.home.viewmodel.PagedPropertyListViewModelFactory
 import com.smarttoolfactory.home.viewmodel.PropertyListFlowViewModelFactory
 import com.smarttoolfactory.home.viewmodel.PropertyListRxJava3ViewModelFactory
-import com.smarttoolfactory.home.viewmodel.PropertyListViewModelFlow
-import com.smarttoolfactory.home.viewmodel.PropertyListViewModelRxJava3
 import dagger.Module
 import dagger.Provides
 import dagger.hilt.InstallIn
diff --git a/features/home/src/main/java/com/smarttoolfactory/home/propertylist/AbstractPropertyListVM.kt b/features/home/src/main/java/com/smarttoolfactory/home/propertylist/AbstractPropertyListVM.kt
@@ -1,4 +1,4 @@
-package com.smarttoolfactory.home.viewmodel
+package com.smarttoolfactory.home.propertylist
 
 import androidx.lifecycle.LiveData
 import androidx.lifecycle.ViewModel
diff --git a/features/home/src/main/java/com/smarttoolfactory/home/propertylist/flow/PropertyListFlowFragment.kt b/features/home/src/main/java/com/smarttoolfactory/home/propertylist/flow/PropertyListFlowFragment.kt
@@ -1,4 +1,4 @@
-package com.smarttoolfactory.home.propertylist
+package com.smarttoolfactory.home.propertylist.flow
 
 import android.os.Bundle
 import androidx.core.os.bundleOf
@@ -12,7 +12,6 @@ import com.smarttoolfactory.home.adapter.PropertyItemListAdapter
 import com.smarttoolfactory.home.databinding.FragmentPropertyListBinding
 import com.smarttoolfactory.home.di.DaggerHomeComponent
 import com.smarttoolfactory.home.viewmodel.HomeToolbarVM
-import com.smarttoolfactory.home.viewmodel.PropertyListViewModelFlow
 import dagger.hilt.android.EntryPointAccessors
 import javax.inject.Inject
 
diff --git a/features/home/src/main/java/com/smarttoolfactory/home/propertylist/flow/PropertyListViewModelFlow.kt b/features/home/src/main/java/com/smarttoolfactory/home/propertylist/flow/PropertyListViewModelFlow.kt
@@ -1,4 +1,4 @@
-package com.smarttoolfactory.home.viewmodel
+package com.smarttoolfactory.home.propertylist.flow
 
 import androidx.hilt.lifecycle.ViewModelInject
 import androidx.lifecycle.LiveData
@@ -10,6 +10,7 @@ import com.smarttoolfactory.core.viewstate.ViewState
 import com.smarttoolfactory.domain.ORDER_BY_NONE
 import com.smarttoolfactory.domain.model.PropertyItem
 import com.smarttoolfactory.domain.usecase.GetPropertiesUseCaseFlow
+import com.smarttoolfactory.home.propertylist.AbstractPropertyListVM
 import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.flow.launchIn
 import kotlinx.coroutines.flow.onEach
diff --git a/features/home/src/main/java/com/smarttoolfactory/home/propertylist/paged/PagedPropertyListFragment.kt b/features/home/src/main/java/com/smarttoolfactory/home/propertylist/paged/PagedPropertyListFragment.kt
@@ -1,7 +1,6 @@
-package com.smarttoolfactory.home.propertylist
+package com.smarttoolfactory.home.propertylist.paged
 
 import android.os.Bundle
-import android.widget.Toast
 import androidx.core.os.bundleOf
 import androidx.fragment.app.activityViewModels
 import androidx.recyclerview.widget.LinearLayoutManager
@@ -14,7 +13,6 @@ import com.smarttoolfactory.home.adapter.PropertyItemListAdapter
 import com.smarttoolfactory.home.databinding.FragmentPropertyListPagedBinding
 import com.smarttoolfactory.home.di.DaggerHomeComponent
 import com.smarttoolfactory.home.viewmodel.HomeToolbarVM
-import com.smarttoolfactory.home.viewmodel.PagedPropertyListViewModel
 import dagger.hilt.android.EntryPointAccessors
 import javax.inject.Inject
 
@@ -143,6 +141,5 @@ class PagedPropertyListFragment :
 
     override fun onScrollToBottom() {
         viewModel.getPropertyList()
-        Toast.makeText(requireContext(), "BOTTOM SCROLL", Toast.LENGTH_SHORT).show()
     }
 }
diff --git a/features/home/src/main/java/com/smarttoolfactory/home/propertylist/paged/PagedPropertyListViewModel.kt b/features/home/src/main/java/com/smarttoolfactory/home/propertylist/paged/PagedPropertyListViewModel.kt
@@ -1,4 +1,4 @@
-package com.smarttoolfactory.home.viewmodel
+package com.smarttoolfactory.home.propertylist.paged
 
 import androidx.hilt.lifecycle.ViewModelInject
 import androidx.lifecycle.LiveData
@@ -10,6 +10,7 @@ import com.smarttoolfactory.core.viewstate.ViewState
 import com.smarttoolfactory.domain.ORDER_BY_NONE
 import com.smarttoolfactory.domain.model.PropertyItem
 import com.smarttoolfactory.domain.usecase.GetPropertiesUseCasePaged
+import com.smarttoolfactory.home.propertylist.AbstractPropertyListVM
 import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.flow.launchIn
 import kotlinx.coroutines.flow.onEach
diff --git a/features/home/src/main/java/com/smarttoolfactory/home/propertylist/rxjava/PropertyListRxjava3Fragment.kt b/features/home/src/main/java/com/smarttoolfactory/home/propertylist/rxjava/PropertyListRxjava3Fragment.kt
@@ -1,4 +1,4 @@
-package com.smarttoolfactory.home.propertylist
+package com.smarttoolfactory.home.propertylist.rxjava
 
 import android.os.Bundle
 import androidx.core.os.bundleOf
@@ -12,7 +12,6 @@ import com.smarttoolfactory.home.adapter.PropertyItemListAdapter
 import com.smarttoolfactory.home.databinding.FragmentPropertyListBinding
 import com.smarttoolfactory.home.di.DaggerHomeComponent
 import com.smarttoolfactory.home.viewmodel.HomeToolbarVM
-import com.smarttoolfactory.home.viewmodel.PropertyListViewModelRxJava3
 import dagger.hilt.android.EntryPointAccessors
 import javax.inject.Inject
 
diff --git a/features/home/src/main/java/com/smarttoolfactory/home/propertylist/rxjava/PropertyListViewModelRxJava3.kt b/features/home/src/main/java/com/smarttoolfactory/home/propertylist/rxjava/PropertyListViewModelRxJava3.kt
@@ -1,4 +1,4 @@
-package com.smarttoolfactory.home.viewmodel
+package com.smarttoolfactory.home.propertylist.rxjava
 
 import androidx.hilt.lifecycle.ViewModelInject
 import androidx.lifecycle.LiveData
@@ -10,6 +10,7 @@ import com.smarttoolfactory.core.viewstate.ViewState
 import com.smarttoolfactory.domain.ORDER_BY_NONE
 import com.smarttoolfactory.domain.model.PropertyItem
 import com.smarttoolfactory.domain.usecase.GetPropertiesUseCaseRxJava3
+import com.smarttoolfactory.home.propertylist.AbstractPropertyListVM
 import io.reactivex.rxjava3.android.schedulers.AndroidSchedulers
 import io.reactivex.rxjava3.schedulers.Schedulers
 
diff --git a/features/home/src/main/java/com/smarttoolfactory/home/viewmodel/ViewModelFactory.kt b/features/home/src/main/java/com/smarttoolfactory/home/viewmodel/ViewModelFactory.kt
@@ -5,6 +5,9 @@ import androidx.lifecycle.ViewModelProvider
 import com.smarttoolfactory.domain.usecase.GetPropertiesUseCaseFlow
 import com.smarttoolfactory.domain.usecase.GetPropertiesUseCasePaged
 import com.smarttoolfactory.domain.usecase.GetPropertiesUseCaseRxJava3
+import com.smarttoolfactory.home.propertylist.flow.PropertyListViewModelFlow
+import com.smarttoolfactory.home.propertylist.paged.PagedPropertyListViewModel
+import com.smarttoolfactory.home.propertylist.rxjava.PropertyListViewModelRxJava3
 import javax.inject.Inject
 import kotlinx.coroutines.CoroutineScope
 
diff --git a/features/home/src/main/res/layout/fragment_property_list.xml b/features/home/src/main/res/layout/fragment_property_list.xml
@@ -9,7 +9,7 @@
 
         <variable
             name="viewModel"
-            type="com.smarttoolfactory.home.viewmodel.AbstractPropertyListVM" />
+            type="com.smarttoolfactory.home.propertylist.AbstractPropertyListVM" />
     </data>
 
     <androidx.swiperefreshlayout.widget.SwipeRefreshLayout
diff --git a/features/home/src/main/res/layout/fragment_property_list_paged.xml b/features/home/src/main/res/layout/fragment_property_list_paged.xml
@@ -9,7 +9,7 @@
 
         <variable
             name="viewModel"
-            type="com.smarttoolfactory.home.viewmodel.PagedPropertyListViewModel" />
+            type="com.smarttoolfactory.home.propertylist.paged.PagedPropertyListViewModel" />
     </data>
 
     <androidx.swiperefreshlayout.widget.SwipeRefreshLayout
diff --git a/features/home/src/main/res/navigation/nav_graph_property_list_flow.xml b/features/home/src/main/res/navigation/nav_graph_property_list_flow.xml
@@ -7,7 +7,7 @@
 
     <fragment
         android:id="@+id/propertyListFragment"
-        android:name="com.smarttoolfactory.home.propertylist.PropertyListFlowFragment"
+        android:name="com.smarttoolfactory.home.propertylist.flow.PropertyListFlowFragment"
         android:label="Properties with Flow"
         tools:layout="@layout/fragment_property_list">
     </fragment>
diff --git a/features/home/src/main/res/navigation/nav_graph_property_list_paged.xml b/features/home/src/main/res/navigation/nav_graph_property_list_paged.xml
@@ -7,7 +7,7 @@
 
     <fragment
         android:id="@+id/propertyListFragment"
-        android:name="com.smarttoolfactory.home.propertylist.PagedPropertyListFragment"
+        android:name="com.smarttoolfactory.home.propertylist.paged.PagedPropertyListFragment"
         android:label="Flow + Pagination"
         tools:layout="@layout/fragment_property_list_paged" />
 
diff --git a/features/home/src/main/res/navigation/nav_graph_property_list_rxjava3.xml b/features/home/src/main/res/navigation/nav_graph_property_list_rxjava3.xml
@@ -7,7 +7,7 @@
 
     <fragment
         android:id="@+id/propertyListFragment"
-        android:name="com.smarttoolfactory.home.propertylist.PropertyListRxjava3Fragment"
+        android:name="com.smarttoolfactory.home.propertylist.rxjava.PropertyListRxjava3Fragment"
         android:label="Properties with RxJava3"
         tools:layout="@layout/fragment_property_list">
     </fragment>
diff --git a/features/home/src/test/java/com/smarttoolfactory/home/viewmodel/PropertyListViewModelFlowTest.kt b/features/home/src/test/java/com/smarttoolfactory/home/viewmodel/PropertyListViewModelFlowTest.kt
@@ -5,6 +5,7 @@ import com.google.common.truth.Truth
 import com.smarttoolfactory.core.viewstate.Status
 import com.smarttoolfactory.domain.model.PropertyItem
 import com.smarttoolfactory.domain.usecase.GetPropertiesUseCaseFlow
+import com.smarttoolfactory.home.propertylist.flow.PropertyListViewModelFlow
 import com.smarttoolfactory.test_utils.RESPONSE_JSON_PATH
 import com.smarttoolfactory.test_utils.rule.TestCoroutineRule
 import com.smarttoolfactory.test_utils.test_observer.test
diff --git a/features/home/src/test/java/com/smarttoolfactory/home/viewmodel/PropertyListViewModelRxJava3Test.kt b/features/home/src/test/java/com/smarttoolfactory/home/viewmodel/PropertyListViewModelRxJava3Test.kt
@@ -5,7 +5,8 @@ import com.google.common.truth.Truth
 import com.smarttoolfactory.core.viewstate.Status
 import com.smarttoolfactory.domain.model.PropertyItem
 import com.smarttoolfactory.domain.usecase.GetPropertiesUseCaseRxJava3
-import com.smarttoolfactory.home.viewmodel.AbstractPropertyListVM.Companion.ORDER_BY_NONE
+import com.smarttoolfactory.home.propertylist.AbstractPropertyListVM.Companion.ORDER_BY_NONE
+import com.smarttoolfactory.home.propertylist.rxjava.PropertyListViewModelRxJava3
 import com.smarttoolfactory.test_utils.RESPONSE_JSON_PATH
 import com.smarttoolfactory.test_utils.rule.RxImmediateSchedulerRule
 import com.smarttoolfactory.test_utils.test_observer.test
diff --git a/libraries/data/src/main/java/com/smarttoolfactory/data/model/local/PropertyEntity.kt b/libraries/data/src/main/java/com/smarttoolfactory/data/model/local/PropertyEntity.kt
@@ -10,6 +10,9 @@ import com.smarttoolfactory.data.model.IEntity
  *
  * * Implements [IEntity] marker interface for mapping this database items from REST DTOs or
  * to UI items
+ *
+ * * Insert order is required for getting data same order they are inserted since ordering is
+ * done in server side.
  */
 @Entity(tableName = "property", primaryKeys = ["insert_order"])
 data class PropertyEntity(
diff --git a/libraries/domain/src/main/java/com/smarttoolfactory/domain/usecase/GetPropertiesUseCasePaged.kt b/libraries/domain/src/main/java/com/smarttoolfactory/domain/usecase/GetPropertiesUseCasePaged.kt
@@ -49,8 +49,7 @@ class GetPropertiesUseCasePaged @Inject constructor(
                             propertyEntity.insertOrder = propertyCount + index
                         }
                         savePropertyEntities(it)
-                        val data = getPropertyEntitiesFromLocal()
-                        data
+                        getPropertyEntitiesFromLocal()
                     }
                 }
             }

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-package com.smarttoolfactory.home.viewmodel`
	`1`	`+package com.smarttoolfactory.home.propertylist`
`2`	`2`
`3`	`3`	`import androidx.lifecycle.LiveData`
`4`	`4`	`import androidx.lifecycle.ViewModel`