Completed
Push — master ( 1fd6be...155036 )
by Zack
13s
created
plugin-and-theme-hooks/class-gravityview-plugin-hooks-ultimate-member.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -43,12 +43,12 @@  discard block
 block discarded – undo
43 43
 	function parse_um_profile_post_content( $args = array() ) {
44 44
 		global $ultimatemember;
45 45
 
46
-		if( ! $ultimatemember || ! is_object( $ultimatemember ) || ! class_exists( 'GravityView_View_Data' ) ) {
46
+		if ( ! $ultimatemember || ! is_object( $ultimatemember ) || ! class_exists( 'GravityView_View_Data' ) ) {
47 47
 			return;
48 48
 		}
49 49
 
50 50
 		// @todo Support Ultimate Member 2.0 - for now, prevent fatal error
51
-		if( ! isset( $ultimatemember->profile ) ) {
51
+		if ( ! isset( $ultimatemember->profile ) ) {
52 52
 			return;
53 53
 		}
54 54
 
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
 			return;
65 65
 		}
66 66
 
67
-		GravityView_View_Data::getInstance()->parse_post_content( $active_tab[0]->post_content );
67
+		GravityView_View_Data::getInstance()->parse_post_content( $active_tab[ 0 ]->post_content );
68 68
 
69 69
 		wp_reset_postdata();
70 70
 	}
Please login to merge, or discard this patch.
includes/extensions/delete-entry/class-delete-entry.php 1 patch
Spacing   +83 added lines, -83 removed lines patch added patch discarded remove patch
@@ -42,12 +42,12 @@  discard block
 block discarded – undo
42 42
 
43 43
 		add_action( 'wp', array( $this, 'process_delete' ), 10000 );
44 44
 
45
-		add_filter( 'gravityview_entry_default_fields', array( $this, 'add_default_field'), 10, 3 );
45
+		add_filter( 'gravityview_entry_default_fields', array( $this, 'add_default_field' ), 10, 3 );
46 46
 
47 47
 		add_action( 'gravityview_before', array( $this, 'display_message' ) );
48 48
 
49 49
 		// For the Delete Entry Link, you don't want visible to all users.
50
-		add_filter( 'gravityview_field_visibility_caps', array( $this, 'modify_visibility_caps'), 10, 5 );
50
+		add_filter( 'gravityview_field_visibility_caps', array( $this, 'modify_visibility_caps' ), 10, 5 );
51 51
 
52 52
 		// Modify the field options based on the name of the field type
53 53
 		add_filter( 'gravityview_template_delete_link_options', array( $this, 'delete_link_field_options' ), 10, 5 );
@@ -55,10 +55,10 @@  discard block
 block discarded – undo
55 55
 		// add template path to check for field
56 56
 		add_filter( 'gravityview_template_paths', array( $this, 'add_template_path' ) );
57 57
 
58
-		add_action( 'gravityview/edit-entry/publishing-action/after', array( $this, 'add_delete_button'), 10, 3 );
58
+		add_action( 'gravityview/edit-entry/publishing-action/after', array( $this, 'add_delete_button' ), 10, 3 );
59 59
 
60
-		add_action ( 'gravityview/delete-entry/deleted', array( $this, 'process_connected_posts' ), 10, 2 );
61
-		add_action ( 'gravityview/delete-entry/trashed', array( $this, 'process_connected_posts' ), 10, 2 );
60
+		add_action( 'gravityview/delete-entry/deleted', array( $this, 'process_connected_posts' ), 10, 2 );
61
+		add_action( 'gravityview/delete-entry/trashed', array( $this, 'process_connected_posts' ), 10, 2 );
62 62
 	}
63 63
 
64 64
 	/**
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
 	 */
