Skip to content

Commit

Permalink
add permission check for notification
Browse files Browse the repository at this point in the history
  • Loading branch information
Strubbl committed Jul 11, 2023
1 parent dc2e329 commit 137e321
Showing 1 changed file with 13 additions and 0 deletions.
13 changes: 13 additions & 0 deletions app/src/main/java/fr/gaulupeau/apps/Poche/tts/TtsService.java
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
package fr.gaulupeau.apps.Poche.tts;

import android.Manifest;
import android.app.Notification;
import android.app.PendingIntent;
import android.app.Service;
Expand All @@ -8,6 +9,7 @@
import android.content.Context;
import android.content.Intent;
import android.content.IntentFilter;
import android.content.pm.PackageManager;
import android.graphics.Bitmap;
import android.media.AudioManager;
import android.media.MediaPlayer;
Expand All @@ -26,6 +28,7 @@
import android.util.Log;

import androidx.annotation.NonNull;
import androidx.core.app.ActivityCompat;
import androidx.core.app.NotificationCompat;
import androidx.core.app.NotificationManagerCompat;
import androidx.core.util.Pair;
Expand Down Expand Up @@ -1152,6 +1155,16 @@ private void setForegroundAndNotification(boolean forceForeground) {
.from(getApplicationContext());

if (showNotification) {
if (ActivityCompat.checkSelfPermission(this, Manifest.permission.POST_NOTIFICATIONS) != PackageManager.PERMISSION_GRANTED) {
// TODO: Consider calling
// ActivityCompat#requestPermissions
// here to request the missing permissions, and then overriding
// public void onRequestPermissionsResult(int requestCode, String[] permissions,
// int[] grantResults)
// to handle the case where the user grants the permission. See the documentation
// for ActivityCompat#requestPermissions for more details.
return;
}
notificationManager.notify(NOTIFICATION_ID, generateNotification());
} else {
notificationManager.cancel(NOTIFICATION_ID);
Expand Down

0 comments on commit 137e321

Please sign in to comment.