D6094: Runtime Permissions: optional Permissions

Albert Vaca Cintora noreply at phabricator.kde.org
Thu Jun 22 21:11:58 UTC 2017


albertvaka added inline comments.

INLINE COMMENTS

> TelephonyPlugin.java:296
>      public String[] getRequiredPermissions() {
> -        return new String[]{Manifest.permission.READ_PHONE_STATE, Manifest.permission.READ_CONTACTS, Manifest.permission.SEND_SMS};
> +        return new String[]{Manifest.permission.READ_PHONE_STATE, Manifest.permission.SEND_SMS};
>      }

I think this one only needs READ_SMS?

REPOSITORY
  R225 KDE Connect - Android application

REVISION DETAIL
  https://phabricator.kde.org/D6094

To: nicolasfella, #kde_connect
Cc: albertvaka, #kde_connect, tfella, aboudhar, seebauer, progwolff, MayeulC, menasshock, ach, apol, hkaelberer
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20170622/e8da2b4b/attachment.html>


More information about the KDEConnect mailing list