Completed
Branch FET-9046-messages-queue (bdca24)
by
unknown
547:25 queued 529:16
created
core/libraries/messages/EE_Messages_Base.lib.php 2 patches
Braces   +9 added lines, -5 removed lines patch added patch discarded remove patch
@@ -1,6 +1,7 @@  discard block
 block discarded – undo
1 1
 <?php
2
-if (!defined('EVENT_ESPRESSO_VERSION') )
2
+if (!defined('EVENT_ESPRESSO_VERSION') ) {
3 3
 	exit('NO direct script access allowed');
4
+}
4 5
 
5 6
 /**
6 7
  * Event Espresso
@@ -209,8 +210,9 @@  discard block
 block discarded – undo
209 210
 	 */
210 211
 	public function get_existing_admin_settings( $messenger = NULL ) {
211 212
 		// if admin_settings property empty lets try setting it.
212
-		if ( method_exists($this, '_set_existing_admin_settings') && empty( $this->_existing_admin_settings ) )
213
-			$this->_set_existing_admin_settings( $messenger );
213
+		if ( method_exists($this, '_set_existing_admin_settings') && empty( $this->_existing_admin_settings ) ) {
214
+					$this->_set_existing_admin_settings( $messenger );
215
+		}
214 216
 
215 217
 		return EEH_Class_Tools::has_property($this, '_existing_admin_settings') ? $this->_existing_admin_settings : null;
216 218
 	}
@@ -265,8 +267,10 @@  discard block
 block discarded – undo
265 267
 			$page = $page . '_' . $action;
266 268
 		}
267 269
 
268
-		if ( !isset( $this->admin_registered_pages[$page]) )
269
-			return false; //todo: a place to throw an exception?  We need to indicate there is no registered page so this function is not being called correctly.
270
+		if ( !isset( $this->admin_registered_pages[$page]) ) {
271
+					return false;
272
+		}
273
+		//todo: a place to throw an exception?  We need to indicate there is no registered page so this function is not being called correctly.
270 274
 
271 275
 		//k made it here so let's call the method
