Avoid undefined indexes in notifications

[twisted1919]

trafficstars

If you have a webhook waiting information, with code similar to :

/* parse the incoming request in a meaningful response */
$response = app()->paymentManager->getProvider('2checkout')->getGateway()->acceptNotification()->send();

And someone does a post request without any data, it will trigger warnings like Undefined index: sale_id because there is basically no check made to see if the given array keys do exist.

This pull request checks using isset to make sure the array keys do exist before trying to fetch their value. Initially i thought to set the values to empty strings if they are not set, but in my second commit i used null as it makes more sense in this context. I also included a test for it.

Please keep in mind i am not familiar at all with how your library works internally, so please review the changes carefully.

Thank you.

[coveralls]

Coverage increased (+0.4%) to 64.858% when pulling b92c0c4386cfe6bb93df55d74d97841e0d3cce4b on twisted1919:master into c317aad071cddb0bd022fb9e3a6e87cad38de730 on collizo4sky:master.

[xwiz]

Please merge @w3guy