release: SDK 1.19.1

Author: arnaud-roland

Sources/build.gradle

@@ -9,7 +9,7 @@ buildscript {
         google()
     }
     dependencies {
-        classpath 'com.android.tools.build:gradle:7.1.2'
+        classpath 'com.android.tools.build:gradle:7.2.1'
         classpath "io.codearte.gradle.nexus:gradle-nexus-staging-plugin:0.21.1"
         // NOTE: Do not place your application dependencies here; they belong
         // in the individual module build.gradle files

Sources/gradle.properties

@@ -19,3 +19,4 @@ org.gradle.configureondemand=false
 android.useAndroidX=true
 android.enableJetifier=true
 gnsp.disableApplyOnlyOnRootProjectEnforcement=true
+kotlin.stdlib.default.dependency=false

Sources/gradle/wrapper/gradle-wrapper.properties

@@ -1,5 +1,5 @@
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-7.2-bin.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-7.3.3-bin.zip
 zipStoreBase=GRADLE_USER_HOME
 zipStorePath=wrapper/dists

Sources/sdk/build.gradle

@@ -1,6 +1,5 @@
 apply plugin: 'com.android.library'
 apply plugin: 'kotlin-android'
-apply plugin: 'kotlin-android-extensions'
 
 def SDK_VERSION = "SDK_VERSION"
 def ENABLE_DEBUG_LOGGER = "ENABLE_DEBUG_LOGGER"
@@ -26,7 +25,7 @@ android {
         minSdkVersion 15
         targetSdkVersion 32
         versionCode 1
-        versionName "1.19.0"
+        versionName "1.19.1"
         buildConfigField "String", SDK_VERSION, "\"$versionName\""
         buildConfigField "Integer", API_LEVEL, '50'
         buildConfigField "Integer", MESSAGING_API_LEVEL, '12'
@@ -80,6 +79,13 @@ android {
         baseline file('lint-baseline.xml')
         lintConfig file('lint.xml')
     }
+
+    publishing {
+        singleVariant("release") {
+            withSourcesJar()
+            withJavadocJar()
+        }
+    }
 }
 
 // Prevent from having accidental kotlin in production code
@@ -91,6 +97,13 @@ afterEvaluate {
     }
 }
 
+// Print the sdk version name
+task printVersion {
+    doLast {
+        println project.version
+    }
+}
+
 dependencies {
     compileOnly "com.google.android.gms:play-services-ads:$playServicesVersion"
     compileOnly "com.google.android.gms:play-services-gcm:$playServicesVersion"
@@ -147,7 +160,7 @@ dependencies {
     testImplementation "org.powermock:powermock-api-mockito2:2.0.7"
     testImplementation "org.powermock:powermock-classloading-xstream:2.0.7"
 
-    testImplementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:$kotlin_version"
+    testImplementation "org.jetbrains.kotlin:kotlin-stdlib-jdk8:$kotlin_version"
     testImplementation "org.jetbrains.kotlinx:kotlinx-coroutines-core:$kotlin_couroutines_version"
     testImplementation "org.jetbrains.kotlinx:kotlinx-coroutines-android:$kotlin_couroutines_version"
 }

Sources/sdk/maven-publish.gradle

@@ -19,50 +19,21 @@ if (signingFile.exists()) {
     }
 }
 
