EE_Event

Complexity

Total Complexity

169

Size/Duplication

Total Lines	1277
Duplicated Lines	9.48 %

Coupling/Cohesion

Components	2
Dependencies	14

78 Methods

Rating	Name	Duplication	Size	Complexity
A	new_instance()	0	4	2
A	new_instance_from_db()	0	3	1
A	set()	0	9	2
B	set_status()	21	26	4
A	datetimes()	0	3	1
A	datetimes_in_chronological_order()	0	3	1
A	datetimes_ordered()	0	3	1
A	first_datetime()	0	3	1
A	primary_datetime()	0	7	2
B	tickets()	0	25	5
A	additional_limit()	0	3	1
A	allow_overflow()	0	3	1
A	created()	0	3	1
A	description()	0	3	1
A	description_filtered()	0	3	1
A	display_description()	0	3	1
A	display_ticket_selector()	0	3	1
A	external_url()	0	3	1
A	member_only()	0	3	1
A	phone()	0	3	1
A	modified()	0	3	1
A	name()	0	3	1
A	order()	0	3	1
A	default_registration_status()	0	4	2
A	short_description()	0	10	3
A	slug()	0	3	1
A	timezone_string()	0	3	1
A	visible_on()	0	3	1
A	wp_user()	0	3	1
A	donations()	0	3	1
A	set_additional_limit()	0	3	1
A	set_created()	0	3	1
A	set_description()	0	3	1
A	set_display_description()	0	3	1
A	set_display_ticket_selector()	0	3	1
A	set_external_url()	0	3	1
A	set_member_only()	0	3	1
A	set_event_phone()	0	3	1
A	set_modified()	0	3	1
A	set_name()	0	3	1
A	set_order()	0	3	1
A	set_short_description()	0	3	1
A	set_slug()	0	3	1
A	set_timezone_string()	0	3	1
A	set_visible_on()	0	3	1
A	set_wp_user()	0	3	1
A	set_default_registration_status()	0	3	1
A	set_donations()	0	3	1
A	add_venue()	0	3	1
A	remove_venue()	0	3	1
A	venues()	0	3	1
C	_has_ID_and_is_published()	0	4	7
B	is_upcoming()	25	25	6
B	is_active()	25	25	6
B	is_expired()	21	21	6
A	is_inactive()	0	7	2
D	perform_sold_out_status_check()	0	40	10
A	spaces_remaining_for_sale()	0	15	3
F	total_available_spaces()	0	132	25
A	is_sold_out()	0	8	2
A	is_postponed()	0	3	1
A	is_cancelled()	0	3	1
C	get_active_status()	0	50	11
A	pretty_active_status()	0	9	2
A	get_number_of_tickets_sold()	0	13	4
A	get_count_of_all_registrations()	0	3	1
A	get_ticket_with_earliest_start_time()	0	5	1
A	get_ticket_with_latest_end_time()	0	5	1
B	tickets_on_sale()	0	12	5
A	get_permalink()	0	8	2
A	first_event_category()	0	5	1
A	get_all_event_categories()	0	5	1
A	question_groups()	0	4	2
A	get_icon()	0	3	1
A	get_admin_details_link()	0	3	1
A	get_admin_edit_link()	10	10	1
A	get_admin_settings_link()	9	9	1
A	get_admin_overview_link()	10	10	1

<?php if ( !defined( 'EVENT_ESPRESSO_VERSION' ) ) {
	exit( 'No direct script access allowed' );
}

/**
 * EE_Event
 *
 * @package 			Event Espresso
 * @subpackage 	includes/models/
 * @author 				Mike Nelson
 */
class EE_Event extends EE_CPT_Base implements EEI_Line_Item_Object, EEI_Admin_Links, EEI_Has_Icon {

	/**
	 * cached value for the the logical active status for the event
	 * @see get_active_status()
	 * @var string
	 */
	protected $_active_status = '';

	/**
	 * This is just used for caching the Primary Datetime for the Event on initial retrieval
	 * @var EE_Datetime
	 */
	protected $_Primary_Datetime;



	/**
	 *
	 * @param array $props_n_values  incoming values
	 * @param string $timezone  incoming timezone (if not set the timezone set for the website will be
	 *                          		used.)
	 * @param array $date_formats  incoming date_formats in an array where the first value is the
	 *                             		    date_format and the second value is the time format
	 * @return EE_Event
	 */
	public static function new_instance( $props_n_values = array(), $timezone = null, $date_formats = array() ) {
		$has_object = parent::_check_for_object( $props_n_values, __CLASS__ );

It seems like you call parent on a different method (_check_for_object() instead of new_instance()). Are you sure this is correct? If so, you might want to change this to $this->_check_for_object().

		return $has_object ? $has_object : new self( $props_n_values, false, $timezone, $date_formats );
	}



	/**
	 * @param array $props_n_values  incoming values from the database
	 * @param string $timezone  incoming timezone as set by the model.  If not set the timezone for
	 *                          		the website will be used.
	 * @return EE_Event
	 */
	public static function new_instance_from_db( $props_n_values = array(), $timezone = null ) {
		return new self( $props_n_values, TRUE, $timezone );
	}



