From 26d2a8c78c70446121e1f7a3bf941c65c8ebb4bf Mon Sep 17 00:00:00 2001
From: Pascallike <antonio.cerciello@gmail.com>
Date: Tue, 7 Mar 2017 11:25:15 +0100
Subject: [PATCH] StringUtils to check empty string

---
 .../geonet/kernel/harvest/harvester/oaipmh/Harvester.java    | 5 +++--
 .../fao/geonet/kernel/harvest/harvester/oaipmh/Search.java   | 5 +++--
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/harvesters/src/main/java/org/fao/geonet/kernel/harvest/harvester/oaipmh/Harvester.java b/harvesters/src/main/java/org/fao/geonet/kernel/harvest/harvester/oaipmh/Harvester.java
index 3ec76f6a336..98f63c7b277 100644
--- a/harvesters/src/main/java/org/fao/geonet/kernel/harvest/harvester/oaipmh/Harvester.java
+++ b/harvesters/src/main/java/org/fao/geonet/kernel/harvest/harvester/oaipmh/Harvester.java
@@ -34,6 +34,7 @@
 import java.util.Set;
 import java.util.concurrent.atomic.AtomicBoolean;
 
+import org.apache.commons.lang.StringUtils;
 import org.eclipse.emf.common.command.AbortExecutionException;
 import org.fao.geonet.GeonetContext;
 import org.fao.geonet.Logger;
@@ -328,7 +329,7 @@ private void addMetadata(XmlRequest t, RecordInfo ri, String processName, Map<St
         
         
         // Apply the xsl filter choosed by UI
-        if (!params.xslfilter.equals("")) {
+        if (StringUtils.isNotEmpty(params.xslfilter)) {
             md = HarvesterUtil.processMetadata(dataMan.getSchema(schema),
                     md, processName, processParams, log);
         }
@@ -490,7 +491,7 @@ private void updateMetadata(XmlRequest t, RecordInfo ri, String id, String proce
 			String schema = dataMan.autodetectSchema(md, null);
 
 			// Apply the xsl filter choosed by UI
-			if (!params.xslfilter.equals("")) {
+			if (StringUtils.isNotEmpty(params.xslfilter)) {
 			    md = HarvesterUtil.processMetadata(dataMan.getSchema(schema),
 			            md, processName, processParams, log);
 			}
diff --git a/harvesters/src/main/java/org/fao/geonet/kernel/harvest/harvester/oaipmh/Search.java b/harvesters/src/main/java/org/fao/geonet/kernel/harvest/harvester/oaipmh/Search.java
index 7a5ff3f84ef..edd7b9b5609 100644
--- a/harvesters/src/main/java/org/fao/geonet/kernel/harvest/harvester/oaipmh/Search.java
+++ b/harvesters/src/main/java/org/fao/geonet/kernel/harvest/harvester/oaipmh/Search.java
@@ -25,6 +25,7 @@
 
 import java.util.Date;
 
+import org.apache.commons.lang.StringUtils;
 import org.fao.geonet.Util;
 import org.fao.geonet.domain.ISODate;
 import org.fao.geonet.exceptions.BadInputEx;
@@ -60,7 +61,7 @@ public Search(Element search) throws BadInputEx
 
 		try
 		{
-		    if (!from.equals("") && !from.equals("Invalid Date")) {
+		    if (StringUtils.isNotEmpty(from) && !from.equalsIgnoreCase("Invalid Date")) {
 				fromDate = new ISODate(from);
 				from     = fromDate.getDateAsString();
 			} else {
@@ -77,7 +78,7 @@ public Search(Element search) throws BadInputEx
 
 		try
 		{
-		    if (!until.equals("") && !until.equals("Invalid Date")) {
+		    if (StringUtils.isNotEmpty(until) && !until.equalsIgnoreCase("Invalid Date")) {
 				untilDate = new ISODate(until);
 				until     = untilDate.getDateAsString();
 			} else {