rentjuice/airbrake-php-client — pull request #1 by pilif (2012-04-10 15:28:58+00:00)
Title: Better handling of array data and backtraces

Hi,

this pull request, amongst trying to unifiy the indentation style that was mixed between spaces and tabs all over the file, adds three things to the plugin:

* the backtrace sent to AirBrake is now better suited for the service. `Exception->getTrace()` doesn't actually contain the location where the error happened by default. This makes it quite annoying to actually find the location of the error if you just have the backtrace
* The ``-Element in the XML you send to AirBrake should be the name of the exception class - not the name of the class where the Exception has happened. AirBrake uses this to keep track of various error types. This pull request contains a commit rectifying this.
* The `` elements can actually have further ``-elements nested in them. This allows for a much nicer inspection in AirBrake than having to read through print_r-output, so this pull-request changes the serialization to support that.

As the whitespace was already mixed all over the file, I have unified it in the first commit. If you don't like this, I can redo the request with the original indentation in place. If you can live with it, but can't live with any of the proposed changes, feel free to cherry-pick - the commits are independent of each other (aside of the whitespace one).

I'm sorry for the whitespace thing, but I noticed the discrepancy only too later after doing all the other changes, so I'm trying my luck with this as it at least takes care of having consistent whitespace across the file.

I hope this helps.

Philip