	/**
	 * Overrides parent set() method so that all calls to set( 'status', $status ) can be routed to internal methods
	 *
	 * @param string $field_name
	 * @param mixed  $field_value
	 * @param bool   $use_default
	 */
	public function set( $field_name, $field_value, $use_default = false ) {
		switch ( $field_name ) {
			case 'status' :
				$this->set_status( $field_value, $use_default );
				break;
			default :
				parent::set( $field_name, $field_value, $use_default );
		}
	}



	/**
	 *    set_status
	 *
	 * Checks if event status is being changed to SOLD OUT
	 * and updates event meta data with previous event status
	 * so that we can revert things if/when the event is no longer sold out
	 *
	 * @access public
	 * @param string $new_status
	 * @param bool   $use_default
	 * @return bool|void
	 * @throws \EE_Error
	 */
	public function set_status( $new_status = null, $use_default = false ) {
		// get current Event status
		$old_status = $this->status();
		// if status has changed
		if ( $old_status != $new_status ) {

This code seems to be duplicated across your project.

			// TO sold_out
			if ( $new_status == EEM_Event::sold_out ) {
				// save the previous event status so that we can revert if the event is no longer sold out
				$this->add_post_meta( '_previous_event_status', $old_status );
				do_action( 'AHEE__EE_Event__set_status__to_sold_out', $this, $old_status, $new_status );
				// OR FROM  sold_out
			} else if ( $old_status == EEM_Event::sold_out ) {
				$this->delete_post_meta( '_previous_event_status' );
				do_action( 'AHEE__EE_Event__set_status__from_sold_out', $this, $old_status, $new_status );
			}
			// update status
			parent::set( 'status', $new_status, $use_default );

It seems like you call parent on a different method (set() instead of set_status()). Are you sure this is correct? If so, you might want to change this to $this->set().

			do_action( 'AHEE__EE_Event__set_status__after_update', $this );
			return true;
		} else {
			// even though the old value matches the new value, it's still good to
			// allow the parent set method to have a say
			parent::set( 'status', $new_status, $use_default );

It seems like you call parent on a different method (set() instead of set_status()). Are you sure this is correct? If so, you might want to change this to $this->set().

			return true;
		}
	}



	/**
	 * Gets all the datetimes for this event
	 *
	 * @param array $query_params like EEM_Base::get_all
	 * @return EE_Datetime[]
	 */
	public function datetimes( $query_params = array() ) {
		return $this->get_many_related( 'Datetime', $query_params );
	}



	/**
	 * Gets all the datetimes for this event, ordered by DTT_EVT_start in ascending order
	 *
	 * @return EE_Datetime[]
	 */
	public function datetimes_in_chronological_order() {
		return $this->get_many_related( 'Datetime', array( 'order_by' => array( 'DTT_EVT_start' => 'ASC' ) ) );
	}



	/**
	 * Gets all the datetimes for this event, ordered by the DTT_order on the datetime.
	 * @darren, we should probably UNSET timezone on the EEM_Datetime model
	 * after running our query, so that this timezone isn't set for EVERY query
	 * on EEM_Datetime for the rest of the request, no?
	 *
	 * @param boolean $show_expired whether or not to include expired events
	 * @param boolean $show_deleted whether or not to include deleted events
	 * @param null $limit
	 * @return \EE_Datetime[]
	 */
	public function datetimes_ordered( $show_expired = true, $show_deleted = false, $limit = null ) {
		return EEM_Datetime::instance( $this->_timezone )->get_datetimes_for_event_ordered_by_DTT_order( $this->ID(), $show_expired, $show_deleted, $limit );
	}



	/**
	 * Returns one related datetime. Mostly only used by some legacy code.
	 * @return EE_Datetime
	 */
	public function first_datetime() {
		return $this->get_first_related( 'Datetime' );
	}



	/**
	 * Returns the 'primary' datetime for the event
	 * @param bool $try_to_exclude_expired
	 * @param bool $try_to_exclude_deleted
	 * @return EE_Datetime
	 */
	public function primary_datetime( $try_to_exclude_expired = TRUE, $try_to_exclude_deleted = TRUE ) {
		if ( !empty ( $this->_Primary_Datetime ) ) {
			return $this->_Primary_Datetime;
		}
		$this->_Primary_Datetime = EEM_Datetime::instance( $this->_timezone )->get_primary_datetime_for_event( $this->ID(), $try_to_exclude_expired, $try_to_exclude_deleted );
		return $this->_Primary_Datetime;
	}



	/**
	 * Gets all the tickets available for purchase of this event
	 * @param array $query_params like EEM_Base::get_all
	 * @return EE_Ticket[]
	 */
	public function tickets( $query_params = array() ) {
		//first get all datetimes
		$datetimes = $this->datetimes_ordered();
		if ( ! $datetimes ) {

The expression $datetimes of type EE_Base_Class[] is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

			return array();
		}

		$datetime_ids = array();
		foreach ( $datetimes as $datetime ) {
			$datetime_ids[] = $datetime->ID();
		}

		$where_params = array( 'Datetime.DTT_ID' => array( 'IN', $datetime_ids ) );

		//if incoming $query_params has where conditions let's merge but not override existing.
		if ( is_array( $query_params ) && isset( $query_params[0]) ) {
			$where_params = array_merge( $query_params[0], $where_params );
			unset( $query_params[0] );
		}

		//now add $where_params to $query_params
		$query_params[0] = $where_params;

		return EEM_Ticket::instance()->get_all( $query_params );
	}



