realurl-extbase-empty-param-hack.diff

Index: class.tx_realurl.php
===================================================================
--- class.tx_realurl.php	(revision 63)
+++ class.tx_realurl.php	(revision 64)
@@ -828,10 +828,12 @@
 		// the same speaking URL in the cache table!)
 		if (isset($paramKeyValues['cHash'])) {

+				//rebuild will be triggered for empty values
 			if ($this->rebuildCHash) {
 				$cHashParameters = array_merge($this->cHashParameters, $paramKeyValues);
 				unset($cHashParameters['cHash']);
-				$cHashParameters = t3lib_div::cHashParams(t3lib_div::implodeArrayForUrl('', $cHashParameters));
+					//we will not accept empty values for cHash generation since we do not add them on rebuilding query string
+				$cHashParameters = t3lib_div::cHashParams(t3lib_div::implodeArrayForUrl('', $cHashParameters, '', true));
 				if (method_exists('t3lib_div', 'calculateCHash')) {
 					$paramKeyValues['cHash'] = t3lib_div::calculateCHash($cHashParameters);
 				}
@@ -1471,7 +1473,9 @@
 				// we stripped empty segments at the end of the URL on encoding.
 				// Reconstruct them or cHash check will fail in TSFE.
 				// Related to bug #15906.
-				$GET_string .= '&' . rawurlencode($setup['GETvar']) . '=';
+
+					// per definition we will not accept empty values within decode
+				// $GET_string .= '&' . rawurlencode($setup['GETvar']) . '=';
 			}
 			else {
 				// Get value and remove from path parts:
@@ -1551,7 +1555,10 @@
 								if (isset($this->extConf['init']['emptySegmentValue']) && $this->extConf['init']['emptySegmentValue'] === $value) {
 									$value = '';
 								}
-								$GET_string .= '&' . rawurlencode($setup['GETvar']) . '=' . rawurlencode($value);
+									// per definition we will not accept empty values within decode
+								if (strlen($value) > 0) {
+									$GET_string .= '&' . rawurlencode($setup['GETvar']) . '=' . rawurlencode($value);
+								}
 							}
 						} else {
 							array_unshift($pathParts, $origValue);