-sourceSets {
-    main {
-        java.srcDirs += 'build/generated'
-    }
-}
-
-task androidJavadocs(type: Javadoc) {
-    source = sourceSets.main.java.srcDirs
-    classpath += project.files(android.getBootClasspath().join(File.pathSeparator))
-    options.addStringOption('Xdoclint:none', '-quiet')
-    android.libraryVariants.all { variant ->
-        if (variant.name == 'release') {
-            owner.classpath += variant.getCompileClasspath()
-        }
-    }
-    exclude '**/gif/README'
-    exclude '**/R.html', '**/R.*.html', '**/index.html'
-}
-
-task androidJavadocsJar(type: Jar, dependsOn: androidJavadocs) {
-    archiveClassifier.set("javadoc")
-    from androidJavadocs.destinationDir
-}
-
-task androidSourcesJar(type: Jar) {
-    archiveClassifier.set("sources")
-    from sourceSets.main.java.srcDirs
-}
-
 nexusStaging {
     packageGroup = mavenGroupId
 }
 
 publishing {
     publications {
-        batchsdk(MavenPublication) {
+        release(MavenPublication) {
 
             groupId mavenGroupId
             artifactId mavenArtifact
             version android.defaultConfig.versionName
 
-            artifact(project.rootProject.file("../public-sdk/Batch.aar"))
-            artifact androidJavadocsJar
-            artifact androidSourcesJar
+            afterEvaluate {
+                from components.release
+            }
 
             pom {
                 name = "Batch.com Android SDK"
@@ -210,19 +181,6 @@ publishing {
                     url = "https://batch.com"
                 }
             }
-
-            pom.withXml {
-                def dependenciesNode = asNode().appendNode('dependencies')
-
-                // Add dependencies manually as we have only one
-                // See : https://discuss.gradle.org/t/maven-publish-doesnt-include-dependencies-in-the-project-pom-file/8544
-                def dependencyNode = dependenciesNode.appendNode('dependency')
-                dependencyNode.appendNode('groupId', 'androidx.core')
-                dependencyNode.appendNode('artifactId', 'core')
-                dependencyNode.appendNode('version', androidXLibraryVersion)
-                dependencyNode.appendNode('scope', 'compile')
-                dependencyNode.appendNode('type', 'aar')
-            }
         }
     }
 
@@ -246,7 +204,7 @@ publishing {
 }
 
 signing {
-    sign publishing.publications.batchsdk
+    sign publishing.publications.release
 }
 
 // Do not sign if we run the publishToMavenLocal task, useful for dev

Sources/sdk/metalava.gradle

@@ -10,6 +10,14 @@
  * We can compare a previously generated signature file with the current API using the metalavaCompareSignature task.
  */
 
+configurations {
+    metalavaRuntime
+}
+
+dependencies {
+    metalavaRuntime 'com.android.tools.metalava:metalava:1.0.0-alpha04'
+}
+
 /**
  * Recursively list all packages from a directory
  * The package must contains at least one .java file to be included
@@ -39,7 +47,7 @@ void getPackagesInDirectory(File rootDirectory, File directory, Set<String> pack
  */
 List<String> getBaseMetalavaArgs() {
     def args = [
-            '-cp', project.rootProject.file('../prebuilt/*'),
+            '-cp', configurations.metalavaRuntime.asPath,
             'com.android.tools.metalava.Driver',
             '--format=v2',
     ]
@@ -100,17 +108,24 @@ task metalavaGenerateSignature {
         addBaseline(signatureArgs, project.file('metalava-baseline.txt'))
         signatureArgs.addAll([
                 '--source-path', android.sourceSets.main.java.srcDirs.toArray().join(':'),
-                '--api', project.rootProject.file('../metalava-api/' + android.defaultConfig.versionName + '/api.txt')
+                '--api', project.rootProject.file('../metalava-api/api-latest.txt')
         ])
         exec {
             executable = 'java'
             args = signatureArgs
         }
+    }
+}
 
+// Generate a release metalava signature
+// This builds api-latest and copies it to a versioned release
+task("metalavaGenerateTaggedSignature", dependsOn: ["metalavaGenerateSignature"]) {
+    doFirst {
         copy {
-            from project.rootProject.file('../metalava-api/' + android.defaultConfig.versionName + '/api.txt')
-            into project.rootProject.file('../metalava-api')
-            rename 'api.txt', 'api-latest.txt'
+            from project.rootProject.file('../metalava-api/api-latest.txt')
+            into project.rootProject.file('../metalava-api/' + android.defaultConfig.versionName + '/')
+
+            rename 'api-latest.txt', 'api.txt'
         }
     }
 }

Sources/sdk/src/main/java/com/batch/android/BatchEventDispatcher.java

@@ -13,6 +13,10 @@ public interface BatchEventDispatcher {
     /**
      * Get the name of the dispatcher
      * This information is only used for analytics
+     *
+     * /!\ `default` method can lead to an AbstractMethodError if java 8 language features are not available (api < 24).
+     * Ensure catching the exception when using this method or use reflection check.
+     *
      * @return the name of the dispatcher
      */
     @Nullable
@@ -23,6 +27,10 @@ default String getName() {
     /**
      * Get the version of the dispatcher
      * This information is only used for analytics
+     *
+     * /!\ `default` method can lead to an AbstractMethodError if java 8 language features are not available (api < 24).
+     * Ensure catching the exception when using this method or use reflection check.
+     *
      * @return the version of the dispatcher
      */
     default int getVersion() {

Sources/sdk/src/main/java/com/batch/android/BatchPushMessageReceiver.java

@@ -92,9 +92,10 @@ protected boolean presentNotification(@NonNull Context context, @NonNull Intent
             try {
                 startPresentationService(context, intent);
                 return true;
-            } catch (IllegalStateException e) {
-                // This exception can happen on Android O, fallback on scheduling a Job.
+            } catch (IllegalStateException | SecurityException e) {
+                // IllegalStateException can happen on Android O, fallback on scheduling a Job.
                 // On earlier Android versions, it should not happen.
+                // The SecurityException happens on some OEMs, like Wiko, for an unknown reason.
                 if (android.os.Build.VERSION.SDK_INT >= android.os.Build.VERSION_CODES.O) {
                     Logger.internal(TAG, "Failed to start service, scheduling Job");
                     return scheduleJob(context, intent);

Sources/sdk/src/main/java/com/batch/android/eventdispatcher/DispatcherSerializer.java

@@ -50,7 +50,7 @@ public static JSONObject serialize(@NonNull Set<BatchEventDispatcher> dispatcher
                     ? dispatcher.getName()
                     : CUSTOM_DISPATCHER_NAME;
                 json.put(name, dispatcher.getVersion());
-            } catch (JSONException ignored) {}
+            } catch (JSONException | AbstractMethodError ignored) {}
         }
         return json;
     }

Sources/sdk/src/main/java/com/batch/android/messaging/fragment/UniversalTemplateFragment.java

@@ -40,7 +40,6 @@
 import java.io.IOException;
 import java.util.Locale;
 import java.util.Map;
-import org.jetbrains.annotations.NotNull;
 
 /**
  * Universal messaging template fragment class. Extends DialogFragment so it can be displayed in its own window easily.
@@ -122,7 +121,7 @@ public Dialog onCreateDialog(Bundle savedInstanceState) {
     @Nullable
     @Override
     public View onCreateView(
-        @NotNull LayoutInflater inflater,
+        @NonNull LayoutInflater inflater,
         @Nullable ViewGroup container,
         @Nullable Bundle savedInstanceState
     ) {