	/**
	 * @return bool
	 */
	function additional_limit() {
		return $this->get( 'EVT_additional_limit' );
	}



	/**
	 * @return bool
	 */
	function allow_overflow() {
		return $this->get( 'EVT_allow_overflow' );
	}



	/**
	 * @return bool
	 */
	function created() {
		return $this->get( 'EVT_created' );
	}



	/**
	 * @return bool
	 */
	function description() {
		return $this->get( 'EVT_desc' );
	}



	/**
	 * Runs do_shortcode and wpautop on the description
	 * @return string of html
	 */
	function description_filtered() {
		return $this->get_pretty( 'EVT_desc' );
	}



	/**
	 * @return bool
	 */
	function display_description() {
		return $this->get( 'EVT_display_desc' );
	}



	/**
	 * @return bool
	 */
	function display_ticket_selector() {
		return (bool)$this->get( 'EVT_display_ticket_selector' );
	}



	/**
	 * @return bool
	 */
	function external_url() {
		return $this->get( 'EVT_external_URL' );
	}



	/**
	 * @return bool
	 */
	function member_only() {
		return $this->get( 'EVT_member_only' );
	}



	/**
	 * @return bool
	 */
	function phone() {
		return $this->get( 'EVT_phone' );
	}



	/**
	 * @return bool
	 */
	function modified() {
		return $this->get( 'EVT_modified' );
	}



	/**
	 * @return bool
	 */
	function name() {
		return $this->get( 'EVT_name' );

The return type of return $this->get('EVT_name'); (boolean) is incompatible with the return type declared by the interface EEI_Line_Item_Object::name of type string.

	}



	/**
	 * @return bool
	 */
	function order() {
		return $this->get( 'EVT_order' );
	}



	/**
	 * @return bool|string
	 */
	function default_registration_status() {
		$event_default_registration_status = $this->get( 'EVT_default_registration_status' );
		return !empty( $event_default_registration_status ) ? $event_default_registration_status : EE_Registry::instance()->CFG->registration->default_STS_ID;
	}



	/**
	 * @param int  $num_words
	 * @param null $more
	 * @param bool $not_full_desc
	 * @return bool|string
	 */
	function short_description( $num_words = 55, $more = NULL, $not_full_desc = FALSE ) {
		$short_desc = $this->get( 'EVT_short_desc' );
		if ( !empty( $short_desc ) || $not_full_desc ) {
			return $short_desc;
		}
		else {
			$full_desc = $this->get( 'EVT_desc' );
			return wp_trim_words( $full_desc, $num_words, $more );
		}
	}



	/**
	 * @return bool
	 */
	function slug() {
		return $this->get( 'EVT_slug' );
	}



	/**
	 * @return bool
	 */
	function timezone_string() {
		return $this->get( 'EVT_timezone_string' );
	}



	/**
	 * @return bool
	 */
	function visible_on() {
		return $this->get( 'EVT_visible_on' );
	}



	/**
	 * @return bool
	 */
	function wp_user() {
		return $this->get( 'EVT_wp_user' );
	}



	/**
	 * @return bool
	 */
	function donations() {
		return $this->get( 'EVT_donations' );
	}



	/**
	 * @param $limit
	 */
	function set_additional_limit( $limit ) {
		$this->set( 'EVT_additional_limit', $limit );
	}



	/**
	 * @param $created
	 */
	function set_created( $created ) {
		$this->set( 'EVT_created', $created );
	}



	/**
	 * @param $desc
	 */
	function set_description( $desc ) {
		$this->set( 'EVT_desc', $desc );
	}



	/**
	 * @param $display_desc
	 */
	function set_display_description( $display_desc ) {
		$this->set( 'EVT_display_desc', $display_desc );
	}



	/**
	 * @param $display_ticket_selector
	 */
	function set_display_ticket_selector( $display_ticket_selector ) {
		$this->set( 'EVT_display_ticket_selector', $display_ticket_selector );
	}



	/**
	 * @param $external_url
	 */
	function set_external_url( $external_url ) {
		$this->set( 'EVT_external_URL', $external_url );
	}



	/**
	 * @param $member_only
	 */
	function set_member_only( $member_only ) {
		$this->set( 'EVT_member_only', $member_only );
	}



	/**
	 * @param $event_phone
	 */
	function set_event_phone( $event_phone ) {
		$this->set( 'EVT_phone', $event_phone );
	}



	/**
	 * @param $modified
	 */
	function set_modified( $modified ) {
		$this->set( 'EVT_modified', $modified );
	}



	/**
	 * @param $name
	 */
	function set_name( $name ) {
		$this->set( 'EVT_name', $name );
	}



	/**
	 * @param $order
	 */
	function set_order( $order ) {
		$this->set( 'EVT_order', $order );
	}



	/**
	 * @param $short_desc
	 */
	function set_short_description( $short_desc ) {
		$this->set( 'EVT_short_desc', $short_desc );
	}