272 276
 		if ( FALSE === ( $content = call_user_func_array( array( $this, '_get_admin_content_' . $page), array($actives, $extra) ) ) ) {
Please login to merge, or discard this patch.
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -1,5 +1,5 @@  discard block
 block discarded – undo
1 1
 <?php
2
-if (!defined('EVENT_ESPRESSO_VERSION') )
2
+if ( ! defined('EVENT_ESPRESSO_VERSION'))
3 3
 	exit('NO direct script access allowed');
4 4
 
5 5
 /**
@@ -181,14 +181,14 @@  discard block
 block discarded – undo
181 181
 	 * @access protected
182 182
 	 * @param string $messenger
183 183
 	 */
184
-	protected function _set_existing_admin_settings( $messenger = '' ) {
184
+	protected function _set_existing_admin_settings($messenger = '') {
185 185
 		$active_messengers = EEH_MSG_Template::get_active_messengers_in_db();
186
-		$active_message_types = ! empty( $messenger ) ? $active_messengers[$messenger]['settings'][$messenger . '-message_types'] : array();
186
+		$active_message_types = ! empty($messenger) ? $active_messengers[$messenger]['settings'][$messenger.'-message_types'] : array();
187 187
 
188 188
 
189 189
 		$actives = $this->_messages_item_type == 'messenger' ? $active_messengers : $active_message_types;
190 190
 
191
-		$this->_existing_admin_settings = isset($actives[$this->name]['settings'] ) ? $actives[$this->name]['settings'] : null;
191
+		$this->_existing_admin_settings = isset($actives[$this->name]['settings']) ? $actives[$this->name]['settings'] : null;
192 192
 	}
193 193
 
194 194
 
@@ -204,10 +204,10 @@  discard block
 block discarded – undo
204 204
 	 * @param string $messenger
205 205
 	 * @return array          settings
206 206
 	 */
207
-	public function get_existing_admin_settings( $messenger = '' ) {
207
+	public function get_existing_admin_settings($messenger = '') {
208 208
 		// if admin_settings property empty lets try setting it.
209
-		if ( method_exists($this, '_set_existing_admin_settings') && empty( $this->_existing_admin_settings ) )
210
-			$this->_set_existing_admin_settings( $messenger );
209
+		if (method_exists($this, '_set_existing_admin_settings') && empty($this->_existing_admin_settings))
210
+			$this->_set_existing_admin_settings($messenger);
211 211
 
212 212
 		return EEH_Class_Tools::has_property($this, '_existing_admin_settings') ? $this->_existing_admin_settings : null;
213 213
 	}
@@ -222,10 +222,10 @@  discard block
 block discarded – undo
222 222
 	 * @return array   an array of valid shortcodes.
223 223
 	 */
224 224
 	public function get_valid_shortcodes() {
225
-		$valid_shortcodes = apply_filters( 'FHEE__' . get_class($this) . '__get_valid_shortcodes', $this->_valid_shortcodes, $this );
225
+		$valid_shortcodes = apply_filters('FHEE__'.get_class($this).'__get_valid_shortcodes', $this->_valid_shortcodes, $this);
226 226
 
227 227
 		//The below filter applies to ALL messengers and message types so use with care!
228
-		$valid_shortcodes = apply_filters( 'FHEE__EE_Messages_Base__get_valid_shortcodes', $valid_shortcodes, $this );
228
+		$valid_shortcodes = apply_filters('FHEE__EE_Messages_Base__get_valid_shortcodes', $valid_shortcodes, $this);
229 229
 		return $valid_shortcodes;
230 230
 	}
231 231
 
@@ -256,17 +256,17 @@  discard block
 block discarded – undo
256 256
 	 * @access protected
257 257
 	 * @return string $content for page.
258 258
 	 */
259
-	protected function _get_admin_page_content( $page, $action, $extra, $actives ) {
259
+	protected function _get_admin_page_content($page, $action, $extra, $actives) {
260 260
 		//we can also further refine the context by action (if present).
261
-		if ( !empty($action) ) {
262
-			$page = $page . '_' . $action;
261
+		if ( ! empty($action)) {
262
+			$page = $page.'_'.$action;
263 263
 		}
264 264
 
265
-		if ( !isset( $this->admin_registered_pages[$page]) )
265
+		if ( ! isset($this->admin_registered_pages[$page]))
266 266
 			return false; //todo: a place to throw an exception?  We need to indicate there is no registered page so this function is not being called correctly.
267 267
 
268 268
 		//k made it here so let's call the method
269
-		if ( FALSE === ( $content = call_user_func_array( array( $this, '_get_admin_content_' . $page), array($actives, $extra) ) ) ) {
269
+		if (FALSE === ($content = call_user_func_array(array($this, '_get_admin_content_'.$page), array($actives, $extra)))) {
270 270
 			return false; //todo this needs to be an exception once we've got exceptions in place.
271 271
 		}
272 272
 		return $content;
Please login to merge, or discard this patch.
core/libraries/messages/EE_Registration_Base_message_type.lib.php 2 patches
Braces   +9 added lines, -5 removed lines patch added patch discarded remove patch
@@ -6,7 +6,9 @@  discard block
 block discarded – undo
6 6
  * @subpackage messages
7 7
  * @since           4.5.0
8 8
  */
9
-if ( ! defined('EVENT_ESPRESSO_VERSION')) exit('No direct script access allowed');
9
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
10
+	exit('No direct script access allowed');
11
+}
10 12
 
11 13
 /**
12 14
  * This class contains common methods/properties shared among all registration message types.
@@ -125,16 +127,18 @@  discard block
 block discarded – undo
125 127
 	 * @return array array of EE_Messages_Addressee objects
126 128
 	 */
127 129
 	protected function _admin_addressees() {
128
-		if ( $this->_single_message )
129
-			return array();
130
+		if ( $this->_single_message ) {
131
+					return array();
132
+		}
130 133
 		return parent::_admin_addressees();
131 134
 	}
132 135
 
133 136
 
134 137
 
135 138
 	protected function _primary_attendee_addressees() {
136
-		if ( $this->_single_message )
137
-			return array();
139
+		if ( $this->_single_message ) {
140
+					return array();
141
+		}
138 142
 
139 143
 		return parent::_primary_attendee_addressees();
140 144
 	}
Please login to merge, or discard this patch.
Spacing   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -38,35 +38,35 @@  discard block
 block discarded – undo
38 38
 	}
39 39
 
40 40
 
41
-	protected function _get_admin_content_events_edit_for_messenger( EE_Messenger $messenger ) {
41
+	protected function _get_admin_content_events_edit_for_messenger(EE_Messenger $messenger) {
42 42
 		//this is just a test
43
-		return $this->name . ' Message Type for ' . $messenger->name . ' Messenger ';
43
+		return $this->name.' Message Type for '.$messenger->name.' Messenger ';
44 44
 	}
45 45
 
46 46
 
47 47
 
48 48
 
49 49
 	protected function _set_data_handler() {
50
-		if ( is_array( $this->_data ) ) {
51
-			$data_type = reset( $this->_data );
50
+		if (is_array($this->_data)) {
51
+			$data_type = reset($this->_data);
52 52
 
53
-			if ( is_array( $data_type ) ) {
53
+			if (is_array($data_type)) {
54 54
 				//grab the first item and see if its a registration.
55
-				$maybe_reg = isset( $data_type[0] ) && is_array( $data_type[0] ) ? reset( $data_type[0] ) : reset( $data_type );
56
-				if ( $maybe_reg instanceof EE_Registration ) {
55
+				$maybe_reg = isset($data_type[0]) && is_array($data_type[0]) ? reset($data_type[0]) : reset($data_type);
56
+				if ($maybe_reg instanceof EE_Registration) {
57 57
 					//is $data_type itself just an array of registrations?
58
-					if ( isset( $data_type[1] ) && $data_type[1] instanceof EE_Registration  ) {
58
+					if (isset($data_type[1]) && $data_type[1] instanceof EE_Registration) {
59 59
 						$regs = $data_type;
60 60
 					} else {
61
-						$regs = is_array( $data_type[0] ) ? $data_type[0] : array( $maybe_reg );
61
+						$regs = is_array($data_type[0]) ? $data_type[0] : array($maybe_reg);
62 62
 					}
63 63
 
64
-					foreach ( $regs as $reg ) {
65
-						if ( $reg instanceof EE_Registration ) {
64
+					foreach ($regs as $reg) {
65
+						if ($reg instanceof EE_Registration) {
66 66
 							$this->_regs_for_sending[] = $reg->ID();
67 67
 						}
68 68
 					}
69
-					$this->_data = isset( $this->_data[1] ) ? array( $maybe_reg->transaction(), null, $this->_data[1] ) : array( $maybe_reg->transaction() );
69
+					$this->_data = isset($this->_data[1]) ? array($maybe_reg->transaction(), null, $this->_data[1]) : array($maybe_reg->transaction());
70 70
 					$this->_data_handler = 'Gateways';
71 71
 				} else {
72 72
 					$this->_data_handler = 'Gateways';
@@ -83,18 +83,18 @@  discard block
 block discarded – undo
83 83
 
84 84
 
85 85
 
86
-	protected function _get_data_for_context( $context, EE_Registration $registration, $id ) {
87
-		if ( $context  == 'admin' ) {
86
+	protected function _get_data_for_context($context, EE_Registration $registration, $id) {
87
+		if ($context == 'admin') {
88 88
 			//use the registration to get the transaction.
89 89
 			$transaction = $registration->transaction();
90 90
 
91 91
 			//bail early if no transaction
92
-			if ( ! $transaction instanceof EE_Transaction ) {
93
-				throw new EE_Error( __('The given registration does not have an associated transaction. Something is wrong.', 'event_espresso' ) );
92
+			if ( ! $transaction instanceof EE_Transaction) {
93
+				throw new EE_Error(__('The given registration does not have an associated transaction. Something is wrong.', 'event_espresso'));
94 94
 			}
95 95
 
96
-			$payment = !empty( $id ) ? EEM_Payment::instance()->get_one( array( array( 'PAY_ID' => $id, 'TXN_ID' => $transaction->ID() ) ) ) : 0;
97
-			return array( $transaction, $payment );
96
+			$payment = ! empty($id) ? EEM_Payment::instance()->get_one(array(array('PAY_ID' => $id, 'TXN_ID' => $transaction->ID()))) : 0;
97
+			return array($transaction, $payment);
98 98
 		} else {
99 99
 			return $registration;
100 100
 		}
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
 	 * @return array array of EE_Messages_Addressee objects
121 121
 	 */
122 122
 	protected function _admin_addressees() {
123
-		if ( $this->_single_message )
123
+		if ($this->_single_message)
124 124
 			return array();
125 125
 		return parent::_admin_addressees();
126 126
 	}
@@ -128,7 +128,7 @@  discard block
 block discarded – undo
128 128
 
129 129
 
130 130
 	protected function _primary_attendee_addressees() {
131
-		if ( $this->_single_message )
131
+		if ($this->_single_message)
132 132
 			return array();
133 133
 
134 134
 		return parent::_primary_attendee_addressees();
Please login to merge, or discard this patch.
core/libraries/messages/data_class/EE_Messages_Addressee.class.php 2 patches
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-if (!defined('EVENT_ESPRESSO_VERSION') )
3
+if ( ! defined('EVENT_ESPRESSO_VERSION'))
4 4
 	exit('NO direct script access allowed');
5 5
 
6 6
 /**
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
 	/**
35 35
 	 * Identifier properties for the recipient
36 36
 	 */
37
-	public $user_id;  //if available we'll use this to set the fname and lname (admin)
37
+	public $user_id; //if available we'll use this to set the fname and lname (admin)
38 38
 	public $fname; //this will always be the admin fname (set later via incoming user_id)
39 39
 	public $lname; //this will always be the admin lname (set later via incoming user_id)
40 40
 	public $primary_registration_id;
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
149 149
 	 * @param array $addressee_data We're expecting an incoming array of data that will be used to fill the properties for the object.
150 150
 	 * @return void
151 151
 	 */
152
-	public function __construct( $addressee_data ) {
152
+	public function __construct($addressee_data) {
153 153
 		$this->_data = $addressee_data;
154 154
 		$this->_set_properties();
155 155
 	}
@@ -165,15 +165,15 @@  discard block
 block discarded – undo
165 165
 	 */
166 166
 	protected function _set_properties() {
167 167
 
168
-		foreach ( $this->_data as $prop => $value ) {
169
-			if( EEH_Class_Tools::has_property( $this, $prop ) )
168
+		foreach ($this->_data as $prop => $value) {
169
+			if (EEH_Class_Tools::has_property($this, $prop))
170 170
 				$this->$prop = $value;
171 171
 		}
172 172
 
173 173
 		//if user_id present we'll use this to set the fname and lname and admin_email.
174
-		if ( !empty( $this->user_id ) ) {
174
+		if ( ! empty($this->user_id)) {
175 175
 			$this->user_id = (int) $this->user_id;
176
-			$user = get_userdata( $this->user_id );
176
+			$user = get_userdata($this->user_id);
177 177
 			$this->fname = $user->user_firstname;
178 178
 			$this->lname = $user->user_lastname;
179 179
 			$this->admin_email = $user->user_email;
Please login to merge, or discard this patch.
Braces   +5 added lines, -3 removed lines patch added patch discarded remove patch
@@ -1,7 +1,8 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-if (!defined('EVENT_ESPRESSO_VERSION') )
3
+if (!defined('EVENT_ESPRESSO_VERSION') ) {
4 4
 	exit('NO direct script access allowed');
5
+}
5 6
 
6 7
 /**
7 8
  * Event Espresso
@@ -166,8 +167,9 @@  discard block
 block discarded – undo
166 167
 	protected function _set_properties() {
167 168
 
168 169
 		foreach ( $this->_data as $prop => $value ) {
169
-			if( EEH_Class_Tools::has_property( $this, $prop ) )
170
-				$this->$prop = $value;
170
+			if( EEH_Class_Tools::has_property( $this, $prop ) ) {
171
+							$this->$prop = $value;
172
+			}
171 173
 		}
172 174
 
173 175
 		//if user_id present we'll use this to set the fname and lname and admin_email.
Please login to merge, or discard this patch.
libraries/messages/data_class/EE_Messages_Gateways_incoming_data.class.php 2 patches
Braces   +8 added lines, -5 removed lines patch added patch discarded remove patch
@@ -1,7 +1,8 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-if (!defined('EVENT_ESPRESSO_VERSION') )
3
+if (!defined('EVENT_ESPRESSO_VERSION') ) {
4 4
 	exit('NO direct script access allowed');
5
+}
5 6
 
6 7
 /**
7 8
  * Event Espresso
@@ -46,11 +47,13 @@  discard block
 block discarded – undo
46 47
 	public function __construct( $data ) {
47 48
 
48 49
 		//test for valid params
49
-		if ( ! ( $data[0] instanceof EE_Transaction ))
50
-			throw new EE_Error( __('Incoming data for the Gateways data handler must have an EE_Transaction object as the value for the first array index.', 'event_espresso') );
50
+		if ( ! ( $data[0] instanceof EE_Transaction )) {
51
+					throw new EE_Error( __('Incoming data for the Gateways data handler must have an EE_Transaction object as the value for the first array index.', 'event_espresso') );
52
+		}
51 53
 
52
-		if ( empty( $data[1] ) || ! $data[1] instanceof  EE_Payment  )
53
-			$pmt_obj = $this->_get_empty_payment_obj( $data[0] );
54
+		if ( empty( $data[1] ) || ! $data[1] instanceof  EE_Payment  ) {
55
+					$pmt_obj = $this->_get_empty_payment_obj( $data[0] );
56
+		}
54 57
 
55 58
 		if ( ! empty( $data[2] ) ) {
56 59
 			$filtered_reg_status = $data[2];
Please login to merge, or discard this patch.
Spacing   +22 added lines, -22 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-if (!defined('EVENT_ESPRESSO_VERSION') )
3
+if ( ! defined('EVENT_ESPRESSO_VERSION'))
4 4
 	exit('NO direct script access allowed');
5 5
 
6 6
 /**
@@ -43,25 +43,25 @@  discard block
 block discarded – undo
43 43
 	 * @param array $data
44 44
 	 * @throws \EE_Error
45 45
 	 */
46
-	public function __construct( $data ) {
46
+	public function __construct($data) {
47 47
 
48 48
 		//test for valid params
49
-		if ( ! ( $data[0] instanceof EE_Transaction ))
50
-			throw new EE_Error( __('Incoming data for the Gateways data handler must have an EE_Transaction object as the value for the first array index.', 'event_espresso') );
49
+		if ( ! ($data[0] instanceof EE_Transaction))
50
+			throw new EE_Error(__('Incoming data for the Gateways data handler must have an EE_Transaction object as the value for the first array index.', 'event_espresso'));
51 51
 
52
-		if ( empty( $data[1] ) || ! $data[1] instanceof  EE_Payment  )
53
-			$pmt_obj = $this->_get_empty_payment_obj( $data[0] );
52
+		if (empty($data[1]) || ! $data[1] instanceof  EE_Payment)
53
+			$pmt_obj = $this->_get_empty_payment_obj($data[0]);
54 54
 
55
-		if ( ! empty( $data[2] ) ) {
55
+		if ( ! empty($data[2])) {
56 56
 			$filtered_reg_status = $data[2];
57 57
 		}
58 58
 
59 59
 		$data = array(
60 60
 			'txn_obj' => $data[0],
61 61
 			'pmt_obj' => isset($pmt_obj) ? $pmt_obj : $data[1],
62
-			'filtered_reg_status' => isset( $filtered_reg_status ) ? $filtered_reg_status : null
62
+			'filtered_reg_status' => isset($filtered_reg_status) ? $filtered_reg_status : null
63 63
 			);
64
-		parent::__construct( $data );
64
+		parent::__construct($data);
65 65
 	}
66 66
 
67 67
 
@@ -75,18 +75,18 @@  discard block
 block discarded – undo
75 75
 	 *
76 76
 	 * @return array   The prepped data for db
77 77
 	 */
78
-	static public function convert_data_for_persistent_storage( $data ) {
78
+	static public function convert_data_for_persistent_storage($data) {
79 79
 		$prepped_data = array();
80 80
 
81
-		if ( $data[0] instanceof EE_Transaction ) {
81
+		if ($data[0] instanceof EE_Transaction) {
82 82
 			$prepped_data['Transaction'] = $data[0]->ID();
83 83
 		}
84 84
 
85
-		if ( isset( $data[1] ) && $data[1] instanceof EE_Payment ) {
85
+		if (isset($data[1]) && $data[1] instanceof EE_Payment) {
86 86
 			$prepped_data['Payment'] = $data[1]->ID();
87 87
 		}
88 88
 
89
-		if ( ! empty( $data[2] ) ) {
89
+		if ( ! empty($data[2])) {
90 90
 			$prepped_data['filter'] = $data[2];
91 91
 		}
92 92
 
@@ -106,11 +106,11 @@  discard block
 block discarded – undo
106 106
 	 *
107 107
 	 * @return array
108 108
 	 */
109
-	static public function convert_data_from_persistent_storage( $data ) {
109
+	static public function convert_data_from_persistent_storage($data) {
110 110
 		$prepped_data = array(
111
-			0 => isset( $data['Transaction'] ) ? EEM_Transaction::instance()->get_one_by_ID( $data['Transaction'] ) : null,
112
-			1 => isset( $data['Payment'] ) ? EEM_Payment::instance()->get_one_by_ID( $data['Payment'] ) : null,
113
-			2 => isset( $data['filter'] ) ? $data['filter'] : null
111
+			0 => isset($data['Transaction']) ? EEM_Transaction::instance()->get_one_by_ID($data['Transaction']) : null,
112
+			1 => isset($data['Payment']) ? EEM_Payment::instance()->get_one_by_ID($data['Payment']) : null,
113
+			2 => isset($data['filter']) ? $data['filter'] : null
114 114
 		);
115 115
 		return $prepped_data;
116 116
 	}
@@ -121,8 +121,8 @@  discard block
 block discarded – undo
121 121
 	 * @param \EE_Transaction $txn
122 122
 	 * @return \EE_Payment
123 123
 	 */
124
-	private function _get_empty_payment_obj( EE_Transaction $txn ) {
125
-		$PMT = EE_Payment::new_instance( array(
124
+	private function _get_empty_payment_obj(EE_Transaction $txn) {
125
+		$PMT = EE_Payment::new_instance(array(
126 126
 			'STS_ID' => EEM_Payment::status_id_pending,
127 127
 			'PAY_timestamp' => time(),
128 128
 			'PMD_ID' => $txn->payment_method_ID(),
@@ -150,9 +150,9 @@  discard block
 block discarded – undo
150 150
 
151 151
 
152 152
 		//other data from the session (if possible)
153
-		$this->user_id = isset( $session_data['user_id'] ) ? $session_data['user_id'] : '';
154
-		$this->ip_address = isset( $session_data['ip_address'] ) ? $session_data['ip_address'] : '';
155
-		$this->user_agent = isset( $session_data['user_agent'] ) ? $session_data['user_agent'] : '';
153
+		$this->user_id = isset($session_data['user_id']) ? $session_data['user_id'] : '';
154
+		$this->ip_address = isset($session_data['ip_address']) ? $session_data['ip_address'] : '';
155
+		$this->user_agent = isset($session_data['user_agent']) ? $session_data['user_agent'] : '';
156 156
 		$this->init_access = $this->last_access = '';
157 157
 
158 158
 		$this->reg_objs = $this->txn->get_many_related('Registration');
Please login to merge, or discard this patch.
core/libraries/messages/data_class/EE_Messages_incoming_data.core.php 2 patches
Braces   +9 added lines, -5 removed lines patch added patch discarded remove patch
@@ -1,7 +1,8 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-if (!defined('EVENT_ESPRESSO_VERSION') )
3
+if (!defined('EVENT_ESPRESSO_VERSION') ) {
4 4
 	exit('NO direct script access allowed');
5
+}
5 6
 
6 7
 /**
7 8
  * Event Espresso
@@ -323,8 +324,9 @@  discard block
 block discarded – undo
323 324
 		$regchk = array_shift($regchk);
324 325
 		$line_items = array();
325 326
 		//verify that reg_objs is set
326
-		if ( !is_array( $this->reg_objs) && ! reset( $this->reg_objs ) instanceof EE_Registration )
327
-			throw new EE_Error( __('In order to assemble the data correctly, the "reg_objs" property must be an array of EE_Registration objects', 'event_espresso') );
327
+		if ( !is_array( $this->reg_objs) && ! reset( $this->reg_objs ) instanceof EE_Registration ) {
328
+					throw new EE_Error( __('In order to assemble the data correctly, the "reg_objs" property must be an array of EE_Registration objects', 'event_espresso') );
329
+		}
328 330
 
329 331
 		//get all attendee and events associated with the registrations in this transaction
330 332
 		$events = $event_setup = $evtcache = $tickets = $datetimes = array();
@@ -384,8 +386,10 @@  discard block
 block discarded – undo
384 386
 					$eventsetup[$evt_id]['dtt_objs'][$dtt_id] = $datetime;
385 387
 					$registrations[$reg->ID()]['dtt_objs'][$dtt_id] = $datetime;
386 388
 
387
-					if ( isset( $datetimes[$dtt_id] ) )
388
-						continue; //already have this info in the datetimes array.
389
+					if ( isset( $datetimes[$dtt_id] ) ) {
390
+											continue;
391
+					}
392
+					//already have this info in the datetimes array.
389 393
 
390 394
 					$datetimes[$dtt_id]['tkt_objs'][] = $ticket;
391 395
 					$datetimes[$dtt_id]['datetime'] = $datetime;
Please login to merge, or discard this patch.
Spacing   +26 added lines, -26 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-if (!defined('EVENT_ESPRESSO_VERSION') )
3
+if ( ! defined('EVENT_ESPRESSO_VERSION'))
4 4
 	exit('NO direct script access allowed');
5 5
 
6 6
 /**
@@ -286,7 +286,7 @@  discard block
 block discarded – undo
286 286
 	 * @param mixed $data incoming data object|array.  Suggested that child classes use type hinting for expected
287 287
 	 * data object.  But here parent will be generic because we don't know what's coming in.
288 288
 	 */
289
-	public function __construct( $data ) {
289
+	public function __construct($data) {
290 290
 		$this->_data = $data;
291 291
 		$this->_setup_data();
292 292
 
@@ -311,7 +311,7 @@  discard block
 block discarded – undo
311 311
 	 *
312 312
 	 * @return mixed   The prepped data for db
313 313
 	 */
314
-	static public function convert_data_for_persistent_storage( $data ) {
314
+	static public function convert_data_for_persistent_storage($data) {
315 315
 		return $data;
316 316
 	}
317 317
 
@@ -327,7 +327,7 @@  discard block
 block discarded – undo
327 327
 	 *
328 328
 	 * @return mixed
329 329
 	 */
330
-	static public function convert_data_from_persistent_storage( $data ) {
330
+	static public function convert_data_from_persistent_storage($data) {
331 331
 		return $data;
332 332
 	}
333 333
 
@@ -356,19 +356,19 @@  discard block
 block discarded – undo
356 356
 		$regchk = array_shift($regchk);
357 357
 		$line_items = array();
358 358
 		//verify that reg_objs is set
359
-		if ( !is_array( $this->reg_objs) && ! reset( $this->reg_objs ) instanceof EE_Registration )
360
-			throw new EE_Error( __('In order to assemble the data correctly, the "reg_objs" property must be an array of EE_Registration objects', 'event_espresso') );
359
+		if ( ! is_array($this->reg_objs) && ! reset($this->reg_objs) instanceof EE_Registration)
360
+			throw new EE_Error(__('In order to assemble the data correctly, the "reg_objs" property must be an array of EE_Registration objects', 'event_espresso'));
361 361
 
362 362
 		//get all attendee and events associated with the registrations in this transaction
363 363
 		$events = $event_setup = $evtcache = $tickets = $datetimes = array();
364 364
 		$answers = $questions = $attendees = $line_items = $registrations = array();
365 365
 		$total_ticket_count = 0;
366 366
 
367
-		if ( !empty( $this->reg_objs ) ) {
367
+		if ( ! empty($this->reg_objs)) {
368 368
 			$event_attendee_count = array();
369
-			foreach ( $this->reg_objs as $reg ) {
369
+			foreach ($this->reg_objs as $reg) {
370 370
 				//account for filtered registrations by status.
371
-				if ( ! empty( $this->filtered_reg_status ) && $this->filtered_reg_status !== $reg->status_ID() ) {
371
+				if ( ! empty($this->filtered_reg_status) && $this->filtered_reg_status !== $reg->status_ID()) {
372 372
 					continue;
373 373
 				}
374 374
 
@@ -378,7 +378,7 @@  discard block
 block discarded – undo
378 378
 				$relateddatetime = $ticket->datetimes();
379 379
 				$total_ticket_count++;
380 380
 				$tickets[$ticket->ID()]['ticket'] = $ticket;
381
-				$tickets[$ticket->ID()]['count'] = is_array($tickets[$ticket->ID()]) && isset( $tickets[$ticket->ID()]['count'] ) ? $tickets[$ticket->ID()]['count'] + 1 : 1;
381
+				$tickets[$ticket->ID()]['count'] = is_array($tickets[$ticket->ID()]) && isset($tickets[$ticket->ID()]['count']) ? $tickets[$ticket->ID()]['count'] + 1 : 1;
382 382
 				$tickets[$ticket->ID()]['att_objs'][$reg->attendee_ID()] = $reg->attendee();
383 383
 				$tickets[$ticket->ID()]['dtt_objs'] = $relateddatetime;
384 384
 				$tickets[$ticket->ID()]['reg_objs'][$reg->ID()] = $reg;
@@ -388,7 +388,7 @@  discard block
 block discarded – undo
388 388
 				$eventsetup[$evt_id]['reg_objs'][$reg->ID()] = $reg;
389 389
 				$eventsetup[$evt_id]['tkt_objs'][$ticket->ID()] = $ticket;
390 390
 				$eventsetup[$evt_id]['att_objs'][$reg->attendee_ID()] = $reg->attendee();
391
-				$event_attendee_count[$evt_id] = isset( $event_attendee_count[$evt_id] ) ? $event_attendee_count[$evt_id] + 1 : 0;
391
+				$event_attendee_count[$evt_id] = isset($event_attendee_count[$evt_id]) ? $event_attendee_count[$evt_id] + 1 : 0;
392 392
 				$attendees[$reg->attendee_ID()]['line_ref'][] = $evt_id;
393 393
 				$attendees[$reg->attendee_ID()]['att_obj'] = $reg->attendee();
394 394
 				$attendees[$reg->attendee_ID()]['reg_objs'][$reg->ID()] = $reg;
@@ -405,19 +405,19 @@  discard block
 block discarded – undo
405 405
 
406 406
 				//set up answer objects
407 407
 				$rel_ans = $reg->get_many_related('Answer');
408
-				foreach ( $rel_ans as $ansid => $answer ) {
409
-					if ( !isset( $questions[$ansid] ) ) {
408
+				foreach ($rel_ans as $ansid => $answer) {
409
+					if ( ! isset($questions[$ansid])) {
410 410
 						$questions[$ansid] = $answer->get_first_related('Question');
411 411
 					}
412 412
 					$answers[$ansid] = $answer;
413 413
 					$registrations[$reg->ID()]['ans_objs'][$ansid] = $answer;
414 414
 				}
415 415
 
416
-				foreach ( $relateddatetime as $dtt_id => $datetime ) {
416
+				foreach ($relateddatetime as $dtt_id => $datetime) {
417 417
 					$eventsetup[$evt_id]['dtt_objs'][$dtt_id] = $datetime;
418 418
 					$registrations[$reg->ID()]['dtt_objs'][$dtt_id] = $datetime;
419 419
 
420
-					if ( isset( $datetimes[$dtt_id] ) )
420
+					if (isset($datetimes[$dtt_id]))
421 421
 						continue; //already have this info in the datetimes array.
422 422
 
423 423
 					$datetimes[$dtt_id]['tkt_objs'][] = $ticket;
@@ -429,9 +429,9 @@  discard block
 block discarded – undo
429 429
 
430 430
 			//let's loop through the unique event=>reg items and setup data on them
431 431
 
432
-			if ( !empty( $eventsetup) ) {
433
-				foreach ( $eventsetup as $evt_id => $items ) {
434
-					$ticket_line_items_for_event = EEM_Line_Item::instance()->get_all(array(array('Ticket.Datetime.EVT_ID'=>$evt_id,'TXN_ID'=>$this->txn->ID()), 'default_where_conditions' => 'none'));
432
+			if ( ! empty($eventsetup)) {
433
+				foreach ($eventsetup as $evt_id => $items) {
434
+					$ticket_line_items_for_event = EEM_Line_Item::instance()->get_all(array(array('Ticket.Datetime.EVT_ID'=>$evt_id, 'TXN_ID'=>$this->txn->ID()), 'default_where_conditions' => 'none'));
435 435
 					$events[$evt_id] = array(
436 436
 						'ID' => $evt_id,
437 437
 						'event' => $evtcache[$evt_id],
@@ -440,13 +440,13 @@  discard block
 block discarded – undo
440 440
 						'reg_objs' => $items['reg_objs'],
441 441
 						'tkt_objs' => $items['tkt_objs'],
442 442
 						'att_objs' => $items['att_objs'],
443
-						'dtt_objs' => isset( $items['dtt_objs'] ) ? $items['dtt_objs'] : array(),
443
+						'dtt_objs' => isset($items['dtt_objs']) ? $items['dtt_objs'] : array(),
444 444
 						'line_items' => $ticket_line_items_for_event
445 445
 					);
446 446
 
447 447
 					//make sure the tickets have the line items setup for them.
448
-					foreach ( $ticket_line_items_for_event as $line_id => $line_item ) {
449
-						if ( $line_item instanceof EE_Line_Item ) {
448
+					foreach ($ticket_line_items_for_event as $line_id => $line_item) {
449
+						if ($line_item instanceof EE_Line_Item) {
450 450
 							$tickets[$line_item->ticket()->ID()]['line_item'] = $line_item;
451 451
 							$tickets[$line_item->ticket()->ID()]['sub_line_items'] = $line_item->children();
452 452
 							$line_items[$line_item->ID()]['children'] = $line_item->children();
@@ -470,7 +470,7 @@  discard block
 block discarded – undo
470 470
 		$this->total_ticket_count = $total_ticket_count;
471 471
 		$this->registrations = $registrations;
472 472
 
473
-		$this->tax_line_items =  $this->txn->tax_items();
473
+		$this->tax_line_items = $this->txn->tax_items();
474 474
 		$this->additional_line_items = $this->txn->non_ticket_line_items();
475 475
 		$this->payments = $this->txn->payments();
476 476
 
@@ -480,10 +480,10 @@  discard block
 block discarded – undo
480 480
 		//let's get just the primary_attendee_data!  First we get the primary registration object.
481 481
 		$primary_reg = $this->txn->primary_registration(TRUE);
482 482
 		// verify
483
-		if( $primary_reg instanceof EE_Registration ) {
483
+		if ($primary_reg instanceof EE_Registration) {
484 484
 
485 485
 			// get attendee object
486
-			if( $primary_reg->attendee() instanceof EE_Attendee ) {
486
+			if ($primary_reg->attendee() instanceof EE_Attendee) {
487 487
 
488 488
 				//now we can setup the primary_attendee_data array
489 489
 				$this->primary_attendee_data = array(
@@ -495,11 +495,11 @@  discard block
 block discarded – undo
495 495
 				);
496 496
 
497 497
 			} else {
498
-				EE_Error::add_error( __('Incoming data does not have a valid Attendee object for the primary registrant.', 'event_espresso'), __FILE__, __FUNCTION__, __LINE__ );
498
+				EE_Error::add_error(__('Incoming data does not have a valid Attendee object for the primary registrant.', 'event_espresso'), __FILE__, __FUNCTION__, __LINE__);
499 499
 			}
500 500
 
501 501
 		} else {
502
-			EE_Error::add_error( __('Incoming data does not have a valid Registration object for the primary registrant.', 'event_espresso'), __FILE__, __FUNCTION__, __LINE__ );
502
+			EE_Error::add_error(__('Incoming data does not have a valid Registration object for the primary registrant.', 'event_espresso'), __FILE__, __FUNCTION__, __LINE__);
503 503
 		}
504 504
 	}
505 505
 
Please login to merge, or discard this patch.
core/libraries/messages/defaults/EE_Message_Template_Defaults.core.php 3 patches
Braces   +5 added lines, -2 removed lines patch added patch discarded remove patch
@@ -1,7 +1,8 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-if (!defined('EVENT_ESPRESSO_VERSION') )
3
+if (!defined('EVENT_ESPRESSO_VERSION') ) {
4 4
 	exit('NO direct script access allowed');
5
+}
5 6
 
6 7
 /**
7 8
  * Event Espresso
@@ -276,7 +277,9 @@  discard block
 block discarded – undo
276 277
 			//let's insert the above and get our GRP_ID, then reset the template data array to just include the GRP_ID
277 278
 			$grp_id = $this->_EEM_data->insert( $main_template_data );
278 279
 
279
-			if ( empty( $grp_id ) ) return $grp_id;
280
+			if ( empty( $grp_id ) ) {
281
+				return $grp_id;
282
+			}
280 283
 			$this->_GRP_ID = $grp_id;
281 284
 		}
282 285
 
Please login to merge, or discard this patch.
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -115,8 +115,8 @@
 block discarded – undo
115 115
 	/**
116 116
 	 * constructor
117 117
 	 * @param EE_Messages $messages the EE_Messages object.
118
-	 * @param $messenger_name	should be the name of a valid active messenger
119
-	 * @param $message_type_name  should be the name of a valid active message type
118
+	 * @param string $messenger_name	should be the name of a valid active messenger
119
+	 * @param string $message_type_name  should be the name of a valid active message type
120 120
 	 * @param int $GRP_ID Optional.  If included then we're just regenerating the template
121 121
 	 *                    		 fields for the given group not the message template group itself
122 122
 	 *
Please login to merge, or discard this patch.
Spacing   +35 added lines, -35 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-if (!defined('EVENT_ESPRESSO_VERSION') )
3
+if ( ! defined('EVENT_ESPRESSO_VERSION'))
4 4
 	exit('NO direct script access allowed');
5 5
 
6 6
 /**
@@ -123,7 +123,7 @@  discard block
 block discarded – undo
123 123
 	 * @access public
124 124
 	 * @return void
125 125
 	 */
126
-	public function __construct( EE_Messages $messages, $messenger_name, $message_type_name, $GRP_ID = 0 ) {
126
+	public function __construct(EE_Messages $messages, $messenger_name, $message_type_name, $GRP_ID = 0) {
127 127
 		$this->_EE_MSG = $messages;
128 128
 
129 129
 		//set the model object
@@ -135,10 +135,10 @@  discard block
 block discarded – undo
135 135
 		$this->_mt_name = $message_type_name;
136 136
 
137 137
 		//make sure required props have been set
138
-		if ( empty( $this->_m_name) || empty( $this->_mt_name ) ) {
138
+		if (empty($this->_m_name) || empty($this->_mt_name)) {
139 139
 			$msg[] = __('Message Templates cannot be generated because the Messenger and Message Types haven\'t been defined for the generator.', 'event_espresso');
140 140
 			$msg[] = __('You need to set the "$m_name" and "$mt_name" properties', 'event_espresso');
141
-			throw new EE_Error( implode( '||', $msg ) );
141
+			throw new EE_Error(implode('||', $msg));
142 142
 		}
143 143
 
144 144
 		$this->_init();
@@ -154,20 +154,20 @@  discard block
 block discarded – undo
154 154
 	 * @param string $template_pack This corresponds to a template pack class reference which will contain information about where to obtain the templates.
155 155
 	 * @return void
156 156
 	 */
157
-	final private function _set_templates( $template_pack ) {
157
+	final private function _set_templates($template_pack) {
158 158
 
159 159
 		//get the corresponding template pack object (if present.  If not then we just load the default and add a notice).  The class name should be something like 'EE_Messages_Template_Pack_Default' where "default' would be the incoming template pack reference.
160
-		$class_name = 'EE_Messages_Template_Pack_' . str_replace( ' ', '_', ucwords( str_replace( '_', ' ', $template_pack ) ) );
160
+		$class_name = 'EE_Messages_Template_Pack_'.str_replace(' ', '_', ucwords(str_replace('_', ' ', $template_pack)));
161 161
 
162
-		if ( ! class_exists( $class_name ) ) {
163
-			EE_Error::add_error( sprintf( __('The template pack represented by a class corresponding to "%s" does not exist.  Likely the autoloader for this class has the wrong path or the incoming reference is mispelled.  The default template pack  been used to generate the templates instead.', 'event_espresso'), $class_name ), __FILE__, __FUNCTION__, __LINE__ );
162
+		if ( ! class_exists($class_name)) {
163
+			EE_Error::add_error(sprintf(__('The template pack represented by a class corresponding to "%s" does not exist.  Likely the autoloader for this class has the wrong path or the incoming reference is mispelled.  The default template pack  been used to generate the templates instead.', 'event_espresso'), $class_name), __FILE__, __FUNCTION__, __LINE__);
164 164
 			$class_name = 'EE_Messages_Template_Pack_Default';
165 165
 		}
166 166
 
167 167
 		$template_pack = new $class_name;
168 168
 
169 169
 		//get all the templates from the template pack.
170
-		$this->_templates = $template_pack->get_templates( $this->_messenger, $this->_message_type );
170
+		$this->_templates = $template_pack->get_templates($this->_messenger, $this->_message_type);
171 171
 	}
172 172
 
173 173
 
@@ -185,29 +185,29 @@  discard block
 block discarded – undo
185 185
 		$active_message_types = $this->_EE_MSG->get_installed_message_types();
186 186
 
187 187
 		//check if messenger is active
188
-		if ( !isset($active_messengers[$this->_m_name] ) ) {
188
+		if ( ! isset($active_messengers[$this->_m_name])) {
189 189
 			$msg[] = __('Message Templates cannot be generated because the given messenger is not active', 'event_espresso');
190
-			$msg[] = sprintf( __('The "$_m_name" property has "%s" as its value.  Check the spelling and make sure it matches an available messenger', 'event_espresso'), $this->_m_name );
191
-			throw new EE_Error( implode( '||', $msg ) );
190
+			$msg[] = sprintf(__('The "$_m_name" property has "%s" as its value.  Check the spelling and make sure it matches an available messenger', 'event_espresso'), $this->_m_name);
191
+			throw new EE_Error(implode('||', $msg));
192 192
 		}
193 193
 
194 194
 		//check if message type is installed
195
-		if ( !isset($active_messengers[$this->_m_name] ) ) {
195
+		if ( ! isset($active_messengers[$this->_m_name])) {
196 196
 			$msg[] = __('Message Templates cannot be generated because the given message type is not installed', 'event_espresso');
197
-			$msg[] = sprintf( __('The "$_mt_name" property has "%s" as its value.  Check the spelling and make sure it matches an available message type', 'event_espresso'), $this->_mt_name );
198
-			throw new EE_Error( implode( '||', $msg ) );
197
+			$msg[] = sprintf(__('The "$_mt_name" property has "%s" as its value.  Check the spelling and make sure it matches an available message type', 'event_espresso'), $this->_mt_name);
198
+			throw new EE_Error(implode('||', $msg));
199 199
 		}
200 200
 
201 201
 		$this->_messenger = $active_messengers[$this->_m_name];
202 202
 		$this->_message_type = $active_message_types[$this->_mt_name];
203 203
 
204 204
 		//verify we have the messenger and message type objects
205
-		if ( ! $this->_messenger instanceof EE_Messenger ) {
206
-			throw new EE_Error( sprintf( __('The _messenger property must be an instance of EE_Messenger by this point.  It isn\'t. Something has gone wrong. Here is the value it holds:<br /> %s', 'event_espresso' ), print_r( $this->_messenger, TRUE ) ) );
205
+		if ( ! $this->_messenger instanceof EE_Messenger) {
206
+			throw new EE_Error(sprintf(__('The _messenger property must be an instance of EE_Messenger by this point.  It isn\'t. Something has gone wrong. Here is the value it holds:<br /> %s', 'event_espresso'), print_r($this->_messenger, TRUE)));
207 207
 		}
208 208
 
209
-		if ( ! $this->_message_type instanceof EE_message_type ) {
210
-			throw new EE_Error( sprintf( __('The _message_type property must be an instance of EE_message_type by this point.  It isn\'t. Something has gone wrong. Here is the value it holds:<br /> %s', 'event_espresso' ), print_r( $this->_message_type, TRUE ) ) );
209
+		if ( ! $this->_message_type instanceof EE_message_type) {
210
+			throw new EE_Error(sprintf(__('The _message_type property must be an instance of EE_message_type by this point.  It isn\'t. Something has gone wrong. Here is the value it holds:<br /> %s', 'event_espresso'), print_r($this->_message_type, TRUE)));
211 211
 		}
212 212
 
213 213
 
@@ -235,13 +235,13 @@  discard block
 block discarded – undo
235 235
 	public function create_new_templates() {
236 236
 		$template_pack = 'default';
237 237
 		//if we have the GRP_ID then let's use that to see if there is a set template pack and use that for the new templates.
238
-		if ( !empty( $this->_GRP_ID ) ) {
239
-			$mtpg = EEM_Message_Template_Group::instance()->get_one_by_ID( $this->_GRP_ID );
238
+		if ( ! empty($this->_GRP_ID)) {
239
+			$mtpg = EEM_Message_Template_Group::instance()->get_one_by_ID($this->_GRP_ID);
240 240
 			$template_pack = $mtpg instanceof EE_Message_Template_Group ? $mtpg->get_template_pack_name() : 'default';
241 241
 			//we also need to reset the template variation to default
242
-			$mtpg->set_template_pack_variation( 'default' );
242
+			$mtpg->set_template_pack_variation('default');
243 243
 		}
244
-		return $this->_create_new_templates( $template_pack );
244
+		return $this->_create_new_templates($template_pack);
245 245
 	}
246 246
 
247 247
 
@@ -254,15 +254,15 @@  discard block
 block discarded – undo
254 254
 	 * @param string $template_pack This corresponds to a template pack class reference which will contain information about where to obtain the templates.
255 255
 	 * @return mixed (array|bool)            success array or false.
256 256
 	 */
257
-	private function _create_new_templates( $template_pack ) {
257
+	private function _create_new_templates($template_pack) {
258 258
 
259
-		$this->_set_templates( $template_pack );
259
+		$this->_set_templates($template_pack);
260 260
 
261 261
 		//necessary properties are set, let's save the default templates
262 262
 
263
-		if ( empty( $this->_GRP_ID ) ) {
263
+		if (empty($this->_GRP_ID)) {
264 264
 
265
-			$main_template_data =  array(
265
+			$main_template_data = array(
266 266
 				'MTP_messenger' => $this->_messenger->name,
267 267
 				'MTP_message_type' => $this->_message_type->name,
268 268
 				'MTP_is_override' => 0,
@@ -274,24 +274,24 @@  discard block
 block discarded – undo
274 274
 
275 275
 
276 276
 			//let's insert the above and get our GRP_ID, then reset the template data array to just include the GRP_ID
277
-			$grp_id = $this->_EEM_data->insert( $main_template_data );
277
+			$grp_id = $this->_EEM_data->insert($main_template_data);
278 278
 
279
-			if ( empty( $grp_id ) ) return $grp_id;
279
+			if (empty($grp_id)) return $grp_id;
280 280
 			$this->_GRP_ID = $grp_id;
281 281
 		}
282 282
 
283
-		$template_data = array( 'GRP_ID' => $this->_GRP_ID );
283
+		$template_data = array('GRP_ID' => $this->_GRP_ID);
284 284
 
285
-		foreach ( $this->_contexts as $context => $details ) {
286
-			foreach ( $this->_fields as $field => $field_type ) {
287
-				if ( $field != 'extra' ) {
285
+		foreach ($this->_contexts as $context => $details) {
286
+			foreach ($this->_fields as $field => $field_type) {
287
+				if ($field != 'extra') {
288 288
 					$template_data['MTP_context'] = $context;
289 289
 					$template_data['MTP_template_field'] = $field;
290 290
 					$template_data['MTP_content'] = $this->_templates[$context][$field];
291 291
 
292 292
 					$MTP = EEM_Message_Template::instance()->insert($template_data);
293
-					if ( !$MTP ) {
294
-						EE_Error::add_error( sprintf(__('There was an error in saving new template data for %s messenger, %s message type, %s context and %s template field.', 'event_espresso'), $this->_messenger->name, $this->_message_type->name, $context, $field), __FILE__, __FUNCTION__, __LINE__  );
293
+					if ( ! $MTP) {
294
+						EE_Error::add_error(sprintf(__('There was an error in saving new template data for %s messenger, %s message type, %s context and %s template field.', 'event_espresso'), $this->_messenger->name, $this->_message_type->name, $context, $field), __FILE__, __FUNCTION__, __LINE__);
295 295
 						return false;
296 296
 					}
297 297
 				}
Please login to merge, or discard this patch.
libraries/messages/defaults/EE_Messages_Template_Pack_Default.class.php 2 patches
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -21,8 +21,8 @@  discard block
 block discarded – undo
21 21
 		$this->label = __('Default', 'event_espresso');
22 22
 		$this->dbref = 'default';
23 23
 		$this->description = __('This is the default template pack included with Event Espress core messages system.', 'event_espresso');
24
-		$this->_base_url = EE_PLUGIN_DIR_URL . 'core/libraries/messages/defaults/default/';
25
-		$this->_base_path = EE_LIBRARIES . 'messages/defaults/default/';
24
+		$this->_base_url = EE_PLUGIN_DIR_URL.'core/libraries/messages/defaults/default/';
25
+		$this->_base_path = EE_LIBRARIES.'messages/defaults/default/';
26 26
 		$this->_supports = array(
27 27
 			'email' => array(
28 28
 				'cancelled_registration', 'declined_registration', 'not_approved_registration', 'pending_approval', 'registration',
@@ -33,9 +33,9 @@  discard block
 block discarded – undo
33 33
 				'receipt', 'invoice'
34 34
 				)
35 35
 			);
36
-		$this->_default_variation_labels= array(
37
-			'email' => __('Default', 'event_espresso' ),
38
-			'html' =>  __('Simple', 'event_espresso' )
36
+		$this->_default_variation_labels = array(
37
+			'email' => __('Default', 'event_espresso'),
38
+			'html' =>  __('Simple', 'event_espresso')
39 39
 			 );
40 40
 		$this->_variations = array(
41 41
 			'html' => array(
@@ -63,18 +63,18 @@  discard block
 block discarded – undo
63 63
 
64 64
 
65 65
 
66
-	public function get_default_variation( $messenger, $message_type, $type, $url, $file_extension ) {
66
+	public function get_default_variation($messenger, $message_type, $type, $url, $file_extension) {
67 67
 		$base = $url ? $this->_base_url : $this->_base_path;
68 68
 		$base_path = $this->_base_path;
69 69
 		//possible variation paths considering whether message type is present or not in the file name.
70
-		$path_string = 'variations/' . $messenger . '_' . $message_type . '_'  . $type . '_default' . $file_extension;
71
-		$default_path_string = 'variations/' . $messenger . '_' . $type . '_default' . $file_extension;
70
+		$path_string = 'variations/'.$messenger.'_'.$message_type.'_'.$type.'_default'.$file_extension;
71
+		$default_path_string = 'variations/'.$messenger.'_'.$type.'_default'.$file_extension;
72 72
 		//first see if fully validated file exists.
73
-		if ( is_readable( $base_path . $path_string ) ) {
74
-			$variation_path = $base . $path_string;
73
+		if (is_readable($base_path.$path_string)) {
74
+			$variation_path = $base.$path_string;
75 75
 		//otherwise see if default exists.
76
-		} elseif ( is_readable( $base_path . $default_path_string ) ) {
77
-			$variation_path = $base . $default_path_string;
76
+		} elseif (is_readable($base_path.$default_path_string)) {
77
+			$variation_path = $base.$default_path_string;
78 78
 		} else {
79 79
 			//no matches found so nothing is present.
80 80
 			$variation_path = '';
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -5,7 +5,9 @@
 block discarded – undo
5 5
  * @subpackage plugin api, messages
6 6
  * @since           4.5.0
7 7
  */
8
-if ( ! defined('EVENT_ESPRESSO_VERSION')) exit('No direct script access allowed');
8
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
9
+	exit('No direct script access allowed');
10
+}
9 11
 
10 12
 /**
11 13
  * Use this to register or deregister a new message template pack variation for the EE messages system.
Please login to merge, or discard this patch.
messages/defaults/default/email_cancelled_registration_content.template.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -31,9 +31,9 @@  discard block
 block discarded – undo
31 31
 						<tbody>
32 32
 							<tr>
33 33
 								<td>
34
-									<h2><?php printf( __( 'Hello, %s', 'event_espresso' ), '[RECIPIENT_FNAME]' ); ?>:</h2>
34
+									<h2><?php printf(__('Hello, %s', 'event_espresso'), '[RECIPIENT_FNAME]'); ?>:</h2>
35 35
 
36
-									<p class="lead"><?php _e( 'Your registration was cancelled for the following tickets:', 'event_espresso' ); ?></p>
36
+									<p class="lead"><?php _e('Your registration was cancelled for the following tickets:', 'event_espresso'); ?></p>
37 37
 								</td>
38 38
 							</tr>
39 39
 						</tbody>
@@ -62,10 +62,10 @@  discard block
 block discarded – undo
62 62
 									<tbody>
63 63
 										<tr>
64 64
 											<td>
65
-												<h5><?php _e( 'Connect with Us:', 'event_espresso' ); ?></h5>
66
-												<a class="soc-btn fb" href="[CO_FACEBOOK_URL]"><?php _e( 'Facebook', 'event_espresso' ); ?></a>
67
-												<a class="soc-btn tw" href="[CO_TWITTER_URL]"><?php _e( 'Twitter', 'event_espresso' ); ?></a>
68
-												<a class="soc-btn gp" href="[CO_GOOGLE_URL]"><?php _e( 'Google+', 'event_espresso' ); ?></a>
65
+												<h5><?php _e('Connect with Us:', 'event_espresso'); ?></h5>
66
+												<a class="soc-btn fb" href="[CO_FACEBOOK_URL]"><?php _e('Facebook', 'event_espresso'); ?></a>
67
+												<a class="soc-btn tw" href="[CO_TWITTER_URL]"><?php _e('Twitter', 'event_espresso'); ?></a>
68
+												<a class="soc-btn gp" href="[CO_GOOGLE_URL]"><?php _e('Google+', 'event_espresso'); ?></a>
69 69
 											</td>
70 70
 										</tr>
71 71
 									</tbody>
@@ -75,9 +75,9 @@  discard block
 block discarded – undo
75 75
 									<tbody>
76 76
 										<tr>
77 77
 											<td>
78
-												<h5><?php _e( 'Contact Info:', 'event_espresso' ); ?></h5>
79
-												<?php _e( 'Phone:', 'event_espresso' ); ?> <strong>[CO_PHONE]</strong>
80
-												<?php _e( 'Email:', 'event_espresso' ); ?>
78
+												<h5><?php _e('Contact Info:', 'event_espresso'); ?></h5>
79
+												<?php _e('Phone:', 'event_espresso'); ?> <strong>[CO_PHONE]</strong>
80
+												<?php _e('Email:', 'event_espresso'); ?>
81 81
 												<strong><a href="mailto:[CO_EMAIL]" target="_blank">[CO_EMAIL]</a></strong>
82 82
 											</td>
83 83
 										</tr>
Please login to merge, or discard this patch.
defaults/default/email_cancelled_registration_content_admin.template.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -31,9 +31,9 @@  discard block
 block discarded – undo
31 31
 						<tbody>
32 32
 							<tr>
33 33
 								<td>
34
-									<h2><?php printf( __( 'Hello, %s:', 'event_espresso' ), '[EVENT_AUTHOR_FNAME]' ); ?></h2>
34
+									<h2><?php printf(__('Hello, %s:', 'event_espresso'), '[EVENT_AUTHOR_FNAME]'); ?></h2>
35 35
 
36
-									<p class="lead"><?php _e( 'Registrations were cancelled for the following events.', 'event_espresso' ); ?></p>
36
+									<p class="lead"><?php _e('Registrations were cancelled for the following events.', 'event_espresso'); ?></p>
37 37
 								</td>
38 38
 							</tr>
39 39
 						</tbody>
@@ -59,10 +59,10 @@  discard block
 block discarded – undo
59 59
 									<tbody>
60 60
 										<tr>
61 61
 											<td>
62
-												<h5><?php _e( 'Connect with Us:', 'event_espresso' ); ?></h5>
63
-												<a class="soc-btn fb" href="[CO_FACEBOOK_URL]"><?php _e( 'Facebook', 'event_espresso' ); ?></a>
64
-												<a class="soc-btn tw" href="[CO_TWITTER_URL]"><?php _e( 'Twitter', 'event_espresso' ); ?></a>
65
-												<a class="soc-btn gp" href="[CO_GOOGLE_URL]"><?php _e( 'Google+', 'event_espresso' ); ?></a>
62
+												<h5><?php _e('Connect with Us:', 'event_espresso'); ?></h5>
63
+												<a class="soc-btn fb" href="[CO_FACEBOOK_URL]"><?php _e('Facebook', 'event_espresso'); ?></a>
64
+												<a class="soc-btn tw" href="[CO_TWITTER_URL]"><?php _e('Twitter', 'event_espresso'); ?></a>
65
+												<a class="soc-btn gp" href="[CO_GOOGLE_URL]"><?php _e('Google+', 'event_espresso'); ?></a>
66 66
 											</td>
67 67
 										</tr>
68 68
 									</tbody>
@@ -72,9 +72,9 @@  discard block
 block discarded – undo
72 72
 									<tbody>
73 73
 										<tr>
74 74
 											<td>
75
-												<h5><?php _e( 'Contact Info:', 'event_espresso' ); ?></h5>
76
-												<?php _e( 'Phone:', 'event_espresso' ); ?> <strong>[CO_PHONE]</strong>
77
-												<?php _e( 'Email:', 'event_espresso' ); ?>
75
+												<h5><?php _e('Contact Info:', 'event_espresso'); ?></h5>
76
+												<?php _e('Phone:', 'event_espresso'); ?> <strong>[CO_PHONE]</strong>
77
+												<?php _e('Email:', 'event_espresso'); ?>
78 78
 												<strong><a href="mailto:[CO_EMAIL]" target="_blank">[CO_EMAIL]</a></strong>
79 79
 											</td>
80 80
 										</tr>
Please login to merge, or discard this patch.