Skip to content

Commit

Permalink
Fix part of #5284: Revert "Fix #5266: Call setUpDrawer( ) directly if…
Browse files Browse the repository at this point in the history
… binding already initialized" (#5285)

Fixes part of #5284

Reverts #5276 as part of preparing to revert #5191.

Note that this reintroduces #5266 but that should stay fixed once #5191
is reverted.
  • Loading branch information
BenHenning authored Dec 23, 2023
1 parent 74f4baf commit 31dfd2a
Showing 1 changed file with 0 additions and 15 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -374,21 +374,6 @@ class NavigationDrawerFragmentPresenter @Inject constructor(
this.drawerLayout = drawerLayout
this.toolbar = toolbar
this.menuItemId = menuItemId

/**
* [setUpDrawer] is called directly if binding is already initialized.
* Otherwise, [setUpDrawer] is called from [handleCreateView].
*
* Note: [binding] is already initialized when [initializeDrawer] is called via [onRestart]
* and [handleCreateView] will not be called in that case.
*/
if (this::binding.isInitialized) {
setUpDrawer(
this.drawerLayout,
this.toolbar,
this.menuItemId
)
}
}

/**
Expand Down

0 comments on commit 31dfd2a

Please sign in to comment.