https://bugs.kde.org/show_bug.cgi?id=392853

Matthijs Tijink <matthijstij...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://commits.kde.org/kde
                   |                            |connect-android/2cd7d4b7a99
                   |                            |aa0073e1a29627b5b10628fa840
                   |                            |fd
         Resolution|---                         |FIXED
             Status|CONFIRMED                   |RESOLVED

--- Comment #3 from Matthijs Tijink <matthijstij...@gmail.com> ---
Git commit 2cd7d4b7a99aa0073e1a29627b5b10628fa840fd by Matthijs Tijink.
Committed on 09/04/2018 at 17:00.
Pushed by mtijink into branch 'master'.

Fix default Find My Phone ringtone

Summary:
Previously, if the user hadn't set a ringtone, the code raised an exception
(because an empty string isn't an uri). Thus, no sound was played.

This fixes it.

Test Plan: Without having set a ringtone, it works. Otherwise, it works too.

Reviewers: #kde_connect, albertvaka

Reviewed By: #kde_connect, albertvaka

Subscribers: nicolasfella, sredman, #kde_connect

Tags: #kde_connect

Differential Revision: https://phabricator.kde.org/D12070

M  +9    -2   
src/org/kde/kdeconnect/Plugins/FindMyPhonePlugin/FindMyPhoneActivity.java

https://commits.kde.org/kdeconnect-android/2cd7d4b7a99aa0073e1a29627b5b10628fa840fd

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to