Fix #2658, #299: Replace builder() with Factory in SingleTypeBuilder and MultiTypeBuilder

app/src/main/java/org/oppia/android/app/administratorcontrols/AdministratorControlsFragmentPresenter.kt

@@ -31,7 +31,8 @@ import javax.inject.Inject
 @FragmentScope
 class AdministratorControlsFragmentPresenter @Inject constructor(
   private val activity: AppCompatActivity,
-  private val fragment: Fragment
+  private val fragment: Fragment,
+  private val multiTypeBuilderFactory: BindableAdapter.MultiTypeBuilder.Factory
 ) {
   private lateinit var binding: AdministratorControlsFragmentBinding
   private lateinit var linearLayoutManager: LinearLayoutManager
@@ -77,8 +78,8 @@ class AdministratorControlsFragmentPresenter @Inject constructor(
   /** Returns the recycler view adapter for the controls panel in  administrator controls fragment. */
   private fun createRecyclerViewAdapter(isMultipane: Boolean):
     BindableAdapter<AdministratorControlsItemViewModel> {
-      return BindableAdapter.MultiTypeBuilder
-        .newBuilder<AdministratorControlsItemViewModel, ViewType> { viewModel ->
+      return multiTypeBuilderFactory
+        .create<AdministratorControlsItemViewModel, ViewType> { viewModel ->
           viewModel.isMultipane.set(isMultipane)
           when (viewModel) {
             is AdministratorControlsGeneralViewModel -> {

app/src/main/java/org/oppia/android/app/administratorcontrols/learneranalytics/ProfileAndDeviceIdFragmentPresenter.kt

@@ -16,8 +16,10 @@ import javax.inject.Inject
 /** Presenter for arranging [ProfileAndDeviceIdFragment]'s UI. */
 class ProfileAndDeviceIdFragmentPresenter @Inject constructor(
   private val fragment: Fragment,
-  private val profileListViewModelFactory: ProfileListViewModel.Factory
+  private val profileListViewModelFactory: ProfileListViewModel.Factory,
+  private val multiTypeBuilderFactory: BindableAdapter.MultiTypeBuilder.Factory
 ) {
+
   private lateinit var binding: ProfileAndDeviceIdFragmentBinding
 
   /** Handles [ProfileAndDeviceIdFragment]'s creation flow. */
@@ -38,16 +40,15 @@ class ProfileAndDeviceIdFragmentPresenter @Inject constructor(
   }
 
   private fun createRecyclerViewAdapter(): BindableAdapter<ProfileListItemViewModel> {
-    return BindableAdapter.MultiTypeBuilder
-      .newBuilder<ProfileListItemViewModel, ProfileListItemViewType> { viewModel ->
-        when (viewModel) {
-          is DeviceIdItemViewModel -> ProfileListItemViewType.DEVICE_ID
-          is ProfileLearnerIdItemViewModel -> ProfileListItemViewType.LEARNER_ID
-          is SyncStatusItemViewModel -> ProfileListItemViewType.SYNC_STATUS
-          else -> error("Encountered unexpected view model: $viewModel")
-        }
+    return multiTypeBuilderFactory.create<ProfileListItemViewModel,
+      ProfileListItemViewType> { viewModel ->
+      when (viewModel) {
+        is DeviceIdItemViewModel -> ProfileListItemViewType.DEVICE_ID
+        is ProfileLearnerIdItemViewModel -> ProfileListItemViewType.LEARNER_ID
+        is SyncStatusItemViewModel -> ProfileListItemViewType.SYNC_STATUS
+        else -> error("Encountered unexpected view model: $viewModel")
       }
-      .setLifecycleOwner(fragment)
+    }
       .registerViewDataBinder(
         viewType = ProfileListItemViewType.DEVICE_ID,
         inflateDataBinding = ProfileListDeviceIdItemBinding::inflate,

app/src/main/java/org/oppia/android/app/completedstorylist/CompletedStoryListFragmentPresenter.kt

@@ -17,7 +17,8 @@ import javax.inject.Inject
 class CompletedStoryListFragmentPresenter @Inject constructor(
   private val activity: AppCompatActivity,
   private val fragment: Fragment,
-  private val viewModelProvider: ViewModelProvider<CompletedStoryListViewModel>
+  private val viewModelProvider: ViewModelProvider<CompletedStoryListViewModel>,
+  private val singleTypeBuilderFactory: BindableAdapter.SingleTypeBuilder.Factory
 ) {
 
   private lateinit var binding: CompletedStoryListFragmentBinding
@@ -52,8 +53,7 @@ class CompletedStoryListFragmentPresenter @Inject constructor(
   }
 
   private fun createRecyclerViewAdapter(): BindableAdapter<CompletedStoryItemViewModel> {
-    return BindableAdapter.SingleTypeBuilder
-      .newBuilder<CompletedStoryItemViewModel>()
+    return singleTypeBuilderFactory.create<CompletedStoryItemViewModel>()
       .registerViewDataBinderWithSameModelType(
         inflateDataBinding = CompletedStoryItemBinding::inflate,
         setViewModel = CompletedStoryItemBinding::setViewModel

app/src/main/java/org/oppia/android/app/devoptions/DeveloperOptionsFragmentPresenter.kt

@@ -24,7 +24,8 @@ import javax.inject.Inject
 @FragmentScope
 class DeveloperOptionsFragmentPresenter @Inject constructor(
   private val activity: AppCompatActivity,
-  private val fragment: Fragment
+  private val fragment: Fragment,
+  private val multiTypeBuilderFactory: BindableAdapter.MultiTypeBuilder.Factory
 ) {
 
   private lateinit var binding: DeveloperOptionsFragmentBinding
@@ -59,28 +60,27 @@ class DeveloperOptionsFragmentPresenter @Inject constructor(
   }
 
   private fun createRecyclerViewAdapter(): BindableAdapter<DeveloperOptionsItemViewModel> {
-    return BindableAdapter.MultiTypeBuilder
-      .newBuilder<DeveloperOptionsItemViewModel, ViewType> { viewModel ->
-        when (viewModel) {
-          is DeveloperOptionsModifyLessonProgressViewModel -> {
-            viewModel.itemIndex.set(0)
-            ViewType.VIEW_TYPE_MODIFY_LESSON_PROGRESS
-          }
-          is DeveloperOptionsViewLogsViewModel -> {
-            viewModel.itemIndex.set(1)
-            ViewType.VIEW_TYPE_VIEW_LOGS
-          }
-          is DeveloperOptionsOverrideAppBehaviorsViewModel -> {
-            viewModel.itemIndex.set(2)
-            ViewType.VIEW_TYPE_OVERRIDE_APP_BEHAVIORS
-          }
-          is DeveloperOptionsTestParsersViewModel -> {
-            viewModel.itemIndex.set(3)
-            ViewType.VIEW_TYPE_TEST_PARSERS
-          }
-          else -> throw IllegalArgumentException("Encountered unexpected view model: $viewModel")
+    return multiTypeBuilderFactory.create<DeveloperOptionsItemViewModel, ViewType> { viewModel ->
+      when (viewModel) {
+        is DeveloperOptionsModifyLessonProgressViewModel -> {
+          viewModel.itemIndex.set(0)
+          ViewType.VIEW_TYPE_MODIFY_LESSON_PROGRESS
         }
+        is DeveloperOptionsViewLogsViewModel -> {
+          viewModel.itemIndex.set(1)
+          ViewType.VIEW_TYPE_VIEW_LOGS
+        }
+        is DeveloperOptionsOverrideAppBehaviorsViewModel -> {
+          viewModel.itemIndex.set(2)
+          ViewType.VIEW_TYPE_OVERRIDE_APP_BEHAVIORS
+        }
+        is DeveloperOptionsTestParsersViewModel -> {
+          viewModel.itemIndex.set(3)
+          ViewType.VIEW_TYPE_TEST_PARSERS
+        }
+        else -> throw IllegalArgumentException("Encountered unexpected view model: $viewModel")
       }
+    }
       .registerViewDataBinder(
         viewType = ViewType.VIEW_TYPE_MODIFY_LESSON_PROGRESS,
         inflateDataBinding = DeveloperOptionsModifyLessonProgressViewBinding::inflate,

app/src/main/java/org/oppia/android/app/devoptions/forcenetworktype/ForceNetworkTypeFragmentPresenter.kt

@@ -21,7 +21,8 @@ class ForceNetworkTypeFragmentPresenter @Inject constructor(
   private val activity: AppCompatActivity,
   private val fragment: Fragment,
   private val networkConnectionUtil: Optional<NetworkConnectionDebugUtil>,
-  private val viewModelProvider: ViewModelProvider<ForceNetworkTypeViewModel>
+  private val viewModelProvider: ViewModelProvider<ForceNetworkTypeViewModel>,
+  private val singleTypeBuilderFactory: BindableAdapter.SingleTypeBuilder.Factory
 ) {
 
   private lateinit var binding: ForceNetworkTypeFragmentBinding
@@ -60,8 +61,7 @@ class ForceNetworkTypeFragmentPresenter @Inject constructor(
   }
 
   private fun createRecyclerViewAdapter(): BindableAdapter<NetworkTypeItemViewModel> {
-    return BindableAdapter.SingleTypeBuilder
-      .newBuilder<NetworkTypeItemViewModel>()
+    return singleTypeBuilderFactory.create<NetworkTypeItemViewModel>()
       .registerViewDataBinderWithSameModelType(
         inflateDataBinding = ForceNetworkTypeNetworkItemViewBinding::inflate,
         setViewModel = this::bindNetworkItemView

app/src/main/java/org/oppia/android/app/devoptions/markchapterscompleted/MarkChaptersCompletedFragmentPresenter.kt

@@ -22,7 +22,8 @@ class MarkChaptersCompletedFragmentPresenter @Inject constructor(
   private val activity: AppCompatActivity,
   private val fragment: Fragment,
   private val viewModel: MarkChaptersCompletedViewModel,
-  private val modifyLessonProgressController: ModifyLessonProgressController
+  private val modifyLessonProgressController: ModifyLessonProgressController,
+  private val multiTypeBuilderFactory: BindableAdapter.MultiTypeBuilder.Factory
 ) : ChapterSelector {
   private lateinit var binding: MarkChaptersCompletedFragmentBinding
   private lateinit var linearLayoutManager: LinearLayoutManager
@@ -92,14 +93,14 @@ class MarkChaptersCompletedFragmentPresenter @Inject constructor(
   }
 
   private fun createRecyclerViewAdapter(): BindableAdapter<MarkChaptersCompletedItemViewModel> {
-    return BindableAdapter.MultiTypeBuilder
-      .newBuilder<MarkChaptersCompletedItemViewModel, ViewType> { viewModel ->
-        when (viewModel) {
-          is StorySummaryViewModel -> ViewType.VIEW_TYPE_STORY
-          is ChapterSummaryViewModel -> ViewType.VIEW_TYPE_CHAPTER
-          else -> throw IllegalArgumentException("Encountered unexpected view model: $viewModel")
-        }
+    return multiTypeBuilderFactory.create<MarkChaptersCompletedItemViewModel,
+      ViewType> { viewModel ->
+      when (viewModel) {
+        is StorySummaryViewModel -> ViewType.VIEW_TYPE_STORY
+        is ChapterSummaryViewModel -> ViewType.VIEW_TYPE_CHAPTER
+        else -> throw IllegalArgumentException("Encountered unexpected view model: $viewModel")
       }
+    }
       .registerViewDataBinder(
         viewType = ViewType.VIEW_TYPE_STORY,
         inflateDataBinding = MarkChaptersCompletedStorySummaryViewBinding::inflate,

app/src/main/java/org/oppia/android/app/devoptions/markstoriescompleted/MarkStoriesCompletedFragmentPresenter.kt

@@ -20,7 +20,8 @@ class MarkStoriesCompletedFragmentPresenter @Inject constructor(
   private val activity: AppCompatActivity,
   private val fragment: Fragment,
   private val viewModel: MarkStoriesCompletedViewModel,
-  private val modifyLessonProgressController: ModifyLessonProgressController
+  private val modifyLessonProgressController: ModifyLessonProgressController,
+  private val singleTypeBuilderFactory: BindableAdapter.SingleTypeBuilder.Factory
 ) : StorySelector {
   private lateinit var binding: MarkStoriesCompletedFragmentBinding
   private lateinit var linearLayoutManager: LinearLayoutManager
@@ -91,8 +92,7 @@ class MarkStoriesCompletedFragmentPresenter @Inject constructor(
   }
 
   private fun createRecyclerViewAdapter(): BindableAdapter<StorySummaryViewModel> {
-    return BindableAdapter.SingleTypeBuilder
-      .newBuilder<StorySummaryViewModel>()
+    return singleTypeBuilderFactory.create<StorySummaryViewModel>()
       .registerViewDataBinderWithSameModelType(
         inflateDataBinding = MarkStoriesCompletedStorySummaryViewBinding::inflate,
         setViewModel = this::bindStorySummaryView

app/src/main/java/org/oppia/android/app/devoptions/marktopicscompleted/MarkTopicsCompletedFragmentPresenter.kt

@@ -20,7 +20,8 @@ class MarkTopicsCompletedFragmentPresenter @Inject constructor(
   private val activity: AppCompatActivity,
   private val fragment: Fragment,
   private val viewModel: MarkTopicsCompletedViewModel,
-  private val modifyLessonProgressController: ModifyLessonProgressController
+  private val modifyLessonProgressController: ModifyLessonProgressController,
+  private val singleTypeBuilderFactory: BindableAdapter.SingleTypeBuilder.Factory
 ) : TopicSelector {
   private lateinit var binding: MarkTopicsCompletedFragmentBinding
   private lateinit var linearLayoutManager: LinearLayoutManager
@@ -88,8 +89,7 @@ class MarkTopicsCompletedFragmentPresenter @Inject constructor(
   }
 
   private fun createRecyclerViewAdapter(): BindableAdapter<TopicViewModel> {
-    return BindableAdapter.SingleTypeBuilder
-      .newBuilder<TopicViewModel>()
+    return singleTypeBuilderFactory.create<TopicViewModel>()
       .registerViewDataBinderWithSameModelType(
         inflateDataBinding = MarkTopicsCompletedTopicViewBinding::inflate,
         setViewModel = this::bindTopicSummaryView

app/src/main/java/org/oppia/android/app/devoptions/vieweventlogs/ViewEventLogsFragmentPresenter.kt

@@ -18,7 +18,8 @@ import javax.inject.Inject
 class ViewEventLogsFragmentPresenter @Inject constructor(
   private val activity: AppCompatActivity,
   private val fragment: Fragment,
-  private val viewModelProvider: ViewModelProvider<ViewEventLogsViewModel>
+  private val viewModelProvider: ViewModelProvider<ViewEventLogsViewModel>,
+  private val singleTypeBuilderFactory: BindableAdapter.SingleTypeBuilder.Factory
 ) {
 
   private lateinit var binding: ViewEventLogsFragmentBinding
@@ -56,8 +57,7 @@ class ViewEventLogsFragmentPresenter @Inject constructor(
   }
 
   private fun createRecyclerViewAdapter(): BindableAdapter<EventLogItemViewModel> {
-    return BindableAdapter.SingleTypeBuilder
-      .newBuilder<EventLogItemViewModel>()
+    return singleTypeBuilderFactory.create<EventLogItemViewModel>()
       .registerViewDataBinderWithSameModelType(
         inflateDataBinding = ViewEventLogsEventLogItemViewBinding::inflate,
         setViewModel = ViewEventLogsEventLogItemViewBinding::setViewModel

app/src/main/java/org/oppia/android/app/fragment/FragmentComponentImpl.kt

@@ -64,6 +64,7 @@ import org.oppia.android.app.settings.profile.ProfileResetPinFragment
 import org.oppia.android.app.shim.IntentFactoryShimModule
 import org.oppia.android.app.shim.ViewBindingShimModule
 import org.oppia.android.app.story.StoryFragment
+import org.oppia.android.app.testing.DragDropTestFragment
 import org.oppia.android.app.testing.ExplorationTestActivityPresenter
 import org.oppia.android.app.testing.ImageRegionSelectionTestFragment
 import org.oppia.android.app.topic.TopicFragment
@@ -114,6 +115,7 @@ interface FragmentComponentImpl : FragmentComponent, ViewComponentBuilderInjecto
   fun inject(conceptCardFragment: ConceptCardFragment)
   fun inject(developerOptionsFragment: DeveloperOptionsFragment)
   fun inject(downloadsTabFragment: DownloadsTabFragment)
+  fun inject(dragDropTestFragment: DragDropTestFragment)
   fun inject(exitProfileDialogFragment: ExitProfileDialogFragment)
   fun inject(explorationFragment: ExplorationFragment)
   fun inject(explorationManagerFragment: ExplorationManagerFragment)

app/src/main/java/org/oppia/android/app/help/HelpFragmentPresenter.kt

@@ -18,7 +18,8 @@ import javax.inject.Inject
 class HelpFragmentPresenter @Inject constructor(
   private val activity: AppCompatActivity,
   private val fragment: Fragment,
-  private val viewModelProvider: ViewModelProvider<HelpListViewModel>
+  private val viewModelProvider: ViewModelProvider<HelpListViewModel>,
+  private val singleTypeBuilderFactory: BindableAdapter.SingleTypeBuilder.Factory
 ) {
   private lateinit var binding: HelpFragmentBinding
 
@@ -48,8 +49,7 @@ class HelpFragmentPresenter @Inject constructor(
   }
 
   private fun createRecyclerViewAdapter(): BindableAdapter<HelpItemViewModel> {
-    return BindableAdapter.SingleTypeBuilder
-      .newBuilder<HelpItemViewModel>()
+    return singleTypeBuilderFactory.create<HelpItemViewModel>()
       .registerViewDataBinderWithSameModelType(
         inflateDataBinding = HelpItemBinding::inflate,
         setViewModel = HelpItemBinding::setViewModel

app/src/main/java/org/oppia/android/app/help/faq/FAQListFragmentPresenter.kt

@@ -22,7 +22,8 @@ import javax.inject.Inject
 class FAQListFragmentPresenter @Inject constructor(
   private val activity: AppCompatActivity,
   private val fragment: Fragment,
-  private val viewModelProvider: ViewModelProvider<FAQListViewModel>
+  private val viewModelProvider: ViewModelProvider<FAQListViewModel>,
+  private val multiTypeBuilderFactory: BindableAdapter.MultiTypeBuilder.Factory
 ) {
   private lateinit var binding: FaqListFragmentBinding
 
@@ -48,14 +49,13 @@ class FAQListFragmentPresenter @Inject constructor(
   }
 
   private fun createRecyclerViewAdapter(): BindableAdapter<FAQItemViewModel> {
-    return BindableAdapter.MultiTypeBuilder
-      .newBuilder<FAQItemViewModel, ViewType> { viewModel ->
-        when (viewModel) {
-          is FAQHeaderViewModel -> ViewType.VIEW_TYPE_HEADER
-          is FAQContentViewModel -> ViewType.VIEW_TYPE_CONTENT
-          else -> throw IllegalArgumentException("Encountered unexpected view model: $viewModel")
-        }
+    return multiTypeBuilderFactory.create<FAQItemViewModel, ViewType> { viewModel ->
+      when (viewModel) {
+        is FAQHeaderViewModel -> ViewType.VIEW_TYPE_HEADER
+        is FAQContentViewModel -> ViewType.VIEW_TYPE_CONTENT
+        else -> throw IllegalArgumentException("Encountered unexpected view model: $viewModel")
       }
+    }
       .registerViewDataBinder(
         viewType = ViewType.VIEW_TYPE_HEADER,
         inflateDataBinding = FaqItemHeaderBinding::inflate,

app/src/main/java/org/oppia/android/app/help/thirdparty/LicenseListFragmentPresenter.kt

@@ -18,7 +18,8 @@ import javax.inject.Inject
 class LicenseListFragmentPresenter @Inject constructor(
   private val activity: AppCompatActivity,
   private val fragment: Fragment,
-  private val resourceHandler: AppLanguageResourceHandler
+  private val resourceHandler: AppLanguageResourceHandler,
+  private val singleTypeBuilderFactory: BindableAdapter.SingleTypeBuilder.Factory
 ) {
   private lateinit var binding: LicenseListFragmentBinding
 
@@ -51,8 +52,7 @@ class LicenseListFragmentPresenter @Inject constructor(
   }
 
   private fun createRecyclerViewAdapter(): BindableAdapter<LicenseItemViewModel> {
-    return BindableAdapter.SingleTypeBuilder
-      .newBuilder<LicenseItemViewModel>()
+    return singleTypeBuilderFactory.create<LicenseItemViewModel>()
       .registerViewDataBinderWithSameModelType(
         inflateDataBinding = LicenseItemBinding::inflate,
         setViewModel = LicenseItemBinding::setViewModel

app/src/main/java/org/oppia/android/app/help/thirdparty/ThirdPartyDependencyListFragmentPresenter.kt

@@ -18,7 +18,8 @@ import javax.inject.Inject
 class ThirdPartyDependencyListFragmentPresenter @Inject constructor(
   private val activity: AppCompatActivity,
   private val fragment: Fragment,
-  private val viewModelProvider: ViewModelProvider<ThirdPartyDependencyListViewModel>
+  private val viewModelProvider: ViewModelProvider<ThirdPartyDependencyListViewModel>,
+  private val singleTypeBuilderFactory: BindableAdapter.SingleTypeBuilder.Factory
 ) {
   private lateinit var binding: ThirdPartyDependencyListFragmentBinding
 
@@ -50,8 +51,7 @@ class ThirdPartyDependencyListFragmentPresenter @Inject constructor(
   }
 
   private fun createRecyclerViewAdapter(): BindableAdapter<ThirdPartyDependencyItemViewModel> {
-    return BindableAdapter.SingleTypeBuilder
-      .newBuilder<ThirdPartyDependencyItemViewModel>()
+    return singleTypeBuilderFactory.create<ThirdPartyDependencyItemViewModel>()
       .registerViewDataBinderWithSameModelType(
         inflateDataBinding = ThirdPartyDependencyItemBinding::inflate,
         setViewModel = ThirdPartyDependencyItemBinding::setViewModel