eventespresso / event-espresso-core

core/libraries/messages/EE_Payment_Base_message_type.lib.php

Braces +3 added lines, -1 removed lines

@@ -5,7 +5,9 @@
  * @subpackage plugin api, messages
  * @since           4.5.0
  */
-if ( ! defined('EVENT_ESPRESSO_VERSION')) exit('No direct script access allowed');
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
+}
 
 /**
  * Use this to register or deregister a new message template pack variation for the EE messages system.

Spacing +7 added lines, -7 removed lines

@@ -51,26 +51,26 @@
 
 
 
-	protected function _get_data_for_context( $context, EE_Registration $registration, $id ) {
+	protected function _get_data_for_context($context, EE_Registration $registration, $id) {
 
 		//use the registration to get the transaction.
 		$transaction = $registration->transaction();
 
 		//bail early if no transaction
-		if ( ! $transaction instanceof EE_Transaction ) {
-			throw new EE_Error( __('The given registration does not have an associated transaction. Something is wrong.', 'event_espresso' ) );
+		if ( ! $transaction instanceof EE_Transaction) {
+			throw new EE_Error(__('The given registration does not have an associated transaction. Something is wrong.', 'event_espresso'));
 		}
 
-		$payment = ! empty( $id ) ? EEM_Payment::instance()->get_one( array( array( 'PAY_ID' => $id, 'TXN_ID' => $transaction->ID() ) ) ) : 0;
+		$payment = ! empty($id) ? EEM_Payment::instance()->get_one(array(array('PAY_ID' => $id, 'TXN_ID' => $transaction->ID()))) : 0;
 
-		return array( $transaction, $payment );
+		return array($transaction, $payment);
 	}
 
 
 
-	protected function _get_admin_content_events_edit_for_messenger( EE_messenger $messenger ) {
+	protected function _get_admin_content_events_edit_for_messenger(EE_messenger $messenger) {
 		//this is just a test
-		return $this->name . ' Message Type for ' . $messenger->name . ' Messenger ';
+		return $this->name.' Message Type for '.$messenger->name.' Messenger ';
 	}
 
 	/**

core/libraries/messages/data_class/EE_Messages_Preview_incoming_data.class.php

Braces +2 added lines, -1 removed lines

@@ -6,8 +6,9 @@
  * @package Event Espresso
  * @subpackage messages
  */
-if (!defined('EVENT_ESPRESSO_VERSION') )
+if (!defined('EVENT_ESPRESSO_VERSION') ) {
 	exit('NO direct script access allowed');
+}
 
 /**
  *

Spacing +84 added lines, -84 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-if (!defined('EVENT_ESPRESSO_VERSION') )
+if ( ! defined('EVENT_ESPRESSO_VERSION'))
 	exit('NO direct script access allowed');
 
 /**
@@ -29,8 +29,8 @@ 
 	 * For the constructor of this special preview class.  We're either looking for an event id or empty data.  If we have an event id (or ids) then we'll use that as the source for the "dummy" data.  If the data is empty then we'll get the first three published events from the users database and use that as a source.
 	 * @param array $data
 	 */