	/**
	 * @param $slug
	 */
	function set_slug( $slug ) {
		$this->set( 'EVT_slug', $slug );
	}



	/**
	 * @param $timezone_string
	 */
	function set_timezone_string( $timezone_string ) {
		$this->set( 'EVT_timezone_string', $timezone_string );
	}



	/**
	 * @param $visible_on
	 */
	function set_visible_on( $visible_on ) {
		$this->set( 'EVT_visible_on', $visible_on );
	}



	/**
	 * @param $wp_user
	 */
	function set_wp_user( $wp_user ) {
		$this->set( 'EVT_wp_user', $wp_user );
	}



	/**
	 * @param $default_registration_status
	 */
	function set_default_registration_status( $default_registration_status ) {
		$this->set( 'EVT_default_registration_status', $default_registration_status );
	}



	/**
	 * @param $donations
	 */
	function set_donations( $donations ) {
		$this->set( 'EVT_donations', $donations );
	}



	/**
	 * Adds a venue to this event
	 * @param EE_Venue /int $venue_id_or_obj
	 * @return EE_Venue
	 */
	function add_venue( $venue_id_or_obj ) {
		return $this->_add_relation_to( $venue_id_or_obj, 'Venue' );
	}



	/**
	 * Removes a venue from the event
	 * @param EE_Venue /int $venue_id_or_obj
	 * @return EE_Venue
	 */
	function remove_venue( $venue_id_or_obj ) {
		return $this->_remove_relation_to( $venue_id_or_obj, 'Venue' );
	}



	/**
	 * Gets all the venues related ot the event. May provide additional $query_params if desired
	 * @param array $query_params like EEM_Base::get_all's $query_params
	 * @return EE_Venue[]
	 */
	function venues( $query_params = array() ) {
		return $this->get_many_related( 'Venue', $query_params );
	}



	/**
	 * check if event id is present and if event is published
	 * @access public
	 * @return boolean true yes, false no
	 */
	private function _has_ID_and_is_published() {
		// first check if event id is present and not NULL, then check if this event is published (or any of the equivalent "published" statuses)
		return ( $this->ID() && $this->ID() !== NULL && ( $this->status() == 'publish' || $this->status() == EEM_Event::sold_out || $this->status() == EEM_Event::postponed || $this->status() == EEM_Event::cancelled ) ) ? TRUE : FALSE;
	}



	/**
	 * This simply compares the internal dates with NOW and determines if the event is upcoming or not.
	 * @access public
	 * @return boolean true yes, false no
	 */
	public function is_upcoming() {

This method seems to be duplicated in your project.

		// check if event id is present and if this event is published
		if ( $this->is_inactive() ) {
			return FALSE;
		}
		// set initial value
		$upcoming = FALSE;
		//next let's get all datetimes and loop through them
		$datetimes = $this->datetimes_in_chronological_order();
		foreach ( $datetimes as $datetime ) {
			if ( $datetime instanceof EE_Datetime ) {
				//if this dtt is expired then we continue cause one of the other datetimes might be upcoming.
				if ( $datetime->is_expired() ) {
					continue;
				}
				//if this dtt is active then we return false.
				if ( $datetime->is_active() ) {
					return FALSE;
				}
				//otherwise let's check upcoming status
				$upcoming = $datetime->is_upcoming();
			}
		}
		return $upcoming;
	}



	/**
	 * @return bool
	 */
	public function is_active() {

This method seems to be duplicated in your project.

		// check if event id is present and if this event is published
		if ( $this->is_inactive() ) {
			return FALSE;
		}
		// set initial value
		$active = FALSE;
		//next let's get all datetimes and loop through them
		$datetimes = $this->datetimes_in_chronological_order();
		foreach ( $datetimes as $datetime ) {
			if ( $datetime instanceof EE_Datetime ) {
				//if this dtt is expired then we continue cause one of the other datetimes might be active.
				if ( $datetime->is_expired() ) {
					continue;
				}
				//if this dtt is upcoming then we return false.
				if ( $datetime->is_upcoming() ) {
					return FALSE;
				}
				//otherwise let's check active status
				$active = $datetime->is_active();
			}
		}
		return $active;
	}



	/**
	 * @return bool
	 */
	public function is_expired() {

This method seems to be duplicated in your project.

		// check if event id is present and if this event is published
		if ( $this->is_inactive() ) {
			return FALSE;
		}
		// set initial value
		$expired = FALSE;
		//first let's get all datetimes and loop through them
		$datetimes = $this->datetimes_in_chronological_order();
		foreach ( $datetimes as $datetime ) {
			if ( $datetime instanceof EE_Datetime ) {
				//if this dtt is upcoming or active then we return false.
				if ( $datetime->is_upcoming() || $datetime->is_active() ) {
					return FALSE;
				}
				//otherwise let's check active status
				$expired = $datetime->is_expired();
			}
		}
		return $expired;
	}



	/**
	 * @return bool
	 */
	public function is_inactive() {
		// check if event id is present and if this event is published
		if ( $this->_has_ID_and_is_published() ) {
			return FALSE;
		}
		return TRUE;
	}



