eventespresso / event-espresso-core

core/db_classes/EE_Registration.class.php

Indentation +66 added lines, -66 removed lines

@@ -119,11 +119,11 @@ 
 		) {
 			// TO approved
 			if ( $new_STS_ID === EEM_Registration::status_id_approved ) {
-                \EEH_Debug_Tools::printr(__FUNCTION__, 'BEFORE HOOK ' . __CLASS__, __FILE__, __LINE__, 2);
-                // reserve a space by incrementing ticket and datetime sold values
+				\EEH_Debug_Tools::printr(__FUNCTION__, 'BEFORE HOOK ' . __CLASS__, __FILE__, __LINE__, 2);
+				// reserve a space by incrementing ticket and datetime sold values
 				$this->_reserve_registration_space();
 				do_action( 'AHEE__EE_Registration__set_status__to_approved', $this, $old_STS_ID, $new_STS_ID );
-                \EEH_Debug_Tools::printr(__FUNCTION__, 'AFTER HOOK ' . __CLASS__, __FILE__, __LINE__, 2);
+				\EEH_Debug_Tools::printr(__FUNCTION__, 'AFTER HOOK ' . __CLASS__, __FILE__, __LINE__, 2);
 				exit();
 			// OR FROM  approved
 			} else if ( $old_STS_ID === EEM_Registration::status_id_approved ) {
@@ -648,9 +648,9 @@ 
 
 
 	/**
-	*		get  Attendee Number
-	* 		@access		public
-	*/
+	 *		get  Attendee Number
+	 * 		@access		public
+	 */
 	public function count() {
 		return $this->get( 'REG_count' );
 	}
@@ -1032,20 +1032,20 @@ 
 
 
 
-    /**
-     * toggle Check-in status for this registration
-     * Check-ins are toggled in the following order:
-     * never checked in -> checked in
-     * checked in -> checked out
-     * checked out -> checked in
-     *
-     * @param  int $DTT_ID  include specific datetime to toggle Check-in for.
-     *                      If not included or null, then it is assumed latest datetime is being toggled.
-     * @param bool $verify  If true then can_checkin() is used to verify whether the person
-     *                      can be checked in or not.  Otherwise this forces change in checkin status.
-     * @return bool|int     the chk_in status toggled to OR false if nothing got changed.
-     * @throws EE_Error
-     */
+	/**
+	 * toggle Check-in status for this registration
+	 * Check-ins are toggled in the following order:
+	 * never checked in -> checked in
+	 * checked in -> checked out
+	 * checked out -> checked in
+	 *
+	 * @param  int $DTT_ID  include specific datetime to toggle Check-in for.
+	 *                      If not included or null, then it is assumed latest datetime is being toggled.
+	 * @param bool $verify  If true then can_checkin() is used to verify whether the person
+	 *                      can be checked in or not.  Otherwise this forces change in checkin status.
+	 * @return bool|int     the chk_in status toggled to OR false if nothing got changed.
+	 * @throws EE_Error
+	 */
 	public function toggle_checkin_status( $DTT_ID = null, $verify = false ) {
 		if ( empty( $DTT_ID ) ) {
 			$datetime = $this->get_latest_related_datetime();
@@ -1102,13 +1102,13 @@ 
 
 
 
-    /**
-     * Returns the latest datetime related to this registration (via the ticket attached to the registration).
-     * "Latest" is defined by the `DTT_EVT_start` column.
-     *
-     * @return EE_Datetime|null
-     * @throws \EE_Error
-     */
+	/**
+	 * Returns the latest datetime related to this registration (via the ticket attached to the registration).
+	 * "Latest" is defined by the `DTT_EVT_start` column.
+	 *
+	 * @return EE_Datetime|null
+	 * @throws \EE_Error
+	 */
 	public function get_latest_related_datetime() {
 		return EEM_Datetime::instance()->get_one(
 			array(
@@ -1122,12 +1122,12 @@ 
 
 
 
-    /**
-     * Returns the earliest datetime related to this registration (via the ticket attached to the registration).
-     * "Earliest" is defined by the `DTT_EVT_start` column.
-     *
-     * @throws \EE_Error
-     */
+	/**
+	 * Returns the earliest datetime related to this registration (via the ticket attached to the registration).
+	 * "Earliest" is defined by the `DTT_EVT_start` column.
+	 *
+	 * @throws \EE_Error
+	 */
 	public function get_earliest_related_datetime() {
 		return EEM_Datetime::instance()->get_one(
 			array(
@@ -1141,18 +1141,18 @@ 
 
 
 
-    /**
-     * This method simply returns the check-in status for this registration and the given datetime.
-     * If neither the datetime nor the checkin values are provided as arguments,
-     * then this will return the LATEST check-in status for the registration across all datetimes it belongs to.
-     *
-     * @param  int $DTT_ID        The ID of the datetime we're checking against
-     *                            (if empty we'll get the primary datetime for
-     *                            this registration (via event) and use it's ID);
-     * @param EE_Checkin $checkin If present, we use the given checkin object rather than the dtt_id.
-     * @return int                Integer representing Check-in status.
-     * @throws \EE_Error
-     */
+	/**
+	 * This method simply returns the check-in status for this registration and the given datetime.
+	 * If neither the datetime nor the checkin values are provided as arguments,
+	 * then this will return the LATEST check-in status for the registration across all datetimes it belongs to.
+	 *
+	 * @param  int $DTT_ID        The ID of the datetime we're checking against
+	 *                            (if empty we'll get the primary datetime for
+	 *                            this registration (via event) and use it's ID);
+	 * @param EE_Checkin $checkin If present, we use the given checkin object rather than the dtt_id.
+	 * @return int                Integer representing Check-in status.
+	 * @throws \EE_Error
+	 */
 	public function check_in_status_for_datetime( $DTT_ID = 0, $checkin = null ) {
 		$checkin_query_params = array(
 			'order_by' => array( 'CHK_timestamp' => 'DESC' )
@@ -1164,8 +1164,8 @@ 
 
 		//get checkin object (if exists)
 		$checkin = $checkin instanceof EE_Checkin
-            ? $checkin
-            : $this->get_first_related( 'Checkin', $checkin_query_params );
+			? $checkin
+			: $this->get_first_related( 'Checkin', $checkin_query_params );
 		if ( $checkin instanceof EE_Checkin ) {
 			if ( $checkin->get( 'CHK_in' ) ) {
 				return EE_Registration::checkin_status_in; //checked in
@@ -1418,7 +1418,7 @@ 
 
 
 
-    /*************************** DEPRECATED ***************************/
+	/*************************** DEPRECATED ***************************/
 
 
 
@@ -1468,24 +1468,24 @@ 
 
 
 
-    /**
-     * Gets the primary datetime related to this registration via the related Event to this registration
-     *
-     * @deprecated 4.9.17
-     * @return EE_Datetime
-     */
-    public function get_related_primary_datetime() {
-        EE_Error::doing_it_wrong(
-            __METHOD__,
-            esc_html__(
-                'Use EE_Registration::get_latest_related_datetime() or EE_Registration::get_earliest_related_datetime()',
-                'event_espresso'
-            ),
-            '4.9.17',
-            '5.0.0'
-        );
-        return $this->event()->primary_datetime();
-    }
+	/**
+	 * Gets the primary datetime related to this registration via the related Event to this registration
+	 *
+	 * @deprecated 4.9.17
+	 * @return EE_Datetime
+	 */
+	public function get_related_primary_datetime() {
+		EE_Error::doing_it_wrong(
+			__METHOD__,
+			esc_html__(
+				'Use EE_Registration::get_latest_related_datetime() or EE_Registration::get_earliest_related_datetime()',
+				'event_espresso'
+			),
+			'4.9.17',
+			'5.0.0'
+		);
+		return $this->event()->primary_datetime();
+	}
 
 
 }

Spacing +197 added lines, -197 removed lines

@@ -1,7 +1,7 @@ 
 <?php use EventEspresso\core\exceptions\EntityNotFoundException;
 
-if ( !defined( 'EVENT_ESPRESSO_VERSION' ) ) {
-	exit( 'No direct script access allowed' );
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
 }
 /**
  * EE_Registration class
@@ -43,9 +43,9 @@ 
 	 *                             		    date_format and the second value is the time format
 	 * @return EE_Registration
 	 */
-	public static function new_instance( $props_n_values = array(), $timezone = null, $date_formats = array() ) {
-		$has_object = parent::_check_for_object( $props_n_values, __CLASS__, $timezone, $date_formats );
-		return $has_object ? $has_object : new self( $props_n_values, false, $timezone, $date_formats );
+	public static function new_instance($props_n_values = array(), $timezone = null, $date_formats = array()) {
+		$has_object = parent::_check_for_object($props_n_values, __CLASS__, $timezone, $date_formats);
+		return $has_object ? $has_object : new self($props_n_values, false, $timezone, $date_formats);
 	}
 
 
@@ -56,8 +56,8 @@ 
 	 *                          		the website will be used.
 	 * @return EE_Registration
 	 */
-	public static function new_instance_from_db( $props_n_values = array(), $timezone = null ) {
-		return new self( $props_n_values, TRUE, $timezone );
+	public static function new_instance_from_db($props_n_values = array(), $timezone = null) {
+		return new self($props_n_values, TRUE, $timezone);
 	}
 
 
@@ -67,8 +67,8 @@ 
 	 *
 	 * @param        int $EVT_ID Event ID
 	 */
-	public function set_event( $EVT_ID = 0 ) {
-		$this->set( 'EVT_ID', $EVT_ID );
+	public function set_event($EVT_ID = 0) {
+		$this->set('EVT_ID', $EVT_ID);
 	}
 
 
@@ -79,18 +79,18 @@ 
 	 * @param mixed  $field_value
 	 * @param bool   $use_default
 	 */
-	public function set( $field_name, $field_value, $use_default = FALSE ) {
-		switch( $field_name ) {
+	public function set($field_name, $field_value, $use_default = FALSE) {
+		switch ($field_name) {
 			case 'REG_code' :
-				if ( ! empty( $field_value ) && $this->reg_code() == '' ) {
-					$this->set_reg_code( $field_value, $use_default );
+				if ( ! empty($field_value) && $this->reg_code() == '') {
+					$this->set_reg_code($field_value, $use_default);
 				}
 				break;
 			case 'STS_ID' :
-				$this->set_status( $field_value, $use_default );
+				$this->set_status($field_value, $use_default);
 				break;
 			default :
-				parent::set( $field_name, $field_value, $use_default );
+				parent::set($field_name, $field_value, $use_default);
 		}
 	}
 
@@ -107,45 +107,45 @@ 
 	 * @return bool
 	 * @throws \EE_Error
 	 */
-	public function set_status( $new_STS_ID = NULL, $use_default = FALSE ) {
+	public function set_status($new_STS_ID = NULL, $use_default = FALSE) {
 		// get current REG_Status
 		$old_STS_ID = $this->status_ID();
 		// if status has changed
 		if (
 			$this->ID() // ensure registration is in the db
 			&& $old_STS_ID != $new_STS_ID // and that status has actually changed
-			&& ! empty( $old_STS_ID ) // and that old status is actually set
-			&& ! empty( $new_STS_ID ) // as well as the new status
+			&& ! empty($old_STS_ID) // and that old status is actually set
+			&& ! empty($new_STS_ID) // as well as the new status
 		) {
 			// TO approved
-			if ( $new_STS_ID === EEM_Registration::status_id_approved ) {
-                \EEH_Debug_Tools::printr(__FUNCTION__, 'BEFORE HOOK ' . __CLASS__, __FILE__, __LINE__, 2);
+			if ($new_STS_ID === EEM_Registration::status_id_approved) {
+                \EEH_Debug_Tools::printr(__FUNCTION__, 'BEFORE HOOK '.__CLASS__, __FILE__, __LINE__, 2);
                 // reserve a space by incrementing ticket and datetime sold values
 				$this->_reserve_registration_space();
-				do_action( 'AHEE__EE_Registration__set_status__to_approved', $this, $old_STS_ID, $new_STS_ID );
-                \EEH_Debug_Tools::printr(__FUNCTION__, 'AFTER HOOK ' . __CLASS__, __FILE__, __LINE__, 2);
+				do_action('AHEE__EE_Registration__set_status__to_approved', $this, $old_STS_ID, $new_STS_ID);
+                \EEH_Debug_Tools::printr(__FUNCTION__, 'AFTER HOOK '.__CLASS__, __FILE__, __LINE__, 2);
 				exit();
 			// OR FROM  approved
-			} else if ( $old_STS_ID === EEM_Registration::status_id_approved ) {
+			} else if ($old_STS_ID === EEM_Registration::status_id_approved) {
 				// release a space by decrementing ticket and datetime sold values
 				$this->_release_registration_space();
-				do_action( 'AHEE__EE_Registration__set_status__from_approved', $this, $old_STS_ID, $new_STS_ID );
+				do_action('AHEE__EE_Registration__set_status__from_approved', $this, $old_STS_ID, $new_STS_ID);
 			}
 			// update status
-			parent::set( 'STS_ID', $new_STS_ID, $use_default );
+			parent::set('STS_ID', $new_STS_ID, $use_default);
 			/** @type EE_Registration_Processor $registration_processor */
-			$registration_processor = EE_Registry::instance()->load_class( 'Registration_Processor' );
+			$registration_processor = EE_Registry::instance()->load_class('Registration_Processor');
 			/** @type EE_Transaction_Processor $transaction_processor */
-			$transaction_processor = EE_Registry::instance()->load_class( 'Transaction_Processor' );
+			$transaction_processor = EE_Registry::instance()->load_class('Transaction_Processor');
 			/** @type EE_Transaction_Payments $transaction_payments */
-			$transaction_payments = EE_Registry::instance()->load_class( 'Transaction_Payments' );
+			$transaction_payments = EE_Registry::instance()->load_class('Transaction_Payments');
 			// these reg statuses should not be considered in any calculations involving monies owing
-			$closed_reg_statuses = ! empty( $closed_reg_statuses )
+			$closed_reg_statuses = ! empty($closed_reg_statuses)
 				? $closed_reg_statuses
 				: EEM_Registration::closed_reg_statuses();
 			if (
-				in_array( $new_STS_ID, $closed_reg_statuses )
-				&& ! in_array( $old_STS_ID, $closed_reg_statuses )
+				in_array($new_STS_ID, $closed_reg_statuses)
+				&& ! in_array($old_STS_ID, $closed_reg_statuses)
 			) {
 				// cancelled or declined registration
 				$registration_processor->update_registration_after_being_canceled_or_declined(
@@ -158,24 +158,24 @@ 
 					false
 				);
 			} else if (
-				in_array( $old_STS_ID, $closed_reg_statuses )
-				&& ! in_array( $new_STS_ID, $closed_reg_statuses )
+				in_array($old_STS_ID, $closed_reg_statuses)
+				&& ! in_array($new_STS_ID, $closed_reg_statuses)
 			) {
 				// reinstating cancelled or declined registration
 				$registration_processor->update_canceled_or_declined_registration_after_being_reinstated(
 					$this,
 					$closed_reg_statuses
 				);
-				$transaction_processor->update_transaction_after_reinstating_canceled_registration( $this );
+				$transaction_processor->update_transaction_after_reinstating_canceled_registration($this);
 			}
-			$transaction_payments->recalculate_transaction_total( $this->transaction(), false );
-			$this->transaction()->update_status_based_on_total_paid( true );
-			do_action( 'AHEE__EE_Registration__set_status__after_update', $this );
+			$transaction_payments->recalculate_transaction_total($this->transaction(), false);
+			$this->transaction()->update_status_based_on_total_paid(true);
+			do_action('AHEE__EE_Registration__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( 'STS_ID', $new_STS_ID, $use_default );
+			parent::set('STS_ID', $new_STS_ID, $use_default);
 			return TRUE;
 		}
 	}
@@ -186,7 +186,7 @@ 
 	 *        get Status ID
 	 */
 	public function status_ID() {
-		return $this->get( 'STS_ID' );
+		return $this->get('STS_ID');
 	}
 
 
@@ -211,12 +211,12 @@ 
 	 * @param boolean $include_archived whether to include archived tickets or not.
 	 * @return EE_Ticket
 	 */
-	public function ticket( $include_archived = TRUE ) {
+	public function ticket($include_archived = TRUE) {
 		$query_params = array();
-		if ( $include_archived ) {
-			$query_params[ 'default_where_conditions' ] = 'none';
+		if ($include_archived) {
+			$query_params['default_where_conditions'] = 'none';
 		}
-		return $this->get_first_related( 'Ticket', $query_params );
+		return $this->get_first_related('Ticket', $query_params);
 	}
 
 
@@ -244,7 +244,7 @@ 
 	 */
 	public function wp_user() {
 		$event = $this->event();
-		if ( $event instanceof EE_Event ) {
+		if ($event instanceof EE_Event) {
 			return $event->wp_user();
 		}
 		return 0;
@@ -269,8 +269,8 @@ 
 	 *
 	 * @param        int $ATT_ID Attendee ID
 	 */
-	public function set_attendee_id( $ATT_ID = 0 ) {
-		$this->set( 'ATT_ID', $ATT_ID );
+	public function set_attendee_id($ATT_ID = 0) {
+		$this->set('ATT_ID', $ATT_ID);
 	}
 
 
@@ -280,8 +280,8 @@ 
 	 *
 	 * @param        int $TXN_ID Transaction ID
 	 */
-	public function set_transaction_id( $TXN_ID = 0 ) {
-		$this->set( 'TXN_ID', $TXN_ID );
+	public function set_transaction_id($TXN_ID = 0) {
+		$this->set('TXN_ID', $TXN_ID);
 	}
 
 
@@ -291,8 +291,8 @@ 
 	 *
 	 * @param    string $REG_session PHP Session ID
 	 */
-	public function set_session( $REG_session = '' ) {
-		$this->set( 'REG_session', $REG_session );
+	public function set_session($REG_session = '') {
+		$this->set('REG_session', $REG_session);
 	}
 
 
@@ -302,8 +302,8 @@ 
 	 *
 	 * @param    string $REG_url_link Registration URL Link
 	 */
-	public function set_reg_url_link( $REG_url_link = '' ) {
-		$this->set( 'REG_url_link', $REG_url_link );
+	public function set_reg_url_link($REG_url_link = '') {
+		$this->set('REG_url_link', $REG_url_link);
 	}
 
 
@@ -313,8 +313,8 @@ 
 	 *
 	 * @param        int $REG_count Primary Attendee
 	 */
-	public function set_count( $REG_count = 1 ) {
-		$this->set( 'REG_count', $REG_count );
+	public function set_count($REG_count = 1) {
+		$this->set('REG_count', $REG_count);
 	}
 
 
@@ -324,8 +324,8 @@ 
 	 *
 	 * @param        boolean $REG_group_size Group Registration
 	 */
-	public function set_group_size( $REG_group_size = FALSE ) {
-		$this->set( 'REG_group_size', $REG_group_size );
+	public function set_group_size($REG_group_size = FALSE) {
+		$this->set('REG_group_size', $REG_group_size);
 	}
 
 
@@ -401,8 +401,8 @@ 
 	 *
 	 * @param        mixed ( int or string ) $REG_date Registration Date - Unix timestamp or string representation of Date
 	 */
-	public function set_reg_date( $REG_date = FALSE ) {
-		$this->set( 'REG_date', $REG_date );
+	public function set_reg_date($REG_date = FALSE) {
+		$this->set('REG_date', $REG_date);
 	}
 
 
@@ -413,8 +413,8 @@ 
 	 * @access    public
 	 * @param    float $REG_final_price
 	 */
-	public function set_final_price( $REG_final_price = 0.00 ) {
-		$this->set( 'REG_final_price', $REG_final_price );
+	public function set_final_price($REG_final_price = 0.00) {
+		$this->set('REG_final_price', $REG_final_price);
 	}
 
 
@@ -425,8 +425,8 @@ 
 	 * @access    public
 	 * @param    float $REG_paid
 	 */
-	public function set_paid( $REG_paid = 0.00 ) {
-		$this->set( 'REG_paid', $REG_paid );
+	public function set_paid($REG_paid = 0.00) {
+		$this->set('REG_paid', $REG_paid);
 	}
 
 
@@ -436,8 +436,8 @@ 
 	 *
 	 * @param        boolean $REG_att_is_going Attendee Is Going
 	 */
-	public function set_att_is_going( $REG_att_is_going = FALSE ) {
-		$this->set( 'REG_att_is_going', $REG_att_is_going );
+	public function set_att_is_going($REG_att_is_going = FALSE) {
+		$this->set('REG_att_is_going', $REG_att_is_going);
 	}
 
 
@@ -447,7 +447,7 @@ 
 	 * @return EE_Attendee
 	 */
 	public function attendee() {
-		return $this->get_first_related( 'Attendee' );
+		return $this->get_first_related('Attendee');
 	}
 
 
@@ -456,7 +456,7 @@ 
 	 *        get Event ID
 	 */
 	public function event_ID() {
-		return $this->get( 'EVT_ID' );
+		return $this->get('EVT_ID');
 	}
 
 
@@ -466,7 +466,7 @@ 
 	 */
 	public function event_name() {
 		$event = $this->event_obj();
-		if ( $event ) {
+		if ($event) {
 			return $event->name();
 		} else {
 			return NULL;
@@ -480,7 +480,7 @@ 
 	 * @return EE_Event
 	 */
 	public function event_obj() {
-		return $this->get_first_related( 'Event' );
+		return $this->get_first_related('Event');
 	}
 
 
@@ -489,7 +489,7 @@ 
 	 *        get Attendee ID
 	 */
 	public function attendee_ID() {
-		return $this->get( 'ATT_ID' );
+		return $this->get('ATT_ID');
 	}
 
 
@@ -498,7 +498,7 @@ 
 	 *        get PHP Session ID
 	 */
 	public function session_ID() {
-		return $this->get( 'REG_session' );
+		return $this->get('REG_session');
 	}
 
 
@@ -508,7 +508,7 @@ 
 	 * @param string $messenger 'pdf' or 'html'.  Default 'html'.
 	 * @return string
 	 */
-	public function receipt_url( $messenger = 'html' ) {
+	public function receipt_url($messenger = 'html') {
 
 		/**
 		 * The below will be deprecated one version after this.  We check first if there is a custom receipt template already in use on old system.  If there is then we just return the standard url for it.
@@ -516,12 +516,12 @@ 
 		 * @since 4.5.0
 		 */
 		$template_relative_path = 'modules/gateways/Invoice/lib/templates/receipt_body.template.php';
-		$has_custom = EEH_Template::locate_template( $template_relative_path , array(), TRUE, TRUE, TRUE );
+		$has_custom = EEH_Template::locate_template($template_relative_path, array(), TRUE, TRUE, TRUE);
 
-		if ( $has_custom ) {
-			return add_query_arg( array( 'receipt' => 'true' ), $this->invoice_url( 'launch' ) );
+		if ($has_custom) {
+			return add_query_arg(array('receipt' => 'true'), $this->invoice_url('launch'));
 		}
-		return apply_filters( 'FHEE__EE_Registration__receipt_url__receipt_url', '', $this, $messenger, 'receipt' );
+		return apply_filters('FHEE__EE_Registration__receipt_url__receipt_url', '', $this, $messenger, 'receipt');
 	}
 
 
@@ -532,28 +532,28 @@ 
 	 * @param string $messenger 'pdf' or 'html'.  Default 'html'.
 	 * @return string
 	 */
-	public function invoice_url( $messenger = 'html' ) {
+	public function invoice_url($messenger = 'html') {
 		/**
 		 * The below will be deprecated one version after this.  We check first if there is a custom invoice template already in use on old system.  If there is then we just return the standard url for it.
 		 *
 		 * @since 4.5.0
 		 */
 		$template_relative_path = 'modules/gateways/Invoice/lib/templates/invoice_body.template.php';
-		$has_custom = EEH_Template::locate_template( $template_relative_path , array(), TRUE, TRUE, TRUE );
+		$has_custom = EEH_Template::locate_template($template_relative_path, array(), TRUE, TRUE, TRUE);
 
-		if ( $has_custom ) {
-			if ( $messenger == 'html' ) {
-				return $this->invoice_url( 'launch' );
+		if ($has_custom) {
+			if ($messenger == 'html') {
+				return $this->invoice_url('launch');
 			}
 			$route = $messenger == 'download' || $messenger == 'pdf' ? 'download_invoice' : 'launch_invoice';
 
-			$query_args = array( 'ee' => $route, 'id' => $this->reg_url_link() );
-			if ( $messenger == 'html' ) {
+			$query_args = array('ee' => $route, 'id' => $this->reg_url_link());
+			if ($messenger == 'html') {
 				$query_args['html'] = TRUE;
 			}
-			return add_query_arg( $query_args, get_permalink( EE_Registry::instance()->CFG->core->thank_you_page_id ) );
+			return add_query_arg($query_args, get_permalink(EE_Registry::instance()->CFG->core->thank_you_page_id));
 		}
-		return apply_filters( 'FHEE__EE_Registration__invoice_url__invoice_url', '', $this, $messenger, 'invoice' );
+		return apply_filters('FHEE__EE_Registration__invoice_url__invoice_url', '', $this, $messenger, 'invoice');
 	}
 
 
@@ -566,7 +566,7 @@ 
 	 * @throws \EE_Error
 	 */
 	public function reg_url_link() {
-		return (string)$this->get( 'REG_url_link' );
+		return (string) $this->get('REG_url_link');
 	}
 
 
@@ -576,8 +576,8 @@ 
 	 * @param string $type 'download','launch', or 'html' (default is 'launch')
 	 * @return void
 	 */
-	public function e_invoice_url( $type = 'launch' ) {
-		echo $this->invoice_url( $type );
+	public function e_invoice_url($type = 'launch') {
+		echo $this->invoice_url($type);
 	}
 
 
@@ -597,7 +597,7 @@ 
 	 * @return string
 	 */
 	public function payment_overview_url() {
-		return add_query_arg( array( 'e_reg_url_link' => $this->reg_url_link(), 'step' => 'payment_options', 'revisit' => TRUE ), EE_Registry::instance()->CFG->core->reg_page_url() );
+		return add_query_arg(array('e_reg_url_link' => $this->reg_url_link(), 'step' => 'payment_options', 'revisit' => TRUE), EE_Registry::instance()->CFG->core->reg_page_url());
 	}
 
 
@@ -608,7 +608,7 @@ 
 	 * @return string
 	 */
 	public function edit_attendee_information_url() {
-		return add_query_arg( array( 'e_reg_url_link' => $this->reg_url_link(), 'step' => 'attendee_information', 'revisit' => TRUE ), EE_Registry::instance()->CFG->core->reg_page_url() );
+		return add_query_arg(array('e_reg_url_link' => $this->reg_url_link(), 'step' => 'attendee_information', 'revisit' => TRUE), EE_Registry::instance()->CFG->core->reg_page_url());
 	}
 
 
@@ -618,7 +618,7 @@ 
 	 * @return string
 	 */
 	public function get_admin_edit_url() {
-		return EEH_URL::add_query_args_and_nonce( array( 'page' => 'espresso_registrations', 'action' => 'view_registration', '_REG_ID' => $this->ID() ), admin_url( 'admin.php' ) );
+		return EEH_URL::add_query_args_and_nonce(array('page' => 'espresso_registrations', 'action' => 'view_registration', '_REG_ID' => $this->ID()), admin_url('admin.php'));
 	}
 
 
@@ -627,7 +627,7 @@ 
 	 *    is_primary_registrant?
 	 */
 	public function is_primary_registrant() {
-		return $this->get( 'REG_count' ) == 1 ? TRUE : FALSE;
+		return $this->get('REG_count') == 1 ? TRUE : FALSE;
 	}
 
 
@@ -636,12 +636,12 @@ 
 	 * This returns the primary registration object for this registration group (which may be this object).
 	 * @return EE_Registration
 	 */
-	public function get_primary_registration()  {
-		if ( $this->is_primary_registrant() )
+	public function get_primary_registration() {
+		if ($this->is_primary_registrant())
 			return $this;
 
 		//k reg_count !== 1 so let's get the EE_Registration object matching this txn_id and reg_count == 1
-		$primary_registrant = EEM_Registration::instance()->get_one( array( array('TXN_ID' => $this->transaction_ID(), 'REG_count' => 1 ) ) );
+		$primary_registrant = EEM_Registration::instance()->get_one(array(array('TXN_ID' => $this->transaction_ID(), 'REG_count' => 1)));
 		return $primary_registrant;
 	}
 
@@ -652,7 +652,7 @@ 
 	* 		@access		public
 	*/
 	public function count() {
-		return $this->get( 'REG_count' );
+		return $this->get('REG_count');
 	}
 
 
@@ -661,7 +661,7 @@ 
 	 *        get Group Size
 	 */
 	public function group_size() {
-		return $this->get( 'REG_group_size' );
+		return $this->get('REG_group_size');
 	}
 
 
@@ -670,7 +670,7 @@ 
 	 *        get Registration Date
 	 */
 	public function date() {
-		return $this->get( 'REG_date' );
+		return $this->get('REG_date');
 	}
 
 
@@ -681,8 +681,8 @@ 
 	 * @param string $time_format
 	 * @return string
 	 */
-	public function pretty_date( $date_format = NULL, $time_format = NULL ) {
-		return $this->get_datetime( 'REG_date', $date_format, $time_format );
+	public function pretty_date($date_format = NULL, $time_format = NULL) {
+		return $this->get_datetime('REG_date', $date_format, $time_format);
 	}
 
 
@@ -694,7 +694,7 @@ 
 	 * @return    float
 	 */
 	public function final_price() {
-		return $this->get( 'REG_final_price' );
+		return $this->get('REG_final_price');
 	}
 
 
@@ -705,7 +705,7 @@ 
 	 * @return string
 	 */
 	public function pretty_final_price() {
-		return $this->get_pretty( 'REG_final_price' );
+		return $this->get_pretty('REG_final_price');
 	}
 
 
@@ -715,7 +715,7 @@ 
 	 * @return 	float
 	 */
 	public function paid() {
-		return $this->get( 'REG_paid' );
+		return $this->get('REG_paid');
 	}
 
 
@@ -725,7 +725,7 @@ 
 	 * @return 	float
 	 */
 	public function pretty_paid() {
-		return $this->get_pretty( 'REG_paid' );
+		return $this->get_pretty('REG_paid');
 	}
 
 
@@ -736,11 +736,11 @@ 
 	 * @param array $requires_payment
 	 * @return bool
 	 */
-	public function owes_monies_and_can_pay( $requires_payment = array()) {
+	public function owes_monies_and_can_pay($requires_payment = array()) {
 		// these reg statuses require payment (if event is not free)
-		$requires_payment = ! empty( $requires_payment ) ? $requires_payment : EEM_Registration::reg_statuses_that_allow_payment();
+		$requires_payment = ! empty($requires_payment) ? $requires_payment : EEM_Registration::reg_statuses_that_allow_payment();
 		if (
-			in_array( $this->status_ID(), $requires_payment ) &&
+			in_array($this->status_ID(), $requires_payment) &&
 			$this->final_price() != 0 &&
 			$this->final_price() != $this->paid()
 		) {
@@ -757,8 +757,8 @@ 
 	 * @param bool $show_icons
 	 * @return void
 	 */
-	public function e_pretty_status( $show_icons = FALSE ) {
-		echo $this->pretty_status( $show_icons );
+	public function e_pretty_status($show_icons = FALSE) {
+		echo $this->pretty_status($show_icons);
 	}
 
 
@@ -769,10 +769,10 @@ 
 	 * @param bool $show_icons
 	 * @return string
 	 */
-	public function pretty_status( $show_icons = FALSE ) {
-		$status = EEM_Status::instance()->localized_status( array( $this->status_ID() => __( 'unknown', 'event_espresso' ) ), FALSE, 'sentence' );
+	public function pretty_status($show_icons = FALSE) {
+		$status = EEM_Status::instance()->localized_status(array($this->status_ID() => __('unknown', 'event_espresso')), FALSE, 'sentence');
 		$icon = '';
-		switch ( $this->status_ID() ) {
+		switch ($this->status_ID()) {
 			case EEM_Registration::status_id_approved:
 				$icon = $show_icons ? '<span class="dashicons dashicons-star-filled ee-icon-size-16 green-text"></span>' : '';
 				break;
@@ -795,7 +795,7 @@ 
 				$icon = $show_icons ? '<span class="dashicons dashicons-clipboard ee-icon-size-16 purple-text"></span>' : '';
 				break;
 		}
-		return $icon . $status[ $this->status_ID() ];
+		return $icon.$status[$this->status_ID()];
 	}
 
 
@@ -804,7 +804,7 @@ 
 	 *        get Attendee Is Going
 	 */
 	public function att_is_going() {
-		return $this->get( 'REG_att_is_going' );
+		return $this->get('REG_att_is_going');
 	}
 
 
@@ -814,8 +814,8 @@ 
 	 * @param array $query_params like EEM_Base::get_all
 	 * @return EE_Answer[]
 	 */
-	public function answers( $query_params = NULL ) {
-		return $this->get_many_related( 'Answer', $query_params );
+	public function answers($query_params = NULL) {
+		return $this->get_many_related('Answer', $query_params);
 	}
 
 
@@ -829,9 +829,9 @@ 
 	 * (because the answer might be an array of answer values, so passing pretty_value=true
 	 * will convert it into some kind of string)
 	 */
-	public function answer_value_to_question( $question, $pretty_value=true ) {
+	public function answer_value_to_question($question, $pretty_value = true) {
 		$question_id = EEM_Question::instance()->ensure_is_ID($question);
-		return EEM_Answer::instance()->get_answer_value_to_question($this,$question_id,$pretty_value);
+		return EEM_Answer::instance()->get_answer_value_to_question($this, $question_id, $pretty_value);
 	}
 
 
@@ -844,13 +844,13 @@ 
 	 */
 	public function question_groups() {
 		$question_groups = array();
-		if ( $this->event() instanceof EE_Event ) {
+		if ($this->event() instanceof EE_Event) {
 			$question_groups = $this->event()->question_groups(
 				array(
 					array(
 						'Event_Question_Group.EQG_primary' => $this->count() == 1 ? true : false
 					),
-					'order_by' => array( 'QSG_order' => 'ASC' )
+					'order_by' => array('QSG_order' => 'ASC')
 				)
 			);
 		}
@@ -867,7 +867,7 @@ 
 	 */
 	public function count_question_groups() {
 		$qg_count = 0;
-		if ( $this->event() instanceof EE_Event ) {
+		if ($this->event() instanceof EE_Event) {
 			$qg_count = $this->event()->count_related(
 				'Question_Group',
 				array(
@@ -888,7 +888,7 @@ 
 	 * @return string
 	 */
 	public function reg_date() {
-		return $this->get_datetime( 'REG_date' );
+		return $this->get_datetime('REG_date');
 	}
 
 
@@ -900,7 +900,7 @@ 
 	 * @return EE_Datetime_Ticket
 	 */
 	public function datetime_ticket() {
-		return $this->get_first_related( 'Datetime_Ticket' );
+		return $this->get_first_related('Datetime_Ticket');
 	}
 
 
@@ -910,15 +910,15 @@ 
 	 * @param EE_Datetime_Ticket $datetime_ticket
 	 * @return EE_Datetime_Ticket
 	 */
-	public function set_datetime_ticket( $datetime_ticket ) {
-		return $this->_add_relation_to( $datetime_ticket, 'Datetime_Ticket' );
+	public function set_datetime_ticket($datetime_ticket) {
+		return $this->_add_relation_to($datetime_ticket, 'Datetime_Ticket');
 	}
 	/**
 	 * Gets deleted
 	 * @return boolean
 	 */
 	public function deleted() {
-		return $this->get( 'REG_deleted' );
+		return $this->get('REG_deleted');
 	}
 
 	/**
@@ -927,7 +927,7 @@ 
 	 * @return boolean
 	 */
 	public function set_deleted($deleted) {
-		$this->set( 'REG_deleted', $deleted );
+		$this->set('REG_deleted', $deleted);
 	}
 
 
@@ -937,7 +937,7 @@ 
 	 * @return EE_Status
 	 */
 	public function status_obj() {
-		return $this->get_first_related( 'Status' );
+		return $this->get_first_related('Status');
 	}
 
 
@@ -948,7 +948,7 @@ 
 	 * @return int
 	 */
 	public function count_checkins() {
-		return $this->get_model()->count_related( $this, 'Checkin' );
+		return $this->get_model()->count_related($this, 'Checkin');
 	}
 
 
@@ -958,7 +958,7 @@ 
 	 * @return int
 	 */
 	public function count_checkins_not_checkedout() {
-		return $this->get_model()->count_related( $this, 'Checkin', array( array( 'CHK_in' => 1 ) ) );
+		return $this->get_model()->count_related($this, 'Checkin', array(array('CHK_in' => 1)));
 	}
 
 
@@ -971,20 +971,20 @@ 
 	 *
 	 * @return bool
 	 */
-	public function can_checkin( $DTT_OR_ID, $check_approved = TRUE ) {
-		$DTT_ID = EEM_Datetime::instance()->ensure_is_ID( $DTT_OR_ID );
+	public function can_checkin($DTT_OR_ID, $check_approved = TRUE) {
+		$DTT_ID = EEM_Datetime::instance()->ensure_is_ID($DTT_OR_ID);
 
 		//first check registration status
-		if (  ( $check_approved && ! $this->is_approved() ) || ! $DTT_ID ) {
+		if (($check_approved && ! $this->is_approved()) || ! $DTT_ID) {
 			return false;
 		}
 		//is there a datetime ticket that matches this dtt_ID?
-		if ( ! ( EEM_Datetime_Ticket::instance()->exists( array( array( 'TKT_ID' => $this->get('TKT_ID' ), 'DTT_ID' => $DTT_ID ) ) ) ) ) {
+		if ( ! (EEM_Datetime_Ticket::instance()->exists(array(array('TKT_ID' => $this->get('TKT_ID'), 'DTT_ID' => $DTT_ID))))) {
 			return false;
 		}
 
 		//final check is against TKT_uses
-		return $this->verify_can_checkin_against_TKT_uses( $DTT_ID );
+		return $this->verify_can_checkin_against_TKT_uses($DTT_ID);
 	}
 
 
@@ -997,10 +997,10 @@ 
 	 * @param int | EE_Datetime  $DTT_OR_ID  The datetime the registration is being checked against
 	 * @return bool   true means can checkin.  false means cannot checkin.
 	 */
-	public function verify_can_checkin_against_TKT_uses( $DTT_OR_ID ) {
-		$DTT_ID = EEM_Datetime::instance()->ensure_is_ID( $DTT_OR_ID );
+	public function verify_can_checkin_against_TKT_uses($DTT_OR_ID) {
+		$DTT_ID = EEM_Datetime::instance()->ensure_is_ID($DTT_OR_ID);
 
-		if ( ! $DTT_ID ) {
+		if ( ! $DTT_ID) {
 			return false;
 		}
 
@@ -1008,23 +1008,23 @@ 
 
 		// if max uses is not set or equals infinity then return true cause its not a factor for whether user can check-in
 		// or not.
-		if ( ! $max_uses || $max_uses === EE_INF ) {
+		if ( ! $max_uses || $max_uses === EE_INF) {
 			return true;
 		}
 
 		//does this datetime have a checkin record?  If so, then the dtt count has already been verified so we can just
 		//go ahead and toggle.
-		if ( EEM_Checkin::instance()->exists( array( array( 'REG_ID' => $this->ID(), 'DTT_ID' => $DTT_ID ) ) ) ) {
+		if (EEM_Checkin::instance()->exists(array(array('REG_ID' => $this->ID(), 'DTT_ID' => $DTT_ID)))) {
 			return true;
 		}
 
 		//made it here so the last check is whether the number of checkins per unique datetime on this registration
 		//disallows further check-ins.
-		$count_unique_dtt_checkins = EEM_Checkin::instance()->count( array( array( 'REG_ID' => $this->ID(), 'CHK_in' => true ) ), 'DTT_ID', true );
+		$count_unique_dtt_checkins = EEM_Checkin::instance()->count(array(array('REG_ID' => $this->ID(), 'CHK_in' => true)), 'DTT_ID', true);
 		// checkins have already reached their max number of uses
 		// so registrant can NOT checkin
-		if ( $count_unique_dtt_checkins >= $max_uses ) {
-			EE_Error::add_error( __( 'Check-in denied because number of datetime uses for the ticket has been reached or exceeded.', 'event_espresso' ), __FILE__, __FUNCTION__, __LINE__ );
+		if ($count_unique_dtt_checkins >= $max_uses) {
+			EE_Error::add_error(__('Check-in denied because number of datetime uses for the ticket has been reached or exceeded.', 'event_espresso'), __FILE__, __FUNCTION__, __LINE__);
 			return false;
 		}
 		return true;
@@ -1046,15 +1046,15 @@ 
      * @return bool|int     the chk_in status toggled to OR false if nothing got changed.
      * @throws EE_Error
      */
-	public function toggle_checkin_status( $DTT_ID = null, $verify = false ) {
-		if ( empty( $DTT_ID ) ) {
+	public function toggle_checkin_status($DTT_ID = null, $verify = false) {
+		if (empty($DTT_ID)) {
 			$datetime = $this->get_latest_related_datetime();
 			$DTT_ID = $datetime instanceof EE_Datetime ? $datetime->ID() : 0;
 		// verify the registration can checkin for the given DTT_ID
-		} elseif ( ! $this->can_checkin( $DTT_ID, $verify ) ) {
+		} elseif ( ! $this->can_checkin($DTT_ID, $verify)) {
 			EE_Error::add_error(
 					sprintf(
-						__( 'The given registration (ID:%1$d) can not be checked in to the given DTT_ID (%2$d), because the registration does not have access', 'event_espresso'),
+						__('The given registration (ID:%1$d) can not be checked in to the given DTT_ID (%2$d), because the registration does not have access', 'event_espresso'),
 						$this->ID(),
 						$DTT_ID
 					),
@@ -1068,8 +1068,8 @@ 
 			EE_Registration::checkin_status_out => EE_Registration::checkin_status_in
 		);
 		//start by getting the current status so we know what status we'll be changing to.
-		$cur_status = $this->check_in_status_for_datetime( $DTT_ID, NULL );
-		$status_to = $status_paths[ $cur_status ];
+		$cur_status = $this->check_in_status_for_datetime($DTT_ID, NULL);
+		$status_to = $status_paths[$cur_status];
 		// database only records true for checked IN or false for checked OUT
 		// no record ( null ) means checked in NEVER, but we obviously don't save that
 		$new_status = $status_to === EE_Registration::checkin_status_in ? true : false;
@@ -1077,24 +1077,24 @@ 
 		// because we are keeping track of Check-ins over time.
 		// Eventually we'll probably want to show a list table
 		// for the individual Check-ins so that they can be managed.
-		$checkin = EE_Checkin::new_instance( array(
+		$checkin = EE_Checkin::new_instance(array(
 				'REG_ID' => $this->ID(),
 				'DTT_ID' => $DTT_ID,
 				'CHK_in' => $new_status
-		) );
+		));
 		// if the record could not be saved then return false
-		if ( $checkin->save() === 0 ) {
-			if ( WP_DEBUG ) {
+		if ($checkin->save() === 0) {
+			if (WP_DEBUG) {
 				global $wpdb;
 				$error = sprintf(
-					__( 'Registration check in update failed because of the following database error: %1$s%2$s', 'event_espresso' ),
+					__('Registration check in update failed because of the following database error: %1$s%2$s', 'event_espresso'),
 					'<br />',
 					$wpdb->last_error
 				);
 			} else {
-				$error = __( 'Registration check in update failed because of an unknown database error', 'event_espresso' );
+				$error = __('Registration check in update failed because of an unknown database error', 'event_espresso');
 			}
-			EE_Error::add_error( $error, __FILE__, __FUNCTION__, __LINE__ );
+			EE_Error::add_error($error, __FILE__, __FUNCTION__, __LINE__);
 			return false;
 		}
 		return $status_to;
@@ -1115,7 +1115,7 @@ 
 				array(
 					'Ticket.Registration.REG_ID' => $this->ID()
 				),
-				'order_by' => array( 'DTT_EVT_start' => 'DESC' )
+				'order_by' => array('DTT_EVT_start' => 'DESC')
 			)
 		);
 	}
@@ -1134,7 +1134,7 @@ 
 				array(
 					'Ticket.Registration.REG_ID' => $this->ID()
 				),
-				'order_by' => array( 'DTT_EVT_start' => 'ASC' )
+				'order_by' => array('DTT_EVT_start' => 'ASC')
 			)
 		);
 	}
@@ -1153,21 +1153,21 @@ 
      * @return int                Integer representing Check-in status.
      * @throws \EE_Error
      */
-	public function check_in_status_for_datetime( $DTT_ID = 0, $checkin = null ) {
+	public function check_in_status_for_datetime($DTT_ID = 0, $checkin = null) {
 		$checkin_query_params = array(
-			'order_by' => array( 'CHK_timestamp' => 'DESC' )
+			'order_by' => array('CHK_timestamp' => 'DESC')
 		);
 
-		if ( $DTT_ID > 0 ) {
-			$checkin_query_params[0] = array( 'DTT_ID' => $DTT_ID );
+		if ($DTT_ID > 0) {
+			$checkin_query_params[0] = array('DTT_ID' => $DTT_ID);
 		}
 
 		//get checkin object (if exists)
 		$checkin = $checkin instanceof EE_Checkin
             ? $checkin
-            : $this->get_first_related( 'Checkin', $checkin_query_params );
-		if ( $checkin instanceof EE_Checkin ) {
-			if ( $checkin->get( 'CHK_in' ) ) {
+            : $this->get_first_related('Checkin', $checkin_query_params);
+		if ($checkin instanceof EE_Checkin) {
+			if ($checkin->get('CHK_in')) {
 				return EE_Registration::checkin_status_in; //checked in
 			}
 			return EE_Registration::checkin_status_out; //had checked in but is now checked out.
@@ -1183,28 +1183,28 @@ 
 	 * @param bool $error  This just flags that you want an error message returned. This is put in so that the error message can be customized with the attendee name.
 	 * @return string         internationalized message
 	 */
-	public function get_checkin_msg( $DTT_ID, $error = FALSE ) {
+	public function get_checkin_msg($DTT_ID, $error = FALSE) {
 		//let's get the attendee first so we can include the name of the attendee
-		$attendee = $this->get_first_related( 'Attendee' );
-		if ( $attendee instanceof EE_Attendee ) {
-			if ( $error ) {
-				return sprintf( __( "%s's check-in status was not changed.", "event_espresso" ), $attendee->full_name() );
+		$attendee = $this->get_first_related('Attendee');
+		if ($attendee instanceof EE_Attendee) {
+			if ($error) {
+				return sprintf(__("%s's check-in status was not changed.", "event_espresso"), $attendee->full_name());
 			}
-			$cur_status = $this->check_in_status_for_datetime( $DTT_ID );
+			$cur_status = $this->check_in_status_for_datetime($DTT_ID);
 			//what is the status message going to be?
-			switch ( $cur_status ) {
+			switch ($cur_status) {
 				case EE_Registration::checkin_status_never :
-					return sprintf( __( "%s has been removed from Check-in records", "event_espresso" ), $attendee->full_name() );
+					return sprintf(__("%s has been removed from Check-in records", "event_espresso"), $attendee->full_name());
 					break;
 				case EE_Registration::checkin_status_in :
-					return sprintf( __( '%s has been checked in', 'event_espresso' ), $attendee->full_name() );
+					return sprintf(__('%s has been checked in', 'event_espresso'), $attendee->full_name());
 					break;
 				case EE_Registration::checkin_status_out :
-					return sprintf( __( '%s has been checked out', 'event_espresso' ), $attendee->full_name() );
+					return sprintf(__('%s has been checked out', 'event_espresso'), $attendee->full_name());
 					break;
 			}
 		}
-		return __( "The check-in status could not be determined.", "event_espresso" );
+		return __("The check-in status could not be determined.", "event_espresso");
 	}
 
 
@@ -1228,7 +1228,7 @@ 
 	 *        get Registration Code
 	 */
 	public function reg_code() {
-		return $this->get( 'REG_code' );
+		return $this->get('REG_code');
 	}
 
 
@@ -1237,7 +1237,7 @@ 
 	 *        get Transaction ID
 	 */
 	public function transaction_ID() {
-		return $this->get( 'TXN_ID' );
+		return $this->get('TXN_ID');
 	}
 
 
@@ -1246,7 +1246,7 @@ 
 	 * @return int
 	 */
 	public function ticket_ID() {
-		return $this->get( 'TKT_ID' );
+		return $this->get('TKT_ID');
 	}
 
 
@@ -1258,17 +1258,17 @@ 
 	 * @param    string $REG_code Registration Code
 	 * @param	boolean $use_default
 	 */
-	public function set_reg_code( $REG_code, $use_default = FALSE ) {
-		if ( empty( $REG_code )) {
-			EE_Error::add_error( __( 'REG_code can not be empty.', 'event_espresso'), __FILE__, __FUNCTION__, __LINE__ );
+	public function set_reg_code($REG_code, $use_default = FALSE) {
+		if (empty($REG_code)) {
+			EE_Error::add_error(__('REG_code can not be empty.', 'event_espresso'), __FILE__, __FUNCTION__, __LINE__);
 			return;
 		}
-		if ( ! $this->reg_code() ) {
-			parent::set( 'REG_code', $REG_code, $use_default );
+		if ( ! $this->reg_code()) {
+			parent::set('REG_code', $REG_code, $use_default);
 		} else {
 			EE_Error::doing_it_wrong(
-				__CLASS__ . '::' . __FUNCTION__,
-				__( 'Can not change a registration REG_code once it has been set.', 'event_espresso' ),
+				__CLASS__.'::'.__FUNCTION__,
+				__('Can not change a registration REG_code once it has been set.', 'event_espresso'),
 				'4.6.0'
 			);
 		}
@@ -1288,17 +1288,17 @@ 
 	 * @return EE_Registration[]  or empty array if this isn't a group registration.
 	 */
 	public function get_all_other_registrations_in_group() {
-		if ( $this->group_size() < 2 ) {
+		if ($this->group_size() < 2) {
 			return array();
 		}
 
 		$query[0] = array(
 			'TXN_ID' => $this->transaction_ID(),
-			'REG_ID' => array( '!=', $this->ID() ),
+			'REG_ID' => array('!=', $this->ID()),
 			'TKT_ID' => $this->ticket_ID()
 			);
 
-		$registrations = $this->get_model()->get_all( $query );
+		$registrations = $this->get_model()->get_all($query);
 		return $registrations;
 	}
 
@@ -1307,14 +1307,14 @@ 
 	 * @return string
 	 */
 	public function get_admin_details_link() {
-		EE_Registry::instance()->load_helper( 'URL' );
+		EE_Registry::instance()->load_helper('URL');
 		return EEH_URL::add_query_args_and_nonce(
 			array(
 				'page' => 'espresso_registrations',
 				'action' => 'view_registration',
 				'_REG_ID' => $this->ID()
 			),
-			admin_url( 'admin.php' )
+			admin_url('admin.php')
 		);
 	}
 
@@ -1339,12 +1339,12 @@ 
 	 * @return string
 	 */
 	public function get_admin_overview_link() {
-		EE_Registry::instance()->load_helper( 'URL' );
+		EE_Registry::instance()->load_helper('URL');
 		return EEH_URL::add_query_args_and_nonce(
 			array(
 				'page' => 'espresso_registrations'
 			),
-			admin_url( 'admin.php' )
+			admin_url('admin.php')
 		);
 	}
 
@@ -1355,8 +1355,8 @@ 
 	 * @return \EE_Registration[]
 	 * @throws \EE_Error
 	 */
-	public function payments( $query_params = array() ) {
-		return $this->get_many_related( 'Payment', $query_params );
+	public function payments($query_params = array()) {
+		return $this->get_many_related('Payment', $query_params);
 	}
 
 
@@ -1366,8 +1366,8 @@ 
 	 * @return \EE_Registration_Payment[]
 	 * @throws \EE_Error
 	 */
-	public function registration_payments( $query_params = array() ) {
-		return $this->get_many_related( 'Registration_Payment', $query_params );
+	public function registration_payments($query_params = array()) {
+		return $this->get_many_related('Registration_Payment', $query_params);
 	}
 
 
@@ -1380,7 +1380,7 @@ 
 	 * @return EE_Payment_Method|null
 	 */
 	public function payment_method() {
-		return EEM_Payment_Method::instance()->get_last_used_for_registration( $this );
+		return EEM_Payment_Method::instance()->get_last_used_for_registration($this);
 	}
 
 

core/domain/services/automated_actions/EventAdminNotification.php

Indentation +46 added lines, -46 removed lines

@@ -23,52 +23,52 @@
 
 
 
-    /**
-     * when a trigger has been "pulled",
-     * either by a do_action or scheduled cron,
-     * this is the callback method that will run
-     * to actually execute the logic
-     * for the automated action
-     *
-     * @throws InvalidEntityException
-     * @throws InvalidArgumentException
-     */
-    public function execute()
-    {
-        \EEH_Debug_Tools::printr(__FUNCTION__, __CLASS__, __FILE__, __LINE__, 2);
-        $params = $this->getParams();
-        if ( ! isset($params[0], $params[1], $params[2], $params['query_params']) ) {
-            throw new InvalidArgumentException();
-        }
-        if ( ! $params[0] instanceof EE_Registration) {
-            throw new InvalidEntityException($params[0], 'EE_Registration');
-        }
-        /** @var EE_Registration $registration */
-        list($registration, $old_STS_ID, $new_STS_ID) = $params;
-        // $registration = $params[0];
-        // $old_STS_ID = $params[1];
-        // $new_STS_ID = $params[2];
-        $query_params = $params['query_params'];
-        \EEH_Debug_Tools::printr($registration, '$registration', __FILE__, __LINE__);
-        \EEH_Debug_Tools::printr($old_STS_ID, '$old_STS_ID', __FILE__, __LINE__);
-        \EEH_Debug_Tools::printr($new_STS_ID, '$new_STS_ID', __FILE__, __LINE__);
-        \EEH_Debug_Tools::printr($query_params, 'Trigger Rules query_params', __FILE__, __LINE__);
-        $where = array();
-        foreach ($query_params as $key => $query_param) {
-            \EEH_Debug_Tools::printr($key, '$key', __FILE__, __LINE__);
-            \EEH_Debug_Tools::printr($query_param, '$query_param', __FILE__, __LINE__);
-            if($key === 0 ) {
-                $where[0] = isset($where[0]) ? array_merge($where[0], $query_param) : $query_param;
-            } else if( is_int($key) ) {
-                $where["AND*{$key}"]= $query_param;
-            } else {
-                $where[$key] = $query_param;
-            }
-            \EEH_Debug_Tools::printr($where, '$where', __FILE__, __LINE__);
-        }
-        $this->setHasRun();
-        exit();
-    }
+	/**
+	 * when a trigger has been "pulled",
+	 * either by a do_action or scheduled cron,
+	 * this is the callback method that will run
+	 * to actually execute the logic
+	 * for the automated action
+	 *
+	 * @throws InvalidEntityException
+	 * @throws InvalidArgumentException
+	 */
+	public function execute()
+	{
+		\EEH_Debug_Tools::printr(__FUNCTION__, __CLASS__, __FILE__, __LINE__, 2);
+		$params = $this->getParams();
+		if ( ! isset($params[0], $params[1], $params[2], $params['query_params']) ) {
+			throw new InvalidArgumentException();
+		}
+		if ( ! $params[0] instanceof EE_Registration) {
+			throw new InvalidEntityException($params[0], 'EE_Registration');
+		}
+		/** @var EE_Registration $registration */
+		list($registration, $old_STS_ID, $new_STS_ID) = $params;
+		// $registration = $params[0];
+		// $old_STS_ID = $params[1];
+		// $new_STS_ID = $params[2];
+		$query_params = $params['query_params'];
+		\EEH_Debug_Tools::printr($registration, '$registration', __FILE__, __LINE__);
+		\EEH_Debug_Tools::printr($old_STS_ID, '$old_STS_ID', __FILE__, __LINE__);
+		\EEH_Debug_Tools::printr($new_STS_ID, '$new_STS_ID', __FILE__, __LINE__);
+		\EEH_Debug_Tools::printr($query_params, 'Trigger Rules query_params', __FILE__, __LINE__);
+		$where = array();
+		foreach ($query_params as $key => $query_param) {
+			\EEH_Debug_Tools::printr($key, '$key', __FILE__, __LINE__);
+			\EEH_Debug_Tools::printr($query_param, '$query_param', __FILE__, __LINE__);
+			if($key === 0 ) {
+				$where[0] = isset($where[0]) ? array_merge($where[0], $query_param) : $query_param;
+			} else if( is_int($key) ) {
+				$where["AND*{$key}"]= $query_param;
+			} else {
+				$where[$key] = $query_param;
+			}
+			\EEH_Debug_Tools::printr($where, '$where', __FILE__, __LINE__);
+		}
+		$this->setHasRun();
+		exit();
+	}
 
 
 

Spacing +4 added lines, -4 removed lines

@@ -37,7 +37,7 @@ 
     {
         \EEH_Debug_Tools::printr(__FUNCTION__, __CLASS__, __FILE__, __LINE__, 2);
         $params = $this->getParams();
-        if ( ! isset($params[0], $params[1], $params[2], $params['query_params']) ) {
+        if ( ! isset($params[0], $params[1], $params[2], $params['query_params'])) {
             throw new InvalidArgumentException();
         }
         if ( ! $params[0] instanceof EE_Registration) {
@@ -57,10 +57,10 @@ 
         foreach ($query_params as $key => $query_param) {
             \EEH_Debug_Tools::printr($key, '$key', __FILE__, __LINE__);
             \EEH_Debug_Tools::printr($query_param, '$query_param', __FILE__, __LINE__);
-            if($key === 0 ) {
+            if ($key === 0) {
                 $where[0] = isset($where[0]) ? array_merge($where[0], $query_param) : $query_param;
-            } else if( is_int($key) ) {
-                $where["AND*{$key}"]= $query_param;
+            } else if (is_int($key)) {
+                $where["AND*{$key}"] = $query_param;
             } else {
                 $where[$key] = $query_param;
             }

core/domain/services/conditional_logic/rules/EventQuery.php

Indentation +29 added lines, -29 removed lines

@@ -21,44 +21,44 @@
 
 
 
-    /**
-     * @return \EEM_Event
-     */
-    protected function get_model()
-    {
-        if (!$this->model instanceof EEM_Event) {
-            return EEM_Event::instance();
-        }
-        return $this->model;
-    }
+	/**
+	 * @return \EEM_Event
+	 */
+	protected function get_model()
+	{
+		if (!$this->model instanceof EEM_Event) {
+			return EEM_Event::instance();
+		}
+		return $this->model;
+	}
 
 
 
-    public function eventCategory()
-    {
-        return array(
-            'Term_Taxonomy.taxonomy' => array($this->comparison, 'espresso_event_categories' ),
-            'Term_Taxonomy.term_id' => array($this->comparison, $this->value),
-        );
-    }
+	public function eventCategory()
+	{
+		return array(
+			'Term_Taxonomy.taxonomy' => array($this->comparison, 'espresso_event_categories' ),
+			'Term_Taxonomy.term_id' => array($this->comparison, $this->value),
+		);
+	}
 
 
 
-    public function eventStart()
-    {
-        return array(
-            'Datetime.DTT_EVT_start' => array($this->comparison, $this->value),
-        );
-    }
+	public function eventStart()
+	{
+		return array(
+			'Datetime.DTT_EVT_start' => array($this->comparison, $this->value),
+		);
+	}
 
 
 
-    public function eventEnd()
-    {
-        return array(
-            'Datetime.DTT_EVT_end' => array($this->comparison, $this->value),
-        );
-    }
+	public function eventEnd()
+	{
+		return array(
+			'Datetime.DTT_EVT_end' => array($this->comparison, $this->value),
+		);
+	}
 
 }
 // End of file EventQuery.php

Spacing +2 added lines, -2 removed lines

@@ -26,7 +26,7 @@ 
      */
     protected function get_model()
     {
-        if (!$this->model instanceof EEM_Event) {
+        if ( ! $this->model instanceof EEM_Event) {
             return EEM_Event::instance();
         }
         return $this->model;
@@ -37,7 +37,7 @@ 
     public function eventCategory()
     {
         return array(
-            'Term_Taxonomy.taxonomy' => array($this->comparison, 'espresso_event_categories' ),
+            'Term_Taxonomy.taxonomy' => array($this->comparison, 'espresso_event_categories'),
             'Term_Taxonomy.term_id' => array($this->comparison, $this->value),
         );
     }

core/services/automated_actions/AutomatedActionManager.php

Indentation +130 added lines, -130 removed lines

@@ -27,136 +27,136 @@
 {
 
 
-    /**
-     * @var Collection $automated_actions
-     */
-    private $automated_actions;
-
-    /**
-     * @type CapabilitiesChecker $capabilities_checker
-     */
-    private $capabilities_checker;
-
-
-
-    /**
-     * AutomatedActionManager constructor
-     *
-     * @param CapabilitiesChecker $capabilities_checker
-     * @param Collection          $automated_actions
-     * @throws InvalidInterfaceException
-     */
-    public function __construct(CapabilitiesChecker $capabilities_checker, Collection $automated_actions = null)
-    {
-        Benchmark::startTimer(__METHOD__);
-        $this->capabilities_checker = $capabilities_checker;
-        $this->automated_actions    = $automated_actions instanceof Collection
-            ? $automated_actions
-            : new Collection('EventEspresso\core\services\automated_actions\AutomatedActionInterface');
-        add_action('AHEE__EE_System__initialize', array($this, 'initialize'));
-        Benchmark::stopTimer(__METHOD__);
-    }
-
-
-
-    /**
-     * @throws \DomainException
-     * @throws InvalidEntityException
-     * @throws InsufficientPermissionsException
-     * @throws InvalidClassException
-     */
-    public function initialize()
-    {
-        $results = $this->getActiveActions();
-        $this->generateActionObjects($results);
-        $this->setTriggers();
-        add_action('shutdown', array($this, 'processActions'));
-    }
-
-
-
-    /**
-     * @return array
-     * @throws \DomainException
-     */
-    protected function getActiveActions()
-    {
-        Benchmark::startTimer(__METHOD__);
-        global $wpdb;
-        $results = $wpdb->get_results(
-            "SELECT * FROM {$wpdb->prefix}esp_automated_action WHERE AMA_is_active = 1;"
-        );
-        if ($results instanceof \WP_Error) {
-            throw new \DomainException(
-                $results->get_error_message()
-            );
-        }
-        $results = is_array($results) ? $results : array($results);
-        Benchmark::stopTimer(__METHOD__);
-        return $results;
-    }
-
-
-
-    /**
-     * @param array $results
-     * @throws InvalidEntityException
-     * @throws \DomainException
-     */
-    protected function generateActionObjects(array $results)
-    {
-        Benchmark::startTimer(__METHOD__);
-        foreach ($results as $result) {
-            $this->automated_actions->add(
-                AutomatedActionFactory::create($result),
-                $result->AMA_name
-            );
-        }
-        Benchmark::stopTimer(__METHOD__);
-    }
-
-
-
-    /**
-     * @throws InsufficientPermissionsException
-     * @throws InvalidClassException
-     */
-    public function setTriggers()
-    {
-        \EEH_Debug_Tools::printr(__FUNCTION__, __CLASS__, __FILE__, __LINE__, 2);
-        Benchmark::startTimer(__METHOD__);
-        foreach ($this->automated_actions as $automated_action) {
-            /** @var AutomatedActionInterface $automated_action */
-            if ($automated_action instanceof RequiresCapCheckInterface) {
-                $this->capabilities_checker->processCapCheck(
-                    $automated_action->getCapCheck()
-                );
-            }
-            $automated_action->setTrigger();
-        }
-        Benchmark::stopTimer(__METHOD__);
-    }
-
-
-
-    /**
-     *
-     */
-    public function processActions()
-    {
-        \EEH_Debug_Tools::printr('', 'SHUTDOWN HOOK', __FILE__, __LINE__, 2);
-        \EEH_Debug_Tools::printr(__FUNCTION__, __CLASS__, __FILE__, __LINE__, 2);
-        Benchmark::startTimer(__METHOD__);
-        foreach ($this->automated_actions as $automated_action) {
-            /** @var AutomatedActionInterface $automated_action */
-            if ( ! $automated_action->triggerPulled() || $automated_action->hasRun()) {
-                continue;
-            }
-            $automated_action->process();
-        }
-        Benchmark::stopTimer(__METHOD__);
-        Benchmark::displayResults();
-    }
+	/**
+	 * @var Collection $automated_actions
+	 */
+	private $automated_actions;
+
+	/**
+	 * @type CapabilitiesChecker $capabilities_checker
+	 */
+	private $capabilities_checker;
+
+
+
+	/**
+	 * AutomatedActionManager constructor
+	 *
+	 * @param CapabilitiesChecker $capabilities_checker
+	 * @param Collection          $automated_actions
+	 * @throws InvalidInterfaceException
+	 */
+	public function __construct(CapabilitiesChecker $capabilities_checker, Collection $automated_actions = null)
+	{
+		Benchmark::startTimer(__METHOD__);
+		$this->capabilities_checker = $capabilities_checker;
+		$this->automated_actions    = $automated_actions instanceof Collection
+			? $automated_actions
+			: new Collection('EventEspresso\core\services\automated_actions\AutomatedActionInterface');
+		add_action('AHEE__EE_System__initialize', array($this, 'initialize'));
+		Benchmark::stopTimer(__METHOD__);
+	}
+
+
+
+	/**
+	 * @throws \DomainException
+	 * @throws InvalidEntityException
+	 * @throws InsufficientPermissionsException
+	 * @throws InvalidClassException
+	 */
+	public function initialize()
+	{
+		$results = $this->getActiveActions();
+		$this->generateActionObjects($results);
+		$this->setTriggers();
+		add_action('shutdown', array($this, 'processActions'));
+	}
+
+
+
+	/**
+	 * @return array
+	 * @throws \DomainException
+	 */
+	protected function getActiveActions()
+	{
+		Benchmark::startTimer(__METHOD__);
+		global $wpdb;
+		$results = $wpdb->get_results(
+			"SELECT * FROM {$wpdb->prefix}esp_automated_action WHERE AMA_is_active = 1;"
+		);
+		if ($results instanceof \WP_Error) {
+			throw new \DomainException(
+				$results->get_error_message()
+			);
+		}
+		$results = is_array($results) ? $results : array($results);
+		Benchmark::stopTimer(__METHOD__);
+		return $results;
+	}
+
+
+
+	/**
+	 * @param array $results
+	 * @throws InvalidEntityException
+	 * @throws \DomainException
+	 */
+	protected function generateActionObjects(array $results)
+	{
+		Benchmark::startTimer(__METHOD__);
+		foreach ($results as $result) {
+			$this->automated_actions->add(
+				AutomatedActionFactory::create($result),
+				$result->AMA_name
+			);
+		}
+		Benchmark::stopTimer(__METHOD__);
+	}
+
+
+
+	/**
+	 * @throws InsufficientPermissionsException
+	 * @throws InvalidClassException
+	 */
+	public function setTriggers()
+	{
+		\EEH_Debug_Tools::printr(__FUNCTION__, __CLASS__, __FILE__, __LINE__, 2);
+		Benchmark::startTimer(__METHOD__);
+		foreach ($this->automated_actions as $automated_action) {
+			/** @var AutomatedActionInterface $automated_action */
+			if ($automated_action instanceof RequiresCapCheckInterface) {
+				$this->capabilities_checker->processCapCheck(
+					$automated_action->getCapCheck()
+				);
+			}
+			$automated_action->setTrigger();
+		}
+		Benchmark::stopTimer(__METHOD__);
+	}
+
+
+
+	/**
+	 *
+	 */
+	public function processActions()
+	{
+		\EEH_Debug_Tools::printr('', 'SHUTDOWN HOOK', __FILE__, __LINE__, 2);
+		\EEH_Debug_Tools::printr(__FUNCTION__, __CLASS__, __FILE__, __LINE__, 2);
+		Benchmark::startTimer(__METHOD__);
+		foreach ($this->automated_actions as $automated_action) {
+			/** @var AutomatedActionInterface $automated_action */
+			if ( ! $automated_action->triggerPulled() || $automated_action->hasRun()) {
+				continue;
+			}
+			$automated_action->process();
+		}
+		Benchmark::stopTimer(__METHOD__);
+		Benchmark::displayResults();
+	}
 
 
 }

core/services/automated_actions/AutomatedAction.php

Indentation +201 added lines, -201 removed lines

@@ -18,268 +18,268 @@
 class AutomatedAction implements AutomatedActionInterface
 {
 
-    /**
-     * @var int $ID
-     */
-    protected $ID;
+	/**
+	 * @var int $ID
+	 */
+	protected $ID;
 
-    /**
-     * @var string $name
-     */
-    protected $name;
+	/**
+	 * @var string $name
+	 */
+	protected $name;
 
-    /**
-     * @var string $description
-     */
-    protected $description;
+	/**
+	 * @var string $description
+	 */
+	protected $description;
 
-    /**
-     * @var AutomatedActionStrategy $strategy
-     */
-    protected $strategy;
+	/**
+	 * @var AutomatedActionStrategy $strategy
+	 */
+	protected $strategy;
 
 
-    /**
-     * @var TriggerStrategy $trigger
-     */
-    protected $trigger;
+	/**
+	 * @var TriggerStrategy $trigger
+	 */
+	protected $trigger;
 
-    /**
-     * @var string $trigger_value
-     */
-    protected $trigger_value;
+	/**
+	 * @var string $trigger_value
+	 */
+	protected $trigger_value;
 
-    /**
-     * @var string $data
-     */
-    protected $data;
+	/**
+	 * @var string $data
+	 */
+	protected $data;
 
 
-    /*
+	/*
      * @var CapCheckInterface $cap_check
      */
-    protected $cap_check;
+	protected $cap_check;
 
 
 
 
-    /**
-     * AutomatedAction constructor.
-     *
-     * @param \stdClass               $args
-     * @param AutomatedActionStrategy $strategy
-     * @param TriggerStrategy         $trigger
-     * @throws \DomainException
-     */
-    public function __construct(\stdClass $args, AutomatedActionStrategy $strategy, TriggerStrategy $trigger)
-    {
-        $this->setID($args);
-        $this->setName($args);
-        $this->setDescription($args);
-        $this->strategy = $strategy;
-        $this->trigger = $trigger;
-        $this->setTriggerValue($args);
-        $this->setData($args);
-    }
+	/**
+	 * AutomatedAction constructor.
+	 *
+	 * @param \stdClass               $args
+	 * @param AutomatedActionStrategy $strategy
+	 * @param TriggerStrategy         $trigger
+	 * @throws \DomainException
+	 */
+	public function __construct(\stdClass $args, AutomatedActionStrategy $strategy, TriggerStrategy $trigger)
+	{
+		$this->setID($args);
+		$this->setName($args);
+		$this->setDescription($args);
+		$this->strategy = $strategy;
+		$this->trigger = $trigger;
+		$this->setTriggerValue($args);
+		$this->setData($args);
+	}
 
 
 
-    /**
-     * @return int
-     */
-    public function getID()
-    {
-        return $this->ID;
-    }
+	/**
+	 * @return int
+	 */
+	public function getID()
+	{
+		return $this->ID;
+	}
 
 
 
-    /**
-     * @param \stdClass $args
-     * @throws \DomainException
-     */
-    protected function setID(\stdClass $args)
-    {
-        $this->ID = isset($args->AMA_ID) ? absint($args->AMA_ID) : 0;
-        if (! $this->ID){
-            throw new \DomainException();
-        }
+	/**
+	 * @param \stdClass $args
+	 * @throws \DomainException
+	 */
+	protected function setID(\stdClass $args)
+	{
+		$this->ID = isset($args->AMA_ID) ? absint($args->AMA_ID) : 0;
+		if (! $this->ID){
+			throw new \DomainException();
+		}
 
-    }
+	}
 
 
 
-    /**
-     * @return string
-     */
-    public function getName()
-    {
-        return $this->name;
-    }
+	/**
+	 * @return string
+	 */
+	public function getName()
+	{
+		return $this->name;
+	}
 
 
 
-    /**
-     * @param \stdClass $args
-     * @throws \DomainException
-     */
-    protected function setName(\stdClass $args)
-    {
-        $this->name = isset($args->AMA_name) ? sanitize_text_field($args->AMA_name) : '';
-        if (empty($this->name)) {
-            throw new \DomainException();
-        }
-    }
+	/**
+	 * @param \stdClass $args
+	 * @throws \DomainException
+	 */
+	protected function setName(\stdClass $args)
+	{
+		$this->name = isset($args->AMA_name) ? sanitize_text_field($args->AMA_name) : '';
+		if (empty($this->name)) {
+			throw new \DomainException();
+		}
+	}
 
 
 
-    /**
-     * @return string
-     */
-    public function getDescription()
-    {
-        return $this->description;
-    }
+	/**
+	 * @return string
+	 */
+	public function getDescription()
+	{
+		return $this->description;
+	}
 
 
 
-    /**
-     * @param \stdClass $args
-     */
-    protected function setDescription(\stdClass $args)
-    {
-        $this->description = isset($args->AMA_description) ? sanitize_text_field($args->AMA_description) : '';
-    }
+	/**
+	 * @param \stdClass $args
+	 */
+	protected function setDescription(\stdClass $args)
+	{
+		$this->description = isset($args->AMA_description) ? sanitize_text_field($args->AMA_description) : '';
+	}
 
 
 
-    /**
-     * @return AutomatedActionStrategy
-     */
-    public function getStrategy()
-    {
-        return $this->strategy;
-    }
+	/**
+	 * @return AutomatedActionStrategy
+	 */
+	public function getStrategy()
+	{
+		return $this->strategy;
+	}
 
 
 
 
-    /**
-     * @return TriggerStrategy
-     */
-    public function getTrigger()
-    {
-        return $this->trigger;
-    }
+	/**
+	 * @return TriggerStrategy
+	 */
+	public function getTrigger()
+	{
+		return $this->trigger;
+	}
 
 
 
-    /**
-     * @return string
-     */
-    public function getTriggerValue()
-    {
-        return $this->trigger_value;
-    }
-
+	/**
+	 * @return string
+	 */
+	public function getTriggerValue()
+	{
+		return $this->trigger_value;
+	}
 
-    /**
-     * @param \stdClass $args
-     */
-    protected function setTriggerValue(\stdClass $args)
-    {
-        $this->trigger_value = isset($args->AMA_trigger_value) ? sanitize_text_field($args->AMA_trigger_value) : '';
-    }
 
+	/**
+	 * @param \stdClass $args
+	 */
+	protected function setTriggerValue(\stdClass $args)
+	{
+		$this->trigger_value = isset($args->AMA_trigger_value) ? sanitize_text_field($args->AMA_trigger_value) : '';
+	}
 
 
-    /**
-     * @return string
-     */
-    public function getData()
-    {
-        return $this->data;
-    }
 
+	/**
+	 * @return string
+	 */
+	public function getData()
+	{
+		return $this->data;
+	}
 
 
-    /**
-     * @param \stdClass $args
-     * @throws \DomainException
-     */
-    protected function setData(\stdClass $args)
-    {
-        $this->data = isset($args->AMA_data) ? sanitize_text_field($args->AMA_data) : '';
-        if ($this->data !== null) {
-            $this->data = json_decode($this->data);
-            if (json_last_error() !== JSON_ERROR_NONE) {
-                throw new \DomainException(json_last_error_msg());
-            }
-        }
-    }
 
+	/**
+	 * @param \stdClass $args
+	 * @throws \DomainException
+	 */
+	protected function setData(\stdClass $args)
+	{
+		$this->data = isset($args->AMA_data) ? sanitize_text_field($args->AMA_data) : '';
+		if ($this->data !== null) {
+			$this->data = json_decode($this->data);
+			if (json_last_error() !== JSON_ERROR_NONE) {
+				throw new \DomainException(json_last_error_msg());
+			}
+		}
+	}
 
 
-    /**
-     * @param CapCheckInterface $cap_check
-     */
-    public function setCapCheck(CapCheckInterface $cap_check)
-    {
-        $this->cap_check = $cap_check;
-    }
+
+	/**
+	 * @param CapCheckInterface $cap_check
+	 */
+	public function setCapCheck(CapCheckInterface $cap_check)
+	{
+		$this->cap_check = $cap_check;
+	}
 
 
 
-    /**
-     * not to be confused with being a setter for the $trigger property,
-     * but instead informs the trigger to set hooks or similar initialization
-     *
-     * @return void
-     */
-    public function setTrigger()
-    {
-        $this->trigger->set($this);
-    }
+	/**
+	 * not to be confused with being a setter for the $trigger property,
+	 * but instead informs the trigger to set hooks or similar initialization
+	 *
+	 * @return void
+	 */
+	public function setTrigger()
+	{
+		$this->trigger->set($this);
+	}
 
 
 
-    /**
-     * @return bool
-     */
-    public function triggerPulled()
-    {
-        // \EEH_Debug_Tools::printr($this->trigger->pulled(), '$this->trigger->pulled()', __FILE__, __LINE__);
-        return $this->trigger->pulled();
-    }
+	/**
+	 * @return bool
+	 */
+	public function triggerPulled()
+	{
+		// \EEH_Debug_Tools::printr($this->trigger->pulled(), '$this->trigger->pulled()', __FILE__, __LINE__);
+		return $this->trigger->pulled();
+	}
 
-
-
-    /**
-     * @return bool
-     */
-    public function hasRun()
-    {
-        // \EEH_Debug_Tools::printr($this->strategy->hasRun(), '$this->strategy->hasRun()', __FILE__, __LINE__);
-        return $this->strategy->hasRun();
-    }
-
-
-
-    /**
-     * by default this is called during the "shutdown"
-     * in AutomatedActionManager::processActions(),
-     * but individual AutomatedActionStrategy classes
-     * can run their callback() method whenever it is best,
-     * as long as they call setHasRun() to avoid duplication
-     */
-    public function process()
-    {
-        \EEH_Debug_Tools::printr(__FUNCTION__, __CLASS__, __FILE__, __LINE__, 2);
-        // \EEH_Debug_Tools::printr($this->description, $this->name, __FILE__, __LINE__);
-        $this->strategy->callback(
-            $this->trigger->getCallbackArgs()
-        );
-    }
+
+
+	/**
+	 * @return bool
+	 */
+	public function hasRun()
+	{
+		// \EEH_Debug_Tools::printr($this->strategy->hasRun(), '$this->strategy->hasRun()', __FILE__, __LINE__);
+		return $this->strategy->hasRun();
+	}
+
+
+
+	/**
+	 * by default this is called during the "shutdown"
+	 * in AutomatedActionManager::processActions(),
+	 * but individual AutomatedActionStrategy classes
+	 * can run their callback() method whenever it is best,
+	 * as long as they call setHasRun() to avoid duplication
+	 */
+	public function process()
+	{
+		\EEH_Debug_Tools::printr(__FUNCTION__, __CLASS__, __FILE__, __LINE__, 2);
+		// \EEH_Debug_Tools::printr($this->description, $this->name, __FILE__, __LINE__);
+		$this->strategy->callback(
+			$this->trigger->getCallbackArgs()
+		);
+	}
 
 }
 // End of file AutomatedAction.php

core/services/automated_actions/TriggerStrategy.php

Indentation +202 added lines, -202 removed lines

@@ -23,221 +23,221 @@
 abstract class TriggerStrategy
 {
 
-    /**
-     * @var RuleManager $rule_manager
-     */
-    private $rule_manager;
+	/**
+	 * @var RuleManager $rule_manager
+	 */
+	private $rule_manager;
 
-    /**
-     * @var AutomatedActionInterface $automated_action
-     */
-    private $automated_action;
+	/**
+	 * @var AutomatedActionInterface $automated_action
+	 */
+	private $automated_action;
 
-    /**
-     * @var array $callback_args
-     */
-    private $callback_args = array();
+	/**
+	 * @var array $callback_args
+	 */
+	private $callback_args = array();
 
-    /**
-     * @var boolean $pulled
-     */
-    private $pulled = false;
+	/**
+	 * @var boolean $pulled
+	 */
+	private $pulled = false;
 
-    /**
-     * @var Collection $rules
-     */
-    private $rules;
+	/**
+	 * @var Collection $rules
+	 */
+	private $rules;
 
 
 
-    /**
-     * TriggerStrategy constructor.
-     *
-     * @param RuleManager $rule_manager
-     */
-    public function __construct(RuleManager $rule_manager)
-    {
-        $this->rule_manager = $rule_manager;
-    }
+	/**
+	 * TriggerStrategy constructor.
+	 *
+	 * @param RuleManager $rule_manager
+	 */
+	public function __construct(RuleManager $rule_manager)
+	{
+		$this->rule_manager = $rule_manager;
+	}
 
 
 
-    /**
-     * @return AutomatedActionInterface
-     */
-    public function getAutomatedAction()
-    {
-        return $this->automated_action;
-    }
+	/**
+	 * @return AutomatedActionInterface
+	 */
+	public function getAutomatedAction()
+	{
+		return $this->automated_action;
+	}
 
 
 
-    /**
-     * @param \EventEspresso\core\services\automated_actions\AutomatedActionInterface $automated_action
-     */
-    public function setAutomatedAction(AutomatedActionInterface $automated_action) {
-        $this->automated_action = $automated_action;
-    }
-
-
-
-
-    /**
-     * @return array
-     */
-    public function getCallbackArgs()
-    {
-        return $this->callback_args;
-    }
-
-
-
-    /**
-     * @param array $callback_args
-     */
-    public function setCallbackArgs(array $callback_args = array())
-    {
-        $this->callback_args = $callback_args;
-    }
-
-
-
-    /**
-     * @return bool
-     */
-    public function pulled()
-    {
-        return $this->pulled;
-    }
-
-
-
-    /**
-     * @param bool $pulled
-     */
-    public function setPulled($pulled = true)
-    {
-        $this->pulled = filter_var($pulled, FILTER_VALIDATE_BOOLEAN);
-    }
-
-
-
-    /**
-     * @return Collection
-     * @throws InvalidArgumentException
-     * @throws InvalidInterfaceException
-     * @throws InvalidEntityException
-     * @throws DomainException
-     */
-    public function getRules()
-    {
-        if ($this->rules === null) {
-            $this->setRules();
-        }
-        return $this->rules;
-    }
-
-
-
-    /**
-     * @return void
-     * @throws InvalidArgumentException
-     * @throws InvalidInterfaceException
-     * @throws InvalidEntityException
-     * @throws DomainException
-     */
-    private function setRules()
-    {
-        $this->rules = $this->rule_manager->retrieveRulesForObject(
-            'Automated Action',
-            $this->automated_action->getID()
-        );
-    }
-
-
-
-    /**
-     * This is required and should be overridden by all non-cron type triggers,
-     * as it allows strategies to set up whatever hooks are necessary to trigger their logic.
-     * Cron related strategies do not need to implement this method
-     * as this one will be used and do nothing but capture the incoming data
-     *
-     * @param AutomatedActionInterface $automated_action
-     */
-    public function set(AutomatedActionInterface $automated_action)
-    {
-        $this->automated_action = $automated_action;
-    }
-
-
-
-    /**
-     * callback method for all strategies (regardless of type),
-     * that runs when the trigger is "pulled".
-     * At this point strategies should ONLY proceed to collect data
-     * required for the action, store it for later processing,
-     * and call setPulled() to indicate that the trigger was successful
-     * (meaning data results for the action were retrieved).
-     * This allows control over when that happens
-     * because the moment the trigger is pulled
-     * may not be the most optimum time for data processing.
-     * By default, logic processing will occur during "shutdown",
-     * but can be overridden by an AutomatedActionStrategy class
-     *
-     * @throws InvalidArgumentException
-     * @throws InvalidInterfaceException
-     * @throws InvalidEntityException
-     * @throws DomainException
-     */
-    public function triggerCallback() {
-        \EEH_Debug_Tools::printr(__FUNCTION__, __CLASS__, __FILE__, __LINE__, 2);
-        $this->setCallbackArgs(func_get_args());
-        $this->execute(
-            $this->getQueryParamsForRules()
-        );
-    }
-
-
-
-    /**
-     * @return array
-     * @throws InvalidArgumentException
-     * @throws InvalidInterfaceException
-     * @throws InvalidEntityException
-     * @throws DomainException
-     */
-    public function getQueryParamsForRules()
-    {
-        return $this->rule_manager->getQueryParamsForRules(
-            $this->getRules()
-        );
-    }
-
-
-
-    /**
-     * when a trigger has been "pulled",
-     * either by a do_action or scheduled cron,
-     * this is the TriggerStrategy method that will run
-     * to continue processing the trigger.
-     * This will typically mean retrieving objects from the db.
-     * By default, this method does nothing but set callback args
-     * to be used for querying during the 'shutdown' hook when
-     * AutomatedActionManager::processActions() runs,
-     * but can be overridden to perform querying now.
-     *
-     * @param array $query_params
-     * @return void
-     */
-    public function execute(array $query_params)
-    {
-        \EEH_Debug_Tools::printr(__FUNCTION__, __CLASS__, __FILE__, __LINE__, 2);
-        $this->setCallbackArgs(
-            array_merge($this->callback_args, array('query_params' => $query_params))
-        );
-        // \EEH_Debug_Tools::printr($this->callback_args, '$this->callback_args', __FILE__, __LINE__);
-        $this->setPulled();
-    }
+	/**
+	 * @param \EventEspresso\core\services\automated_actions\AutomatedActionInterface $automated_action
+	 */
+	public function setAutomatedAction(AutomatedActionInterface $automated_action) {
+		$this->automated_action = $automated_action;
+	}
+
+
+
+
+	/**
+	 * @return array
+	 */
+	public function getCallbackArgs()
+	{
+		return $this->callback_args;
+	}
+
+
+
+	/**
+	 * @param array $callback_args
+	 */
+	public function setCallbackArgs(array $callback_args = array())
+	{
+		$this->callback_args = $callback_args;
+	}
+
+
+
+	/**
+	 * @return bool
+	 */
+	public function pulled()
+	{
+		return $this->pulled;
+	}
+
+
+
+	/**
+	 * @param bool $pulled
+	 */
+	public function setPulled($pulled = true)
+	{
+		$this->pulled = filter_var($pulled, FILTER_VALIDATE_BOOLEAN);
+	}
+
+
+
+	/**
+	 * @return Collection
+	 * @throws InvalidArgumentException
+	 * @throws InvalidInterfaceException
+	 * @throws InvalidEntityException
+	 * @throws DomainException
+	 */
+	public function getRules()
+	{
+		if ($this->rules === null) {
+			$this->setRules();
+		}
+		return $this->rules;
+	}
+
+
+
+	/**
+	 * @return void
+	 * @throws InvalidArgumentException
+	 * @throws InvalidInterfaceException
+	 * @throws InvalidEntityException
+	 * @throws DomainException
+	 */
+	private function setRules()
+	{
+		$this->rules = $this->rule_manager->retrieveRulesForObject(
+			'Automated Action',
+			$this->automated_action->getID()
+		);
+	}
+
+
+
+	/**
+	 * This is required and should be overridden by all non-cron type triggers,
+	 * as it allows strategies to set up whatever hooks are necessary to trigger their logic.
+	 * Cron related strategies do not need to implement this method
+	 * as this one will be used and do nothing but capture the incoming data
+	 *
+	 * @param AutomatedActionInterface $automated_action
+	 */
+	public function set(AutomatedActionInterface $automated_action)
+	{
+		$this->automated_action = $automated_action;
+	}
+
+
+
+	/**
+	 * callback method for all strategies (regardless of type),
+	 * that runs when the trigger is "pulled".
+	 * At this point strategies should ONLY proceed to collect data
+	 * required for the action, store it for later processing,
+	 * and call setPulled() to indicate that the trigger was successful
+	 * (meaning data results for the action were retrieved).
+	 * This allows control over when that happens
+	 * because the moment the trigger is pulled
+	 * may not be the most optimum time for data processing.
+	 * By default, logic processing will occur during "shutdown",
+	 * but can be overridden by an AutomatedActionStrategy class
+	 *
+	 * @throws InvalidArgumentException
+	 * @throws InvalidInterfaceException
+	 * @throws InvalidEntityException
+	 * @throws DomainException
+	 */
+	public function triggerCallback() {
+		\EEH_Debug_Tools::printr(__FUNCTION__, __CLASS__, __FILE__, __LINE__, 2);
+		$this->setCallbackArgs(func_get_args());
+		$this->execute(
+			$this->getQueryParamsForRules()
+		);
+	}
+
+
+
+	/**
+	 * @return array
+	 * @throws InvalidArgumentException
+	 * @throws InvalidInterfaceException
+	 * @throws InvalidEntityException
+	 * @throws DomainException
+	 */
+	public function getQueryParamsForRules()
+	{
+		return $this->rule_manager->getQueryParamsForRules(
+			$this->getRules()
+		);
+	}
+
+
+
+	/**
+	 * when a trigger has been "pulled",
+	 * either by a do_action or scheduled cron,
+	 * this is the TriggerStrategy method that will run
+	 * to continue processing the trigger.
+	 * This will typically mean retrieving objects from the db.
+	 * By default, this method does nothing but set callback args
+	 * to be used for querying during the 'shutdown' hook when
+	 * AutomatedActionManager::processActions() runs,
+	 * but can be overridden to perform querying now.
+	 *
+	 * @param array $query_params
+	 * @return void
+	 */
+	public function execute(array $query_params)
+	{
+		\EEH_Debug_Tools::printr(__FUNCTION__, __CLASS__, __FILE__, __LINE__, 2);
+		$this->setCallbackArgs(
+			array_merge($this->callback_args, array('query_params' => $query_params))
+		);
+		// \EEH_Debug_Tools::printr($this->callback_args, '$this->callback_args', __FILE__, __LINE__);
+		$this->setPulled();
+	}
 
 
 }

core/services/conditional_logic/rules/RuleManager.php

Indentation +123 added lines, -123 removed lines

@@ -23,134 +23,134 @@
 class RuleManager
 {
 
-    const RULE_TYPE_QUERY = 'query';
-    const RULE_TYPE_OBJECT = 'object';
-
-    /**
-     * @var QueryParamGenerator $query_generator
-     */
-    private $query_generator;
-
-
-
-    /**
-     * RuleManager constructor
-     *
-     * @param QueryParamGenerator $query_generator
-     * @throws InvalidInterfaceException
-     */
-    public function __construct(QueryParamGenerator $query_generator)
-    {
-        $this->query_generator = $query_generator;
-    }
-
-
-
-    /**
-     * @param \stdClass[] $results
-     * @return Collection
-     * @throws InvalidInterfaceException
-     * @throws InvalidArgumentException
-     * @throws InvalidEntityException
-     */
-    protected function getRulesCollection(array $results)
-    {
-        Benchmark::startTimer(__METHOD__);
-        $rules = new Collection('EventEspresso\core\services\conditional_logic\rules\Rule');
-        foreach ($results as $result) {
-            $rules->add(new Rule($result));
-        }
-        Benchmark::stopTimer(__METHOD__);
-        return $rules;
-    }
-
-
-
-    /**
-     * @param string $OBJ_name
-     * @param mixed  $OBJ_ID
-     * @return array
-     * @throws DomainException
-     */
-    public function retrieveRules($OBJ_name, $OBJ_ID)
-    {
-        Benchmark::startTimer(__METHOD__);
-        global $wpdb;
-        $results = $wpdb->get_results(
-            $wpdb->prepare(
-                "SELECT * FROM {$wpdb->prefix}esp_object_rule AS object_rule
+	const RULE_TYPE_QUERY = 'query';
+	const RULE_TYPE_OBJECT = 'object';
+
+	/**
+	 * @var QueryParamGenerator $query_generator
+	 */
+	private $query_generator;
+
+
+
+	/**
+	 * RuleManager constructor
+	 *
+	 * @param QueryParamGenerator $query_generator
+	 * @throws InvalidInterfaceException
+	 */
+	public function __construct(QueryParamGenerator $query_generator)
+	{
+		$this->query_generator = $query_generator;
+	}
+
+
+
+	/**
+	 * @param \stdClass[] $results
+	 * @return Collection
+	 * @throws InvalidInterfaceException
+	 * @throws InvalidArgumentException
+	 * @throws InvalidEntityException
+	 */
+	protected function getRulesCollection(array $results)
+	{
+		Benchmark::startTimer(__METHOD__);
+		$rules = new Collection('EventEspresso\core\services\conditional_logic\rules\Rule');
+		foreach ($results as $result) {
+			$rules->add(new Rule($result));
+		}
+		Benchmark::stopTimer(__METHOD__);
+		return $rules;
+	}
+
+
+
+	/**
+	 * @param string $OBJ_name
+	 * @param mixed  $OBJ_ID
+	 * @return array
+	 * @throws DomainException
+	 */
+	public function retrieveRules($OBJ_name, $OBJ_ID)
+	{
+		Benchmark::startTimer(__METHOD__);
+		global $wpdb;
+		$results = $wpdb->get_results(
+			$wpdb->prepare(
+				"SELECT * FROM {$wpdb->prefix}esp_object_rule AS object_rule
                  LEFT JOIN {$wpdb->prefix}esp_rule AS rule
                  ON object_rule.RUL_ID = rule.RUL_ID
                  WHERE object_rule.ORL_OBJ_name = %s
                  AND object_rule.ORL_OBJ_ID = %d
                  ORDER BY object_rule.ORL_order",
-                $OBJ_name,
-                $OBJ_ID
-            )
-        );
-        // \EEH_Debug_Tools::printr($results, '$results', __FILE__, __LINE__);
-        if ($results instanceof \WP_Error) {
-            throw new DomainException(
-                $results->get_error_message()
-            );
-        }
-        $results = is_array($results) ? $results : array($results);
-        Benchmark::stopTimer(__METHOD__);
-        return $results;
-    }
-
-
-
-    /**
-     * @param string $OBJ_name
-     * @param mixed  $OBJ_ID
-     * @return Collection
-     * @throws InvalidArgumentException
-     * @throws InvalidInterfaceException
-     * @throws InvalidEntityException
-     * @throws DomainException
-     */
-    public function retrieveRulesForObject( $OBJ_name = '', $OBJ_ID )
-    {
-        return $this->getRulesCollection(
-            $this->retrieveRules($OBJ_name, $OBJ_ID)
-        );
-    }
-
-
-
-    /**
-     * @param string $OBJ_name
-     * @param mixed  $OBJ_ID
-     * @return array
-     * @throws InvalidArgumentException
-     * @throws InvalidInterfaceException
-     * @throws InvalidEntityException
-     * @throws DomainException
-     */
-    public function getQueryParamsForObjectRules( $OBJ_name = '', $OBJ_ID )
-    {
-        \EEH_Debug_Tools::printr(__FUNCTION__, __CLASS__, __FILE__, __LINE__, 2);
-        return $this->getQueryParamsForRules(
-            $this->getRulesCollection(
-                $this->retrieveRules($OBJ_name, $OBJ_ID)
-            )
-        );
-    }
-
-
-
-    /**
-     * @param Collection $rules
-     * @return array
-     */
-    public function getQueryParamsForRules(Collection $rules)
-    {
-        $this->query_generator->addRules($rules);
-        $SQL = $this->query_generator->generateQueryParams(RuleManager::RULE_TYPE_QUERY);
-        // \EEH_Debug_Tools::printr($SQL, '$SQL', __FILE__, __LINE__);
-        return $SQL;
-    }
+				$OBJ_name,
+				$OBJ_ID
+			)
+		);
+		// \EEH_Debug_Tools::printr($results, '$results', __FILE__, __LINE__);
+		if ($results instanceof \WP_Error) {
+			throw new DomainException(
+				$results->get_error_message()
+			);
+		}
+		$results = is_array($results) ? $results : array($results);
+		Benchmark::stopTimer(__METHOD__);
+		return $results;
+	}
+
+
+
+	/**
+	 * @param string $OBJ_name
+	 * @param mixed  $OBJ_ID
+	 * @return Collection
+	 * @throws InvalidArgumentException
+	 * @throws InvalidInterfaceException
+	 * @throws InvalidEntityException
+	 * @throws DomainException
+	 */
+	public function retrieveRulesForObject( $OBJ_name = '', $OBJ_ID )
+	{
+		return $this->getRulesCollection(
+			$this->retrieveRules($OBJ_name, $OBJ_ID)
+		);
+	}
+
+
+
+	/**
+	 * @param string $OBJ_name
+	 * @param mixed  $OBJ_ID
+	 * @return array
+	 * @throws InvalidArgumentException
+	 * @throws InvalidInterfaceException
+	 * @throws InvalidEntityException
+	 * @throws DomainException
+	 */
+	public function getQueryParamsForObjectRules( $OBJ_name = '', $OBJ_ID )
+	{
+		\EEH_Debug_Tools::printr(__FUNCTION__, __CLASS__, __FILE__, __LINE__, 2);
+		return $this->getQueryParamsForRules(
+			$this->getRulesCollection(
+				$this->retrieveRules($OBJ_name, $OBJ_ID)
+			)
+		);
+	}
+
+
+
+	/**
+	 * @param Collection $rules
+	 * @return array
+	 */
+	public function getQueryParamsForRules(Collection $rules)
+	{
+		$this->query_generator->addRules($rules);
+		$SQL = $this->query_generator->generateQueryParams(RuleManager::RULE_TYPE_QUERY);
+		// \EEH_Debug_Tools::printr($SQL, '$SQL', __FILE__, __LINE__);
+		return $SQL;
+	}
 
 
 }

core/services/conditional_logic/rules/RuleStrategy.php

Indentation +29 added lines, -29 removed lines

@@ -8,35 +8,35 @@
 abstract class RuleStrategy
 {
 
-    /**
-     * @var string $target
-     */
-    protected $target = '';
-
-    /**
-     * @var string $comparison
-     */
-    protected $comparison = '';
-
-    /**
-     * @var string $value
-     */
-    protected $value = '';
-
-
-    /**
-     * @param \EventEspresso\core\services\conditional_logic\rules\Rule $rule
-     * @return mixed
-     */
-    public function getQueryParamForRule(Rule $rule) {
-        $this->comparison = $rule->getComparison();
-        $this->value      = $rule->getValue();
-        $target           = $rule->getTarget();
-        // \EEH_Debug_Tools::printr($target, '$target', __FILE__, __LINE__);
-        return method_exists($this, $target)
-            ? $this->{$target}()
-            : "{$target} {$this->comparison} {$this->value}";
-    }
+	/**
+	 * @var string $target
+	 */
+	protected $target = '';
+
+	/**
+	 * @var string $comparison
+	 */
+	protected $comparison = '';
+
+	/**
+	 * @var string $value
+	 */
+	protected $value = '';
+
+
+	/**
+	 * @param \EventEspresso\core\services\conditional_logic\rules\Rule $rule
+	 * @return mixed
+	 */
+	public function getQueryParamForRule(Rule $rule) {
+		$this->comparison = $rule->getComparison();
+		$this->value      = $rule->getValue();
+		$target           = $rule->getTarget();
+		// \EEH_Debug_Tools::printr($target, '$target', __FILE__, __LINE__);
+		return method_exists($this, $target)
+			? $this->{$target}()
+			: "{$target} {$this->comparison} {$this->value}";
+	}
 
 }
 // End of file RuleStrategy.php

core/services/conditional_logic/rules/QueryParamGenerator.php

Indentation +178 added lines, -178 removed lines

@@ -18,184 +18,184 @@
 class QueryParamGenerator
 {
 
-    /**
-     * @var Collection $rules
-     */
-    private $rules;
-
-    // /**
-    //  * @var array $query_params
-    //  */
-    // private $query_params;
-
-
-
-    public function addRules(Collection $rules)
-    {
-        $this->rules = $rules;
-    }
-
-
-
-    public function resetRules()
-    {
-        $this->rules = null;
-    }
-
-
-
-    public function generateQueryParams($type = '')
-    {
-        \EEH_Debug_Tools::printr(__FUNCTION__, __CLASS__, __FILE__, __LINE__, 2);
-        $rules = $this->getRulesAsArray($type);
-        // \EEH_Debug_Tools::printr($rules, 'Rules As Array', __FILE__, __LINE__);
-        $query_params = $this->addQueryParamForRule($rules, array());
-        // \EEH_Debug_Tools::printr($query_params, '>>> $QUERY_PARAMS', __FILE__, __LINE__);
-        return $query_params;
-    }
-
-
-
-    private function getRulesAsArray($type = '')
-    {
-        // \EEH_Debug_Tools::printr(__FUNCTION__, __CLASS__, __FILE__, __LINE__, 2);
-        $rules_array = array();
-        foreach ($this->rules as $rule) {
-            /** @var Rule $rule */
-            if ($type && $rule->getType() !== $type){
-                continue;
-            }
-            $rules_array = $this->addRuleToArray($rules_array, $rule);
-        }
-        return $rules_array;
-    }
-
-
-
-    private function addRuleToArray(array $rules_array, Rule $rule)
-    {
-        // static $level = 0;
-        // \EEH_Debug_Tools::printr(__FUNCTION__, $level . ') ' .__CLASS__, __FILE__, __LINE__, 2);
-        // \EEH_Debug_Tools::printr((string)$rule->getID(), 'ID', __FILE__, __LINE__);
-        $parent = $rule->getParent();
-        // \EEH_Debug_Tools::printr((string)$parent, 'Parent', __FILE__, __LINE__);
-        if ($rule->getOperator() === ')') {
-            // $level--;
-            // \EEH_Debug_Tools::printr('SUBGROUP', 'CLOSE', __FILE__, __LINE__);
-            return $rules_array;
-        } else if ( isset($rules_array[$parent])) {
-            // $level++;
-            // \EEH_Debug_Tools::printr('PARENT', 'FOUND', __FILE__, __LINE__);
-            $rules_array[$parent] = $this->addRuleToArray($rules_array[$parent], $rule);
-            return $rules_array;
-        } else if (! empty($rules_array)){
-            // \EEH_Debug_Tools::printr('SUB ARRAY', 'FOUND', __FILE__, __LINE__);
-            // \EEH_Debug_Tools::printr($rules_array, '$rules_array', __FILE__, __LINE__);
-            foreach($rules_array as $ID => $rules) {
-                // \EEH_Debug_Tools::printr($ID, '$rules_array $ID', __FILE__, __LINE__);
-                if($ID === $parent) {
-                    // \EEH_Debug_Tools::printr($ID === $parent, '$ID === $parent', __FILE__, __LINE__);
-                    $rules_array[$ID] = $this->addRuleToArray($rules[$ID], $rule);
-                    return $rules_array;
-                }
-            }
-        }
-        // \EEH_Debug_Tools::printr(
-        //     $rule->getOperator()
-        //     . ' '
-        //     . $rule->getTarget()
-        //     . ' '
-        //     . $rule->getComparison()
-        //     . ' '
-        //     . $rule->getValue(), 'ADD RULE',
-        //     __FILE__, __LINE__
-        // );
-        $rules_array[$rule->getID()] = array();
-        $rules_array[$rule->getID()]['rule'] = $rule;
-        return $rules_array;
-    }
-
-
-
-    public function addQueryParamForRule($rules, array $query_params, $operator = ' AND ')
-    {
-        // \EEH_Debug_Tools::printr(__FUNCTION__, __CLASS__, __FILE__, __LINE__, 2);
-        // \EEH_Debug_Tools::printr($rule, '$rule', __FILE__, __LINE__);
-        foreach ($rules as $key => $rule) {
-            // \EEH_Debug_Tools::printr((string)$key, '2) $key', __FILE__, __LINE__);
-            if (is_array($rule)) {
-                // \EEH_Debug_Tools::printr(is_array($rule), 'is_array($rule)', __FILE__, __LINE__);
-                $query_params = $this->addQueryParamForRule($rule, $query_params, $operator);
-            } else {
-                /** @var Rule $rule */
-                // \EEH_Debug_Tools::printr($rule, 'getQueryParamForRule', __FILE__, __LINE__);
-                $operator = $rule->getOperator();
-                $parent = $rule->getParent();
-                $parent = $parent ? $parent : 0;
-                if ( ! isset($query_params[$parent])) {
-                    $query_params[$parent] = array();
-                }
-                \EEH_Debug_Tools::printr($operator, '$operator', __FILE__, __LINE__);
-                \EEH_Debug_Tools::printr((string)$parent, '$parent', __FILE__, __LINE__);
-                switch ($operator) {
-                    default :
-                    case '  ';
-                    case ' AND ';
-                        $query_params[$parent] = array_merge(
-                            $query_params[$parent],
-                            $this->getQueryParamForRule($rule)
-                        );
-                        break;
-                    case ' OR ';
-                        $query_params[$parent] = array_merge(
-                            $query_params[$parent],
-                            array('OR' => $this->getQueryParamForRule($rule))
-                        );
-                        break;
-                    case ' AND ( ';
-                        $query_params[$parent] = array_merge(
-                            $query_params[$parent],
-                            array( $this->getQueryParamForRule($rule) )
-                        );
-                        break;
-                    case ' OR ( ';
-                        $query_params['OR'][$parent] = array_merge(
-                            $query_params['OR'][$parent],
-                            $this->getQueryParamForRule($rule)
-                        );
-                        break;
-                    case ' ) ';
-                        break;
-                }
-            }
-        }
-        // \EEH_Debug_Tools::printr($query_params, '$query_params', __FILE__, __LINE__);
-        return $query_params;
-    }
-
-
-
-    protected function getQueryParamForRule(Rule $rule)
-    {
-        // \EEH_Debug_Tools::printr(__FUNCTION__, __CLASS__, __FILE__, __LINE__, 2);
-        $strategy = $rule->getStrategy();
-        if (
-            ! class_exists($strategy)
-            || ! is_subclass_of($strategy, '\EventEspresso\core\services\conditional_logic\rules\RuleStrategy')
-        ){
-            throw new \DomainException(
-                sprintf(
-                    esc_html__('The "%1$s" class is either missing or not a valid Rule Strategy.', 'event_espresso'),
-                    $strategy
-                )
-            );
-        }
-        /** @var RuleStrategy $strategy */
-        $strategy = new $strategy();
-        $query_param = $strategy->getQueryParamForRule($rule);
-        // \EEH_Debug_Tools::printr($query_param, '$query_param', __FILE__, __LINE__);
-        return $query_param;
-    }
+	/**
+	 * @var Collection $rules
+	 */
+	private $rules;
+
+	// /**
+	//  * @var array $query_params
+	//  */
+	// private $query_params;
+
+
+
+	public function addRules(Collection $rules)
+	{
+		$this->rules = $rules;
+	}
+
+
+
+	public function resetRules()
+	{
+		$this->rules = null;
+	}
+
+
+
+	public function generateQueryParams($type = '')
+	{
+		\EEH_Debug_Tools::printr(__FUNCTION__, __CLASS__, __FILE__, __LINE__, 2);
+		$rules = $this->getRulesAsArray($type);
+		// \EEH_Debug_Tools::printr($rules, 'Rules As Array', __FILE__, __LINE__);
+		$query_params = $this->addQueryParamForRule($rules, array());
+		// \EEH_Debug_Tools::printr($query_params, '>>> $QUERY_PARAMS', __FILE__, __LINE__);
+		return $query_params;
+	}
+
+
+
+	private function getRulesAsArray($type = '')
+	{
+		// \EEH_Debug_Tools::printr(__FUNCTION__, __CLASS__, __FILE__, __LINE__, 2);
+		$rules_array = array();
+		foreach ($this->rules as $rule) {
+			/** @var Rule $rule */
+			if ($type && $rule->getType() !== $type){
+				continue;
+			}
+			$rules_array = $this->addRuleToArray($rules_array, $rule);
+		}
+		return $rules_array;
+	}
+
+
+
+	private function addRuleToArray(array $rules_array, Rule $rule)
+	{
+		// static $level = 0;
+		// \EEH_Debug_Tools::printr(__FUNCTION__, $level . ') ' .__CLASS__, __FILE__, __LINE__, 2);
+		// \EEH_Debug_Tools::printr((string)$rule->getID(), 'ID', __FILE__, __LINE__);
+		$parent = $rule->getParent();
+		// \EEH_Debug_Tools::printr((string)$parent, 'Parent', __FILE__, __LINE__);
+		if ($rule->getOperator() === ')') {
+			// $level--;
+			// \EEH_Debug_Tools::printr('SUBGROUP', 'CLOSE', __FILE__, __LINE__);
+			return $rules_array;
+		} else if ( isset($rules_array[$parent])) {
+			// $level++;
+			// \EEH_Debug_Tools::printr('PARENT', 'FOUND', __FILE__, __LINE__);
+			$rules_array[$parent] = $this->addRuleToArray($rules_array[$parent], $rule);
+			return $rules_array;
+		} else if (! empty($rules_array)){
+			// \EEH_Debug_Tools::printr('SUB ARRAY', 'FOUND', __FILE__, __LINE__);
+			// \EEH_Debug_Tools::printr($rules_array, '$rules_array', __FILE__, __LINE__);
+			foreach($rules_array as $ID => $rules) {
+				// \EEH_Debug_Tools::printr($ID, '$rules_array $ID', __FILE__, __LINE__);
+				if($ID === $parent) {
+					// \EEH_Debug_Tools::printr($ID === $parent, '$ID === $parent', __FILE__, __LINE__);
+					$rules_array[$ID] = $this->addRuleToArray($rules[$ID], $rule);
+					return $rules_array;
+				}
+			}
+		}
+		// \EEH_Debug_Tools::printr(
+		//     $rule->getOperator()
+		//     . ' '
+		//     . $rule->getTarget()
+		//     . ' '
+		//     . $rule->getComparison()
+		//     . ' '
+		//     . $rule->getValue(), 'ADD RULE',
+		//     __FILE__, __LINE__
+		// );
+		$rules_array[$rule->getID()] = array();
+		$rules_array[$rule->getID()]['rule'] = $rule;
+		return $rules_array;
+	}
+
+
+
+	public function addQueryParamForRule($rules, array $query_params, $operator = ' AND ')
+	{
+		// \EEH_Debug_Tools::printr(__FUNCTION__, __CLASS__, __FILE__, __LINE__, 2);
+		// \EEH_Debug_Tools::printr($rule, '$rule', __FILE__, __LINE__);
+		foreach ($rules as $key => $rule) {
+			// \EEH_Debug_Tools::printr((string)$key, '2) $key', __FILE__, __LINE__);
+			if (is_array($rule)) {
+				// \EEH_Debug_Tools::printr(is_array($rule), 'is_array($rule)', __FILE__, __LINE__);
+				$query_params = $this->addQueryParamForRule($rule, $query_params, $operator);
+			} else {
+				/** @var Rule $rule */
+				// \EEH_Debug_Tools::printr($rule, 'getQueryParamForRule', __FILE__, __LINE__);
+				$operator = $rule->getOperator();
+				$parent = $rule->getParent();
+				$parent = $parent ? $parent : 0;
+				if ( ! isset($query_params[$parent])) {
+					$query_params[$parent] = array();
+				}
+				\EEH_Debug_Tools::printr($operator, '$operator', __FILE__, __LINE__);
+				\EEH_Debug_Tools::printr((string)$parent, '$parent', __FILE__, __LINE__);
+				switch ($operator) {
+					default :
+					case '  ';
+					case ' AND ';
+						$query_params[$parent] = array_merge(
+							$query_params[$parent],
+							$this->getQueryParamForRule($rule)
+						);
+						break;
+					case ' OR ';
+						$query_params[$parent] = array_merge(
+							$query_params[$parent],
+							array('OR' => $this->getQueryParamForRule($rule))
+						);
+						break;
+					case ' AND ( ';
+						$query_params[$parent] = array_merge(
+							$query_params[$parent],
+							array( $this->getQueryParamForRule($rule) )
+						);
+						break;
+					case ' OR ( ';
+						$query_params['OR'][$parent] = array_merge(
+							$query_params['OR'][$parent],
+							$this->getQueryParamForRule($rule)
+						);
+						break;
+					case ' ) ';
+						break;
+				}
+			}
+		}
+		// \EEH_Debug_Tools::printr($query_params, '$query_params', __FILE__, __LINE__);
+		return $query_params;
+	}
+
+
+
+	protected function getQueryParamForRule(Rule $rule)
+	{
+		// \EEH_Debug_Tools::printr(__FUNCTION__, __CLASS__, __FILE__, __LINE__, 2);
+		$strategy = $rule->getStrategy();
+		if (
+			! class_exists($strategy)
+			|| ! is_subclass_of($strategy, '\EventEspresso\core\services\conditional_logic\rules\RuleStrategy')
+		){
+			throw new \DomainException(
+				sprintf(
+					esc_html__('The "%1$s" class is either missing or not a valid Rule Strategy.', 'event_espresso'),
+					$strategy
+				)
+			);
+		}
+		/** @var RuleStrategy $strategy */
+		$strategy = new $strategy();
+		$query_param = $strategy->getQueryParamForRule($rule);
+		// \EEH_Debug_Tools::printr($query_param, '$query_param', __FILE__, __LINE__);
+		return $query_param;
+	}
 
 }
 // End of file QueryParamGenerator.php

Spacing +8 added lines, -8 removed lines

@@ -62,7 +62,7 @@ 
         $rules_array = array();
         foreach ($this->rules as $rule) {
             /** @var Rule $rule */
-            if ($type && $rule->getType() !== $type){
+            if ($type && $rule->getType() !== $type) {
                 continue;
             }
             $rules_array = $this->addRuleToArray($rules_array, $rule);
@@ -83,17 +83,17 @@ 
             // $level--;
             // \EEH_Debug_Tools::printr('SUBGROUP', 'CLOSE', __FILE__, __LINE__);
             return $rules_array;
-        } else if ( isset($rules_array[$parent])) {
+        } else if (isset($rules_array[$parent])) {
             // $level++;
             // \EEH_Debug_Tools::printr('PARENT', 'FOUND', __FILE__, __LINE__);
             $rules_array[$parent] = $this->addRuleToArray($rules_array[$parent], $rule);
             return $rules_array;
-        } else if (! empty($rules_array)){
+        } else if ( ! empty($rules_array)) {
             // \EEH_Debug_Tools::printr('SUB ARRAY', 'FOUND', __FILE__, __LINE__);
             // \EEH_Debug_Tools::printr($rules_array, '$rules_array', __FILE__, __LINE__);
-            foreach($rules_array as $ID => $rules) {
+            foreach ($rules_array as $ID => $rules) {
                 // \EEH_Debug_Tools::printr($ID, '$rules_array $ID', __FILE__, __LINE__);
-                if($ID === $parent) {
+                if ($ID === $parent) {
                     // \EEH_Debug_Tools::printr($ID === $parent, '$ID === $parent', __FILE__, __LINE__);
                     $rules_array[$ID] = $this->addRuleToArray($rules[$ID], $rule);
                     return $rules_array;
@@ -136,7 +136,7 @@ 
                     $query_params[$parent] = array();
                 }
                 \EEH_Debug_Tools::printr($operator, '$operator', __FILE__, __LINE__);
-                \EEH_Debug_Tools::printr((string)$parent, '$parent', __FILE__, __LINE__);
+                \EEH_Debug_Tools::printr((string) $parent, '$parent', __FILE__, __LINE__);
                 switch ($operator) {
                     default :
                     case '  ';
@@ -155,7 +155,7 @@ 
                     case ' AND ( ';
                         $query_params[$parent] = array_merge(
                             $query_params[$parent],
-                            array( $this->getQueryParamForRule($rule) )
+                            array($this->getQueryParamForRule($rule))
                         );
                         break;
                     case ' OR ( ';
@@ -182,7 +182,7 @@ 
         if (
             ! class_exists($strategy)
             || ! is_subclass_of($strategy, '\EventEspresso\core\services\conditional_logic\rules\RuleStrategy')
-        ){
+        ) {
             throw new \DomainException(
                 sprintf(
                     esc_html__('The "%1$s" class is either missing or not a valid Rule Strategy.', 'event_espresso'),