Completed
Push — develop ( feab9e...47c1de )
by Zack
16:17
created
includes/admin/class.render.settings.php 1 patch
Spacing   +87 added lines, -87 removed lines patch added patch discarded remove patch
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
 
54 54
 		$is_table_layout = preg_match( '/table/ism', $template_id );
55 55
 
56
-		if( 'field' === $field_type ) {
56
+		if ( 'field' === $field_type ) {
57 57
 
58 58
 			// Default options - fields
59 59
 			$field_options = array(
@@ -105,11 +105,11 @@  discard block
 block discarded – undo
105 105
 			);
106 106
 
107 107
 			// Match Table as well as DataTables
108
-			if( $is_table_layout && 'directory' === $context ) {
109
-				$field_options['width'] = array(
108
+			if ( $is_table_layout && 'directory' === $context ) {
109
+				$field_options[ 'width' ] = array(
110 110
 					'type' => 'number',
111
-					'label' => __('Percent Width', 'gravityview'),
112
-					'desc' => __( 'Leave blank for column width to be based on the field content.', 'gravityview'),
111
+					'label' => __( 'Percent Width', 'gravityview' ),
112
+					'desc' => __( 'Leave blank for column width to be based on the field content.', 'gravityview' ),
113 113
 					'class' => 'code widefat',
114 114
 					'value' => '',
115 115
 					'priority' => 200,
@@ -121,7 +121,7 @@  discard block
 block discarded – undo
121 121
 
122 122
 		// Remove suffix ":" from the labels to standardize style. Using trim() instead of rtrim() for i18n.
123 123
 		foreach ( $field_options as $key => $field_option ) {
124
-			$field_options[ $key ]['label'] = trim( $field_option['label'], ':' );
124
+			$field_options[ $key ][ 'label' ] = trim( $field_option[ 'label' ], ':' );
125 125
 		}
126 126
 
127 127
 		/**
@@ -146,8 +146,8 @@  discard block
 block discarded – undo
146 146
 		 */
147 147
 		$field_options = apply_filters( "gravityview_template_{$input_type}_options", $field_options, $template_id, $field_id, $context, $input_type, $form_id );
148 148
 
149
-		if ( 'directory' === $context && isset( $field_options['show_as_link'] ) && ! isset( $field_options['new_window'] ) ) {
150
-			$field_options['new_window'] = array(
149
+		if ( 'directory' === $context && isset( $field_options[ 'show_as_link' ] ) && ! isset( $field_options[ 'new_window' ] ) ) {
150
+			$field_options[ 'new_window' ] = array(
151 151
 				'type'     => 'checkbox',
152 152
 				'label'    => __( 'Open link in a new tab or window?', 'gravityview' ),
153 153
 				'value'    => false,
@@ -168,7 +168,7 @@  discard block
 block discarded – undo
168 168
 				switch ( $key ) {
169 169
 					case 'show_as_link':
170 170
 						$_group = 'display';
171
-						$field_option['priority'] = 100;
171
+						$field_option[ 'priority' ] = 100;
172 172
 						break;
173 173
 					default:
174 174
 						$_group = \GV\Utils::get( $field_option, 'group', 'display' );
@@ -183,7 +183,7 @@  discard block
 block discarded – undo
183 183
 			}
184 184
 
185 185
 			$field_options = array();
186
-			foreach ( self::get_field_groups() as $group_key => $group_name  ) {
186
+			foreach ( self::get_field_groups() as $group_key => $group_name ) {
187 187
 				$field_options[ $group_key ] = \GV\Utils::get( $option_groups, $group_key, array() );
188 188
 			}
189 189
 
@@ -214,7 +214,7 @@  discard block
 block discarded – undo
214 214
 			return 0;
215 215
 		}
216 216
 
217
-		return ( $a_priority < $b_priority ) ? - 1 : 1;
217
+		return ( $a_priority < $b_priority ) ? -1 : 1;
218 218
 	}
219 219
 
220 220
 	/**
@@ -239,8 +239,8 @@  discard block
 block discarded – undo
239 239
 			'manage_options' => __( 'Administrator', 'gravityview' ),
240 240
 		);
241 241
 
242
-		if( is_multisite() ) {
243
-			$select_cap_choices['manage_network'] = __('Multisite Super Admin', 'gravityview' );
242
+		if ( is_multisite() ) {
243
+			$select_cap_choices[ 'manage_network' ] = __( 'Multisite Super Admin', 'gravityview' );
244 244
 		}
245 245
 
246 246
 		/**
@@ -253,7 +253,7 @@  discard block
 block discarded – undo
253 253
 		 * @param  string $context     Optional. What context are we in? Example: `single` or `directory`
254 254
 		 * @param  string $input_type  Optional. (textarea, list, select, etc.)
255 255
 		 */
256
-		$select_cap_choices = apply_filters('gravityview_field_visibility_caps', $select_cap_choices, $template_id, $field_id, $context, $input_type );
256
+		$select_cap_choices = apply_filters( 'gravityview_field_visibility_caps', $select_cap_choices, $template_id, $field_id, $context, $input_type );
257 257
 
258 258
 		return $select_cap_choices;
259 259
 	}
@@ -280,9 +280,9 @@  discard block
 block discarded – undo
280 280
 	 */
281 281
 	public static function render_field_options( $form_id, $field_type, $template_id, $field_id, $field_label, $area, $input_type = NULL, $uniqid = '', $current = '', $context = 'single', $item = array() ) {
282 282
 
283
-		if( empty( $uniqid ) ) {
283
+		if ( empty( $uniqid ) ) {
284 284
 			//generate a unique field id
285
-			$uniqid = uniqid('', false);
285
+			$uniqid = uniqid( '', false );
286 286
 		}
287 287
 
288 288
 		$grouped = ( 'field' === $field_type );
@@ -290,24 +290,24 @@  discard block
 block discarded – undo
290 290
 		// get field/widget options
291 291
 		$option_groups = self::get_default_field_options( $field_type, $template_id, $field_id, $context, $input_type, $form_id, $grouped );
292 292
 
293
-		if( ! $grouped ) {
293
+		if ( ! $grouped ) {
294 294
 			$option_groups = array( $option_groups );
295 295
 		}
296 296
 
297 297
 		$option_groups = array_filter( $option_groups );
298 298
 
299 299
 		// two different post arrays, depending of the field type
300
-		$name_prefix = $field_type .'s' .'['. $area .']['. $uniqid .']';
300
+		$name_prefix = $field_type . 's' . '[' . $area . '][' . $uniqid . ']';
301 301
 
302 302
 		// build output
303
-		$hidden_fields  = '<input type="hidden" class="field-key" name="'. $name_prefix .'[id]" value="'. esc_attr( $field_id ) .'">';
304
-		$hidden_fields .= '<input type="hidden" class="field-label" name="'. $name_prefix .'[label]" value="'. esc_attr( $field_label ) .'">';
303
+		$hidden_fields  = '<input type="hidden" class="field-key" name="' . $name_prefix . '[id]" value="' . esc_attr( $field_id ) . '">';
304
+		$hidden_fields .= '<input type="hidden" class="field-label" name="' . $name_prefix . '[label]" value="' . esc_attr( $field_label ) . '">';
305 305
 
306 306
 		$form_title = '';
307 307
 		if ( $form_id ) {
308
-			$hidden_fields .= '<input type="hidden" class="field-form-id" name="'. $name_prefix .'[form_id]" value="'. esc_attr( $form_id ) .'">';
308
+			$hidden_fields .= '<input type="hidden" class="field-form-id" name="' . $name_prefix . '[form_id]" value="' . esc_attr( $form_id ) . '">';
309 309
 			$form = GVCommon::get_form( $form_id );
310
-			$form_title = $form['title'];
310
+			$form_title = $form[ 'title' ];
311 311
 		}
312 312
 
313 313
 		// If there are no options, return what we got.
@@ -315,14 +315,14 @@  discard block
 block discarded – undo
315 315
 			return $hidden_fields . '<!-- No Options -->'; // The HTML comment is here for checking if the output is empty in render_label()
316 316
 		}
317 317
 
318
-		$settings_title = esc_attr( sprintf( __( '%s Settings', 'gravityview' ) , strip_tags( html_entity_decode( $field_label ) ) ) );
318
+		$settings_title = esc_attr( sprintf( __( '%s Settings', 'gravityview' ), strip_tags( html_entity_decode( $field_label ) ) ) );
319 319
 
320 320
 		$field_details = '';
321 321
 
322 322
 		// Get the pretty name for the input type
323 323
 		$gv_field = GravityView_Fields::get( $input_type );
324 324
 
325
-		if( $gv_field ) {
325
+		if ( $gv_field ) {
326 326
 			$input_type_label = $gv_field->label;
327 327
 		} else {
328 328
 			$input_type_label = $input_type;
@@ -336,7 +336,7 @@  discard block
 block discarded – undo
336 336
 			}
337 337
 
338 338
 			if ( $grouped ) {
339
-				$group_name     = rgar( self::get_field_groups(), $group_key, '' );
339
+				$group_name = rgar( self::get_field_groups(), $group_key, '' );
340 340
 				$field_settings .= '<fieldset class="item-settings-group item-settings-group-' . esc_attr( $group_key ) . '">';
341 341
 				$field_settings .= '<legend>' . esc_attr( $group_name ) . '</legend>';
342 342
 			}
@@ -353,15 +353,15 @@  discard block
 block discarded – undo
353 353
 				}
354 354
 
355 355
 				$show_if = '';
356
-				if ( ! empty( $option['requires'] ) ) {
357
-					$show_if .= sprintf( ' data-requires="%s"', $option['requires'] );
356
+				if ( ! empty( $option[ 'requires' ] ) ) {
357
+					$show_if .= sprintf( ' data-requires="%s"', $option[ 'requires' ] );
358 358
 				}
359 359
 
360
-				if ( ! empty( $option['requires_not'] ) ) {
361
-					$show_if .= sprintf( ' data-requires-not="%s"', $option['requires_not'] );
360
+				if ( ! empty( $option[ 'requires_not' ] ) ) {
361
+					$show_if .= sprintf( ' data-requires-not="%s"', $option[ 'requires_not' ] );
362 362
 				}
363 363
 
364
-				switch ( $option['type'] ) {
364
+				switch ( $option[ 'type' ] ) {
365 365
 					// Hide hidden fields
366 366
 					case 'hidden':
367 367
 						$field_settings .= '<div class="gv-setting-container gv-setting-container-' . esc_attr( $key ) . ' screen-reader-text">' . $field_output . '</div>';
@@ -379,30 +379,30 @@  discard block
 block discarded – undo
379 379
 		$item_details = '';
380 380
 		$subtitle = '';
381 381
 
382
-		if( 'field' === $field_type ) {
383
-			$subtitle = ! empty( $item['subtitle'] ) ? '<div class="subtitle">' . $item['subtitle'] . '</div>' : '';
382
+		if ( 'field' === $field_type ) {
383
+			$subtitle = ! empty( $item[ 'subtitle' ] ) ? '<div class="subtitle">' . $item[ 'subtitle' ] . '</div>' : '';
384 384
 
385 385
 			$item_details .= '
386 386
 			<div class="gv-field-details--container">
387
-				<label class="gv-field-details--toggle">' . esc_html__( 'Field Details', 'gravityview' ) .' <i class="dashicons dashicons-arrow-right"></i></label>
387
+				<label class="gv-field-details--toggle">' . esc_html__( 'Field Details', 'gravityview' ) . ' <i class="dashicons dashicons-arrow-right"></i></label>
388 388
 				<section class="gv-field-details gv-field-details--closed">';
389 389
 
390 390
 				if ( $field_id && is_numeric( $field_id ) ) {
391 391
 				$item_details .= '
392 392
 					<div class="gv-field-detail gv-field-detail--field">
393
-						<span class="gv-field-detail--label">' . esc_html__( 'Field ID', 'gravityview' ) .'</span><span class="gv-field-detail--value">#{{field_id}}</span>
393
+						<span class="gv-field-detail--label">' . esc_html__( 'Field ID', 'gravityview' ) . '</span><span class="gv-field-detail--value">#{{field_id}}</span>
394 394
 					</div>';
395 395
 			    }
396 396
 
397 397
 				$item_details .= '
398 398
 					<div class="gv-field-detail gv-field-detail--type">
399
-						<span class="gv-field-detail--label">' . esc_html_x( 'Type', 'The type of field being configured (eg: "Single Line Text")', 'gravityview' ) .'</span><span class="gv-field-detail--value">{{input_type_label}}</span>
399
+						<span class="gv-field-detail--label">' . esc_html_x( 'Type', 'The type of field being configured (eg: "Single Line Text")', 'gravityview' ) . '</span><span class="gv-field-detail--value">{{input_type_label}}</span>
400 400
 					</div>';
401 401
 
402
-				if( $form_id ) {
402
+				if ( $form_id ) {
403 403
 					$item_details .= '
404 404
 					<div class="gv-field-detail gv-field-detail--form">
405
-						<span class="gv-field-detail--label">' . esc_html__( 'Form', 'gravityview' ) .'</span><span class="gv-field-detail--value">{{form_title}} (#{{form_id}})</span>
405
+						<span class="gv-field-detail--label">' . esc_html__( 'Form', 'gravityview' ) . '</span><span class="gv-field-detail--value">{{form_title}} (#{{form_id}})</span>
406 406
 					</div>';
407 407
 				}
408 408
 				$item_details .= '
@@ -410,8 +410,8 @@  discard block
 block discarded – undo
410 410
 			</div>';
411 411
 		} else {
412 412
 			$widget_details_content = rgar( $item, 'description', '' );
413
-			if ( ! empty( $item['subtitle'] ) ) {
414
-				$widget_details_content .= ( '' !== $widget_details_content ) ? "\n\n" . $item['subtitle'] : $item['subtitle'];
413
+			if ( ! empty( $item[ 'subtitle' ] ) ) {
414
+				$widget_details_content .= ( '' !== $widget_details_content ) ? "\n\n" . $item[ 'subtitle' ] : $item[ 'subtitle' ];
415 415
 			}
416 416
 
417 417
 			// Intentionally not escaping to allow HTML.
@@ -467,17 +467,17 @@  discard block
 block discarded – undo
467 467
 		 * @deprecated setting index 'default' was replaced by 'value'
468 468
 		 * @see GravityView_FieldType::get_field_defaults
469 469
 		 */
470
-		if( !empty( $option['default'] ) && empty( $option['value'] ) ) {
471
-			$option['value'] = $option['default'];
472
-			_deprecated_function( 'GravityView_FieldType::get_field_defaults', '1.1.7', '[value] instead of [default] when defining the setting '. $name .' details' );
470
+		if ( ! empty( $option[ 'default' ] ) && empty( $option[ 'value' ] ) ) {
471
+			$option[ 'value' ] = $option[ 'default' ];
472
+			_deprecated_function( 'GravityView_FieldType::get_field_defaults', '1.1.7', '[value] instead of [default] when defining the setting ' . $name . ' details' );
473 473
 		}
474 474
 
475 475
 		// prepare to render option field type
476
-		if( isset( $option['type'] ) ) {
476
+		if ( isset( $option[ 'type' ] ) ) {
477 477
 
478 478
 			$type_class = self::load_type_class( $option );
479 479
 
480
-			if( class_exists( $type_class ) ) {
480
+			if ( class_exists( $type_class ) ) {
481 481
 
482 482
 				/** @type GravityView_FieldType $render_type */
483 483
 				$render_type = new $type_class( $name, $option, $curr_value );
@@ -494,7 +494,7 @@  discard block
 block discarded – undo
494 494
 				 * @param[in,out] string $output field class name
495 495
 				 * @param[in] array $option  option field data
496 496
 				 */
497
-				$output = apply_filters( "gravityview/option/output/{$option['type']}" , $output, $option );
497
+				$output = apply_filters( "gravityview/option/output/{$option[ 'type' ]}", $output, $option );
498 498
 			}
499 499
 
500 500
 		} // isset option[type]
@@ -529,27 +529,27 @@  discard block
 block discarded – undo
529 529
 		 * @deprecated setting index 'name' was replaced by 'label'
530 530
 		 * @see GravityView_FieldType::get_field_defaults
531 531
 		 */
532
-		if( isset( $setting['name'] ) && empty( $setting['label'] ) ) {
533
-			$setting['label'] = $setting['name'];
534
-			_deprecated_function( 'GravityView_FieldType::get_field_defaults', '1.1.7', '[label] instead of [name] when defining the setting '. $key .' details' );
532
+		if ( isset( $setting[ 'name' ] ) && empty( $setting[ 'label' ] ) ) {
533
+			$setting[ 'label' ] = $setting[ 'name' ];
534
+			_deprecated_function( 'GravityView_FieldType::get_field_defaults', '1.1.7', '[label] instead of [name] when defining the setting ' . $key . ' details' );
535 535
 		}
536 536
 
537 537
 		$name = esc_attr( sprintf( $name, $key ) );
538
-		$setting['id'] = esc_attr( sprintf( $id, $key ) );
539
-		$setting['tooltip'] = 'gv_' . $key;
538
+		$setting[ 'id' ] = esc_attr( sprintf( $id, $key ) );
539
+		$setting[ 'tooltip' ] = 'gv_' . $key;
540 540
 
541 541
 		// Use default if current setting isn't set.
542
-		$curr_value = isset( $current_settings[ $key ] ) ? $current_settings[ $key ] : $setting['value'];
542
+		$curr_value = isset( $current_settings[ $key ] ) ? $current_settings[ $key ] : $setting[ 'value' ];
543 543
 
544 544
 		// default setting type = text
545
-		$setting['type'] = empty( $setting['type'] ) ? 'text' : $setting['type'];
545
+		$setting[ 'type' ] = empty( $setting[ 'type' ] ) ? 'text' : $setting[ 'type' ];
546 546
 
547 547
 		// merge tags
548
-		if( !isset( $setting['merge_tags'] ) ) {
549
-			if( $setting['type'] === 'text' ) {
550
-				$setting['merge_tags'] = true;
548
+		if ( ! isset( $setting[ 'merge_tags' ] ) ) {
549
+			if ( $setting[ 'type' ] === 'text' ) {
550
+				$setting[ 'merge_tags' ] = true;
551 551
 			} else {
552
-				$setting['merge_tags'] = false;
552
+				$setting[ 'merge_tags' ] = false;
553 553
 			}
554 554
 		}
555 555
 
@@ -557,7 +557,7 @@  discard block
 block discarded – undo
557 557
 
558 558
 		// render the setting
559 559
 		$type_class = self::load_type_class( $setting );
560
-		if( class_exists( $type_class ) ) {
560
+		if ( class_exists( $type_class ) ) {
561 561
 			/** @type GravityView_FieldType $render_type */
562 562
 			$render_type = new $type_class( $name, $setting, $curr_value );
563 563
 			ob_start();
@@ -566,25 +566,25 @@  discard block
 block discarded – undo
566 566
 		}
567 567
 
568 568
 		// Check if setting is specific for a template
569
-		if( !empty( $setting['show_in_template'] ) ) {
570
-			if( !is_array( $setting['show_in_template'] ) ) {
571
-				$setting['show_in_template'] = array( $setting['show_in_template'] );
569
+		if ( ! empty( $setting[ 'show_in_template' ] ) ) {
570
+			if ( ! is_array( $setting[ 'show_in_template' ] ) ) {
571
+				$setting[ 'show_in_template' ] = array( $setting[ 'show_in_template' ] );
572 572
 			}
573
-			$show_if = ' data-show-if="'. implode( ' ', $setting['show_in_template'] ).'"';
573
+			$show_if = ' data-show-if="' . implode( ' ', $setting[ 'show_in_template' ] ) . '"';
574 574
 		} else {
575 575
 			$show_if = '';
576 576
 		}
577 577
 
578
-		if( ! empty( $setting['requires'] ) ) {
579
-			$show_if .= sprintf( ' data-requires="%s"', $setting['requires'] );
578
+		if ( ! empty( $setting[ 'requires' ] ) ) {
579
+			$show_if .= sprintf( ' data-requires="%s"', $setting[ 'requires' ] );
580 580
 		}
581 581
 
582
-		if( ! empty( $setting['requires_not'] ) ) {
583
-			$show_if .= sprintf( ' data-requires-not="%s"', $setting['requires_not'] );
582
+		if ( ! empty( $setting[ 'requires_not' ] ) ) {
583
+			$show_if .= sprintf( ' data-requires-not="%s"', $setting[ 'requires_not' ] );
584 584
 		}
585 585
 
586 586
 		// output
587
-		echo '<tr style="vertical-align: top;" '. $show_if .'>' . $output . '</tr>';
587
+		echo '<tr style="vertical-align: top;" ' . $show_if . '>' . $output . '</tr>';
588 588
 
589 589
 	}
590 590
 
@@ -596,7 +596,7 @@  discard block
 block discarded – undo
596 596
 	 */
597 597
 	public static function load_type_class( $field = NULL ) {
598 598
 
599
-		if( empty( $field['type'] ) ) {
599
+		if ( empty( $field[ 'type' ] ) ) {
600 600
 			return NULL;
601 601
 		}
602 602
 
@@ -605,9 +605,9 @@  discard block
 block discarded – undo
605 605
 		 * @param string $class_suffix  field class suffix; `GravityView_FieldType_{$class_suffix}`
606 606
 		 * @param array $field   field data
607 607
 		 */
608
-		$type_class = apply_filters( "gravityview/setting/class/{$field['type']}", 'GravityView_FieldType_' . $field['type'], $field );
608
+		$type_class = apply_filters( "gravityview/setting/class/{$field[ 'type' ]}", 'GravityView_FieldType_' . $field[ 'type' ], $field );
609 609
 
610
-		if( class_exists( $type_class ) ) {
610
+		if ( class_exists( $type_class ) ) {
611 611
 			return $type_class;
612 612
 		}
613 613
 
@@ -616,9 +616,9 @@  discard block
 block discarded – undo
616 616
 		 * @param string  $field_type_include_path field class file path
617 617
 		 * @param array $field  field data
618 618
 		 */
619
-		$class_file = apply_filters( "gravityview/setting/class_file/{$field['type']}", GRAVITYVIEW_DIR . "includes/admin/field-types/type_{$field['type']}.php", $field );
619
+		$class_file = apply_filters( "gravityview/setting/class_file/{$field[ 'type' ]}", GRAVITYVIEW_DIR . "includes/admin/field-types/type_{$field[ 'type' ]}.php", $field );
620 620
 
621
-		if( $class_file && file_exists( $class_file ) ) {
621
+		if ( $class_file && file_exists( $class_file ) ) {
622 622
 			require_once( $class_file );
623 623
 		}
624 624
 
@@ -640,8 +640,8 @@  discard block
 block discarded – undo
640 640
 
641 641
 		_deprecated_function( __METHOD__, '1.2', 'GravityView_FieldType_checkbox::render_input' );
642 642
 
643
-		$output  = '<input name="'. esc_attr( $name ) .'" type="hidden" value="0">';
644
-		$output .= '<input name="'. esc_attr( $name ) .'" id="'. esc_attr( $id ) .'" type="checkbox" value="1" '. checked( $current, '1', false ) .' >';
643
+		$output  = '<input name="' . esc_attr( $name ) . '" type="hidden" value="0">';
644
+		$output .= '<input name="' . esc_attr( $name ) . '" id="' . esc_attr( $id ) . '" type="checkbox" value="1" ' . checked( $current, '1', false ) . ' >';
645 645
 
646 646
 		return $output;
647 647
 	}
@@ -661,22 +661,22 @@  discard block
 block discarded – undo
661 661
 		_deprecated_function( __METHOD__, '1.2', 'GravityView_FieldType_text::render_input' );
662 662
 
663 663
 		// Show the merge tags if the field is a list view
664
-		$is_list = ( preg_match( '/_list-/ism', $name ));
664
+		$is_list = ( preg_match( '/_list-/ism', $name ) );
665 665
 
666 666
 		// Or is a single entry view
667
-		$is_single = ( preg_match( '/single_/ism', $name ));
667
+		$is_single = ( preg_match( '/single_/ism', $name ) );
668 668
 		$show = ( $is_single || $is_list );
669 669
 
670 670
 		$class = '';
671 671
 		// and $add_merge_tags is not false
672
-		if( $show && $add_merge_tags !== false || $add_merge_tags === 'force' ) {
672
+		if ( $show && $add_merge_tags !== false || $add_merge_tags === 'force' ) {
673 673
 			$class = 'merge-tag-support mt-position-right mt-hide_all_fields ';
674 674
 		}
675 675
 
676
-		$class .= !empty( $args['class'] ) ? $args['class'] : 'widefat';
677
-		$type = !empty( $args['type'] ) ? $args['type'] : 'text';
676
+		$class .= ! empty( $args[ 'class' ] ) ? $args[ 'class' ] : 'widefat';
677
+		$type = ! empty( $args[ 'type' ] ) ? $args[ 'type' ] : 'text';
678 678
 
679
-		return '<input name="'. esc_attr( $name ) .'" id="'. esc_attr( $id ) .'" type="'.esc_attr($type).'" value="'. esc_attr( $current ) .'" class="'.esc_attr( $class ).'">';
679
+		return '<input name="' . esc_attr( $name ) . '" id="' . esc_attr( $id ) . '" type="' . esc_attr( $type ) . '" value="' . esc_attr( $current ) . '" class="' . esc_attr( $class ) . '">';
680 680
 	}
681 681
 
682 682
 	/**
@@ -693,21 +693,21 @@  discard block
 block discarded – undo
693 693
 		_deprecated_function( __METHOD__, '1.2', 'GravityView_FieldType_textarea::render_input' );
694 694
 
695 695
 		// Show the merge tags if the field is a list view
696
-		$is_list = ( preg_match( '/_list-/ism', $name ));
696
+		$is_list = ( preg_match( '/_list-/ism', $name ) );
697 697
 
698 698
 		// Or is a single entry view
699
-		$is_single = ( preg_match( '/single_/ism', $name ));
699
+		$is_single = ( preg_match( '/single_/ism', $name ) );
700 700
 		$show = ( $is_single || $is_list );
701 701
 
702 702
 		$class = '';
703 703
 		// and $add_merge_tags is not false
704
-		if( $show && $add_merge_tags !== false || $add_merge_tags === 'force' ) {
704
+		if ( $show && $add_merge_tags !== false || $add_merge_tags === 'force' ) {
705 705
 			$class = 'merge-tag-support mt-position-right mt-hide_all_fields ';
706 706
 		}
707 707
 
708
-		$class .= !empty( $args['class'] ) ? 'widefat '.$args['class'] : 'widefat';
708
+		$class .= ! empty( $args[ 'class' ] ) ? 'widefat ' . $args[ 'class' ] : 'widefat';
709 709
 
710
-		return '<textarea name="'. esc_attr( $name ) .'" id="'. esc_attr( $id ) .'" class="'.esc_attr( $class ).'">'. esc_textarea( $current ) .'</textarea>';
710
+		return '<textarea name="' . esc_attr( $name ) . '" id="' . esc_attr( $id ) . '" class="' . esc_attr( $class ) . '">' . esc_textarea( $current ) . '</textarea>';
711 711
 	}
712 712
 
713 713
 	/**
@@ -723,9 +723,9 @@  discard block
 block discarded – undo
723 723
 
724 724
 		_deprecated_function( __METHOD__, '1.2', 'GravityView_FieldType_select::render_input' );
725 725
 
726
-		$output = '<select name="'. $name .'" id="'. $id .'">';
727
-		foreach( $choices as $value => $label ) {
728
-			$output .= '<option value="'. esc_attr( $value ) .'" '. selected( $value, $current, false ) .'>'. esc_html( $label ) .'</option>';
726
+		$output = '<select name="' . $name . '" id="' . $id . '">';
727
+		foreach ( $choices as $value => $label ) {
728
+			$output .= '<option value="' . esc_attr( $value ) . '" ' . selected( $value, $current, false ) . '>' . esc_html( $label ) . '</option>';
729 729
 		}
730 730
 		$output .= '</select>';
731 731
 
Please login to merge, or discard this patch.
includes/extensions/entry-notes/class-gravityview-field-notes.php 1 patch
Spacing   +76 added lines, -76 removed lines patch added patch discarded remove patch
@@ -65,19 +65,19 @@  discard block
 block discarded – undo
65 65
 
66 66
 		add_shortcode( 'gv_note_add', array( 'GravityView_Field_Notes', 'get_add_note_part' ) );
67 67
 
68
-		add_action( 'wp', array( $this, 'maybe_delete_notes'), 1000 );
69
-		add_action( 'wp_ajax_nopriv_gv_delete_notes', array( $this, 'maybe_delete_notes') );
70
-		add_action( 'wp_ajax_gv_delete_notes', array( $this, 'maybe_delete_notes') );
68
+		add_action( 'wp', array( $this, 'maybe_delete_notes' ), 1000 );
69
+		add_action( 'wp_ajax_nopriv_gv_delete_notes', array( $this, 'maybe_delete_notes' ) );
70
+		add_action( 'wp_ajax_gv_delete_notes', array( $this, 'maybe_delete_notes' ) );
71 71
 
72
-		add_action( 'wp', array( $this, 'maybe_add_note'), 1000 );
73
-		add_action( 'wp_ajax_nopriv_gv_note_add', array( $this, 'maybe_add_note') );
74
-		add_action( 'wp_ajax_gv_note_add', array( $this, 'maybe_add_note') );
72
+		add_action( 'wp', array( $this, 'maybe_add_note' ), 1000 );
73
+		add_action( 'wp_ajax_nopriv_gv_note_add', array( $this, 'maybe_add_note' ) );
74
+		add_action( 'wp_ajax_gv_note_add', array( $this, 'maybe_add_note' ) );
75 75
 
76 76
 		// add template path to check for field
77 77
 		add_filter( 'gravityview_template_paths', array( $this, 'add_template_path' ) );
78 78
 		add_filter( 'gravityview/template/fields_template_paths', array( $this, 'add_template_path' ) );
79 79
 
80
-		add_action( 'wp_enqueue_scripts', array( $this, 'register_scripts') );
80
+		add_action( 'wp_enqueue_scripts', array( $this, 'register_scripts' ) );
81 81
 		add_action( 'gravityview/field/notes/scripts', array( $this, 'enqueue_scripts' ) );
82 82
 
83 83
 		add_filter( 'gravityview_entry_default_fields', array( $this, 'add_entry_default_field' ), 10, 3 );
@@ -99,8 +99,8 @@  discard block
 block discarded – undo
99 99
 	 */
100 100
 	public function add_entry_default_field( $entry_default_fields, $form, $zone ) {
101 101
 
102
-		if( in_array( $zone, array( 'directory', 'single' ) ) ) {
103
-			$entry_default_fields['notes'] = array(
102
+		if ( in_array( $zone, array( 'directory', 'single' ) ) ) {
103
+			$entry_default_fields[ 'notes' ] = array(
104 104
 				'label' => __( 'Entry Notes', 'gravityview' ),
105 105
 				'type'  => 'notes',
106 106
 				'desc'  => __( 'Display, add, and delete notes for an entry.', 'gravityview' ),
@@ -134,23 +134,23 @@  discard block
 block discarded – undo
134 134
 	public function enqueue_scripts() {
135 135
 		global $wp_actions;
136 136
 
137
-		if( ! wp_script_is( 'gravityview-notes', 'enqueued' ) ) {
137
+		if ( ! wp_script_is( 'gravityview-notes', 'enqueued' ) ) {
138 138
 			wp_enqueue_style( 'gravityview-notes' );
139 139
 			wp_enqueue_script( 'gravityview-notes' );
140 140
 		}
141 141
 
142
-		if( ! wp_script_is( 'gravityview-notes', 'done' ) ) {
142
+		if ( ! wp_script_is( 'gravityview-notes', 'done' ) ) {
143 143
 
144 144
 			$strings = self::strings();
145 145
 
146 146
 			wp_localize_script( 'gravityview-notes', 'GVNotes', array(
147 147
 				'ajaxurl' => admin_url( 'admin-ajax.php' ),
148 148
 				'text' => array(
149
-					'processing' => $strings['processing'],
150
-					'delete_confirm' => $strings['delete-confirm'],
151
-					'note_added' => $strings['added-note'],
152
-					'error_invalid' => $strings['error-invalid'],
153
-					'error_empty_note' => $strings['error-empty-note'],
149
+					'processing' => $strings[ 'processing' ],
150
+					'delete_confirm' => $strings[ 'delete-confirm' ],
151
+					'note_added' => $strings[ 'added-note' ],
152
+					'error_invalid' => $strings[ 'error-invalid' ],
153
+					'error_empty_note' => $strings[ 'error-empty-note' ],
154 154
 				),
155 155
 			) );
156 156
 		}
@@ -166,7 +166,7 @@  discard block
 block discarded – undo
166 166
 	 * @return void
167 167
 	 */
168 168
 	function maybe_add_note() {
169
-		if ( ! isset( $_POST['action'] ) || 'gv_note_add' !== $_POST['action'] ) {
169
+		if ( ! isset( $_POST[ 'action' ] ) || 'gv_note_add' !== $_POST[ 'action' ] ) {
170 170
 			return;
171 171
 		}
172 172
 
@@ -179,12 +179,12 @@  discard block
 block discarded – undo
179 179
 		$post = wp_unslash( $_POST );
180 180
 
181 181
 		if ( $this->doing_ajax ) {
182
-			parse_str( $post['data'], $data );
182
+			parse_str( $post[ 'data' ], $data );
183 183
 		} else {
184 184
 			$data = $post;
185 185
 		}
186 186
 
187
-		$this->process_add_note( (array) $data );
187
+		$this->process_add_note( (array)$data );
188 188
 	}
189 189
 
190 190
 	/**
@@ -210,23 +210,23 @@  discard block
 block discarded – undo
210 210
 		$error = false;
211 211
 		$success = false;
212 212
 
213
-		if( empty( $data['entry-slug'] ) ) {
213
+		if ( empty( $data[ 'entry-slug' ] ) ) {
214 214
 
215
-			$error = self::strings('error-invalid');
215
+			$error = self::strings( 'error-invalid' );
216 216
 			gravityview()->log->error( 'The note is missing an Entry ID.' );
217 217
 
218 218
 		} else {
219 219
 
220
-			$valid = wp_verify_nonce( $data['gv_note_add'], 'gv_note_add_' . $data['entry-slug'] );
220
+			$valid = wp_verify_nonce( $data[ 'gv_note_add' ], 'gv_note_add_' . $data[ 'entry-slug' ] );
221 221
 
222 222
 			$has_cap = GVCommon::has_cap( 'gravityview_add_entry_notes' );
223 223
 
224
-			if( ! $has_cap ) {
224
+			if ( ! $has_cap ) {
225 225
 				$error = self::strings( 'error-cap-add' );
226 226
 				gravityview()->log->error( 'Adding a note failed: the user does not have the "gravityview_add_entry_notes" capability.' );
227 227
 			} elseif ( $valid ) {
228 228
 
229
-				$entry = gravityview_get_entry( $data['entry-slug'], true, false );
229
+				$entry = gravityview_get_entry( $data[ 'entry-slug' ], true, false );
230 230
 
231 231
 				$added = $this->add_note( $entry, $data );
232 232
 
@@ -244,22 +244,22 @@  discard block
 block discarded – undo
244 244
 					$this->maybe_send_entry_notes( $note, $entry, $data );
245 245
 
246 246
 					if ( $note ) {
247
-						$success = self::display_note( $note, ! empty( $data['show-delete'] ) );
247
+						$success = self::display_note( $note, ! empty( $data[ 'show-delete' ] ) );
248 248
 						gravityview()->log->debug( 'The note was successfully created', array( 'data' => compact( 'note', 'data' ) ) );
249 249
 					} else {
250
-						$error = self::strings('error-add-note');
250
+						$error = self::strings( 'error-add-note' );
251 251
 						gravityview()->log->error( 'The note was not successfully created', array( 'data' => compact( 'note', 'data' ) ) );
252 252
 					}
253 253
 				}
254 254
 			} else {
255
-				$error = self::strings('error-invalid');
255
+				$error = self::strings( 'error-invalid' );
256 256
 				gravityview()->log->error( 'Nonce validation failed; the note was not created' );
257 257
 			}
258 258
 		}
259 259
 
260 260
 
261
-		if( $this->doing_ajax ) {
262
-			if( $success ) {
261
+		if ( $this->doing_ajax ) {
262
+			if ( $success ) {
263 263
 				wp_send_json_success( array( 'html' => $success ) );
264 264
 			} else {
265 265
 				$error = $error ? $error : self::strings( 'error-invalid' );
@@ -285,11 +285,11 @@  discard block
 block discarded – undo
285 285
 			return;
286 286
 		}
287 287
 
288
-		if ( isset( $_POST['action'] ) && 'gv_delete_notes' === $_POST['action'] ) {
288
+		if ( isset( $_POST[ 'action' ] ) && 'gv_delete_notes' === $_POST[ 'action' ] ) {
289 289
 
290 290
 			$post = wp_unslash( $_POST );
291 291
 			if ( $this->doing_ajax ) {
292
-				parse_str( $post['data'], $data );
292
+				parse_str( $post[ 'data' ], $data );
293 293
 			} else {
294 294
 				$data = $post;
295 295
 			}
@@ -320,18 +320,18 @@  discard block
 block discarded – undo
320 320
 	 */
321 321
 	function process_delete_notes( $data ) {
322 322
 
323
-		$valid = wp_verify_nonce( $data['gv_delete_notes'], 'gv_delete_notes_' . $data['entry-slug'] );
323
+		$valid = wp_verify_nonce( $data[ 'gv_delete_notes' ], 'gv_delete_notes_' . $data[ 'entry-slug' ] );
324 324
 		$has_cap = GVCommon::has_cap( 'gravityview_delete_entry_notes' );
325 325
 		$success = false;
326 326
 
327 327
 		if ( $valid && $has_cap ) {
328
-			GravityView_Entry_Notes::delete_notes( $data['note'] );
328
+			GravityView_Entry_Notes::delete_notes( $data[ 'note' ] );
329 329
 			$success = true;
330 330
 		}
331 331
 
332
-		if( $this->doing_ajax ) {
332
+		if ( $this->doing_ajax ) {
333 333
 
334
-			if( $success ) {
334
+			if ( $success ) {
335 335
 				wp_send_json_success();
336 336
 			} else {
337 337
 				if ( ! $valid ) {
@@ -364,13 +364,13 @@  discard block
 block discarded – undo
364 364
 
365 365
 	public function field_options( $field_options, $template_id, $field_id, $context, $input_type, $form_id ) {
366 366
 
367
-		unset( $field_options['show_as_link'] );
367
+		unset( $field_options[ 'show_as_link' ] );
368 368
 
369 369
 		$notes_options = array(
370 370
 			'notes' => array(
371 371
 				'type' => 'checkboxes',
372
-				'label' => __('Note Settings', 'gravityview'),
373
-				'desc' => sprintf( _x('Only users with specific capabilities will be able to view, add and delete notes. %sRead more%s.', '%s is opening and closing HTML link', 'gravityview' ), '<a href="https://docs.gravityview.co/article/311-gravityview-capabilities">', '</a>' ),
372
+				'label' => __( 'Note Settings', 'gravityview' ),
373
+				'desc' => sprintf( _x( 'Only users with specific capabilities will be able to view, add and delete notes. %sRead more%s.', '%s is opening and closing HTML link', 'gravityview' ), '<a href="https://docs.gravityview.co/article/311-gravityview-capabilities">', '</a>' ),
374 374
 				'options' => array(
375 375
 					'view' => array(
376 376
 						'label' => __( 'Display notes?', 'gravityview' ),
@@ -422,7 +422,7 @@  discard block
 block discarded – undo
422 422
 			'processing' => __( 'Processing&hellip;', 'gravityview' ),
423 423
 			'other-email' => __( 'Other email address', 'gravityview' ),
424 424
 			'email-label' => __( 'Email address', 'gravityview' ),
425
-			'email-placeholder' => _x('[email protected]', 'Example email address used as a placeholder', 'gravityview'),
425
+			'email-placeholder' => _x( '[email protected]', 'Example email address used as a placeholder', 'gravityview' ),
426 426
 			'subject-label' => __( 'Subject', 'gravityview' ),
427 427
 			'subject' => __( 'Email subject', 'gravityview' ),
428 428
 			'default-email-subject' => __( 'New entry note', 'gravityview' ),
@@ -442,7 +442,7 @@  discard block
 block discarded – undo
442 442
 		 */
443 443
 		$strings = gv_map_deep( apply_filters( 'gravityview/field/notes/strings', $strings ), 'esc_html' );
444 444
 
445
-		if( $key ) {
445
+		if ( $key ) {
446 446
 			return isset( $strings[ $key ] ) ? $strings[ $key ] : '';
447 447
 		}
448 448
 
@@ -464,7 +464,7 @@  discard block
 block discarded – undo
464 464
 	 */
465 465
 	static public function display_note( $note, $show_delete = false, $context = null ) {
466 466
 
467
-		if( ! is_object( $note ) ) {
467
+		if ( ! is_object( $note ) ) {
468 468
 			return '';
469 469
 		}
470 470
 
@@ -519,7 +519,7 @@  discard block
 block discarded – undo
519 519
 		}
520 520
 
521 521
 		$replacements = array(
522
-			'{note_id}' => $note_content['note_id'],
522
+			'{note_id}' => $note_content[ 'note_id' ],
523 523
 			'{row_class}' => 'gv-note',
524 524
 			'{note_detail}' => $note_detail_html
525 525
 		);
@@ -551,13 +551,13 @@  discard block
 block discarded – undo
551 551
 
552 552
 		$user_data = get_userdata( $current_user->ID );
553 553
 
554
-		$note_content = trim( $data['gv-note-content'] );
554
+		$note_content = trim( $data[ 'gv-note-content' ] );
555 555
 
556
-		if( empty( $note_content ) ) {
556
+		if ( empty( $note_content ) ) {
557 557
 			return new WP_Error( 'gv-add-note-empty', __( 'The note is empty.', 'gravityview' ) );
558 558
 		}
559 559
 
560
-		$return = GravityView_Entry_Notes::add_note( $entry['id'], $user_data->ID, $user_data->display_name, $note_content, 'gravityview/field/notes' );
560
+		$return = GravityView_Entry_Notes::add_note( $entry[ 'id' ], $user_data->ID, $user_data->display_name, $note_content, 'gravityview/field/notes' );
561 561
 
562 562
 		return $return;
563 563
 	}
@@ -577,7 +577,7 @@  discard block
 block discarded – undo
577 577
 
578 578
 		$atts = shortcode_atts( array( 'entry' => null ), $atts );
579 579
 
580
-		if( ! GVCommon::has_cap( 'gravityview_add_entry_notes' ) ) {
580
+		if ( ! GVCommon::has_cap( 'gravityview_add_entry_notes' ) ) {
581 581
 			gravityview()->log->error( 'User does not have permission to add entry notes ("gravityview_add_entry_notes").' );
582 582
 			return '';
583 583
 		}
@@ -599,8 +599,8 @@  discard block
 block discarded – undo
599 599
 
600 600
 			$visibility_settings = $gravityview_view->getCurrentFieldSetting( 'notes' );
601 601
 
602
-			if ( $atts['entry'] ) {
603
-				$entry = GFAPI::get_entry( $atts['entry'] );
602
+			if ( $atts[ 'entry' ] ) {
603
+				$entry = GFAPI::get_entry( $atts[ 'entry' ] );
604 604
 			}
605 605
 
606 606
 			if ( ! isset( $entry ) || ! $entry ) {
@@ -618,14 +618,14 @@  discard block
 block discarded – undo
618 618
 
619 619
 		// Only generate the dropdown if the field settings allow it
620 620
 		$email_fields = '';
621
-		if( ! empty( $visibility_settings['email'] ) ) {
621
+		if ( ! empty( $visibility_settings[ 'email' ] ) ) {
622 622
 			$email_fields = self::get_note_email_fields( $entry_slug );
623 623
 		}
624 624
 
625 625
 		$add_note_html = str_replace( '{entry_slug}', $entry_slug, $add_note_html );
626 626
 		$add_note_html = str_replace( '{nonce_field}', $nonce_field, $add_note_html );
627
-		$add_note_html = str_replace( '{show_delete}', intval( empty( $visibility_settings['delete'] ) ? 0 : $visibility_settings['delete'] ), $add_note_html );
628
-		$add_note_html   = str_replace( '{email_fields}', $email_fields, $add_note_html );
627
+		$add_note_html = str_replace( '{show_delete}', intval( empty( $visibility_settings[ 'delete' ] ) ? 0 : $visibility_settings[ 'delete' ] ), $add_note_html );
628
+		$add_note_html = str_replace( '{email_fields}', $email_fields, $add_note_html );
629 629
 		$add_note_html = str_replace( '{url}', esc_url_raw( add_query_arg( array() ) ), $add_note_html );
630 630
 
631 631
 		return $add_note_html;
@@ -650,8 +650,8 @@  discard block
 block discarded – undo
650 650
 		$note_emails = array();
651 651
 
652 652
 		foreach ( $email_fields as $email_field ) {
653
-			if ( ! empty( $entry["{$email_field->id}"] ) && is_email( $entry["{$email_field->id}"] ) ) {
654
-				$note_emails[] = $entry["{$email_field->id}"];
653
+			if ( ! empty( $entry[ "{$email_field->id}" ] ) && is_email( $entry[ "{$email_field->id}" ] ) ) {
654
+				$note_emails[ ] = $entry[ "{$email_field->id}" ];
655 655
 			}
656 656
 		}
657 657
 
@@ -663,7 +663,7 @@  discard block
 block discarded – undo
663 663
 		 */
664 664
 		$note_emails = apply_filters( 'gravityview/field/notes/emails', $note_emails, $entry );
665 665
 
666
-		return (array) $note_emails;
666
+		return (array)$note_emails;
667 667
 	}
668 668
 
669 669
 	/**
@@ -679,7 +679,7 @@  discard block
 block discarded – undo
679 679
 	 */
680 680
 	private static function get_note_email_fields( $entry_slug = '' ) {
681 681
 
682
-		if( ! GVCommon::has_cap( 'gravityview_email_entry_notes' ) ) {
682
+		if ( ! GVCommon::has_cap( 'gravityview_email_entry_notes' ) ) {
683 683
 			gravityview()->log->error( 'User does not have permission to email entry notes ("gravityview_email_entry_notes").' );
684 684
 			return '';
685 685
 		}
@@ -701,27 +701,27 @@  discard block
 block discarded – undo
701 701
 
702 702
 		if ( ! empty( $note_emails ) || $include_custom ) { ?>
703 703
 			<div class="gv-note-email-container">
704
-				<label for="gv-note-email-to-<?php echo $entry_slug_esc; ?>" class="screen-reader-text"><?php echo $strings['also-email'];  ?></label>
704
+				<label for="gv-note-email-to-<?php echo $entry_slug_esc; ?>" class="screen-reader-text"><?php echo $strings[ 'also-email' ]; ?></label>
705 705
 				<select class="gv-note-email-to" name="gv-note-to" id="gv-note-email-to-<?php echo $entry_slug_esc; ?>">
706
-					<option value=""><?php echo $strings['also-email'];  ?></option>
706
+					<option value=""><?php echo $strings[ 'also-email' ]; ?></option>
707 707
 					<?php foreach ( $note_emails as  $email ) {
708 708
 						?>
709 709
 						<option value="<?php echo esc_attr( $email ); ?>"><?php echo esc_html( $email ); ?></option>
710 710
 					<?php }
711
-					if( $include_custom ) { ?>
712
-					<option value="custom"><?php echo self::strings('other-email'); ?></option>
711
+					if ( $include_custom ) { ?>
712
+					<option value="custom"><?php echo self::strings( 'other-email' ); ?></option>
713 713
 					<?php } ?>
714 714
 				</select>
715 715
 				<fieldset class="gv-note-to-container">
716
-					<?php if( $include_custom ) { ?>
716
+					<?php if ( $include_custom ) { ?>
717 717
 					<div class='gv-note-to-custom-container'>
718
-						<label for="gv-note-email-to-custom-<?php echo $entry_slug_esc; ?>"><?php echo $strings['email-label']; ?></label>
719
-						<input type="text" name="gv-note-to-custom" placeholder="<?php echo $strings['email-placeholder']; ?>" id="gv-note-to-custom-<?php echo $entry_slug_esc; ?>" value="" />
718
+						<label for="gv-note-email-to-custom-<?php echo $entry_slug_esc; ?>"><?php echo $strings[ 'email-label' ]; ?></label>
719
+						<input type="text" name="gv-note-to-custom" placeholder="<?php echo $strings[ 'email-placeholder' ]; ?>" id="gv-note-to-custom-<?php echo $entry_slug_esc; ?>" value="" />
720 720
 					</div>
721 721
 					<?php } ?>
722 722
 		            <div class='gv-note-subject-container'>
723
-		                <label for="gv-note-subject-<?php echo $entry_slug_esc; ?>"><?php echo $strings['subject-label']; ?></label>
724
-		                <input type="text" name="gv-note-subject" placeholder="<?php echo $strings['subject']; ?>" id="gv-note-subject-<?php echo $entry_slug_esc; ?>" value="" />
723
+		                <label for="gv-note-subject-<?php echo $entry_slug_esc; ?>"><?php echo $strings[ 'subject-label' ]; ?></label>
724
+		                <input type="text" name="gv-note-subject" placeholder="<?php echo $strings[ 'subject' ]; ?>" id="gv-note-subject-<?php echo $entry_slug_esc; ?>" value="" />
725 725
 		            </div>
726 726
 				</fieldset>
727 727
 			</div>
@@ -744,7 +744,7 @@  discard block
 block discarded – undo
744 744
 	 */
745 745
 	private function maybe_send_entry_notes( $note = false, $entry, $data ) {
746 746
 
747
-		if( ! $note || ! GVCommon::has_cap('gravityview_email_entry_notes') ) {
747
+		if ( ! $note || ! GVCommon::has_cap( 'gravityview_email_entry_notes' ) ) {
748 748
 			gravityview()->log->debug( 'User doesn\'t have "gravityview_email_entry_notes" cap, or $note is empty', array( 'data' => $note ) );
749 749
 			return;
750 750
 		}
@@ -752,7 +752,7 @@  discard block
 block discarded – undo
752 752
 		gravityview()->log->debug( '$data', array( 'data' => $data ) );
753 753
 
754 754
 		//emailing notes if configured
755
-		if ( ! empty( $data['gv-note-to'] ) ) {
755
+		if ( ! empty( $data[ 'gv-note-to' ] ) ) {
756 756
 
757 757
 			$default_data = array(
758 758
 				'gv-note-to' => '',
@@ -762,11 +762,11 @@  discard block
 block discarded – undo
762 762
                 'current-url' => '',
763 763
 			);
764 764
 
765
-			$current_user  = wp_get_current_user();
765
+			$current_user = wp_get_current_user();
766 766
 			$email_data = wp_parse_args( $data, $default_data );
767 767
 
768
-			$from    = $current_user->user_email;
769
-			$to = $email_data['gv-note-to'];
768
+			$from = $current_user->user_email;
769
+			$to = $email_data[ 'gv-note-to' ];
770 770
 
771 771
 			/**
772 772
 			 * Documented in get_note_email_fields
@@ -774,8 +774,8 @@  discard block
 block discarded – undo
774 774
 			 */
775 775
 			$include_custom = apply_filters( 'gravityview/field/notes/custom-email', true );
776 776
 
777
-			if( 'custom' === $to && $include_custom ) {
778
-				$to = $email_data['gv-note-to-custom'];
777
+			if ( 'custom' === $to && $include_custom ) {
778
+				$to = $email_data[ 'gv-note-to-custom' ];
779 779
 				gravityview()->log->debug( 'Sending note to a custom email address: {to}', array( 'to' => $to ) );
780 780
 			}
781 781
 
@@ -786,13 +786,13 @@  discard block
 block discarded – undo
786 786
 
787 787
 			$bcc = false;
788 788
 			$reply_to = $from;
789
-			$subject = trim( $email_data['gv-note-subject'] );
789
+			$subject = trim( $email_data[ 'gv-note-subject' ] );
790 790
 
791 791
 			// We use empty() here because GF uses empty to check against, too. `0` isn't a valid subject to GF
792 792
 			$subject = empty( $subject ) ? self::strings( 'default-email-subject' ) : $subject;
793
-			$message = $email_data['gv-note-content'];
793
+			$message = $email_data[ 'gv-note-content' ];
794 794
 			$email_footer = self::strings( 'email-footer' );
795
-			$from_name     = $current_user->display_name;
795
+			$from_name = $current_user->display_name;
796 796
 			$message_format = 'html';
797 797
 
798 798
 			/**
@@ -823,7 +823,7 @@  discard block
 block discarded – undo
823 823
 
824 824
 			GVCommon::send_email( $from, $to, $bcc, $reply_to, $subject, $message, $from_name, $message_format, '', $entry, false );
825 825
 
826
-			$form  = isset( $entry['form_id'] ) ? GFAPI::get_form( $entry['form_id'] ) : array();
826
+			$form = isset( $entry[ 'form_id' ] ) ? GFAPI::get_form( $entry[ 'form_id' ] ) : array();
827 827
 
828 828
 			/**
829 829
 			 * @see https://www.gravityhelp.com/documentation/article/10146-2/ It's here for compatibility with Gravity Forms
@@ -849,7 +849,7 @@  discard block
 block discarded – undo
849 849
 
850 850
 	    $output = '';
851 851
 
852
-		if( ! empty( $email_footer ) ) {
852
+		if ( ! empty( $email_footer ) ) {
853 853
 		    $url = \GV\Utils::get( $email_data, 'current-url' );
854 854
 			$url = html_entity_decode( $url );
855 855
 			$url = site_url( $url );
Please login to merge, or discard this patch.