making sure id is urlencoded when using updateDocument

[TroodoNmike]

Yesterday I tried to update existing elasticsearch document using Type class. Because all of my documents ids contain forward slashes elastica was throwing an exception and documents were not updated.

So when using:

(existing document)
$doc->getId() -> '/some/path'
$doc->getData() -> array(data here)

Then using
$type->updateDocument($doc);

Documents was not updated.

I have created a test and fix. Let me know if this is valid. Cheers

Mike

[coveralls]

Coverage increased (+0.0%) when pulling ecefc85 on TroodoNmike:master into 990b89f on ruflin:master.

[ruflin]

@TroodoNmike I understand the problem and I think it needs to be fixed. Because you brought up the issue I realised we use urlencode(...) for id's in quite a few other locations. The reason I don't like it is because I think the urlencoding is specific to the transport.

I will merge the pull request for now as it is in line with most of the other code parts but I will create a ticket to find a general solution here.