@@ -94,7 +94,7 @@ discard block |
||
94 | 94 | /** |
95 | 95 | * get Group ID |
96 | 96 | * @access public |
97 | - * @return int |
|
97 | + * @return boolean |
|
98 | 98 | */ |
99 | 99 | public function GRP_ID() { |
100 | 100 | return $this->get( 'GRP_ID' ); |
@@ -139,7 +139,7 @@ discard block |
||
139 | 139 | |
140 | 140 | /** |
141 | 141 | * returns the name saved in the db for this template |
142 | - * @return string |
|
142 | + * @return boolean |
|
143 | 143 | */ |
144 | 144 | public function name() { |
145 | 145 | return $this->get( 'MTP_name' ); |
@@ -149,7 +149,7 @@ discard block |
||
149 | 149 | |
150 | 150 | /** |
151 | 151 | * Returns the description saved in the db for this template group |
152 | - * @return string |
|
152 | + * @return boolean |
|
153 | 153 | */ |
154 | 154 | public function description() { |
155 | 155 | return $this->get( 'MTP_description' ); |
@@ -171,7 +171,7 @@ discard block |
||
171 | 171 | /** |
172 | 172 | * get Message Messenger |
173 | 173 | * @access public |
174 | - * @return string |
|
174 | + * @return boolean |
|
175 | 175 | */ |
176 | 176 | public function messenger() { |
177 | 177 | return $this->get( 'MTP_messenger' ); |
@@ -208,7 +208,7 @@ discard block |
||
208 | 208 | * get Message Type |
209 | 209 | * |
210 | 210 | * @access public |
211 | - * @return string |
|
211 | + * @return boolean |
|
212 | 212 | */ |
213 | 213 | public function message_type() { |
214 | 214 | return $this->get( 'MTP_message_type' ); |
@@ -224,7 +224,7 @@ discard block |
||
224 | 224 | * appropriately. |
225 | 225 | * |
226 | 226 | * @throws EE_Error |
227 | - * @return EE_message_type|false if exception thrown. |
|
227 | + * @return null|EE_message_type if exception thrown. |
|
228 | 228 | */ |
229 | 229 | public function message_type_obj() { |
230 | 230 | $message_type = $this->message_type(); |
@@ -1,5 +1,5 @@ discard block |
||
1 | -<?php if ( !defined( 'EVENT_ESPRESSO_VERSION' ) ) { |
|
2 | - exit( 'No direct script access allowed' ); |
|
1 | +<?php if ( ! defined('EVENT_ESPRESSO_VERSION')) { |
|
2 | + exit('No direct script access allowed'); |
|
3 | 3 | } |
4 | 4 | /** |
5 | 5 | * Event Espresso |
@@ -34,9 +34,9 @@ discard block |
||
34 | 34 | * @param string $timezone |
35 | 35 | * @return EE_Message_Template_Group|mixed |
36 | 36 | */ |
37 | - public static function new_instance( $props_n_values = array(), $timezone = '' ) { |
|
38 | - $has_object = parent::_check_for_object( $props_n_values, __CLASS__, $timezone ); |
|
39 | - return $has_object ? $has_object : new self( $props_n_values, FALSE, $timezone ); |
|
37 | + public static function new_instance($props_n_values = array(), $timezone = '') { |
|
38 | + $has_object = parent::_check_for_object($props_n_values, __CLASS__, $timezone); |
|
39 | + return $has_object ? $has_object : new self($props_n_values, FALSE, $timezone); |
|
40 | 40 | } |
41 | 41 | |
42 | 42 | |
@@ -46,8 +46,8 @@ discard block |
||
46 | 46 | * @param string $timezone |
47 | 47 | * @return EE_Message_Template_Group |
48 | 48 | */ |
49 | - public static function new_instance_from_db( $props_n_values = array(), $timezone = '' ) { |
|
50 | - return new self( $props_n_values, TRUE, $timezone ); |
|
49 | + public static function new_instance_from_db($props_n_values = array(), $timezone = '') { |
|
50 | + return new self($props_n_values, TRUE, $timezone); |
|
51 | 51 | } |
52 | 52 | |
53 | 53 | |
@@ -56,11 +56,11 @@ discard block |
||
56 | 56 | * @param bool $message_type |
57 | 57 | * @throws EE_Error |
58 | 58 | */ |
59 | - public function set_message_type( $message_type = FALSE ) { |
|
60 | - if ( ! $message_type ) { |
|
61 | - throw new EE_Error( __( 'Missing required value for the message_type parameter', 'event_espresso' ) ); |
|
59 | + public function set_message_type($message_type = FALSE) { |
|
60 | + if ( ! $message_type) { |
|
61 | + throw new EE_Error(__('Missing required value for the message_type parameter', 'event_espresso')); |
|
62 | 62 | } |
63 | - $this->set( 'MTP_message_type', $message_type ); |
|
63 | + $this->set('MTP_message_type', $message_type); |
|
64 | 64 | } |
65 | 65 | |
66 | 66 | |
@@ -69,11 +69,11 @@ discard block |
||
69 | 69 | * @param bool $messenger |
70 | 70 | * @throws EE_Error |
71 | 71 | */ |
72 | - public function set_messenger( $messenger = FALSE ) { |
|
73 | - if ( ! $messenger ) { |
|
74 | - throw new EE_Error( __( 'Missing required value for the messenger parameter', 'event_espresso' ) ); |
|
72 | + public function set_messenger($messenger = FALSE) { |
|
73 | + if ( ! $messenger) { |
|
74 | + throw new EE_Error(__('Missing required value for the messenger parameter', 'event_espresso')); |
|
75 | 75 | } |
76 | - $this->set( 'MTP_messenger', $messenger ); |
|
76 | + $this->set('MTP_messenger', $messenger); |
|
77 | 77 | } |
78 | 78 | |
79 | 79 | |
@@ -82,11 +82,11 @@ discard block |
||
82 | 82 | * @param bool $GRP_ID |
83 | 83 | * @throws EE_Error |
84 | 84 | */ |
85 | - public function set_group_template_id( $GRP_ID = FALSE ) { |
|
86 | - if ( ! $GRP_ID ) { |
|
87 | - throw new EE_Error( __( 'Missing required value for the message template group id', 'event_espresso' ) ); |
|
85 | + public function set_group_template_id($GRP_ID = FALSE) { |
|
86 | + if ( ! $GRP_ID) { |
|
87 | + throw new EE_Error(__('Missing required value for the message template group id', 'event_espresso')); |
|
88 | 88 | } |
89 | - $this->set( 'GRP_ID', $GRP_ID ); |
|
89 | + $this->set('GRP_ID', $GRP_ID); |
|
90 | 90 | } |
91 | 91 | |
92 | 92 | |
@@ -97,7 +97,7 @@ discard block |
||
97 | 97 | * @return int |
98 | 98 | */ |
99 | 99 | public function GRP_ID() { |
100 | - return $this->get( 'GRP_ID' ); |
|
100 | + return $this->get('GRP_ID'); |
|
101 | 101 | } |
102 | 102 | |
103 | 103 | |
@@ -108,8 +108,8 @@ discard block |
||
108 | 108 | * @return int |
109 | 109 | */ |
110 | 110 | public function user() { |
111 | - $user_id = $this->get( 'MTP_user_id' ); |
|
112 | - return empty( $user_id ) ? get_current_user_id() : $user_id; |
|
111 | + $user_id = $this->get('MTP_user_id'); |
|
112 | + return empty($user_id) ? get_current_user_id() : $user_id; |
|
113 | 113 | } |
114 | 114 | |
115 | 115 | |
@@ -132,7 +132,7 @@ discard block |
||
132 | 132 | * @return int |
133 | 133 | */ |
134 | 134 | public function count_events() { |
135 | - return $this->count_related( 'Event' ); |
|
135 | + return $this->count_related('Event'); |
|
136 | 136 | } |
137 | 137 | |
138 | 138 | |
@@ -142,7 +142,7 @@ discard block |
||
142 | 142 | * @return string |
143 | 143 | */ |
144 | 144 | public function name() { |
145 | - return $this->get( 'MTP_name' ); |
|
145 | + return $this->get('MTP_name'); |
|
146 | 146 | } |
147 | 147 | |
148 | 148 | |
@@ -152,7 +152,7 @@ discard block |
||
152 | 152 | * @return string |
153 | 153 | */ |
154 | 154 | public function description() { |
155 | - return $this->get( 'MTP_description' ); |
|
155 | + return $this->get('MTP_description'); |
|
156 | 156 | } |
157 | 157 | |
158 | 158 | |
@@ -162,8 +162,8 @@ discard block |
||
162 | 162 | * @param array $query_params like EEM_Base::get_all() |
163 | 163 | * @return EE_Message_Template[] |
164 | 164 | */ |
165 | - public function message_templates( $query_params = array() ) { |
|
166 | - return $this->get_many_related( 'Message_Template', $query_params ); |
|
165 | + public function message_templates($query_params = array()) { |
|
166 | + return $this->get_many_related('Message_Template', $query_params); |
|
167 | 167 | } |
168 | 168 | |
169 | 169 | |
@@ -174,7 +174,7 @@ discard block |
||
174 | 174 | * @return string |
175 | 175 | */ |
176 | 176 | public function messenger() { |
177 | - return $this->get( 'MTP_messenger' ); |
|
177 | + return $this->get('MTP_messenger'); |
|
178 | 178 | } |
179 | 179 | |
180 | 180 | |
@@ -190,12 +190,12 @@ discard block |
||
190 | 190 | */ |
191 | 191 | public function messenger_obj() { |
192 | 192 | $messenger = $this->messenger(); |
193 | - EE_Registry::instance()->load_helper( 'MSG_Template' ); |
|
193 | + EE_Registry::instance()->load_helper('MSG_Template'); |
|
194 | 194 | try { |
195 | - $messenger = EEH_MSG_Template::messenger_obj( $messenger ); |
|
196 | - } catch( EE_Error $e ) { |
|
195 | + $messenger = EEH_MSG_Template::messenger_obj($messenger); |
|
196 | + } catch (EE_Error $e) { |
|
197 | 197 | //if an exception was thrown then let's deactivate this message template group because it means there is no class for this messenger in this group. |
198 | - $this->set( 'MTP_is_active', false ); |
|
198 | + $this->set('MTP_is_active', false); |
|
199 | 199 | $this->save(); |
200 | 200 | return null; |
201 | 201 | } |
@@ -211,7 +211,7 @@ discard block |
||
211 | 211 | * @return string |
212 | 212 | */ |
213 | 213 | public function message_type() { |
214 | - return $this->get( 'MTP_message_type' ); |
|
214 | + return $this->get('MTP_message_type'); |
|
215 | 215 | } |
216 | 216 | |
217 | 217 | |
@@ -228,12 +228,12 @@ discard block |
||
228 | 228 | */ |
229 | 229 | public function message_type_obj() { |
230 | 230 | $message_type = $this->message_type(); |
231 | - EE_Registry::instance()->load_helper( 'MSG_Template' ); |
|
231 | + EE_Registry::instance()->load_helper('MSG_Template'); |
|
232 | 232 | try { |
233 | - $message_type = EEH_MSG_Template::message_type_obj( $message_type ); |
|
234 | - } catch(EE_Error $e) { |
|
233 | + $message_type = EEH_MSG_Template::message_type_obj($message_type); |
|
234 | + } catch (EE_Error $e) { |
|
235 | 235 | //if an exception was thrown then let's deactivate this message template group because it means there is no class for the message type in this group. |
236 | - $this->set( 'MTP_is_active', false ); |
|
236 | + $this->set('MTP_is_active', false); |
|
237 | 237 | $this->save(); |
238 | 238 | return null; |
239 | 239 | } |
@@ -270,13 +270,13 @@ discard block |
||
270 | 270 | */ |
271 | 271 | public function context_templates() { |
272 | 272 | $mtps_arr = array(); |
273 | - $mtps = $this->get_many_related( 'Message_Template' ); |
|
274 | - if ( empty( $mtps ) ) { |
|
273 | + $mtps = $this->get_many_related('Message_Template'); |
|
274 | + if (empty($mtps)) { |
|
275 | 275 | return array(); |
276 | 276 | } |
277 | 277 | //note contexts could have CHECKBOX fields per context. So we return the objects indexed by context AND field. |
278 | - foreach ( $mtps as $mtp ) { |
|
279 | - $mtps_arr[ $mtp->get( 'MTP_context' ) ][ $mtp->get( 'MTP_template_field' ) ] = $mtp; |
|
278 | + foreach ($mtps as $mtp) { |
|
279 | + $mtps_arr[$mtp->get('MTP_context')][$mtp->get('MTP_template_field')] = $mtp; |
|
280 | 280 | } |
281 | 281 | return $mtps_arr; |
282 | 282 | } |
@@ -288,7 +288,7 @@ discard block |
||
288 | 288 | * @return boolean true if it is, false if it isn't |
289 | 289 | */ |
290 | 290 | public function is_global() { |
291 | - return $this->get( 'MTP_is_global' ); |
|
291 | + return $this->get('MTP_is_global'); |
|
292 | 292 | } |
293 | 293 | |
294 | 294 | |
@@ -298,7 +298,7 @@ discard block |
||
298 | 298 | * @return boolean true if it is, false if it isn't |
299 | 299 | */ |
300 | 300 | public function is_active() { |
301 | - return $this->get( 'MTP_is_active' ); |
|
301 | + return $this->get('MTP_is_active'); |
|
302 | 302 | } |
303 | 303 | |
304 | 304 | |
@@ -314,11 +314,11 @@ discard block |
||
314 | 314 | * @param bool $merged If TRUE then we don't return shortcodes indexed by field but instead an array of the unique shortcodes for all the given (or all) fields. |
315 | 315 | * @return mixed (array|bool) an array of shortcodes in the format array( '[shortcode] => 'label') OR FALSE if no shortcodes found. |
316 | 316 | */ |
317 | - public function get_shortcodes( $context, $fields = array(), $merged = FALSE ) { |
|
317 | + public function get_shortcodes($context, $fields = array(), $merged = FALSE) { |
|
318 | 318 | $messenger = $this->messenger(); |
319 | 319 | $message_type = $this->message_type(); |
320 | - EE_Registry::instance()->load_helper( 'MSG_Template' ); |
|
321 | - return EEH_MSG_Template::get_shortcodes( $message_type, $messenger, $fields, $context, $merged ); |
|
320 | + EE_Registry::instance()->load_helper('MSG_Template'); |
|
321 | + return EEH_MSG_Template::get_shortcodes($message_type, $messenger, $fields, $context, $merged); |
|
322 | 322 | } |
323 | 323 | |
324 | 324 | |
@@ -331,20 +331,20 @@ discard block |
||
331 | 331 | * @throws EE_Error |
332 | 332 | * @return array an array of EE_Shortcode objects |
333 | 333 | */ |
334 | - private function _get_shortcode_objects( $sc_refs ) { |
|
334 | + private function _get_shortcode_objects($sc_refs) { |
|
335 | 335 | $sc_objs = array(); |
336 | 336 | EED_Messages::set_autoloaders(); |
337 | - foreach ( $sc_refs as $shortcode_ref ) { |
|
338 | - $ref = ucwords( str_replace( '_', ' ', $shortcode_ref ) ); |
|
339 | - $ref = str_replace( ' ', '_', $ref ); |
|
340 | - $classname = 'EE_' . $ref . '_Shortcodes'; |
|
341 | - if ( ! class_exists( $classname ) ) { |
|
342 | - $msg[ ] = __( 'Shortcode library loading fail.', 'event_espresso' ); |
|
343 | - $msg[ ] = sprintf( __( 'The class name checked was "%s". Please check the spelling and case of this reference and make sure it matches the appropriate shortcode library file name (minus the extension) in the "/library/shortcodes/" directory', 'event_espresso' ), $classname ); |
|
344 | - throw new EE_Error( implode( '||', $msg ) ); |
|
337 | + foreach ($sc_refs as $shortcode_ref) { |
|
338 | + $ref = ucwords(str_replace('_', ' ', $shortcode_ref)); |
|
339 | + $ref = str_replace(' ', '_', $ref); |
|
340 | + $classname = 'EE_'.$ref.'_Shortcodes'; |
|
341 | + if ( ! class_exists($classname)) { |
|
342 | + $msg[] = __('Shortcode library loading fail.', 'event_espresso'); |
|
343 | + $msg[] = sprintf(__('The class name checked was "%s". Please check the spelling and case of this reference and make sure it matches the appropriate shortcode library file name (minus the extension) in the "/library/shortcodes/" directory', 'event_espresso'), $classname); |
|
344 | + throw new EE_Error(implode('||', $msg)); |
|
345 | 345 | } |
346 | - $a = new ReflectionClass( $classname ); |
|
347 | - $sc_objs[ ] = $a->newInstance(); |
|
346 | + $a = new ReflectionClass($classname); |
|
347 | + $sc_objs[] = $a->newInstance(); |
|
348 | 348 | } |
349 | 349 | return $sc_objs; |
350 | 350 | } |
@@ -359,7 +359,7 @@ discard block |
||
359 | 359 | * @return string |
360 | 360 | */ |
361 | 361 | public function get_template_pack_name() { |
362 | - return $this->get_extra_meta( 'MTP_template_pack', TRUE, 'default' ); |
|
362 | + return $this->get_extra_meta('MTP_template_pack', TRUE, 'default'); |
|
363 | 363 | } |
364 | 364 | |
365 | 365 | |
@@ -375,7 +375,7 @@ discard block |
||
375 | 375 | public function get_template_pack() { |
376 | 376 | $pack_name = $this->get_template_pack_name(); |
377 | 377 | |
378 | - return EED_Messages::get_template_pack( $pack_name ); |
|
378 | + return EED_Messages::get_template_pack($pack_name); |
|
379 | 379 | } |
380 | 380 | |
381 | 381 | |
@@ -388,7 +388,7 @@ discard block |
||
388 | 388 | * @return string |
389 | 389 | */ |
390 | 390 | public function get_template_pack_variation() { |
391 | - return $this->get_extra_meta( 'MTP_variation', TRUE, 'default' ); |
|
391 | + return $this->get_extra_meta('MTP_variation', TRUE, 'default'); |
|
392 | 392 | } |
393 | 393 | |
394 | 394 | |
@@ -401,8 +401,8 @@ discard block |
||
401 | 401 | * |
402 | 402 | * @param string $template_pack_name What message template pack is assigned. |
403 | 403 | */ |
404 | - public function set_template_pack_name( $template_pack_name ) { |
|
405 | - return $this->update_extra_meta( 'MTP_template_pack', $template_pack_name ); |
|
404 | + public function set_template_pack_name($template_pack_name) { |
|
405 | + return $this->update_extra_meta('MTP_template_pack', $template_pack_name); |
|
406 | 406 | } |
407 | 407 | |
408 | 408 | |
@@ -415,8 +415,8 @@ discard block |
||
415 | 415 | * |
416 | 416 | * @param string $variation What variation is being set on the message template group. |
417 | 417 | */ |
418 | - public function set_template_pack_variation( $variation ) { |
|
419 | - return $this->update_extra_meta( 'MTP_variation', $variation ); |
|
418 | + public function set_template_pack_variation($variation) { |
|
419 | + return $this->update_extra_meta('MTP_variation', $variation); |
|
420 | 420 | } |
421 | 421 | } |
422 | 422 | //end EE_Message_Template_Group class |
@@ -112,7 +112,7 @@ discard block |
||
112 | 112 | |
113 | 113 | /** |
114 | 114 | * Gets button_url |
115 | - * @return string |
|
115 | + * @return boolean |
|
116 | 116 | */ |
117 | 117 | function button_url() { |
118 | 118 | return $this->get('PMD_button_url'); |
@@ -152,7 +152,7 @@ discard block |
||
152 | 152 | |
153 | 153 | /** |
154 | 154 | * Gets description |
155 | - * @return string |
|
155 | + * @return boolean |
|
156 | 156 | */ |
157 | 157 | function description() { |
158 | 158 | return $this->get('PMD_desc'); |
@@ -172,7 +172,7 @@ discard block |
||
172 | 172 | |
173 | 173 | /** |
174 | 174 | * Gets name |
175 | - * @return string |
|
175 | + * @return boolean |
|
176 | 176 | */ |
177 | 177 | function name() { |
178 | 178 | return $this->get('PMD_name'); |
@@ -212,7 +212,7 @@ discard block |
||
212 | 212 | |
213 | 213 | /** |
214 | 214 | * Gets order |
215 | - * @return int |
|
215 | + * @return boolean |
|
216 | 216 | */ |
217 | 217 | function order() { |
218 | 218 | return $this->get('PMD_order'); |
@@ -232,7 +232,7 @@ discard block |
||
232 | 232 | |
233 | 233 | /** |
234 | 234 | * Gets slug |
235 | - * @return string |
|
235 | + * @return boolean |
|
236 | 236 | */ |
237 | 237 | function slug() { |
238 | 238 | return $this->get('PMD_slug'); |
@@ -252,7 +252,7 @@ discard block |
||
252 | 252 | |
253 | 253 | /** |
254 | 254 | * Gets type |
255 | - * @return string |
|
255 | + * @return boolean |
|
256 | 256 | */ |
257 | 257 | function type() { |
258 | 258 | return $this->get('PMD_type'); |
@@ -272,7 +272,7 @@ discard block |
||
272 | 272 | |
273 | 273 | /** |
274 | 274 | * Gets wp_user |
275 | - * @return int |
|
275 | + * @return boolean |
|
276 | 276 | */ |
277 | 277 | function wp_user() { |
278 | 278 | return $this->get('PMD_wp_user'); |
@@ -291,7 +291,7 @@ discard block |
||
291 | 291 | |
292 | 292 | /** |
293 | 293 | * Overrides parent so when PMD_type is changed we refresh the _type_obj |
294 | - * @param type $field_name |
|
294 | + * @param string $field_name |
|
295 | 295 | * @param type $field_value |
296 | 296 | * @param type $use_default |
297 | 297 | */ |
@@ -307,7 +307,7 @@ discard block |
||
307 | 307 | |
308 | 308 | /** |
309 | 309 | * Gets admin_name |
310 | - * @return string |
|
310 | + * @return boolean |
|
311 | 311 | */ |
312 | 312 | function admin_name() { |
313 | 313 | return $this->get('PMD_admin_name'); |
@@ -327,7 +327,7 @@ discard block |
||
327 | 327 | |
328 | 328 | /** |
329 | 329 | * Gets admin_desc |
330 | - * @return string |
|
330 | + * @return boolean |
|
331 | 331 | */ |
332 | 332 | function admin_desc() { |
333 | 333 | return $this->get('PMD_admin_desc'); |
@@ -347,7 +347,7 @@ discard block |
||
347 | 347 | |
348 | 348 | /** |
349 | 349 | * Gets scope |
350 | - * @return array |
|
350 | + * @return boolean |
|
351 | 351 | */ |
352 | 352 | function scope() { |
353 | 353 | return $this->get('PMD_scope'); |
@@ -1,5 +1,5 @@ discard block |
||
1 | -<?php if (!defined('EVENT_ESPRESSO_VERSION')) exit('No direct script access allowed'); |
|
2 | -do_action( 'AHEE_log', __FILE__, __FUNCTION__, '' ); |
|
1 | +<?php if ( ! defined('EVENT_ESPRESSO_VERSION')) exit('No direct script access allowed'); |
|
2 | +do_action('AHEE_log', __FILE__, __FUNCTION__, ''); |
|
3 | 3 | /** |
4 | 4 | * Event Espresso |
5 | 5 | * |
@@ -23,7 +23,7 @@ discard block |
||
23 | 23 | * |
24 | 24 | * ------------------------------------------------------------------------ |
25 | 25 | */ |
26 | -class EE_Payment_Method extends EE_Base_Class{ |
|
26 | +class EE_Payment_Method extends EE_Base_Class { |
|
27 | 27 | |
28 | 28 | /** |
29 | 29 | * Payment Method type object, which has all the info about this type of payment method, |
@@ -39,9 +39,9 @@ discard block |
||
39 | 39 | * @param array $props_n_values |
40 | 40 | * @return EE_Payment_Method |
41 | 41 | */ |
42 | - public static function new_instance( $props_n_values = array()) { |
|
43 | - $has_object = parent::_check_for_object( $props_n_values, __CLASS__ ); |
|
44 | - return $has_object ? $has_object : new self( $props_n_values, FALSE ); |
|
42 | + public static function new_instance($props_n_values = array()) { |
|
43 | + $has_object = parent::_check_for_object($props_n_values, __CLASS__); |
|
44 | + return $has_object ? $has_object : new self($props_n_values, FALSE); |
|
45 | 45 | } |
46 | 46 | |
47 | 47 | |
@@ -51,8 +51,8 @@ discard block |
||
51 | 51 | * @param array $props_n_values |
52 | 52 | * @return EE_Payment_Method |
53 | 53 | */ |
54 | - public static function new_instance_from_db ( $props_n_values = array()) { |
|
55 | - return new self( $props_n_values, TRUE ); |
|
54 | + public static function new_instance_from_db($props_n_values = array()) { |
|
55 | + return new self($props_n_values, TRUE); |
|
56 | 56 | } |
57 | 57 | |
58 | 58 | |
@@ -81,7 +81,7 @@ discard block |
||
81 | 81 | * @return boolean |
82 | 82 | */ |
83 | 83 | function active() { |
84 | - return array_intersect(array_keys(EEM_Payment_Method::instance()->scopes()),$this->scope()); |
|
84 | + return array_intersect(array_keys(EEM_Payment_Method::instance()->scopes()), $this->scope()); |
|
85 | 85 | } |
86 | 86 | |
87 | 87 | |
@@ -90,10 +90,10 @@ discard block |
||
90 | 90 | * Sets this PM as active by making it usable within the CART scope. Offline gateways |
91 | 91 | * are also usable from the admin-scope as well. DOES NOT SAVE it |
92 | 92 | */ |
93 | - function set_active(){ |
|
93 | + function set_active() { |
|
94 | 94 | $default_scopes = array(EEM_Payment_Method::scope_cart); |
95 | - if($this->type_obj() && |
|
96 | - $this->type_obj()->payment_occurs() == EE_PMT_Base::offline){ |
|
95 | + if ($this->type_obj() && |
|
96 | + $this->type_obj()->payment_occurs() == EE_PMT_Base::offline) { |
|
97 | 97 | $default_scopes[] = EEM_Payment_Method::scope_admin; |
98 | 98 | } |
99 | 99 | $this->set_scope($default_scopes); |
@@ -104,7 +104,7 @@ discard block |
||
104 | 104 | /** |
105 | 105 | * Makes this payment method apply to NO scopes at all. DOES NOT SAVE it. |
106 | 106 | */ |
107 | - function deactivate(){ |
|
107 | + function deactivate() { |
|
108 | 108 | $this->set_scope(array()); |
109 | 109 | } |
110 | 110 | |
@@ -295,8 +295,8 @@ discard block |
||
295 | 295 | * @param type $field_value |
296 | 296 | * @param type $use_default |
297 | 297 | */ |
298 | - function set( $field_name, $field_value, $use_default = FALSE ){ |
|
299 | - if( $field_name == 'PMD_type' ){ |
|
298 | + function set($field_name, $field_value, $use_default = FALSE) { |
|
299 | + if ($field_name == 'PMD_type') { |
|
300 | 300 | //the type has probably changed, so forget about its old type object |
301 | 301 | $this->_type_obj = NULL; |
302 | 302 | } |
@@ -370,18 +370,18 @@ discard block |
||
370 | 370 | * @return EE_PMT_Base |
371 | 371 | * @throws EE_Error |
372 | 372 | */ |
373 | - public function type_obj(){ |
|
374 | - if( ! $this->_type_obj ) { |
|
375 | - EE_Registry::instance()->load_lib( 'Payment_Method_Manager' ); |
|
376 | - if ( EE_Payment_Method_Manager::instance()->payment_method_type_exists( $this->type() )) { |
|
377 | - $class_name = EE_Payment_Method_Manager::instance()->payment_method_class_from_type( $this->type() ); |
|
378 | - if ( ! class_exists( $class_name )) { |
|
379 | - throw new EE_Error(sprintf(__("There is no payment method type of class '%s', did you deactivate an EE addon?", "event_espresso"),$class_name)); |
|
373 | + public function type_obj() { |
|
374 | + if ( ! $this->_type_obj) { |
|
375 | + EE_Registry::instance()->load_lib('Payment_Method_Manager'); |
|
376 | + if (EE_Payment_Method_Manager::instance()->payment_method_type_exists($this->type())) { |
|
377 | + $class_name = EE_Payment_Method_Manager::instance()->payment_method_class_from_type($this->type()); |
|
378 | + if ( ! class_exists($class_name)) { |
|
379 | + throw new EE_Error(sprintf(__("There is no payment method type of class '%s', did you deactivate an EE addon?", "event_espresso"), $class_name)); |
|
380 | 380 | } |
381 | - $r = new ReflectionClass( $class_name ); |
|
382 | - $this->_type_obj = $r->newInstanceArgs( array( $this )); |
|
381 | + $r = new ReflectionClass($class_name); |
|
382 | + $this->_type_obj = $r->newInstanceArgs(array($this)); |
|
383 | 383 | } else { |
384 | - throw new EE_Error( sprintf( __( 'A payment method of type "%1$s" does not exist. Only ones existing are: %2$s', 'event_espresso' ), $this->type(), implode(',', EE_Payment_Method_Manager::instance()->payment_method_type_names() ) ) ); |
|
384 | + throw new EE_Error(sprintf(__('A payment method of type "%1$s" does not exist. Only ones existing are: %2$s', 'event_espresso'), $this->type(), implode(',', EE_Payment_Method_Manager::instance()->payment_method_type_names()))); |
|
385 | 385 | } |
386 | 386 | } |
387 | 387 | return $this->_type_obj; |
@@ -394,18 +394,18 @@ discard block |
||
394 | 394 | * and the extra meta. Mostly used for passing off ot gateways. * |
395 | 395 | * @return array |
396 | 396 | */ |
397 | - public function settings_array(){ |
|
397 | + public function settings_array() { |
|
398 | 398 | $fields = $this->model_field_array(); |
399 | 399 | $extra_meta = $this->all_extra_meta_array(); |
400 | 400 | //remove the model's prefix from the fields |
401 | 401 | $combined_settings_array = array(); |
402 | - foreach($fields as $key => $value){ |
|
403 | - if(strpos($key, 'PMD_')===0){ |
|
402 | + foreach ($fields as $key => $value) { |
|
403 | + if (strpos($key, 'PMD_') === 0) { |
|
404 | 404 | $key_sans_model_prefix = str_replace('PMD_', '', $key); |
405 | 405 | $combined_settings_array [$key_sans_model_prefix] = $value; |
406 | 406 | } |
407 | 407 | } |
408 | - $combined_settings_array = array_merge( $extra_meta,$combined_settings_array ); |
|
408 | + $combined_settings_array = array_merge($extra_meta, $combined_settings_array); |
|
409 | 409 | return $combined_settings_array; |
410 | 410 | } |
411 | 411 | |
@@ -417,12 +417,12 @@ discard block |
||
417 | 417 | * @param string $css_class |
418 | 418 | * @return string of HTML for displaying the button |
419 | 419 | */ |
420 | - public function button_html( $url = '', $css_class = '' ){ |
|
420 | + public function button_html($url = '', $css_class = '') { |
|
421 | 421 | $payment_occurs = $this->type_obj()->payment_occurs(); |
422 | 422 | return ' |
423 | - <div id="' . $this->slug() . '-payment-option-dv" class="'. $payment_occurs .'-payment-gateway reg-page-payment-option-dv' . $css_class . '"> |
|
424 | - <a id="payment-gateway-button-' . $this->slug() . '" class="reg-page-payment-option-lnk" rel="' . $this->slug() . '" href="' . $url . '" > |
|
425 | - <img src="' . $this->button_url() . '" alt="' . sprintf( esc_attr__( 'Pay using %s', 'event_espresso' ), $this->get_pretty('PMD_name','form_input') ) . '" /> |
|
423 | + <div id="' . $this->slug().'-payment-option-dv" class="'.$payment_occurs.'-payment-gateway reg-page-payment-option-dv'.$css_class.'"> |
|
424 | + <a id="payment-gateway-button-' . $this->slug().'" class="reg-page-payment-option-lnk" rel="'.$this->slug().'" href="'.$url.'" > |
|
425 | + <img src="' . $this->button_url().'" alt="'.sprintf(esc_attr__('Pay using %s', 'event_espresso'), $this->get_pretty('PMD_name', 'form_input')).'" /> |
|
426 | 426 | </a> |
427 | 427 | </div> |
428 | 428 | '; |
@@ -435,7 +435,7 @@ discard block |
||
435 | 435 | * (as defined by the gateway and the currently active currencies) |
436 | 436 | * @return EE_Currency[] |
437 | 437 | */ |
438 | - public function get_all_usable_currencies(){ |
|
438 | + public function get_all_usable_currencies() { |
|
439 | 439 | return EEM_Currency::instance()->get_all_currencies_usable_by($this->type_obj()); |
440 | 440 | } |
441 | 441 | |
@@ -444,9 +444,9 @@ discard block |
||
444 | 444 | * @param string $currency_code currency ID (code) |
445 | 445 | * @return boolean |
446 | 446 | */ |
447 | - public function usable_for_currency( $currency_code ) { |
|
448 | - foreach( $this->get_all_usable_currencies() as $currency_obj ) { |
|
449 | - if( $currency_obj->ID() == $currency_code ){ |
|
447 | + public function usable_for_currency($currency_code) { |
|
448 | + foreach ($this->get_all_usable_currencies() as $currency_obj) { |
|
449 | + if ($currency_obj->ID() == $currency_code) { |
|
450 | 450 | return TRUE; |
451 | 451 | } |
452 | 452 | } |
@@ -459,7 +459,7 @@ discard block |
||
459 | 459 | * Returns TRUE if this payment method's gateway is an instance of EE_Onsite_Gateway |
460 | 460 | * @return bool |
461 | 461 | */ |
462 | - public function is_on_site(){ |
|
462 | + public function is_on_site() { |
|
463 | 463 | return $this->type_obj()->payment_occurs() == EE_PMT_Base::onsite; |
464 | 464 | } |
465 | 465 | |
@@ -469,7 +469,7 @@ discard block |
||
469 | 469 | * Returns TRUE if this payment method's gateway is an instance of EE_Offsite_Gateway |
470 | 470 | * @return bool |
471 | 471 | */ |
472 | - public function is_off_site(){ |
|
472 | + public function is_off_site() { |
|
473 | 473 | return $this->type_obj()->payment_occurs() == EE_PMT_Base::offsite; |
474 | 474 | } |
475 | 475 | |
@@ -479,7 +479,7 @@ discard block |
||
479 | 479 | * Returns TRUE if this payment method does not utilize a gateway |
480 | 480 | * @return bool |
481 | 481 | */ |
482 | - public function is_off_line(){ |
|
482 | + public function is_off_line() { |
|
483 | 483 | return $this->type_obj()->payment_occurs() == EE_PMT_Base::offline; |
484 | 484 | } |
485 | 485 | |
@@ -489,10 +489,10 @@ discard block |
||
489 | 489 | * to load the required classes, and don't need them at the time of unserialization |
490 | 490 | * @return array |
491 | 491 | */ |
492 | - public function __sleep(){ |
|
493 | - $properties = get_object_vars( $this ); |
|
494 | - unset( $properties[ '_type_obj' ] ); |
|
495 | - return array_keys( $properties ); |
|
492 | + public function __sleep() { |
|
493 | + $properties = get_object_vars($this); |
|
494 | + unset($properties['_type_obj']); |
|
495 | + return array_keys($properties); |
|
496 | 496 | } |
497 | 497 | |
498 | 498 |
@@ -1,4 +1,6 @@ |
||
1 | -<?php if (!defined('EVENT_ESPRESSO_VERSION')) exit('No direct script access allowed'); |
|
1 | +<?php if (!defined('EVENT_ESPRESSO_VERSION')) { |
|
2 | + exit('No direct script access allowed'); |
|
3 | +} |
|
2 | 4 | do_action( 'AHEE_log', __FILE__, __FUNCTION__, '' ); |
3 | 5 | /** |
4 | 6 | * Event Espresso |
@@ -156,7 +156,7 @@ |
||
156 | 156 | * |
157 | 157 | * @since 4.5.0 |
158 | 158 | * |
159 | - * @return int |
|
159 | + * @return boolean |
|
160 | 160 | */ |
161 | 161 | public function wp_user() { |
162 | 162 | return $this->get('PRT_wp_user'); |
@@ -1,5 +1,5 @@ discard block |
||
1 | -<?php if ( ! defined( 'EVENT_ESPRESSO_VERSION' ) ) { |
|
2 | - exit( 'No direct script access allowed' ); |
|
1 | +<?php if ( ! defined('EVENT_ESPRESSO_VERSION')) { |
|
2 | + exit('No direct script access allowed'); |
|
3 | 3 | } |
4 | 4 | /** |
5 | 5 | * Event Espresso |
@@ -32,9 +32,9 @@ discard block |
||
32 | 32 | * @param array $props_n_values |
33 | 33 | * @return EE_Price_Type |
34 | 34 | */ |
35 | - public static function new_instance( $props_n_values = array() ) { |
|
36 | - $has_object = parent::_check_for_object( $props_n_values, __CLASS__ ); |
|
37 | - return $has_object ? $has_object : new self( $props_n_values ); |
|
35 | + public static function new_instance($props_n_values = array()) { |
|
36 | + $has_object = parent::_check_for_object($props_n_values, __CLASS__); |
|
37 | + return $has_object ? $has_object : new self($props_n_values); |
|
38 | 38 | } |
39 | 39 | |
40 | 40 | |
@@ -43,8 +43,8 @@ discard block |
||
43 | 43 | * @param array $props_n_values |
44 | 44 | * @return EE_Price_Type |
45 | 45 | */ |
46 | - public static function new_instance_from_db( $props_n_values = array() ) { |
|
47 | - return new self( $props_n_values, TRUE ); |
|
46 | + public static function new_instance_from_db($props_n_values = array()) { |
|
47 | + return new self($props_n_values, TRUE); |
|
48 | 48 | } |
49 | 49 | |
50 | 50 | |
@@ -55,8 +55,8 @@ discard block |
||
55 | 55 | * @access public |
56 | 56 | * @param string $PRT_name |
57 | 57 | */ |
58 | - public function set_name( $PRT_name = '' ) { |
|
59 | - $this->set( 'PRT_name', $PRT_name ); |
|
58 | + public function set_name($PRT_name = '') { |
|
59 | + $this->set('PRT_name', $PRT_name); |
|
60 | 60 | } |
61 | 61 | |
62 | 62 | |
@@ -67,8 +67,8 @@ discard block |
||
67 | 67 | * @access public |
68 | 68 | * @param bool $PRT_is_percent |
69 | 69 | */ |
70 | - public function set_is_percent( $PRT_is_percent = FALSE ) { |
|
71 | - $this->set( 'PRT_is_percent', $PRT_is_percent ); |
|
70 | + public function set_is_percent($PRT_is_percent = FALSE) { |
|
71 | + $this->set('PRT_is_percent', $PRT_is_percent); |
|
72 | 72 | } |
73 | 73 | |
74 | 74 | |
@@ -79,8 +79,8 @@ discard block |
||
79 | 79 | * @access public |
80 | 80 | * @param int $PRT_order |
81 | 81 | */ |
82 | - public function set_order( $PRT_order = 0 ) { |
|
83 | - $this->set( 'PRT_order', $PRT_order ); |
|
82 | + public function set_order($PRT_order = 0) { |
|
83 | + $this->set('PRT_order', $PRT_order); |
|
84 | 84 | } |
85 | 85 | |
86 | 86 | |
@@ -89,7 +89,7 @@ discard block |
||
89 | 89 | * |
90 | 90 | */ |
91 | 91 | public function move_to_trash() { |
92 | - $this->set( 'PRT_deleted', TRUE ); |
|
92 | + $this->set('PRT_deleted', TRUE); |
|
93 | 93 | } |
94 | 94 | |
95 | 95 | |
@@ -98,7 +98,7 @@ discard block |
||
98 | 98 | * |
99 | 99 | */ |
100 | 100 | public function restore_from_trash() { |
101 | - $this->set( 'PRT_deleted', FALSE ); |
|
101 | + $this->set('PRT_deleted', FALSE); |
|
102 | 102 | } |
103 | 103 | |
104 | 104 | |
@@ -108,7 +108,7 @@ discard block |
||
108 | 108 | * @access public |
109 | 109 | */ |
110 | 110 | public function name() { |
111 | - return $this->get( 'PRT_name' ); |
|
111 | + return $this->get('PRT_name'); |
|
112 | 112 | } |
113 | 113 | |
114 | 114 | |
@@ -118,7 +118,7 @@ discard block |
||
118 | 118 | * @access public |
119 | 119 | */ |
120 | 120 | public function base_type() { |
121 | - return $this->get( 'PBT_ID' ); |
|
121 | + return $this->get('PBT_ID'); |
|
122 | 122 | } |
123 | 123 | |
124 | 124 | |
@@ -127,7 +127,7 @@ discard block |
||
127 | 127 | * @return mixed |
128 | 128 | */ |
129 | 129 | public function base_type_name() { |
130 | - return $this->get_pretty( 'PBT_ID' ); |
|
130 | + return $this->get_pretty('PBT_ID'); |
|
131 | 131 | } |
132 | 132 | |
133 | 133 | |
@@ -137,7 +137,7 @@ discard block |
||
137 | 137 | * @access public |
138 | 138 | */ |
139 | 139 | public function is_percent() { |
140 | - return $this->get( 'PRT_is_percent' ); |
|
140 | + return $this->get('PRT_is_percent'); |
|
141 | 141 | } |
142 | 142 | |
143 | 143 | |
@@ -146,7 +146,7 @@ discard block |
||
146 | 146 | * @return bool |
147 | 147 | */ |
148 | 148 | public function is_discount() { |
149 | - return $this->get( 'PBT_ID' ) == 2 ? TRUE : FALSE; |
|
149 | + return $this->get('PBT_ID') == 2 ? TRUE : FALSE; |
|
150 | 150 | } |
151 | 151 | |
152 | 152 | |
@@ -169,7 +169,7 @@ discard block |
||
169 | 169 | * @access public |
170 | 170 | */ |
171 | 171 | public function order() { |
172 | - return $this->get( 'PRT_order' ); |
|
172 | + return $this->get('PRT_order'); |
|
173 | 173 | } |
174 | 174 | |
175 | 175 | |
@@ -179,7 +179,7 @@ discard block |
||
179 | 179 | * @access public |
180 | 180 | */ |
181 | 181 | public function deleted() { |
182 | - return $this->get( 'PRT_deleted' ); |
|
182 | + return $this->get('PRT_deleted'); |
|
183 | 183 | } |
184 | 184 | } |
185 | 185 |
@@ -226,7 +226,7 @@ discard block |
||
226 | 226 | * generate_question_form_inputs_for_object |
227 | 227 | * |
228 | 228 | * @access protected |
229 | - * @param bool|object $object $object |
|
229 | + * @param EE_Base_Class|null $object $object |
|
230 | 230 | * @param array $input_types |
231 | 231 | * @return array |
232 | 232 | */ |
@@ -324,7 +324,7 @@ discard block |
||
324 | 324 | * add_temp_option |
325 | 325 | * @access public |
326 | 326 | * @param \EE_Question_Option $QSO EE_Question_Option |
327 | - * @return boolean |
|
327 | + * @return boolean|null |
|
328 | 328 | */ |
329 | 329 | public function add_temp_option( EE_Question_Option $QSO ) { |
330 | 330 | $this->_QST->add_temp_option( $QSO ); |
@@ -336,8 +336,8 @@ discard block |
||
336 | 336 | * set property values for question form input |
337 | 337 | * @access public |
338 | 338 | * @param string $property |
339 | - * @param mixed $value |
|
340 | - * @return mixed |
|
339 | + * @param boolean $value |
|
340 | + * @return boolean|null |
|
341 | 341 | */ |
342 | 342 | public function set( $property = NULL, $value = NULL ) { |
343 | 343 | if ( ! empty( $property ) ) { |
@@ -376,7 +376,7 @@ discard block |
||
376 | 376 | /** |
377 | 377 | * get_meta |
378 | 378 | * @access public |
379 | - * @param mixed $key |
|
379 | + * @param string $key |
|
380 | 380 | * @return mixed |
381 | 381 | */ |
382 | 382 | public function get_meta( $key = FALSE ) { |
@@ -1,5 +1,5 @@ discard block |
||
1 | -<?php if ( !defined( 'EVENT_ESPRESSO_VERSION' ) ) { |
|
2 | - exit( 'No direct script access allowed' ); |
|
1 | +<?php if ( ! defined('EVENT_ESPRESSO_VERSION')) { |
|
2 | + exit('No direct script access allowed'); |
|
3 | 3 | } |
4 | 4 | /** |
5 | 5 | * Event Espresso |
@@ -82,14 +82,14 @@ discard block |
||
82 | 82 | * @access public |
83 | 83 | * @return \EE_Question_Form_Input |
84 | 84 | */ |
85 | - public function __construct( EE_Question $QST = NULL, EE_Answer $ANS = NULL, $q_meta = array() ) { |
|
86 | - if ( empty( $QST ) || empty( $ANS ) ) { |
|
87 | - EE_Error::add_error( __( 'An error occurred. A valid EE_Question or EE_Answer object was not received.', 'event_espresso' ), __FILE__, __FUNCTION__, __LINE__ ); |
|
85 | + public function __construct(EE_Question $QST = NULL, EE_Answer $ANS = NULL, $q_meta = array()) { |
|
86 | + if (empty($QST) || empty($ANS)) { |
|
87 | + EE_Error::add_error(__('An error occurred. A valid EE_Question or EE_Answer object was not received.', 'event_espresso'), __FILE__, __FUNCTION__, __LINE__); |
|
88 | 88 | return NULL; |
89 | 89 | } |
90 | 90 | $this->_QST = $QST; |
91 | 91 | $this->_ANS = $ANS; |
92 | - $this->set_question_form_input_meta( $q_meta ); |
|
92 | + $this->set_question_form_input_meta($q_meta); |
|
93 | 93 | $this->set_question_form_input_init(); |
94 | 94 | } |
95 | 95 | |
@@ -101,9 +101,9 @@ discard block |
||
101 | 101 | * @param array $q_meta |
102 | 102 | * @return void |
103 | 103 | */ |
104 | - public function set_question_form_input_meta( $q_meta = array() ) { |
|
105 | - $default_q_meta = array( 'att_nmbr' => 1, 'ticket_id' => '', 'date' => '', 'time' => '', 'input_name' => '', 'input_id' => '', 'input_class' => '', 'input_prefix' => 'qstn', 'append_qstn_id' => TRUE, 'htmlentities' => TRUE, 'allow_null' => FALSE ); |
|
106 | - $this->_QST_meta = array_merge( $default_q_meta, $q_meta ); |
|
104 | + public function set_question_form_input_meta($q_meta = array()) { |
|
105 | + $default_q_meta = array('att_nmbr' => 1, 'ticket_id' => '', 'date' => '', 'time' => '', 'input_name' => '', 'input_id' => '', 'input_class' => '', 'input_prefix' => 'qstn', 'append_qstn_id' => TRUE, 'htmlentities' => TRUE, 'allow_null' => FALSE); |
|
106 | + $this->_QST_meta = array_merge($default_q_meta, $q_meta); |
|
107 | 107 | } |
108 | 108 | |
109 | 109 | |
@@ -115,10 +115,10 @@ discard block |
||
115 | 115 | */ |
116 | 116 | public function set_question_form_input_init() { |
117 | 117 | $qstn_id = $this->_QST->system_ID() ? $this->_QST->system_ID() : $this->_QST->ID(); |
118 | - $this->_set_input_name( $qstn_id ); |
|
119 | - $this->_set_input_id( $qstn_id ); |
|
120 | - $this->_set_input_class( $qstn_id ); |
|
121 | - $this->set_question_form_input_answer( $qstn_id ); |
|
118 | + $this->_set_input_name($qstn_id); |
|
119 | + $this->_set_input_id($qstn_id); |
|
120 | + $this->_set_input_class($qstn_id); |
|
121 | + $this->set_question_form_input_answer($qstn_id); |
|
122 | 122 | } |
123 | 123 | |
124 | 124 | |
@@ -129,12 +129,12 @@ discard block |
||
129 | 129 | * @param $qstn_id |
130 | 130 | * @return void |
131 | 131 | */ |
132 | - private function _set_input_name( $qstn_id ) { |
|
133 | - if ( ! empty( $qstn_id ) ) { |
|
134 | - $ANS_ID = $this->get( 'ANS_ID' ); |
|
135 | - $qstn_id = ! empty( $ANS_ID ) ? '[' . $qstn_id . '][' . $ANS_ID . ']' : '[' . $qstn_id . ']'; |
|
132 | + private function _set_input_name($qstn_id) { |
|
133 | + if ( ! empty($qstn_id)) { |
|
134 | + $ANS_ID = $this->get('ANS_ID'); |
|
135 | + $qstn_id = ! empty($ANS_ID) ? '['.$qstn_id.']['.$ANS_ID.']' : '['.$qstn_id.']'; |
|
136 | 136 | } |
137 | - $this->QST_input_name = $this->_QST_meta[ 'append_qstn_id' ] && ! empty( $qstn_id ) ? $this->_QST_meta[ 'input_prefix' ] . $this->_QST_meta[ 'input_name' ] . $qstn_id : $this->_QST_meta[ 'input_prefix' ] . $this->_QST_meta[ 'input_name' ]; |
|
137 | + $this->QST_input_name = $this->_QST_meta['append_qstn_id'] && ! empty($qstn_id) ? $this->_QST_meta['input_prefix'].$this->_QST_meta['input_name'].$qstn_id : $this->_QST_meta['input_prefix'].$this->_QST_meta['input_name']; |
|
138 | 138 | } |
139 | 139 | |
140 | 140 | |
@@ -145,13 +145,13 @@ discard block |
||
145 | 145 | * @param string $property |
146 | 146 | * @return mixed |
147 | 147 | */ |
148 | - public function get( $property = NULL ) { |
|
149 | - if ( ! empty( $property ) ) { |
|
150 | - if ( EEM_Question::instance()->has_field( $property ) ) { |
|
151 | - return $this->_QST->get( $property ); |
|
152 | - } else if ( EEM_Answer::instance()->has_field( $property ) ) { |
|
153 | - return $this->_ANS->get( $property ); |
|
154 | - } else if ( $this->_question_form_input_property_exists( __CLASS__, $property ) ) { |
|
148 | + public function get($property = NULL) { |
|
149 | + if ( ! empty($property)) { |
|
150 | + if (EEM_Question::instance()->has_field($property)) { |
|
151 | + return $this->_QST->get($property); |
|
152 | + } else if (EEM_Answer::instance()->has_field($property)) { |
|
153 | + return $this->_ANS->get($property); |
|
154 | + } else if ($this->_question_form_input_property_exists(__CLASS__, $property)) { |
|
155 | 155 | return $this->{$property}; |
156 | 156 | } |
157 | 157 | } |
@@ -167,13 +167,13 @@ discard block |
||
167 | 167 | * @param string $property |
168 | 168 | * @return boolean |
169 | 169 | */ |
170 | - private function _question_form_input_property_exists( $classname, $property ) { |
|
170 | + private function _question_form_input_property_exists($classname, $property) { |
|
171 | 171 | // first try regular property exists method which works as expected in PHP 5.3+ |
172 | - $prop = EEH_Class_Tools::has_property( $classname, $property ); |
|
173 | - if ( ! $prop ) { |
|
172 | + $prop = EEH_Class_Tools::has_property($classname, $property); |
|
173 | + if ( ! $prop) { |
|
174 | 174 | // use reflection for < PHP 5.3 as a double check when property is not found, possible due to access restriction |
175 | - $reflector = new ReflectionClass( $classname ); |
|
176 | - $prop = $reflector->hasProperty( $property ); |
|
175 | + $reflector = new ReflectionClass($classname); |
|
176 | + $prop = $reflector->hasProperty($property); |
|
177 | 177 | } |
178 | 178 | return $prop; |
179 | 179 | } |
@@ -186,9 +186,9 @@ discard block |
||
186 | 186 | * @param $qstn_id |
187 | 187 | * @return void |
188 | 188 | */ |
189 | - private function _set_input_id( $qstn_id ) { |
|
190 | - $input_id = isset( $this->_QST_meta[ 'input_id' ] ) && ! empty( $this->_QST_meta[ 'input_id' ] ) ? $this->_QST_meta[ 'input_id' ] : sanitize_key( strip_tags( $this->_QST->get( 'QST_display_text' ) ) ); |
|
191 | - $this->QST_input_id = $this->_QST_meta[ 'append_qstn_id' ] && ! empty( $qstn_id ) ? $input_id . '-' . $qstn_id : $input_id; |
|
189 | + private function _set_input_id($qstn_id) { |
|
190 | + $input_id = isset($this->_QST_meta['input_id']) && ! empty($this->_QST_meta['input_id']) ? $this->_QST_meta['input_id'] : sanitize_key(strip_tags($this->_QST->get('QST_display_text'))); |
|
191 | + $this->QST_input_id = $this->_QST_meta['append_qstn_id'] && ! empty($qstn_id) ? $input_id.'-'.$qstn_id : $input_id; |
|
192 | 192 | } |
193 | 193 | |
194 | 194 | |
@@ -199,7 +199,7 @@ discard block |
||
199 | 199 | * @return void |
200 | 200 | */ |
201 | 201 | private function _set_input_class() { |
202 | - $this->QST_input_class = isset( $this->_QST_meta[ 'input_class' ] ) ? $this->_QST_meta[ 'input_class' ] : ''; |
|
202 | + $this->QST_input_class = isset($this->_QST_meta['input_class']) ? $this->_QST_meta['input_class'] : ''; |
|
203 | 203 | } |
204 | 204 | |
205 | 205 | |
@@ -210,12 +210,12 @@ discard block |
||
210 | 210 | * @param mixed int | string $qstn_id |
211 | 211 | * @return void |
212 | 212 | */ |
213 | - public function set_question_form_input_answer( $qstn_id ) { |
|
213 | + public function set_question_form_input_answer($qstn_id) { |
|
214 | 214 | // check for answer in $_REQUEST in case we are reprocessing a form after an error |
215 | - if ( isset( $this->_QST_meta[ 'EVT_ID' ] ) && isset( $this->_QST_meta[ 'att_nmbr' ] ) && isset( $this->_QST_meta[ 'date' ] ) && isset( $this->_QST_meta[ 'time' ] ) && isset( $this->_QST_meta[ 'price_id' ] ) ) { |
|
216 | - if ( isset( $_REQUEST[ 'qstn' ][ $this->_QST_meta[ 'EVT_ID' ] ][ $this->_QST_meta[ 'att_nmbr' ] ][ $this->_QST_meta[ 'date' ] ][ $this->_QST_meta[ 'time' ] ][ $this->_QST_meta[ 'price_id' ] ][ $qstn_id ] ) ) { |
|
217 | - $answer = $_REQUEST[ 'qstn' ][ $this->_QST_meta[ 'EVT_ID' ] ][ $this->_QST_meta[ 'att_nmbr' ] ][ $this->_QST_meta[ 'date' ] ][ $this->_QST_meta[ 'time' ] ][ $this->_QST_meta[ 'price_id' ] ][ $qstn_id ]; |
|
218 | - $this->_ANS->set( 'ANS_value', $answer ); |
|
215 | + if (isset($this->_QST_meta['EVT_ID']) && isset($this->_QST_meta['att_nmbr']) && isset($this->_QST_meta['date']) && isset($this->_QST_meta['time']) && isset($this->_QST_meta['price_id'])) { |
|
216 | + if (isset($_REQUEST['qstn'][$this->_QST_meta['EVT_ID']][$this->_QST_meta['att_nmbr']][$this->_QST_meta['date']][$this->_QST_meta['time']][$this->_QST_meta['price_id']][$qstn_id])) { |
|
217 | + $answer = $_REQUEST['qstn'][$this->_QST_meta['EVT_ID']][$this->_QST_meta['att_nmbr']][$this->_QST_meta['date']][$this->_QST_meta['time']][$this->_QST_meta['price_id']][$qstn_id]; |
|
218 | + $this->_ANS->set('ANS_value', $answer); |
|
219 | 219 | } |
220 | 220 | } |
221 | 221 | } |
@@ -230,40 +230,40 @@ discard block |
||
230 | 230 | * @param array $input_types |
231 | 231 | * @return array |
232 | 232 | */ |
233 | - static function generate_question_form_inputs_for_object( $object = FALSE, $input_types = array() ) { |
|
234 | - if ( ! is_object( $object ) ) { |
|
233 | + static function generate_question_form_inputs_for_object($object = FALSE, $input_types = array()) { |
|
234 | + if ( ! is_object($object)) { |
|
235 | 235 | return FALSE; |
236 | 236 | } |
237 | 237 | $inputs = array(); |
238 | - $fields = $object->get_model()->field_settings( FALSE ); |
|
238 | + $fields = $object->get_model()->field_settings(FALSE); |
|
239 | 239 | // $pk = $object->ID(); <<< NO! |
240 | 240 | // EEH_Debug_Tools::printr( $object, get_class( $object ) . '<br /><span style="font-size:10px;font-weight:normal;">' . __FILE__ . '<br />line no: ' . __LINE__ . '</span>', 'auto' ); |
241 | 241 | // EEH_Debug_Tools::printr( $fields, '$fields <br /><span style="font-size:10px;font-weight:normal;">' . __FILE__ . '<br />line no: ' . __LINE__ . '</span>', 'auto' ); |
242 | 242 | // EEH_Debug_Tools::printr( $input_types, '$input_types <br /><span style="font-size:10px;font-weight:normal;">' . __FILE__ . '<br />line no: ' . __LINE__ . '</span>', 'auto' ); |
243 | - foreach ( $fields as $field_ID => $field ) { |
|
244 | - if ( $field instanceof EE_Model_Field_Base ) { |
|
243 | + foreach ($fields as $field_ID => $field) { |
|
244 | + if ($field instanceof EE_Model_Field_Base) { |
|
245 | 245 | // echo '<h4>$field_ID : ' . $field_ID . ' <br /><span style="font-size:10px;font-weight:normal;">' . __FILE__ . '<br />line no: ' . __LINE__ . '</span></h4>'; |
246 | 246 | // EEH_Debug_Tools::printr( $field, '$field <br /><span style="font-size:10px;font-weight:normal;">' . __FILE__ . '<br />line no: ' . __LINE__ . '</span>', 'auto' ); |
247 | - if ( isset( $input_types[ $field_ID ] ) ) { |
|
247 | + if (isset($input_types[$field_ID])) { |
|
248 | 248 | // get saved value for field |
249 | - $value = $object->get( $field_ID ); |
|
249 | + $value = $object->get($field_ID); |
|
250 | 250 | // echo '<h4>$value : ' . $value . ' <br /><span style="font-size:10px;font-weight:normal;">' . __FILE__ . '<br />line no: ' . __LINE__ . '</span></h4>'; |
251 | 251 | // if no saved value, then use default |
252 | 252 | $value = $value !== NULL ? $value : $field->get_default_value(); |
253 | 253 | // if ( $field_ID == 'CNT_active' ) |
254 | 254 | // echo '<h4>$value : ' . $value . ' <br /><span style="font-size:10px;font-weight:normal;">' . __FILE__ . '<br />line no: ' . __LINE__ . '</span></h4>'; |
255 | 255 | // determine question type |
256 | - $type = isset( $input_types[ $field_ID ] ) ? $input_types[ $field_ID ][ 'type' ] : 'TEXT'; |
|
256 | + $type = isset($input_types[$field_ID]) ? $input_types[$field_ID]['type'] : 'TEXT'; |
|
257 | 257 | // input name |
258 | - $input_name = isset( $input_types[ $field_ID ] ) && isset( $input_types[ $field_ID ][ 'input_name' ] ) ? $input_types[ $field_ID ][ 'input_name' ] . '[' . $field_ID . ']' : $field_ID; |
|
258 | + $input_name = isset($input_types[$field_ID]) && isset($input_types[$field_ID]['input_name']) ? $input_types[$field_ID]['input_name'].'['.$field_ID.']' : $field_ID; |
|
259 | 259 | // css class for input |
260 | - $class = isset( $input_types[ $field_ID ][ 'class' ] ) && ! empty( $input_types[ $field_ID ][ 'class' ] ) ? ' ' . $input_types[ $field_ID ][ 'class' ] : ''; |
|
260 | + $class = isset($input_types[$field_ID]['class']) && ! empty($input_types[$field_ID]['class']) ? ' '.$input_types[$field_ID]['class'] : ''; |
|
261 | 261 | // whether to apply htmlentities to answer |
262 | - $htmlentities = isset( $input_types[ $field_ID ][ 'htmlentities' ] ) ? $input_types[ $field_ID ][ 'htmlentities' ] : TRUE; |
|
262 | + $htmlentities = isset($input_types[$field_ID]['htmlentities']) ? $input_types[$field_ID]['htmlentities'] : TRUE; |
|
263 | 263 | // whether to apply htmlentities to answer |
264 | - $label_b4 = isset( $input_types[ $field_ID ][ 'label_b4' ] ) ? $input_types[ $field_ID ][ 'label_b4' ] : FALSE; |
|
264 | + $label_b4 = isset($input_types[$field_ID]['label_b4']) ? $input_types[$field_ID]['label_b4'] : FALSE; |
|
265 | 265 | // whether to apply htmlentities to answer |
266 | - $use_desc_4_label = isset( $input_types[ $field_ID ][ 'use_desc_4_label' ] ) ? $input_types[ $field_ID ][ 'use_desc_4_label' ] : FALSE; |
|
266 | + $use_desc_4_label = isset($input_types[$field_ID]['use_desc_4_label']) ? $input_types[$field_ID]['use_desc_4_label'] : FALSE; |
|
267 | 267 | |
268 | 268 | // create EE_Question_Form_Input object |
269 | 269 | $QFI = new EE_Question_Form_Input( |
@@ -283,9 +283,9 @@ discard block |
||
283 | 283 | ) |
284 | 284 | ), |
285 | 285 | array( |
286 | - 'input_id' => $field_ID . '-' . $object->ID(), |
|
286 | + 'input_id' => $field_ID.'-'.$object->ID(), |
|
287 | 287 | 'input_name' => $input_name, |
288 | - 'input_class' => $field_ID . $class, |
|
288 | + 'input_class' => $field_ID.$class, |
|
289 | 289 | 'input_prefix' => '', |
290 | 290 | 'append_qstn_id' => FALSE, |
291 | 291 | 'htmlentities' => $htmlentities, |
@@ -294,21 +294,21 @@ discard block |
||
294 | 294 | ) |
295 | 295 | ); |
296 | 296 | // does question type have options ? |
297 | - if ( in_array( $type, array( 'DROPDOWN', 'RADIO_BTN', 'CHECKBOX' ) ) && isset ( $input_types[ $field_ID ] ) && isset ( $input_types[ $field_ID ][ 'options' ] ) ) { |
|
298 | - foreach ( $input_types[ $field_ID ][ 'options' ] as $option ) { |
|
299 | - $option = stripslashes_deep( $option ); |
|
300 | - $option_id = ! empty( $option[ 'id' ] ) ? $option[ 'id' ] : 0; |
|
301 | - $QSO = EE_Question_Option::new_instance( array( 'QSO_value' => (string)$option_id, 'QSO_desc' => $option[ 'text' ], 'QSO_deleted' => FALSE ) ); |
|
297 | + if (in_array($type, array('DROPDOWN', 'RADIO_BTN', 'CHECKBOX')) && isset ($input_types[$field_ID]) && isset ($input_types[$field_ID]['options'])) { |
|
298 | + foreach ($input_types[$field_ID]['options'] as $option) { |
|
299 | + $option = stripslashes_deep($option); |
|
300 | + $option_id = ! empty($option['id']) ? $option['id'] : 0; |
|
301 | + $QSO = EE_Question_Option::new_instance(array('QSO_value' => (string) $option_id, 'QSO_desc' => $option['text'], 'QSO_deleted' => FALSE)); |
|
302 | 302 | // all QST (and ANS) properties can be accessed indirectly thru QFI |
303 | - $QFI->add_temp_option( $QSO ); |
|
303 | + $QFI->add_temp_option($QSO); |
|
304 | 304 | } |
305 | 305 | } |
306 | 306 | // we don't want ppl manually changing primary keys cuz that would just lead to total craziness man |
307 | - if ( $field_ID == $object->get_model()->primary_key_name() ) { |
|
308 | - $QFI->set( 'QST_disabled', TRUE ); |
|
307 | + if ($field_ID == $object->get_model()->primary_key_name()) { |
|
308 | + $QFI->set('QST_disabled', TRUE); |
|
309 | 309 | } |
310 | 310 | //EEH_Debug_Tools::printr( $QFI, '$QFI <br /><span style="font-size:10px;font-weight:normal;">' . __FILE__ . '<br />line no: ' . __LINE__ . '</span>', 'auto' ); |
311 | - $inputs[ $field_ID ] = $QFI; |
|
311 | + $inputs[$field_ID] = $QFI; |
|
312 | 312 | // if ( $field_ID == 'CNT_active' ) { |
313 | 313 | // EEH_Debug_Tools::printr( $QFI, '$QFI <br /><span style="font-size:10px;font-weight:normal;">' . __FILE__ . '<br />line no: ' . __LINE__ . '</span>', 'auto' ); |
314 | 314 | // } |
@@ -326,8 +326,8 @@ discard block |
||
326 | 326 | * @param \EE_Question_Option $QSO EE_Question_Option |
327 | 327 | * @return boolean |
328 | 328 | */ |
329 | - public function add_temp_option( EE_Question_Option $QSO ) { |
|
330 | - $this->_QST->add_temp_option( $QSO ); |
|
329 | + public function add_temp_option(EE_Question_Option $QSO) { |
|
330 | + $this->_QST->add_temp_option($QSO); |
|
331 | 331 | } |
332 | 332 | |
333 | 333 | |
@@ -339,13 +339,13 @@ discard block |
||
339 | 339 | * @param mixed $value |
340 | 340 | * @return mixed |
341 | 341 | */ |
342 | - public function set( $property = NULL, $value = NULL ) { |
|
343 | - if ( ! empty( $property ) ) { |
|
344 | - if ( EEM_Question::instance()->has_field( $property ) ) { |
|
345 | - $this->_QST->set( $property, $value ); |
|
346 | - } else if ( EEM_Answer::instance()->has_field( $property ) ) { |
|
347 | - $this->_ANS->set( $property, $value ); |
|
348 | - } else if ( $this->_question_form_input_property_exists( __CLASS__, $property ) ) { |
|
342 | + public function set($property = NULL, $value = NULL) { |
|
343 | + if ( ! empty($property)) { |
|
344 | + if (EEM_Question::instance()->has_field($property)) { |
|
345 | + $this->_QST->set($property, $value); |
|
346 | + } else if (EEM_Answer::instance()->has_field($property)) { |
|
347 | + $this->_ANS->set($property, $value); |
|
348 | + } else if ($this->_question_form_input_property_exists(__CLASS__, $property)) { |
|
349 | 349 | echo "<hr>$property is a prop of QFI"; |
350 | 350 | $this->{$property} = $value; |
351 | 351 | return TRUE; |
@@ -366,9 +366,9 @@ discard block |
||
366 | 366 | * whether it was trashed or not. |
367 | 367 | * @return EE_Question_Option |
368 | 368 | */ |
369 | - public function options( $notDeletedOptionsOnly = TRUE, $selected_value_to_always_include = NULL ) { |
|
369 | + public function options($notDeletedOptionsOnly = TRUE, $selected_value_to_always_include = NULL) { |
|
370 | 370 | $temp_options = $this->_QST->temp_options(); |
371 | - return ! empty( $temp_options ) ? $temp_options : $this->_QST->options( $notDeletedOptionsOnly, $selected_value_to_always_include ); |
|
371 | + return ! empty($temp_options) ? $temp_options : $this->_QST->options($notDeletedOptionsOnly, $selected_value_to_always_include); |
|
372 | 372 | } |
373 | 373 | |
374 | 374 | |
@@ -379,8 +379,8 @@ discard block |
||
379 | 379 | * @param mixed $key |
380 | 380 | * @return mixed |
381 | 381 | */ |
382 | - public function get_meta( $key = FALSE ) { |
|
383 | - return $key && isset( $this->_QST_meta[ $key ] ) ? $this->_QST_meta[ $key ] : FALSE; |
|
382 | + public function get_meta($key = FALSE) { |
|
383 | + return $key && isset($this->_QST_meta[$key]) ? $this->_QST_meta[$key] : FALSE; |
|
384 | 384 | } |
385 | 385 | |
386 | 386 |
@@ -65,7 +65,7 @@ discard block |
||
65 | 65 | /** |
66 | 66 | * Gets the question group's internal name |
67 | 67 | * @access public |
68 | - * @return string |
|
68 | + * @return boolean |
|
69 | 69 | */ |
70 | 70 | public function identifier() { |
71 | 71 | return $this->get( 'QSG_identifier' ); |
@@ -89,7 +89,7 @@ discard block |
||
89 | 89 | * Gets the question group's order number in a sequence |
90 | 90 | * of other question groups |
91 | 91 | * @access public |
92 | - * @return int |
|
92 | + * @return boolean |
|
93 | 93 | */ |
94 | 94 | public function order() { |
95 | 95 | return $this->get( 'QSG_order' ); |
@@ -138,7 +138,7 @@ discard block |
||
138 | 138 | * |
139 | 139 | * @since 4.5.0 |
140 | 140 | * |
141 | - * @return int |
|
141 | + * @return boolean |
|
142 | 142 | */ |
143 | 143 | public function wp_user() { |
144 | 144 | return $this->get('QSG_wp_user'); |
@@ -213,7 +213,7 @@ discard block |
||
213 | 213 | /** |
214 | 214 | * Adds the question to this question group |
215 | 215 | * @param EE_Question || int $question object or ID |
216 | - * @return boolean if successful |
|
216 | + * @return EE_Base_Class if successful |
|
217 | 217 | */ |
218 | 218 | public function add_question( $questionObjectOrID ) { |
219 | 219 | return $this->_add_relation_to( $questionObjectOrID, 'Question' ); |
@@ -224,7 +224,7 @@ discard block |
||
224 | 224 | /** |
225 | 225 | * Removes the question from this question group |
226 | 226 | * @param EE_Question || int $question object or ID |
227 | - * @return boolean of success |
|
227 | + * @return EE_Base_Class of success |
|
228 | 228 | */ |
229 | 229 | public function remove_question( $questionObjectOrID ) { |
230 | 230 | return $this->_remove_relation_to( $questionObjectOrID, 'Question' ); |
@@ -1,5 +1,5 @@ discard block |
||
1 | -<?php if ( !defined( 'EVENT_ESPRESSO_VERSION' ) ) { |
|
2 | - exit( 'No direct script access allowed' ); |
|
1 | +<?php if ( ! defined('EVENT_ESPRESSO_VERSION')) { |
|
2 | + exit('No direct script access allowed'); |
|
3 | 3 | } |
4 | 4 | /** |
5 | 5 | * Event Espresso |
@@ -32,9 +32,9 @@ discard block |
||
32 | 32 | * @param array $props_n_values |
33 | 33 | * @return EE_Question_Group|mixed |
34 | 34 | */ |
35 | - public static function new_instance( $props_n_values = array() ) { |
|
36 | - $has_object = parent::_check_for_object( $props_n_values, __CLASS__ ); |
|
37 | - return $has_object ? $has_object : new self( $props_n_values ); |
|
35 | + public static function new_instance($props_n_values = array()) { |
|
36 | + $has_object = parent::_check_for_object($props_n_values, __CLASS__); |
|
37 | + return $has_object ? $has_object : new self($props_n_values); |
|
38 | 38 | } |
39 | 39 | |
40 | 40 | |
@@ -43,8 +43,8 @@ discard block |
||
43 | 43 | * @param array $props_n_values |
44 | 44 | * @return EE_Question_Group |
45 | 45 | */ |
46 | - public static function new_instance_from_db( $props_n_values = array() ) { |
|
47 | - return new self( $props_n_values, TRUE ); |
|
46 | + public static function new_instance_from_db($props_n_values = array()) { |
|
47 | + return new self($props_n_values, TRUE); |
|
48 | 48 | } |
49 | 49 | |
50 | 50 | |
@@ -55,8 +55,8 @@ discard block |
||
55 | 55 | * @param bool $pretty |
56 | 56 | * @return string |
57 | 57 | */ |
58 | - public function name( $pretty = FALSE ) { |
|
59 | - return $pretty ? $this->get_pretty( 'QSG_name' ) : $this->get( 'QSG_name' ); |
|
58 | + public function name($pretty = FALSE) { |
|
59 | + return $pretty ? $this->get_pretty('QSG_name') : $this->get('QSG_name'); |
|
60 | 60 | } |
61 | 61 | |
62 | 62 | |
@@ -68,7 +68,7 @@ discard block |
||
68 | 68 | * @return string |
69 | 69 | */ |
70 | 70 | public function identifier() { |
71 | - return $this->get( 'QSG_identifier' ); |
|
71 | + return $this->get('QSG_identifier'); |
|
72 | 72 | } |
73 | 73 | |
74 | 74 | |
@@ -79,8 +79,8 @@ discard block |
||
79 | 79 | * @param bool $pretty |
80 | 80 | * @return string |
81 | 81 | */ |
82 | - public function desc( $pretty = FALSE ) { |
|
83 | - return $pretty ? $this->get_pretty( 'QSG_desc' ) : $this->get( 'QSG_desc' ); |
|
82 | + public function desc($pretty = FALSE) { |
|
83 | + return $pretty ? $this->get_pretty('QSG_desc') : $this->get('QSG_desc'); |
|
84 | 84 | } |
85 | 85 | |
86 | 86 | |
@@ -92,7 +92,7 @@ discard block |
||
92 | 92 | * @return int |
93 | 93 | */ |
94 | 94 | public function order() { |
95 | - return $this->get( 'QSG_order' ); |
|
95 | + return $this->get('QSG_order'); |
|
96 | 96 | } |
97 | 97 | |
98 | 98 | |
@@ -103,7 +103,7 @@ discard block |
||
103 | 103 | * @return boolean |
104 | 104 | */ |
105 | 105 | public function show_group_name() { |
106 | - return $this->get( 'QSG_show_group_name' ); |
|
106 | + return $this->get('QSG_show_group_name'); |
|
107 | 107 | } |
108 | 108 | |
109 | 109 | |
@@ -115,7 +115,7 @@ discard block |
||
115 | 115 | * @return boolean |
116 | 116 | */ |
117 | 117 | public function show_group_desc() { |
118 | - return $this->get( 'QSG_show_group_desc' ); |
|
118 | + return $this->get('QSG_show_group_desc'); |
|
119 | 119 | } |
120 | 120 | |
121 | 121 | |
@@ -128,7 +128,7 @@ discard block |
||
128 | 128 | * @return boolean |
129 | 129 | */ |
130 | 130 | public function system_group() { |
131 | - return $this->get( 'QSG_system' ); |
|
131 | + return $this->get('QSG_system'); |
|
132 | 132 | } |
133 | 133 | |
134 | 134 | |
@@ -153,7 +153,7 @@ discard block |
||
153 | 153 | * @return boolean |
154 | 154 | */ |
155 | 155 | public function deleted() { |
156 | - return $this->get( 'QST_deleted' ); |
|
156 | + return $this->get('QST_deleted'); |
|
157 | 157 | } |
158 | 158 | |
159 | 159 | |
@@ -164,8 +164,8 @@ discard block |
||
164 | 164 | */ |
165 | 165 | public function questions_in_and_not_in_group() { |
166 | 166 | $questions_in_group = $this->questions(); |
167 | - $exclude_question_ids = ! empty( $questions_in_group ) ? array_keys( $questions_in_group ) : array(); |
|
168 | - $questions_not_in_group = $this->questions_not_in_group( $exclude_question_ids ); |
|
167 | + $exclude_question_ids = ! empty($questions_in_group) ? array_keys($questions_in_group) : array(); |
|
168 | + $questions_not_in_group = $this->questions_not_in_group($exclude_question_ids); |
|
169 | 169 | return $questions_in_group + $questions_not_in_group; |
170 | 170 | } |
171 | 171 | |
@@ -176,9 +176,9 @@ discard block |
||
176 | 176 | * @param array $query_params |
177 | 177 | * @return EE_Question[] |
178 | 178 | */ |
179 | - public function questions( $query_params = array() ) { |
|
180 | - $query_params = ! empty( $query_params ) ? $query_params : array( 'order_by' => array( 'Question_Group_Question.QGQ_order' => 'ASC' ) ); |
|
181 | - return $this->ID() ? $this->get_many_related( 'Question', $query_params ) : array(); |
|
179 | + public function questions($query_params = array()) { |
|
180 | + $query_params = ! empty($query_params) ? $query_params : array('order_by' => array('Question_Group_Question.QGQ_order' => 'ASC')); |
|
181 | + return $this->ID() ? $this->get_many_related('Question', $query_params) : array(); |
|
182 | 182 | } |
183 | 183 | |
184 | 184 | |
@@ -188,14 +188,14 @@ discard block |
||
188 | 188 | * @param mixed $question_IDS_in_group if empty array then all questions returned. if FALSE then we first get questions in this group and exclude them from questions get all. IF empty array then we just return all questions. |
189 | 189 | * @return EE_Question[] |
190 | 190 | */ |
191 | - public function questions_not_in_group( $question_IDS_in_group = FALSE ) { |
|
192 | - if ( $question_IDS_in_group === FALSE ) { |
|
191 | + public function questions_not_in_group($question_IDS_in_group = FALSE) { |
|
192 | + if ($question_IDS_in_group === FALSE) { |
|
193 | 193 | $questions = $this->questions(); |
194 | - $question_IDS_in_group = ! empty( $questions ) ? array_keys( $questions ) : array(); |
|
194 | + $question_IDS_in_group = ! empty($questions) ? array_keys($questions) : array(); |
|
195 | 195 | } |
196 | - $_where = ! empty( $question_IDS_in_group ) ? array( 'QST_ID' => array( 'not_in', $question_IDS_in_group ) ) : array(); |
|
196 | + $_where = ! empty($question_IDS_in_group) ? array('QST_ID' => array('not_in', $question_IDS_in_group)) : array(); |
|
197 | 197 | |
198 | - return EEM_Question::instance()->get_all( array( $_where, 'order_by' => array( 'QST_ID' => 'ASC' ) ) ); |
|
198 | + return EEM_Question::instance()->get_all(array($_where, 'order_by' => array('QST_ID' => 'ASC'))); |
|
199 | 199 | } |
200 | 200 | |
201 | 201 | |
@@ -205,7 +205,7 @@ discard block |
||
205 | 205 | * @return EE_Event[] |
206 | 206 | */ |
207 | 207 | public function events() { |
208 | - return $this->get_many_related( 'Event' ); |
|
208 | + return $this->get_many_related('Event'); |
|
209 | 209 | } |
210 | 210 | |
211 | 211 | |
@@ -215,8 +215,8 @@ discard block |
||
215 | 215 | * @param EE_Question || int $question object or ID |
216 | 216 | * @return boolean if successful |
217 | 217 | */ |
218 | - public function add_question( $questionObjectOrID ) { |
|
219 | - return $this->_add_relation_to( $questionObjectOrID, 'Question' ); |
|
218 | + public function add_question($questionObjectOrID) { |
|
219 | + return $this->_add_relation_to($questionObjectOrID, 'Question'); |
|
220 | 220 | } |
221 | 221 | |
222 | 222 | |
@@ -226,8 +226,8 @@ discard block |
||
226 | 226 | * @param EE_Question || int $question object or ID |
227 | 227 | * @return boolean of success |
228 | 228 | */ |
229 | - public function remove_question( $questionObjectOrID ) { |
|
230 | - return $this->_remove_relation_to( $questionObjectOrID, 'Question' ); |
|
229 | + public function remove_question($questionObjectOrID) { |
|
230 | + return $this->_remove_relation_to($questionObjectOrID, 'Question'); |
|
231 | 231 | } |
232 | 232 | |
233 | 233 | |
@@ -237,9 +237,9 @@ discard block |
||
237 | 237 | * @param $qst_order |
238 | 238 | * @return int |
239 | 239 | */ |
240 | - public function update_question_order( $questionObjectOrID, $qst_order ) { |
|
241 | - $qst_ID = $questionObjectOrID instanceof EE_Question ? $questionObjectOrID->ID() : (int)$questionObjectOrID; |
|
242 | - return EEM_Question_Group_Question::instance()->update( array( 'QGQ_order' => $qst_order ), array( array( 'QST_ID' => $qst_ID, 'QSG_ID' => $this->ID() ) ) ); |
|
240 | + public function update_question_order($questionObjectOrID, $qst_order) { |
|
241 | + $qst_ID = $questionObjectOrID instanceof EE_Question ? $questionObjectOrID->ID() : (int) $questionObjectOrID; |
|
242 | + return EEM_Question_Group_Question::instance()->update(array('QGQ_order' => $qst_order), array(array('QST_ID' => $qst_ID, 'QSG_ID' => $this->ID()))); |
|
243 | 243 | } |
244 | 244 | |
245 | 245 | |
@@ -250,9 +250,9 @@ discard block |
||
250 | 250 | */ |
251 | 251 | public function has_questions_with_answers() { |
252 | 252 | $has_answers = FALSE; |
253 | - $questions = $this->get_many_related( 'Question' ); |
|
254 | - foreach ( $questions as $question ) { |
|
255 | - if ( $question->count_related( 'Answer' ) > 0 ) |
|
253 | + $questions = $this->get_many_related('Question'); |
|
254 | + foreach ($questions as $question) { |
|
255 | + if ($question->count_related('Answer') > 0) |
|
256 | 256 | $has_answers = TRUE; |
257 | 257 | } |
258 | 258 | return $has_answers; |
@@ -268,7 +268,7 @@ discard block |
||
268 | 268 | */ |
269 | 269 | public function set_order_to_latest() { |
270 | 270 | $latest_order = $this->get_model()->get_latest_question_group_order(); |
271 | - $latest_order ++; |
|
272 | - $this->set( 'QSG_order', $latest_order ); |
|
271 | + $latest_order++; |
|
272 | + $this->set('QSG_order', $latest_order); |
|
273 | 273 | } |
274 | 274 | } |
@@ -252,8 +252,9 @@ |
||
252 | 252 | $has_answers = FALSE; |
253 | 253 | $questions = $this->get_many_related( 'Question' ); |
254 | 254 | foreach ( $questions as $question ) { |
255 | - if ( $question->count_related( 'Answer' ) > 0 ) |
|
256 | - $has_answers = TRUE; |
|
255 | + if ( $question->count_related( 'Answer' ) > 0 ) { |
|
256 | + $has_answers = TRUE; |
|
257 | + } |
|
257 | 258 | } |
258 | 259 | return $has_answers; |
259 | 260 | } |
@@ -40,7 +40,7 @@ discard block |
||
40 | 40 | /** |
41 | 41 | * registration_ID |
42 | 42 | * @access public |
43 | - * @return int |
|
43 | + * @return boolean |
|
44 | 44 | */ |
45 | 45 | public function registration_ID() { |
46 | 46 | return $this->get( 'REG_ID' ); |
@@ -51,7 +51,7 @@ discard block |
||
51 | 51 | /** |
52 | 52 | * payment_ID |
53 | 53 | * @access public |
54 | - * @return int |
|
54 | + * @return boolean |
|
55 | 55 | */ |
56 | 56 | public function payment_ID() { |
57 | 57 | return $this->get( 'PAY_ID' ); |
@@ -62,7 +62,7 @@ discard block |
||
62 | 62 | /** |
63 | 63 | * amount |
64 | 64 | * @access public |
65 | - * @return float |
|
65 | + * @return boolean |
|
66 | 66 | */ |
67 | 67 | public function amount() { |
68 | 68 | return $this->get( 'RPY_amount' ); |
@@ -19,9 +19,9 @@ discard block |
||
19 | 19 | * @param string $timezone |
20 | 20 | * @return EE_Registration_Payment |
21 | 21 | */ |
22 | - public static function new_instance( $props_n_values = array(), $timezone = '' ) { |
|
23 | - $has_object = parent::_check_for_object( $props_n_values, __CLASS__ ); |
|
24 | - return $has_object ? $has_object : new self( $props_n_values, false, $timezone ); |
|
22 | + public static function new_instance($props_n_values = array(), $timezone = '') { |
|
23 | + $has_object = parent::_check_for_object($props_n_values, __CLASS__); |
|
24 | + return $has_object ? $has_object : new self($props_n_values, false, $timezone); |
|
25 | 25 | } |
26 | 26 | |
27 | 27 | |
@@ -31,8 +31,8 @@ discard block |
||
31 | 31 | * @param string $timezone |
32 | 32 | * @return EE_Registration_Payment |
33 | 33 | */ |
34 | - public static function new_instance_from_db( $props_n_values = array(), $timezone = '' ) { |
|
35 | - return new self( $props_n_values, true, $timezone ); |
|
34 | + public static function new_instance_from_db($props_n_values = array(), $timezone = '') { |
|
35 | + return new self($props_n_values, true, $timezone); |
|
36 | 36 | } |
37 | 37 | |
38 | 38 | |
@@ -43,7 +43,7 @@ discard block |
||
43 | 43 | * @return int |
44 | 44 | */ |
45 | 45 | public function registration_ID() { |
46 | - return $this->get( 'REG_ID' ); |
|
46 | + return $this->get('REG_ID'); |
|
47 | 47 | } |
48 | 48 | |
49 | 49 | |
@@ -54,7 +54,7 @@ discard block |
||
54 | 54 | * @return int |
55 | 55 | */ |
56 | 56 | public function payment_ID() { |
57 | - return $this->get( 'PAY_ID' ); |
|
57 | + return $this->get('PAY_ID'); |
|
58 | 58 | } |
59 | 59 | |
60 | 60 | |
@@ -65,7 +65,7 @@ discard block |
||
65 | 65 | * @return float |
66 | 66 | */ |
67 | 67 | public function amount() { |
68 | - return $this->get( 'RPY_amount' ); |
|
68 | + return $this->get('RPY_amount'); |
|
69 | 69 | } |
70 | 70 | |
71 | 71 | |
@@ -75,8 +75,8 @@ discard block |
||
75 | 75 | * @access public |
76 | 76 | * @param float $amount |
77 | 77 | */ |
78 | - public function set_amount( $amount = 0.000 ) { |
|
79 | - $this->set( 'RPY_amount', $amount ); |
|
78 | + public function set_amount($amount = 0.000) { |
|
79 | + $this->set('RPY_amount', $amount); |
|
80 | 80 | } |
81 | 81 | |
82 | 82 |
@@ -50,7 +50,7 @@ discard block |
||
50 | 50 | |
51 | 51 | |
52 | 52 | /** |
53 | - * @return string |
|
53 | + * @return boolean |
|
54 | 54 | */ |
55 | 55 | public function country_iso() { |
56 | 56 | return $this->get( 'CNT_ISO' ); |
@@ -59,7 +59,7 @@ discard block |
||
59 | 59 | |
60 | 60 | |
61 | 61 | /** |
62 | - * @return string |
|
62 | + * @return boolean |
|
63 | 63 | */ |
64 | 64 | public function abbrev() { |
65 | 65 | return $this->get( 'STA_abbrev' ); |
@@ -77,7 +77,7 @@ discard block |
||
77 | 77 | |
78 | 78 | |
79 | 79 | /** |
80 | - * @return string |
|
80 | + * @return boolean |
|
81 | 81 | */ |
82 | 82 | public function name() { |
83 | 83 | return $this->get( 'STA_name' ); |
@@ -1,5 +1,5 @@ discard block |
||
1 | -<?php if ( !defined( 'EVENT_ESPRESSO_VERSION' ) ) { |
|
2 | - exit( 'No direct script access allowed' ); |
|
1 | +<?php if ( ! defined('EVENT_ESPRESSO_VERSION')) { |
|
2 | + exit('No direct script access allowed'); |
|
3 | 3 | } |
4 | 4 | /** |
5 | 5 | * Event Espresso |
@@ -32,9 +32,9 @@ discard block |
||
32 | 32 | * @param array $props_n_values |
33 | 33 | * @return EE_State|mixed |
34 | 34 | */ |
35 | - public static function new_instance( $props_n_values = array() ) { |
|
36 | - $has_object = parent::_check_for_object( $props_n_values, __CLASS__ ); |
|
37 | - return $has_object ? $has_object : new self( $props_n_values ); |
|
35 | + public static function new_instance($props_n_values = array()) { |
|
36 | + $has_object = parent::_check_for_object($props_n_values, __CLASS__); |
|
37 | + return $has_object ? $has_object : new self($props_n_values); |
|
38 | 38 | } |
39 | 39 | |
40 | 40 | |
@@ -43,8 +43,8 @@ discard block |
||
43 | 43 | * @param array $props_n_values |
44 | 44 | * @return EE_State |
45 | 45 | */ |
46 | - public static function new_instance_from_db( $props_n_values = array() ) { |
|
47 | - return new self( $props_n_values, TRUE ); |
|
46 | + public static function new_instance_from_db($props_n_values = array()) { |
|
47 | + return new self($props_n_values, TRUE); |
|
48 | 48 | } |
49 | 49 | |
50 | 50 | |
@@ -53,7 +53,7 @@ discard block |
||
53 | 53 | * @return string |
54 | 54 | */ |
55 | 55 | public function country_iso() { |
56 | - return $this->get( 'CNT_ISO' ); |
|
56 | + return $this->get('CNT_ISO'); |
|
57 | 57 | } |
58 | 58 | |
59 | 59 | |
@@ -62,7 +62,7 @@ discard block |
||
62 | 62 | * @return string |
63 | 63 | */ |
64 | 64 | public function abbrev() { |
65 | - return $this->get( 'STA_abbrev' ); |
|
65 | + return $this->get('STA_abbrev'); |
|
66 | 66 | } |
67 | 67 | |
68 | 68 | |
@@ -71,7 +71,7 @@ discard block |
||
71 | 71 | * @return bool |
72 | 72 | */ |
73 | 73 | public function active() { |
74 | - return $this->get( 'STA_active' ); |
|
74 | + return $this->get('STA_active'); |
|
75 | 75 | } |
76 | 76 | |
77 | 77 | |
@@ -80,7 +80,7 @@ discard block |
||
80 | 80 | * @return string |
81 | 81 | */ |
82 | 82 | public function name() { |
83 | - return $this->get( 'STA_name' ); |
|
83 | + return $this->get('STA_name'); |
|
84 | 84 | } |
85 | 85 | |
86 | 86 | |
@@ -89,7 +89,7 @@ discard block |
||
89 | 89 | * @return EE_Country |
90 | 90 | */ |
91 | 91 | public function country() { |
92 | - return $this->get_first_related( 'Country' ); |
|
92 | + return $this->get_first_related('Country'); |
|
93 | 93 | } |
94 | 94 | |
95 | 95 | |
@@ -97,8 +97,8 @@ discard block |
||
97 | 97 | /** |
98 | 98 | * @param $iso |
99 | 99 | */ |
100 | - public function set_country_iso( $iso ) { |
|
101 | - $this->set( 'CNT_ISO', $iso ); |
|
100 | + public function set_country_iso($iso) { |
|
101 | + $this->set('CNT_ISO', $iso); |
|
102 | 102 | } |
103 | 103 | |
104 | 104 | |
@@ -106,8 +106,8 @@ discard block |
||
106 | 106 | /** |
107 | 107 | * @param $abbrev |
108 | 108 | */ |
109 | - public function set_abbrev( $abbrev ) { |
|
110 | - $this->set( 'STA_abbrev', $abbrev ); |
|
109 | + public function set_abbrev($abbrev) { |
|
110 | + $this->set('STA_abbrev', $abbrev); |
|
111 | 111 | } |
112 | 112 | |
113 | 113 | |
@@ -115,8 +115,8 @@ discard block |
||
115 | 115 | /** |
116 | 116 | * @param $active |
117 | 117 | */ |
118 | - public function set_active( $active ) { |
|
119 | - $this->set( 'STA_active', $active ); |
|
118 | + public function set_active($active) { |
|
119 | + $this->set('STA_active', $active); |
|
120 | 120 | } |
121 | 121 | |
122 | 122 | |
@@ -124,8 +124,8 @@ discard block |
||
124 | 124 | /** |
125 | 125 | * @param $name |
126 | 126 | */ |
127 | - public function set_name( $name ) { |
|
128 | - $this->set( 'STA_name', $name ); |
|
127 | + public function set_name($name) { |
|
128 | + $this->set('STA_name', $name); |
|
129 | 129 | } |
130 | 130 | } |
131 | 131 |
@@ -66,7 +66,7 @@ discard block |
||
66 | 66 | /** |
67 | 67 | * Sets code |
68 | 68 | * @param string $code |
69 | - * @return boolean |
|
69 | + * @return boolean|null |
|
70 | 70 | */ |
71 | 71 | function set_code( $code ) { |
72 | 72 | $this->set( 'STS_code', $code ); |
@@ -76,7 +76,7 @@ discard block |
||
76 | 76 | |
77 | 77 | /** |
78 | 78 | * Gets desc |
79 | - * @return string |
|
79 | + * @return boolean |
|
80 | 80 | */ |
81 | 81 | function desc() { |
82 | 82 | return $this->get( 'STS_desc' ); |
@@ -87,7 +87,7 @@ discard block |
||
87 | 87 | /** |
88 | 88 | * Sets desc |
89 | 89 | * @param string $desc |
90 | - * @return boolean |
|
90 | + * @return boolean|null |
|
91 | 91 | */ |
92 | 92 | function set_desc( $desc ) { |
93 | 93 | $this->set( 'STS_desc', $desc ); |
@@ -97,7 +97,7 @@ discard block |
||
97 | 97 | |
98 | 98 | /** |
99 | 99 | * Gets type |
100 | - * @return string |
|
100 | + * @return boolean |
|
101 | 101 | */ |
102 | 102 | function type() { |
103 | 103 | return $this->get( 'STS_type' ); |
@@ -108,7 +108,7 @@ discard block |
||
108 | 108 | /** |
109 | 109 | * Sets type |
110 | 110 | * @param string $type |
111 | - * @return boolean |
|
111 | + * @return boolean|null |
|
112 | 112 | */ |
113 | 113 | function set_type( $type ) { |
114 | 114 | $this->set( 'STS_type', $type ); |
@@ -129,7 +129,7 @@ discard block |
||
129 | 129 | /** |
130 | 130 | * Sets can_edit |
131 | 131 | * @param boolean $can_edit |
132 | - * @return boolean |
|
132 | + * @return boolean|null |
|
133 | 133 | */ |
134 | 134 | function set_can_edit( $can_edit ) { |
135 | 135 | $this->set( 'STS_can_edit', $can_edit ); |
@@ -150,7 +150,7 @@ discard block |
||
150 | 150 | /** |
151 | 151 | * Sets open |
152 | 152 | * @param boolean $open |
153 | - * @return boolean |
|
153 | + * @return boolean|null |
|
154 | 154 | */ |
155 | 155 | function set_open( $open ) { |
156 | 156 | $this->set( 'STS_open', $open ); |
@@ -1,5 +1,5 @@ discard block |
||
1 | -<?php if ( !defined( 'EVENT_ESPRESSO_VERSION' ) ) { |
|
2 | - exit( 'No direct script access allowed' ); |
|
1 | +<?php if ( ! defined('EVENT_ESPRESSO_VERSION')) { |
|
2 | + exit('No direct script access allowed'); |
|
3 | 3 | } |
4 | 4 | /** |
5 | 5 | * Event Espresso |
@@ -32,9 +32,9 @@ discard block |
||
32 | 32 | * @param array $props_n_values |
33 | 33 | * @return EE_Status |
34 | 34 | */ |
35 | - public static function new_instance( $props_n_values = array() ) { |
|
36 | - $has_object = parent::_check_for_object( $props_n_values, __CLASS__ ); |
|
37 | - return $has_object ? $has_object : new self( $props_n_values ); |
|
35 | + public static function new_instance($props_n_values = array()) { |
|
36 | + $has_object = parent::_check_for_object($props_n_values, __CLASS__); |
|
37 | + return $has_object ? $has_object : new self($props_n_values); |
|
38 | 38 | } |
39 | 39 | |
40 | 40 | |
@@ -43,8 +43,8 @@ discard block |
||
43 | 43 | * @param array $props_n_values |
44 | 44 | * @return EE_Status |
45 | 45 | */ |
46 | - public static function new_instance_from_db( $props_n_values = array() ) { |
|
47 | - return new self( $props_n_values, TRUE ); |
|
46 | + public static function new_instance_from_db($props_n_values = array()) { |
|
47 | + return new self($props_n_values, TRUE); |
|
48 | 48 | } |
49 | 49 | |
50 | 50 | |
@@ -55,10 +55,10 @@ discard block |
||
55 | 55 | * @param string $schema |
56 | 56 | * @return string |
57 | 57 | */ |
58 | - function code( $plural = FALSE, $schema = 'upper' ) { |
|
59 | - $id = $this->get( 'STS_ID' ); |
|
60 | - $code = EEM_Status::instance()->localized_status( array( $id => $this->get( 'STS_code' ) ), $plural, $schema ); |
|
61 | - return $code[ $id ]; |
|
58 | + function code($plural = FALSE, $schema = 'upper') { |
|
59 | + $id = $this->get('STS_ID'); |
|
60 | + $code = EEM_Status::instance()->localized_status(array($id => $this->get('STS_code')), $plural, $schema); |
|
61 | + return $code[$id]; |
|
62 | 62 | } |
63 | 63 | |
64 | 64 | |
@@ -68,8 +68,8 @@ discard block |
||
68 | 68 | * @param string $code |
69 | 69 | * @return boolean |
70 | 70 | */ |
71 | - function set_code( $code ) { |
|
72 | - $this->set( 'STS_code', $code ); |
|
71 | + function set_code($code) { |
|
72 | + $this->set('STS_code', $code); |
|
73 | 73 | } |
74 | 74 | |
75 | 75 | |
@@ -79,7 +79,7 @@ discard block |
||
79 | 79 | * @return string |
80 | 80 | */ |
81 | 81 | function desc() { |
82 | - return $this->get( 'STS_desc' ); |
|
82 | + return $this->get('STS_desc'); |
|
83 | 83 | } |
84 | 84 | |
85 | 85 | |
@@ -89,8 +89,8 @@ discard block |
||
89 | 89 | * @param string $desc |
90 | 90 | * @return boolean |
91 | 91 | */ |
92 | - function set_desc( $desc ) { |
|
93 | - $this->set( 'STS_desc', $desc ); |
|
92 | + function set_desc($desc) { |
|
93 | + $this->set('STS_desc', $desc); |
|
94 | 94 | } |
95 | 95 | |
96 | 96 | |
@@ -100,7 +100,7 @@ discard block |
||
100 | 100 | * @return string |
101 | 101 | */ |
102 | 102 | function type() { |
103 | - return $this->get( 'STS_type' ); |
|
103 | + return $this->get('STS_type'); |
|
104 | 104 | } |
105 | 105 | |
106 | 106 | |
@@ -110,8 +110,8 @@ discard block |
||
110 | 110 | * @param string $type |
111 | 111 | * @return boolean |
112 | 112 | */ |
113 | - function set_type( $type ) { |
|
114 | - $this->set( 'STS_type', $type ); |
|
113 | + function set_type($type) { |
|
114 | + $this->set('STS_type', $type); |
|
115 | 115 | } |
116 | 116 | |
117 | 117 | |
@@ -121,7 +121,7 @@ discard block |
||
121 | 121 | * @return boolean |
122 | 122 | */ |
123 | 123 | function can_edit() { |
124 | - return $this->get( 'STS_can_edit' ); |
|
124 | + return $this->get('STS_can_edit'); |
|
125 | 125 | } |
126 | 126 | |
127 | 127 | |
@@ -131,8 +131,8 @@ discard block |
||
131 | 131 | * @param boolean $can_edit |
132 | 132 | * @return boolean |
133 | 133 | */ |
134 | - function set_can_edit( $can_edit ) { |
|
135 | - $this->set( 'STS_can_edit', $can_edit ); |
|
134 | + function set_can_edit($can_edit) { |
|
135 | + $this->set('STS_can_edit', $can_edit); |
|
136 | 136 | } |
137 | 137 | |
138 | 138 | |
@@ -142,7 +142,7 @@ discard block |
||
142 | 142 | * @return boolean |
143 | 143 | */ |
144 | 144 | function open() { |
145 | - return $this->get( 'STS_open' ); |
|
145 | + return $this->get('STS_open'); |
|
146 | 146 | } |
147 | 147 | |
148 | 148 | |
@@ -152,8 +152,8 @@ discard block |
||
152 | 152 | * @param boolean $open |
153 | 153 | * @return boolean |
154 | 154 | */ |
155 | - function set_open( $open ) { |
|
156 | - $this->set( 'STS_open', $open ); |
|
155 | + function set_open($open) { |
|
156 | + $this->set('STS_open', $open); |
|
157 | 157 | } |
158 | 158 | } |
159 | 159 |
@@ -66,7 +66,7 @@ discard block |
||
66 | 66 | |
67 | 67 | /** |
68 | 68 | * Gets name |
69 | - * @return string |
|
69 | + * @return boolean |
|
70 | 70 | */ |
71 | 71 | function name() { |
72 | 72 | return $this->get( 'name' ); |
@@ -77,7 +77,7 @@ discard block |
||
77 | 77 | /** |
78 | 78 | * Sets name |
79 | 79 | * @param string $name |
80 | - * @return boolean |
|
80 | + * @return boolean|null |
|
81 | 81 | */ |
82 | 82 | function set_name( $name ) { |
83 | 83 | $this->set( 'name', $name ); |
@@ -87,7 +87,7 @@ discard block |
||
87 | 87 | |
88 | 88 | /** |
89 | 89 | * Gets slug |
90 | - * @return string |
|
90 | + * @return boolean |
|
91 | 91 | */ |
92 | 92 | function slug() { |
93 | 93 | return $this->get( 'slug' ); |
@@ -98,7 +98,7 @@ discard block |
||
98 | 98 | /** |
99 | 99 | * Sets slug |
100 | 100 | * @param string $slug |
101 | - * @return boolean |
|
101 | + * @return boolean|null |
|
102 | 102 | */ |
103 | 103 | function set_slug( $slug ) { |
104 | 104 | $this->set( 'slug', $slug ); |
@@ -1,5 +1,5 @@ discard block |
||
1 | -<?php if ( !defined( 'EVENT_ESPRESSO_VERSION' ) ) { |
|
2 | - exit( 'No direct script access allowed' ); |
|
1 | +<?php if ( ! defined('EVENT_ESPRESSO_VERSION')) { |
|
2 | + exit('No direct script access allowed'); |
|
3 | 3 | } |
4 | 4 | /** |
5 | 5 | * Event Espresso |
@@ -34,11 +34,11 @@ discard block |
||
34 | 34 | * @param bool $bydb |
35 | 35 | * @param string $timezone |
36 | 36 | */ |
37 | - protected function __construct( $fieldValues = array(), $bydb = FALSE, $timezone = '' ) { |
|
38 | - if ( ! isset( $fieldValues[ 'slug' ] ) ) { |
|
39 | - $fieldValues[ 'slug' ] = $fieldValues[ 'name' ]; |
|
37 | + protected function __construct($fieldValues = array(), $bydb = FALSE, $timezone = '') { |
|
38 | + if ( ! isset($fieldValues['slug'])) { |
|
39 | + $fieldValues['slug'] = $fieldValues['name']; |
|
40 | 40 | } |
41 | - parent::__construct( $fieldValues, $bydb, $timezone ); |
|
41 | + parent::__construct($fieldValues, $bydb, $timezone); |
|
42 | 42 | } |
43 | 43 | |
44 | 44 | |
@@ -47,9 +47,9 @@ discard block |
||
47 | 47 | * @param array $props_n_values |
48 | 48 | * @return EE_Term|mixed |
49 | 49 | */ |
50 | - public static function new_instance( $props_n_values = array() ) { |
|
51 | - $has_object = parent::_check_for_object( $props_n_values, __CLASS__ ); |
|
52 | - return $has_object ? $has_object : new self( $props_n_values ); |
|
50 | + public static function new_instance($props_n_values = array()) { |
|
51 | + $has_object = parent::_check_for_object($props_n_values, __CLASS__); |
|
52 | + return $has_object ? $has_object : new self($props_n_values); |
|
53 | 53 | } |
54 | 54 | |
55 | 55 | |
@@ -58,8 +58,8 @@ discard block |
||
58 | 58 | * @param array $props_n_values |
59 | 59 | * @return EE_Term |
60 | 60 | */ |
61 | - public static function new_instance_from_db( $props_n_values = array() ) { |
|
62 | - return new self( $props_n_values, TRUE ); |
|
61 | + public static function new_instance_from_db($props_n_values = array()) { |
|
62 | + return new self($props_n_values, TRUE); |
|
63 | 63 | } |
64 | 64 | |
65 | 65 | |
@@ -69,7 +69,7 @@ discard block |
||
69 | 69 | * @return string |
70 | 70 | */ |
71 | 71 | function name() { |
72 | - return $this->get( 'name' ); |
|
72 | + return $this->get('name'); |
|
73 | 73 | } |
74 | 74 | |
75 | 75 | |
@@ -79,8 +79,8 @@ discard block |
||
79 | 79 | * @param string $name |
80 | 80 | * @return boolean |
81 | 81 | */ |
82 | - function set_name( $name ) { |
|
83 | - $this->set( 'name', $name ); |
|
82 | + function set_name($name) { |
|
83 | + $this->set('name', $name); |
|
84 | 84 | } |
85 | 85 | |
86 | 86 | |
@@ -90,7 +90,7 @@ discard block |
||
90 | 90 | * @return string |
91 | 91 | */ |
92 | 92 | function slug() { |
93 | - return $this->get( 'slug' ); |
|
93 | + return $this->get('slug'); |
|
94 | 94 | } |
95 | 95 | |
96 | 96 | |
@@ -100,8 +100,8 @@ discard block |
||
100 | 100 | * @param string $slug |
101 | 101 | * @return boolean |
102 | 102 | */ |
103 | - function set_slug( $slug ) { |
|
104 | - $this->set( 'slug', $slug ); |
|
103 | + function set_slug($slug) { |
|
104 | + $this->set('slug', $slug); |
|
105 | 105 | } |
106 | 106 | } |
107 | 107 |