mirror of
https://github.com/silverstripe/silverstripe-framework
synced 2024-10-22 12:05:37 +00:00
b3931cc707
API CHANGE HTTP::setGetVar() combines any GET parameters in PHP array notation (e.g. "foo[bar]=val") instead of replacing the whole array (merged from r98373) BUGFIX HTTP::setGetVar() uses parse_url() and http_build_query() to add query parameters to an existing URL, instead of doing its own regex-based parsing. This means existing GET parameters are correctly url encoded. (merged from r98373) git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/sapphire/branches/2.3@98375 467b73ca-7a2a-4603-9d3b-597d59a354a9
76 lines
2.2 KiB
PHP
76 lines
2.2 KiB
PHP
<?php
|
|
/**
|
|
* Tests the {@link HTTP} class
|
|
*
|
|
* @package sapphire
|
|
* @subpackage tests
|
|
*/
|
|
class HTTPTest extends SapphireTest {
|
|
|
|
/**
|
|
* Tests {@link HTTP::getLinksIn()}
|
|
*/
|
|
public function testGetLinksIn() {
|
|
$content = '
|
|
<h2>My page</h2>
|
|
<p>A boy went <a href="home/">home</a> to see his <span><a href="mother/">mother</a></span>.</p>
|
|
';
|
|
|
|
$links = HTTP::getLinksIn($content);
|
|
|
|
$this->assertTrue(is_array($links));
|
|
$this->assertTrue(count($links) == 2);
|
|
}
|
|
|
|
/**
|
|
* Tests {@link HTTP::setGetVar()}
|
|
*/
|
|
public function testSetGetVar() {
|
|
// HACK No easy way to get the current URL without the query string or fragment
|
|
$base = Director::absoluteBaseURL() . 'dev/tests/HTTPTest';
|
|
|
|
$this->assertEquals(
|
|
$base . '?foo=bar',
|
|
HTTP::setGetVar('foo', 'bar'),
|
|
'Omitting a URL falls back to current URL'
|
|
);
|
|
|
|
$this->assertEquals(
|
|
Director::absoluteBaseURL() . 'relative/url?foo=bar',
|
|
HTTP::setGetVar('foo', 'bar', 'relative/url'),
|
|
'Relative URL without slash prefix returns URL with absolute base'
|
|
);
|
|
|
|
$this->assertEquals(
|
|
Director::absoluteBaseURL() . '/relative/url?foo=bar',
|
|
HTTP::setGetVar('foo', 'bar', '/relative/url'),
|
|
'Relative URL with slash prefix returns URL with absolute base'
|
|
);
|
|
|
|
$this->assertEquals(
|
|
Director::absoluteBaseURL() . '/relative/url?baz=buz&foo=bar',
|
|
HTTP::setGetVar('foo', 'bar', '/relative/url?baz=buz'),
|
|
'Relative URL with existing query params, and new added key'
|
|
);
|
|
|
|
$this->assertEquals(
|
|
'http://test.com/?foo=new&buz=baz',
|
|
HTTP::setGetVar('foo', 'new', 'http://test.com/?foo=old&buz=baz'),
|
|
'Absolute URL without path and multipe existing query params, overwriting an existing parameter'
|
|
);
|
|
|
|
$this->assertEquals(
|
|
'http://test.com/?foo=new',
|
|
HTTP::setGetVar('foo', 'new', 'http://test.com/?foo=&foo=old'),
|
|
'Absolute URL and empty query param'
|
|
);
|
|
|
|
$this->assertEquals(
|
|
// http_build_query() escapes angular brackets, they should be correctly urldecoded by the browser client
|
|
'http://test.com/?foo%5Btest%5D=one&foo%5Btest%5D=two',
|
|
HTTP::setGetVar('foo[test]', 'two', 'http://test.com/?foo[test]=one'),
|
|
'Absolute URL and PHP array query string notation'
|
|
);
|
|
}
|
|
|
|
} |