fixes #295: URL::_buildCoreUrl() added the port number twice if it is nonstandard.

core/trunk/core/class/Url.php

@@ -185,8 +185,12 @@ class Url
 
     private static function _buildHost()
     {
-        return array_key_exists('HTTP_X_FORWARDED_HOST', $_SERVER)
-                ? $_SERVER['HTTP_X_FORWARDED_HOST'] : $_SERVER['HTTP_HOST'];
+        if (array_key_exists('HTTP_X_FORWARDED_HOST', $_SERVER)) {
+            return $_SERVER['HTTP_X_FORWARDED_HOST'];
+        } else {
+            $hostParts = explode(':',$_SERVER['HTTP_HOST']);
+            return $hostParts[0];
+        }
     }
 
     private static function _buildPort()

core/trunk/core/tests/class/UrlTest.php

@@ -25,7 +25,7 @@ class UrlTest extends PHPUnit_Framework_TestCase
         $_SERVER['HTTPS'] = (strpos($url_parts[0], 's') !== false)
                                 ? 'on': '';
         $host_parts = explode(':', $url_parts[2]);
-        $_SERVER['HTTP_HOST'] = $host_parts[0];
+        $_SERVER['HTTP_HOST'] = $url_parts[2];
         if (count($host_parts) == 2) {
             $_SERVER['SERVER_PORT'] = $host_parts[1];
         } else {

core/trunk/core/tests/class/UrlWithProxyTest.php

@@ -25,7 +25,7 @@ class UrlWithProxyTest extends PHPUnit_Framework_TestCase
         $_SERVER['HTTPS'] = (strpos($url_parts[0], 's') !== false)
         ? 'on': '';
         $host_parts = explode(':', $url_parts[2]);
-        $_SERVER['HTTP_HOST'] = $host_parts[0];
+        $_SERVER['HTTP_HOST'] = $url_parts[2];
         if (count($host_parts) == 2) {
             $_SERVER['SERVER_PORT'] = $host_parts[1];
         } else {