-	public function __construct( $data = array() ) {
-		$this->_data = isset( $data['event_ids'] ) ? $data['event_ids'] : array();
+	public function __construct($data = array()) {
+		$this->_data = isset($data['event_ids']) ? $data['event_ids'] : array();
 		$this->_setup_attendees_events();
 		parent::__construct($data);
 	}
@@ -46,7 +46,7 @@ 
 	 *
 	 * @return array   The prepped data for db
 	 */
-	static public function convert_data_for_persistent_storage( $data ) {
+	static public function convert_data_for_persistent_storage($data) {
 		return $data;
 	}
 
@@ -61,7 +61,7 @@ 
 	 *
 	 * @return array
 	 */
-	static public function convert_data_from_persistent_storage( $data ) {
+	static public function convert_data_from_persistent_storage($data) {
 		return $data;
 	}
 
@@ -78,12 +78,12 @@ 
 		$attendees = $this->_get_some_attendees();
 
 		//if empty $data we'll do a query to get some events from the server. otherwise we'll retrieve the event data for the given ids.
-		$events = $this->_get_some_events( $this->_data );
+		$events = $this->_get_some_events($this->_data);
 
 		$answers_n_questions = $this->_get_some_q_and_as();
 
-		if ( count( $events ) < 1 ) {
-			throw new EE_Error( __('We can\'t generate a preview for you because there are no active events in your database', 'event_espresso' ) );
+		if (count($events) < 1) {
+			throw new EE_Error(__('We can\'t generate a preview for you because there are no active events in your database', 'event_espresso'));
 		}
 
 
@@ -93,14 +93,14 @@ 
 
 		//we'll actually use the generated line_item identifiers for our loop
 		$dtts = $tkts = array();
-		foreach( $events as $id => $event ) {
-			if ( ! $event instanceof EE_Event ) {
+		foreach ($events as $id => $event) {
+			if ( ! $event instanceof EE_Event) {
 				continue;
 			}
 			$this->_events[$id]['ID'] = $id;
 			$this->_events[$id]['name'] = $event->get('EVT_name');
 			$datetime = $event->get_first_related('Datetime');
-			$tickets = $datetime instanceof EE_Datetime ? $datetime->get_many_related('Ticket', array('default_where_conditions' => 'none') ) : array();
+			$tickets = $datetime instanceof EE_Datetime ? $datetime->get_many_related('Ticket', array('default_where_conditions' => 'none')) : array();
 			$this->_events[$id]['event'] = $event;
 			$this->_events[$id]['reg_objs'] = array();
 			$this->_events[$id]['tkt_objs'] = $tickets;
@@ -108,51 +108,51 @@ 
 
 			$dttcache = array();
 			$tkts = array();
-			foreach ( $tickets as $ticket ) {
-				if ( ! $ticket instanceof EE_Ticket ) {
+			foreach ($tickets as $ticket) {
+				if ( ! $ticket instanceof EE_Ticket) {
 					continue;
 				}
 				$reldatetime = $ticket->datetimes();
-				$tkts[ $ticket->ID() ] = array();
-				$tkts[ $ticket->ID() ][ 'ticket' ] = $ticket;
-				$tkts[ $ticket->ID() ][ 'dtt_objs' ] = $reldatetime;
-				$tkts[ $ticket->ID() ][ 'att_objs' ] = $attendees;
-				$tkts[ $ticket->ID() ][ 'count' ] = count( $attendees );
-				$tkts[ $ticket->ID() ][ 'EE_Event' ] = $event;
-				foreach ( $reldatetime as $datetime ) {
-					if ( $datetime instanceof EE_Datetime && ! isset( $dtts[ $datetime->ID() ] ) ) {
-						$this->_events[ $id ][ 'dtt_objs' ][ $datetime->ID() ] = $datetime;
-						$dtts[ $datetime->ID() ][ 'datetime' ] = $datetime;
-						$dtts[ $datetime->ID() ][ 'tkt_objs' ][] = $ticket;
-						$dtts[ $datetime->ID() ][ 'evt_objs' ][] = $event;
-						$dttcache[ $datetime->ID() ] = $datetime;
+				$tkts[$ticket->ID()] = array();
+				$tkts[$ticket->ID()]['ticket'] = $ticket;
+				$tkts[$ticket->ID()]['dtt_objs'] = $reldatetime;
+				$tkts[$ticket->ID()]['att_objs'] = $attendees;
+				$tkts[$ticket->ID()]['count'] = count($attendees);
+				$tkts[$ticket->ID()]['EE_Event'] = $event;
+				foreach ($reldatetime as $datetime) {
+					if ($datetime instanceof EE_Datetime && ! isset($dtts[$datetime->ID()])) {
+						$this->_events[$id]['dtt_objs'][$datetime->ID()] = $datetime;
+						$dtts[$datetime->ID()]['datetime'] = $datetime;
+						$dtts[$datetime->ID()]['tkt_objs'][] = $ticket;
+						$dtts[$datetime->ID()]['evt_objs'][] = $event;
+						$dttcache[$datetime->ID()] = $datetime;
 					}
 				}
 			}
 
-			$this->_events[$id]['total_attendees'] = count( $attendees );
+			$this->_events[$id]['total_attendees'] = count($attendees);
 			$this->_events[$id]['att_objs'] = $attendees;
 
 			//let's also setup the dummy attendees property!
-			foreach ( $attendees as $att_key => $attendee ) {
-				if ( ! $attendee instanceof EE_Attendee ) {
+			foreach ($attendees as $att_key => $attendee) {
+				if ( ! $attendee instanceof EE_Attendee) {
 					continue;
 				}
-				$this->_attendees[$att_key]['line_ref'][] = $id;  //so later it can be determined what events this attendee registered for!
+				$this->_attendees[$att_key]['line_ref'][] = $id; //so later it can be determined what events this attendee registered for!
 				$this->_attendees[$att_key]['evt_objs'][] = $event;
 				$this->_attendees[$att_key]['att_obj'] = $attendee;
 				//$this->_attendees[$att_key]['registration_id'] = 0;
 				$this->_attendees[$att_key]['attendee_email'] = $attendee->email();
 				$this->_attendees[$att_key]['tkt_objs'] = $tickets;
-				if ( $att_key == 999999991 ) {
+				if ($att_key == 999999991) {
 					$this->_attendees[$att_key]['ans_objs'][999] = $answers_n_questions['answers'][999];
 					$this->_attendees[$att_key]['ans_objs'][1002] = $answers_n_questions['answers'][1002];
 					$this->_attendees[$att_key]['ans_objs'][1005] = $answers_n_questions['answers'][1005];
-				} elseif ( $att_key == 999999992 ) {
+				} elseif ($att_key == 999999992) {
 					$this->_attendees[$att_key]['ans_objs'][1000] = $answers_n_questions['answers'][1000];
 					$this->_attendees[$att_key]['ans_objs'][1003] = $answers_n_questions['answers'][1003];
 					$this->_attendees[$att_key]['ans_objs'][1006] = $answers_n_questions['answers'][1006];
-				} elseif ( $att_key == 999999993 ) {
+				} elseif ($att_key == 999999993) {
 					$this->_attendees[$att_key]['ans_objs'][1001] = $answers_n_questions['answers'][1001];
 					$this->_attendees[$att_key]['ans_objs'][1004] = $answers_n_questions['answers'][1004];
 					$this->_attendees[$att_key]['ans_objs'][1007] = $answers_n_questions['answers'][1007];
@@ -225,11 +225,11 @@ 
 
 		//let's generate the attendee objects
 		$attendees = array();
-		$var_array = array('fname','lname','email','address','address2','city','staid','cntry','zip','phone','deleted','attid');
+		$var_array = array('fname', 'lname', 'email', 'address', 'address2', 'city', 'staid', 'cntry', 'zip', 'phone', 'deleted', 'attid');
 
 		//EE_Registry::instance()->load_class( 'Attendee', array(), FALSE, false, TRUE );
-		foreach ( $dummy_attendees as $dummy ) {
-			$att = array_combine( $var_array, $dummy );
+		foreach ($dummy_attendees as $dummy) {
+			$att = array_combine($var_array, $dummy);
 			extract($att);
 			/** @var $fname string */
 			/** @var $lname string */
@@ -346,15 +346,15 @@ 
 
 		$qsts = array();
 		//first the questions
-		foreach ( $quests_array as $qst ) {
-			$qstobj = array_combine( $qst_columns, $qst );
+		foreach ($quests_array as $qst) {
+			$qstobj = array_combine($qst_columns, $qst);
 			$qsts[$qstobj['QST_ID']] = EE_Question::new_instance($qstobj);
 		}
 
 		//now the answers (and we'll setup our arrays)
 		$q_n_as = array();
-		foreach ( $ans_array as $ans ) {
-			$ansobj = array_combine( $ans_columns, $ans );
+		foreach ($ans_array as $ans) {
+			$ansobj = array_combine($ans_columns, $ans);
 			$ansobj = EE_Answer::new_instance($ansobj);
 			$q_n_as['answers'][$ansobj->ID()] = $ansobj;
 			$q_n_as['questions'][$ansobj->ID()] = $qsts[$ansobj->get('QST_ID')];
@@ -376,25 +376,25 @@ 
 	 * @param  array  $event_ids if set, this will be an array of event ids to obtain events for.
 	 * @return array    An array of event objects from the db.
 	 */
-	private function _get_some_events( $event_ids = array() ) {
+	private function _get_some_events($event_ids = array()) {
 
 		//HEY, if we have an evt_id then we want to make sure we use that for the preview (because a specific event template is being viewed);
-		$event_ids = isset( $_REQUEST['evt_id'] ) && !empty( $_REQUEST['evt_id'] )
-			? array( $_REQUEST['evt_id'] )
+		$event_ids = isset($_REQUEST['evt_id']) && ! empty($_REQUEST['evt_id'])
+			? array($_REQUEST['evt_id'])
 			: $event_ids;
 
-		$limit = !empty( $event_ids )
+		$limit = ! empty($event_ids)
 			? NULL
-			: apply_filters( 'FHEE__EE_Messages_Preview_incoming_data___get_some_events__limit', '0,1' );
+			: apply_filters('FHEE__EE_Messages_Preview_incoming_data___get_some_events__limit', '0,1');
 
-		$where = ! empty( $event_ids )
+		$where = ! empty($event_ids)
 			? array(
-				'EVT_ID' => array( 'IN', $event_ids ),
-				'Datetime.Ticket.TKT_ID' => array( '>', 1 )
+				'EVT_ID' => array('IN', $event_ids),
+				'Datetime.Ticket.TKT_ID' => array('>', 1)
 			)
-			: array( 'Datetime.Ticket.TKT_ID' => array( '>', 1 ) );
+			: array('Datetime.Ticket.TKT_ID' => array('>', 1));
 
-		$events = EE_Registry::instance()->load_model('Event')->get_all(array($where, 'limit' => $limit ) );
+		$events = EE_Registry::instance()->load_model('Event')->get_all(array($where, 'limit' => $limit));
 
 		return $events;
 	}
@@ -407,17 +407,17 @@ 
 	protected function _setup_data() {
 
 		//need to figure out the running total for test purposes so... we're going to create a temp cart and add the tickets to it!
-		if ( EE_Registry::instance()->SSN instanceof EE_Session ) {
-			EE_Registry::instance()->SSN->clear_session( __CLASS__, __FUNCTION__ );
+		if (EE_Registry::instance()->SSN instanceof EE_Session) {
+			EE_Registry::instance()->SSN->clear_session(__CLASS__, __FUNCTION__);
 			$session = EE_Registry::instance()->SSN;
 		} else {
-			$session = EE_Registry::instance()->load_core( 'Session' );
+			$session = EE_Registry::instance()->load_core('Session');
 		}
-		$cart = EE_Cart::instance( null, $session );
+		$cart = EE_Cart::instance(null, $session);
 
 
 		//add tickets to cart
-		foreach ( $this->tickets as $ticket ) {
+		foreach ($this->tickets as $ticket) {
 			$cart->add_ticket_to_cart($ticket['ticket']);
 		}
 
@@ -441,14 +441,14 @@ 
 		//note we're setting up a reg object for each attendee in each event but ALSO adding to the reg_object array.
 		$this->reg_objs = array();
 		$regid = 9999990;
-		foreach ( $this->_attendees as $key => $attendee ) {
+		foreach ($this->_attendees as $key => $attendee) {
 			//note we need to setup reg_objects for each event this attendee belongs to
 			$regatt = $attendee['att_obj'] instanceof EE_Attendee ? $attendee['att_obj']->ID() : null;
 			$regtxn = $this->txn->ID();
 			$regcnt = 1;
-			foreach ( $attendee['line_ref'] as $evtid ) {
-				foreach ( $this->_events[$evtid]['tkt_objs'] as $ticket ) {
-					if ( ! $ticket instanceof EE_Ticket ) {
+			foreach ($attendee['line_ref'] as $evtid) {
+				foreach ($this->_events[$evtid]['tkt_objs'] as $ticket) {
+					if ( ! $ticket instanceof EE_Ticket) {
 						continue;
 					}
 					$reg_array = array(
@@ -460,14 +460,14 @@ 
 						'REG_date' => time(),
 						'REG_final_price' => $ticket->get('TKT_price'),
 						'REG_session' => 'dummy_session_id',
-						'REG_code' => $regid . '-dummy-generated-code',
-						'REG_url_link' => $regcnt . '-daafpapasdlfakasdfpqasdfasdf',
+						'REG_code' => $regid.'-dummy-generated-code',
+						'REG_url_link' => $regcnt.'-daafpapasdlfakasdfpqasdfasdf',
 						'REG_count' => $regcnt,
 						'REG_group_size' => $this->_events[$evtid]['total_attendees'],
 						'REG_att_is_going' => TRUE,
 						'REG_ID' => $regid
 					);
-					$REG_OBJ =  EE_Registration::new_instance( $reg_array );
+					$REG_OBJ = EE_Registration::new_instance($reg_array);
 					$this->_attendees[$key]['reg_objs'][$regid] = $REG_OBJ;
 					$this->_events[$evtid]['reg_objs'][] = $REG_OBJ;
 					$this->reg_objs[] = $REG_OBJ;
@@ -483,40 +483,40 @@ 
 
 		//setup line items!
 		EE_Registry::instance()->load_helper('Line_Item');
-		$line_item_total = EEH_Line_Item::create_total_line_item( $this->txn );
+		$line_item_total = EEH_Line_Item::create_total_line_item($this->txn);
 
 		//add tickets
-		foreach ( $this->tickets as $tktid => $item ) {
+		foreach ($this->tickets as $tktid => $item) {
 			$qty = $item['count'];
 			$ticket = $item['ticket'];
-			EEH_Line_Item::add_ticket_purchase( $line_item_total, $ticket, $qty );
+			EEH_Line_Item::add_ticket_purchase($line_item_total, $ticket, $qty);
 		}
 
-		$shipping_line_item = EE_Line_Item::new_instance( array(
-			'LIN_name' => __( 'Shipping Surcharge', 'event_espresso' ),
-			'LIN_desc' => __( 'Sent via Millenium Falcon', 'event_espresso' ),
+		$shipping_line_item = EE_Line_Item::new_instance(array(
+			'LIN_name' => __('Shipping Surcharge', 'event_espresso'),
+			'LIN_desc' => __('Sent via Millenium Falcon', 'event_espresso'),
 			'LIN_unit_price' => 20,
 			'LIN_quantity' => 1,
 			'LIN_is_taxable' => TRUE,
 			'LIN_total' => 20,
 			'LIN_type' => EEM_Line_Item::type_line_item
 		));
-		EEH_Line_Item::add_item($line_item_total, $shipping_line_item );
-		$this->additional_line_items = array( $shipping_line_item );
+		EEH_Line_Item::add_item($line_item_total, $shipping_line_item);
+		$this->additional_line_items = array($shipping_line_item);
 
 		//now let's add taxes
-		EEH_Line_Item::apply_taxes( $line_item_total );
+		EEH_Line_Item::apply_taxes($line_item_total);
 
 		//now we should be able to get the items we need from this object
-		$event_line_items = EEH_Line_Item::get_pre_tax_subtotal( $line_item_total )->children();
+		$event_line_items = EEH_Line_Item::get_pre_tax_subtotal($line_item_total)->children();
 		$line_items = array();
-		foreach ( $event_line_items as $line_id => $line_item ) {
-			if ( ! $line_item instanceof EE_Line_Item || $line_item->OBJ_type() !== 'Event' ) {
+		foreach ($event_line_items as $line_id => $line_item) {
+			if ( ! $line_item instanceof EE_Line_Item || $line_item->OBJ_type() !== 'Event') {
 				continue;
 			}
-			$ticket_line_items = EEH_Line_Item::get_ticket_line_items( $line_item );
-			foreach ( $ticket_line_items as $ticket_line_id => $ticket_line_item ) {
-				if ( ! $ticket_line_item instanceof EE_Line_Item ) {
+			$ticket_line_items = EEH_Line_Item::get_ticket_line_items($line_item);
+			foreach ($ticket_line_items as $ticket_line_id => $ticket_line_item) {
+				if ( ! $ticket_line_item instanceof EE_Line_Item) {
 					continue;
 				}
 				$this->tickets[$ticket_line_item->OBJ_ID()]['line_item'] = $ticket_line_item;
@@ -532,12 +532,12 @@ 
 		//add proper total to transaction object.
 		$grand_total = $line_item_total->recalculate_total_including_taxes();
 		$this->grand_total_line_item = $line_item_total;
-		$this->txn->set_total( $grand_total );
+		$this->txn->set_total($grand_total);
 
 
 		//add additional details for each registration
-		foreach ( $this->reg_objs as $reg ) {
-			if ( ! $reg instanceof EE_Registration ) {
+		foreach ($this->reg_objs as $reg) {
+			if ( ! $reg instanceof EE_Registration) {
 				continue;
 			}
 			$this->_registrations[$reg->ID()]['tkt_obj'] = $this->tickets[$reg->get('TKT_ID')]['ticket'];
@@ -562,17 +562,17 @@ 
 				? $this->_attendees[999999991]['att_obj']->fname()
 				: '',
 
-			'lname' => $this->_attendees[ 999999991 ][ 'att_obj' ] instanceof EE_Attendee
+			'lname' => $this->_attendees[999999991]['att_obj'] instanceof EE_Attendee
 				? $this->_attendees[999999991]['att_obj']->lname()
 				: '',
 
-			'email' => $this->_attendees[ 999999991 ][ 'att_obj' ] instanceof EE_Attendee
+			'email' => $this->_attendees[999999991]['att_obj'] instanceof EE_Attendee
 				? $this->_attendees[999999991]['att_obj']->email()
 				: '',
 
 			'att_obj' => $this->_attendees[999999991]['att_obj'],
 
-			'reg_obj' => array_shift( $attendees_to_shift[999999991]['reg_objs'] )
+			'reg_obj' => array_shift($attendees_to_shift[999999991]['reg_objs'])
 		);
 
 		//reg_info property
@@ -580,7 +580,7 @@ 
 		$this->reg_info = array();
 
 		//let's set a reg_obj for messengers expecting one.
-		$this->reg_obj = array_pop( $this->_attendees[999999991]['reg_objs'] );
+		$this->reg_obj = array_pop($this->_attendees[999999991]['reg_objs']);
 
 
 		//the below are just dummy items.

core/libraries/messages/data_class/EE_Messages_REG_incoming_data.class.php

Braces +2 added lines, -1 removed lines

@@ -6,8 +6,9 @@
  * @package Event Espresso
  * @subpackage messages
  */
-if (!defined('EVENT_ESPRESSO_VERSION') )
+if (!defined('EVENT_ESPRESSO_VERSION') ) {
 	exit('NO direct script access allowed');
+}
 
 /**
  *

Spacing +23 added lines, -23 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-if (!defined('EVENT_ESPRESSO_VERSION') )
+if ( ! defined('EVENT_ESPRESSO_VERSION'))
 	exit('NO direct script access allowed');
 
 /**
@@ -29,20 +29,20 @@ 
 	 * @param EE_Registration|array $data
 	 * @throws \EE_Error
 	 */
-	public function __construct( $data ) {
+	public function __construct($data) {
 		$filtered_reg_status = null;
 
-		if ( ! is_array( $data ) && $data instanceof EE_Registration ) {
+		if ( ! is_array($data) && $data instanceof EE_Registration) {
 			$this->reg_obj = $data;
 		} else {
-			$this->reg_obj = is_array( $data ) && isset( $data[0] ) && $data[0] instanceof EE_Registration ? $data[0] : null;
-			$filtered_reg_status = is_array( $data ) && ! empty( $data[1] ) ? $data[1] : null;
+			$this->reg_obj = is_array($data) && isset($data[0]) && $data[0] instanceof EE_Registration ? $data[0] : null;
+			$filtered_reg_status = is_array($data) && ! empty($data[1]) ? $data[1] : null;
 		}
 
-		if ( ! $this->reg_obj instanceof EE_Registration ) {
+		if ( ! $this->reg_obj instanceof EE_Registration) {
 			throw new EE_Error(
 				sprintf(
-					__( '%1$s requires the incoming data argument to be an instance of %2$s or an array where the first value is an instance of %2$s', 'event_espresso'),
+					__('%1$s requires the incoming data argument to be an instance of %2$s or an array where the first value is an instance of %2$s', 'event_espresso'),
 					'EE_Messages_REG_incoming_data',
 					'EE_Registration'
 				)
@@ -64,18 +64,18 @@ 
 	 *
 	 * @return array   The prepped data for db
 	 */
-	static public function convert_data_for_persistent_storage( $data ) {
+	static public function convert_data_for_persistent_storage($data) {
 		$prepped_data = array();
-		if ( ! is_array( $data ) && $data instanceof EE_Registration ) {
+		if ( ! is_array($data) && $data instanceof EE_Registration) {
 			$prepped_data['Registration'] = $data->ID();
 			return $prepped_data;
-		} elseif ( ! is_array( $data ) ) {
+		} elseif ( ! is_array($data)) {
 			return array();
 		} else {
-			if ( $data[0] instanceof EE_Registration ) {
+			if ($data[0] instanceof EE_Registration) {
 				$prepped_data['Registration'] = $data[0];
 			}
-			if ( ! empty( $data[1] ) ) {
+			if ( ! empty($data[1])) {
 				$prepped_data['filter'] = $data[1];
 			}
 		}
@@ -91,17 +91,17 @@ 
 	 *
 	 * @return mixed
 	 */
-	static public function convert_data_from_persistent_storage( $data ) {
+	static public function convert_data_from_persistent_storage($data) {
 		$registration = null;
 		//$data['Registration'] could be either an ID (back compat) or a registration object (prepped using old system).
-		if ( isset( $data[ 'Registration' ] ) ) {
-			$registration = $data[ 'Registration' ] instanceof EE_Registration
-				? $data[ 'Registration' ]
-				: EEM_Registration::instance()->get_one_by_ID( $data[ 'Registration' ] );
+		if (isset($data['Registration'])) {
+			$registration = $data['Registration'] instanceof EE_Registration
+				? $data['Registration']
+				: EEM_Registration::instance()->get_one_by_ID($data['Registration']);
 		}
 		$prepped_data = array(
 			0 => $registration,
-			1 => isset( $data['filter'] ) ? $data['filter'] : null
+			1 => isset($data['filter']) ? $data['filter'] : null
 		);
 		return $prepped_data;
 	}
@@ -119,18 +119,18 @@ 
 		$this->txn = $this->reg_obj->transaction();
 		//possible session stuff?
 		$session = $this->txn->session_data();
-		$session_data =  $session instanceof EE_Session ? $session->get_session_data() : array();
+		$session_data = $session instanceof EE_Session ? $session->get_session_data() : array();
 
 		//other data from the session (if possible)
-		$this->user_id = isset( $session_data['user_id'] ) ? $session_data['user_id'] : '';
-		$this->ip_address = isset( $session_data['ip_address'] ) ? $session_data['ip_address'] : '';
-		$this->user_agent = isset( $session_data['user_agent'] ) ? $session_data['user_agent'] : '';
+		$this->user_id = isset($session_data['user_id']) ? $session_data['user_id'] : '';
+		$this->ip_address = isset($session_data['ip_address']) ? $session_data['ip_address'] : '';
+		$this->user_agent = isset($session_data['user_agent']) ? $session_data['user_agent'] : '';
 		$this->init_access = $this->last_access = '';
 
 		$this->payment = $this->txn->get_first_related('Payment');
 		// if there is no payments associated with the transaction
 		// then we just create a default payment object for potential parsing.
-		$this->payment = empty( $this->payment )
+		$this->payment = empty($this->payment)
 			? EE_Payment::new_instance(
 				array(
 					'STS_ID'               => EEM_Payment::status_id_pending,

core/libraries/messages/defaults/default/email_not_approved_registration_attendee_list_admin.template.php

Spacing +2 added lines, -2 removed lines

@@ -1,11 +1,11 @@
 <h4><a href="[EDIT_ATTENDEE_LINK]">[FNAME] [LNAME]</a></h4>
 <ul>
 	<li>
-		<strong><?php _e( 'Registration Code:', 'event_espresso' ); ?></strong>
+		<strong><?php _e('Registration Code:', 'event_espresso'); ?></strong>
 		<a href="[EDIT_ATTENDEE_LINK]">[REGISTRATION_CODE]</a>
 	</li>
 	<li>
-		<strong><?php _e( 'Tickets:', 'event_espresso' ); ?></strong>
+		<strong><?php _e('Tickets:', 'event_espresso'); ?></strong>
 		<ul>[TICKET_LIST]</ul>
 	</li>
 </ul>

core/libraries/messages/defaults/default/email_pending_approval_event_list.template.php

Spacing +2 added lines, -2 removed lines

@@ -3,7 +3,7 @@ 
 		<tbody>
 			<tr>
 				<td>
-					<p class="callout"><strong><?php _e( 'Event:', 'event_espresso' ); ?> [EVENT_LINK]</strong></p>
+					<p class="callout"><strong><?php _e('Event:', 'event_espresso'); ?> [EVENT_LINK]</strong></p>
 				</td>
 			</tr>
 		</tbody>
@@ -16,7 +16,7 @@ 
 			<tbody>
 				<tr>
 					<td>
-						<h3><?php _e( 'Registrant(s):', 'event_espresso' ); ?></h3>
+						<h3><?php _e('Registrant(s):', 'event_espresso'); ?></h3>
 						[ATTENDEE_LIST]
 					</td>
 				</tr>

core/libraries/messages/message_type/EE_Invoice_message_type.class.php

Spacing +5 added lines, -5 removed lines

@@ -32,7 +32,7 @@ 
 
 
 	protected function _set_admin_pages() {
-		$this->admin_registered_pages = array( 'events_edit' => TRUE );
+		$this->admin_registered_pages = array('events_edit' => TRUE);
 	}
 
 
@@ -45,17 +45,17 @@ 
 
 	protected function _set_with_messengers() {
 		$this->_with_messengers = array(
-			'html' => array( 'pdf' )
+			'html' => array('pdf')
 			);
 	}
 
 
 
-	protected function _get_data_for_context( $context, EE_Registration $registration, $id ) {
+	protected function _get_data_for_context($context, EE_Registration $registration, $id) {
 		//receipt message type data handler is 'Gateways' and it expects a transaction object.
 		$transaction = $registration->transaction();
-		if ( $transaction instanceof EE_Transaction ) {
-			return array( $transaction );
+		if ($transaction instanceof EE_Transaction) {
+			return array($transaction);
 		}
 		return array();
 	}

Braces +3 added lines, -1 removed lines

@@ -5,7 +5,9 @@
  * @subpackage plugin api, messages
  * @since           4.5.0
  */
-if ( ! defined('EVENT_ESPRESSO_VERSION')) exit('No direct script access allowed');
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
+}
 
 /**
  * Use this to register or deregister a new message template pack variation for the EE messages system.

Indentation +6 added lines, -6 removed lines

@@ -87,12 +87,12 @@
 
 
 	/**
-	* used to set the valid shortcodes for the receipt message type
-	*
-	* @since   4.5.0
-	*
-	* @return  void
-	*/
+	 * used to set the valid shortcodes for the receipt message type
+	 *
+	 * @since   4.5.0
+	 *
+	 * @return  void
+	 */
 	protected function _set_valid_shortcodes() {
 		$this->_valid_shortcodes['purchaser'] = array(
 			'attendee_list', 'attendee', 'datetime_list', 'datetime', 'event_list', 'event', 'event_meta', 'messenger', 'organization', 'primary_registration_list', 'primary_registration_details', 'ticket_list', 'ticket', 'transaction', 'venue', 'line_item_list', 'payment_list', 'line_item', 'payment'

core/libraries/messages/message_type/EE_Not_Approved_Registration_message_type.class.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-if (!defined('EVENT_ESPRESSO_VERSION') )
+if ( ! defined('EVENT_ESPRESSO_VERSION'))
 	exit('NO direct script access allowed');
 
 /**

Braces +2 added lines, -1 removed lines

@@ -6,8 +6,9 @@
  * @package Event Espresso
  * @subpackage messages
  */
-if (!defined('EVENT_ESPRESSO_VERSION') )
+if (!defined('EVENT_ESPRESSO_VERSION') ) {
 	exit('NO direct script access allowed');
+}
 
 /**
  *

core/libraries/messages/message_type/EE_Payment_Refund_message_type.class.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-if (!defined('EVENT_ESPRESSO_VERSION') )
+if ( ! defined('EVENT_ESPRESSO_VERSION'))
 	exit('NO direct script access allowed');
 
 /**

Braces +2 added lines, -1 removed lines

@@ -6,8 +6,9 @@
  * @package Event Espresso
  * @subpackage messages
  */
-if (!defined('EVENT_ESPRESSO_VERSION') )
+if (!defined('EVENT_ESPRESSO_VERSION') ) {
 	exit('NO direct script access allowed');
+}
 
 /**
  *

core/libraries/messages/message_type/EE_Payment_message_type.class.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-if (!defined('EVENT_ESPRESSO_VERSION') )
+if ( ! defined('EVENT_ESPRESSO_VERSION'))
 	exit('NO direct script access allowed');
 
 /**

Braces +2 added lines, -1 removed lines

@@ -6,8 +6,9 @@
  * @package Event Espresso
  * @subpackage messages
  */
-if (!defined('EVENT_ESPRESSO_VERSION') )
+if (!defined('EVENT_ESPRESSO_VERSION') ) {
 	exit('NO direct script access allowed');
+}
 
 /**
  *