Some more String.isEmpty() replacements

Also fixes header comparison error

git-svn-id: https://joinup.ec.europa.eu/svn/pdf-over/trunk@327 174cde9d-5d70-4d2a-aa98-46368bc2aaf7

4 files changed, 13 insertions, 13 deletions

diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/LocalBKUState.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/LocalBKUState.java
index 2f498a98..908ed244 100644
--- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/LocalBKUState.java
+++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/LocalBKUState.java
@@ -96,14 +96,14 @@ public class LocalBKUState extends State {
 					String server = ""; //$NON-NLS-1$
 					String userAgent = ""; //$NON-NLS-1$
 					String signatureLayout = null; 
-					if (method.getResponseHeader(BKU_RESPONSE_HEADER_SERVER) != null&& 
-						!method.getResponseHeader(BKU_RESPONSE_HEADER_SERVER).equals("")) { //$NON-NLS-1$
+					if (method.getResponseHeader(BKU_RESPONSE_HEADER_SERVER) != null &&
+						!method.getResponseHeader(BKU_RESPONSE_HEADER_SERVER).getValue().isEmpty()) {
 						server = method.getResponseHeader(
 								BKU_RESPONSE_HEADER_SERVER).getValue();
 					}
 
 					if (method.getResponseHeader(BKU_RESPONSE_HEADER_USERAGENT) != null &&
-						!method.getResponseHeader(BKU_RESPONSE_HEADER_USERAGENT).equals("")) { //$NON-NLS-1$
+						!method.getResponseHeader(BKU_RESPONSE_HEADER_USERAGENT).getValue().isEmpty()) {
 						userAgent = method.getResponseHeader(
 								BKU_RESPONSE_HEADER_USERAGENT).getValue();
 					}
@@ -111,7 +111,7 @@ public class LocalBKUState extends State {
 					if (method
 							.getResponseHeader(BKU_RESPONSE_HEADER_SIGNATURE_LAYOUT) != null &&
 						!method
-							.getResponseHeader(BKU_RESPONSE_HEADER_SIGNATURE_LAYOUT).equals("")) { //$NON-NLS-1$
+							.getResponseHeader(BKU_RESPONSE_HEADER_SIGNATURE_LAYOUT).getValue().isEmpty()) {
 						signatureLayout = method.getResponseHeader(
 								BKU_RESPONSE_HEADER_SIGNATURE_LAYOUT).getValue();
 					}
diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/MobileBKUState.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/MobileBKUState.java
index 1a13c794..bc831766 100644
--- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/MobileBKUState.java
+++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/MobileBKUState.java
@@ -177,9 +177,9 @@ public class MobileBKUState extends State {
 
 			// check if we have everything we need!
 			if (mobileStatus.getPhoneNumber() != null
-					&& !mobileStatus.getPhoneNumber().equals("") //$NON-NLS-1$
+					&& !mobileStatus.getPhoneNumber().isEmpty()
 					&& mobileStatus.getMobilePassword() != null
-					&& !mobileStatus.getMobilePassword().equals("")) { //$NON-NLS-1$
+					&& !mobileStatus.getMobilePassword().isEmpty()) {
 				// post to bku
 				Thread postCredentialsThread = new Thread(
 						new PostCredentialsThread(this));
@@ -212,7 +212,7 @@ public class MobileBKUState extends State {
 					// We need number and password => show UI!
 
 					if (mobileStatus.getErrorMessage() != null
-							&& !mobileStatus.getErrorMessage().equals("")) { //$NON-NLS-1$
+							&& !mobileStatus.getErrorMessage().isEmpty()) {
 						// set possible error message
 						ui.setErrorMessage(mobileStatus.getErrorMessage());
 						mobileStatus.setErrorMessage(null);
@@ -220,13 +220,13 @@ public class MobileBKUState extends State {
 					
 
 					if (ui.getMobileNumber() == null
-							|| ui.getMobileNumber().equals("")) { //$NON-NLS-1$
+							|| ui.getMobileNumber().isEmpty()) {
 						// set possible phone number
 						ui.setMobileNumber(mobileStatus.getPhoneNumber());
 					}
 
 					if (ui.getMobilePassword() == null
-							|| ui.getMobilePassword().equals("")) { //$NON-NLS-1$
+							|| ui.getMobilePassword().isEmpty()) {
 						// set possible password
 						ui.setMobilePassword(mobileStatus.getMobilePassword());
 					}
diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/PositioningState.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/PositioningState.java
index 2d610f24..dbd70fce 100644
--- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/PositioningState.java
+++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/PositioningState.java
@@ -68,7 +68,7 @@ public class PositioningState extends State {
 		SignatureParameter param = this.stateMachine.getPDFSigner().getPDFSigner().newParameter();
 		Emblem emblem = new FileNameEmblem(this.stateMachine.getConfigProvider().getDefaultEmblem());
 		param.setEmblem(emblem);
-		if(this.stateMachine.getConfigProvider().getSignatureNote() != null && !this.stateMachine.getConfigProvider().getSignatureNote().equals("")) { //$NON-NLS-1$
+		if(this.stateMachine.getConfigProvider().getSignatureNote() != null && !this.stateMachine.getConfigProvider().getSignatureNote().isEmpty()) {
 			param.setProperty("SIG_NOTE", this.stateMachine.getConfigProvider().getSignatureNote()); //$NON-NLS-1$
 		}
 		
diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/PrepareSigningState.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/PrepareSigningState.java
index c58417ac..1fdc4228 100644
--- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/PrepareSigningState.java
+++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/PrepareSigningState.java
@@ -76,7 +76,7 @@ public class PrepareSigningState extends State {
 					System.setProperty("https.proxyPort", Integer.toString(proxyPort)); //$NON-NLS-1$
 				} 
 				
-				if(proxyHost != null && !proxyHost.equals("")) { //$NON-NLS-1$
+				if(proxyHost != null && !proxyHost.isEmpty()) {
 					System.setProperty("http.proxyHost", proxyHost); //$NON-NLS-1$
 					System.setProperty("https.proxyHost", proxyHost); //$NON-NLS-1$
 				} 
@@ -93,11 +93,11 @@ public class PrepareSigningState extends State {
 				this.state.signatureParameter.setSignatureDevice(status.getBKU());
 				this.state.signatureParameter.setSignaturePosition(status.getSignaturePosition());
 				
-				if(configuration.getDefaultEmblem() != null && !configuration.getDefaultEmblem().equals("")) { //$NON-NLS-1$
+				if(configuration.getDefaultEmblem() != null && !configuration.getDefaultEmblem().isEmpty()) {
 					this.state.signatureParameter.setEmblem(new FileNameEmblem(configuration.getDefaultEmblem()));
 				}
 				
-				if(configuration.getSignatureNote() != null && !configuration.getSignatureNote().equals("")) { //$NON-NLS-1$
+				if(configuration.getSignatureNote() != null && !configuration.getSignatureNote().isEmpty()) {
 					this.state.signatureParameter.setProperty("SIG_NOTE", configuration.getSignatureNote()); //$NON-NLS-1$
 				}