	/**
	 *    perform_sold_out_status_check
	 *    checks all of this events's datetime  reg_limit - sold values to determine if ANY datetimes have spaces available...
	 *    if NOT, then the event status will get toggled to 'sold_out'
	 *
	 * @access public
	 * @return bool    return the ACTUAL sold out state.
	 */
	public function perform_sold_out_status_check() {
		// get all unexpired untrashed tickets
		$tickets = $this->tickets( array(
			array(
				'TKT_end_date'   => array( '>=', EEM_Ticket::instance()->current_time_for_query( 'TKT_end_date' ) ),
				'TKT_deleted'    => false
			)
		));
		// if all the tickets are just expired, then don't update the event status to sold out
		if ( empty( $tickets )) {
			return true;
		}
		// set initial value
		$spaces_remaining = 0;
		foreach( $tickets as $ticket ) {
			if ( $ticket instanceof EE_Ticket ) {
				$spaces_remaining += $ticket->qty( 'saleable' );
			}
		}
		if ( $spaces_remaining === 0 ) {
			$this->set_status( EEM_Event::sold_out );
			if ( !is_admin() || ( is_admin() && defined( 'DOING_AJAX' ) ) ) {
				$this->save();
			}
			$sold_out = TRUE;
		} else {
			$sold_out = FALSE;
			// was event previously marked as sold out ?
			if ( $this->status() == EEM_Event::sold_out ) {
				// revert status to previous value, if it was set
				$previous_event_status = $this->get_post_meta( '_previous_event_status', true );
				if ( $previous_event_status ) {
					$this->set_status( $previous_event_status );
				}
			}
		}
		//note: I considered changing the EEM_Event status away from sold_out if this status check reveals that it's no longer sold out (yet the status is still set as sold out) but the problem is... what do we change the status BACK to?  We can't always assume that the previous event status was 'published' because this status check is always done in the admin and its entirely possible the event admin manually changes to sold_out status from some other status.  We also don't want a draft event to become a "publish event" because the sold out check reveals its NOT sold out.
		// So I'll forgo the automatic switch away from sold out status for now and instead just return the $sold out status... so this check can be used to validate the TRUE sold out status regardless of what the Event status is set to.
		return $sold_out;
	}


	/**
	 * This returns the total remaining spaces for sale on this event.
	 *
	 * ############################
	 * VERY IMPORTANT FOR DEVELOPERS:
	 * While included here, this method is still being tested internally, so its signature and behaviour COULD change. While
	 * this comment block is in place, usage is at your own risk and know that it may change in future builds.
	 * ############################
	 *
	 * @uses EE_Event::total_available_spaces()
	 * @return float|int  (EE_INF is returned as float)
	 */
	public function spaces_remaining_for_sale() {
		//first get total available spaces including consideration for tickets that have already sold.
		$spaces_available = $this->total_available_spaces( true );

		//if total available = 0, then exit right away because that means everything is expired.
		if ( $spaces_available === 0 ) {
			return 0;
		}

		//subtract total approved registrations from spaces available to get how many are remaining.
		$spots_taken = EEM_Registration::instance()->count( array( array( 'EVT_ID' => $this->ID(), 'STS_ID' => EEM_Registration::status_id_approved ) ), 'REG_ID', true );
		$spaces_remaining = $spaces_available - $spots_taken;

		return $spaces_remaining > 0 ? $spaces_remaining : 0;
	}





