Show empty ringtone name in case of RuntimeException

Fixes #116
pull/138/merge
Alinson S. Xavier 9 years ago
parent 88e8aad0d8
commit 59d42fe62f

@ -134,6 +134,7 @@ public class SettingsFragment extends PreferenceFragmentCompat
private void updateRingtoneDescription() private void updateRingtoneDescription()
{ {
String ringtoneName = ReminderHelper.getRingtoneName(getContext()); String ringtoneName = ReminderHelper.getRingtoneName(getContext());
if(ringtoneName == null) return;
Preference ringtonePreference = findPreference("reminderSound"); Preference ringtonePreference = findPreference("reminderSound");
ringtonePreference.setSummary(ringtoneName); ringtonePreference.setSummary(ringtoneName);
} }

@ -144,15 +144,18 @@ public class ReminderHelper
fragment.startActivityForResult(intent, requestCode); fragment.startActivityForResult(intent, requestCode);
} }
@Nullable
public static String getRingtoneName(Context context) public static String getRingtoneName(Context context)
{
try
{ {
Uri ringtoneUri = getRingtoneUri(context); Uri ringtoneUri = getRingtoneUri(context);
String ringtoneName = context.getResources().getString(R.string.none); String ringtoneName = context.getResources().getString(R.string.none);
if(ringtoneUri != null) if (ringtoneUri != null)
{ {
Ringtone ringtone = RingtoneManager.getRingtone(context, ringtoneUri); Ringtone ringtone = RingtoneManager.getRingtone(context, ringtoneUri);
if(ringtone != null) if (ringtone != null)
{ {
ringtoneName = ringtone.getTitle(context); ringtoneName = ringtone.getTitle(context);
ringtone.stop(); ringtone.stop();
@ -161,4 +164,10 @@ public class ReminderHelper
return ringtoneName; return ringtoneName;
} }
catch (RuntimeException e)
{
e.printStackTrace();
return null;
}
}
} }

Loading…
Cancel
Save