From c933bd2a58b3f51097250f3f1b10ba714b0f3e30 Mon Sep 17 00:00:00 2001
From: Sebastian Schuberth <sschuberth@gmail.com>
Date: Thu, 22 Oct 2015 15:11:25 +0200
Subject: [PATCH] Make use of "Groovy truth" for Strings

See http://www.groovy-lang.org/semantics.html#_strings.
---
 .../gradle/plugins/license/LicenseReporter.groovy    | 12 ++++--------
 .../gradle/plugins/license/LicenseResolver.groovy    |  2 +-
 2 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/src/main/groovy/nl/javadude/gradle/plugins/license/LicenseReporter.groovy b/src/main/groovy/nl/javadude/gradle/plugins/license/LicenseReporter.groovy
index a69f806..c1c041c 100644
--- a/src/main/groovy/nl/javadude/gradle/plugins/license/LicenseReporter.groovy
+++ b/src/main/groovy/nl/javadude/gradle/plugins/license/LicenseReporter.groovy
@@ -35,7 +35,7 @@ class LicenseReporter {
                                 def attributes = [name: l.licenseName]
 
                                 // Miss attribute if it's empty
-                                if (!isNullOrEmpty(l.licenseTextUrl)) {
+                                if (l.licenseTextUrl) {
                                     attributes << [url: l.licenseTextUrl]
                                 }
 
@@ -46,10 +46,6 @@ class LicenseReporter {
         }
     }
 
-    private static boolean isNullOrEmpty(String s) {
-        return s == null || s.isEmpty()
-    }
-
     /**
      * Generate xml report grouping by licenses.
      *
@@ -66,7 +62,7 @@ class LicenseReporter {
                     def attributes = [name: entry.key.licenseName]
 
                     // Miss attribute if it's empty
-                    if(!isNullOrEmpty(entry.key.licenseTextUrl)) {
+                    if(entry.key.licenseTextUrl) {
                         attributes << [url:  entry.key.licenseTextUrl]
                     }
                     license(attributes) {
@@ -140,7 +136,7 @@ class LicenseReporter {
                                     td(entry.dependencyFileName, class: 'licenseName')
                                     td(license.licenseName, class: 'licenseName')
                                     td(class: 'license') {
-                                        if (!isNullOrEmpty(license.licenseTextUrl)) {
+                                        if (license.licenseTextUrl) {
                                             a(href: license.licenseTextUrl, "Show license agreement")
                                         }
                                     }
@@ -218,7 +214,7 @@ class LicenseReporter {
                             tr {
                                 td(entry.key.licenseName, class: 'licenseName')
                                 td(class: 'license') {
-                                    if (!isNullOrEmpty(entry.key.licenseTextUrl)) {
+                                    if (entry.key.licenseTextUrl) {
                                         a(href: entry.key.licenseTextUrl, "License agreement")
                                     }
                                 }
diff --git a/src/main/groovy/nl/javadude/gradle/plugins/license/LicenseResolver.groovy b/src/main/groovy/nl/javadude/gradle/plugins/license/LicenseResolver.groovy
index 7989cc1..396ac06 100644
--- a/src/main/groovy/nl/javadude/gradle/plugins/license/LicenseResolver.groovy
+++ b/src/main/groovy/nl/javadude/gradle/plugins/license/LicenseResolver.groovy
@@ -230,7 +230,7 @@ class LicenseResolver {
 
         if (pomData.hasLicense()) {
             pomData
-        } else if (xml.parent.text() != null && !xml.parent().text().isEmpty()) {
+        } else if (xml.parent.text()) {
             String parentGroup = xml.parent.groupId.text().trim()
             String parentName = xml.parent.artifactId.text().trim()
             String parentVersion = xml.parent.version.text().trim()