Ok, you've convinced me :)

On Mon, May 27, 2013 at 5:44 AM, Joe Hoyle <notifications@github.com>
wrote:

>> @@ -110,4 +122,35 @@ public static function get_by_id( $id ) {
>>  		}
>>  	}
>>  
>> +	/**
>> +	 * Query for events by filter params
>> +	 * 
>> +	 * @param $args array
>> +	 * @return Notify_Event[]
>> +	 */
>> +	public static function query( $args ) {
> I actually quite like this pattern, it's pretty common in other ORMs, it means there is only one class to interface with for `Notify_Events`s, I think it creates a good place for "query" functions to live - I do get it's not very "WordPressy" but neither is any of this project!
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/humanmade/Notify-Humans/pull/4/files#r4399994