	/**
	 * This returns the total spaces available for an event while considering all the qtys on the tickets and the reg limits
	 * on the datetimes attached to this event.
	 *
	 * ############################
	 * VERY IMPORTANT FOR DEVELOPERS:
	 * While included here, this method is still being tested internally, so its signature and behaviour COULD change. While
	 * this comment block is in place, usage is at your own risk and know that it may change in future builds.
	 * ############################
	 *
	 * Note: by "spaces available" we are not returning how many spaces remain.  That is a calculation involving using the value
	 * from this method and subtracting the approved registrations for the event.
	 *
	 * @param   bool    $current_total_available    Whether to consider any tickets that have already sold in our calculation.
	 *                                              If this is false, then we return the most tickets that could ever be sold
	 *                                              for this event with the datetime and tickets setup on the event under optimal
	 *                                              selling conditions.  Otherwise we return a live calculation of spaces available
	 *                                              based on tickets sold.  Depending on setup and stage of sales, this
	 *                                              may appear to equal remaining tickets.  However, the more tickets are
	 *                                              sold out, the more accurate the "live" total is.
	 *
	 * @return  int|float  (Note: if EE_INF is returned its considered a float by PHP)
	 */
	public function total_available_spaces( $current_total_available = false ) {
		$spaces_available = 0;

		//first get all tickets on the event and include expired tickets
		$tickets = $this->tickets( array( 'default_where_conditions' => 'none' ) );
		$ticket_sums = array();
		$datetime_limits = array();

		//loop through tickets and normalize them
		foreach ( $tickets as $ticket ) {
			$datetimes = $ticket->datetimes( array( 'order_by' => array( 'DTT_reg_limit' => 'ASC' ) ) );

			if ( empty( $datetimes ) ) {
				continue;
			}

			//first datetime should be the lowest datetime
			$least_datetime = reset( $datetimes );

			//lets reset the ticket quantity to be the lower of either the lowest datetime reg limit or the ticket quantity
			//IF datetimes sold (and we're not doing current live total available, then use spaces remaining for datetime, not reg_limit.
			if ( $current_total_available ) {
				if ( $ticket->is_remaining() ) {
					$remaining = $ticket->remaining();
				} else {
					$spaces_available += $ticket->sold();
					//and we don't cache this ticket to our list because its sold out.
					continue;
				}
			} else {
				$remaining = min( $ticket->qty(), $least_datetime->reg_limit() );

The method reg_limit does not exist on object<EE_Base_Class>? Since you implemented __call, maybe consider adding a @method annotation.

			}

			//if $ticket_limit == infinity then let's drop out right away and just return that because any infinity amount trumps all other "available" amounts.
			if ( $remaining == EE_INF ) {
				return EE_INF;
			}

			//multiply normalized $tkt quantity by the number of datetimes on the ticket as the "sum"
			//also include the sum of all the datetime reg limits on the ticket for breaking ties.
			$ticket_sums[$ticket->ID()]['sum'] = $remaining * count( $datetimes );
			$ticket_sums[$ticket->ID()]['datetime_sums'] = 0;
			foreach ( $datetimes as $datetime ) {
				if ( $datetime->reg_limit() === EE_INF ) {
					$ticket_sums[$ticket->ID()]['datetime_sums'] = EE_INF;
				} else {
					$ticket_sums[ $ticket->ID() ]['datetime_sums'] += $current_total_available ? $datetime->spaces_remaining() : $datetime->reg_limit();
				}
				$datetime_limits[$datetime->ID()] = $current_total_available ? $datetime->spaces_remaining() : $datetime->reg_limit();
			}
			$ticket_sums[$ticket->ID()]['ticket'] = $ticket;
		}

		//The order is sorted by lowest available first (which is calculated for each ticket by multiplying the normalized
		//ticket quantity by the number of datetimes on the ticket).  For tie-breakers, then the next sort is based on the
		//ticket with the greatest sum of all remaining datetime->spaces_remaining() ( or $datetime->reg_limit() if not
		//$current_total_available ) for the datetimes on the ticket.
		usort( $ticket_sums, function( $a, $b ) {
			if ( $a['sum'] == $b['sum'] ) {
				if ( $a['datetime_sums'] == $b['datetime_sums'] ) {
					return 0;
				}

				return $a['datetime_sums'] < $b['datetime_sums'] ? 1 : -1;
			}
			return ( $a['sum'] < $b['sum'] ) ? -1 : 1;
		});

		//now let's loop through the sorted tickets and simulate sellouts
		foreach ( $ticket_sums as $ticket_info ) {
			if ( $ticket_info['ticket'] instanceof EE_Ticket ) {

				$datetimes = $ticket_info['ticket']->datetimes( array( 'order_by' => array( 'DTT_reg_limit' => 'ASC' ) ) );
				//need to sort these $datetimes by remaining (only if $current_total_available)
				//setup datetimes for simulation
				$ticket_datetimes_remaining = array();
				foreach( $datetimes as $datetime ) {
					$ticket_datetimes_remaining[$datetime->ID()]['rem'] = $datetime_limits[$datetime->ID()];
					$ticket_datetimes_remaining[$datetime->ID()]['datetime'] = $datetime;
				}
				usort( $ticket_datetimes_remaining, function( $a, $b ) {
					if ( $a['rem'] == $b['rem'] ) {
						return 0;
					}
					return ( $a['rem'] < $b['rem'] ) ? -1 : 1;
				});


				//get the remaining on the first datetime (which should be the one with the least remaining) and that is
				//what we add to the spaces_available running total.  Then we need to decrease the remaining on our datetime tracker.
				$lowest_datetime = reset( $ticket_datetimes_remaining );

				//need to get the lower of; what the remaining is on the lowest datetime, and the remaining on the ticket.
				// If this ends up being 0 (because of previous tickets in our simulation selling out), then it has already
				// been tracked on $spaces available and this ticket is now sold out for the simulation, so we can continue
				// to the next ticket.
				if ( $current_total_available ) {
					$remaining = min( $lowest_datetime['rem'], $ticket_info['ticket']->remaining() );
				} else {
					$remaining = min( $lowest_datetime['rem'], $ticket_info['ticket']->qty() );
				}

				//if $remaining is infinite that means that all datetimes on this ticket are infinite but we've made it here because all
				//tickets have a quantity.  So we don't have to track datetimes, we can just use ticket quantities for total
				//available.
				if ( $remaining === EE_INF ) {
					$spaces_available += $ticket_info['ticket']->qty();
					continue;
				}

				//if ticket has sold amounts then we also need to add that (but only if doing live counts)
				if ( $current_total_available ) {
					$spaces_available += $ticket_info['ticket']->sold();
				}

				if ( $remaining <= 0 ) {
					continue;
				} else {
					$spaces_available += $remaining;
				}

				//loop through the datetimes and sell them out!
				foreach ( $ticket_datetimes_remaining as $datetime_info ) {
					if ( $datetime_info['datetime'] instanceof EE_Datetime ) {
						$datetime_limits[ $datetime_info['datetime']->ID() ] += - $remaining;
					}
				}
			}
		}

		return $spaces_available;
	}



