ZF-4802: setCookie() concats the the new cookie to the previous header cookie string

Description

Add a cookie using the setHeaders() function, then add another cookie using setCookie() - they end up being a single value

Reproduction code:


<?php
require_once 'Zend/Http/Client.php';

$client = new Zend_Http_Client();
$client->setHeaders(array('cookie' => 'are you'));
$client->setCookie('funky', 'monkey');

echo $client->getHeader('cookie'); // will output "are youfunky=monkey; " instead of "are you; funky=monkey; "

Comments

Fixed in revision 12425. Added a check if cookie seperator is present in the string, if not, add it.

forgot to add an @group in my second unit test for the issue. Was added in revision 12426

will need to review this patch, there are hard coded values in tests as well as underscores in test name.

(I will fix this this week, but will keep it assigned to you)

-ralph

removed hardcoded paths from tests in r12522

and underscores have been removed from test method names in r12523

closing issue again, I fixed the raised issues :)

Changing issues in preparation for the 1.7.0 release.