concrete5/concrete5-5.7.0 — pull request #756 by EC-Joe (2014-08-22 04:10:44+00:00)
Title: Tidy Up Conversation Editor base class

PHPDocs

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)

whew... sorry this was just going to be a PHPDoc commit but the more I got into it the more opportunities I found, I'll try to make my commits a bit more granular next time