Skip to content

Commit 91e241d

Browse files
committed
Merge branch 'master' into major_release_4.0.0
2 parents 982efec + cfb5574 commit 91e241d

39 files changed

+884
-747
lines changed

OneSignalSDK.jar

135 Bytes
Binary file not shown.

OneSignalSDK/onesignal/consumer-proguard-rules.pro

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@
3333

3434
-keep class com.onesignal.shortcutbadger.impl.AdwHomeBadger { <init>(...); }
3535
-keep class com.onesignal.shortcutbadger.impl.ApexHomeBadger { <init>(...); }
36-
-keep class com.onesignal.shortcutbadger.impl.AsusHomeLauncher { <init>(...); }
36+
-keep class com.onesignal.shortcutbadger.impl.AsusHomeBadger { <init>(...); }
3737
-keep class com.onesignal.shortcutbadger.impl.DefaultBadger { <init>(...); }
3838
-keep class com.onesignal.shortcutbadger.impl.EverythingMeHomeBadger { <init>(...); }
3939
-keep class com.onesignal.shortcutbadger.impl.HuaweiHomeBadger { <init>(...); }

OneSignalSDK/onesignal/maven-push.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ class Global {
2424
static def POM_NAME = 'OneSignal'
2525
static def POM_ARTIFACT_ID = 'OneSignal'
2626
static def POM_PACKAGING = 'aar'
27-
static def VERSION_NAME = '3.15.1'
27+
static def VERSION_NAME = '3.15.3'
2828

2929
static def GROUP_ID = 'com.onesignal'
3030
static def POM_DESCRIPTION = 'OneSignal Android SDK'

OneSignalSDK/onesignal/src/main/java/com/onesignal/BadgeCountUpdater.java

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,6 @@
3131
import android.content.pm.ApplicationInfo;
3232
import android.content.pm.PackageManager;
3333
import android.database.Cursor;
34-
import android.database.sqlite.SQLiteDatabase;
3534
import android.os.Build;
3635
import android.os.Bundle;
3736
import android.service.notification.StatusBarNotification;
@@ -73,14 +72,14 @@ private static boolean areBadgesEnabled(Context context) {
7372
return areBadgeSettingsEnabled(context) && OSUtils.areNotificationsEnabled(context);
7473
}
7574

76-
static void update(SQLiteDatabase readableDb, Context context) {
75+
static void update(OneSignalDb db, Context context) {
7776
if (!areBadgesEnabled(context))
7877
return;
7978

8079
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.M)
8180
updateStandard(context);
8281
else
83-
updateFallback(readableDb, context);
82+
updateFallback(db, context);
8483
}
8584

8685
@RequiresApi(api = Build.VERSION_CODES.M)
@@ -97,8 +96,8 @@ private static void updateStandard(Context context) {
9796
updateCount(runningCount, context);
9897
}
9998

100-
private static void updateFallback(SQLiteDatabase readableDb, Context context) {
101-
Cursor cursor = readableDb.query(
99+
private static void updateFallback(OneSignalDb db, Context context) {
100+
Cursor cursor = db.query(
102101
NotificationTable.TABLE_NAME,
103102
null,
104103
OneSignalDbHelper.recentUninteractedWithNotificationsWhere().toString(),

OneSignalSDK/onesignal/src/main/java/com/onesignal/GenerateNotification.java

Lines changed: 6 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,6 @@
3636
import android.content.pm.PackageManager;
3737
import android.content.res.Resources;
3838
import android.database.Cursor;
39-
import android.database.sqlite.SQLiteDatabase;
4039
import android.graphics.Bitmap;
4140
import android.graphics.BitmapFactory;
4241
import android.net.Uri;
@@ -470,8 +469,6 @@ private static void createSummaryNotification(OSNotificationGenerationJob notifi
470469
Cursor cursor = null;
471470

472471
try {
473-
SQLiteDatabase readableDb = dbHelper.getSQLiteDatabaseWithRetries();
474-
475472
String[] retColumn = { NotificationTable.COLUMN_NAME_ANDROID_NOTIFICATION_ID,
476473
NotificationTable.COLUMN_NAME_FULL_DATA,
477474
NotificationTable.COLUMN_NAME_IS_SUMMARY,
@@ -487,7 +484,7 @@ private static void createSummaryNotification(OSNotificationGenerationJob notifi
487484
if (!updateSummary && notificationJob.getAndroidId() != -1)
488485
whereStr += " AND " + NotificationTable.COLUMN_NAME_ANDROID_NOTIFICATION_ID + " <> " + notificationJob.getAndroidId();
489486

490-
cursor = readableDb.query(
487+
cursor = dbHelper.query(
491488
NotificationTable.TABLE_NAME,
492489
retColumn,
493490
whereStr,
@@ -709,28 +706,11 @@ private static Intent createBaseSummaryIntent(int summaryNotificationId, JSONObj
709706
private static void createSummaryIdDatabaseEntry(OneSignalDbHelper dbHelper, String group, int id) {
710707
// There currently isn't a visible notification from for this group_id.
711708
// Save the group summary notification id so it can be updated later.
712-
SQLiteDatabase writableDb = null;
713-
try {
714-
writableDb = dbHelper.getSQLiteDatabaseWithRetries();
715-
writableDb.beginTransaction();
716-
717-
ContentValues values = new ContentValues();
718-
values.put(NotificationTable.COLUMN_NAME_ANDROID_NOTIFICATION_ID, id);
719-
values.put(NotificationTable.COLUMN_NAME_GROUP_ID, group);
720-
values.put(NotificationTable.COLUMN_NAME_IS_SUMMARY, 1);
721-
writableDb.insertOrThrow(NotificationTable.TABLE_NAME, null, values);
722-
writableDb.setTransactionSuccessful();
723-
} catch (Throwable t) {
724-
OneSignal.Log(OneSignal.LOG_LEVEL.ERROR, "Error adding summary notification record! ", t);
725-
} finally {
726-
if (writableDb != null) {
727-
try {
728-
writableDb.endTransaction(); // May throw if transaction was never opened or DB is full.
729-
} catch (Throwable t) {
730-
OneSignal.Log(OneSignal.LOG_LEVEL.ERROR, "Error closing transaction! ", t);
731-
}
732-
}
733-
}
709+
ContentValues values = new ContentValues();
710+
values.put(NotificationTable.COLUMN_NAME_ANDROID_NOTIFICATION_ID, id);
711+
values.put(NotificationTable.COLUMN_NAME_GROUP_ID, group);
712+
values.put(NotificationTable.COLUMN_NAME_IS_SUMMARY, 1);
713+
dbHelper.insertOrThrow(NotificationTable.TABLE_NAME, null, values);
734714
}
735715

736716
// Keep 'throws Throwable' as 'onesignal_bgimage_notif_layout' may not be available

0 commit comments

Comments
 (0)