Skip to content

Commit

Permalink
[4124] Fix UninitializedPropertyAccessException in AllergensAlertFrag…
Browse files Browse the repository at this point in the history
…ment (#4130)
  • Loading branch information
naivekook committed Aug 8, 2021
1 parent 03f504f commit 5141f8d
Showing 1 changed file with 4 additions and 4 deletions.
Expand Up @@ -66,15 +66,14 @@ class AllergensAlertFragment : NavigationBaseFragment() {
private lateinit var adapter: AllergensAdapter
private val mSettings by lazy { requireActivity().getSharedPreferences("prefs", 0) }
private val dataObserver by lazy { AllergensObserver() }
private val appLang: String by lazy { localeManager.getLanguage() }

override fun onCreateView(inflater: LayoutInflater, container: ViewGroup?, savedInstanceState: Bundle?): View {
setHasOptionsMenu(true)
_binding = FragmentAlertAllergensBinding.inflate(inflater)
return binding.root
}

private val appLang: String by lazy { localeManager.getLanguage() }

override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
super.onViewCreated(view, savedInstanceState)

Expand Down Expand Up @@ -112,7 +111,9 @@ class AllergensAlertFragment : NavigationBaseFragment() {

override fun onDestroyView() {
super.onDestroyView()
adapter.unregisterAdapterDataObserver(dataObserver)
if (this::adapter.isInitialized) {
adapter.unregisterAdapterDataObserver(dataObserver)
}
_binding = null
}

Expand Down Expand Up @@ -219,7 +220,6 @@ class AllergensAlertFragment : NavigationBaseFragment() {
}

companion object {
private val LOG_TAG = AllergensAlertFragment::class.simpleName

@JvmStatic
fun newInstance() = AllergensAlertFragment().apply { arguments = Bundle() }
Expand Down

0 comments on commit 5141f8d

Please sign in to comment.