From 6459137bb2df1b4af71ada1710c4fb9549415c96 Mon Sep 17 00:00:00 2001
From: Scott Battaglia <scott.battaglia@gmail.com>
Date: Tue, 24 Nov 2009 15:19:15 +0000
Subject: [PATCH] CASC-99

moved the line to make sure we're not accidentally prepending "&"
---
 .../cas/client/validation/AbstractUrlBasedTicketValidator.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cas-client-core/src/main/java/org/jasig/cas/client/validation/AbstractUrlBasedTicketValidator.java b/cas-client-core/src/main/java/org/jasig/cas/client/validation/AbstractUrlBasedTicketValidator.java
index 3323791..fcfcfc8 100644
--- a/cas-client-core/src/main/java/org/jasig/cas/client/validation/AbstractUrlBasedTicketValidator.java
+++ b/cas-client-core/src/main/java/org/jasig/cas/client/validation/AbstractUrlBasedTicketValidator.java
@@ -111,12 +111,12 @@ public abstract class AbstractUrlBasedTicketValidator implements TicketValidator
             buffer.append(suffix);
 
             for (final Iterator iter = urlParameters.entrySet().iterator(); iter.hasNext();) {
-                buffer.append(i++ == 0 ? "?" : "&");
                 final Map.Entry entry = (Map.Entry) iter.next();
                 final String key = (String) entry.getKey();
                 final String value = (String) entry.getValue();
 
                 if (value != null) {
+                    buffer.append(i++ == 0 ? "?" : "&");
 	                buffer.append(key);
 	                buffer.append("=");
 	                buffer.append(value);