	/**
	 * Checks if the event is set to sold out
	 * @param  bool $actual whether or not to perform calculations to not only figure the actual status but also to flip the status if necessary to sold out If false, we just check the existing status of the event
	 * @return boolean
	 */
	public function is_sold_out( $actual = FALSE ) {
		if ( ! $actual ) {
			return $this->status() == EEM_Event::sold_out;
		}
		else {
			return $this->perform_sold_out_status_check();
		}
	}



	/**
	 * Checks if the event is marked as postponed
	 * @return boolean
	 */
	public function is_postponed() {
		return $this->status() == EEM_Event::postponed;
	}



	/**
	 * Checks if the event is marked as cancelled
	 * @return boolean
	 */
	public function is_cancelled() {
		return $this->status() == EEM_Event::cancelled;
	}



	/**
	 * Get the logical active status in a hierarchical order for all the datetimes.  Note
	 *
	 * Basically, we order the datetimes by EVT_start_date.  Then first test on whether the event is published.  If its
	 * NOT published then we test for whether its expired or not.  IF it IS published then we test first on whether an
	 * event has any active dates.  If no active dates then we check for any upcoming dates.  If no upcoming dates then
	 * the event is considered expired.
	 *
	 * NOTE: this method does NOT calculate whether the datetimes are sold out when event is published.  Sold Out is a status
	 * set on the EVENT when it is not published and thus is done
	 *
	 * @param bool $reset
	 *
	 * @return bool | string - based on EE_Datetime active constants or FALSE if error.
	 */
	public function get_active_status( $reset = false ) {
		// if the active status has already been set, then just use that value (unless we are resetting it)
		if ( ! empty( $this->_active_status ) && ! $reset ) {
			return $this->_active_status;

The return type of return $this->_active_status; (string) is incompatible with the return type documented by EE_Event::get_active_status of type boolean.

		}
		//first check if event id is present on this object
		if ( ! $this->ID() ) {
			return false;
		}

		$where_params_for_event  = array( array( 'EVT_ID' => $this->ID() ) );

		//if event is published:
		if ( $this->status() === 'publish' ) {
			//active?
			if ( EEM_Datetime::instance()->get_datetime_count_for_status( EE_Datetime::active, $where_params_for_event ) > 0 ) {
				$this->_active_status = EE_Datetime::active;
			} else {
				//upcoming?
				if ( EEM_Datetime::instance()->get_datetime_count_for_status( EE_Datetime::upcoming, $where_params_for_event  ) > 0 ) {
					$this->_active_status = EE_Datetime::upcoming;
				} else {
					//expired?
					if ( EEM_Datetime::instance()->get_datetime_count_for_status( EE_Datetime::expired, $where_params_for_event  ) > 0 ) {
						$this->_active_status = EE_Datetime::expired;
					} else {
						//it would be odd if things make it this far because it basically means there are no datetime's
						//attached to the event.  So in this case it will just be considered inactive.
						$this->_active_status = EE_Datetime::inactive;
					}
				}
			}
		} else {
			//the event is not published, so let's just set it's active status according to its' post status
			switch ( $this->status() ) {
				case EEM_Event::sold_out :
					$this->_active_status = EE_Datetime::sold_out;
					break;
				case EEM_Event::cancelled :
					$this->_active_status = EE_Datetime::cancelled;
					break;
				case EEM_Event::postponed :
					$this->_active_status = EE_Datetime::postponed;
					break;
				default :
					$this->_active_status = EE_Datetime::inactive;
			}
		}
		return $this->_active_status;
	}



	/**
	 *    pretty_active_status
	 *
	 * @access public
	 * @param boolean $echo     whether to return (FALSE), or echo out the result (TRUE)
	 * @return mixed void|string
	 */
	public function pretty_active_status( $echo = TRUE ) {
		$active_status = $this->get_active_status();
		$status = '<span class="ee-status event-active-status-' . $active_status . '">' . EEH_Template::pretty_status( $active_status, FALSE, 'sentence' ) . '</span>';

$active_status is of type boolean, but the function expects a string.

		if ( $echo ) {
			echo $status;
			return '';
		}
		return $status;
	}



	/**
	 * @return bool|int
	 */
	public function get_number_of_tickets_sold() {
		$tkt_sold = 0;
		if ( !$this->ID() ) {
			return 0;
		}
		$datetimes = $this->datetimes();
		foreach ( $datetimes as $datetime ) {
			if ( $datetime instanceof EE_Datetime ) {
				$tkt_sold += $datetime->sold();
			}
		}
		return $tkt_sold;
	}



	/**
	 * This just returns a count of all the registrations for this event
	 * @access  public
	 * @return int
	 */
	public function get_count_of_all_registrations() {
		return EEM_Event::instance()->count_related( $this, 'Registration' );
	}



	/**
	 * This returns the ticket with the earliest start time that is available for this event (across all datetimes attached to the event)
	 * @return EE_Ticket
	 */
	public function get_ticket_with_earliest_start_time() {
		$where[ 'Datetime.EVT_ID' ] = $this->ID();

$where was never initialized. Although not strictly required by PHP, it is generally a good practice to add $where = array(); before regardless.

		$query_params = array( $where, 'order_by' => array( 'TKT_start_date' => 'ASC' ) );
		return EE_Registry::instance()->load_model( 'Ticket' )->get_one( $query_params );

The method get_one cannot be called on \EE_Registry::instance()->load_model('Ticket') (of type boolean).

	}



