Completed
Branch BUG-5580-mike-implementation (468206)
by
unknown
15:25 queued 13s
created
core/libraries/form_sections/base/EE_Form_Section_Proper.form.php 3 patches
Braces   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -209,7 +209,7 @@  discard block
 block discarded – undo
209 209
 			if(isset($default_data[$subsection_name])){
210 210
 				if($subsection instanceof EE_Form_Input_Base){
211 211
 					$subsection->set_default($default_data[$subsection_name]);
212
-				}elseif($subsection instanceof EE_Form_Section_Proper){
212
+				} elseif($subsection instanceof EE_Form_Section_Proper){
213 213
 					$subsection->populate_defaults($default_data[$subsection_name]);
214 214
 				}
215 215
 			}
@@ -440,7 +440,7 @@  discard block
 block discarded – undo
440 440
 		foreach($this->subsections() as $subsection){
441 441
 			if( $subsection instanceof EE_Form_Input_Base ){
442 442
 				$inputs[ $subsection->html_name() ] = $subsection;
443
-			}elseif($subsection instanceof EE_Form_Section_Proper ){
443
+			} elseif($subsection instanceof EE_Form_Section_Proper ){
444 444
 				$inputs += $subsection->inputs_in_subsections();
445 445
 			}
446 446
 		}
@@ -548,7 +548,7 @@  discard block
 block discarded – undo
548 548
 		foreach($this->get_validatable_subsections() as $subsection){
549 549
 			try{
550 550
 				$subsection->_normalize($req_data);
551
-			}catch( EE_Validation_Error $e ){
551
+			} catch( EE_Validation_Error $e ){
552 552
 				$subsection->add_validation_error( $e );
553 553
 			}
554 554
 		}
@@ -866,7 +866,7 @@  discard block
 block discarded – undo
866 866
 	public function html_name_prefix(){
867 867
 		if( $this->parent_section() instanceof EE_Form_Section_Proper ){
868 868
 			return $this->parent_section()->html_name_prefix() . '[' . $this->name() . ']';
869
-		}else{
869
+		} else{
870 870
 			return $this->name();
871 871
 		}
872 872
 	}
@@ -918,7 +918,7 @@  discard block
 block discarded – undo
918 918
 				if( $subsection->form_data_present_in( $req_data ) ) {
919 919
 					return TRUE;
920 920
 				}