70 70
 	static function getInstance() {
71 71
 
72
-		if( empty( self::$instance ) ) {
72
+		if ( empty( self::$instance ) ) {
73 73
 			self::$instance = new self;
74 74
 		}
75 75
 
@@ -105,20 +105,20 @@  discard block
 block discarded – undo
105 105
 	function delete_link_field_options( $field_options, $template_id, $field_id, $context, $input_type ) {
106 106
 
107 107
 		// Always a link, never a filter
108
-		unset( $field_options['show_as_link'], $field_options['search_filter'] );
108
+		unset( $field_options[ 'show_as_link' ], $field_options[ 'search_filter' ] );
109 109
 
110 110
 		// Delete Entry link should only appear to visitors capable of editing entries
111
-		unset( $field_options['only_loggedin'], $field_options['only_loggedin_cap'] );
111
+		unset( $field_options[ 'only_loggedin' ], $field_options[ 'only_loggedin_cap' ] );
112 112
 
113
-		$add_option['delete_link'] = array(
113
+		$add_option[ 'delete_link' ] = array(
114 114
 			'type' => 'text',
115 115
 			'label' => __( 'Delete Link Text', 'gravityview' ),
116 116
 			'desc' => NULL,
117
-			'value' => __('Delete Entry', 'gravityview'),
117
+			'value' => __( 'Delete Entry', 'gravityview' ),
118 118
 			'merge_tags' => true,
119 119
 		);
120 120
 
121
-		$field_options['allow_edit_cap'] = array(
121
+		$field_options[ 'allow_edit_cap' ] = array(
122 122
 			'type' => 'select',
123 123
 			'label' => __( 'Allow the following users to delete the entry:', 'gravityview' ),
124 124
 			'choices' => GravityView_Render_Settings::get_cap_choices( $template_id, $field_id, $context, $input_type ),
@@ -142,8 +142,8 @@  discard block
 block discarded – undo
142 142
 	 */
143 143
 	function add_default_field( $entry_default_fields, $form = array(), $zone = '' ) {
144 144
 
145
-		if( 'edit' !== $zone ) {
146
-			$entry_default_fields['delete_link'] = array(
145
+		if ( 'edit' !== $zone ) {
146
+			$entry_default_fields[ 'delete_link' ] = array(
147 147
 				'label' => __( 'Delete Entry', 'gravityview' ),
148 148
 				'type'  => 'delete_link',
149 149
 				'desc'  => __( 'A link to delete the entry. Respects the Delete Entry permissions.', 'gravityview' ),
@@ -160,7 +160,7 @@  discard block
 block discarded – undo
160 160
 	 */
161 161
 	function add_available_field( $available_fields = array() ) {
162 162
 
163
-		$available_fields['delete_link'] = array(
163
+		$available_fields[ 'delete_link' ] = array(
164 164
 			'label_text' => __( 'Delete Entry', 'gravityview' ),
165 165
 			'field_id' => 'delete_link',
166 166
 			'label_type' => 'field',
@@ -188,12 +188,12 @@  discard block
 block discarded – undo
188 188
 		$caps = $visibility_caps;
189 189
 
190 190
 		// If we're configuring fields in the edit context, we want a limited selection
191
-		if( $field_id === 'delete_link' ) {
191
+		if ( $field_id === 'delete_link' ) {
192 192
 
193 193
 			// Remove other built-in caps.
194
-			unset( $caps['publish_posts'], $caps['gravityforms_view_entries'], $caps['delete_others_posts'] );
194
+			unset( $caps[ 'publish_posts' ], $caps[ 'gravityforms_view_entries' ], $caps[ 'delete_others_posts' ] );
195 195
 
196
-			$caps['read'] = _x('Entry Creator', 'User capability', 'gravityview');
196
+			$caps[ 'read' ] = _x( 'Entry Creator', 'User capability', 'gravityview' );
197 197
 		}
198 198
 
199 199
 		return $caps;
@@ -206,7 +206,7 @@  discard block
 block discarded – undo
206 206
 	 * @param [type] $entry [description]
207 207
 	 */
208 208
 	function set_entry( $entry = null ) {
209
-		$this->entry = empty( $entry ) ? GravityView_View::getInstance()->entries[0] : $entry;
209
+		$this->entry = empty( $entry ) ? GravityView_View::getInstance()->entries[ 0 ] : $entry;
210 210
 	}
211 211
 
212 212
 	/**
@@ -236,13 +236,13 @@  discard block
 block discarded – undo
236 236
 
237 237
         $base = GravityView_API::directory_link( $post_id, true );
238 238
 
239
-		if( empty( $base ) ) {
240
-			do_action( 'gravityview_log_error', __METHOD__ . ' - Post ID does not exist: '.$post_id );
239
+		if ( empty( $base ) ) {
240
+			do_action( 'gravityview_log_error', __METHOD__ . ' - Post ID does not exist: ' . $post_id );
241 241
 			return NULL;
242 242
 		}
243 243
 
244 244
 		// Use the slug instead of the ID for consistent security
245
-		$entry_slug = GravityView_API::get_entry_slug( $entry['id'], $entry );
245
+		$entry_slug = GravityView_API::get_entry_slug( $entry[ 'id' ], $entry );
246 246
 
247 247
         $view_id = empty( $view_id ) ? gravityview_get_view_id() : $view_id;
248 248
 
@@ -253,7 +253,7 @@  discard block
 block discarded – undo
253 253
             'view_id' => $view_id,
254 254
 		), $base );
255 255
 
256
-		$url = wp_nonce_url( $actionurl, 'delete_'.$entry_slug, 'delete' );
256
+		$url = wp_nonce_url( $actionurl, 'delete_' . $entry_slug, 'delete' );
257 257
 
258 258
 		return $url;
259 259
 	}
@@ -270,7 +270,7 @@  discard block
 block discarded – undo
270 270
 	function add_delete_button( $form = array(), $entry = array(), $view_id = NULL ) {
271 271
 
272 272
 		// Only show the link to those who are allowed to see it.
273
-		if( !self::check_user_cap_delete_entry( $entry ) ) {
273
+		if ( ! self::check_user_cap_delete_entry( $entry ) ) {
274 274
 			return;
275 275
 		}
276 276
 
@@ -281,7 +281,7 @@  discard block
 block discarded – undo
281 281
 		$show_delete_button = apply_filters( 'gravityview/delete-entry/show-delete-button', true );
282 282
 
283 283
 		// If the button is hidden by the filter, don't show.
284
-		if( !$show_delete_button ) {
284
+		if ( ! $show_delete_button ) {
285 285
 			return;
286 286
 		}
287 287
 
@@ -311,27 +311,27 @@  discard block
 block discarded – undo
311 311
 	function process_delete() {
312 312
 
313 313
 		// If the form is submitted
314
-		if( isset( $_GET['action'] ) && 'delete' === $_GET['action'] && isset( $_GET['entry_id'] ) ) {
314
+		if ( isset( $_GET[ 'action' ] ) && 'delete' === $_GET[ 'action' ] && isset( $_GET[ 'entry_id' ] ) ) {
315 315
 
316 316
 			// Make sure it's a GravityView request
317
-			$valid_nonce_key = wp_verify_nonce( $_GET['delete'], self::get_nonce_key( $_GET['entry_id'] ) );
317
+			$valid_nonce_key = wp_verify_nonce( $_GET[ 'delete' ], self::get_nonce_key( $_GET[ 'entry_id' ] ) );
318 318
 
319
-			if( ! $valid_nonce_key ) {
320
-				do_action('gravityview_log_debug', __METHOD__ . ' Delete entry not processed: nonce validation failed.' );
319
+			if ( ! $valid_nonce_key ) {
320
+				do_action( 'gravityview_log_debug', __METHOD__ . ' Delete entry not processed: nonce validation failed.' );
321 321
 				return;
322 322
 			}
323 323
 
324 324
 			// Get the entry slug
325
-			$entry_slug = esc_attr( $_GET['entry_id'] );
325
+			$entry_slug = esc_attr( $_GET[ 'entry_id' ] );
326 326
 
327 327
 			// See if there's an entry there
328 328
 			$entry = gravityview_get_entry( $entry_slug, true, false );
329 329
 
330
-			if( $entry ) {
330
+			if ( $entry ) {
331 331
 
332 332
 				$has_permission = $this->user_can_delete_entry( $entry );
333 333
 
334
-				if( is_wp_error( $has_permission ) ) {
334
+				if ( is_wp_error( $has_permission ) ) {
335 335
 
336 336
 					$messages = array(
337 337
 						'message' => urlencode( $has_permission->get_error_message() ),
@@ -343,7 +343,7 @@  discard block
 block discarded – undo
343 343
 					// Delete the entry
344 344
 					$delete_response = $this->delete_or_trash_entry( $entry );
345 345
 
346
-					if( is_wp_error( $delete_response ) ) {
346
+					if ( is_wp_error( $delete_response ) ) {
347 347
 
348 348
 						$messages = array(
349 349
 							'message' => urlencode( $delete_response->get_error_message() ),
@@ -362,10 +362,10 @@  discard block
 block discarded – undo
362 362
 
363 363
 			} else {
364 364
 
365
-				do_action('gravityview_log_debug', __METHOD__ . ' Delete entry failed: there was no entry with the entry slug '. $entry_slug );
365
+				do_action( 'gravityview_log_debug', __METHOD__ . ' Delete entry failed: there was no entry with the entry slug ' . $entry_slug );
366 366
 
367 367
 				$messages = array(
368
-					'message' => urlencode( __('The entry does not exist.', 'gravityview') ),
368
+					'message' => urlencode( __( 'The entry does not exist.', 'gravityview' ) ),
369 369
 					'status' => 'error',
370 370
 				);
371 371
 			}
@@ -405,18 +405,18 @@  discard block
 block discarded – undo
405 405
 	 */
406 406
 	private function delete_or_trash_entry( $entry ) {
407 407
 
408
-		$entry_id = $entry['id'];
408
+		$entry_id = $entry[ 'id' ];
409 409
 		
410 410
 		$mode = $this->get_delete_mode();
411 411
 
412
-		if( 'delete' === $mode ) {
412
+		if ( 'delete' === $mode ) {
413 413
 
414 414
 			do_action( 'gravityview_log_debug', __METHOD__ . ' Starting delete entry: ', $entry_id );
415 415
 
416 416
 			// Delete the entry
417 417
 			$delete_response = GFAPI::delete_entry( $entry_id );
418 418
 
419
-			if( ! is_wp_error( $delete_response ) ) {
419
+			if ( ! is_wp_error( $delete_response ) ) {
420 420
 				$delete_response = 'deleted';
421 421
 
422 422
 				/**
@@ -437,8 +437,8 @@  discard block
 block discarded – undo
437 437
 			$trashed = GFAPI::update_entry_property( $entry_id, 'status', 'trash' );
438 438
 			new GravityView_Cache;
439 439
 
440
-			if( ! $trashed ) {
441
-				$delete_response = new WP_Error( 'trash_entry_failed', __('Moving the entry to the trash failed.', 'gravityview' ) );
440
+			if ( ! $trashed ) {
441
+				$delete_response = new WP_Error( 'trash_entry_failed', __( 'Moving the entry to the trash failed.', 'gravityview' ) );
442 442
 			} else {
443 443
 
444 444
 				/**
@@ -469,7 +469,7 @@  discard block
 block discarded – undo
469 469
 	public function process_connected_posts( $entry_id = 0, $entry = array() ) {
470 470
 
471 471
 		// The entry had no connected post
472
-		if( empty( $entry['post_id'] ) ) {
472
+		if ( empty( $entry[ 'post_id' ] ) ) {
473 473
 			return;
474 474
 		}
475 475
 
@@ -480,22 +480,22 @@  discard block
 block discarded – undo
480 480
 		 */
481 481
 		$delete_post = apply_filters( 'gravityview/delete-entry/delete-connected-post', true );
482 482
 		
483
-		if( false === $delete_post ) {
483
+		if ( false === $delete_post ) {
484 484
 			return;
485 485
 		}
486 486
 
487 487
 		$action = current_action();
488 488
 
489
-		if( 'gravityview/delete-entry/deleted' === $action ) {
490
-			$result = wp_delete_post( $entry['post_id'], true );
489
+		if ( 'gravityview/delete-entry/deleted' === $action ) {
490
+			$result = wp_delete_post( $entry[ 'post_id' ], true );
491 491
 		} else {
492
-			$result = wp_trash_post( $entry['post_id'] );
492
+			$result = wp_trash_post( $entry[ 'post_id' ] );
493 493
 		}
494 494
 
495
-		if( false === $result ) {
496
-			do_action( 'gravityview_log_error', __METHOD__ . ' (called by '.$action.'): Error processing the Post connected to the entry.', $entry );
495
+		if ( false === $result ) {
496
+			do_action( 'gravityview_log_error', __METHOD__ . ' (called by ' . $action . '): Error processing the Post connected to the entry.', $entry );
497 497
 		} else {
498
-			do_action( 'gravityview_log_debug', __METHOD__ . ' (called by '.$action.'): Successfully processed Post connected to the entry.', $entry );
498
+			do_action( 'gravityview_log_debug', __METHOD__ . ' (called by ' . $action . '): Successfully processed Post connected to the entry.', $entry );
499 499
 		}
500 500
 	}
501 501
 
@@ -508,13 +508,13 @@  discard block
 block discarded – undo
508 508
 	public function verify_nonce() {
509 509
 
510 510
 		// No delete entry request was made
511
-		if( empty( $_GET['entry_id'] ) || empty( $_GET['delete'] ) ) {
511
+		if ( empty( $_GET[ 'entry_id' ] ) || empty( $_GET[ 'delete' ] ) ) {
512 512
 			return false;
513 513
 		}
514 514
 
515
-		$nonce_key = self::get_nonce_key( $_GET['entry_id'] );
515
+		$nonce_key = self::get_nonce_key( $_GET[ 'entry_id' ] );
516 516
 
517
-		$valid = wp_verify_nonce( $_GET['delete'], $nonce_key );
517
+		$valid = wp_verify_nonce( $_GET[ 'delete' ], $nonce_key );
518 518
 
519 519
 		/**
520 520
 		 * @filter `gravityview/delete-entry/verify_nonce` Override Delete Entry nonce validation. Return true to declare nonce valid.
@@ -536,7 +536,7 @@  discard block
 block discarded – undo
536 536
 	 */
537 537
 	public static function get_confirm_dialog() {
538 538
 
539
-		$confirm = __('Are you sure you want to delete this entry? This cannot be undone.', 'gravityview');
539
+		$confirm = __( 'Are you sure you want to delete this entry? This cannot be undone.', 'gravityview' );
540 540
 
541 541
 		/**
542 542
 		 * @filter `gravityview/delete-entry/confirm-text` Modify the Delete Entry Javascript confirmation text
@@ -544,7 +544,7 @@  discard block
 block discarded – undo
544 544
 		 */
545 545
 		$confirm = apply_filters( 'gravityview/delete-entry/confirm-text', $confirm );
546 546
 
547
-		return 'return window.confirm(\''. esc_js( $confirm ) .'\');';
547
+		return 'return window.confirm(\'' . esc_js( $confirm ) . '\');';
548 548
 	}
549 549
 
550 550
 	/**
@@ -562,16 +562,16 @@  discard block
 block discarded – undo
562 562
 
563 563
 		$error = NULL;
564 564
 
565
-		if( ! $this->verify_nonce() ) {
566
-			$error = __( 'The link to delete this entry is not valid; it may have expired.', 'gravityview');
565
+		if ( ! $this->verify_nonce() ) {
566
+			$error = __( 'The link to delete this entry is not valid; it may have expired.', 'gravityview' );
567 567
 		}
568 568
 
569
-		if( ! self::check_user_cap_delete_entry( $entry ) ) {
570
-			$error = __( 'You do not have permission to delete this entry.', 'gravityview');
569
+		if ( ! self::check_user_cap_delete_entry( $entry ) ) {
570
+			$error = __( 'You do not have permission to delete this entry.', 'gravityview' );
571 571
 		}
572 572
 
573
-		if( $entry['status'] === 'trash' ) {
574
-			if( 'trash' === $this->get_delete_mode() ) {
573
+		if ( $entry[ 'status' ] === 'trash' ) {
574
+			if ( 'trash' === $this->get_delete_mode() ) {
575 575
 				$error = __( 'The entry is already in the trash.', 'gravityview' );
576 576
 			} else {
577 577
 				$error = __( 'You cannot delete the entry; it is already in the trash.', 'gravityview' );
@@ -579,11 +579,11 @@  discard block
 block discarded – undo
579 579
 		}
580 580
 
581 581
 		// No errors; everything's fine here!
582
-		if( empty( $error ) ) {
582
+		if ( empty( $error ) ) {
583 583
 			return true;
584 584
 		}
585 585
 
586
-		do_action('gravityview_log_error', 'GravityView_Delete_Entry[user_can_delete_entry]' . $error );
586
+		do_action( 'gravityview_log_error', 'GravityView_Delete_Entry[user_can_delete_entry]' . $error );
587 587
 
588 588
 		return new WP_Error( 'gravityview-delete-entry-permissions', $error );
589 589
 	}
@@ -605,32 +605,32 @@  discard block
 block discarded – undo
605 605
 
606 606
 		$current_user = wp_get_current_user();
607 607
 
608
-		$entry_id = isset( $entry['id'] ) ? $entry['id'] : NULL;
608
+		$entry_id = isset( $entry[ 'id' ] ) ? $entry[ 'id' ] : NULL;
609 609
 
610 610
 		// Or if they can delete any entries (as defined in Gravity Forms), we're good.
611
-		if( GVCommon::has_cap( array( 'gravityforms_delete_entries', 'gravityview_delete_others_entries' ), $entry_id ) ) {
611
+		if ( GVCommon::has_cap( array( 'gravityforms_delete_entries', 'gravityview_delete_others_entries' ), $entry_id ) ) {
612 612
 
613
-			do_action('gravityview_log_debug', 'GravityView_Delete_Entry[check_user_cap_delete_entry] Current user has `gravityforms_delete_entries` or `gravityview_delete_others_entries` capability.' );
613
+			do_action( 'gravityview_log_debug', 'GravityView_Delete_Entry[check_user_cap_delete_entry] Current user has `gravityforms_delete_entries` or `gravityview_delete_others_entries` capability.' );
614 614
 
615 615
 			return true;
616 616
 		}
617 617
 
618 618
 
619 619
 		// If field options are passed, check if current user can view the link
620
-		if( !empty( $field ) ) {
620
+		if ( ! empty( $field ) ) {
621 621
 
622 622
 			// If capability is not defined, something is not right!
623
-			if( empty( $field['allow_edit_cap'] ) ) {
623
+			if ( empty( $field[ 'allow_edit_cap' ] ) ) {
624 624
 
625 625
 				do_action( 'gravityview_log_error', 'GravityView_Delete_Entry[check_user_cap_delete_entry] Cannot read delete entry field caps', $field );
626 626
 
627 627
 				return false;
628 628
 			}
629 629
 
630
-			if( GVCommon::has_cap( $field['allow_edit_cap'] ) ) {
630
+			if ( GVCommon::has_cap( $field[ 'allow_edit_cap' ] ) ) {
631 631
 
632 632
 				// Do not return true if cap is read, as we need to check if the current user created the entry
633
-				if( $field['allow_edit_cap'] !== 'read' ) {
633
+				if ( $field[ 'allow_edit_cap' ] !== 'read' ) {
634 634
 					return true;
635 635
 				}
636 636
 
@@ -643,9 +643,9 @@  discard block
 block discarded – undo
643 643
 
644 644
 		}
645 645
 
646
-		if( !isset( $entry['created_by'] ) ) {
646
+		if ( ! isset( $entry[ 'created_by' ] ) ) {
647 647
 
648
-			do_action('gravityview_log_error', 'GravityView_Delete_Entry[check_user_cap_delete_entry] Entry `created_by` doesn\'t exist.');
648
+			do_action( 'gravityview_log_error', 'GravityView_Delete_Entry[check_user_cap_delete_entry] Entry `created_by` doesn\'t exist.' );
649 649
 
650 650
 			return false;
651 651
 		}
@@ -653,24 +653,24 @@  discard block
 block discarded – undo
653 653
 		$view_id = empty( $view_id ) ? $gravityview_view->getViewId() : $view_id;
654 654
 
655 655
 		// Only checks user_delete view option if view is already set
656
-		if( $view_id ) {
656
+		if ( $view_id ) {
657 657
 
658 658
 			$current_view = gravityview_get_current_view_data( $view_id );
659 659
 
660
-			$user_delete = isset( $current_view['atts']['user_delete'] ) ? $current_view['atts']['user_delete'] : false;
660
+			$user_delete = isset( $current_view[ 'atts' ][ 'user_delete' ] ) ? $current_view[ 'atts' ][ 'user_delete' ] : false;
661 661
 
662
-			if( empty( $user_delete ) ) {
662
+			if ( empty( $user_delete ) ) {
663 663
 
664
-				do_action('gravityview_log_debug', 'GravityView_Delete_Entry[check_user_cap_delete_entry] User Delete is disabled. Returning false.' );
664
+				do_action( 'gravityview_log_debug', 'GravityView_Delete_Entry[check_user_cap_delete_entry] User Delete is disabled. Returning false.' );
665 665
 
666 666
 				return false;
667 667
 			}
668 668
 		}
669 669
 
670 670
 		// If the logged-in user is the same as the user who created the entry, we're good.
671
-		if( is_user_logged_in() && intval( $current_user->ID ) === intval( $entry['created_by'] ) ) {
671
+		if ( is_user_logged_in() && intval( $current_user->ID ) === intval( $entry[ 'created_by' ] ) ) {
672 672
 
673
-			do_action('gravityview_log_debug', sprintf( 'GravityView_Delete_Entry[check_user_cap_delete_entry] User %s created the entry.', $current_user->ID ) );
673
+			do_action( 'gravityview_log_debug', sprintf( 'GravityView_Delete_Entry[check_user_cap_delete_entry] User %s created the entry.', $current_user->ID ) );
674 674
 
675 675
 			return true;
676 676
 		}
@@ -693,31 +693,31 @@  discard block
 block discarded – undo
693 693
 	 */
694 694
 	public function display_message( $current_view_id = 0 ) {
695 695
 
696
-		if( empty( $_GET['status'] ) || ! self::verify_nonce() ) {
696
+		if ( empty( $_GET[ 'status' ] ) || ! self::verify_nonce() ) {
697 697
 			return;
698 698
 		}
699 699
 
700 700
 		// Entry wasn't deleted from current View
701
-		if( isset( $_GET['view_id'] ) && intval( $_GET['view_id'] ) !== intval( $current_view_id ) ) {
701
+		if ( isset( $_GET[ 'view_id' ] ) && intval( $_GET[ 'view_id' ] ) !== intval( $current_view_id ) ) {
702 702
 			return;
703 703
 		}
704 704
 
705
-		$status = esc_attr( $_GET['status'] );
706
-		$message_from_url = rgget('message');
705
+		$status = esc_attr( $_GET[ 'status' ] );
706
+		$message_from_url = rgget( 'message' );
707 707
 		$message_from_url = rawurldecode( stripslashes_deep( $message_from_url ) );
708 708
 		$class = '';
709 709
 
710 710
 		switch ( $status ) {
711 711
 			case 'error':
712 712
 				$class = ' gv-error error';
713
-				$error_message = __('There was an error deleting the entry: %s', 'gravityview');
713
+				$error_message = __( 'There was an error deleting the entry: %s', 'gravityview' );
714 714
 				$message = sprintf( $error_message, $message_from_url );
715 715
 				break;
716 716
 			case 'trashed':
717
-				$message = __('The entry was successfully moved to the trash.', 'gravityview');
717
+				$message = __( 'The entry was successfully moved to the trash.', 'gravityview' );
718 718
 				break;
719 719
 			default:
720
-				$message = __('The entry was successfully deleted.', 'gravityview');
720
+				$message = __( 'The entry was successfully deleted.', 'gravityview' );
721 721
 				break;
722 722
 		}
723 723
 
@@ -731,7 +731,7 @@  discard block
 block discarded – undo
731 731
 		$message = apply_filters( 'gravityview/delete-entry/message', esc_attr( $message ), $status, $message_from_url );
732 732
 
733 733
 		// DISPLAY ERROR/SUCCESS MESSAGE
734
-		echo '<div class="gv-notice' . esc_attr( $class ) .'">'. $message .'</div>';
734
+		echo '<div class="gv-notice' . esc_attr( $class ) . '">' . $message . '</div>';
735 735
 	}
736 736
 
737 737
 
Please login to merge, or discard this patch.
includes/plugin-and-theme-hooks/class-gravityview-plugin-hooks-acf.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -44,20 +44,20 @@  discard block
 block discarded – undo
44 44
 	function add_meta_keys_from_post( $meta_keys = array(), $post_id = 0 ) {
45 45
 
46 46
 		// Can never be too careful: double-check that ACF is active and the function exists
47
-		if ( ! function_exists( 'get_field_objects' ) || ! class_exists('acf_field_functions' ) ) {
47
+		if ( ! function_exists( 'get_field_objects' ) || ! class_exists( 'acf_field_functions' ) ) {
48 48
 			return $meta_keys;
49 49
 		}
50 50
 
51 51
 		$acf_field_functions = new acf_field_functions;
52 52
 
53
-		remove_action('acf/format_value', array( $acf_field_functions, 'format_value'), 5 );
53
+		remove_action( 'acf/format_value', array( $acf_field_functions, 'format_value' ), 5 );
54 54
 
55 55
 		$acf_keys = get_field_objects( $post_id, array( 'load_value' => false ) );
56 56
 
57 57
 		// Restore existing filters added by format_value
58
-		add_action('acf/format_value', array( $acf_field_functions, 'format_value'), 5, 3 );
58
+		add_action( 'acf/format_value', array( $acf_field_functions, 'format_value' ), 5, 3 );
59 59
 
60
-		if( $acf_keys ) {
60
+		if ( $acf_keys ) {
61 61
 			return array_merge( array_keys( $acf_keys ), $meta_keys );
62 62
 		}
63 63
 
@@ -72,9 +72,9 @@  discard block
 block discarded – undo
72 72
 	 * @return void
73 73
 	 */
74 74
 	private function fix_posted_fields() {
75
-		if( is_admin() && isset( $_POST['action'] ) && isset( $_POST['post_type'] ) ) {
76
-			if( 'editpost' === $_POST['action'] && 'gravityview' === $_POST['post_type'] ) {
77
-				$_POST['fields'] = _gravityview_process_posted_fields();
75
+		if ( is_admin() && isset( $_POST[ 'action' ] ) && isset( $_POST[ 'post_type' ] ) ) {
76
+			if ( 'editpost' === $_POST[ 'action' ] && 'gravityview' === $_POST[ 'post_type' ] ) {
77
+				$_POST[ 'fields' ] = _gravityview_process_posted_fields();
78 78
 			}
79 79
 		}
80 80
 	}
Please login to merge, or discard this patch.
gravityview.php 1 patch
Spacing   +25 added lines, -25 removed lines patch added patch discarded remove patch
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
 	 */
91 91
 	public static function getInstance() {
92 92
 
93
-		if( empty( self::$instance ) ) {
93
+		if ( empty( self::$instance ) ) {
94 94
 			self::$instance = new self;
95 95
 		}
96 96
 
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
101 101
 
102 102
 		self::require_files();
103 103
 
104
-		if( ! GravityView_Compatibility::is_valid() ) {
104
+		if ( ! GravityView_Compatibility::is_valid() ) {
105 105
 			return;
106 106
 		}
107 107
 
@@ -116,15 +116,15 @@  discard block
 block discarded – undo
116 116
 	 */
117 117
 	private static function require_files() {
118 118
 		require_once( GRAVITYVIEW_DIR . 'includes/helper-functions.php' );
119
-		require_once( GRAVITYVIEW_DIR . 'includes/class-common.php');
120
-		require_once( GRAVITYVIEW_DIR . 'includes/class-gravityview-html-elements.php');
121
-		require_once( GRAVITYVIEW_DIR . 'includes/connector-functions.php');
119
+		require_once( GRAVITYVIEW_DIR . 'includes/class-common.php' );
120
+		require_once( GRAVITYVIEW_DIR . 'includes/class-gravityview-html-elements.php' );
121
+		require_once( GRAVITYVIEW_DIR . 'includes/connector-functions.php' );
122 122
 		require_once( GRAVITYVIEW_DIR . 'includes/class-gravityview-compatibility.php' );
123 123
 		require_once( GRAVITYVIEW_DIR . 'includes/class-gravityview-roles-capabilities.php' );
124 124
 		require_once( GRAVITYVIEW_DIR . 'includes/class-gravityview-admin-notices.php' );
125 125
 		require_once( GRAVITYVIEW_DIR . 'includes/class-admin.php' );
126
-		require_once( GRAVITYVIEW_DIR . 'includes/class-post-types.php');
127
-		require_once( GRAVITYVIEW_DIR . 'includes/class-cache.php');
126
+		require_once( GRAVITYVIEW_DIR . 'includes/class-post-types.php' );
127
+		require_once( GRAVITYVIEW_DIR . 'includes/class-cache.php' );
128 128
 	}
129 129
 
130 130
 	/**
@@ -165,9 +165,9 @@  discard block
 block discarded – undo
165 165
 		// Load Extensions
166 166
 		// @todo: Convert to a scan of the directory or a method where this all lives
167 167
 		include_once( GRAVITYVIEW_DIR . 'includes/class-gravityview-gfformsmodel.php' );
168
-		include_once( GRAVITYVIEW_DIR .'includes/extensions/edit-entry/class-edit-entry.php' );
169
-		include_once( GRAVITYVIEW_DIR .'includes/extensions/delete-entry/class-delete-entry.php' );
170
-		include_once( GRAVITYVIEW_DIR .'includes/extensions/entry-notes/class-gravityview-field-notes.php' );
168
+		include_once( GRAVITYVIEW_DIR . 'includes/extensions/edit-entry/class-edit-entry.php' );
169
+		include_once( GRAVITYVIEW_DIR . 'includes/extensions/delete-entry/class-delete-entry.php' );
170
+		include_once( GRAVITYVIEW_DIR . 'includes/extensions/entry-notes/class-gravityview-field-notes.php' );
171 171
 
172 172
 		// Load WordPress Widgets
173 173
 		include_once( GRAVITYVIEW_DIR . 'includes/wordpress-widgets/register-wordpress-widgets.php' );
@@ -186,7 +186,7 @@  discard block
 block discarded – undo
186 186
 		include_once( GRAVITYVIEW_DIR . 'includes/class-frontend-views.php' );
187 187
 		include_once( GRAVITYVIEW_DIR . 'includes/class-gravityview-admin-bar.php' );
188 188
 		include_once( GRAVITYVIEW_DIR . 'includes/class-gravityview-entry-list.php' );
189
-		include_once( GRAVITYVIEW_DIR . 'includes/class-gravityview-merge-tags.php'); /** @since 1.8.4 */
189
+		include_once( GRAVITYVIEW_DIR . 'includes/class-gravityview-merge-tags.php' ); /** @since 1.8.4 */
190 190
 		include_once( GRAVITYVIEW_DIR . 'includes/class-data.php' );
191 191
 		include_once( GRAVITYVIEW_DIR . 'includes/class-gravityview-shortcode.php' );
192 192
 		include_once( GRAVITYVIEW_DIR . 'includes/class-gravityview-entry-link-shortcode.php' );
@@ -201,7 +201,7 @@  discard block
 block discarded – undo
201 201
 	 * @return bool
202 202
 	 */
203 203
 	public static function is_network_activated() {
204
-		return is_multisite() && ( function_exists('is_plugin_active_for_network') && is_plugin_active_for_network( 'gravityview/gravityview.php' ) );
204
+		return is_multisite() && ( function_exists( 'is_plugin_active_for_network' ) && is_plugin_active_for_network( 'gravityview/gravityview.php' ) );
205 205
 	}
206 206
 
207 207
 
@@ -222,12 +222,12 @@  discard block
 block discarded – undo
222 222
 
223 223
 			if ( php_sapi_name() == 'cli' ) {
224 224
 				printf( __( "GravityView requires PHP Version %s or newer. You're using Version %s. Please ask your host to upgrade your server's PHP.", 'gravityview' ),
225
-					GV_FUTURE_MIN_PHP_VERSION , phpversion() );
225
+					GV_FUTURE_MIN_PHP_VERSION, phpversion() );
226 226
 			} else {
227 227
 				printf( '<body style="padding: 0; margin: 0; font-family: -apple-system, BlinkMacSystemFont, \'Segoe UI\', Roboto, Oxygen-Sans, Ubuntu, Cantarell, \'Helvetica Neue\', sans-serif;">' );
228 228
 				printf( '<img src="' . plugins_url( 'assets/images/astronaut-200x263.png', GRAVITYVIEW_FILE ) . '" alt="The GravityView Astronaut Says:" style="float: left; height: 60px; margin-right : 10px;" />' );
229 229
 				printf( __( "%sGravityView requires PHP Version %s or newer.%s \n\nYou're using Version %s. Please ask your host to upgrade your server's PHP.", 'gravityview' ),
230
-					'<h3 style="font-size:16px; margin: 0 0 8px 0;">', GV_FUTURE_MIN_PHP_VERSION , "</h3>\n\n", $version );
230
+					'<h3 style="font-size:16px; margin: 0 0 8px 0;">', GV_FUTURE_MIN_PHP_VERSION, "</h3>\n\n", $version );
231 231
 				printf( '</body>' );
232 232
 			}
233 233
 
@@ -297,7 +297,7 @@  discard block
 block discarded – undo
297 297
 	 * @since 1.7.5.1
298 298
 	 */
299 299
 	public static function include_widget_class() {
300
-		include_once( GRAVITYVIEW_DIR .'includes/widgets/class-gravityview-widget.php' );
300
+		include_once( GRAVITYVIEW_DIR . 'includes/widgets/class-gravityview-widget.php' );
301 301
 	}
302 302
 
303 303
 
@@ -319,7 +319,7 @@  discard block
 block discarded – undo
319 319
 		}
320 320
 		if ( ! $loaded ) {
321 321
 			$locale = apply_filters( 'plugin_locale', get_locale(), 'gravityview' );
322
-			$mofile = dirname( __FILE__ ) . '/languages/gravityview-'. $locale .'.mo';
322
+			$mofile = dirname( __FILE__ ) . '/languages/gravityview-' . $locale . '.mo';
323 323
 			load_textdomain( 'gravityview', $mofile );
324 324
 		}
325 325
 
@@ -356,12 +356,12 @@  discard block
 block discarded – undo
356 356
 	 */
357 357
 	public function frontend_actions( $force = false ) {
358 358
 
359
-		if( self::is_admin() && ! $force ) { return; }
359
+		if ( self::is_admin() && ! $force ) { return; }
360 360
 
361 361
 		include_once( GRAVITYVIEW_DIR . 'includes/class-gravityview-image.php' );
362
-		include_once( GRAVITYVIEW_DIR .'includes/class-template.php' );
363
-		include_once( GRAVITYVIEW_DIR .'includes/class-api.php' );
364
-		include_once( GRAVITYVIEW_DIR .'includes/class-frontend-views.php' );
362
+		include_once( GRAVITYVIEW_DIR . 'includes/class-template.php' );
363
+		include_once( GRAVITYVIEW_DIR . 'includes/class-api.php' );
364
+		include_once( GRAVITYVIEW_DIR . 'includes/class-frontend-views.php' );
365 365
 		include_once( GRAVITYVIEW_DIR . 'includes/class-gravityview-change-entry-creator.php' );
366 366
 
367 367
 
@@ -386,8 +386,8 @@  discard block
 block discarded – undo
386 386
 	 */
387 387
 	public static function get_default_widget_areas() {
388 388
 		$default_areas = array(
389
-			array( '1-1' => array( array( 'areaid' => 'top', 'title' => __('Top', 'gravityview' ) , 'subtitle' => '' ) ) ),
390
-			array( '1-2' => array( array( 'areaid' => 'left', 'title' => __('Left', 'gravityview') , 'subtitle' => '' ) ), '2-2' => array( array( 'areaid' => 'right', 'title' => __('Right', 'gravityview') , 'subtitle' => '' ) ) ),
389
+			array( '1-1' => array( array( 'areaid' => 'top', 'title' => __( 'Top', 'gravityview' ), 'subtitle' => '' ) ) ),
390
+			array( '1-2' => array( array( 'areaid' => 'left', 'title' => __( 'Left', 'gravityview' ), 'subtitle' => '' ) ), '2-2' => array( array( 'areaid' => 'right', 'title' => __( 'Right', 'gravityview' ), 'subtitle' => '' ) ) ),
391 391
 		);
392 392
 
393 393
 		/**
@@ -405,7 +405,7 @@  discard block
 block discarded – undo
405 405
      * @param mixed $data Additional data to display
406 406
      * @return void
407 407
      */
408
-    public static function log_debug( $message, $data = null ){
408
+    public static function log_debug( $message, $data = null ) {
409 409
 	    /**
410 410
 	     * @action `gravityview_log_debug` Log a debug message that shows up in the Gravity Forms Logging Addon and also the Debug Bar plugin output
411 411
 	     * @param string $message Message to display
@@ -419,7 +419,7 @@  discard block
 block discarded – undo
419 419
      * @param  string $message log message
420 420
      * @return void
421 421
      */
422
-    public static function log_error( $message, $data = null ){
422
+    public static function log_error( $message, $data = null ) {
423 423
 	    /**
424 424
 	     * @action `gravityview_log_error` Log an error message that shows up in the Gravity Forms Logging Addon and also the Debug Bar plugin output
425 425
 	     * @param string $message Error message to display
@@ -430,4 +430,4 @@  discard block
 block discarded – undo
430 430
 
431 431
 } // end class GravityView_Plugin
432 432
 
433
-add_action('plugins_loaded', array('GravityView_Plugin', 'getInstance'), 1);
433
+add_action( 'plugins_loaded', array( 'GravityView_Plugin', 'getInstance' ), 1 );
Please login to merge, or discard this patch.
includes/class-gravityview-gfformsmodel.php 1 patch
Indentation   +127 added lines, -127 removed lines patch added patch discarded remove patch
@@ -24,132 +24,132 @@
 block discarded – undo
24 24
 	}
25 25
 
26 26
 	/**
27
-     * Given information provided in an entry, get array of media IDs
28
-     *
29
-     * This is necessary because GF doesn't expect to need to update post images, only to create them.
30
-     *
31
-     * @see GFFormsModel::create_post()
32
-     *
33
-     * @since 1.17
34
-     *
35
-     * @param array $form Gravity Forms form array
36
-     * @param array $entry Gravity Forms entry array
37
-     *
38
-     * @return array Array of "Field ID" => "Media IDs"
39
-     */
40
-    public static function get_post_field_images( $form, $entry ) {
41
-
42
-        $post_data = self::get_post_fields( $form, $entry );
43
-
44
-        $media = get_attached_media( 'image', $entry['post_id'] );
45
-
46
-        $post_images = array();
47
-
48
-        foreach ( $media as $media_item ) {
49
-            foreach( (array) $post_data['images'] as $post_data_item ) {
50
-                if(
51
-                    rgar( $post_data_item, 'title' ) === $media_item->post_title &&
52
-                    rgar( $post_data_item, 'description' ) === $media_item->post_content &&
53
-                    rgar( $post_data_item, 'caption' ) === $media_item->post_excerpt
54
-                ) {
55
-                    $post_images["{$post_data_item['field_id']}"] = $media_item->ID;
56
-                }
57
-            }
58
-        }
59
-
60
-        return $post_images;
61
-    }
62
-
63
-    /**
64
-     * Alias of GFFormsModel::get_post_fields(); just making it public
65
-     *
66
-     * @see GFFormsModel::get_post_fields()
67
-     *
68
-     * @since 1.17
69
-     *
70
-     * @param array $form Gravity Forms form array
71
-     * @param array $entry Gravity Forms entry array
72
-     *
73
-     * @return array
74
-     */
75
-    public static function get_post_fields( $form, $entry ) {
76
-
77
-        $reflection = new ReflectionMethod( 'GFFormsModel', 'get_post_fields' );
78
-
79
-        /**
80
-         * If the method changes to public, use Gravity Forms' method
81
-         * @todo: If/when the method is public, remove the unneeded copied code.
82
-         */
83
-        if( $reflection->isPublic() ) {
84
-            return parent::get_post_fields( $form, $entry );
85
-        }
86
-
87
-        // It was private; let's make it public
88
-        $reflection->setAccessible( true );
89
-
90
-        return $reflection->invoke( new GFFormsModel, $form, $entry );
91
-    }
92
-
93
-    /**
94
-     * Copied function from Gravity Forms plugin \GFFormsModel::copy_post_image since the method is private.
95
-     *
96
-     * @since 1.16.2
97
-     *
98
-     * @param string $url URL of the post image to update
99
-     * @param int $post_id ID of the post image to update
100
-     * @return array|bool Array with `file`, `url` and `type` keys. False: failed to copy file to final directory path.
101
-     */
102
-    public static function copy_post_image( $url, $post_id ) {
103
-
104
-        $reflection = new ReflectionMethod( 'GFFormsModel', 'copy_post_image' );
105
-
106
-        /**
107
-         * If the method changes to public, use Gravity Forms' method
108
-         * @todo: If/when the method is public, remove the unneeded copied code.
109
-         */
110
-        if( $reflection->isPublic() ) {
111
-            return parent::copy_post_image( $url, $post_id );
112
-        }
113
-
114
-        // It was private; let's make it public
115
-        $reflection->setAccessible( true );
116
-
117
-        return $reflection->invoke( new GFFormsModel, $url, $post_id );
118
-    }
119
-
120
-    /**
121
-     * Copied function from Gravity Forms plugin \GFFormsModel::media_handle_upload since the method is private.
122
-     *
123
-     * Note: The method became public in GF 1.9.17.7
124
-     *
125
-     * @see GFFormsModel::media_handle_upload
126
-     * @see GravityView_Edit_Entry_Render::maybe_update_post_fields
127
-     *
128
-     * @uses copy_post_image
129
-     * @uses wp_insert_attachment
130
-     * @uses wp_update_attachment_metadata
131
-     *
132
-     * @param string $url URL of the post image to update
133
-     * @param int $post_id ID of the post image to update
134
-     * @param array $post_data Array of data for the eventual attachment post type that is created using {@see wp_insert_attachment}. Supports `post_mime_type`, `guid`, `post_parent`, `post_title`, `post_content` keys.
135
-     * @return bool|int ID of attachment Post created. Returns false if file not created by copy_post_image
136
-     */
137
-    public static function media_handle_upload( $url, $post_id, $post_data = array() ) {
138
-
139
-        $reflection = new ReflectionMethod( 'GFFormsModel', 'media_handle_upload' );
140
-
141
-        /**
142
-         * If the method changes to public, use Gravity Forms' method
143
-         * @todo: If/when the method is public, remove the unneeded copied code.
144
-         */
145
-        if( $reflection->isPublic() ) {
146
-            return parent::media_handle_upload( $url, $post_id, $post_data );
147
-        }
148
-
149
-        // It was private; let's make it public
150
-        $reflection->setAccessible( true );
151
-
152
-        return $reflection->invoke( new GFFormsModel, $url, $post_id, $post_data );
153
-    }
27
+	 * Given information provided in an entry, get array of media IDs
28
+	 *
29
+	 * This is necessary because GF doesn't expect to need to update post images, only to create them.
30
+	 *
31
+	 * @see GFFormsModel::create_post()
32
+	 *
33
+	 * @since 1.17
34
+	 *
35
+	 * @param array $form Gravity Forms form array
36
+	 * @param array $entry Gravity Forms entry array
37
+	 *
38
+	 * @return array Array of "Field ID" => "Media IDs"
39
+	 */
40
+	public static function get_post_field_images( $form, $entry ) {
41
+
42
+		$post_data = self::get_post_fields( $form, $entry );
43
+
44
+		$media = get_attached_media( 'image', $entry['post_id'] );
45
+
46
+		$post_images = array();
47
+
48
+		foreach ( $media as $media_item ) {
49
+			foreach( (array) $post_data['images'] as $post_data_item ) {
50
+				if(
51
+					rgar( $post_data_item, 'title' ) === $media_item->post_title &&
52
+					rgar( $post_data_item, 'description' ) === $media_item->post_content &&
53
+					rgar( $post_data_item, 'caption' ) === $media_item->post_excerpt
54
+				) {
55
+					$post_images["{$post_data_item['field_id']}"] = $media_item->ID;
56
+				}
57
+			}
58
+		}
59
+
60
+		return $post_images;
61
+	}
62
+
63
+	/**
64
+	 * Alias of GFFormsModel::get_post_fields(); just making it public
65
+	 *
66
+	 * @see GFFormsModel::get_post_fields()
67
+	 *
68
+	 * @since 1.17
69
+	 *
70
+	 * @param array $form Gravity Forms form array
71
+	 * @param array $entry Gravity Forms entry array
72
+	 *
73
+	 * @return array
74
+	 */
75
+	public static function get_post_fields( $form, $entry ) {
76
+
77
+		$reflection = new ReflectionMethod( 'GFFormsModel', 'get_post_fields' );
78
+
79
+		/**
80
+		 * If the method changes to public, use Gravity Forms' method
81
+		 * @todo: If/when the method is public, remove the unneeded copied code.
82
+		 */
83
+		if( $reflection->isPublic() ) {
84
+			return parent::get_post_fields( $form, $entry );
85
+		}
86
+
87
+		// It was private; let's make it public
88
+		$reflection->setAccessible( true );
89
+
90
+		return $reflection->invoke( new GFFormsModel, $form, $entry );
91
+	}
92
+
93
+	/**
94
+	 * Copied function from Gravity Forms plugin \GFFormsModel::copy_post_image since the method is private.
95
+	 *
96
+	 * @since 1.16.2
97
+	 *
98
+	 * @param string $url URL of the post image to update
99
+	 * @param int $post_id ID of the post image to update
100
+	 * @return array|bool Array with `file`, `url` and `type` keys. False: failed to copy file to final directory path.
101
+	 */
102
+	public static function copy_post_image( $url, $post_id ) {
103
+
104
+		$reflection = new ReflectionMethod( 'GFFormsModel', 'copy_post_image' );
105
+
106
+		/**
107
+		 * If the method changes to public, use Gravity Forms' method
108
+		 * @todo: If/when the method is public, remove the unneeded copied code.
109
+		 */
110
+		if( $reflection->isPublic() ) {
111
+			return parent::copy_post_image( $url, $post_id );
112
+		}
113
+
114
+		// It was private; let's make it public
115
+		$reflection->setAccessible( true );
116
+
117
+		return $reflection->invoke( new GFFormsModel, $url, $post_id );
118
+	}
119
+
120
+	/**
121
+	 * Copied function from Gravity Forms plugin \GFFormsModel::media_handle_upload since the method is private.
122
+	 *
123
+	 * Note: The method became public in GF 1.9.17.7
124
+	 *
125
+	 * @see GFFormsModel::media_handle_upload
126
+	 * @see GravityView_Edit_Entry_Render::maybe_update_post_fields
127
+	 *
128
+	 * @uses copy_post_image
129
+	 * @uses wp_insert_attachment
130
+	 * @uses wp_update_attachment_metadata
131
+	 *
132
+	 * @param string $url URL of the post image to update
133
+	 * @param int $post_id ID of the post image to update
134
+	 * @param array $post_data Array of data for the eventual attachment post type that is created using {@see wp_insert_attachment}. Supports `post_mime_type`, `guid`, `post_parent`, `post_title`, `post_content` keys.
135
+	 * @return bool|int ID of attachment Post created. Returns false if file not created by copy_post_image
136
+	 */
137
+	public static function media_handle_upload( $url, $post_id, $post_data = array() ) {
138
+
139
+		$reflection = new ReflectionMethod( 'GFFormsModel', 'media_handle_upload' );
140
+
141
+		/**
142
+		 * If the method changes to public, use Gravity Forms' method
143
+		 * @todo: If/when the method is public, remove the unneeded copied code.
144
+		 */
145
+		if( $reflection->isPublic() ) {
146
+			return parent::media_handle_upload( $url, $post_id, $post_data );
147
+		}
148
+
149
+		// It was private; let's make it public
150
+		$reflection->setAccessible( true );
151
+
152
+		return $reflection->invoke( new GFFormsModel, $url, $post_id, $post_data );
153
+	}
154 154
 
155 155
 }
156 156
\ No newline at end of file
Please login to merge, or discard this patch.
includes/extensions/edit-entry/class-edit-entry-render.php 2 patches
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -359,9 +359,9 @@  discard block
 block discarded – undo
359 359
 
360 360
 		/**
361 361
 		 * @filter `gravityview/edit_entry/unset_hidden_field_values` Whether to delete values of fields hidden by conditional logic
362
-         * @since 1.22.2
363
-         * @param bool $unset_hidden_field_values Default: true
364
-         * @param GravityView_Edit_Entry_Render $this This object
362
+		 * @since 1.22.2
363
+		 * @param bool $unset_hidden_field_values Default: true
364
+		 * @param GravityView_Edit_Entry_Render $this This object
365 365
 		 */
366 366
 		$unset_hidden_field_values = apply_filters( 'gravityview/edit_entry/unset_hidden_field_values', true, $this );
367 367
 
@@ -369,7 +369,7 @@  discard block
 block discarded – undo
369 369
 			return;
370 370
 		}
371 371
 
372
-        if ( version_compare( GravityView_GFFormsModel::get_database_version(), '2.3-dev-1', '>=' ) && method_exists( 'GFFormsModel', 'get_entry_meta_table_name' ) ) {
372
+		if ( version_compare( GravityView_GFFormsModel::get_database_version(), '2.3-dev-1', '>=' ) && method_exists( 'GFFormsModel', 'get_entry_meta_table_name' ) ) {
373 373
 			$entry_meta_table = GFFormsModel::get_entry_meta_table_name();
374 374
 			$current_fields = $wpdb->get_results( $wpdb->prepare( "SELECT meta_key, meta_value FROM $entry_meta_table WHERE entry_id=%d", $this->entry['id'] ) );
375 375
 		} else {
Please login to merge, or discard this patch.
Spacing   +233 added lines, -233 removed lines patch added patch discarded remove patch
@@ -106,16 +106,16 @@  discard block
 block discarded – undo
106 106
 	function load() {
107 107
 
108 108
 		/** @define "GRAVITYVIEW_DIR" "../../../" */
109
-		include_once( GRAVITYVIEW_DIR .'includes/class-admin-approve-entries.php' );
109
+		include_once( GRAVITYVIEW_DIR . 'includes/class-admin-approve-entries.php' );
110 110
 
111 111
 		// Don't display an embedded form when editing an entry
112 112
 		add_action( 'wp_head', array( $this, 'prevent_render_form' ) );
113 113
 		add_action( 'wp_footer', array( $this, 'prevent_render_form' ) );
114 114
 
115 115
 		// Stop Gravity Forms processing what is ours!
116
-		add_filter( 'wp', array( $this, 'prevent_maybe_process_form'), 8 );
116
+		add_filter( 'wp', array( $this, 'prevent_maybe_process_form' ), 8 );
117 117
 
118
-		add_filter( 'gravityview_is_edit_entry', array( $this, 'is_edit_entry') );
118
+		add_filter( 'gravityview_is_edit_entry', array( $this, 'is_edit_entry' ) );
119 119
 
120 120
 		add_action( 'gravityview_edit_entry', array( $this, 'init' ) );
121 121
 
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
 		add_filter( 'gform_plupload_settings', array( $this, 'modify_fileupload_settings' ), 10, 3 );
127 127
 
128 128
 		// Add fields expected by GFFormDisplay::validate()
129
-		add_filter( 'gform_pre_validation', array( $this, 'gform_pre_validation') );
129
+		add_filter( 'gform_pre_validation', array( $this, 'gform_pre_validation' ) );
130 130
 
131 131
 		// Fix multiselect value for GF 2.2
132 132
 		add_filter( 'gravityview/edit_entry/field_value_multiselect', array( $this, 'fix_multiselect_value_serialization' ), 10, 3 );
@@ -143,8 +143,8 @@  discard block
 block discarded – undo
143 143
 	 * @return void
144 144
 	 */
145 145
 	public function prevent_render_form() {
146
-		if( $this->is_edit_entry() ) {
147
-			if( 'wp_head' === current_filter() ) {
146
+		if ( $this->is_edit_entry() ) {
147
+			if ( 'wp_head' === current_filter() ) {
148 148
 				add_filter( 'gform_shortcode_form', '__return_empty_string' );
149 149
 			} else {
150 150
 				remove_filter( 'gform_shortcode_form', '__return_empty_string' );
@@ -159,13 +159,13 @@  discard block
 block discarded – undo
159 159
 	 */
160 160
 	public function prevent_maybe_process_form() {
161 161
 
162
-		if( ! empty( $_POST ) ) {
162
+		if ( ! empty( $_POST ) ) {
163 163
 			do_action( 'gravityview_log_debug', 'GravityView_Edit_Entry[prevent_maybe_process_form] $_POSTed data (sanitized): ', esc_html( print_r( $_POST, true ) ) );
164 164
 		}
165 165
 
166
-		if( $this->is_edit_entry_submission() ) {
167
-			remove_action( 'wp',  array( 'RGForms', 'maybe_process_form'), 9 );
168
-			remove_action( 'wp',  array( 'GFForms', 'maybe_process_form'), 9 );
166
+		if ( $this->is_edit_entry_submission() ) {
167
+			remove_action( 'wp', array( 'RGForms', 'maybe_process_form' ), 9 );
168
+			remove_action( 'wp', array( 'GFForms', 'maybe_process_form' ), 9 );
169 169
 		}
170 170
 	}
171 171
 
@@ -175,7 +175,7 @@  discard block
 block discarded – undo
175 175
 	 */
176 176
 	public function is_edit_entry() {
177 177
 
178
-		$is_edit_entry = GravityView_frontend::is_single_entry() && ! empty( $_GET['edit'] );
178
+		$is_edit_entry = GravityView_frontend::is_single_entry() && ! empty( $_GET[ 'edit' ] );
179 179
 
180 180
 		return ( $is_edit_entry || $this->is_edit_entry_submission() );
181 181
 	}
@@ -186,7 +186,7 @@  discard block
 block discarded – undo
186 186
 	 * @return boolean
187 187
 	 */
188 188
 	public function is_edit_entry_submission() {
189
-		return !empty( $_POST[ self::$nonce_field ] );
189
+		return ! empty( $_POST[ self::$nonce_field ] );
190 190
 	}
191 191
 
192 192
 	/**
@@ -197,15 +197,15 @@  discard block
 block discarded – undo
197 197
 
198 198
 
199 199
 		$entries = $gravityview_view->getEntries();
200
-		self::$original_entry = $entries[0];
201
-		$this->entry = $entries[0];
200
+		self::$original_entry = $entries[ 0 ];
201
+		$this->entry = $entries[ 0 ];
202 202
 
203 203
 		self::$original_form = $gravityview_view->getForm();
204 204
 		$this->form = $gravityview_view->getForm();
205 205
 		$this->form_id = $gravityview_view->getFormId();
206 206
 		$this->view_id = $gravityview_view->getViewId();
207 207
 
208
-		self::$nonce_key = GravityView_Edit_Entry::get_nonce_key( $this->view_id, $this->form_id, $this->entry['id'] );
208
+		self::$nonce_key = GravityView_Edit_Entry::get_nonce_key( $this->view_id, $this->form_id, $this->entry[ 'id' ] );
209 209
 	}
210 210
 
211 211
 
@@ -226,14 +226,14 @@  discard block
 block discarded – undo
226 226
 
227 227
 		// Multiple Views embedded, don't proceed if nonce fails
228 228
 		$multiple_views = defined( 'GRAVITYVIEW_FUTURE_CORE_LOADED' ) ? gravityview()->views->count() > 1 : $gv_data->has_multiple_views();
229
-		if( $multiple_views && ! wp_verify_nonce( $_GET['edit'], self::$nonce_key ) ) {
230
-			do_action('gravityview_log_error', __METHOD__ . ': Nonce validation failed for the Edit Entry request; returning' );
229
+		if ( $multiple_views && ! wp_verify_nonce( $_GET[ 'edit' ], self::$nonce_key ) ) {
230
+			do_action( 'gravityview_log_error', __METHOD__ . ': Nonce validation failed for the Edit Entry request; returning' );
231 231
 			return;
232 232
 		}
233 233
 
234 234
 		// Sorry, you're not allowed here.
235
-		if( false === $this->user_can_edit_entry( true ) ) {
236
-			do_action('gravityview_log_error', __METHOD__ . ': User is not allowed to edit this entry; returning', $this->entry );
235
+		if ( false === $this->user_can_edit_entry( true ) ) {
236
+			do_action( 'gravityview_log_error', __METHOD__ . ': User is not allowed to edit this entry; returning', $this->entry );
237 237
 			return;
238 238
 		}
239 239
 
@@ -253,9 +253,9 @@  discard block
 block discarded – undo
253 253
 	private function print_scripts() {
254 254
 		$gravityview_view = GravityView_View::getInstance();
255 255
 
256
-		wp_register_script( 'gform_gravityforms', GFCommon::get_base_url().'/js/gravityforms.js', array( 'jquery', 'gform_json', 'gform_placeholder', 'sack', 'plupload-all', 'gravityview-fe-view' ) );
256
+		wp_register_script( 'gform_gravityforms', GFCommon::get_base_url() . '/js/gravityforms.js', array( 'jquery', 'gform_json', 'gform_placeholder', 'sack', 'plupload-all', 'gravityview-fe-view' ) );
257 257
 
258
-		GFFormDisplay::enqueue_form_scripts($gravityview_view->getForm(), false);
258
+		GFFormDisplay::enqueue_form_scripts( $gravityview_view->getForm(), false );
259 259
 
260 260
 		// Sack is required for images
261 261
 		wp_print_scripts( array( 'sack', 'gform_gravityforms' ) );
@@ -267,32 +267,32 @@  discard block
 block discarded – undo
267 267
 	 */
268 268
 	private function process_save() {
269 269
 
270
-		if( empty( $_POST ) || ! isset( $_POST['lid'] ) ) {
270
+		if ( empty( $_POST ) || ! isset( $_POST[ 'lid' ] ) ) {
271 271
 			return;
272 272
 		}
273 273
 
274 274
 		// Make sure the entry, view, and form IDs are all correct
275 275
 		$valid = $this->verify_nonce();
276 276
 
277
-		if( !$valid ) {
278
-			do_action('gravityview_log_error', __METHOD__ . ' Nonce validation failed.' );
277
+		if ( ! $valid ) {
278
+			do_action( 'gravityview_log_error', __METHOD__ . ' Nonce validation failed.' );
279 279
 			return;
280 280
 		}
281 281
 
282
-		if( $this->entry['id'] !== $_POST['lid'] ) {
283
-			do_action('gravityview_log_error', __METHOD__ . ' Entry ID did not match posted entry ID.' );
282
+		if ( $this->entry[ 'id' ] !== $_POST[ 'lid' ] ) {
283
+			do_action( 'gravityview_log_error', __METHOD__ . ' Entry ID did not match posted entry ID.' );
284 284
 			return;
285 285
 		}
286 286
 
287
-		do_action('gravityview_log_debug', __METHOD__ . ': $_POSTed data (sanitized): ', esc_html( print_r( $_POST, true ) ) );
287
+		do_action( 'gravityview_log_debug', __METHOD__ . ': $_POSTed data (sanitized): ', esc_html( print_r( $_POST, true ) ) );
288 288
 
289 289
 		$this->process_save_process_files( $this->form_id );
290 290
 
291 291
 		$this->validate();
292 292
 
293
-		if( $this->is_valid ) {
293
+		if ( $this->is_valid ) {
294 294
 
295
-			do_action('gravityview_log_debug', __METHOD__ . ': Submission is valid.' );
295
+			do_action( 'gravityview_log_debug', __METHOD__ . ': Submission is valid.' );
296 296
 
297 297
 			/**
298 298
 			 * @hack This step is needed to unset the adminOnly from form fields, to add the calculation fields
@@ -302,22 +302,22 @@  discard block
 block discarded – undo
302 302
 			/**
303 303
 			 * @hack to avoid the capability validation of the method save_lead for GF 1.9+
304 304
 			 */
305
-			unset( $_GET['page'] );
305
+			unset( $_GET[ 'page' ] );
306 306
 
307
-			$date_created = $this->entry['date_created'];
307
+			$date_created = $this->entry[ 'date_created' ];
308 308
 
309 309
 			/**
310 310
 			 * @hack to force Gravity Forms to use $read_value_from_post in GFFormsModel::save_lead()
311 311
 			 * @since 1.17.2
312 312
 			 */
313
-			unset( $this->entry['date_created'] );
313
+			unset( $this->entry[ 'date_created' ] );
314 314
 
315 315
 			GFFormsModel::save_lead( $form, $this->entry );
316 316
 
317 317
 			// Delete the values for hidden inputs
318 318
 			$this->unset_hidden_field_values();
319 319
 			
320
-			$this->entry['date_created'] = $date_created;
320
+			$this->entry[ 'date_created' ] = $date_created;
321 321
 
322 322
 			// Process calculation fields
323 323
 			$this->update_calculation_fields();
@@ -337,10 +337,10 @@  discard block
 block discarded – undo
337 337
 			 * @param string $entry_id Numeric ID of the entry that was updated
338 338
 			 * @param GravityView_Edit_Entry_Render $this This object
339 339
 			 */
340
-			do_action( 'gravityview/edit_entry/after_update', $this->form, $this->entry['id'], $this );
340
+			do_action( 'gravityview/edit_entry/after_update', $this->form, $this->entry[ 'id' ], $this );
341 341
 
342 342
 		} else {
343
-			do_action('gravityview_log_error', __METHOD__ . ': Submission is NOT valid.', $this->entry );
343
+			do_action( 'gravityview_log_error', __METHOD__ . ': Submission is NOT valid.', $this->entry );
344 344
 		}
345 345
 
346 346
 	} // process_save
@@ -365,16 +365,16 @@  discard block
 block discarded – undo
365 365
 		 */
366 366
 		$unset_hidden_field_values = apply_filters( 'gravityview/edit_entry/unset_hidden_field_values', true, $this );
367 367
 
368
-		if( ! $unset_hidden_field_values ) {
368
+		if ( ! $unset_hidden_field_values ) {
369 369
 			return;
370 370
 		}
371 371
 
372 372
         if ( version_compare( GravityView_GFFormsModel::get_database_version(), '2.3-dev-1', '>=' ) && method_exists( 'GFFormsModel', 'get_entry_meta_table_name' ) ) {
373 373
 			$entry_meta_table = GFFormsModel::get_entry_meta_table_name();
374
-			$current_fields = $wpdb->get_results( $wpdb->prepare( "SELECT meta_key, meta_value FROM $entry_meta_table WHERE entry_id=%d", $this->entry['id'] ) );
374
+			$current_fields = $wpdb->get_results( $wpdb->prepare( "SELECT meta_key, meta_value FROM $entry_meta_table WHERE entry_id=%d", $this->entry[ 'id' ] ) );
375 375
 		} else {
376 376
 			$lead_detail_table = GFFormsModel::get_lead_details_table_name();
377
-			$current_fields = $wpdb->get_results( $wpdb->prepare( "SELECT id, field_number FROM $lead_detail_table WHERE lead_id=%d", $this->entry['id'] ) );
377
+			$current_fields = $wpdb->get_results( $wpdb->prepare( "SELECT id, field_number FROM $lead_detail_table WHERE lead_id=%d", $this->entry[ 'id' ] ) );
378 378
 		}
379 379
 
380 380
 		foreach ( $this->entry as $input_id => $field_value ) {
@@ -455,7 +455,7 @@  discard block
 block discarded – undo
455 455
 		}
456 456
 
457 457
 		/** No file is being uploaded. */
458
-		if ( empty( $_FILES[ $input_name ]['name'] ) ) {
458
+		if ( empty( $_FILES[ $input_name ][ 'name' ] ) ) {
459 459
 			/** So return the original upload */
460 460
 			return $entry[ $input_id ];
461 461
 		}
@@ -473,11 +473,11 @@  discard block
 block discarded – undo
473 473
 	 * @return mixed
474 474
 	 */
475 475
 	public function modify_fileupload_settings( $plupload_init, $form_id, $instance ) {
476
-		if( ! $this->is_edit_entry() ) {
476
+		if ( ! $this->is_edit_entry() ) {
477 477
 			return $plupload_init;
478 478
 		}
479 479
 
480
-		$plupload_init['gf_vars']['max_files'] = 0;
480
+		$plupload_init[ 'gf_vars' ][ 'max_files' ] = 0;
481 481
 
482 482
 		return $plupload_init;
483 483
 	}
@@ -492,22 +492,22 @@  discard block
 block discarded – undo
492 492
 		$form = $this->form;
493 493
 
494 494
 		/** @var GF_Field $field */
495
-		foreach( $form['fields'] as $k => &$field ) {
495
+		foreach ( $form[ 'fields' ] as $k => &$field ) {
496 496
 
497 497
 			/**
498 498
 			 * Remove the fields with calculation formulas before save to avoid conflicts with GF logic
499 499
 			 * @since 1.16.3
500 500
 			 * @var GF_Field $field
501 501
 			 */
502
-			if( $field->has_calculation() ) {
503
-				unset( $form['fields'][ $k ] );
502
+			if ( $field->has_calculation() ) {
503
+				unset( $form[ 'fields' ][ $k ] );
504 504
 			}
505 505
 
506 506
 			$field->adminOnly = false;
507 507
 
508
-			if( isset( $field->inputs ) && is_array( $field->inputs ) ) {
509
-				foreach( $field->inputs as $key => $input ) {
510
-					$field->inputs[ $key ][ 'id' ] = (string)$input['id'];
508
+			if ( isset( $field->inputs ) && is_array( $field->inputs ) ) {
509
+				foreach ( $field->inputs as $key => $input ) {
510
+					$field->inputs[ $key ][ 'id' ] = (string)$input[ 'id' ];
511 511
 				}
512 512
 			}
513 513
 		}
@@ -521,21 +521,21 @@  discard block
 block discarded – undo
521 521
 		$update = false;
522 522
 
523 523
 		// get the most up to date entry values
524
-		$entry = GFAPI::get_entry( $this->entry['id'] );
524
+		$entry = GFAPI::get_entry( $this->entry[ 'id' ] );
525 525
 
526
-		if( !empty( $this->fields_with_calculation ) ) {
526
+		if ( ! empty( $this->fields_with_calculation ) ) {
527 527
 			$update = true;
528 528
 			foreach ( $this->fields_with_calculation as $calc_field ) {
529 529
 				$inputs = $calc_field->get_entry_inputs();
530 530
 				if ( is_array( $inputs ) ) {
531 531
 					foreach ( $inputs as $input ) {
532
-						$input_name = 'input_' . str_replace( '.', '_', $input['id'] );
532
+						$input_name = 'input_' . str_replace( '.', '_', $input[ 'id' ] );
533 533
 						list( $prefix, $field_id, $input_id ) = rgexplode( '_', $input_name, 3 );
534 534
 
535 535
 						switch ( $input_id ) {
536 536
 							case 1:
537 537
 								/** Never void the labels. */
538
-								$value = $entry[ $input['id'] ];
538
+								$value = $entry[ $input[ 'id' ] ];
539 539
 								break;
540 540
 							case 2:
541 541
 								/** Always recalcualte the final price. */
@@ -543,25 +543,25 @@  discard block
 block discarded – undo
543 543
 								break;
544 544
 							case 3:
545 545
 								/** Fetch the quantity form the request. */
546
-								$value = rgpost( $input_name, $entry[ $input['id'] ] );
546
+								$value = rgpost( $input_name, $entry[ $input[ 'id' ] ] );
547 547
 								break;
548 548
 						}
549 549
 
550
-						$entry[ strval( $input['id'] ) ] = RGFormsModel::prepare_value( $form, $calc_field, $value, $input_name, $entry['id'], $entry );
550
+						$entry[ strval( $input[ 'id' ] ) ] = RGFormsModel::prepare_value( $form, $calc_field, $value, $input_name, $entry[ 'id' ], $entry );
551 551
 					}
552 552
 				} else {
553
-					$input_name = 'input_' . str_replace( '.', '_', $calc_field->id);
554
-					$entry[ strval( $calc_field->id ) ] = RGFormsModel::prepare_value( $form, $calc_field, '', $input_name, $entry['id'], $entry );
553
+					$input_name = 'input_' . str_replace( '.', '_', $calc_field->id );
554
+					$entry[ strval( $calc_field->id ) ] = RGFormsModel::prepare_value( $form, $calc_field, '', $input_name, $entry[ 'id' ], $entry );
555 555
 				}
556 556
 			}
557 557
 
558 558
 		}
559 559
 
560
-		if( $update ) {
560
+		if ( $update ) {
561 561
 
562 562
 			$return_entry = GFAPI::update_entry( $entry );
563 563
 
564
-			if( is_wp_error( $return_entry ) ) {
564
+			if ( is_wp_error( $return_entry ) ) {
565 565
 				do_action( 'gravityview_log_error', 'Updating the entry calculation fields failed', $return_entry );
566 566
 			} else {
567 567
 				do_action( 'gravityview_log_debug', 'Updating the entry calculation fields succeeded' );
@@ -592,18 +592,18 @@  discard block
 block discarded – undo
592 592
 
593 593
 		$input_name = 'input_' . $field_id;
594 594
 
595
-		if ( !empty( $_FILES[ $input_name ]['name'] ) ) {
595
+		if ( ! empty( $_FILES[ $input_name ][ 'name' ] ) ) {
596 596
 
597 597
 			// We have a new image
598 598
 
599
-			$value = RGFormsModel::prepare_value( $form, $field, $value, $input_name, $entry['id'] );
599
+			$value = RGFormsModel::prepare_value( $form, $field, $value, $input_name, $entry[ 'id' ] );
600 600
 
601 601
 			$ary = ! empty( $value ) ? explode( '|:|', $value ) : array();
602 602
 			$img_url = rgar( $ary, 0 );
603 603
 
604
-			$img_title	   = count( $ary ) > 1 ? $ary[1] : '';
605
-			$img_caption	 = count( $ary ) > 2 ? $ary[2] : '';
606
-			$img_description = count( $ary ) > 3 ? $ary[3] : '';
604
+			$img_title	   = count( $ary ) > 1 ? $ary[ 1 ] : '';
605
+			$img_caption	 = count( $ary ) > 2 ? $ary[ 2 ] : '';
606
+			$img_description = count( $ary ) > 3 ? $ary[ 3 ] : '';
607 607
 
608 608
 			$image_meta = array(
609 609
 				'post_excerpt' => $img_caption,
@@ -612,7 +612,7 @@  discard block
 block discarded – undo
612 612
 
613 613
 			//adding title only if it is not empty. It will default to the file name if it is not in the array
614 614
 			if ( ! empty( $img_title ) ) {
615
-				$image_meta['post_title'] = $img_title;
615
+				$image_meta[ 'post_title' ] = $img_title;
616 616
 			}
617 617
 
618 618
 			/**
@@ -626,13 +626,13 @@  discard block
 block discarded – undo
626 626
 				set_post_thumbnail( $post_id, $media_id );
627 627
 			}
628 628
 
629
-		} elseif ( !empty( $_POST[ $input_name ] ) && is_array( $value ) ) {
629
+		} elseif ( ! empty( $_POST[ $input_name ] ) && is_array( $value ) ) {
630 630
 
631 631
 			$img_url = $_POST[ $input_name ];
632 632
 
633
-			$img_title	   = rgar( $_POST, $input_name.'_1' );
634
-			$img_caption	 = rgar( $_POST, $input_name .'_4' );
635
-			$img_description = rgar( $_POST, $input_name .'_7' );
633
+			$img_title	   = rgar( $_POST, $input_name . '_1' );
634
+			$img_caption	 = rgar( $_POST, $input_name . '_4' );
635
+			$img_description = rgar( $_POST, $input_name . '_7' );
636 636
 
637 637
 			$value = ! empty( $img_url ) ? $img_url . "|:|" . $img_title . "|:|" . $img_caption . "|:|" . $img_description : '';
638 638
 
@@ -670,16 +670,16 @@  discard block
 block discarded – undo
670 670
 	 */
671 671
 	private function maybe_update_post_fields( $form ) {
672 672
 
673
-		if( empty( $this->entry['post_id'] ) ) {
673
+		if ( empty( $this->entry[ 'post_id' ] ) ) {
674 674
 			do_action( 'gravityview_log_debug', __METHOD__ . ': This entry has no post fields. Continuing...' );
675 675
 			return;
676 676
 		}
677 677
 
678
-		$post_id = $this->entry['post_id'];
678
+		$post_id = $this->entry[ 'post_id' ];
679 679
 
680 680
 		// Security check
681
-		if( false === GVCommon::has_cap( 'edit_post', $post_id ) ) {
682
-			do_action( 'gravityview_log_error', 'The current user does not have the ability to edit Post #'.$post_id );
681
+		if ( false === GVCommon::has_cap( 'edit_post', $post_id ) ) {
682
+			do_action( 'gravityview_log_error', 'The current user does not have the ability to edit Post #' . $post_id );
683 683
 			return;
684 684
 		}
685 685
 
@@ -691,25 +691,25 @@  discard block
 block discarded – undo
691 691
 
692 692
 			$field = RGFormsModel::get_field( $form, $field_id );
693 693
 
694
-			if( ! $field ) {
694
+			if ( ! $field ) {
695 695
 				continue;
696 696
 			}
697 697
 
698
-			if( GFCommon::is_post_field( $field ) && 'post_category' !== $field->type ) {
698
+			if ( GFCommon::is_post_field( $field ) && 'post_category' !== $field->type ) {
699 699
 
700 700
 				// Get the value of the field, including $_POSTed value
701 701
 				$value = RGFormsModel::get_field_value( $field );
702 702
 
703 703
 				// Use temporary entry variable, to make values available to fill_post_template() and update_post_image()
704 704
 				$entry_tmp = $this->entry;
705
-				$entry_tmp["{$field_id}"] = $value;
705
+				$entry_tmp[ "{$field_id}" ] = $value;
706 706
 
707
-				switch( $field->type ) {
707
+				switch ( $field->type ) {
708 708
 
709 709
 					case 'post_title':
710 710
 						$post_title = $value;
711
-						if( rgar( $form, 'postTitleTemplateEnabled' ) ) {
712
-							$post_title = $this->fill_post_template( $form['postTitleTemplate'], $form, $entry_tmp );
711
+						if ( rgar( $form, 'postTitleTemplateEnabled' ) ) {
712
+							$post_title = $this->fill_post_template( $form[ 'postTitleTemplate' ], $form, $entry_tmp );
713 713
 						}
714 714
 						$updated_post->post_title = $post_title;
715 715
 						$updated_post->post_name  = $post_title;
@@ -718,8 +718,8 @@  discard block
 block discarded – undo
718 718
 
719 719
 					case 'post_content':
720 720
 						$post_content = $value;
721
-						if( rgar( $form, 'postContentTemplateEnabled' ) ) {
722
-							$post_content = $this->fill_post_template( $form['postContentTemplate'], $form, $entry_tmp, true );
721
+						if ( rgar( $form, 'postContentTemplateEnabled' ) ) {
722
+							$post_content = $this->fill_post_template( $form[ 'postContentTemplate' ], $form, $entry_tmp, true );
723 723
 						}
724 724
 						$updated_post->post_content = $post_content;
725 725
 						unset( $post_content );
@@ -737,12 +737,12 @@  discard block
 block discarded – undo
737 737
 							$value = $value[ $field_id ];
738 738
 						}
739 739
 
740
-						if( ! empty( $field->customFieldTemplateEnabled ) ) {
740
+						if ( ! empty( $field->customFieldTemplateEnabled ) ) {
741 741
 							$value = $this->fill_post_template( $field->customFieldTemplate, $form, $entry_tmp, true );
742 742
 						}
743 743
 
744 744
 						if ( $this->is_field_json_encoded( $field ) && ! is_string( $value ) ) {
745
-							$value = function_exists('wp_json_encode') ? wp_json_encode( $value ) : json_encode( $value );
745
+							$value = function_exists( 'wp_json_encode' ) ? wp_json_encode( $value ) : json_encode( $value );
746 746
 						}
747 747
 
748 748
 						update_post_meta( $post_id, $field->postCustomFieldName, $value );
@@ -755,7 +755,7 @@  discard block
 block discarded – undo
755 755
 				}
756 756
 
757 757
 				// update entry after
758
-				$this->entry["{$field_id}"] = $value;
758
+				$this->entry[ "{$field_id}" ] = $value;
759 759
 
760 760
 				$update_entry = true;
761 761
 
@@ -764,25 +764,25 @@  discard block
 block discarded – undo
764 764
 
765 765
 		}
766 766
 
767
-		if( $update_entry ) {
767
+		if ( $update_entry ) {
768 768
 
769 769
 			$return_entry = GFAPI::update_entry( $this->entry );
770 770
 
771
-			if( is_wp_error( $return_entry ) ) {
771
+			if ( is_wp_error( $return_entry ) ) {
772 772
 			   do_action( 'gravityview_log_error', 'Updating the entry post fields failed', array( '$this->entry' => $this->entry, '$return_entry' => $return_entry ) );
773 773
 			} else {
774
-				do_action( 'gravityview_log_debug', 'Updating the entry post fields for post #'.$post_id.' succeeded' );
774
+				do_action( 'gravityview_log_debug', 'Updating the entry post fields for post #' . $post_id . ' succeeded' );
775 775
 			}
776 776
 
777 777
 		}
778 778
 
779 779
 		$return_post = wp_update_post( $updated_post, true );
780 780
 
781
-		if( is_wp_error( $return_post ) ) {
781
+		if ( is_wp_error( $return_post ) ) {
782 782
 			$return_post->add_data( $updated_post, '$updated_post' );
783 783
 			do_action( 'gravityview_log_error', 'Updating the post content failed', compact( 'updated_post', 'return_post' ) );
784 784
 		} else {
785
-			do_action( 'gravityview_log_debug', 'Updating the post content for post #'.$post_id.' succeeded', $updated_post );
785
+			do_action( 'gravityview_log_debug', 'Updating the post content for post #' . $post_id . ' succeeded', $updated_post );
786 786
 		}
787 787
 	}
788 788
 
@@ -800,7 +800,7 @@  discard block
 block discarded – undo
800 800
 		$input_type = RGFormsModel::get_input_type( $field );
801 801
 
802 802
 		// Only certain custom field types are supported
803
-		switch( $input_type ) {
803
+		switch ( $input_type ) {
804 804
 			case 'fileupload':
805 805
 			case 'list':
806 806
 			case 'multiselect':
@@ -837,7 +837,7 @@  discard block
 block discarded – undo
837 837
 		$output = GFCommon::replace_variables( $output, $form, $entry, false, false, false );
838 838
 
839 839
 		// replace conditional shortcodes
840
-		if( $do_shortcode ) {
840
+		if ( $do_shortcode ) {
841 841
 			$output = do_shortcode( $output );
842 842
 		}
843 843
 
@@ -856,19 +856,19 @@  discard block
 block discarded – undo
856 856
 	 */
857 857
 	private function after_update() {
858 858
 
859
-		do_action( 'gform_after_update_entry', $this->form, $this->entry['id'], self::$original_entry );
860
-		do_action( "gform_after_update_entry_{$this->form['id']}", $this->form, $this->entry['id'], self::$original_entry );
859
+		do_action( 'gform_after_update_entry', $this->form, $this->entry[ 'id' ], self::$original_entry );
860
+		do_action( "gform_after_update_entry_{$this->form[ 'id' ]}", $this->form, $this->entry[ 'id' ], self::$original_entry );
861 861
 
862 862
 		// Re-define the entry now that we've updated it.
863
-		$entry = RGFormsModel::get_lead( $this->entry['id'] );
863
+		$entry = RGFormsModel::get_lead( $this->entry[ 'id' ] );
864 864
 
865 865
 		$entry = GFFormsModel::set_entry_meta( $entry, $this->form );
866 866
 
867 867
 		if ( version_compare( GravityView_GFFormsModel::get_database_version(), '2.3-dev-1', '<' ) ) {
868 868
 			// We need to clear the cache because Gravity Forms caches the field values, which
869 869
 			// we have just updated.
870
-			foreach ($this->form['fields'] as $key => $field) {
871
-				GFFormsModel::refresh_lead_field_value( $entry['id'], $field->id );
870
+			foreach ( $this->form[ 'fields' ] as $key => $field ) {
871
+				GFFormsModel::refresh_lead_field_value( $entry[ 'id' ], $field->id );
872 872
 			}
873 873
 		}
874 874
 
@@ -887,7 +887,7 @@  discard block
 block discarded – undo
887 887
 
888 888
 		<div class="gv-edit-entry-wrapper"><?php
889 889
 
890
-			$javascript = gravityview_ob_include( GravityView_Edit_Entry::$file .'/partials/inline-javascript.php', $this );
890
+			$javascript = gravityview_ob_include( GravityView_Edit_Entry::$file . '/partials/inline-javascript.php', $this );
891 891
 
892 892
 			/**
893 893
 			 * Fixes weird wpautop() issue
@@ -903,7 +903,7 @@  discard block
 block discarded – undo
903 903
 					 * @param string $edit_entry_title Modify the "Edit Entry" title
904 904
 					 * @param GravityView_Edit_Entry_Render $this This object
905 905
 					 */
906
-					$edit_entry_title = apply_filters('gravityview_edit_entry_title', __('Edit Entry', 'gravityview'), $this );
906
+					$edit_entry_title = apply_filters( 'gravityview_edit_entry_title', __( 'Edit Entry', 'gravityview' ), $this );
907 907
 
908 908
 					echo esc_attr( $edit_entry_title );
909 909
 			?></span>
@@ -949,20 +949,20 @@  discard block
 block discarded – undo
949 949
 	 */
950 950
 	private function maybe_print_message() {
951 951
 
952
-		if( rgpost('action') === 'update' ) {
952
+		if ( rgpost( 'action' ) === 'update' ) {
953 953
 
954 954
 			$back_link = esc_url( remove_query_arg( array( 'page', 'view', 'edit' ) ) );
955 955
 
956
-			if( ! $this->is_valid ){
956
+			if ( ! $this->is_valid ) {
957 957
 
958 958
 				// Keeping this compatible with Gravity Forms.
959
-				$validation_message = "<div class='validation_error'>" . __('There was a problem with your submission.', 'gravityview') . " " . __('Errors have been highlighted below.', 'gravityview') . "</div>";
960
-				$message = apply_filters("gform_validation_message_{$this->form['id']}", apply_filters("gform_validation_message", $validation_message, $this->form), $this->form);
959
+				$validation_message = "<div class='validation_error'>" . __( 'There was a problem with your submission.', 'gravityview' ) . " " . __( 'Errors have been highlighted below.', 'gravityview' ) . "</div>";
960
+				$message = apply_filters( "gform_validation_message_{$this->form[ 'id' ]}", apply_filters( "gform_validation_message", $validation_message, $this->form ), $this->form );
961 961
 
962
-				echo GVCommon::generate_notice( $message , 'gv-error' );
962
+				echo GVCommon::generate_notice( $message, 'gv-error' );
963 963
 
964 964
 			} else {
965
-				$entry_updated_message = sprintf( esc_attr__('Entry Updated. %sReturn to Entry%s', 'gravityview'), '<a href="'. $back_link .'">', '</a>' );
965
+				$entry_updated_message = sprintf( esc_attr__( 'Entry Updated. %sReturn to Entry%s', 'gravityview' ), '<a href="' . $back_link . '">', '</a>' );
966 966
 
967 967
 				/**
968 968
 				 * @filter `gravityview/edit_entry/success` Modify the edit entry success message (including the anchor link)
@@ -972,7 +972,7 @@  discard block
 block discarded – undo
972 972
 				 * @param array $entry Gravity Forms entry array
973 973
 				 * @param string $back_link URL to return to the original entry. @since 1.6
974 974
 				 */
975
-				$message = apply_filters( 'gravityview/edit_entry/success', $entry_updated_message , $this->view_id, $this->entry, $back_link );
975
+				$message = apply_filters( 'gravityview/edit_entry/success', $entry_updated_message, $this->view_id, $this->entry, $back_link );
976 976
 
977 977
 				echo GVCommon::generate_notice( $message );
978 978
 			}
@@ -996,21 +996,21 @@  discard block
 block discarded – undo
996 996
 		 */
997 997
 		do_action( 'gravityview/edit-entry/render/before', $this );
998 998
 
999
-		add_filter( 'gform_pre_render', array( $this, 'filter_modify_form_fields'), 5000, 3 );
1000
-		add_filter( 'gform_submit_button', array( $this, 'render_form_buttons') );
999
+		add_filter( 'gform_pre_render', array( $this, 'filter_modify_form_fields' ), 5000, 3 );
1000
+		add_filter( 'gform_submit_button', array( $this, 'render_form_buttons' ) );
1001 1001
 		add_filter( 'gform_disable_view_counter', '__return_true' );
1002 1002
 
1003 1003
 		add_filter( 'gform_field_input', array( $this, 'verify_user_can_edit_post' ), 5, 5 );
1004 1004
 		add_filter( 'gform_field_input', array( $this, 'modify_edit_field_input' ), 10, 5 );
1005 1005
 
1006 1006
 		// We need to remove the fake $_GET['page'] arg to avoid rendering form as if in admin.
1007
-		unset( $_GET['page'] );
1007
+		unset( $_GET[ 'page' ] );
1008 1008
 
1009 1009
 		// TODO: Verify multiple-page forms
1010 1010
 
1011 1011
 		ob_start(); // Prevent PHP warnings possibly caused by prefilling list fields for conditional logic
1012 1012
 
1013
-		$html = GFFormDisplay::get_form( $this->form['id'], false, false, true, $this->entry );
1013
+		$html = GFFormDisplay::get_form( $this->form[ 'id' ], false, false, true, $this->entry );
1014 1014
 
1015 1015
 		ob_get_clean();
1016 1016
 
@@ -1036,7 +1036,7 @@  discard block
 block discarded – undo
1036 1036
 	 * @return string
1037 1037
 	 */
1038 1038
 	public function render_form_buttons() {
1039
-		return gravityview_ob_include( GravityView_Edit_Entry::$file .'/partials/form-buttons.php', $this );
1039
+		return gravityview_ob_include( GravityView_Edit_Entry::$file . '/partials/form-buttons.php', $this );
1040 1040
 	}
1041 1041
 
1042 1042
 
@@ -1056,10 +1056,10 @@  discard block
 block discarded – undo
1056 1056
 	public function filter_modify_form_fields( $form, $ajax = false, $field_values = '' ) {
1057 1057
 
1058 1058
 		// In case we have validated the form, use it to inject the validation results into the form render
1059
-		if( isset( $this->form_after_validation ) ) {
1059
+		if ( isset( $this->form_after_validation ) ) {
1060 1060
 			$form = $this->form_after_validation;
1061 1061
 		} else {
1062
-			$form['fields'] = $this->get_configured_edit_fields( $form, $this->view_id );
1062
+			$form[ 'fields' ] = $this->get_configured_edit_fields( $form, $this->view_id );
1063 1063
 		}
1064 1064
 
1065 1065
 		$form = $this->filter_conditional_logic( $form );
@@ -1067,8 +1067,8 @@  discard block
 block discarded – undo
1067 1067
 		$form = $this->prefill_conditional_logic( $form );
1068 1068
 
1069 1069
 		// for now we don't support Save and Continue feature.
1070
-		if( ! self::$supports_save_and_continue ) {
1071
-			unset( $form['save'] );
1070
+		if ( ! self::$supports_save_and_continue ) {
1071
+			unset( $form[ 'save' ] );
1072 1072
 		}
1073 1073
 
1074 1074
 		return $form;
@@ -1089,29 +1089,29 @@  discard block
 block discarded – undo
1089 1089
 	 */
1090 1090
 	public function verify_user_can_edit_post( $field_content = '', $field, $value, $lead_id = 0, $form_id ) {
1091 1091
 
1092
-		if( GFCommon::is_post_field( $field ) ) {
1092
+		if ( GFCommon::is_post_field( $field ) ) {
1093 1093
 
1094 1094
 			$message = null;
1095 1095
 
1096 1096
 			// First, make sure they have the capability to edit the post.
1097
-			if( false === current_user_can( 'edit_post', $this->entry['post_id'] ) ) {
1097
+			if ( false === current_user_can( 'edit_post', $this->entry[ 'post_id' ] ) ) {
1098 1098
 
1099 1099
 				/**
1100 1100
 				 * @filter `gravityview/edit_entry/unsupported_post_field_text` Modify the message when someone isn't able to edit a post
1101 1101
 				 * @param string $message The existing "You don't have permission..." text
1102 1102
 				 */
1103
-				$message = apply_filters('gravityview/edit_entry/unsupported_post_field_text', __('You don&rsquo;t have permission to edit this post.', 'gravityview') );
1103
+				$message = apply_filters( 'gravityview/edit_entry/unsupported_post_field_text', __( 'You don&rsquo;t have permission to edit this post.', 'gravityview' ) );
1104 1104
 
1105
-			} elseif( null === get_post( $this->entry['post_id'] ) ) {
1105
+			} elseif ( null === get_post( $this->entry[ 'post_id' ] ) ) {
1106 1106
 				/**
1107 1107
 				 * @filter `gravityview/edit_entry/no_post_text` Modify the message when someone is editing an entry attached to a post that no longer exists
1108 1108
 				 * @param string $message The existing "This field is not editable; the post no longer exists." text
1109 1109
 				 */
1110
-				$message = apply_filters('gravityview/edit_entry/no_post_text', __('This field is not editable; the post no longer exists.', 'gravityview' ) );
1110
+				$message = apply_filters( 'gravityview/edit_entry/no_post_text', __( 'This field is not editable; the post no longer exists.', 'gravityview' ) );
1111 1111
 			}
1112 1112
 
1113
-			if( $message ) {
1114
-				$field_content = sprintf('<div class="ginput_container ginput_container_' . $field->type . '">%s</div>', wpautop( $message ) );
1113
+			if ( $message ) {
1114
+				$field_content = sprintf( '<div class="ginput_container ginput_container_' . $field->type . '">%s</div>', wpautop( $message ) );
1115 1115
 			}
1116 1116
 		}
1117 1117
 
@@ -1136,8 +1136,8 @@  discard block
 block discarded – undo
1136 1136
 
1137 1137
 		// If the form has been submitted, then we don't need to pre-fill the values,
1138 1138
 		// Except for fileupload type and when a field input is overridden- run always!!
1139
-		if(
1140
-			( $this->is_edit_entry_submission() && !in_array( $field->type, array( 'fileupload', 'post_image' ) ) )
1139
+		if (
1140
+			( $this->is_edit_entry_submission() && ! in_array( $field->type, array( 'fileupload', 'post_image' ) ) )
1141 1141
 			&& false === ( $gv_field && is_callable( array( $gv_field, 'get_field_input' ) ) )
1142 1142
 			&& ! GFCommon::is_product_field( $field->type )
1143 1143
 			|| ! empty( $field_content )
@@ -1157,7 +1157,7 @@  discard block
 block discarded – undo
1157 1157
 		$return = null;
1158 1158
 
1159 1159
 		/** @var GravityView_Field $gv_field */
1160
-		if( $gv_field && is_callable( array( $gv_field, 'get_field_input' ) ) ) {
1160
+		if ( $gv_field && is_callable( array( $gv_field, 'get_field_input' ) ) ) {
1161 1161
 			$return = $gv_field->get_field_input( $this->form, $field_value, $this->entry, $field );
1162 1162
 		} else {
1163 1163
 			$return = $field->get_field_input( $this->form, $field_value, $this->entry );
@@ -1166,7 +1166,7 @@  discard block
 block discarded – undo
1166 1166
 		// If there was output, it's an error
1167 1167
 		$warnings = ob_get_clean();
1168 1168
 
1169
-		if( !empty( $warnings ) ) {
1169
+		if ( ! empty( $warnings ) ) {
1170 1170
 			do_action( 'gravityview_log_error', __METHOD__ . $warnings, $field_value );
1171 1171
 		}
1172 1172
 
@@ -1191,7 +1191,7 @@  discard block
 block discarded – undo
1191 1191
 		$override_saved_value = apply_filters( 'gravityview/edit_entry/pre_populate/override', false, $field );
1192 1192
 
1193 1193
 		// We're dealing with multiple inputs (e.g. checkbox) but not time or date (as it doesn't store data in input IDs)
1194
-		if( isset( $field->inputs ) && is_array( $field->inputs ) && !in_array( $field->type, array( 'time', 'date' ) ) ) {
1194
+		if ( isset( $field->inputs ) && is_array( $field->inputs ) && ! in_array( $field->type, array( 'time', 'date' ) ) ) {
1195 1195
 
1196 1196
 			$field_value = array();
1197 1197
 
@@ -1200,10 +1200,10 @@  discard block
 block discarded – undo
1200 1200
 
1201 1201
 			foreach ( (array)$field->inputs as $input ) {
1202 1202
 
1203
-				$input_id = strval( $input['id'] );
1203
+				$input_id = strval( $input[ 'id' ] );
1204 1204
 				
1205 1205
 				if ( isset( $this->entry[ $input_id ] ) && ! gv_empty( $this->entry[ $input_id ], false, false ) ) {
1206
-					$field_value[ $input_id ] =  'post_category' === $field->type ? GFCommon::format_post_category( $this->entry[ $input_id ], true ) : $this->entry[ $input_id ];
1206
+					$field_value[ $input_id ] = 'post_category' === $field->type ? GFCommon::format_post_category( $this->entry[ $input_id ], true ) : $this->entry[ $input_id ];
1207 1207
 					$allow_pre_populated = false;
1208 1208
 				}
1209 1209
 
@@ -1211,7 +1211,7 @@  discard block
 block discarded – undo
1211 1211
 
1212 1212
 			$pre_value = $field->get_value_submission( array(), false );
1213 1213
 
1214
-			$field_value = ! $allow_pre_populated && ! ( $override_saved_value && !gv_empty( $pre_value, false, false ) ) ? $field_value : $pre_value;
1214
+			$field_value = ! $allow_pre_populated && ! ( $override_saved_value && ! gv_empty( $pre_value, false, false ) ) ? $field_value : $pre_value;
1215 1215
 
1216 1216
 		} else {
1217 1217
 
@@ -1222,13 +1222,13 @@  discard block
 block discarded – undo
1222 1222
 
1223 1223
 			// saved field entry value (if empty, fallback to the pre-populated value, if exists)
1224 1224
 			// or pre-populated value if not empty and set to override saved value
1225
-			$field_value = !gv_empty( $this->entry[ $id ], false, false ) && ! ( $override_saved_value && !gv_empty( $pre_value, false, false ) ) ? $this->entry[ $id ] : $pre_value;
1225
+			$field_value = ! gv_empty( $this->entry[ $id ], false, false ) && ! ( $override_saved_value && ! gv_empty( $pre_value, false, false ) ) ? $this->entry[ $id ] : $pre_value;
1226 1226
 
1227 1227
 			// in case field is post_category but inputType is select, multi-select or radio, convert value into array of category IDs.
1228
-			if ( 'post_category' === $field->type && !gv_empty( $field_value, false, false ) ) {
1228
+			if ( 'post_category' === $field->type && ! gv_empty( $field_value, false, false ) ) {
1229 1229
 				$categories = array();
1230 1230
 				foreach ( explode( ',', $field_value ) as $cat_string ) {
1231
-					$categories[] = GFCommon::format_post_category( $cat_string, true );
1231
+					$categories[ ] = GFCommon::format_post_category( $cat_string, true );
1232 1232
 				}
1233 1233
 				$field_value = 'multiselect' === $field->get_input_type() ? $categories : implode( '', $categories );
1234 1234
 			}
@@ -1256,7 +1256,7 @@  discard block
 block discarded – undo
1256 1256
 		 * @param GF_Field $field Gravity Forms field object
1257 1257
 		 * @param GravityView_Edit_Entry_Render $this Current object
1258 1258
 		 */
1259
-		$field_value = apply_filters( 'gravityview/edit_entry/field_value_' . $field->type , $field_value, $field, $this );
1259
+		$field_value = apply_filters( 'gravityview/edit_entry/field_value_' . $field->type, $field_value, $field, $this );
1260 1260
 
1261 1261
 		return $field_value;
1262 1262
 	}
@@ -1273,12 +1273,12 @@  discard block
 block discarded – undo
1273 1273
 	 */
1274 1274
 	public function gform_pre_validation( $form ) {
1275 1275
 
1276
-		if( ! $this->verify_nonce() ) {
1276
+		if ( ! $this->verify_nonce() ) {
1277 1277
 			return $form;
1278 1278
 		}
1279 1279
 
1280 1280
 		// Fix PHP warning regarding undefined index.
1281
-		foreach ( $form['fields'] as &$field) {
1281
+		foreach ( $form[ 'fields' ] as &$field ) {
1282 1282
 
1283 1283
 			// This is because we're doing admin form pretending to be front-end, so Gravity Forms
1284 1284
 			// expects certain field array items to be set.
@@ -1286,7 +1286,7 @@  discard block
 block discarded – undo
1286 1286
 				$field->{$key} = isset( $field->{$key} ) ? $field->{$key} : NULL;
1287 1287
 			}
1288 1288
 
1289
-			switch( RGFormsModel::get_input_type( $field ) ) {
1289
+			switch ( RGFormsModel::get_input_type( $field ) ) {
1290 1290
 
1291 1291
 				/**
1292 1292
 				 * this whole fileupload hack is because in the admin, Gravity Forms simply doesn't update any fileupload field if it's empty, but it DOES in the frontend.
@@ -1300,37 +1300,37 @@  discard block
 block discarded – undo
1300 1300
 					// Set the previous value
1301 1301
 					$entry = $this->get_entry();
1302 1302
 
1303
-					$input_name = 'input_'.$field->id;
1304
-					$form_id = $form['id'];
1303
+					$input_name = 'input_' . $field->id;
1304
+					$form_id = $form[ 'id' ];
1305 1305
 
1306 1306
 					$value = NULL;
1307 1307
 
1308 1308
 					// Use the previous entry value as the default.
1309
-					if( isset( $entry[ $field->id ] ) ) {
1309
+					if ( isset( $entry[ $field->id ] ) ) {
1310 1310
 						$value = $entry[ $field->id ];
1311 1311
 					}
1312 1312
 
1313 1313
 					// If this is a single upload file
1314
-					if( !empty( $_FILES[ $input_name ] ) && !empty( $_FILES[ $input_name ]['name'] ) ) {
1315
-						$file_path = GFFormsModel::get_file_upload_path( $form['id'], $_FILES[ $input_name ]['name'] );
1316
-						$value = $file_path['url'];
1314
+					if ( ! empty( $_FILES[ $input_name ] ) && ! empty( $_FILES[ $input_name ][ 'name' ] ) ) {
1315
+						$file_path = GFFormsModel::get_file_upload_path( $form[ 'id' ], $_FILES[ $input_name ][ 'name' ] );
1316
+						$value = $file_path[ 'url' ];
1317 1317
 
1318 1318
 					} else {
1319 1319
 
1320 1320
 						// Fix PHP warning on line 1498 of form_display.php for post_image fields
1321 1321
 						// Fix PHP Notice:  Undefined index:  size in form_display.php on line 1511
1322
-						$_FILES[ $input_name ] = array('name' => '', 'size' => '' );
1322
+						$_FILES[ $input_name ] = array( 'name' => '', 'size' => '' );
1323 1323
 
1324 1324
 					}
1325 1325
 
1326
-					if( rgar($field, "multipleFiles") ) {
1326
+					if ( rgar( $field, "multipleFiles" ) ) {
1327 1327
 
1328 1328
 						// If there are fresh uploads, process and merge them.
1329 1329
 						// Otherwise, use the passed values, which should be json-encoded array of URLs
1330
-						if( isset( GFFormsModel::$uploaded_files[$form_id][$input_name] ) ) {
1330
+						if ( isset( GFFormsModel::$uploaded_files[ $form_id ][ $input_name ] ) ) {
1331 1331
 							$value = empty( $value ) ? '[]' : $value;
1332 1332
 							$value = stripslashes_deep( $value );
1333
-							$value = GFFormsModel::prepare_value( $form, $field, $value, $input_name, $entry['id'], array());
1333
+							$value = GFFormsModel::prepare_value( $form, $field, $value, $input_name, $entry[ 'id' ], array() );
1334 1334
 						}
1335 1335
 
1336 1336
 					} else {
@@ -1348,8 +1348,8 @@  discard block
 block discarded – undo
1348 1348
 
1349 1349
 				case 'number':
1350 1350
 					// Fix "undefined index" issue at line 1286 in form_display.php
1351
-					if( !isset( $_POST['input_'.$field->id ] ) ) {
1352
-						$_POST['input_'.$field->id ] = NULL;
1351
+					if ( ! isset( $_POST[ 'input_' . $field->id ] ) ) {
1352
+						$_POST[ 'input_' . $field->id ] = NULL;
1353 1353
 					}
1354 1354
 					break;
1355 1355
 			}
@@ -1391,7 +1391,7 @@  discard block
 block discarded – undo
1391 1391
 		 * You can enter whatever you want!
1392 1392
 		 * We try validating, and customize the results using `self::custom_validation()`
1393 1393
 		 */
1394
-		add_filter( 'gform_validation_'. $this->form_id, array( $this, 'custom_validation' ), 10, 4);
1394
+		add_filter( 'gform_validation_' . $this->form_id, array( $this, 'custom_validation' ), 10, 4 );
1395 1395
 
1396 1396
 		// Needed by the validate funtion
1397 1397
 		$failed_validation_page = NULL;
@@ -1399,14 +1399,14 @@  discard block
 block discarded – undo
1399 1399
 
1400 1400
 		// Prevent entry limit from running when editing an entry, also
1401 1401
 		// prevent form scheduling from preventing editing
1402
-		unset( $this->form['limitEntries'], $this->form['scheduleForm'] );
1402
+		unset( $this->form[ 'limitEntries' ], $this->form[ 'scheduleForm' ] );
1403 1403
 
1404 1404
 		// Hide fields depending on Edit Entry settings
1405
-		$this->form['fields'] = $this->get_configured_edit_fields( $this->form, $this->view_id );
1405
+		$this->form[ 'fields' ] = $this->get_configured_edit_fields( $this->form, $this->view_id );
1406 1406
 
1407 1407
 		$this->is_valid = GFFormDisplay::validate( $this->form, $field_values, 1, $failed_validation_page );
1408 1408
 
1409
-		remove_filter( 'gform_validation_'. $this->form_id, array( $this, 'custom_validation' ), 10 );
1409
+		remove_filter( 'gform_validation_' . $this->form_id, array( $this, 'custom_validation' ), 10 );
1410 1410
 		remove_filter( 'gform_validation_' . $this->form_id, array( $this, 'user_registration_validation' ), 10 );
1411 1411
 	}
1412 1412
 
@@ -1423,7 +1423,7 @@  discard block
 block discarded – undo
1423 1423
 		$user_registration = GF_User_Registration::get_instance();
1424 1424
 
1425 1425
 		$entry = $this->get_entry();
1426
-		$form = $validation_results['form'];
1426
+		$form = $validation_results[ 'form' ];
1427 1427
 
1428 1428
 		if ( ! $feed = $user_registration->get_single_submission_feed( $entry, $form ) ) {
1429 1429
 			return $validation_results;
@@ -1439,18 +1439,18 @@  discard block
 block discarded – undo
1439 1439
 		if ( $user_email != $value && email_exists( $value ) ) {
1440 1440
 			$email_field->failed_validation = 1;
1441 1441
 			$email_field->validation_message = __( 'This email is already in use', 'gravityview' );
1442
-			$validation_results['is_valid'] = false;
1442
+			$validation_results[ 'is_valid' ] = false;
1443 1443
 		}
1444 1444
 
1445 1445
 		$value = RGFormsModel::get_field_value( $username_field );
1446 1446
 		if ( $username != $value ) {
1447 1447
 			$username_field->failed_validation = 1;
1448 1448
 			$username_field->validation_message = __( 'Usernames cannot be changed', 'gravityview' );
1449
-			$validation_results['is_valid'] = false;
1449
+			$validation_results[ 'is_valid' ] = false;
1450 1450
 		}
1451 1451
 
1452 1452
 		// We'll need this result when rendering the form ( on GFFormDisplay::get_form )
1453
-		$this->form_after_validation = $validation_results['form'];
1453
+		$this->form_after_validation = $validation_results[ 'form' ];
1454 1454
 
1455 1455
 		return $validation_results;
1456 1456
 	}
@@ -1468,13 +1468,13 @@  discard block
 block discarded – undo
1468 1468
 	 */
1469 1469
 	public function custom_validation( $validation_results ) {
1470 1470
 
1471
-		do_action('gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Validation results: ', $validation_results );
1471
+		do_action( 'gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Validation results: ', $validation_results );
1472 1472
 
1473
-		do_action('gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] $_POSTed data (sanitized): ', esc_html( print_r( $_POST, true ) ) );
1473
+		do_action( 'gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] $_POSTed data (sanitized): ', esc_html( print_r( $_POST, true ) ) );
1474 1474
 
1475 1475
 		$gv_valid = true;
1476 1476
 
1477
-		foreach ( $validation_results['form']['fields'] as $key => &$field ) {
1477
+		foreach ( $validation_results[ 'form' ][ 'fields' ] as $key => &$field ) {
1478 1478
 
1479 1479
 			$value = RGFormsModel::get_field_value( $field );
1480 1480
 			$field_type = RGFormsModel::get_input_type( $field );
@@ -1487,35 +1487,35 @@  discard block
 block discarded – undo
1487 1487
 				case 'post_image':
1488 1488
 
1489 1489
 					// in case nothing is uploaded but there are already files saved
1490
-					if( !empty( $field->failed_validation ) && !empty( $field->isRequired ) && !empty( $value ) ) {
1490
+					if ( ! empty( $field->failed_validation ) && ! empty( $field->isRequired ) && ! empty( $value ) ) {
1491 1491
 						$field->failed_validation = false;
1492 1492
 						unset( $field->validation_message );
1493 1493
 					}
1494 1494
 
1495 1495
 					// validate if multi file upload reached max number of files [maxFiles] => 2
1496
-					if( rgobj( $field, 'maxFiles') && rgobj( $field, 'multipleFiles') ) {
1496
+					if ( rgobj( $field, 'maxFiles' ) && rgobj( $field, 'multipleFiles' ) ) {
1497 1497
 
1498 1498
 						$input_name = 'input_' . $field->id;
1499 1499
 						//uploaded
1500
-						$file_names = isset( GFFormsModel::$uploaded_files[ $validation_results['form']['id'] ][ $input_name ] ) ? GFFormsModel::$uploaded_files[ $validation_results['form']['id'] ][ $input_name ] : array();
1500
+						$file_names = isset( GFFormsModel::$uploaded_files[ $validation_results[ 'form' ][ 'id' ] ][ $input_name ] ) ? GFFormsModel::$uploaded_files[ $validation_results[ 'form' ][ 'id' ] ][ $input_name ] : array();
1501 1501
 
1502 1502
 						//existent
1503 1503
 						$entry = $this->get_entry();
1504 1504
 						$value = NULL;
1505
-						if( isset( $entry[ $field->id ] ) ) {
1505
+						if ( isset( $entry[ $field->id ] ) ) {
1506 1506
 							$value = json_decode( $entry[ $field->id ], true );
1507 1507
 						}
1508 1508
 
1509 1509
 						// count uploaded files and existent entry files
1510 1510
 						$count_files = count( $file_names ) + count( $value );
1511 1511
 
1512
-						if( $count_files > $field->maxFiles ) {
1512
+						if ( $count_files > $field->maxFiles ) {
1513 1513
 							$field->validation_message = __( 'Maximum number of files reached', 'gravityview' );
1514 1514
 							$field->failed_validation = 1;
1515 1515
 							$gv_valid = false;
1516 1516
 
1517 1517
 							// in case of error make sure the newest upload files are removed from the upload input
1518
-							GFFormsModel::$uploaded_files[ $validation_results['form']['id'] ] = null;
1518
+							GFFormsModel::$uploaded_files[ $validation_results[ 'form' ][ 'id' ] ] = null;
1519 1519
 						}
1520 1520
 
1521 1521
 					}
@@ -1526,7 +1526,7 @@  discard block
 block discarded – undo
1526 1526
 			}
1527 1527
 
1528 1528
 			// This field has failed validation.
1529
-			if( !empty( $field->failed_validation ) ) {
1529
+			if ( ! empty( $field->failed_validation ) ) {
1530 1530
 
1531 1531
 				do_action( 'gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Field is invalid.', array( 'field' => $field, 'value' => $value ) );
1532 1532
 
@@ -1544,32 +1544,32 @@  discard block
 block discarded – undo
1544 1544
 				}
1545 1545
 
1546 1546
 				// You can't continue inside a switch, so we do it after.
1547
-				if( empty( $field->failed_validation ) ) {
1547
+				if ( empty( $field->failed_validation ) ) {
1548 1548
 					continue;
1549 1549
 				}
1550 1550
 
1551 1551
 				// checks if the No Duplicates option is not validating entry against itself, since
1552 1552
 				// we're editing a stored entry, it would also assume it's a duplicate.
1553
-				if( !empty( $field->noDuplicates ) ) {
1553
+				if ( ! empty( $field->noDuplicates ) ) {
1554 1554
 
1555 1555
 					$entry = $this->get_entry();
1556 1556
 
1557 1557
 					// If the value of the entry is the same as the stored value
1558 1558
 					// Then we can assume it's not a duplicate, it's the same.
1559
-					if( !empty( $entry ) && $value == $entry[ $field->id ] ) {
1559
+					if ( ! empty( $entry ) && $value == $entry[ $field->id ] ) {
1560 1560
 						//if value submitted was not changed, then don't validate
1561 1561
 						$field->failed_validation = false;
1562 1562
 
1563 1563
 						unset( $field->validation_message );
1564 1564
 
1565
-						do_action('gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Field not a duplicate; it is the same entry.', $entry );
1565
+						do_action( 'gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Field not a duplicate; it is the same entry.', $entry );
1566 1566
 
1567 1567
 						continue;
1568 1568
 					}
1569 1569
 				}
1570 1570
 
1571 1571
 				// if here then probably we are facing the validation 'At least one field must be filled out'
1572
-				if( GFFormDisplay::is_empty( $field, $this->form_id  ) && empty( $field->isRequired ) ) {
1572
+				if ( GFFormDisplay::is_empty( $field, $this->form_id ) && empty( $field->isRequired ) ) {
1573 1573
 					unset( $field->validation_message );
1574 1574
 					$field->validation_message = false;
1575 1575
 					continue;
@@ -1581,12 +1581,12 @@  discard block
 block discarded – undo
1581 1581
 
1582 1582
 		}
1583 1583
 
1584
-		$validation_results['is_valid'] = $gv_valid;
1584
+		$validation_results[ 'is_valid' ] = $gv_valid;
1585 1585
 
1586
-		do_action('gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Validation results.', $validation_results );
1586
+		do_action( 'gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Validation results.', $validation_results );
1587 1587
 
1588 1588
 		// We'll need this result when rendering the form ( on GFFormDisplay::get_form )
1589
-		$this->form_after_validation = $validation_results['form'];
1589
+		$this->form_after_validation = $validation_results[ 'form' ];
1590 1590
 
1591 1591
 		return $validation_results;
1592 1592
 	}
@@ -1599,7 +1599,7 @@  discard block
 block discarded – undo
1599 1599
 	 */
1600 1600
 	public function get_entry() {
1601 1601
 
1602
-		if( empty( $this->entry ) ) {
1602
+		if ( empty( $this->entry ) ) {
1603 1603
 			// Get the database value of the entry that's being edited
1604 1604
 			$this->entry = gravityview_get_entry( GravityView_frontend::is_single_entry() );
1605 1605
 		}
@@ -1634,10 +1634,10 @@  discard block
 block discarded – undo
1634 1634
 		}
1635 1635
 
1636 1636
 		// If edit tab not yet configured, show all fields
1637
-		$edit_fields = !empty( $properties['edit_edit-fields'] ) ? $properties['edit_edit-fields'] : NULL;
1637
+		$edit_fields = ! empty( $properties[ 'edit_edit-fields' ] ) ? $properties[ 'edit_edit-fields' ] : NULL;
1638 1638
 
1639 1639
 		// Hide fields depending on admin settings
1640
-		$fields = $this->filter_fields( $form['fields'], $edit_fields );
1640
+		$fields = $this->filter_fields( $form[ 'fields' ], $edit_fields );
1641 1641
 
1642 1642
 		// If Edit Entry fields are configured, remove adminOnly field settings. Otherwise, don't.
1643 1643
 		$fields = $this->filter_admin_only_fields( $fields, $edit_fields, $form, $view_id );
@@ -1669,7 +1669,7 @@  discard block
 block discarded – undo
1669 1669
 	 */
1670 1670
 	private function filter_fields( $fields, $configured_fields ) {
1671 1671
 
1672
-		if( empty( $fields ) || !is_array( $fields ) ) {
1672
+		if ( empty( $fields ) || ! is_array( $fields ) ) {
1673 1673
 			return $fields;
1674 1674
 		}
1675 1675
 
@@ -1682,18 +1682,18 @@  discard block
 block discarded – undo
1682 1682
 
1683 1683
 			// Remove the fields that have calculation properties and keep them to be used later
1684 1684
 			// @since 1.16.2
1685
-			if( $field->has_calculation() ) {
1686
-				$this->fields_with_calculation[] = $field;
1685
+			if ( $field->has_calculation() ) {
1686
+				$this->fields_with_calculation[ ] = $field;
1687 1687
 				// don't remove the calculation fields on form render.
1688 1688
 			}
1689 1689
 
1690
-			if( in_array( $field->type, $field_type_blacklist ) ) {
1690
+			if ( in_array( $field->type, $field_type_blacklist ) ) {
1691 1691
 				unset( $fields[ $key ] );
1692 1692
 			}
1693 1693
 		}
1694 1694
 
1695 1695
 		// The Edit tab has not been configured, so we return all fields by default.
1696
-		if( empty( $configured_fields ) ) {
1696
+		if ( empty( $configured_fields ) ) {
1697 1697
 			return $fields;
1698 1698
 		}
1699 1699
 
@@ -1703,8 +1703,8 @@  discard block
 block discarded – undo
1703 1703
 			/** @var GF_Field $field */
1704 1704
 			foreach ( $fields as $field ) {
1705 1705
 
1706
-				if( intval( $configured_field['id'] ) === intval( $field->id ) && $this->user_can_edit_field( $configured_field, false ) ) {
1707
-					$edit_fields[] = $this->merge_field_properties( $field, $configured_field );
1706
+				if ( intval( $configured_field[ 'id' ] ) === intval( $field->id ) && $this->user_can_edit_field( $configured_field, false ) ) {
1707
+					$edit_fields[ ] = $this->merge_field_properties( $field, $configured_field );
1708 1708
 					break;
1709 1709
 				}
1710 1710
 
@@ -1727,14 +1727,14 @@  discard block
 block discarded – undo
1727 1727
 
1728 1728
 		$return_field = $field;
1729 1729
 
1730
-		if( empty( $field_setting['show_label'] ) ) {
1730
+		if ( empty( $field_setting[ 'show_label' ] ) ) {
1731 1731
 			$return_field->label = '';
1732
-		} elseif ( !empty( $field_setting['custom_label'] ) ) {
1733
-			$return_field->label = $field_setting['custom_label'];
1732
+		} elseif ( ! empty( $field_setting[ 'custom_label' ] ) ) {
1733
+			$return_field->label = $field_setting[ 'custom_label' ];
1734 1734
 		}
1735 1735
 
1736
-		if( !empty( $field_setting['custom_class'] ) ) {
1737
-			$return_field->cssClass .= ' '. gravityview_sanitize_html_class( $field_setting['custom_class'] );
1736
+		if ( ! empty( $field_setting[ 'custom_class' ] ) ) {
1737
+			$return_field->cssClass .= ' ' . gravityview_sanitize_html_class( $field_setting[ 'custom_class' ] );
1738 1738
 		}
1739 1739
 
1740 1740
 		/**
@@ -1772,16 +1772,16 @@  discard block
 block discarded – undo
1772 1772
 		 */
1773 1773
 		$use_gf_adminonly_setting = apply_filters( 'gravityview/edit_entry/use_gf_admin_only_setting', empty( $edit_fields ), $form, $view_id );
1774 1774
 
1775
-		if( $use_gf_adminonly_setting && false === GVCommon::has_cap( 'gravityforms_edit_entries', $this->entry['id'] ) ) {
1776
-			foreach( $fields as $k => $field ) {
1777
-				if( $field->adminOnly ) {
1775
+		if ( $use_gf_adminonly_setting && false === GVCommon::has_cap( 'gravityforms_edit_entries', $this->entry[ 'id' ] ) ) {
1776
+			foreach ( $fields as $k => $field ) {
1777
+				if ( $field->adminOnly ) {
1778 1778
 					unset( $fields[ $k ] );
1779 1779
 				}
1780 1780
 			}
1781 1781
 			return $fields;
1782 1782
 		}
1783 1783
 
1784
-		foreach( $fields as &$field ) {
1784
+		foreach ( $fields as &$field ) {
1785 1785
 			$field->adminOnly = false;
1786 1786
 		}
1787 1787
 
@@ -1805,22 +1805,22 @@  discard block
 block discarded – undo
1805 1805
 	 */
1806 1806
 	function prefill_conditional_logic( $form ) {
1807 1807
 
1808
-		if( ! GFFormDisplay::has_conditional_logic( $form ) ) {
1808
+		if ( ! GFFormDisplay::has_conditional_logic( $form ) ) {
1809 1809
 			return $form;
1810 1810
 		}
1811 1811
 
1812 1812
 		// Have Conditional Logic pre-fill fields as if the data were default values
1813 1813
 		/** @var GF_Field $field */
1814
-		foreach ( $form['fields'] as &$field ) {
1814
+		foreach ( $form[ 'fields' ] as &$field ) {
1815 1815
 
1816
-			if( 'checkbox' === $field->type ) {
1816
+			if ( 'checkbox' === $field->type ) {
1817 1817
 				foreach ( $field->get_entry_inputs() as $key => $input ) {
1818
-					$input_id = $input['id'];
1818
+					$input_id = $input[ 'id' ];
1819 1819
 					$choice = $field->choices[ $key ];
1820 1820
 					$value = rgar( $this->entry, $input_id );
1821 1821
 					$match = RGFormsModel::choice_value_match( $field, $choice, $value );
1822
-					if( $match ) {
1823
-						$field->choices[ $key ]['isSelected'] = true;
1822
+					if ( $match ) {
1823
+						$field->choices[ $key ][ 'isSelected' ] = true;
1824 1824
 					}
1825 1825
 				}
1826 1826
 			} else {
@@ -1828,15 +1828,15 @@  discard block
 block discarded – undo
1828 1828
 				// We need to run through each field to set the default values
1829 1829
 				foreach ( $this->entry as $field_id => $field_value ) {
1830 1830
 
1831
-					if( floatval( $field_id ) === floatval( $field->id ) ) {
1831
+					if ( floatval( $field_id ) === floatval( $field->id ) ) {
1832 1832
 
1833
-						if( 'list' === $field->type ) {
1833
+						if ( 'list' === $field->type ) {
1834 1834
 							$list_rows = maybe_unserialize( $field_value );
1835 1835
 
1836 1836
 							$list_field_value = array();
1837
-							foreach ( (array) $list_rows as $row ) {
1838
-								foreach ( (array) $row as $column ) {
1839
-									$list_field_value[] = $column;
1837
+							foreach ( (array)$list_rows as $row ) {
1838
+								foreach ( (array)$row as $column ) {
1839
+									$list_field_value[ ] = $column;
1840 1840
 								}
1841 1841
 							}
1842 1842
 
@@ -1871,16 +1871,16 @@  discard block
 block discarded – undo
1871 1871
 		 */
1872 1872
 		$use_conditional_logic = apply_filters( 'gravityview/edit_entry/conditional_logic', true, $form );
1873 1873
 
1874
-		if( $use_conditional_logic ) {
1874
+		if ( $use_conditional_logic ) {
1875 1875
 			return $form;
1876 1876
 		}
1877 1877
 
1878
-		foreach( $form['fields'] as &$field ) {
1878
+		foreach ( $form[ 'fields' ] as &$field ) {
1879 1879
 			/* @var GF_Field $field */
1880 1880
 			$field->conditionalLogic = null;
1881 1881
 		}
1882 1882
 
1883
-		unset( $form['button']['conditionalLogic'] );
1883
+		unset( $form[ 'button' ][ 'conditionalLogic' ] );
1884 1884
 
1885 1885
 		return $form;
1886 1886
 
@@ -1897,7 +1897,7 @@  discard block
 block discarded – undo
1897 1897
 	 */
1898 1898
 	public function manage_conditional_logic( $has_conditional_logic, $form ) {
1899 1899
 
1900
-		if( ! $this->is_edit_entry() ) {
1900
+		if ( ! $this->is_edit_entry() ) {
1901 1901
 			return $has_conditional_logic;
1902 1902
 		}
1903 1903
 
@@ -1929,44 +1929,44 @@  discard block
 block discarded – undo
1929 1929
 		 *  2. There are two entries embedded using oEmbed
1930 1930
 		 *  3. One of the entries has just been saved
1931 1931
 		 */
1932
-		if( !empty( $_POST['lid'] ) && !empty( $_GET['entry'] ) && ( $_POST['lid'] !== $_GET['entry'] ) ) {
1932
+		if ( ! empty( $_POST[ 'lid' ] ) && ! empty( $_GET[ 'entry' ] ) && ( $_POST[ 'lid' ] !== $_GET[ 'entry' ] ) ) {
1933 1933
 
1934 1934
 			$error = true;
1935 1935
 
1936 1936
 		}
1937 1937
 
1938
-		if( !empty( $_GET['entry'] ) && (string)$this->entry['id'] !== $_GET['entry'] ) {
1938
+		if ( ! empty( $_GET[ 'entry' ] ) && (string)$this->entry[ 'id' ] !== $_GET[ 'entry' ] ) {
1939 1939
 
1940 1940
 			$error = true;
1941 1941
 
1942
-		} elseif( ! $this->verify_nonce() ) {
1942
+		} elseif ( ! $this->verify_nonce() ) {
1943 1943
 
1944 1944
 			/**
1945 1945
 			 * If the Entry is embedded, there may be two entries on the same page.
1946 1946
 			 * If that's the case, and one is being edited, the other should fail gracefully and not display an error.
1947 1947
 			 */
1948
-			if( GravityView_oEmbed::getInstance()->get_entry_id() ) {
1948
+			if ( GravityView_oEmbed::getInstance()->get_entry_id() ) {
1949 1949
 				$error = true;
1950 1950
 			} else {
1951
-				$error = __( 'The link to edit this entry is not valid; it may have expired.', 'gravityview');
1951
+				$error = __( 'The link to edit this entry is not valid; it may have expired.', 'gravityview' );
1952 1952
 			}
1953 1953
 
1954 1954
 		}
1955 1955
 
1956
-		if( ! GravityView_Edit_Entry::check_user_cap_edit_entry( $this->entry ) ) {
1957
-			$error = __( 'You do not have permission to edit this entry.', 'gravityview');
1956
+		if ( ! GravityView_Edit_Entry::check_user_cap_edit_entry( $this->entry ) ) {
1957
+			$error = __( 'You do not have permission to edit this entry.', 'gravityview' );
1958 1958
 		}
1959 1959
 
1960
-		if( $this->entry['status'] === 'trash' ) {
1961
-			$error = __('You cannot edit the entry; it is in the trash.', 'gravityview' );
1960
+		if ( $this->entry[ 'status' ] === 'trash' ) {
1961
+			$error = __( 'You cannot edit the entry; it is in the trash.', 'gravityview' );
1962 1962
 		}
1963 1963
 
1964 1964
 		// No errors; everything's fine here!
1965
-		if( empty( $error ) ) {
1965
+		if ( empty( $error ) ) {
1966 1966
 			return true;
1967 1967
 		}
1968 1968
 
1969
-		if( $echo && $error !== true ) {
1969
+		if ( $echo && $error !== true ) {
1970 1970
 
1971 1971
 			$error = esc_html( $error );
1972 1972
 
@@ -1974,13 +1974,13 @@  discard block
 block discarded – undo
1974 1974
 			 * @since 1.9
1975 1975
 			 */
1976 1976
 			if ( ! empty( $this->entry ) ) {
1977
-				$error .= ' ' . gravityview_get_link( '#', _x('Go back.', 'Link shown when invalid Edit Entry link is clicked', 'gravityview' ), array( 'onclick' => "window.history.go(-1); return false;" ) );
1977
+				$error .= ' ' . gravityview_get_link( '#', _x( 'Go back.', 'Link shown when invalid Edit Entry link is clicked', 'gravityview' ), array( 'onclick' => "window.history.go(-1); return false;" ) );
1978 1978
 			}
1979 1979
 
1980
-			echo GVCommon::generate_notice( wpautop( $error ), 'gv-error error');
1980
+			echo GVCommon::generate_notice( wpautop( $error ), 'gv-error error' );
1981 1981
 		}
1982 1982
 
1983
-		do_action('gravityview_log_error', 'GravityView_Edit_Entry[user_can_edit_entry]' . $error );
1983
+		do_action( 'gravityview_log_error', 'GravityView_Edit_Entry[user_can_edit_entry]' . $error );
1984 1984
 
1985 1985
 		return false;
1986 1986
 	}
@@ -1997,20 +1997,20 @@  discard block
 block discarded – undo
1997 1997
 
1998 1998
 		$error = NULL;
1999 1999
 
2000
-		if( ! $this->check_user_cap_edit_field( $field ) ) {
2001
-			$error = __( 'You do not have permission to edit this field.', 'gravityview');
2000
+		if ( ! $this->check_user_cap_edit_field( $field ) ) {
2001
+			$error = __( 'You do not have permission to edit this field.', 'gravityview' );
2002 2002
 		}
2003 2003
 
2004 2004
 		// No errors; everything's fine here!
2005
-		if( empty( $error ) ) {
2005
+		if ( empty( $error ) ) {
2006 2006
 			return true;
2007 2007
 		}
2008 2008
 
2009
-		if( $echo ) {
2010
-			echo GVCommon::generate_notice( wpautop( esc_html( $error ) ), 'gv-error error');
2009
+		if ( $echo ) {
2010
+			echo GVCommon::generate_notice( wpautop( esc_html( $error ) ), 'gv-error error' );
2011 2011
 		}
2012 2012
 
2013
-		do_action('gravityview_log_error', 'GravityView_Edit_Entry[user_can_edit_field]' . $error );
2013
+		do_action( 'gravityview_log_error', 'GravityView_Edit_Entry[user_can_edit_field]' . $error );
2014 2014
 
2015 2015
 		return false;
2016 2016
 
@@ -2028,15 +2028,15 @@  discard block
 block discarded – undo
2028 2028
 	private function check_user_cap_edit_field( $field ) {
2029 2029
 
2030 2030
 		// If they can edit any entries (as defined in Gravity Forms), we're good.
2031
-		if( GVCommon::has_cap( array( 'gravityforms_edit_entries', 'gravityview_edit_others_entries' ) ) ) {
2031
+		if ( GVCommon::has_cap( array( 'gravityforms_edit_entries', 'gravityview_edit_others_entries' ) ) ) {
2032 2032
 			return true;
2033 2033
 		}
2034 2034
 
2035
-		$field_cap = isset( $field['allow_edit_cap'] ) ? $field['allow_edit_cap'] : false;
2035
+		$field_cap = isset( $field[ 'allow_edit_cap' ] ) ? $field[ 'allow_edit_cap' ] : false;
2036 2036
 
2037 2037
 		// If the field has custom editing capaibilities set, check those
2038
-		if( $field_cap ) {
2039
-			return GVCommon::has_cap( $field['allow_edit_cap'] );
2038
+		if ( $field_cap ) {
2039
+			return GVCommon::has_cap( $field[ 'allow_edit_cap' ] );
2040 2040
 		}
2041 2041
 
2042 2042
 		return false;
@@ -2050,17 +2050,17 @@  discard block
 block discarded – undo
2050 2050
 	public function verify_nonce() {
2051 2051
 
2052 2052
 		// Verify form submitted for editing single
2053
-		if( $this->is_edit_entry_submission() ) {
2053
+		if ( $this->is_edit_entry_submission() ) {
2054 2054
 			$valid = wp_verify_nonce( $_POST[ self::$nonce_field ], self::$nonce_field );
2055 2055
 		}
2056 2056
 
2057 2057
 		// Verify
2058
-		else if( ! $this->is_edit_entry() ) {
2058
+		else if ( ! $this->is_edit_entry() ) {
2059 2059
 			$valid = false;
2060 2060
 		}
2061 2061
 
2062 2062
 		else {
2063
-			$valid = wp_verify_nonce( $_GET['edit'], self::$nonce_key );
2063
+			$valid = wp_verify_nonce( $_GET[ 'edit' ], self::$nonce_key );
2064 2064
 		}
2065 2065
 
2066 2066
 		/**
Please login to merge, or discard this patch.
includes/class-data.php 1 patch
Spacing   +128 added lines, -128 removed lines patch added patch discarded remove patch
@@ -21,12 +21,12 @@  discard block
 block discarded – undo
21 21
 			gravityview()->request = new \GV\Dummy_Request();
22 22
 		}
23 23
 
24
-		if( !empty( $passed_post ) ) {
24
+		if ( ! empty( $passed_post ) ) {
25 25
 
26 26
 			$id_or_id_array = $this->maybe_get_view_id( $passed_post );
27 27
 
28 28
 			if ( defined( 'GRAVITYVIEW_FUTURE_CORE_LOADED' ) ) {
29
-				foreach( is_array( $id_or_id_array ) ? $id_or_id_array : array( $id_or_id_array ) as $view_id ) {
29
+				foreach ( is_array( $id_or_id_array ) ? $id_or_id_array : array( $id_or_id_array ) as $view_id ) {
30 30
 					if ( \GV\View::exists( $view_id ) && ! gravityview()->views->contains( $view_id ) ) {
31 31
 						gravityview()->views->add( \GV\View::by_id( $view_id ) );
32 32
 					}
@@ -72,9 +72,9 @@  discard block
 block discarded – undo
72 72
 	 */
73 73
 	public function maybe_get_view_id( $passed_post ) {
74 74
 		$ids = array();
75
-		if( ! empty( $passed_post ) ) {
75
+		if ( ! empty( $passed_post ) ) {
76 76
 
77
-			if( is_numeric( $passed_post ) ) {
77
+			if ( is_numeric( $passed_post ) ) {
78 78
 				$passed_post = get_post( $passed_post );
79 79
 			}
80 80
 
@@ -83,14 +83,14 @@  discard block
 block discarded – undo
83 83
 				$passed_post = array( $passed_post );
84 84
 			}
85 85
 
86
-			if( is_array( $passed_post ) ) {
86
+			if ( is_array( $passed_post ) ) {
87 87
 
88
-				foreach ( $passed_post as &$post) {
88
+				foreach ( $passed_post as &$post ) {
89 89
 					if ( defined( 'GRAVITYVIEW_FUTURE_CORE_LOADED' ) && is_object( $post ) && property_exists( $post, 'ID' ) ) {
90 90
 						$post = get_post( $post ); // Convert into real WP_Post, thanks globals
91 91
 						$views = \GV\View_Collection::from_post( $post );
92 92
 						foreach ( $views->all() as $view ) {
93
-							$ids []= $view->ID;
93
+							$ids [ ] = $view->ID;
94 94
 
95 95
 							/** And as a side-effect... add each view to the global scope. */
96 96
 							if ( ! gravityview()->views->contains( $view->ID ) ) {
@@ -99,19 +99,19 @@  discard block
 block discarded – undo
99 99
 						}
100 100
 					} else {
101 101
 						/** Deprecated, see \GV\View_Collection::from_post */
102
-						if( ( get_post_type( $post ) === 'gravityview' ) ) {
103
-							$ids[] = $post->ID;
104
-						} else{
102
+						if ( ( get_post_type( $post ) === 'gravityview' ) ) {
103
+							$ids[ ] = $post->ID;
104
+						} else {
105 105
 							// Parse the Post Content
106 106
 							$id = $this->parse_post_content( $post->post_content );
107
-							if( $id ) {
108
-								$ids = array_merge( $ids, (array) $id );
107
+							if ( $id ) {
108
+								$ids = array_merge( $ids, (array)$id );
109 109
 							}
110 110
 
111 111
 							// Parse the Post Meta
112 112
 							$id = $this->parse_post_meta( $post->ID );
113
-							if( $id ) {
114
-								$ids = array_merge( $ids, (array) $id );
113
+							if ( $id ) {
114
+								$ids = array_merge( $ids, (array)$id );
115 115
 							}
116 116
 						}
117 117
 					}
@@ -125,37 +125,37 @@  discard block
 block discarded – undo
125 125
 					if ( defined( 'GRAVITYVIEW_FUTURE_CORE_LOADED' ) ) {
126 126
 						$shortcodes = \GV\Shortcode::parse( $passed_post );
127 127
 						foreach ( $shortcodes as $shortcode ) {
128
-							if ( $shortcode->name == 'gravityview' && !empty( $shortcode->atts['id'] ) ) {
129
-								$ids []= $shortcode->atts['id'];
128
+							if ( $shortcode->name == 'gravityview' && ! empty( $shortcode->atts[ 'id' ] ) ) {
129
+								$ids [ ] = $shortcode->atts[ 'id' ];
130 130
 
131 131
 								/** And as a side-effect... add each view to the global scope. */
132
-								if ( ! gravityview()->views->contains( $shortcode->atts['id'] ) && \GV\View::exists( $shortcode->atts['id'] ) ) {
133
-									gravityview()->views->add( $shortcode->atts['id'] );
132
+								if ( ! gravityview()->views->contains( $shortcode->atts[ 'id' ] ) && \GV\View::exists( $shortcode->atts[ 'id' ] ) ) {
133
+									gravityview()->views->add( $shortcode->atts[ 'id' ] );
134 134
 								}
135 135
 							}
136 136
 						}
137 137
 					} else {
138 138
 						/** Deprecated, use \GV\Shortcode::parse. */
139 139
 						$id = $this->parse_post_content( $passed_post );
140
-						if( $id ) {
141
-							$ids = array_merge( $ids, (array) $id );
140
+						if ( $id ) {
141
+							$ids = array_merge( $ids, (array)$id );
142 142
 						}
143 143
 					}
144 144
 
145 145
 				} else {
146 146
 					$id = $this->get_id_from_atts( $passed_post );
147
-					$ids[] = intval( $id );
147
+					$ids[ ] = intval( $id );
148 148
 				}
149 149
 			}
150 150
 		}
151 151
 
152
-		if( empty($ids) ) {
152
+		if ( empty( $ids ) ) {
153 153
 			return NULL;
154 154
 		}
155 155
 
156 156
 		// If it's just one ID, return that.
157 157
 		// Otherwise, return array of IDs
158
-		return ( sizeof( $ids ) === 1 ) ? $ids[0] : $ids;
158
+		return ( sizeof( $ids ) === 1 ) ? $ids[ 0 ] : $ids;
159 159
 	}
160 160
 
161 161
 	/**
@@ -163,7 +163,7 @@  discard block
 block discarded – undo
163 163
 	 */
164 164
 	public static function getInstance( $passed_post = NULL ) {
165 165
 
166
-		if( empty( self::$instance ) ) {
166
+		if ( empty( self::$instance ) ) {
167 167
 			self::$instance = new GravityView_View_Data( $passed_post );
168 168
 		}
169 169
 
@@ -180,8 +180,8 @@  discard block
 block discarded – undo
180 180
 				return array();
181 181
 			}
182 182
 			return array_combine(
183
-				array_map( function ( $view ) { return $view->ID; }, gravityview()->views->all() ),
184
-				array_map( function ( $view ) { return $view->as_data(); }, gravityview()->views->all() )
183
+				array_map( function( $view ) { return $view->ID; }, gravityview()->views->all() ),
184
+				array_map( function( $view ) { return $view->as_data(); }, gravityview()->views->all() )
185 185
 			);
186 186
 		}
187 187
 		return $this->views;
@@ -208,19 +208,19 @@  discard block
 block discarded – undo
208 208
 			return $view->as_data();
209 209
 		}
210 210
 
211
-		if( ! is_numeric( $view_id) ) {
212
-			do_action('gravityview_log_error', sprintf('GravityView_View_Data[get_view] $view_id passed is not numeric.', $view_id) );
211
+		if ( ! is_numeric( $view_id ) ) {
212
+			do_action( 'gravityview_log_error', sprintf( 'GravityView_View_Data[get_view] $view_id passed is not numeric.', $view_id ) );
213 213
 			return false;
214 214
 		}
215 215
 
216 216
 		// Backup: the view hasn't been fetched yet. Doing it now.
217 217
 		if ( ! isset( $this->views[ $view_id ] ) ) {
218
-			do_action('gravityview_log_debug', sprintf('GravityView_View_Data[get_view] View #%s not set yet.', $view_id) );
218
+			do_action( 'gravityview_log_debug', sprintf( 'GravityView_View_Data[get_view] View #%s not set yet.', $view_id ) );
219 219
 			return $this->add_view( $view_id, $atts );
220 220
 		}
221 221
 
222 222
 		if ( empty( $this->views[ $view_id ] ) ) {
223
-			do_action('gravityview_log_debug', sprintf('GravityView_View_Data[get_view] Returning; View #%s was empty.', $view_id) );
223
+			do_action( 'gravityview_log_debug', sprintf( 'GravityView_View_Data[get_view] Returning; View #%s was empty.', $view_id ) );
224 224
 			return false;
225 225
 		}
226 226
 
@@ -265,8 +265,8 @@  discard block
 block discarded – undo
265 265
 		}
266 266
 
267 267
 		// Handle array of IDs
268
-		if( is_array( $view_id ) ) {
269
-			foreach( $view_id as $id ) {
268
+		if ( is_array( $view_id ) ) {
269
+			foreach ( $view_id as $id ) {
270 270
 
271 271
 				$this->add_view( $id, $atts );
272 272
 			}
@@ -275,21 +275,21 @@  discard block
 block discarded – undo
275 275
 		}
276 276
 
277 277
 		// The view has been set already; returning stored view.
278
-		if ( !empty( $this->views[ $view_id ] ) ) {
279
-			do_action('gravityview_log_debug', sprintf('GravityView_View_Data[add_view] Returning; View #%s already exists.', $view_id) );
278
+		if ( ! empty( $this->views[ $view_id ] ) ) {
279
+			do_action( 'gravityview_log_debug', sprintf( 'GravityView_View_Data[add_view] Returning; View #%s already exists.', $view_id ) );
280 280
 			return $this->views[ $view_id ];
281 281
 		}
282 282
 
283
-		if( ! $this->view_exists( $view_id ) ) {
284
-			do_action('gravityview_log_debug', sprintf('GravityView_View_Data[add_view] Returning; View #%s does not exist.', $view_id) );
283
+		if ( ! $this->view_exists( $view_id ) ) {
284
+			do_action( 'gravityview_log_debug', sprintf( 'GravityView_View_Data[add_view] Returning; View #%s does not exist.', $view_id ) );
285 285
 			return false;
286 286
 		}
287 287
 
288 288
 		$form_id = gravityview_get_form_id( $view_id );
289 289
 
290
-		if( empty( $form_id ) ) {
290
+		if ( empty( $form_id ) ) {
291 291
 
292
-			do_action('gravityview_log_debug', sprintf('GravityView_View_Data[add_view] Returning; Post ID #%s does not have a connected form.', $view_id) );
292
+			do_action( 'gravityview_log_debug', sprintf( 'GravityView_View_Data[add_view] Returning; Post ID #%s does not have a connected form.', $view_id ) );
293 293
 
294 294
 			return false;
295 295
 		}
@@ -297,21 +297,21 @@  discard block
 block discarded – undo
297 297
 		// Get the settings for the View ID
298 298
 		$view_settings = gravityview_get_template_settings( $view_id );
299 299
 
300
-		do_action('gravityview_log_debug', sprintf('GravityView_View_Data[add_view] Settings pulled in from View #%s', $view_id), $view_settings );
300
+		do_action( 'gravityview_log_debug', sprintf( 'GravityView_View_Data[add_view] Settings pulled in from View #%s', $view_id ), $view_settings );
301 301
 
302 302
 		// Merge the view settings with the defaults
303 303
 		$view_defaults = wp_parse_args( $view_settings, defined( 'GRAVITYVIEW_FUTURE_CORE_LOADED' ) ? \GV\View_Settings::defaults() : self::get_default_args() );
304 304
 
305
-		do_action('gravityview_log_debug', 'GravityView_View_Data[add_view] View Defaults after merging View Settings with the default args.', $view_defaults );
305
+		do_action( 'gravityview_log_debug', 'GravityView_View_Data[add_view] View Defaults after merging View Settings with the default args.', $view_defaults );
306 306
 
307
-		if( ! empty( $atts ) && is_array( $atts ) ) {
307
+		if ( ! empty( $atts ) && is_array( $atts ) ) {
308 308
 
309
-			do_action('gravityview_log_debug', 'GravityView_View_Data[add_view] $atts before merging  with the $view_defaults', $atts );
309
+			do_action( 'gravityview_log_debug', 'GravityView_View_Data[add_view] $atts before merging  with the $view_defaults', $atts );
310 310
 
311 311
 			// Get the settings from the shortcode and merge them with defaults.
312 312
 			$atts = shortcode_atts( $view_defaults, $atts );
313 313
 
314
-			do_action('gravityview_log_debug', 'GravityView_View_Data[add_view] $atts after merging  with the $view_defaults', $atts );
314
+			do_action( 'gravityview_log_debug', 'GravityView_View_Data[add_view] $atts after merging  with the $view_defaults', $atts );
315 315
 
316 316
 		} else {
317 317
 
@@ -320,7 +320,7 @@  discard block
 block discarded – undo
320 320
 
321 321
 		}
322 322
 
323
-		unset( $atts['id'], $view_defaults, $view_settings );
323
+		unset( $atts[ 'id' ], $view_defaults, $view_settings );
324 324
 
325 325
 		$data = array(
326 326
 			'id' => $view_id,
@@ -333,7 +333,7 @@  discard block
 block discarded – undo
333 333
 			'form' => gravityview_get_form( $form_id ),
334 334
 		);
335 335
 
336
-		do_action('gravityview_log_debug', sprintf('GravityView_View_Data[add_view] View #%s being added.', $view_id), $data );
336
+		do_action( 'gravityview_log_debug', sprintf( 'GravityView_View_Data[add_view] View #%s being added.', $view_id ), $data );
337 337
 
338 338
 		$this->views[ $view_id ] = $data;
339 339
 
@@ -383,15 +383,15 @@  discard block
 block discarded – undo
383 383
 			throw new Exception( __METHOD__ . ' should not be called anymore. Why was it?' );
384 384
 		}
385 385
 
386
-		if( empty( $dir_fields ) || !is_array( $dir_fields ) ) {
386
+		if ( empty( $dir_fields ) || ! is_array( $dir_fields ) ) {
387 387
 			return $dir_fields;
388 388
 		}
389 389
 
390
-		foreach( $dir_fields as $area => $fields ) {
390
+		foreach ( $dir_fields as $area => $fields ) {
391 391
 
392
-			foreach( (array)$fields as $uniqid => $properties ) {
392
+			foreach ( (array)$fields as $uniqid => $properties ) {
393 393
 
394
-				if( $this->hide_field_check_conditions( $properties ) ) {
394
+				if ( $this->hide_field_check_conditions( $properties ) ) {
395 395
 					unset( $dir_fields[ $area ][ $uniqid ] );
396 396
 				}
397 397
 
@@ -418,7 +418,7 @@  discard block
 block discarded – undo
418 418
 		}
419 419
 
420 420
 		// logged-in visibility
421
-		if( ! empty( $properties['only_loggedin'] ) && ! GVCommon::has_cap( $properties['only_loggedin_cap'] ) ) {
421
+		if ( ! empty( $properties[ 'only_loggedin' ] ) && ! GVCommon::has_cap( $properties[ 'only_loggedin_cap' ] ) ) {
422 422
 			return true;
423 423
 		}
424 424
 
@@ -449,14 +449,14 @@  discard block
 block discarded – undo
449 449
 		// Get the settings from the shortcode and merge them with defaults.
450 450
 		$atts = wp_parse_args( $atts, defined( 'GRAVITYVIEW_FUTURE_CORE_LOADED' ) ? \GV\View_Settings::defaults() : self::get_default_args() );
451 451
 
452
-		$view_id = ! empty( $atts['view_id'] ) ? (int)$atts['view_id'] : NULL;
452
+		$view_id = ! empty( $atts[ 'view_id' ] ) ? (int)$atts[ 'view_id' ] : NULL;
453 453
 
454
-		if( empty( $view_id ) && !empty( $atts['id'] ) ) {
455
-			$view_id = (int)$atts['id'];
454
+		if ( empty( $view_id ) && ! empty( $atts[ 'id' ] ) ) {
455
+			$view_id = (int)$atts[ 'id' ];
456 456
 		}
457 457
 
458
-		if( empty( $view_id ) ) {
459
-			do_action('gravityview_log_error', 'GravityView_View_Data[get_id_from_atts] Returning; no ID defined (Atts)', $atts );
458
+		if ( empty( $view_id ) ) {
459
+			do_action( 'gravityview_log_error', 'GravityView_View_Data[get_id_from_atts] Returning; no ID defined (Atts)', $atts );
460 460
 			return;
461 461
 		}
462 462
 
@@ -480,68 +480,68 @@  discard block
 block discarded – undo
480 480
 		if ( defined( 'GRAVITYVIEW_FUTURE_CORE_LOADED' ) ) {
481 481
 			$ids = array();
482 482
 			foreach ( \GV\Shortcode::parse( $content ) as $shortcode ) {
483
-				if ( $shortcode->name == 'gravityview' && is_numeric( $shortcode->atts['id'] ) ) {
484
-					if ( \GV\View::exists( $shortcode->atts['id'] ) && ! gravityview()->views->contains( $shortcode->atts['id'] ) ) {
485
-						gravityview()->views->add( \GV\View::by_id( $shortcode->atts['id'] ) );
483
+				if ( $shortcode->name == 'gravityview' && is_numeric( $shortcode->atts[ 'id' ] ) ) {
484
+					if ( \GV\View::exists( $shortcode->atts[ 'id' ] ) && ! gravityview()->views->contains( $shortcode->atts[ 'id' ] ) ) {
485
+						gravityview()->views->add( \GV\View::by_id( $shortcode->atts[ 'id' ] ) );
486 486
 					}
487 487
 					/**
488 488
 					 * The original function outputs the ID even though it wasn't added by ::add_view()
489 489
 					 * Wether this is a bug or not remains a mystery. But we need to emulate this behavior
490 490
 					 * until better times.
491 491
 					 */
492
-					$ids []= $shortcode->atts['id'];
492
+					$ids [ ] = $shortcode->atts[ 'id' ];
493 493
 				}
494 494
 			}
495 495
 			if ( empty ( $ids ) ) {
496 496
 				return null;
497 497
 			}
498
-			return ( sizeof( $ids ) === 1 ) ? $ids[0] : $ids;
498
+			return ( sizeof( $ids ) === 1 ) ? $ids[ 0 ] : $ids;
499 499
 		}
500 500
 
501 501
 		/**
502 502
 		 * @hack This is so that the shortcode is registered for the oEmbed preview in the Admin
503 503
 		 * @since 1.6
504 504
 		 */
505
-		if( ! shortcode_exists('gravityview') && class_exists( 'GravityView_Shortcode' ) ) {
505
+		if ( ! shortcode_exists( 'gravityview' ) && class_exists( 'GravityView_Shortcode' ) ) {
506 506
 			new GravityView_Shortcode;
507 507
 		}
508 508
 
509 509
 		$shortcodes = gravityview_has_shortcode_r( $content, 'gravityview' );
510 510
 
511
-		if( empty( $shortcodes ) ) {
511
+		if ( empty( $shortcodes ) ) {
512 512
 			return NULL;
513 513
 		}
514 514
 
515
-		do_action('gravityview_log_debug', 'GravityView_View_Data[parse_post_content] Parsing content, found shortcodes:', $shortcodes );
515
+		do_action( 'gravityview_log_debug', 'GravityView_View_Data[parse_post_content] Parsing content, found shortcodes:', $shortcodes );
516 516
 
517 517
 		$ids = array();
518 518
 
519
-		foreach ($shortcodes as $key => $shortcode) {
519
+		foreach ( $shortcodes as $key => $shortcode ) {
520 520
 
521
-			$shortcode[3] = htmlspecialchars_decode( $shortcode[3], ENT_QUOTES );
521
+			$shortcode[ 3 ] = htmlspecialchars_decode( $shortcode[ 3 ], ENT_QUOTES );
522 522
 
523
-			$args = shortcode_parse_atts( $shortcode[3] );
523
+			$args = shortcode_parse_atts( $shortcode[ 3 ] );
524 524
 
525
-			if( empty( $args['id'] ) ) {
526
-				do_action('gravityview_log_error', 'GravityView_View_Data[parse_post_content] Returning; no ID defined in shortcode atts', $shortcode );
525
+			if ( empty( $args[ 'id' ] ) ) {
526
+				do_action( 'gravityview_log_error', 'GravityView_View_Data[parse_post_content] Returning; no ID defined in shortcode atts', $shortcode );
527 527
 				continue;
528 528
 			}
529 529
 
530
-			do_action('gravityview_log_debug', sprintf('GravityView_View_Data[parse_post_content] Adding view #%s with shortcode args', $args['id']), $args );
530
+			do_action( 'gravityview_log_debug', sprintf( 'GravityView_View_Data[parse_post_content] Adding view #%s with shortcode args', $args[ 'id' ] ), $args );
531 531
 
532 532
 			// Store the View to the object for later fetching.
533
-			$this->add_view( $args['id'], $args );
533
+			$this->add_view( $args[ 'id' ], $args );
534 534
 
535
-			$ids[] = $args['id'];
535
+			$ids[ ] = $args[ 'id' ];
536 536
 		}
537 537
 
538
-		if( empty($ids) ) {
538
+		if ( empty( $ids ) ) {
539 539
 			return NULL;
540 540
 		}
541 541
 
542 542
 		// If it's just one ID, return that.
543 543
 		// Otherwise, return array of IDs
544
-		return ( sizeof( $ids ) === 1 ) ? $ids[0] : $ids;
544
+		return ( sizeof( $ids ) === 1 ) ? $ids[ 0 ] : $ids;
545 545
 
546 546
 	}
547 547
 
@@ -570,7 +570,7 @@  discard block
 block discarded – undo
570 570
 		 */
571 571
 		$meta_keys = (array)apply_filters( 'gravityview/data/parse/meta_keys', array(), $post_id );
572 572
 
573
-		if( empty( $meta_keys ) ) {
573
+		if ( empty( $meta_keys ) ) {
574 574
 			return NULL;
575 575
 		}
576 576
 
@@ -578,16 +578,16 @@  discard block
 block discarded – undo
578 578
 
579 579
 		$meta_content = '';
580 580
 
581
-		foreach( $meta_keys as $key ) {
582
-			$meta = get_post_meta( $post_id, $key , true );
583
-			if( ! is_string( $meta ) ) {
581
+		foreach ( $meta_keys as $key ) {
582
+			$meta = get_post_meta( $post_id, $key, true );
583
+			if ( ! is_string( $meta ) ) {
584 584
 				continue;
585 585
 			}
586 586
 			$meta_content .= $meta . ' ';
587 587
 		}
588 588
 
589
-		if( empty( $meta_content ) ) {
590
-			do_action('gravityview_log_error', sprintf( 'GravityView_View_Data[parse_post_meta] Returning; Empty custom fields for Post #%s (Custom fields keys:)', $post_id ), $meta_keys );
589
+		if ( empty( $meta_content ) ) {
590
+			do_action( 'gravityview_log_error', sprintf( 'GravityView_View_Data[parse_post_meta] Returning; Empty custom fields for Post #%s (Custom fields keys:)', $post_id ), $meta_keys );
591 591
 			return NULL;
592 592
 		}
593 593
 
@@ -615,16 +615,16 @@  discard block
 block discarded – undo
615 615
 		$message = NULL;
616 616
 
617 617
 		// Not invalid if not set!
618
-		if( empty( $post_id ) || empty( $view_id ) ) {
618
+		if ( empty( $post_id ) || empty( $view_id ) ) {
619 619
 
620
-			if( $empty_is_valid ) {
620
+			if ( $empty_is_valid ) {
621 621
 				return true;
622 622
 			}
623 623
 
624 624
 			$message = esc_html__( 'The ID is required.', 'gravityview' );
625 625
 		}
626 626
 
627
-		if( ! $message ) {
627
+		if ( ! $message ) {
628 628
 			$status = get_post_status( $post_id );
629 629
 
630 630
 			// Nothing exists with that post ID.
@@ -636,7 +636,7 @@  discard block
 block discarded – undo
636 636
 			}
637 637
 		}
638 638
 
639
-		if( ! $message ) {
639
+		if ( ! $message ) {
640 640
 
641 641
 			// Nothing exists with that post ID.
642 642
 			if ( empty( $status ) || in_array( $status, array( 'revision', 'attachment' ) ) ) {
@@ -645,8 +645,8 @@  discard block
 block discarded – undo
645 645
 
646 646
 		}
647 647
 
648
-		if( ! $message ) {
649
-			if ( defined( 'GRAVITYVIEW_FUTURE_CORE_LOADED' ) && $post = get_post( $post_id ) )  {
648
+		if ( ! $message ) {
649
+			if ( defined( 'GRAVITYVIEW_FUTURE_CORE_LOADED' ) && $post = get_post( $post_id ) ) {
650 650
 				$views = GV\View_Collection::from_post( $post );
651 651
 				$view_ids_in_post = array_map( function( $view ) { return $view->ID; }, $views->all() );
652 652
 			} else {
@@ -655,21 +655,21 @@  discard block
 block discarded – undo
655 655
 			}
656 656
 
657 657
 			// The post or page specified does not contain the shortcode.
658
-			if ( false === in_array( $view_id, (array) $view_ids_in_post ) ) {
658
+			if ( false === in_array( $view_id, (array)$view_ids_in_post ) ) {
659 659
 				$message = sprintf( esc_html__( 'The Post ID entered is not valid. You may have entered a post or page that does not contain the selected View. Make sure the post contains the following shortcode: %s', 'gravityview' ), '<br /><code>[gravityview id="' . intval( $view_id ) . '"]</code>' );
660 660
 			}
661 661
 		}
662 662
 
663
-		if( ! $message ) {
663
+		if ( ! $message ) {
664 664
 
665 665
 			// It's a View
666 666
 			if ( ( defined( 'GRAVITYVIEW_FUTURE_CORE_LOADED' ) && \GV\View::exists( $post_id ) )
667 667
 				|| 'gravityview' === get_post_type( $post_id ) ) {
668
-				$message = esc_html__( 'The ID is already a View.', 'gravityview' );;
668
+				$message = esc_html__( 'The ID is already a View.', 'gravityview' ); ;
669 669
 			}
670 670
 		}
671 671
 
672
-		if( $message ) {
672
+		if ( $message ) {
673 673
 			return new WP_Error( 'invalid_embed_id', $message );
674 674
 		}
675 675
 
@@ -686,7 +686,7 @@  discard block
 block discarded – undo
686 686
 
687 687
 		$args = defined( 'GRAVITYVIEW_FUTURE_CORE_LOADED' ) ? \GV\View_Settings::defaults( $with_details ) : self::get_default_args( $with_details );
688 688
 
689
-		if( !isset( $args[ $key ] ) ) { return NULL; }
689
+		if ( ! isset( $args[ $key ] ) ) { return NULL; }
690 690
 
691 691
 		return $args[ $key ];
692 692
 	}
@@ -717,7 +717,7 @@  discard block
 block discarded – undo
717 717
 
718 718
 		$default_settings = array(
719 719
 			'id' => array(
720
-				'label' => __('View ID', 'gravityview'),
720
+				'label' => __( 'View ID', 'gravityview' ),
721 721
 				'type' => 'number',
722 722
 				'group'	=> 'default',
723 723
 				'value' => NULL,
@@ -725,7 +725,7 @@  discard block
 block discarded – undo
725 725
 				'show_in_shortcode' => false,
726 726
 			),
727 727
 			'page_size' => array(
728
-				'label' 	=> __('Number of entries per page', 'gravityview'),
728
+				'label' 	=> __( 'Number of entries per page', 'gravityview' ),
729 729
 				'type' => 'number',
730 730
 				'class'	=> 'small-text',
731 731
 				'group'	=> 'default',
@@ -733,7 +733,7 @@  discard block
 block discarded – undo
733 733
 				'show_in_shortcode' => true,
734 734
 			),
735 735
 			'offset' => array(
736
-				'label' 	=> __('Offset entries starting from', 'gravityview'),
736
+				'label' 	=> __( 'Offset entries starting from', 'gravityview' ),
737 737
 				'type' => 'number',
738 738
 				'class'	=> 'small-text',
739 739
 				'group'	=> 'default',
@@ -757,8 +757,8 @@  discard block
 block discarded – undo
757 757
 			),
758 758
 			'admin_show_all_statuses' => array(
759 759
 				'label' => __( 'Show all entries to administrators', 'gravityview' ),
760
-				'desc'	=> __('Administrators will be able to see entries with any approval status.', 'gravityview'),
761
-				'tooltip' => __('Logged-out visitors and non-administrators will only see approved entries, while administrators will see entries with all statuses. This makes it easier for administrators to moderate entries from a View.', 'gravityview'),
760
+				'desc'	=> __( 'Administrators will be able to see entries with any approval status.', 'gravityview' ),
761
+				'tooltip' => __( 'Logged-out visitors and non-administrators will only see approved entries, while administrators will see entries with all statuses. This makes it easier for administrators to moderate entries from a View.', 'gravityview' ),
762 762
 				'requires' => 'show_only_approved',
763 763
 				'type' => 'checkbox',
764 764
 				'group'	=> 'default',
@@ -783,40 +783,40 @@  discard block
 block discarded – undo
783 783
 			'user_edit' => array(
784 784
 				'label'	=> __( 'Allow User Edit', 'gravityview' ),
785 785
 				'group'	=> 'default',
786
-				'desc'	=> __('Allow logged-in users to edit entries they created.', 'gravityview'),
786
+				'desc'	=> __( 'Allow logged-in users to edit entries they created.', 'gravityview' ),
787 787
 				'value'	=> 0,
788
-				'tooltip' => __('Display "Edit Entry" fields to non-administrator users if they created the entry. Edit Entry fields will always be displayed to site administrators.', 'gravityview'),
788
+				'tooltip' => __( 'Display "Edit Entry" fields to non-administrator users if they created the entry. Edit Entry fields will always be displayed to site administrators.', 'gravityview' ),
789 789
 				'type'	=> 'checkbox',
790 790
 				'show_in_shortcode' => true,
791 791
 			),
792 792
 			'user_delete' => array(
793 793
 				'label'	=> __( 'Allow User Delete', 'gravityview' ),
794 794
 				'group'	=> 'default',
795
-				'desc'	=> __('Allow logged-in users to delete entries they created.', 'gravityview'),
795
+				'desc'	=> __( 'Allow logged-in users to delete entries they created.', 'gravityview' ),
796 796
 				'value'	=> 0,
797
-				'tooltip' => __('Display "Delete Entry" fields to non-administrator users if they created the entry. Delete Entry fields will always be displayed to site administrators.', 'gravityview'),
797
+				'tooltip' => __( 'Display "Delete Entry" fields to non-administrator users if they created the entry. Delete Entry fields will always be displayed to site administrators.', 'gravityview' ),
798 798
 				'type'	=> 'checkbox',
799 799
 				'show_in_shortcode' => true,
800 800
 			),
801 801
 			'sort_field' => array(
802
-				'label'	=> __('Sort by field', 'gravityview'),
802
+				'label'	=> __( 'Sort by field', 'gravityview' ),
803 803
 				'type' => 'select',
804 804
 				'value' => '',
805 805
 				'group'	=> 'sort',
806 806
 				'options' => array(
807
-					'' => __( 'Default', 'gravityview'),
808
-					'date_created' => __( 'Date Created', 'gravityview'),
807
+					'' => __( 'Default', 'gravityview' ),
808
+					'date_created' => __( 'Date Created', 'gravityview' ),
809 809
 				),
810 810
 				'show_in_shortcode' => true,
811 811
 			),
812 812
 			'sort_direction' => array(
813
-				'label' 	=> __('Sort direction', 'gravityview'),
813
+				'label' 	=> __( 'Sort direction', 'gravityview' ),
814 814
 				'type' => 'select',
815 815
 				'value' => 'ASC',
816 816
 				'group'	=> 'sort',
817 817
 				'options' => array(
818
-					'ASC' => __('ASC', 'gravityview'),
819
-					'DESC' => __('DESC', 'gravityview'),
818
+					'ASC' => __( 'ASC', 'gravityview' ),
819
+					'DESC' => __( 'DESC', 'gravityview' ),
820 820
 				),
821 821
 				'show_in_shortcode' => true,
822 822
 			),
@@ -831,69 +831,69 @@  discard block
 block discarded – undo
831 831
 				'show_in_template' => array( 'default_table', 'preset_business_data', 'preset_issue_tracker', 'preset_resume_board', 'preset_job_board' ),
832 832
 			),
833 833
 			'start_date' => array(
834
-				'label' 	=> __('Filter by Start Date', 'gravityview'),
834
+				'label' 	=> __( 'Filter by Start Date', 'gravityview' ),
835 835
 				'class'	=> 'gv-datepicker',
836
-				'desc'	=> __('Show entries submitted after this date. Supports relative dates, such as "-1 week" or "-1 month".', 'gravityview' ),
836
+				'desc'	=> __( 'Show entries submitted after this date. Supports relative dates, such as "-1 week" or "-1 month".', 'gravityview' ),
837 837
 				'type' => 'text',
838 838
 				'value' => '',
839 839
 				'group'	=> 'filter',
840 840
 				'show_in_shortcode' => true,
841 841
 			),
842 842
 			'end_date' => array(
843
-				'label' 	=> __('Filter by End Date', 'gravityview'),
843
+				'label' 	=> __( 'Filter by End Date', 'gravityview' ),
844 844
 				'class'	=> 'gv-datepicker',
845
-				'desc'	=> __('Show entries submitted before this date. Supports relative dates, such as "now" or "-3 days".', 'gravityview' ),
845
+				'desc'	=> __( 'Show entries submitted before this date. Supports relative dates, such as "now" or "-3 days".', 'gravityview' ),
846 846
 				'type' => 'text',
847 847
 				'value' => '',
848 848
 				'group'	=> 'filter',
849 849
 				'show_in_shortcode' => true,
850 850
 			),
851 851
 			'class' => array(
852
-				'label' 	=> __('CSS Class', 'gravityview'),
853
-				'desc'	=> __('CSS class to add to the wrapping HTML container.', 'gravityview'),
852
+				'label' 	=> __( 'CSS Class', 'gravityview' ),
853
+				'desc'	=> __( 'CSS class to add to the wrapping HTML container.', 'gravityview' ),
854 854
 				'group'	=> 'default',
855 855
 				'type' => 'text',
856 856
 				'value' => '',
857 857
 				'show_in_shortcode' => false,
858 858
 			),
859 859
 			'search_value' => array(
860
-				'label' 	=> __('Search Value', 'gravityview'),
861
-				'desc'	=> __('Define a default search value for the View', 'gravityview'),
860
+				'label' 	=> __( 'Search Value', 'gravityview' ),
861
+				'desc'	=> __( 'Define a default search value for the View', 'gravityview' ),
862 862
 				'type' => 'text',
863 863
 				'value' => '',
864 864
 				'group'	=> 'filter',
865 865
 				'show_in_shortcode' => false,
866 866
 			),
867 867
 			'search_field' => array(
868
-				'label' 	=> __('Search Field', 'gravityview'),
869
-				'desc'	=> __('If Search Value is set, you can define a specific field to search in. Otherwise, all fields will be searched.', 'gravityview'),
868
+				'label' 	=> __( 'Search Field', 'gravityview' ),
869
+				'desc'	=> __( 'If Search Value is set, you can define a specific field to search in. Otherwise, all fields will be searched.', 'gravityview' ),
870 870
 				'type' => 'number',
871 871
 				'value' => '',
872 872
 				'group'	=> 'filter',
873 873
 				'show_in_shortcode' => false,
874 874
 			),
875 875
 			'single_title' => array(
876
-				'label'	=> __('Single Entry Title', 'gravityview'),
876
+				'label'	=> __( 'Single Entry Title', 'gravityview' ),
877 877
 				'type'	=> 'text',
878
-				'desc'	=> __('When viewing a single entry, change the title of the page to this setting. Otherwise, the title will not change between the Multiple Entries and Single Entry views.', 'gravityview'),
878
+				'desc'	=> __( 'When viewing a single entry, change the title of the page to this setting. Otherwise, the title will not change between the Multiple Entries and Single Entry views.', 'gravityview' ),
879 879
 				'group'	=> 'default',
880 880
 				'value'	=> '',
881 881
 				'show_in_shortcode' => false,
882 882
 				'full_width' => true,
883 883
 			),
884 884
 			'back_link_label' => array(
885
-				'label'	=> __('Back Link Label', 'gravityview'),
885
+				'label'	=> __( 'Back Link Label', 'gravityview' ),
886 886
 				'group'	=> 'default',
887
-				'desc'	=> __('The text of the link that returns to the multiple entries view.', 'gravityview'),
887
+				'desc'	=> __( 'The text of the link that returns to the multiple entries view.', 'gravityview' ),
888 888
 				'type'	=> 'text',
889 889
 				'value'	=> '',
890 890
 				'show_in_shortcode' => false,
891 891
 				'full_width' => true,
892 892
 			),
893 893
 			'embed_only' => array(
894
-				'label'	=> __('Prevent Direct Access', 'gravityview'),
894
+				'label'	=> __( 'Prevent Direct Access', 'gravityview' ),
895 895
 				'group'	=> 'default',
896
-				'desc'	=> __('Only allow access to this View when embedded using the shortcode.', 'gravityview'),
896
+				'desc'	=> __( 'Only allow access to this View when embedded using the shortcode.', 'gravityview' ),
897 897
 				'type'	=> 'checkbox',
898 898
 				'value'	=> '',
899 899
 				'tooltip' => false,
@@ -908,7 +908,7 @@  discard block
 block discarded – undo
908 908
 		);
909 909
 
910 910
 		if ( version_compare( \GFCommon::$version, '2.3-beta-4', '>=' ) ) {
911
-			$default_settings['sort_direction']['options']['RAND'] = __( 'Random', 'gravityview' );
911
+			$default_settings[ 'sort_direction' ][ 'options' ][ 'RAND' ] = __( 'Random', 'gravityview' );
912 912
 		}
913 913
 
914 914
 		/**
@@ -920,12 +920,12 @@  discard block
 block discarded – undo
920 920
 		$default_settings = apply_filters( 'gravityview_default_args', $default_settings );
921 921
 
922 922
 		// By default, we only want the key => value pairing, not the whole array.
923
-		if( empty( $with_details ) ) {
923
+		if ( empty( $with_details ) ) {
924 924
 
925 925
 			$defaults = array();
926 926
 
927
-			foreach( $default_settings as $key => $value ) {
928
-				$defaults[ $key ] = $value['value'];
927
+			foreach ( $default_settings as $key => $value ) {
928
+				$defaults[ $key ] = $value[ 'value' ];
929 929
 			}
930 930
 
931 931
 			return $defaults;
@@ -934,12 +934,12 @@  discard block
 block discarded – undo
934 934
 		// But sometimes, we want all the details.
935 935
 		else {
936 936
 
937
-			foreach ($default_settings as $key => $value) {
937
+			foreach ( $default_settings as $key => $value ) {
938 938
 
939 939
 				// If the $group argument is set for the method,
940 940
 				// ignore any settings that aren't in that group.
941
-				if( !empty( $group ) && is_string( $group ) ) {
942
-					if( empty( $value['group'] ) || $value['group'] !== $group ) {
941
+				if ( ! empty( $group ) && is_string( $group ) ) {
942
+					if ( empty( $value[ 'group' ] ) || $value[ 'group' ] !== $group ) {
943 943
 						unset( $default_settings[ $key ] );
944 944
 					}
945 945
 				}
Please login to merge, or discard this patch.
includes/class-gravityview-uninstall.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -42,9 +42,9 @@  discard block
 block discarded – undo
42 42
 		$tables = array();
43 43
 
44 44
 		if ( version_compare( GravityView_GFFormsModel::get_database_version(), '2.3-dev-1', '>=' ) ) {
45
-			$tables []= GFFormsModel::get_entry_meta_table_name();
45
+			$tables [ ] = GFFormsModel::get_entry_meta_table_name();
46 46
 		}
47
-		$tables []= GFFormsModel::get_lead_meta_table_name();
47
+		$tables [ ] = GFFormsModel::get_lead_meta_table_name();
48 48
 
49 49
 		$suppress = $wpdb->suppress_errors();
50 50
 		foreach ( $tables as $meta_table ) {
@@ -70,13 +70,13 @@  discard block
 block discarded – undo
70 70
 		$tables = array();
71 71
 
72 72
 		if ( version_compare( GravityView_GFFormsModel::get_database_version(), '2.3-dev-1', '>=' ) && method_exists( 'GFFormsModel', 'get_entry_notes_table_name' ) ) {
73
-			$tables[] = GFFormsModel::get_entry_notes_table_name();
73
+			$tables[ ] = GFFormsModel::get_entry_notes_table_name();
74 74
 		}
75 75
 
76
-		$tables[] = GFFormsModel::get_lead_notes_table_name();
76
+		$tables[ ] = GFFormsModel::get_lead_notes_table_name();
77 77
 
78
-		$disapproved = __('Disapproved the Entry for GravityView', 'gravityview');
79
-		$approved = __('Approved the Entry for GravityView', 'gravityview');
78
+		$disapproved = __( 'Disapproved the Entry for GravityView', 'gravityview' );
79
+		$approved = __( 'Approved the Entry for GravityView', 'gravityview' );
80 80
 
81 81
 		$suppress = $wpdb->suppress_errors();
82 82
 		foreach ( $tables as $notes_table ) {
Please login to merge, or discard this patch.
future/includes/class-gv-settings-view.php 1 patch
Spacing   +36 added lines, -36 removed lines patch added patch discarded remove patch
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
 
35 35
 		$default_settings = array(
36 36
 			'id' => array(
37
-				'label' => __('View ID', 'gravityview'),
37
+				'label' => __( 'View ID', 'gravityview' ),
38 38
 				'type' => 'number',
39 39
 				'group'	=> 'default',
40 40
 				'value' => NULL,
@@ -42,7 +42,7 @@  discard block
 block discarded – undo
42 42
 				'show_in_shortcode' => false,
43 43
 			),
44 44
 			'page_size' => array(
45
-				'label' 	=> __('Number of entries per page', 'gravityview'),
45
+				'label' 	=> __( 'Number of entries per page', 'gravityview' ),
46 46
 				'type' => 'number',
47 47
 				'class'	=> 'small-text',
48 48
 				'group'	=> 'default',
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
50 50
 				'show_in_shortcode' => true,
51 51
 			),
52 52
 			'offset' => array(
53
-				'label' 	=> __('Offset entries starting from', 'gravityview'),
53
+				'label' 	=> __( 'Offset entries starting from', 'gravityview' ),
54 54
 				'type' => 'number',
55 55
 				'class'	=> 'small-text',
56 56
 				'group'	=> 'default',
@@ -74,8 +74,8 @@  discard block
 block discarded – undo
74 74
 			),
75 75
 			'admin_show_all_statuses' => array(
76 76
 				'label' => __( 'Show all entries to administrators', 'gravityview' ),
77
-				'desc'	=> __('Administrators will be able to see entries with any approval status.', 'gravityview'),
78
-				'tooltip' => __('Logged-out visitors and non-administrators will only see approved entries, while administrators will see entries with all statuses. This makes it easier for administrators to moderate entries from a View.', 'gravityview'),
77
+				'desc'	=> __( 'Administrators will be able to see entries with any approval status.', 'gravityview' ),
78
+				'tooltip' => __( 'Logged-out visitors and non-administrators will only see approved entries, while administrators will see entries with all statuses. This makes it easier for administrators to moderate entries from a View.', 'gravityview' ),
79 79
 				'requires' => 'show_only_approved',
80 80
 				'type' => 'checkbox',
81 81
 				'group'	=> 'default',
@@ -100,40 +100,40 @@  discard block
 block discarded – undo
100 100
 			'user_edit' => array(
101 101
 				'label'	=> __( 'Allow User Edit', 'gravityview' ),
102 102
 				'group'	=> 'default',
103
-				'desc'	=> __('Allow logged-in users to edit entries they created.', 'gravityview'),
103
+				'desc'	=> __( 'Allow logged-in users to edit entries they created.', 'gravityview' ),
104 104
 				'value'	=> 0,
105
-				'tooltip' => __('Display "Edit Entry" fields to non-administrator users if they created the entry. Edit Entry fields will always be displayed to site administrators.', 'gravityview'),
105
+				'tooltip' => __( 'Display "Edit Entry" fields to non-administrator users if they created the entry. Edit Entry fields will always be displayed to site administrators.', 'gravityview' ),
106 106
 				'type'	=> 'checkbox',
107 107
 				'show_in_shortcode' => true,
108 108
 			),
109 109
 			'user_delete' => array(
110 110
 				'label'	=> __( 'Allow User Delete', 'gravityview' ),
111 111
 				'group'	=> 'default',
112
-				'desc'	=> __('Allow logged-in users to delete entries they created.', 'gravityview'),
112
+				'desc'	=> __( 'Allow logged-in users to delete entries they created.', 'gravityview' ),
113 113
 				'value'	=> 0,
114
-				'tooltip' => __('Display "Delete Entry" fields to non-administrator users if they created the entry. Delete Entry fields will always be displayed to site administrators.', 'gravityview'),
114
+				'tooltip' => __( 'Display "Delete Entry" fields to non-administrator users if they created the entry. Delete Entry fields will always be displayed to site administrators.', 'gravityview' ),
115 115
 				'type'	=> 'checkbox',
116 116
 				'show_in_shortcode' => true,
117 117
 			),
118 118
 			'sort_field' => array(
119
-				'label'	=> __('Sort by field', 'gravityview'),
119
+				'label'	=> __( 'Sort by field', 'gravityview' ),
120 120
 				'type' => 'select',
121 121
 				'value' => '',
122 122
 				'group'	=> 'sort',
123 123
 				'options' => array(
124
-					'' => __( 'Default', 'gravityview'),
125
-					'date_created' => __( 'Date Created', 'gravityview'),
124
+					'' => __( 'Default', 'gravityview' ),
125
+					'date_created' => __( 'Date Created', 'gravityview' ),
126 126
 				),
127 127
 				'show_in_shortcode' => true,
128 128
 			),
129 129
 			'sort_direction' => array(
130
-				'label' 	=> __('Sort direction', 'gravityview'),
130
+				'label' 	=> __( 'Sort direction', 'gravityview' ),
131 131
 				'type' => 'select',
132 132
 				'value' => 'ASC',
133 133
 				'group'	=> 'sort',
134 134
 				'options' => array(
135
-					'ASC' => __('ASC', 'gravityview'),
136
-					'DESC' => __('DESC', 'gravityview'),
135
+					'ASC' => __( 'ASC', 'gravityview' ),
136
+					'DESC' => __( 'DESC', 'gravityview' ),
137 137
 				),
138 138
 				'show_in_shortcode' => true,
139 139
 			),
@@ -148,69 +148,69 @@  discard block
 block discarded – undo
148 148
 				'show_in_template' => array( 'default_table', 'preset_business_data', 'preset_issue_tracker', 'preset_resume_board', 'preset_job_board' ),
149 149
 			),
150 150
 			'start_date' => array(
151
-				'label' 	=> __('Filter by Start Date', 'gravityview'),
151
+				'label' 	=> __( 'Filter by Start Date', 'gravityview' ),
152 152
 				'class'	=> 'gv-datepicker',
153
-				'desc'	=> __('Show entries submitted after this date. Supports relative dates, such as "-1 week" or "-1 month".', 'gravityview' ),
153
+				'desc'	=> __( 'Show entries submitted after this date. Supports relative dates, such as "-1 week" or "-1 month".', 'gravityview' ),
154 154
 				'type' => 'text',
155 155
 				'value' => '',
156 156
 				'group'	=> 'filter',
157 157
 				'show_in_shortcode' => true,
158 158
 			),
159 159
 			'end_date' => array(
160
-				'label' 	=> __('Filter by End Date', 'gravityview'),
160
+				'label' 	=> __( 'Filter by End Date', 'gravityview' ),
161 161
 				'class'	=> 'gv-datepicker',
162
-				'desc'	=> __('Show entries submitted before this date. Supports relative dates, such as "now" or "-3 days".', 'gravityview' ),
162
+				'desc'	=> __( 'Show entries submitted before this date. Supports relative dates, such as "now" or "-3 days".', 'gravityview' ),
163 163
 				'type' => 'text',
164 164
 				'value' => '',
165 165
 				'group'	=> 'filter',
166 166
 				'show_in_shortcode' => true,
167 167
 			),
168 168
 			'class' => array(
169
-				'label' 	=> __('CSS Class', 'gravityview'),
170
-				'desc'	=> __('CSS class to add to the wrapping HTML container.', 'gravityview'),
169
+				'label' 	=> __( 'CSS Class', 'gravityview' ),
170
+				'desc'	=> __( 'CSS class to add to the wrapping HTML container.', 'gravityview' ),
171 171
 				'group'	=> 'default',
172 172
 				'type' => 'text',
173 173
 				'value' => '',
174 174
 				'show_in_shortcode' => false,
175 175
 			),
176 176
 			'search_value' => array(
177
-				'label' 	=> __('Search Value', 'gravityview'),
178
-				'desc'	=> __('Define a default search value for the View', 'gravityview'),
177
+				'label' 	=> __( 'Search Value', 'gravityview' ),
178
+				'desc'	=> __( 'Define a default search value for the View', 'gravityview' ),
179 179
 				'type' => 'text',
180 180
 				'value' => '',
181 181
 				'group'	=> 'filter',
182 182
 				'show_in_shortcode' => false,
183 183
 			),
184 184
 			'search_field' => array(
185
-				'label' 	=> __('Search Field', 'gravityview'),
186
-				'desc'	=> __('If Search Value is set, you can define a specific field to search in. Otherwise, all fields will be searched.', 'gravityview'),
185
+				'label' 	=> __( 'Search Field', 'gravityview' ),
186
+				'desc'	=> __( 'If Search Value is set, you can define a specific field to search in. Otherwise, all fields will be searched.', 'gravityview' ),
187 187
 				'type' => 'number',
188 188
 				'value' => '',
189 189
 				'group'	=> 'filter',
190 190
 				'show_in_shortcode' => false,
191 191
 			),
192 192
 			'single_title' => array(
193
-				'label'	=> __('Single Entry Title', 'gravityview'),
193
+				'label'	=> __( 'Single Entry Title', 'gravityview' ),
194 194
 				'type'	=> 'text',
195
-				'desc'	=> __('When viewing a single entry, change the title of the page to this setting. Otherwise, the title will not change between the Multiple Entries and Single Entry views.', 'gravityview'),
195
+				'desc'	=> __( 'When viewing a single entry, change the title of the page to this setting. Otherwise, the title will not change between the Multiple Entries and Single Entry views.', 'gravityview' ),
196 196
 				'group'	=> 'default',
197 197
 				'value'	=> '',
198 198
 				'show_in_shortcode' => false,
199 199
 				'full_width' => true,
200 200
 			),
201 201
 			'back_link_label' => array(
202
-				'label'	=> __('Back Link Label', 'gravityview'),
202
+				'label'	=> __( 'Back Link Label', 'gravityview' ),
203 203
 				'group'	=> 'default',
204
-				'desc'	=> __('The text of the link that returns to the multiple entries view.', 'gravityview'),
204
+				'desc'	=> __( 'The text of the link that returns to the multiple entries view.', 'gravityview' ),
205 205
 				'type'	=> 'text',
206 206
 				'value'	=> '',
207 207
 				'show_in_shortcode' => false,
208 208
 				'full_width' => true,
209 209
 			),
210 210
 			'embed_only' => array(
211
-				'label'	=> __('Prevent Direct Access', 'gravityview'),
211
+				'label'	=> __( 'Prevent Direct Access', 'gravityview' ),
212 212
 				'group'	=> 'default',
213
-				'desc'	=> __('Only allow access to this View when embedded using the shortcode.', 'gravityview'),
213
+				'desc'	=> __( 'Only allow access to this View when embedded using the shortcode.', 'gravityview' ),
214 214
 				'type'	=> 'checkbox',
215 215
 				'value'	=> '',
216 216
 				'tooltip' => false,
@@ -225,7 +225,7 @@  discard block
 block discarded – undo
225 225
 		);
226 226
 
227 227
 		if ( version_compare( \GFCommon::$version, '2.3-beta-4', '>=' ) ) {
228
-			$default_settings['sort_direction']['options']['RAND'] = __( 'Random', 'gravityview' );
228
+			$default_settings[ 'sort_direction' ][ 'options' ][ 'RAND' ] = __( 'Random', 'gravityview' );
229 229
 		}
230 230
 
231 231
 		/**
@@ -245,19 +245,19 @@  discard block
 block discarded – undo
245 245
 		// By default, we only want the key => value pairing, not the whole array.
246 246
 		if ( ! $detailed ) {
247 247
 			$defaults = array();
248
-			foreach( $default_settings as $key => $value ) {
249
-				$defaults[ $key ] = $value['value'];
248
+			foreach ( $default_settings as $key => $value ) {
249
+				$defaults[ $key ] = $value[ 'value' ];
250 250
 			}
251 251
 			return $defaults;
252 252
 
253 253
 		// But sometimes, we want all the details.
254 254
 		} else {
255
-			foreach ($default_settings as $key => $value) {
255
+			foreach ( $default_settings as $key => $value ) {
256 256
 
257 257
 				// If the $group argument is set for the method,
258 258
 				// ignore any settings that aren't in that group.
259 259
 				if ( ! empty( $group ) && is_string( $group ) ) {
260
-					if ( empty( $value['group'] ) || $value['group'] !== $group ) {
260
+					if ( empty( $value[ 'group' ] ) || $value[ 'group' ] !== $group ) {
261 261
 						unset( $default_settings[ $key ] );
262 262
 					}
263 263
 				}
Please login to merge, or discard this patch.