JCGay notifier clean up workaround

Author: krystian.panek

src/main/kotlin/com/cognifide/gradle/aem/api/AemConfig.kt

@@ -575,7 +575,7 @@ class AemConfig(
      */
     @Internal
     @JsonIgnore
-    var notificationConfig: (AemNotifier) -> Notifier = { it.dorkbox() }
+    var notificationConfig: (AemNotifier) -> Notifier = { it.factory() }
 
     /**
      * Initialize defaults that depends on concrete type of project.

src/main/kotlin/com/cognifide/gradle/aem/api/AemNotifier.kt

@@ -78,13 +78,23 @@ class AemNotifier private constructor(private val project: Project) {
     }
 
     fun custom(notifier: (title: String, text: String, level: LogLevel) -> Unit): Notifier {
-        return object: Notifier {
+        return object : Notifier {
             override fun notify(title: String, text: String, level: LogLevel) {
                 notifier(title, text, level)
             }
         }
     }
 
+    fun factory(): Notifier {
+        val name = project.properties["aem.notification.config"] ?: "dorkbox"
+
+        return when (name) {
+            "dorkbox" -> dorkbox()
+            "jcgay" -> jcgay()
+            else -> throw AemException("Unsupported notifier: '$name'")
+        }
+    }
+
     companion object {
 
         const val IMAGE_PATH = "/com/cognifide/gradle/aem/META-INF/vault/definition/thumbnail.png"

src/main/kotlin/com/cognifide/gradle/aem/internal/notifier/JcGayNotifier.kt

@@ -8,26 +8,44 @@ import fr.jcgay.notification.Notification
 import fr.jcgay.notification.SendNotification
 import org.gradle.api.Project
 import org.gradle.api.logging.LogLevel
+import org.gradle.util.GFileUtils
+import fr.jcgay.notification.Notifier as Base
 
 class JcGayNotifier(
         val project: Project,
         val appBuilder: Application.Builder.() -> Unit,
         val messageBuilder: Notification.Builder.() -> Unit
 ) : Notifier {
 
-    private val icon by lazy {
-        Icon.create(javaClass.getResource(AemNotifier.IMAGE_PATH), "default")
+    private val notifier by lazy {
+        val props = project.rootProject.extensions.extraProperties
+        if (!props.has(EXT_PROP)) {
+            props.set(EXT_PROP, SendNotification()
+                    .setApplication(Application.builder()
+                            .id(AemPlugin.ID)
+                            .name(AemPlugin.NAME)
+                            .icon(icon)
+                            .apply(appBuilder)
+                            .build())
+                    .initNotifier())
+        }
+
+        val notifier = props.get(EXT_PROP) as Base
+
+        if (project == project.rootProject) {
+            project.gradle.buildFinished {
+                notifier.close()
+
+                // TODO https://github.com/jcgay/send-notification/issues/10
+                GFileUtils.deleteFileQuietly(project.file("null"))
+            }
+        }
+
+        notifier
     }
 
-    private val notifier by lazy {
-        SendNotification()
-                .setApplication(Application.builder()
-                        .id(AemPlugin.ID)
-                        .name(AemPlugin.NAME)
-                        .icon(icon)
-                        .apply(appBuilder)
-                        .build())
-                .initNotifier()
+    private val icon by lazy {
+        Icon.create(javaClass.getResource(AemNotifier.IMAGE_PATH), "default")
     }
 
     override fun notify(title: String, text: String, level: LogLevel) {
@@ -47,6 +65,8 @@ class JcGayNotifier(
 
     companion object {
 
+        val EXT_PROP = JcGayNotifier::class.java.name
+
         val LOG_LEVEL_NOTIFY_MAP = mapOf(
                 LogLevel.ERROR to Notification.Level.ERROR,
                 LogLevel.WARN to Notification.Level.WARNING