921
-			}elseif( $subsection instanceof EE_Form_Section_Proper ) {
921
+			} elseif( $subsection instanceof EE_Form_Section_Proper ) {
922 922
 				if( $subsection->form_data_present_in( $req_data ) ) {
923 923
 					return TRUE;
924 924
 				}
Please login to merge, or discard this patch.
Spacing   +157 added lines, -157 removed lines patch added patch discarded remove patch
@@ -7,7 +7,7 @@  discard block
 block discarded – undo
7 7
  * before the hook wp_enqueue_scripts is called (so that the form section can enqueue its needed scripts).
8 8
  * However, you may output the form (usually by calling get_html_and_js) anywhere you like.
9 9
  */
10
-class EE_Form_Section_Proper extends EE_Form_Section_Validatable{
10
+class EE_Form_Section_Proper extends EE_Form_Section_Validatable {
11 11
 
12 12
 	/**
13 13
 	 * Subsections
@@ -68,38 +68,38 @@  discard block
 block discarded – undo
68 68
 	 * } @see EE_Form_Section_Validatable::__construct()
69 69
 	 *
70 70
 	 */
71
-	public function __construct( $options_array = array() ){
71
+	public function __construct($options_array = array()) {
72 72
 		EE_Registry::instance()->load_helper('Formatter');
73
-		$options_array = apply_filters( 'FHEE__EE_Form_Section_Proper___construct__options_array', $options_array, $this );
73
+		$options_array = apply_filters('FHEE__EE_Form_Section_Proper___construct__options_array', $options_array, $this);
74 74
 		//call parent first, as it may be setting the name
75 75
 		parent::__construct($options_array);
76 76
 		//if they've included subsections in the constructor, add them now
77
-		if( isset( $options_array['include'] )){
77
+		if (isset($options_array['include'])) {
78 78
 			//we are going to make sure we ONLY have those subsections to include
79 79
 			//AND we are going to make sure they're in that specified order
80 80
 			$reordered_subsections = array();
81
-			foreach($options_array['include'] as $input_name){
82
-				if(isset($this->_subsections[$input_name])){
81
+			foreach ($options_array['include'] as $input_name) {
82
+				if (isset($this->_subsections[$input_name])) {
83 83
 					$reordered_subsections[$input_name] = $this->_subsections[$input_name];
84 84
 				}
85 85
 			}
86 86
 			$this->_subsections = $reordered_subsections;
87 87
 		}
88
-		if(isset($options_array['exclude'])){
88
+		if (isset($options_array['exclude'])) {
89 89
 			$exclude = $options_array['exclude'];
90 90
 			$this->_subsections = array_diff_key($this->_subsections, array_flip($exclude));
91 91
 		}
92
-		if(isset($options_array['layout_strategy'])){
92
+		if (isset($options_array['layout_strategy'])) {
93 93
 			$this->_layout_strategy = $options_array['layout_strategy'];
94 94
 		}
95
-		if( ! $this->_layout_strategy){
95
+		if ( ! $this->_layout_strategy) {
96 96
 			$this->_layout_strategy = new EE_Two_Column_Layout();
97 97
 		}
98 98
 		$this->_layout_strategy->_construct_finalize($this);
99 99
 
100
-		add_action( 'wp_enqueue_scripts', array( 'EE_Form_Section_Proper', 'wp_enqueue_scripts' ));
101
-		add_action( 'admin_enqueue_scripts', array( 'EE_Form_Section_Proper', 'wp_enqueue_scripts' ));
102
-		add_action( 'wp_footer', array( $this, 'ensure_scripts_localized' ), 1 );
100
+		add_action('wp_enqueue_scripts', array('EE_Form_Section_Proper', 'wp_enqueue_scripts'));
101
+		add_action('admin_enqueue_scripts', array('EE_Form_Section_Proper', 'wp_enqueue_scripts'));
102
+		add_action('wp_footer', array($this, 'ensure_scripts_localized'), 1);
103 103
 	}
104 104
 
105 105
 
@@ -111,25 +111,25 @@  discard block
 block discarded – undo
111 111
 	 * @param string                 $name
112 112
 	 * @throws \EE_Error
113 113
 	 */
114
-	public function _construct_finalize( $parent_form_section, $name ) {
114
+	public function _construct_finalize($parent_form_section, $name) {
115 115
 		parent::_construct_finalize($parent_form_section, $name);
116 116
 		$this->_set_default_name_if_empty();
117 117
 		$this->_set_default_html_id_if_empty();
118
-		foreach( $this->_subsections as $subsection_name => $subsection ){
119
-			if ( $subsection instanceof EE_Form_Section_Base ) {
120
-				$subsection->_construct_finalize( $this, $subsection_name );
118
+		foreach ($this->_subsections as $subsection_name => $subsection) {
119
+			if ($subsection instanceof EE_Form_Section_Base) {
120
+				$subsection->_construct_finalize($this, $subsection_name);
121 121
 			} else {
122 122
 				throw new EE_Error(
123 123
 					sprintf(
124
-						__( 'Subsection "%s" is not an instanceof EE_Form_Section_Base on form "%s". It is a "%s"', 'event_espresso' ),
124
+						__('Subsection "%s" is not an instanceof EE_Form_Section_Base on form "%s". It is a "%s"', 'event_espresso'),
125 125
 						$subsection_name,
126 126
 						get_class($this),
127
-						$subsection ? get_class($subsection) : __( 'NULL', 'event_espresso' )
127
+						$subsection ? get_class($subsection) : __('NULL', 'event_espresso')
128 128
 					)
129 129
 				);
130 130
 			}
131 131
 		}
132
-		do_action( 'AHEE__EE_Form_Section_Proper___construct_finalize__end', $this, $parent_form_section, $name );
132
+		do_action('AHEE__EE_Form_Section_Proper___construct_finalize__end', $this, $parent_form_section, $name);
133 133
 	}
134 134
 
135 135
 
@@ -138,7 +138,7 @@  discard block
 block discarded – undo
138 138
 	 * Gets the layout strategy for this form section
139 139
 	 * @return EE_Form_Section_Layout_Base
140 140
 	 */
141
-	public function get_layout_strategy(){
141
+	public function get_layout_strategy() {
142 142
 		return $this->_layout_strategy;
143 143
 	}
144 144
 
@@ -150,7 +150,7 @@  discard block
 block discarded – undo
150 150
 	 * @param EE_Form_Input_Base $input
151 151
 	 * @return string
152 152
 	 */
153
-	public function get_html_for_input($input){
153
+	public function get_html_for_input($input) {
154 154
 		return $this->_layout_strategy->layout_input($input);
155 155
 	}
156 156
 
@@ -163,7 +163,7 @@  discard block
 block discarded – undo
163 163
 	 * @param null $form_data
164 164
 	 * @return boolean
165 165
 	 */
166
-	public function was_submitted($form_data = NULL){
166
+	public function was_submitted($form_data = NULL) {
167 167
 		return $this->form_data_present_in($form_data);
168 168
 	}
169 169
 
@@ -184,12 +184,12 @@  discard block
 block discarded – undo
184 184
 	 * to skip this step.
185 185
 	 * @return void
186 186
 	 */
187
-	public function receive_form_submission($req_data = NULL, $validate = TRUE){
188
-		if( $req_data === NULL){
187
+	public function receive_form_submission($req_data = NULL, $validate = TRUE) {
188
+		if ($req_data === NULL) {
189 189
 			$req_data = $_REQUEST;
190 190
 		}
191 191
 		$this->_normalize($req_data);
192
-		if( $validate ){
192
+		if ($validate) {
193 193
 			$this->_validate();
194 194
 		}
195 195
 	}
@@ -204,12 +204,12 @@  discard block
 block discarded – undo
204 204
 	 * the value being an array formatted in teh same way
205 205
 	 * @param array $default_data
206 206
 	 */
207
-	public function populate_defaults($default_data){
208
-		foreach($this->subsections() as $subsection_name => $subsection){
209
-			if(isset($default_data[$subsection_name])){
210
-				if($subsection instanceof EE_Form_Input_Base){
207
+	public function populate_defaults($default_data) {
208
+		foreach ($this->subsections() as $subsection_name => $subsection) {
209
+			if (isset($default_data[$subsection_name])) {
210
+				if ($subsection instanceof EE_Form_Input_Base) {
211 211
 					$subsection->set_default($default_data[$subsection_name]);
212
-				}elseif($subsection instanceof EE_Form_Section_Proper){
212
+				}elseif ($subsection instanceof EE_Form_Section_Proper) {
213 213
 					$subsection->populate_defaults($default_data[$subsection_name]);
214 214
 				}
215 215
 			}
@@ -227,8 +227,8 @@  discard block
 block discarded – undo
227 227
 	 * (realizing that the subsections' html names might not be set yet, etc.)
228 228
 	 * @return EE_Form_Section_Base
229 229
 	 */
230
-	public function get_subsection($name, $require_construction_to_be_finalized = TRUE ){
231
-		if( $require_construction_to_be_finalized ){
230
+	public function get_subsection($name, $require_construction_to_be_finalized = TRUE) {
231
+		if ($require_construction_to_be_finalized) {
232 232
 			$this->ensure_construct_finalized_called();
233 233
 		}
234 234
 		return isset($this->_subsections[$name]) ? $this->_subsections[$name] : NULL;
@@ -240,10 +240,10 @@  discard block
 block discarded – undo
240 240
 	 * Gets all the validatable subsections of this form section
241 241
 	 * @return EE_Form_Section_Validatable[]
242 242
 	 */
243
-	public function get_validatable_subsections(){
243
+	public function get_validatable_subsections() {
244 244
 		$validatable_subsections = array();
245
-		foreach($this->subsections() as $name=>$obj){
246
-			if($obj instanceof EE_Form_Section_Validatable){
245
+		foreach ($this->subsections() as $name=>$obj) {
246
+			if ($obj instanceof EE_Form_Section_Validatable) {
247 247
 				$validatable_subsections[$name] = $obj;
248 248
 			}
249 249
 		}
@@ -263,10 +263,10 @@  discard block
 block discarded – undo
263 263
 	 * @return EE_Form_Input_Base
264 264
 	 * @throws EE_Error
265 265
 	 */
266
-	public function get_input($name, $require_construction_to_be_finalized = TRUE ){
266
+	public function get_input($name, $require_construction_to_be_finalized = TRUE) {
267 267
 		$subsection = $this->get_subsection($name, $require_construction_to_be_finalized);
268
-		if( ! $subsection instanceof EE_Form_Input_Base){
269
-			throw new EE_Error(sprintf(__("Subsection '%s' is not an instanceof EE_Form_Input_Base on form '%s'. It is a '%s'", 'event_espresso'),$name, get_class($this),$subsection ? get_class($subsection) : __("NULL", 'event_espresso')));
268
+		if ( ! $subsection instanceof EE_Form_Input_Base) {
269
+			throw new EE_Error(sprintf(__("Subsection '%s' is not an instanceof EE_Form_Input_Base on form '%s'. It is a '%s'", 'event_espresso'), $name, get_class($this), $subsection ? get_class($subsection) : __("NULL", 'event_espresso')));
270 270
 		}
271 271
 		return $subsection;
272 272
 	}
@@ -284,10 +284,10 @@  discard block
 block discarded – undo
284 284
 	 * @return EE_Form_Section_Proper
285 285
 	 * @throws EE_Error
286 286
 	 */
287
-	public function get_proper_subsection($name, $require_construction_to_be_finalized = TRUE ){
288
-		$subsection = $this->get_subsection( $name, $require_construction_to_be_finalized );
289
-		if( ! $subsection instanceof EE_Form_Section_Proper){
290
-			throw new EE_Error(sprintf(__("Subsection '%'s is not an instanceof EE_Form_Section_Proper on form '%s'", 'event_espresso'),$name, get_class($this)));
287
+	public function get_proper_subsection($name, $require_construction_to_be_finalized = TRUE) {
288
+		$subsection = $this->get_subsection($name, $require_construction_to_be_finalized);
289
+		if ( ! $subsection instanceof EE_Form_Section_Proper) {
290
+			throw new EE_Error(sprintf(__("Subsection '%'s is not an instanceof EE_Form_Section_Proper on form '%s'", 'event_espresso'), $name, get_class($this)));
291 291
 		}
292 292
 		return $subsection;
293 293
 	}
@@ -300,7 +300,7 @@  discard block
 block discarded – undo
300 300
 	 * @param string $name
301 301
 	 * @return mixed depending on the input's type and its normalization strategy
302 302
 	 */
303
-	public function get_input_value($name){
303
+	public function get_input_value($name) {
304 304
 		$input = $this->get_input($name);
305 305
 		return $input->normalized_value();
306 306
 	}
@@ -313,16 +313,16 @@  discard block
 block discarded – undo
313 313
 	 * @return boolean
314 314
 	 */
315 315
 	public function is_valid() {
316
-		if( ! $this->has_received_submission()){
316
+		if ( ! $this->has_received_submission()) {
317 317
 			throw new EE_Error(sprintf(__("You cannot check if a form is valid before receiving the form submission using receive_form_submission", "event_espresso")));
318 318
 		}
319
-		if( ! parent::is_valid()){
319
+		if ( ! parent::is_valid()) {
320 320
 			return false;
321 321
 		}
322 322
 		//ok so no errors general to this entire form section. so let's check the subsections
323
-		foreach( $this->get_validatable_subsections() as $subsection ){
324
-			if( ! $subsection->is_valid() || $subsection->get_validation_error_string() != '' ){
325
-				$this->set_submission_error_message( $subsection->get_validation_error_string() );
323
+		foreach ($this->get_validatable_subsections() as $subsection) {
324
+			if ( ! $subsection->is_valid() || $subsection->get_validation_error_string() != '') {
325
+				$this->set_submission_error_message($subsection->get_validation_error_string());
326 326
 				return false;
327 327
 			}
328 328
 		}
@@ -335,11 +335,11 @@  discard block
 block discarded – undo
335 335
 	 * gets teh default name of this form section if none is specified
336 336
 	 * @return string
337 337
 	 */
338
-	protected function _set_default_name_if_empty(){
339
-		if( ! $this->_name ){
338
+	protected function _set_default_name_if_empty() {
339
+		if ( ! $this->_name) {
340 340
 			$classname = get_class($this);
341 341
 			$default_name = str_replace("EE_", "", $classname);
342
-			$this->_name =  $default_name;
342
+			$this->_name = $default_name;
343 343
 		}
344 344
 	}
345 345
 
@@ -350,7 +350,7 @@  discard block
 block discarded – undo
350 350
 	 * Also returns the HTML for the form, except for the form opening and closing tags
351 351
 	 * (as the form section doesn't know where you necessarily want to send the information to), and except for a submit button.
352 352
 	 */
353
-	public function get_html_and_js(){
353
+	public function get_html_and_js() {
354 354
 		$this->enqueue_js();
355 355
 		return $this->get_html();
356 356
 	}
@@ -361,7 +361,7 @@  discard block
 block discarded – undo
361 361
 	 * returns HTML for displaying this form section. recursively calls display_section() on all subsections
362 362
 	 * @return string
363 363
 	 */
364
-	public function get_html(){
364
+	public function get_html() {
365 365
 		$this->ensure_construct_finalized_called();
366 366
 		return $this->_layout_strategy->layout_form();
367 367
 	}
@@ -372,7 +372,7 @@  discard block
 block discarded – undo
372 372
 	 * enqueues JS for the form
373 373
 	 * @return string
374 374
 	 */
375
-	public function enqueue_js(){
375
+	public function enqueue_js() {
376 376
 		$this->_enqueue_and_localize_form_js();
377 377
 	}
378 378
 
@@ -387,9 +387,9 @@  discard block
 block discarded – undo
387 387
 	 * could change until it's actually outputted)
388 388
 	 * @return void
389 389
 	 */
390
-	public static function wp_enqueue_scripts(){
391
-		add_filter( 'FHEE_load_jquery_validate', '__return_true' );
392
-		wp_register_script( 'ee_form_section_validation', EE_GLOBAL_ASSETS_URL . 'scripts' . DS . 'form_section_validation.js', array( 'jquery-validate', 'jquery-ui-datepicker' ), EVENT_ESPRESSO_VERSION, TRUE );
390
+	public static function wp_enqueue_scripts() {
391
+		add_filter('FHEE_load_jquery_validate', '__return_true');
392
+		wp_register_script('ee_form_section_validation', EE_GLOBAL_ASSETS_URL.'scripts'.DS.'form_section_validation.js', array('jquery-validate', 'jquery-ui-datepicker'), EVENT_ESPRESSO_VERSION, TRUE);
393 393
 	}
394 394
 
395 395
 
@@ -399,14 +399,14 @@  discard block
 block discarded – undo
399 399
 	 * This needs to be called AFTER we've called $this->_enqueue_jquery_validate_script,
400 400
 	 * but before the wordpress hook wp_loaded
401 401
 	 */
402
-	public function _enqueue_and_localize_form_js(){
402
+	public function _enqueue_and_localize_form_js() {
403 403
 		$this->ensure_construct_finalized_called();
404 404
 		//actually, we don't want to localize just yet. There may be other forms on the page.
405 405
 		//so we need to add our form section data to a static variable accessible by all form sections
406 406
 		//and localize it just before the footer
407 407
 		$this->localize_validation_rules();
408
-		add_action( 'wp_footer', array( 'EE_Form_Section_Proper', 'localize_script_for_all_forms' ), 2 );
409
-		add_action( 'admin_footer', array( 'EE_Form_Section_Proper', 'localize_script_for_all_forms' ) );
408
+		add_action('wp_footer', array('EE_Form_Section_Proper', 'localize_script_for_all_forms'), 2);
409
+		add_action('admin_footer', array('EE_Form_Section_Proper', 'localize_script_for_all_forms'));
410 410
 	}
411 411
 
412 412
 
@@ -416,11 +416,11 @@  discard block
 block discarded – undo
416 416
 	 * @param bool $return_for_subsection
417 417
 	 * @return void
418 418
 	 */
419
-	public function localize_validation_rules( $return_for_subsection = FALSE ){
419
+	public function localize_validation_rules($return_for_subsection = FALSE) {
420 420
 		// we only want to localize vars ONCE for the entire form, so if the form section doesn't have a parent, then it must be the top dog
421
-		if ( ! $this->parent_section() || $return_for_subsection ) {
422
-			EE_Form_Section_Proper::$_js_localization['form_data'][ $this->html_id() ] = array(
423
-				'form_section_id'=> $this->html_id( TRUE ),
421
+		if ( ! $this->parent_section() || $return_for_subsection) {
422
+			EE_Form_Section_Proper::$_js_localization['form_data'][$this->html_id()] = array(
423
+				'form_section_id'=> $this->html_id(TRUE),
424 424
 				'validation_rules'=> $this->get_jquery_validation_rules(),
425 425
 				'errors'=> $this->subsection_validation_errors_by_html_name()
426 426
 			);
@@ -435,12 +435,12 @@  discard block
 block discarded – undo
435 435
 	 * Keys are their form names, and values are the inputs themselves
436 436
 	 * @return EE_Form_Input_Base
437 437
 	 */
438
-	public function inputs_in_subsections(){
438
+	public function inputs_in_subsections() {
439 439
 		$inputs = array();
440
-		foreach($this->subsections() as $subsection){
441
-			if( $subsection instanceof EE_Form_Input_Base ){
442
-				$inputs[ $subsection->html_name() ] = $subsection;
443
-			}elseif($subsection instanceof EE_Form_Section_Proper ){
440
+		foreach ($this->subsections() as $subsection) {
441
+			if ($subsection instanceof EE_Form_Input_Base) {
442
+				$inputs[$subsection->html_name()] = $subsection;
443
+			}elseif ($subsection instanceof EE_Form_Section_Proper) {
444 444
 				$inputs += $subsection->inputs_in_subsections();
445 445
 			}
446 446
 		}
@@ -453,12 +453,12 @@  discard block
 block discarded – undo
453 453
 	 * and values are a string of all their validation errors
454 454
 	 * @return string[]
455 455
 	 */
456
-	public function subsection_validation_errors_by_html_name(){
456
+	public function subsection_validation_errors_by_html_name() {
457 457
 		$inputs = $this->inputs();
458 458
 		$errors = array();
459
-		foreach( $inputs as $form_input ){
460
-			if ( $form_input instanceof EE_Form_Input_Base && $form_input->get_validation_errors() ){
461
-				$errors[ $form_input->html_name() ] = $form_input->get_validation_error_string();
459
+		foreach ($inputs as $form_input) {
460
+			if ($form_input instanceof EE_Form_Input_Base && $form_input->get_validation_errors()) {
461
+				$errors[$form_input->html_name()] = $form_input->get_validation_error_string();
462 462
 			}
463 463
 		}
464 464
 		return $errors;
@@ -470,12 +470,12 @@  discard block
 block discarded – undo
470 470
 	 * passes all the form data required by the JS to the JS, and enqueues the few required JS files.
471 471
 	 * Should be setup by each form during the _enqueues_and_localize_form_js
472 472
 	 */
473
-	public static function localize_script_for_all_forms(){
473
+	public static function localize_script_for_all_forms() {
474 474
 		//allow inputs and stuff to hook in their JS and stuff here
475 475
 		do_action('AHEE__EE_Form_Section_Proper__localize_script_for_all_forms__begin');
476 476
 		EE_Form_Section_Proper::$_js_localization['localized_error_messages'] = EE_Form_Section_Proper::_get_localized_error_messages();
477
-		wp_enqueue_script( 'ee_form_section_validation' );
478
-		wp_localize_script( 'ee_form_section_validation', 'ee_form_section_vars', EE_Form_Section_Proper::$_js_localization );
477
+		wp_enqueue_script('ee_form_section_validation');
478
+		wp_localize_script('ee_form_section_validation', 'ee_form_section_vars', EE_Form_Section_Proper::$_js_localization);
479 479
 	}
480 480
 
481 481
 
@@ -483,8 +483,8 @@  discard block
 block discarded – undo
483 483
 	/**
484 484
 	 * ensure_scripts_localized
485 485
 	 */
486
-	public function ensure_scripts_localized(){
487
-		if ( ! EE_Form_Section_Proper::$_scripts_localized ) {
486
+	public function ensure_scripts_localized() {
487
+		if ( ! EE_Form_Section_Proper::$_scripts_localized) {
488 488
 			$this->_enqueue_and_localize_form_js();
489 489
 		}
490 490
 	}
@@ -496,10 +496,10 @@  discard block
 block discarded – undo
496 496
 	 * is that the key here should be the same as the custom validation rule put in the JS file
497 497
 	 * @return array keys are custom validation rules, and values are internationalized strings
498 498
 	 */
499
-	private static function _get_localized_error_messages(){
499
+	private static function _get_localized_error_messages() {
500 500
 		return array(
501 501
 			'validUrl'=>  __("This is not a valid absolute URL. Eg, http://domain.com/monkey.jpg", "event_espresso"),
502
-			'regex' => __( 'Please check your input', 'event_espresso' ),
502
+			'regex' => __('Please check your input', 'event_espresso'),
503 503
 		);
504 504
 	}
505 505
 
@@ -527,10 +527,10 @@  discard block
 block discarded – undo
527 527
 	 * Gets the JS to put inside the jquery validation rules for subsection of this form section. See parent function for more...
528 528
 	 * @return array
529 529
 	 */
530
-	function get_jquery_validation_rules(){
530
+	function get_jquery_validation_rules() {
531 531
 		$jquery_validation_rules = array();
532
-		foreach($this->get_validatable_subsections() as $subsection){
533
-			$jquery_validation_rules = array_merge( $jquery_validation_rules,  $subsection->get_jquery_validation_rules() );
532
+		foreach ($this->get_validatable_subsections() as $subsection) {
533
+			$jquery_validation_rules = array_merge($jquery_validation_rules, $subsection->get_jquery_validation_rules());
534 534
 		}
535 535
 		return $jquery_validation_rules;
536 536
 	}
@@ -545,11 +545,11 @@  discard block
 block discarded – undo
545 545
 	protected function _normalize($req_data) {
546 546
 		$this->_received_submission = TRUE;
547 547
 		$this->_validation_errors = array();
548
-		foreach($this->get_validatable_subsections() as $subsection){
549
-			try{
548
+		foreach ($this->get_validatable_subsections() as $subsection) {
549
+			try {
550 550
 				$subsection->_normalize($req_data);
551
-			}catch( EE_Validation_Error $e ){
552
-				$subsection->add_validation_error( $e );
551
+			} catch (EE_Validation_Error $e) {
552
+				$subsection->add_validation_error($e);
553 553
 			}
554 554
 		}
555 555
 	}
@@ -563,9 +563,9 @@  discard block
 block discarded – undo
563 563
 	 * calling parent::_validate() first.
564 564
 	 */
565 565
 	protected function _validate() {
566
-		foreach($this->get_validatable_subsections() as $subsection_name => $subsection){
567
-			if(method_exists($this,'_validate_'.$subsection_name)){
568
-				call_user_func_array(array($this,'_validate_'.$subsection_name), array($subsection));
566
+		foreach ($this->get_validatable_subsections() as $subsection_name => $subsection) {
567
+			if (method_exists($this, '_validate_'.$subsection_name)) {
568
+				call_user_func_array(array($this, '_validate_'.$subsection_name), array($subsection));
569 569
 			}
570 570
 			$subsection->_validate();
571 571
 		}
@@ -577,13 +577,13 @@  discard block
 block discarded – undo
577 577
 	 * Gets all the validated inputs for the form section
578 578
 	 * @return array
579 579
 	 */
580
-	public function valid_data(){
580
+	public function valid_data() {
581 581
 		$inputs = array();
582
-		foreach( $this->subsections() as $subsection_name =>$subsection ){
583
-			if ( $subsection instanceof EE_Form_Section_Proper ) {
584
-				$inputs[ $subsection_name ] = $subsection->valid_data();
585
-			} else if ( $subsection instanceof EE_Form_Input_Base ){
586
-				$inputs[ $subsection_name ] = $subsection->normalized_value();
582
+		foreach ($this->subsections() as $subsection_name =>$subsection) {
583
+			if ($subsection instanceof EE_Form_Section_Proper) {
584
+				$inputs[$subsection_name] = $subsection->valid_data();
585
+			} else if ($subsection instanceof EE_Form_Input_Base) {
586
+				$inputs[$subsection_name] = $subsection->normalized_value();
587 587
 			}
588 588
 		}
589 589
 		return $inputs;
@@ -595,11 +595,11 @@  discard block
 block discarded – undo
595 595
 	 * Gets all the inputs on this form section
596 596
 	 * @return EE_Form_Input_Base[]
597 597
 	 */
598
-	public function inputs(){
598
+	public function inputs() {
599 599
 		$inputs = array();
600
-		foreach( $this->subsections() as $subsection_name =>$subsection ){
601
-			if ( $subsection instanceof EE_Form_Input_Base ){
602
-				$inputs[ $subsection_name ] = $subsection;
600
+		foreach ($this->subsections() as $subsection_name =>$subsection) {
601
+			if ($subsection instanceof EE_Form_Input_Base) {
602
+				$inputs[$subsection_name] = $subsection;
603 603
 			}
604 604
 		}
605 605
 		return $inputs;
@@ -611,10 +611,10 @@  discard block
 block discarded – undo
611 611
 	 * Gets all the subsections which are a proper form
612 612
 	 * @return EE_Form_Section_Proper[]
613 613
 	 */
614
-	public function subforms(){
614
+	public function subforms() {
615 615
 		$form_sections = array();
616
-		foreach($this->subsections() as $name=>$obj){
617
-			if($obj instanceof EE_Form_Section_Proper){
616
+		foreach ($this->subsections() as $name=>$obj) {
617
+			if ($obj instanceof EE_Form_Section_Proper) {
618 618
 				$form_sections[$name] = $obj;
619 619
 			}
620 620
 		}
@@ -629,7 +629,7 @@  discard block
 block discarded – undo
629 629
 	 * if you only want form inputs or proper form sections.
630 630
 	 * @return EE_Form_Section_Proper[]
631 631
 	 */
632
-	public function subsections(){
632
+	public function subsections() {
633 633
 		$this->ensure_construct_finalized_called();
634 634
 		return $this->_subsections;
635 635
 	}
@@ -647,8 +647,8 @@  discard block
 block discarded – undo
647 647
 	 * it can be a multidimensional array where keys are always subsection names and values are either the
648 648
 	 * input's normalized value, or an array like the top-level array
649 649
 	 */
650
-	public function input_values( $include_subform_inputs = false, $flatten = false ){
651
-		return $this->_input_values( false, $include_subform_inputs, $flatten );
650
+	public function input_values($include_subform_inputs = false, $flatten = false) {
651
+		return $this->_input_values(false, $include_subform_inputs, $flatten);
652 652
 	}
653 653
 
654 654
 	/**
@@ -664,8 +664,8 @@  discard block
 block discarded – undo
664 664
 	 * it can be a multidimensional array where keys are always subsection names and values are either the
665 665
 	 * input's normalized value, or an array like the top-level array
666 666
 	 */
667
-	public function input_pretty_values(  $include_subform_inputs = false, $flatten = false ){
668
-		return $this->_input_values( true, $include_subform_inputs, $flatten );
667
+	public function input_pretty_values($include_subform_inputs = false, $flatten = false) {
668
+		return $this->_input_values(true, $include_subform_inputs, $flatten);
669 669
 	}
670 670
 
671 671
 	/**
@@ -678,17 +678,17 @@  discard block
 block discarded – undo
678 678
 	 * it can be a multidimensional array where keys are always subsection names and values are either the
679 679
 	 * input's normalized value, or an array like the top-level array
680 680
 	 */
681
-	public function _input_values( $pretty = false, $include_subform_inputs = false, $flatten = false ) {
681
+	public function _input_values($pretty = false, $include_subform_inputs = false, $flatten = false) {
682 682
 		$input_values = array();
683
-		foreach( $this->subsections() as $subsection_name => $subsection ) {
684
-			if( $subsection instanceof EE_Form_Input_Base ) {
685
-				$input_values[ $subsection_name ] = $pretty ? $subsection->pretty_value() : $subsection->normalized_value();
686
-			} else if( $subsection instanceof EE_Form_Section_Proper && $include_subform_inputs ) {
687
-				$subform_input_values = $subsection->_input_values( $pretty, $include_subform_inputs, $flatten );
688
-				if( $flatten ) {
689
-					$input_values = array_merge( $input_values, $subform_input_values );
683
+		foreach ($this->subsections() as $subsection_name => $subsection) {
684
+			if ($subsection instanceof EE_Form_Input_Base) {
685
+				$input_values[$subsection_name] = $pretty ? $subsection->pretty_value() : $subsection->normalized_value();
686
+			} else if ($subsection instanceof EE_Form_Section_Proper && $include_subform_inputs) {
687
+				$subform_input_values = $subsection->_input_values($pretty, $include_subform_inputs, $flatten);
688
+				if ($flatten) {
689
+					$input_values = array_merge($input_values, $subform_input_values);
690 690
 				} else {
691
-					$input_values[ $subsection_name ] = $subform_input_values;
691
+					$input_values[$subsection_name] = $subform_input_values;
692 692
 				}
693 693
 			}
694 694
 		}
@@ -702,7 +702,7 @@  discard block
 block discarded – undo
702 702
 	 * (ie, had receive_form_submission called on it yet)
703 703
 	 * @return boolean
704 704
 	 */
705
-	public function has_received_submission(){
705
+	public function has_received_submission() {
706 706
 		$this->ensure_construct_finalized_called();
707 707
 		return $this->_received_submission;
708 708
 	}
@@ -715,8 +715,8 @@  discard block
 block discarded – undo
715 715
 	 * @param array $inputs_to_exclude values are the input names
716 716
 	 * @return void
717 717
 	 */
718
-	public function exclude($inputs_to_exclude = array()){
719
-		foreach($inputs_to_exclude as $input_to_exclude_name){
718
+	public function exclude($inputs_to_exclude = array()) {
719
+		foreach ($inputs_to_exclude as $input_to_exclude_name) {
720 720
 			unset($this->_subsections[$input_to_exclude_name]);
721 721
 		}
722 722
 	}
@@ -726,8 +726,8 @@  discard block
 block discarded – undo
726 726
 	/**
727 727
 	 * @param array $inputs_to_hide
728 728
 	 */
729
-	public function hide($inputs_to_hide= array()){
730
-		foreach($inputs_to_hide as $input_to_hide){
729
+	public function hide($inputs_to_hide = array()) {
730
+		foreach ($inputs_to_hide as $input_to_hide) {
731 731
 			$input = $this->get_input($input_to_hide);
732 732
 
733 733
 			$input->set_display_strategy(new EE_Hidden_Display_Strategy());
@@ -752,13 +752,13 @@  discard block
 block discarded – undo
752 752
 	 * 																								or if $subsection_name_to_target is null, before or after entire subsections array
753 753
 	 * @return void
754 754
 	 */
755
-	public function add_subsections( $new_subsections, $subsection_name_to_target = NULL, $add_before = true ){
756
-		foreach($new_subsections as $subsection_name => $subsection){
757
-			if( ! $subsection instanceof EE_Form_Section_Base){
755
+	public function add_subsections($new_subsections, $subsection_name_to_target = NULL, $add_before = true) {
756
+		foreach ($new_subsections as $subsection_name => $subsection) {
757
+			if ( ! $subsection instanceof EE_Form_Section_Base) {
758 758
 				EE_Error::add_error(
759 759
 					sprintf(
760 760
 						__("Trying to add a %s as a subsection (it was named '%s') to the form section '%s'. It was removed.", "event_espresso"),
761
-						get_class( $subsection ),
761
+						get_class($subsection),
762 762
 						$subsection_name,
763 763
 						$this->name()
764 764
 					)
@@ -767,8 +767,8 @@  discard block
 block discarded – undo
767 767
 			}
768 768
 		}
769 769
 
770
-		EE_Registry::instance()->load_helper( 'Array' );
771
-		$this->_subsections = EEH_Array::insert_into_array( $this->_subsections, $new_subsections, $subsection_name_to_target, $add_before );
770
+		EE_Registry::instance()->load_helper('Array');
771
+		$this->_subsections = EEH_Array::insert_into_array($this->_subsections, $new_subsections, $subsection_name_to_target, $add_before);
772 772
 
773 773
 		/*$subsections_before = array();
774 774
 		if( $subsection_name_to_target ){
@@ -799,8 +799,8 @@  discard block
 block discarded – undo
799 799
 				$this->_subsections = $new_subsections;
800 800
 			}
801 801
 		}*/
802
-		if( $this->_construction_finalized ){
803
-			foreach($this->_subsections as $name => $subsection){
802
+		if ($this->_construction_finalized) {
803
+			foreach ($this->_subsections as $name => $subsection) {
804 804
 				$subsection->_construct_finalize($this, $name);
805 805
 			}
806 806
 		}
@@ -811,8 +811,8 @@  discard block
 block discarded – undo
811 811
 	/**
812 812
 	 * Just gets all validatable subsections to clean their sensitive data
813 813
 	 */
814
-	public function clean_sensitive_data(){
815
-		foreach($this->get_validatable_subsections() as $subsection){
814
+	public function clean_sensitive_data() {
815
+		foreach ($this->get_validatable_subsections() as $subsection) {
816 816
 			$subsection->clean_sensitive_data();
817 817
 		}
818 818
 	}
@@ -822,8 +822,8 @@  discard block
 block discarded – undo
822 822
 	/**
823 823
 	 * @param string $form_submission_error_message
824 824
 	 */
825
-	public function set_submission_error_message( $form_submission_error_message = '' ) {
826
-		$this->_form_submission_error_message .= ! empty( $form_submission_error_message ) ? $form_submission_error_message : __( 'Form submission failed due to errors', 'event_espresso' );
825
+	public function set_submission_error_message($form_submission_error_message = '') {
826
+		$this->_form_submission_error_message .= ! empty($form_submission_error_message) ? $form_submission_error_message : __('Form submission failed due to errors', 'event_espresso');
827 827
 	}
828 828
 
829 829
 
@@ -840,8 +840,8 @@  discard block
 block discarded – undo
840 840
 	/**
841 841
 	 * @param string $form_submission_success_message
842 842
 	 */
843
-	public function set_submission_success_message( $form_submission_success_message ) {
844
-		$this->_form_submission_success_message .= ! empty( $form_submission_success_message ) ? $form_submission_success_message : __( 'Form submitted successfully', 'event_espresso' );
843
+	public function set_submission_success_message($form_submission_success_message) {
844
+		$this->_form_submission_success_message .= ! empty($form_submission_success_message) ? $form_submission_success_message : __('Form submitted successfully', 'event_espresso');
845 845
 	}
846 846
 
847 847
 
@@ -863,10 +863,10 @@  discard block
 block discarded – undo
863 863
 	 * EE_Form_Input_Base::_set_default_html_name_if_empty
864 864
 	 * @return string
865 865
 	 */
866
-	public function html_name_prefix(){
867
-		if( $this->parent_section() instanceof EE_Form_Section_Proper ){
868
-			return $this->parent_section()->html_name_prefix() . '[' . $this->name() . ']';
869
-		}else{
866
+	public function html_name_prefix() {
867
+		if ($this->parent_section() instanceof EE_Form_Section_Proper) {
868
+			return $this->parent_section()->html_name_prefix().'['.$this->name().']';
869
+		} else {
870 870
 			return $this->name();
871 871
 		}
872 872
 	}
@@ -877,7 +877,7 @@  discard block
 block discarded – undo
877 877
 	 * was set, which is probably nothing, or the classname)
878 878
 	 * @return string
879 879
 	 */
880
-	public function name(){
880
+	public function name() {
881 881
 		$this->ensure_construct_finalized_called();
882 882
 		return parent::name();
883 883
 	}
@@ -886,7 +886,7 @@  discard block
 block discarded – undo
886 886
 	 *
887 887
 	 * @return EE_Form_Section_Proper
888 888
 	 */
889
-	public function parent_section(){
889
+	public function parent_section() {
890 890
 		$this->ensure_construct_finalized_called();
891 891
 		return parent::parent_section();
892 892
 	}
@@ -895,9 +895,9 @@  discard block
 block discarded – undo
895 895
 	 * make sure construction finalized was called, otherwise children might not be ready
896 896
 	 * @return void
897 897
 	 */
898
-	public function ensure_construct_finalized_called(){
899
-		if( ! $this->_construction_finalized ){
900
-			$this->_construct_finalize($this->_parent_section, $this->_name );
898
+	public function ensure_construct_finalized_called() {
899
+		if ( ! $this->_construction_finalized) {
900
+			$this->_construct_finalize($this->_parent_section, $this->_name);
901 901
 		}
902 902
 	}
903 903
 
@@ -909,17 +909,17 @@  discard block
 block discarded – undo
909 909
 	 * @param array $req_data
910 910
 	 * @return boolean
911 911
 	 */
912
-	public function form_data_present_in( $req_data = NULL ) {
913
-		if( $req_data === NULL){
912
+	public function form_data_present_in($req_data = NULL) {
913
+		if ($req_data === NULL) {
914 914
 			$req_data = $_POST;
915 915
 		}
916
-		foreach( $this->subsections() as $subsection ) {
917
-			if($subsection instanceof EE_Form_Input_Base ) {
918
-				if( $subsection->form_data_present_in( $req_data ) ) {
916
+		foreach ($this->subsections() as $subsection) {
917
+			if ($subsection instanceof EE_Form_Input_Base) {
918
+				if ($subsection->form_data_present_in($req_data)) {
919 919
 					return TRUE;
920 920
 				}
921
-			}elseif( $subsection instanceof EE_Form_Section_Proper ) {
922
-				if( $subsection->form_data_present_in( $req_data ) ) {
921
+			}elseif ($subsection instanceof EE_Form_Section_Proper) {
922
+				if ($subsection->form_data_present_in($req_data)) {
923 923
 					return TRUE;
924 924
 				}
925 925
 			}
@@ -936,14 +936,14 @@  discard block
 block discarded – undo
936 936
 	 */
937 937
 	public function get_validation_errors_accumulated() {
938 938
 		$validation_errors = $this->get_validation_errors();
939
-		foreach($this->get_validatable_subsections() as $subsection ) {
940
-			if( $subsection instanceof EE_Form_Section_Proper ) {
939
+		foreach ($this->get_validatable_subsections() as $subsection) {
940
+			if ($subsection instanceof EE_Form_Section_Proper) {
941 941
 				$validation_errors_on_this_subsection = $subsection->get_validation_errors_accumulated();
942 942
 			} else {
943
-				$validation_errors_on_this_subsection =  $subsection->get_validation_errors();
943
+				$validation_errors_on_this_subsection = $subsection->get_validation_errors();
944 944
 			}
945
-			if( $validation_errors_on_this_subsection ){
946
-				$validation_errors = array_merge( $validation_errors, $validation_errors_on_this_subsection );
945
+			if ($validation_errors_on_this_subsection) {
946
+				$validation_errors = array_merge($validation_errors, $validation_errors_on_this_subsection);
947 947
 			}
948 948
 		}
949 949
 		return $validation_errors;
Please login to merge, or discard this patch.
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -225,7 +225,7 @@
 block discarded – undo
225 225
 	 * leave this as TRUE so that the inputs will be properly configured. However,
226 226
 	 * some client code may be ok with construction finalize being called later
227 227
 	 * (realizing that the subsections' html names might not be set yet, etc.)
228
-	 * @return EE_Form_Section_Base
228
+	 * @return EE_Form_Section_Validatable|null
229 229
 	 */
230 230
 	public function get_subsection($name, $require_construction_to_be_finalized = TRUE ){
231 231
 		if( $require_construction_to_be_finalized ){
Please login to merge, or discard this patch.
core/libraries/form_sections/base/EE_Form_Section_Validatable.form.php 2 patches
Spacing   +15 added lines, -15 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
 /**
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
  *
35 35
  * ------------------------------------------------------------------------
36 36
  */
37
-abstract class EE_Form_Section_Validatable extends EE_Form_Section_Base{
37
+abstract class EE_Form_Section_Validatable extends EE_Form_Section_Base {
38 38
 
39 39
 	/**
40 40
 	 * Array of validation errors in this section. Does not contain validation errors in subsections, however.
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
 	 * called get_validation_errors_accumulated
50 50
 	 * @return EE_Validation_Error[]
51 51
 	 */
52
-	public function get_validation_errors(){
52
+	public function get_validation_errors() {
53 53
 		return $this->_validation_errors;
54 54
 	}
55 55
 	/**
@@ -57,16 +57,16 @@  discard block
 block discarded – undo
57 57
 	 * If we want this to be customizable, we may decide to create a strategy for displaying it.
58 58
 	 * @return string
59 59
 	 */
60
-	public function get_validation_error_string(){
60
+	public function get_validation_error_string() {
61 61
 		$validation_error_messages = array();
62
-		if($this->get_validation_errors()){
63
-			foreach($this->get_validation_errors() as $validation_error){
64
-				if ( $validation_error instanceof EE_Validation_Error ) {
65
-					$validation_error_messages[] =$validation_error->getMessage();
62
+		if ($this->get_validation_errors()) {
63
+			foreach ($this->get_validation_errors() as $validation_error) {
64
+				if ($validation_error instanceof EE_Validation_Error) {
65
+					$validation_error_messages[] = $validation_error->getMessage();
66 66
 				}
67 67
 			}
68 68
 		}
69
-		return implode(", ",$validation_error_messages);
69
+		return implode(", ", $validation_error_messages);
70 70
 	}
71 71
 
72 72
 
@@ -82,9 +82,9 @@  discard block
 block discarded – undo
82 82
 	 * @return boolean
83 83
 	 */
84 84
 	public function is_valid() {
85
-		if(count($this->_validation_errors)){
85
+		if (count($this->_validation_errors)) {
86 86
 			return false;
87
-		}else{
87
+		} else {
88 88
 			return true;
89 89
 		}
90 90
 	}
@@ -109,11 +109,11 @@  discard block
 block discarded – undo
109 109
 	 * @param Exception $previous_exception if there was an exception that caused the error, that exception
110 110
 	 * @return void
111 111
 	 */
112
-	function add_validation_error( $message_or_object, $error_code = NULL, $previous_exception = NULL ){
113
-		if($message_or_object instanceof EE_Validation_Error){
114
-			$validation_error= $message_or_object;
112
+	function add_validation_error($message_or_object, $error_code = NULL, $previous_exception = NULL) {
113
+		if ($message_or_object instanceof EE_Validation_Error) {
114
+			$validation_error = $message_or_object;
115 115
 			$validation_error->set_form_section($this);
116
-		}else{
116
+		} else {
117 117
 			$validation_error = new EE_Validation_Error($message_or_object, $error_code, $this, $previous_exception);
118 118
 		}
119 119
 		$this->_validation_errors[] = $validation_error;
Please login to merge, or discard this patch.
Braces   +4 added lines, -3 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
@@ -84,7 +85,7 @@  discard block
 block discarded – undo
84 85
 	public function is_valid() {
85 86
 		if(count($this->_validation_errors)){
86 87
 			return false;
87
-		}else{
88
+		} else{
88 89
 			return true;
89 90
 		}
90 91
 	}
@@ -113,7 +114,7 @@  discard block
 block discarded – undo
113 114
 		if($message_or_object instanceof EE_Validation_Error){
114 115
 			$validation_error= $message_or_object;
115 116
 			$validation_error->set_form_section($this);
116
-		}else{
117
+		} else{
117 118
 			$validation_error = new EE_Validation_Error($message_or_object, $error_code, $this, $previous_exception);
118 119
 		}
119 120
 		$this->_validation_errors[] = $validation_error;
Please login to merge, or discard this patch.
core/libraries/form_sections/helpers/EE_Validation_Error.error.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-class EE_Validation_Error extends Exception{
3
+class EE_Validation_Error extends Exception {
4 4
 	/**
5 5
 	 * Form Section from which this error originated.
6 6
 	 * @var EE_Form_Section
@@ -20,7 +20,7 @@  discard block
 block discarded – undo
20 20
 	 * @param EE_Form_Section_Validatable $form_section
21 21
 	 * @param Exception $previous if there was an exception that caused this exception
22 22
 	 */
23
-	function __construct( $message = null, $string_code = null,$form_section = null, $previous = null){
23
+	function __construct($message = null, $string_code = null, $form_section = null, $previous = null) {
24 24
 		$this->_form_section = $form_section;
25 25
 		$this->_string_code = $string_code;
26 26
 		parent::__construct($message, 500, $previous);
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 	 * returns teh form section which caused the error.
31 31
 	 * @return EE_Form_Section_Validatable
32 32
 	 */
33
-	public function get_form_section(){
33
+	public function get_form_section() {
34 34
 		return $this->_form_section;
35 35
 	}
36 36
 	/**
@@ -38,7 +38,7 @@  discard block
 block discarded – undo
38 38
 	 * @param EE_Form_Section_Validatable $form_section
39 39
 	 * @return void
40 40
 	 */
41
-	public function set_form_section($form_section){
41
+	public function set_form_section($form_section) {
42 42
 		$this->_form_section = $form_section;
43 43
 	}
44 44
 
Please login to merge, or discard this patch.
core/libraries/form_sections/inputs/EE_Admin_File_Uploader_Input.input.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -8,15 +8,15 @@
 block discarded – undo
8 8
  * @since                4.6
9 9
  *
10 10
  */
11
-class EE_Admin_File_Uploader_Input extends EE_Form_Input_Base{
11
+class EE_Admin_File_Uploader_Input extends EE_Form_Input_Base {
12 12
 
13 13
 	/**
14 14
 	 * @param array $input_settings
15 15
 	 */
16
-	function __construct($input_settings = array()){
16
+	function __construct($input_settings = array()) {
17 17
 		$this->_set_display_strategy(new EE_Admin_File_Uploader_Display_Strategy());
18 18
 		$this->_set_normalization_strategy(new EE_Text_Normalization());
19
-		$this->_add_validation_strategy(new EE_URL_Validation_Strategy( isset( $input_settings[ 'validation_error_message' ] ) ? $input_settings[ 'validation_error_message' ] : NULL ) );
19
+		$this->_add_validation_strategy(new EE_URL_Validation_Strategy(isset($input_settings['validation_error_message']) ? $input_settings['validation_error_message'] : NULL));
20 20
 		parent::__construct($input_settings);
21 21
 	}
22 22
 }
23 23
\ No newline at end of file
Please login to merge, or discard this patch.
core/libraries/form_sections/inputs/EE_Checkbox_Multi_Input.input.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -1,4 +1,4 @@  discard block
 block discarded – undo
1
-<?php if (!defined('EVENT_ESPRESSO_VERSION')) { exit('No direct script access allowed'); }
1
+<?php if ( ! defined('EVENT_ESPRESSO_VERSION')) { exit('No direct script access allowed'); }
2 2
 /**
3 3
  *
4 4
  * Class EE_Checkbox_Multi_Input
@@ -11,16 +11,16 @@  discard block
 block discarded – undo
11 11
  * @since 				$VID:$
12 12
  *
13 13
  */
14
-class EE_Checkbox_Multi_Input extends EE_Form_Input_With_Options_Base{
14
+class EE_Checkbox_Multi_Input extends EE_Form_Input_With_Options_Base {
15 15
 
16 16
 	/**
17 17
 	 * @param array $input_settings
18 18
 	 * @param array | EE_Question_Option[] $answer_options
19 19
 	 */
20
-	public function __construct( $answer_options, $input_settings = array() ) {
21
-		$this->_set_display_strategy( new EE_Checkbox_Display_Strategy() );
22
-		$this->_add_validation_strategy( new EE_Many_Valued_Validation_Strategy( array( new EE_Enum_Validation_Strategy( isset( $input_settings[ 'validation_error_message' ] ) ? $input_settings[ 'validation_error_message' ] : NULL ) )));
20
+	public function __construct($answer_options, $input_settings = array()) {
21
+		$this->_set_display_strategy(new EE_Checkbox_Display_Strategy());
22
+		$this->_add_validation_strategy(new EE_Many_Valued_Validation_Strategy(array(new EE_Enum_Validation_Strategy(isset($input_settings['validation_error_message']) ? $input_settings['validation_error_message'] : NULL))));
23 23
 		$this->_multiple_selections = TRUE;
24
-		parent::__construct( $answer_options, $input_settings );
24
+		parent::__construct($answer_options, $input_settings);
25 25
 	}
26 26
 }
27 27
\ No newline at end of file
Please login to merge, or discard this patch.
core/libraries/form_sections/inputs/EE_Country_Select_Input.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -1,4 +1,4 @@  discard block
 block discarded – undo
1
-<?php  if ( ! defined( 'EVENT_ESPRESSO_VERSION' )) { exit( 'No direct script access allowed' ); }
1
+<?php  if ( ! defined('EVENT_ESPRESSO_VERSION')) { exit('No direct script access allowed'); }
2 2
 /**
3 3
  * Class EE_Country_Select_Input
4 4
  *
@@ -11,20 +11,20 @@  discard block
 block discarded – undo
11 11
  * @since 				$VID:$
12 12
  *
13 13
  */
14
-class EE_Country_Select_Input extends EE_Select_Input{
14
+class EE_Country_Select_Input extends EE_Select_Input {
15 15
 
16 16
 	/**
17 17
 	 * @param array $country_options
18 18
 	 * @param array $input_settings
19 19
 	 * @return EE_Country_Select_Input
20 20
 	 */
21
-	public function __construct( $country_options = NULL, $input_settings = array() ){
21
+	public function __construct($country_options = NULL, $input_settings = array()) {
22 22
 		$country_options = apply_filters(
23 23
 			'FHEE__EE_Country_Select_Input____construct__country_options',
24
-			$this->get_country_answer_options( $country_options ),
24
+			$this->get_country_answer_options($country_options),
25 25
 			$this
26 26
 		);
27
-		parent::__construct( $country_options, $input_settings );
27
+		parent::__construct($country_options, $input_settings);
28 28
 	}
29 29
 
30 30
 
@@ -35,16 +35,16 @@  discard block
 block discarded – undo
35 35
 	 * @param array $country_options
36 36
 	 * @return array
37 37
 	 */
38
-	public function get_country_answer_options( $country_options = NULL ){
38
+	public function get_country_answer_options($country_options = NULL) {
39 39
 		// if passed something that is NOT an array
40
-		if ( ! is_array( $country_options )) {
40
+		if ( ! is_array($country_options)) {
41 41
 			// get possibly cached list of countries
42 42
 			$countries = EEM_Country::instance()->get_all_active_countries();
43
-			if ( ! empty( $countries )) {
44
-				$country_options[ '' ] = '';
45
-				foreach( $countries as $country ){
46
-					if ( $country instanceof EE_Country ) {
47
-						$country_options[ $country->ID() ] = $country->name();
43
+			if ( ! empty($countries)) {
44
+				$country_options[''] = '';
45
+				foreach ($countries as $country) {
46
+					if ($country instanceof EE_Country) {
47
+						$country_options[$country->ID()] = $country->name();
48 48
 					}
49 49
 				}
50 50
 			} else {
Please login to merge, or discard this patch.
core/libraries/form_sections/inputs/EE_Credit_Card_Input.input.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -7,16 +7,16 @@
 block discarded – undo
7 7
  * @subpackage
8 8
  * @author				Mike Nelson
9 9
  */
10
-class EE_Credit_Card_Input extends EE_Form_Input_Base{
10
+class EE_Credit_Card_Input extends EE_Form_Input_Base {
11 11
 
12 12
 	/**
13 13
 	 * @param array $input_settings
14 14
 	 */
15
-	function __construct($input_settings = array()){
16
-		$input_settings['required'] = isset( $input_settings['required'] ) ? $input_settings['required'] : TRUE;
15
+	function __construct($input_settings = array()) {
16
+		$input_settings['required'] = isset($input_settings['required']) ? $input_settings['required'] : TRUE;
17 17
 		$this->_set_display_strategy(new EE_Text_Input_Display_Strategy());
18 18
 		$this->_set_normalization_strategy(new EE_Text_Normalization());
19
-		$this->_add_validation_strategy(new EE_Credit_Card_Validation_Strategy( isset( $input_settings[ 'validation_error_message' ] ) ? $input_settings[ 'validation_error_message' ] : NULL ));
19
+		$this->_add_validation_strategy(new EE_Credit_Card_Validation_Strategy(isset($input_settings['validation_error_message']) ? $input_settings['validation_error_message'] : NULL));
20 20
 		$this->set_sensitive_data_removal_strategy(new EE_Credit_Card_Sensitive_Data_Removal());
21 21
 		parent::__construct($input_settings);
22 22
 	}
Please login to merge, or discard this patch.
core/libraries/form_sections/inputs/EE_Credit_Card_Month_Input.input.php 2 patches
Spacing   +5 added lines, -5 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
 /**
@@ -24,14 +24,14 @@  discard block
 block discarded – undo
24 24
  *
25 25
  * ------------------------------------------------------------------------
26 26
  */
27
-class EE_Credit_Card_Month_Input extends EE_Month_Input{
27
+class EE_Credit_Card_Month_Input extends EE_Month_Input {
28 28
 
29 29
 	/**
30 30
 	 * @param bool  $leading_zero
31 31
 	 * @param array $input_settings
32 32
 	 */
33
-	function __construct( $leading_zero = false, $input_settings = array()){
34
-		$this->set_sensitive_data_removal_strategy( new EE_All_Sensitive_Data_Removal() );
35
-		parent::__construct($leading_zero,$input_settings);
33
+	function __construct($leading_zero = false, $input_settings = array()) {
34
+		$this->set_sensitive_data_removal_strategy(new EE_All_Sensitive_Data_Removal());
35
+		parent::__construct($leading_zero, $input_settings);
36 36
 	}
37 37
 }
38 38
\ No newline at end of file
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,7 +1,8 @@
 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
  * Event Espresso
7 8
  *
Please login to merge, or discard this patch.
core/libraries/form_sections/inputs/EE_Credit_Card_Year_Input.input.php 2 patches
Spacing   +5 added lines, -5 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
 /**
@@ -9,7 +9,7 @@  discard block
 block discarded – undo
9 9
  * @subpackage
10 10
  * @author				Mike Nelson
11 11
  */
12
-class EE_Credit_Card_Year_Input extends EE_Year_Input{
12
+class EE_Credit_Card_Year_Input extends EE_Year_Input {
13 13
 
14 14
 	/**
15 15
 	 * @param array $input_settings
@@ -17,8 +17,8 @@  discard block
 block discarded – undo
17 17
 	 * @param int   $years_behind
18 18
 	 * @param int   $years_ahead
19 19
 	 */
20
-	function __construct( $input_settings = array(), $four_digit_year = true, $years_behind = 0, $years_ahead = 15 ){
21
-		$this->set_sensitive_data_removal_strategy( new EE_All_Sensitive_Data_Removal() );
22
-		parent::__construct( $input_settings, $four_digit_year, $years_behind, $years_ahead );
20
+	function __construct($input_settings = array(), $four_digit_year = true, $years_behind = 0, $years_ahead = 15) {
21
+		$this->set_sensitive_data_removal_strategy(new EE_All_Sensitive_Data_Removal());
22
+		parent::__construct($input_settings, $four_digit_year, $years_behind, $years_ahead);
23 23
 	}
24 24
 }
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,7 +1,8 @@
 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
  * Event Espresso
7 8
  *
Please login to merge, or discard this patch.