	/**
	 * This returns the ticket with the latest end time that is available for this event (across all datetimes attached to the event)
	 * @return EE_Ticket
	 */
	public function get_ticket_with_latest_end_time() {
		$where[ 'Datetime.EVT_ID' ] = $this->ID();

$where was never initialized. Although not strictly required by PHP, it is generally a good practice to add $where = array(); before regardless.

		$query_params = array( $where, 'order_by' => array( 'TKT_end_date' => 'DESC' ) );
		return EE_Registry::instance()->load_model( 'Ticket' )->get_one( $query_params );

The method get_one cannot be called on \EE_Registry::instance()->load_model('Ticket') (of type boolean).

	}



	/**
	 * This returns whether there are any tickets on sale for this event.
	 * @return bool true = YES tickets on sale.
	 */
	public function tickets_on_sale() {
		$earliest_ticket = $this->get_ticket_with_earliest_start_time();
		$latest_ticket = $this->get_ticket_with_latest_end_time();
		if ( !$latest_ticket instanceof EE_Ticket && !$earliest_ticket instanceof EE_Ticket ) {
			return FALSE;
		}
		//check on sale for these two tickets.
		if ( $latest_ticket->is_on_sale() || $earliest_ticket->is_on_sale() ) {
			return TRUE;
		}
		return FALSE;
	}



	/**
	 * Gets the URL for viewing this event on the front-end. Overrides parent
	 * to check for an external URL first
	 * @return string
	 */
	public function get_permalink() {
		if ( $this->external_url() ) {
			return $this->external_url();

The return type of return $this->external_url(); (boolean) is incompatible with the return type of the parent method EE_CPT_Base::get_permalink of type string.

		}
		else {
			return parent::get_permalink();
		}
	}



	/**
	 * Gets the first term for 'espresso_event_categories' we can find
	 * @param array $query_params like EEM_Base::get_all
	 * @return EE_Term
	 */
	public function first_event_category( $query_params = array() ) {
		$query_params[ 0 ][ 'Term_Taxonomy.taxonomy' ] = 'espresso_event_categories';
		$query_params[ 0 ][ 'Term_Taxonomy.Event.EVT_ID' ] = $this->ID();
		return EEM_Term::instance()->get_one( $query_params );
	}



	/**
	 * Gets all terms for 'espresso_event_categories' we can find
	 * @param array $query_params
	 * @return EE_Term[]
	 */
	public function get_all_event_categories( $query_params = array() ) {
		$query_params[ 0 ][ 'Term_Taxonomy.taxonomy' ] = 'espresso_event_categories';
		$query_params[ 0 ][ 'Term_Taxonomy.Event.EVT_ID' ] = $this->ID();
		return EEM_Term::instance()->get_all( $query_params );
	}

	/**
	 * Gets all the question groups, ordering them by QSG_order ascending
	 * @param array $query_params @see EEM_Base::get_all
	 * @return EE_Question_Group[]
	 */
	public function question_groups($query_params = array()){
		$query_params = ! empty( $query_params ) ? $query_params : array( 'order_by' => array( 'QSG_order' => 'ASC' ));
		return $this->get_many_related('Question_Group', $query_params);
	}




	/**
	 * Implementation for EEI_Has_Icon interface method.
	 * @see EEI_Visual_Representation for comments
	 * @return string
	 */
	public function get_icon() {
		return '<span class="dashicons dashicons-flag"></span>';
	}




	/**
	 * Implementation for EEI_Admin_Links interface method.
	 * @see EEI_Admin_Links for comments
	 * @return string
	 */
	public function get_admin_details_link() {
		return $this->get_admin_edit_link();
	}






	/**
	 * Implementation for EEI_Admin_Links interface method.
	 * @see EEI_Admin_Links for comments
	 * @return string
	 */
	public function get_admin_edit_link() {

This method seems to be duplicated in your project.

		EE_Registry::instance()->load_helper('URL');
		return EEH_URL::add_query_args_and_nonce( array(
			'page' => 'espresso_events',
			'action' => 'edit',
			'post' => $this->ID()
			),
			admin_url( 'admin.php' )
		);
	}



	/**
	 * Implementation for EEI_Admin_Links interface method.
	 * @see EEI_Admin_Links for comments
	 * @return string
	 */
	public function get_admin_settings_link() {

This method seems to be duplicated in your project.

		EE_Registry::instance()->load_helper('URL');
		return EEH_URL::add_query_args_and_nonce( array(
			'page' => 'espresso_events',
			'action' => 'default_event_settings'
			),
			admin_url( 'admin.php' )
		);
	}





	/**
	 * Implementation for EEI_Admin_Links interface method.
	 * @see EEI_Admin_Links for comments
	 * @return string
	 */
	public function get_admin_overview_link() {

This method seems to be duplicated in your project.

		EE_Registry::instance()->load_helper('URL');
		return EEH_URL::add_query_args_and_nonce( array(
			'page' => 'espresso_events',
			'action' => 'default'
		),
			admin_url( 'admin.php' )
		);
	}

}