Merge pull request #4309 from medici/amc-crashfix

added a null safety while accessing the graph view
This commit is contained in:
shankar yadav
2022-11-09 20:30:20 +05:30
committed by GitHub Enterprise

View File

@@ -36,6 +36,7 @@ import com.navi.amc.utils.Constant.DATA
import com.navi.amc.utils.Constant.SHOW_BOTTOMSHEET
import com.navi.amc.utils.getBottomSheet
import com.navi.amc.utils.orFalse
import com.navi.amc.utils.orTrue
import com.navi.base.model.ActionData
import com.navi.base.utils.BaseUtils.cacheDirUri
import com.navi.common.downloader.DownloadUtil
@@ -101,8 +102,8 @@ class FundDetailsFragment : AmcBaseFragment() {
val rect = Rect()
scroll.getHitRect(rect)
val view =
binding.container.findViewWithTag<FundGraphView>("graph").getSubtitle()
if (!view.getLocalVisibleRect(rect)) {
binding.container.findViewWithTag<FundGraphView>("graph")?.getSubtitle()
if (!view?.getLocalVisibleRect(rect).orTrue()) {
binding.fundOnscrollView.apply {
left.setSpannableString(viewModel.fundReturn.value?.leftText)
right.setSpannableString(viewModel.fundReturn.value?.rightText)