PHPDoc'ed the Editor class

Added the class properties as protected variables, forcing the use of getters and setters

Switched out some legacy execute functions with the new doctrine functions

Added a new function, createFromRecord, which can be used to instantiate an object so that we don't have duplicate code for doing this when looking up by handle vs ID.

Switched the getList to have an optional pkgID param to eliminate redundant code between that and the getListByPackage function

Changed the getList function to run a query that gets all records, and makes use of the new createFromRecord() function in order to instantiate the objects, but this cuts down on the number of queries that have to get one (instead of 1 query for each record + a query to lookup all the handles, it's not just 1 query)
PSR2 for the rest of the stuff I somehow didn't touch
Merge pull request #756 from ExchangeCore/Editor-Docs

Tidy Up Conversation Editor base class