mirror_php/guzzle
1
0
Fork 0
mirror of https://github.com/guzzle/guzzle.git synced 2025-02-25 02:22:57 +01:00
Code Issues Releases Wiki Activity

QueryString::$prefix is now completely unnecessary - removed

Browse Source
This commit is contained in:
David Abdemoulaie 2013-01-30 17:32:43 -06:00
parent 74b55d0c85
commit 8aabcebef1
3 changed files with 0 additions and 48 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
src/Guzzle/Http/Message/EntityEnclosingRequest.php
View File

@ -39,7 +39,6 @@ class EntityEnclosingRequest extends Request implements EntityEnclosingRequestIn
public function __construct($method, $url, $headers = array())
{
$this->postFields = new QueryString();
$this->postFields->setPrefix('');
parent::__construct($method, $url, $headers);
}

29
src/Guzzle/Http/QueryString.php
View File

@ -24,11 +24,6 @@ class QueryString extends Collection
*/
protected $valueSeparator = '=';
/**
* @var string The query string prefix
*/
protected $prefix = '?';
/**
* @var bool URL encode fields and values?
*/
@ -178,16 +173,6 @@ class QueryString extends Collection
return $this->fieldSeparator;
}
/**
* Get the query string prefix
*
* @return string
*/
public function getPrefix()
{
return $this->prefix;
}
/**
* Get the query string value separator
*
@ -252,20 +237,6 @@ class QueryString extends Collection
return $this;
}
/**
* Set the query string prefix
*
* @param string $prefix Prefix to use with the query string (e.g. '?')
*
* @return QueryString
*/
public function setPrefix($prefix)
{
$this->prefix = $prefix;
return $this;
}
/**
* Set the query string value separator
*

18
tests/Guzzle/Tests/Http/QueryStringTest.php
View File

@ -26,14 +26,6 @@ class QueryStringTest extends \Guzzle\Tests\GuzzleTestCase
$this->assertEquals('&', $this->q->getFieldSeparator());
}
/**
* @covers Guzzle\Http\QueryString::getPrefix
*/
public function testGetPrefix()
{
$this->assertEquals('?', $this->q->getPrefix());
}
/**
* @covers Guzzle\Http\QueryString::getValueSeparator
*/
@ -62,16 +54,6 @@ class QueryStringTest extends \Guzzle\Tests\GuzzleTestCase
$this->assertEquals('/', $this->q->getFieldSeparator());
}
/**
* @covers Guzzle\Http\QueryString::setPrefix
* @covers Guzzle\Http\QueryString::getPrefix
*/
public function testSetPrefix()
{
$this->assertEquals($this->q, $this->q->setPrefix(''));
$this->assertEquals('', $this->q->getPrefix());
}
/**
* @covers Guzzle\Http\QueryString::setValueSeparator
* @covers Guzzle\Http\QueryString::getValueSeparator