From 6840b3df74dd38746f95a64dfb42204ad2708752 Mon Sep 17 00:00:00 2001 From: Gavriil Sitnikov Date: Sat, 8 Apr 2017 22:43:26 +0300 Subject: [PATCH] warning of PhoneSpan fixed --- .../touchin/roboswag/components/utils/spans/PhoneSpan.java | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/main/java/ru/touchin/roboswag/components/utils/spans/PhoneSpan.java b/src/main/java/ru/touchin/roboswag/components/utils/spans/PhoneSpan.java index f001e23..5f84f91 100644 --- a/src/main/java/ru/touchin/roboswag/components/utils/spans/PhoneSpan.java +++ b/src/main/java/ru/touchin/roboswag/components/utils/spans/PhoneSpan.java @@ -1,5 +1,6 @@ package ru.touchin.roboswag.components.utils.spans; +import android.content.ActivityNotFoundException; import android.content.Intent; import android.net.Uri; import android.support.annotation.NonNull; @@ -19,7 +20,6 @@ public class PhoneSpan extends URLSpan { super(phoneNumber); } - @SuppressWarnings("PMD.AvoidCatchingThrowable") @Override public void onClick(@NonNull final View widget) { super.onClick(widget); @@ -27,9 +27,8 @@ public class PhoneSpan extends URLSpan { final Intent intent = new Intent(Intent.ACTION_DIAL); intent.setData(Uri.parse(getURL())); widget.getContext().startActivity(intent); - // it should catch throwable to not crash in production if there are problems with startActivity() - } catch (final Throwable throwable) { - Lc.assertion(throwable); + } catch (final ActivityNotFoundException exception) { + Lc.assertion(exception); } }