diff --git a/androidApp/src/main/java/bou/amine/apps/readerforselfossv2/android/fragments/ArticleFragment.kt b/androidApp/src/main/java/bou/amine/apps/readerforselfossv2/android/fragments/ArticleFragment.kt index 1ebb9dd..83fd978 100644 --- a/androidApp/src/main/java/bou/amine/apps/readerforselfossv2/android/fragments/ArticleFragment.kt +++ b/androidApp/src/main/java/bou/amine/apps/readerforselfossv2/android/fragments/ArticleFragment.kt @@ -157,7 +157,7 @@ class ArticleFragment : Fragment(), DIAware { ) } catch (e: InflateException) { e.sendSilentlyWithAcraWithName("webview not available") - if (context != null) { + try { AlertDialog.Builder(requireContext()) .setMessage(requireContext().getString(R.string.webview_dialog_issue_message)) .setTitle(requireContext().getString(R.string.webview_dialog_issue_title)) @@ -169,6 +169,8 @@ class ArticleFragment : Fragment(), DIAware { } .create() .show() + } catch (e: IllegalStateException) { + e.sendSilentlyWithAcraWithName("Context required is null") } } @@ -218,7 +220,7 @@ class ArticleFragment : Fragment(), DIAware { R.id.share_action -> requireActivity().shareLink(url, contentTitle) R.id.open_action -> requireActivity().openItemUrlInBrowserAsNewTask(this@ArticleFragment.item) R.id.unread_action -> - if (context != null) { + try { if (this@ArticleFragment.item.unread) { CoroutineScope(Dispatchers.IO).launch { repository.markAsRead(this@ArticleFragment.item) @@ -240,6 +242,8 @@ class ArticleFragment : Fragment(), DIAware { Toast.LENGTH_LONG, ).show() } + } catch (e: IllegalStateException) { + e.sendSilentlyWithAcraWithName("Context required is null") } else -> Unit @@ -389,7 +393,7 @@ class ArticleFragment : Fragment(), DIAware { } private fun htmlToWebview() { - if (context != null) { + try { val attrs: IntArray = intArrayOf(android.R.attr.fontFamily) val a: TypedArray = requireContext().obtainStyledAttributes(resId, attrs) @@ -534,6 +538,8 @@ class ArticleFragment : Fragment(), DIAware { "utf-8", null, ) + } catch (e: IllegalStateException) { + e.sendSilentlyWithAcraWithName("Context required is null") } } @@ -549,10 +555,10 @@ class ArticleFragment : Fragment(), DIAware { private fun openInBrowserAfterFailing() { binding.progressBar.visibility = View.GONE - if (context != null) { + try { requireContext().openItemUrlInBrowserAsNewTask(this@ArticleFragment.item) - } else { - Exception("openInBrowserAfterFailing context is null").sendSilentlyWithAcraWithName("openInBrowserAfterFailing > $context") + } catch (e: IllegalStateException) { + e.sendSilentlyWithAcraWithName("Context required is null") } }