From b52e49dee78b8627fb29e129593290127eebb6ef Mon Sep 17 00:00:00 2001
From: Sean Harvey <sean@silverstripe.com>
Date: Wed, 16 Dec 2009 05:48:06 +0000
Subject: [PATCH] BUGFIX ComplexTableField - fixed sprintf() not enough
 arguments when the referrer has a % character in it, the referrer is now an
 argument instead of being hardcoded in the string (from r95093)

git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/sapphire/trunk@95633 467b73ca-7a2a-4603-9d3b-597d59a354a9
---
 forms/ComplexTableField.php | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/forms/ComplexTableField.php b/forms/ComplexTableField.php
index 2a442be32..d146c22a0 100755
--- a/forms/ComplexTableField.php
+++ b/forms/ComplexTableField.php
@@ -814,7 +814,8 @@ class ComplexTableField_ItemRequest extends RequestHandler {
 		$referrer = isset($_SERVER['HTTP_REFERER']) ? $_SERVER['HTTP_REFERER'] : null;
 		
 		$closeLink = sprintf(
-			'<small><a href="' . str_replace('%', '%%', $referrer) . '" onclick="javascript:window.top.GB_hide(); return false;">(%s)</a></small>',
+			'<small><a href="%s" onclick="javascript:window.top.GB_hide(); return false;">(%s)</a></small>',
+			$referrer,
 			_t('ComplexTableField.CLOSEPOPUP', 'Close Popup')
 		);
 		$message = sprintf(