Completed
Push — develop ( 2c39bb...a4e86c )
by Gennady
15:59 queued 10s
created
includes/extensions/delete-entry/class-delete-entry.php 2 patches
Doc Comments   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -222,7 +222,6 @@  discard block
 block discarded – undo
222 222
 	 *
223 223
 	 * @since 1.5.1
224 224
 	 * @param  array 	   $visibility_caps        Array of capabilities to display in field dropdown.
225
-	 * @param  string      $field_type  Type of field options to render (`field` or `widget`)
226 225
 	 * @param  string      $template_id Table slug
227 226
 	 * @param  float       $field_id    GF Field ID - Example: `3`, `5.2`, `entry_link`, `created_by`
228 227
 	 * @param  string      $context     What context are we in? Example: `single` or `directory`
@@ -651,7 +650,7 @@  discard block
 block discarded – undo
651 650
 	 *
652 651
 	 * @param  array $entry Gravity Forms entry array
653 652
 	 * @param array $field Field settings (optional)
654
-	 * @param int|\GV\View $view Pass a View ID to check caps against. If not set, check against current View (@deprecated no longer optional)
653
+	 * @param integer $view Pass a View ID to check caps against. If not set, check against current View (@deprecated no longer optional)
655 654
 	 * @return bool
656 655
 	 */
657 656
 	public static function check_user_cap_delete_entry( $entry, $field = array(), $view = 0 ) {
Please login to merge, or discard this patch.
Spacing   +72 added lines, -72 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, 4 );
58
+		add_action( 'gravityview/edit-entry/publishing-action/after', array( $this, 'add_delete_button' ), 10, 4 );
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
 		add_filter( 'gravityview/field/is_visible', array( $this, 'maybe_not_visible' ), 10, 3 );
64 64
 	}
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
 	 */
72 72
 	static function getInstance() {
73 73
 
74
-		if( empty( self::$instance ) ) {
74
+		if ( empty( self::$instance ) ) {
75 75
 			self::$instance = new self;
76 76
 		}
77 77
 
@@ -151,20 +151,20 @@  discard block
 block discarded – undo
151 151
 	function delete_link_field_options( $field_options, $template_id, $field_id, $context, $input_type ) {
152 152
 
153 153
 		// Always a link, never a filter
154
-		unset( $field_options['show_as_link'], $field_options['search_filter'] );
154
+		unset( $field_options[ 'show_as_link' ], $field_options[ 'search_filter' ] );
155 155
 
156 156
 		// Delete Entry link should only appear to visitors capable of editing entries
157
-		unset( $field_options['only_loggedin'], $field_options['only_loggedin_cap'] );
157
+		unset( $field_options[ 'only_loggedin' ], $field_options[ 'only_loggedin_cap' ] );
158 158
 
159
-		$add_option['delete_link'] = array(
159
+		$add_option[ 'delete_link' ] = array(
160 160
 			'type' => 'text',
161 161
 			'label' => __( 'Delete Link Text', 'gravityview' ),
162 162
 			'desc' => NULL,
163
-			'value' => __('Delete Entry', 'gravityview'),
163
+			'value' => __( 'Delete Entry', 'gravityview' ),
164 164
 			'merge_tags' => true,
165 165
 		);
166 166
 
167
-		$field_options['allow_edit_cap'] = array(
167
+		$field_options[ 'allow_edit_cap' ] = array(
168 168
 			'type' => 'select',
169 169
 			'label' => __( 'Allow the following users to delete the entry:', 'gravityview' ),
170 170
 			'choices' => GravityView_Render_Settings::get_cap_choices( $template_id, $field_id, $context, $input_type ),
@@ -188,8 +188,8 @@  discard block
 block discarded – undo
188 188
 	 */
189 189
 	function add_default_field( $entry_default_fields, $form = array(), $zone = '' ) {
190 190
 
191
-		if( 'edit' !== $zone ) {
192
-			$entry_default_fields['delete_link'] = array(
191
+		if ( 'edit' !== $zone ) {
192
+			$entry_default_fields[ 'delete_link' ] = array(
193 193
 				'label' => __( 'Delete Entry', 'gravityview' ),
194 194
 				'type'  => 'delete_link',
195 195
 				'desc'  => __( 'A link to delete the entry. Respects the Delete Entry permissions.', 'gravityview' ),
@@ -206,7 +206,7 @@  discard block
 block discarded – undo
206 206
 	 */
207 207
 	function add_available_field( $available_fields = array() ) {
208 208
 
209
-		$available_fields['delete_link'] = array(
209
+		$available_fields[ 'delete_link' ] = array(
210 210
 			'label_text' => __( 'Delete Entry', 'gravityview' ),
211 211
 			'field_id' => 'delete_link',
212 212
 			'label_type' => 'field',
@@ -234,12 +234,12 @@  discard block
 block discarded – undo
234 234
 		$caps = $visibility_caps;
235 235
 
236 236
 		// If we're configuring fields in the edit context, we want a limited selection
237
-		if( $field_id === 'delete_link' ) {
237
+		if ( $field_id === 'delete_link' ) {
238 238
 
239 239
 			// Remove other built-in caps.
240
-			unset( $caps['publish_posts'], $caps['gravityforms_view_entries'], $caps['delete_others_posts'] );
240
+			unset( $caps[ 'publish_posts' ], $caps[ 'gravityforms_view_entries' ], $caps[ 'delete_others_posts' ] );
241 241
 
242
-			$caps['read'] = _x('Entry Creator', 'User capability', 'gravityview');
242
+			$caps[ 'read' ] = _x( 'Entry Creator', 'User capability', 'gravityview' );
243 243
 		}
244 244
 
245 245
 		return $caps;
@@ -252,7 +252,7 @@  discard block
 block discarded – undo
252 252
 	 * @param [type] $entry [description]
253 253
 	 */
254 254
 	function set_entry( $entry = null ) {
255
-		$this->entry = empty( $entry ) ? GravityView_View::getInstance()->entries[0] : $entry;
255
+		$this->entry = empty( $entry ) ? GravityView_View::getInstance()->entries[ 0 ] : $entry;
256 256
 	}
257 257
 
258 258
 	/**
@@ -285,7 +285,7 @@  discard block
 block discarded – undo
285 285
 
286 286
 		self::getInstance()->set_entry( $entry );
287 287
 
288
-        $base = GravityView_API::directory_link( $post_id ? : $view_id, true );
288
+        $base = GravityView_API::directory_link( $post_id ?: $view_id, true );
289 289
 
290 290
 		if ( empty( $base ) ) {
291 291
 			gravityview()->log->error( 'Post ID does not exist: {post_id}', array( 'post_id' => $post_id ) );
@@ -293,7 +293,7 @@  discard block
 block discarded – undo
293 293
 		}
294 294
 
295 295
 		// Use the slug instead of the ID for consistent security
296
-		$entry_slug = GravityView_API::get_entry_slug( $entry['id'], $entry );
296
+		$entry_slug = GravityView_API::get_entry_slug( $entry[ 'id' ], $entry );
297 297
 
298 298
 		$actionurl = add_query_arg( array(
299 299
 			'action'	=> 'delete',
@@ -302,7 +302,7 @@  discard block
 block discarded – undo
302 302
             'view_id' => $view_id,
303 303
 		), $base );
304 304
 
305
-		$url = wp_nonce_url( $actionurl, 'delete_'.$entry_slug, 'delete' );
305
+		$url = wp_nonce_url( $actionurl, 'delete_' . $entry_slug, 'delete' );
306 306
 
307 307
 		return $url;
308 308
 	}
@@ -324,7 +324,7 @@  discard block
 block discarded – undo
324 324
 	public function add_delete_button( $form = array(), $entry = array(), $view_id = null, $post_id = null ) {
325 325
 
326 326
 		// Only show the link to those who are allowed to see it.
327
-		if( !self::check_user_cap_delete_entry( $entry, array(), $view_id ) ) {
327
+		if ( ! self::check_user_cap_delete_entry( $entry, array(), $view_id ) ) {
328 328
 			return;
329 329
 		}
330 330
 
@@ -335,13 +335,13 @@  discard block
 block discarded – undo
335 335
 		$show_delete_button = apply_filters( 'gravityview/delete-entry/show-delete-button', true );
336 336
 
337 337
 		// If the button is hidden by the filter, don't show.
338
-		if( !$show_delete_button ) {
338
+		if ( ! $show_delete_button ) {
339 339
 			return;
340 340
 		}
341 341
 
342 342
 		$attributes = array(
343 343
 			'class' => 'btn btn-sm button button-small alignright pull-right btn-danger gv-button-delete',
344
-			'tabindex' => ( GFCommon::$tab_index ++ ),
344
+			'tabindex' => ( GFCommon::$tab_index++ ),
345 345
 			'onclick' => self::get_confirm_dialog(),
346 346
 		);
347 347
 
@@ -365,27 +365,27 @@  discard block
 block discarded – undo
365 365
 	function process_delete() {
366 366
 
367 367
 		// If the form is submitted
368
-		if( isset( $_GET['action'] ) && 'delete' === $_GET['action'] && isset( $_GET['entry_id'] ) ) {
368
+		if ( isset( $_GET[ 'action' ] ) && 'delete' === $_GET[ 'action' ] && isset( $_GET[ 'entry_id' ] ) ) {
369 369
 
370 370
 			// Make sure it's a GravityView request
371
-			$valid_nonce_key = wp_verify_nonce( $_GET['delete'], self::get_nonce_key( $_GET['entry_id'] ) );
371
+			$valid_nonce_key = wp_verify_nonce( $_GET[ 'delete' ], self::get_nonce_key( $_GET[ 'entry_id' ] ) );
372 372
 
373
-			if( ! $valid_nonce_key ) {
373
+			if ( ! $valid_nonce_key ) {
374 374
 				gravityview()->log->debug( 'Delete entry not processed: nonce validation failed.' );
375 375
 				return;
376 376
 			}
377 377
 
378 378
 			// Get the entry slug
379
-			$entry_slug = esc_attr( $_GET['entry_id'] );
379
+			$entry_slug = esc_attr( $_GET[ 'entry_id' ] );
380 380
 
381 381
 			// See if there's an entry there
382 382
 			$entry = gravityview_get_entry( $entry_slug, true, false );
383 383
 
384
-			if( $entry ) {
384
+			if ( $entry ) {
385 385
 
386 386
 				$has_permission = $this->user_can_delete_entry( $entry );
387 387
 
388
-				if( is_wp_error( $has_permission ) ) {
388
+				if ( is_wp_error( $has_permission ) ) {
389 389
 
390 390
 					$messages = array(
391 391
 						'message' => urlencode( $has_permission->get_error_message() ),
@@ -397,7 +397,7 @@  discard block
 block discarded – undo
397 397
 					// Delete the entry
398 398
 					$delete_response = $this->delete_or_trash_entry( $entry );
399 399
 
400
-					if( is_wp_error( $delete_response ) ) {
400
+					if ( is_wp_error( $delete_response ) ) {
401 401
 
402 402
 						$messages = array(
403 403
 							'message' => urlencode( $delete_response->get_error_message() ),
@@ -419,7 +419,7 @@  discard block
 block discarded – undo
419 419
 				gravityview()->log->debug( 'Delete entry failed: there was no entry with the entry slug {entry_slug}', array( 'entry_slug' => $entry_slug ) );
420 420
 
421 421
 				$messages = array(
422
-					'message' => urlencode( __('The entry does not exist.', 'gravityview') ),
422
+					'message' => urlencode( __( 'The entry does not exist.', 'gravityview' ) ),
423 423
 					'status' => 'error',
424 424
 				);
425 425
 			}
@@ -459,18 +459,18 @@  discard block
 block discarded – undo
459 459
 	 */
460 460
 	private function delete_or_trash_entry( $entry ) {
461 461
 
462
-		$entry_id = $entry['id'];
462
+		$entry_id = $entry[ 'id' ];
463 463
 		
464 464
 		$mode = $this->get_delete_mode();
465 465
 
466
-		if( 'delete' === $mode ) {
466
+		if ( 'delete' === $mode ) {
467 467
 
468 468
 			gravityview()->log->debug( 'Starting delete entry: {entry_id}', array( 'entry_id' => $entry_id ) );
469 469
 
470 470
 			// Delete the entry
471 471
 			$delete_response = GFAPI::delete_entry( $entry_id );
472 472
 
473
-			if( ! is_wp_error( $delete_response ) ) {
473
+			if ( ! is_wp_error( $delete_response ) ) {
474 474
 				$delete_response = 'deleted';
475 475
 
476 476
 				/**
@@ -491,8 +491,8 @@  discard block
 block discarded – undo
491 491
 			$trashed = GFAPI::update_entry_property( $entry_id, 'status', 'trash' );
492 492
 			new GravityView_Cache;
493 493
 
494
-			if( ! $trashed ) {
495
-				$delete_response = new WP_Error( 'trash_entry_failed', __('Moving the entry to the trash failed.', 'gravityview' ) );
494
+			if ( ! $trashed ) {
495
+				$delete_response = new WP_Error( 'trash_entry_failed', __( 'Moving the entry to the trash failed.', 'gravityview' ) );
496 496
 			} else {
497 497
 
498 498
 				/**
@@ -523,7 +523,7 @@  discard block
 block discarded – undo
523 523
 	public function process_connected_posts( $entry_id = 0, $entry = array() ) {
524 524
 
525 525
 		// The entry had no connected post
526
-		if( empty( $entry['post_id'] ) ) {
526
+		if ( empty( $entry[ 'post_id' ] ) ) {
527 527
 			return;
528 528
 		}
529 529
 
@@ -534,19 +534,19 @@  discard block
 block discarded – undo
534 534
 		 */
535 535
 		$delete_post = apply_filters( 'gravityview/delete-entry/delete-connected-post', true );
536 536
 		
537
-		if( false === $delete_post ) {
537
+		if ( false === $delete_post ) {
538 538
 			return;
539 539
 		}
540 540
 
541 541
 		$action = current_action();
542 542
 
543
-		if( 'gravityview/delete-entry/deleted' === $action ) {
544
-			$result = wp_delete_post( $entry['post_id'], true );
543
+		if ( 'gravityview/delete-entry/deleted' === $action ) {
544
+			$result = wp_delete_post( $entry[ 'post_id' ], true );
545 545
 		} else {
546
-			$result = wp_trash_post( $entry['post_id'] );
546
+			$result = wp_trash_post( $entry[ 'post_id' ] );
547 547
 		}
548 548
 
549
-		if( false === $result ) {
549
+		if ( false === $result ) {
550 550
 			gravityview()->log->error( '(called by {action}): Error processing the Post connected to the entry.', array( 'action' => $action, 'data' => $entry ) );
551 551
 		} else {
552 552
 			gravityview()->log->debug( '(called by {action}): Successfully processed Post connected to the entry.', array( 'action' => $action, 'data' => $entry ) );
@@ -562,13 +562,13 @@  discard block
 block discarded – undo
562 562
 	public function verify_nonce() {
563 563
 
564 564
 		// No delete entry request was made
565
-		if( empty( $_GET['entry_id'] ) || empty( $_GET['delete'] ) ) {
565
+		if ( empty( $_GET[ 'entry_id' ] ) || empty( $_GET[ 'delete' ] ) ) {
566 566
 			return false;
567 567
 		}
568 568
 
569
-		$nonce_key = self::get_nonce_key( $_GET['entry_id'] );
569
+		$nonce_key = self::get_nonce_key( $_GET[ 'entry_id' ] );
570 570
 
571
-		$valid = wp_verify_nonce( $_GET['delete'], $nonce_key );
571
+		$valid = wp_verify_nonce( $_GET[ 'delete' ], $nonce_key );
572 572
 
573 573
 		/**
574 574
 		 * @filter `gravityview/delete-entry/verify_nonce` Override Delete Entry nonce validation. Return true to declare nonce valid.
@@ -590,7 +590,7 @@  discard block
 block discarded – undo
590 590
 	 */
591 591
 	public static function get_confirm_dialog() {
592 592
 
593
-		$confirm = __('Are you sure you want to delete this entry? This cannot be undone.', 'gravityview');
593
+		$confirm = __( 'Are you sure you want to delete this entry? This cannot be undone.', 'gravityview' );
594 594
 
595 595
 		/**
596 596
 		 * @filter `gravityview/delete-entry/confirm-text` Modify the Delete Entry Javascript confirmation text
@@ -598,7 +598,7 @@  discard block
 block discarded – undo
598 598
 		 */
599 599
 		$confirm = apply_filters( 'gravityview/delete-entry/confirm-text', $confirm );
600 600
 
601
-		return 'return window.confirm(\''. esc_js( $confirm ) .'\');';
601
+		return 'return window.confirm(\'' . esc_js( $confirm ) . '\');';
602 602
 	}
603 603
 
604 604
 	/**
@@ -616,16 +616,16 @@  discard block
 block discarded – undo
616 616
 
617 617
 		$error = NULL;
618 618
 
619
-		if( ! $this->verify_nonce() ) {
620
-			$error = __( 'The link to delete this entry is not valid; it may have expired.', 'gravityview');
619
+		if ( ! $this->verify_nonce() ) {
620
+			$error = __( 'The link to delete this entry is not valid; it may have expired.', 'gravityview' );
621 621
 		}
622 622
 
623
-		if( ! self::check_user_cap_delete_entry( $entry, array(), $view_id ) ) {
624
-			$error = __( 'You do not have permission to delete this entry.', 'gravityview');
623
+		if ( ! self::check_user_cap_delete_entry( $entry, array(), $view_id ) ) {
624
+			$error = __( 'You do not have permission to delete this entry.', 'gravityview' );
625 625
 		}
626 626
 
627
-		if( $entry['status'] === 'trash' ) {
628
-			if( 'trash' === $this->get_delete_mode() ) {
627
+		if ( $entry[ 'status' ] === 'trash' ) {
628
+			if ( 'trash' === $this->get_delete_mode() ) {
629 629
 				$error = __( 'The entry is already in the trash.', 'gravityview' );
630 630
 			} else {
631 631
 				$error = __( 'You cannot delete the entry; it is already in the trash.', 'gravityview' );
@@ -633,7 +633,7 @@  discard block
 block discarded – undo
633 633
 		}
634 634
 
635 635
 		// No errors; everything's fine here!
636
-		if( empty( $error ) ) {
636
+		if ( empty( $error ) ) {
637 637
 			return true;
638 638
 		}
639 639
 
@@ -661,17 +661,17 @@  discard block
 block discarded – undo
661 661
 			$view = \GV\View::by_id( $view_id );
662 662
 		} else {
663 663
 			if ( ! $view instanceof \GV\View ) {
664
-				$view = \GV\View::by_id ( $view );
664
+				$view = \GV\View::by_id( $view );
665 665
 			}
666 666
 			$view_id = $view->ID;
667 667
 		}
668 668
 
669 669
 		$current_user = wp_get_current_user();
670 670
 
671
-		$entry_id = isset( $entry['id'] ) ? $entry['id'] : NULL;
671
+		$entry_id = isset( $entry[ 'id' ] ) ? $entry[ 'id' ] : NULL;
672 672
 
673 673
 		// Or if they can delete any entries (as defined in Gravity Forms), we're good.
674
-		if( GVCommon::has_cap( array( 'gravityforms_delete_entries', 'gravityview_delete_others_entries' ), $entry_id ) ) {
674
+		if ( GVCommon::has_cap( array( 'gravityforms_delete_entries', 'gravityview_delete_others_entries' ), $entry_id ) ) {
675 675
 
676 676
 			gravityview()->log->debug( 'Current user has `gravityforms_delete_entries` or `gravityview_delete_others_entries` capability.' );
677 677
 
@@ -680,20 +680,20 @@  discard block
 block discarded – undo
680 680
 
681 681
 
682 682
 		// If field options are passed, check if current user can view the link
683
-		if( !empty( $field ) ) {
683
+		if ( ! empty( $field ) ) {
684 684
 
685 685
 			// If capability is not defined, something is not right!
686
-			if( empty( $field['allow_edit_cap'] ) ) {
686
+			if ( empty( $field[ 'allow_edit_cap' ] ) ) {
687 687
 
688 688
 				gravityview()->log->error( 'Cannot read delete entry field caps', array( 'data' => $field ) );
689 689
 
690 690
 				return false;
691 691
 			}
692 692
 
693
-			if( GVCommon::has_cap( $field['allow_edit_cap'] ) ) {
693
+			if ( GVCommon::has_cap( $field[ 'allow_edit_cap' ] ) ) {
694 694
 
695 695
 				// Do not return true if cap is read, as we need to check if the current user created the entry
696
-				if( $field['allow_edit_cap'] !== 'read' ) {
696
+				if ( $field[ 'allow_edit_cap' ] !== 'read' ) {
697 697
 					return true;
698 698
 				}
699 699
 
@@ -706,9 +706,9 @@  discard block
 block discarded – undo
706 706
 
707 707
 		}
708 708
 
709
-		if( !isset( $entry['created_by'] ) ) {
709
+		if ( ! isset( $entry[ 'created_by' ] ) ) {
710 710
 
711
-			gravityview()->log->error( 'Entry `created_by` doesn\'t exist.');
711
+			gravityview()->log->error( 'Entry `created_by` doesn\'t exist.' );
712 712
 
713 713
 			return false;
714 714
 		}
@@ -725,7 +725,7 @@  discard block
 block discarded – undo
725 725
 		}
726 726
 
727 727
 		// If the logged-in user is the same as the user who created the entry, we're good.
728
-		if( is_user_logged_in() && intval( $current_user->ID ) === intval( $entry['created_by'] ) ) {
728
+		if ( is_user_logged_in() && intval( $current_user->ID ) === intval( $entry[ 'created_by' ] ) ) {
729 729
 
730 730
 			gravityview()->log->debug( 'User {user_id} created the entry.', array( 'user_id' => $current_user->ID ) );
731 731
 
@@ -750,16 +750,16 @@  discard block
 block discarded – undo
750 750
 	 */
751 751
 	public function display_message( $current_view_id = 0 ) {
752 752
 
753
-		if( empty( $_GET['status'] ) || ! self::verify_nonce() ) {
753
+		if ( empty( $_GET[ 'status' ] ) || ! self::verify_nonce() ) {
754 754
 			return;
755 755
 		}
756 756
 
757 757
 		// Entry wasn't deleted from current View
758
-		if( isset( $_GET['view_id'] ) && intval( $_GET['view_id'] ) !== intval( $current_view_id ) ) {
758
+		if ( isset( $_GET[ 'view_id' ] ) && intval( $_GET[ 'view_id' ] ) !== intval( $current_view_id ) ) {
759 759
 			return;
760 760
 		}
761 761
 
762
-		$status = esc_attr( $_GET['status'] );
762
+		$status = esc_attr( $_GET[ 'status' ] );
763 763
 		$message_from_url = \GV\Utils::_GET( 'message' );
764 764
 		$message_from_url = rawurldecode( stripslashes_deep( $message_from_url ) );
765 765
 		$class = '';
@@ -767,14 +767,14 @@  discard block
 block discarded – undo
767 767
 		switch ( $status ) {
768 768
 			case 'error':
769 769
 				$class = ' gv-error error';
770
-				$error_message = __('There was an error deleting the entry: %s', 'gravityview');
770
+				$error_message = __( 'There was an error deleting the entry: %s', 'gravityview' );
771 771
 				$message = sprintf( $error_message, $message_from_url );
772 772
 				break;
773 773
 			case 'trashed':
774
-				$message = __('The entry was successfully moved to the trash.', 'gravityview');
774
+				$message = __( 'The entry was successfully moved to the trash.', 'gravityview' );
775 775
 				break;
776 776
 			default:
777
-				$message = __('The entry was successfully deleted.', 'gravityview');
777
+				$message = __( 'The entry was successfully deleted.', 'gravityview' );
778 778
 				break;
779 779
 		}
780 780
 
@@ -788,7 +788,7 @@  discard block
 block discarded – undo
788 788
 		$message = apply_filters( 'gravityview/delete-entry/message', esc_attr( $message ), $status, $message_from_url );
789 789
 
790 790
 		// DISPLAY ERROR/SUCCESS MESSAGE
791
-		echo '<div class="gv-notice' . esc_attr( $class ) .'">'. $message .'</div>';
791
+		echo '<div class="gv-notice' . esc_attr( $class ) . '">' . $message . '</div>';
792 792
 	}
793 793
 
794 794
 
Please login to merge, or discard this patch.
includes/extensions/edit-entry/class-edit-entry.php 2 patches
Indentation   +159 added lines, -159 removed lines patch added patch discarded remove patch
@@ -18,84 +18,84 @@  discard block
 block discarded – undo
18 18
 
19 19
 class GravityView_Edit_Entry {
20 20
 
21
-    /**
22
-     * @var string
23
-     */
21
+	/**
22
+	 * @var string
23
+	 */
24 24
 	static $file;
25 25
 
26 26
 	static $instance;
27 27
 
28
-    /**
29
-     * Component instances.
30
-     * @var array
31
-     */
32
-    public $instances = array();
28
+	/**
29
+	 * Component instances.
30
+	 * @var array
31
+	 */
32
+	public $instances = array();
33 33
 
34 34
 
35 35
 	function __construct() {
36 36
 
37
-        self::$file = plugin_dir_path( __FILE__ );
37
+		self::$file = plugin_dir_path( __FILE__ );
38 38
 
39
-        if( is_admin() ) {
40
-            $this->load_components( 'admin' );
41
-        }
39
+		if( is_admin() ) {
40
+			$this->load_components( 'admin' );
41
+		}
42 42
 
43 43
 
44
-        $this->load_components( 'render' );
44
+		$this->load_components( 'render' );
45 45
 
46
-        $this->load_components( 'locking' );
46
+		$this->load_components( 'locking' );
47 47
 
48
-        // If GF User Registration Add-on exists
49
-        $this->load_components( 'user-registration' );
48
+		// If GF User Registration Add-on exists
49
+		$this->load_components( 'user-registration' );
50 50
 
51
-        $this->add_hooks();
51
+		$this->add_hooks();
52 52
 
53 53
 		// Process hooks for addons that may or may not be present
54 54
 		$this->addon_specific_hooks();
55 55
 	}
56 56
 
57 57
 
58
-    static function getInstance() {
58
+	static function getInstance() {
59 59
 
60
-        if( empty( self::$instance ) ) {
61
-            self::$instance = new GravityView_Edit_Entry;
62
-        }
60
+		if( empty( self::$instance ) ) {
61
+			self::$instance = new GravityView_Edit_Entry;
62
+		}
63 63
 
64
-        return self::$instance;
65
-    }
64
+		return self::$instance;
65
+	}
66 66
 
67 67
 
68
-    private function load_components( $component ) {
68
+	private function load_components( $component ) {
69 69
 
70
-        $dir = trailingslashit( self::$file );
70
+		$dir = trailingslashit( self::$file );
71 71
 
72
-        $filename  = $dir . 'class-edit-entry-' . $component . '.php';
73
-        $classname = 'GravityView_Edit_Entry_' . str_replace( ' ', '_', ucwords( str_replace( '-', ' ', $component ) ) );
72
+		$filename  = $dir . 'class-edit-entry-' . $component . '.php';
73
+		$classname = 'GravityView_Edit_Entry_' . str_replace( ' ', '_', ucwords( str_replace( '-', ' ', $component ) ) );
74 74
 
75
-        // Loads component and pass extension's instance so that component can
76
-        // talk each other.
77
-        require_once $filename;
78
-        $this->instances[ $component ] = new $classname( $this );
79
-        $this->instances[ $component ]->load();
75
+		// Loads component and pass extension's instance so that component can
76
+		// talk each other.
77
+		require_once $filename;
78
+		$this->instances[ $component ] = new $classname( $this );
79
+		$this->instances[ $component ]->load();
80 80
 
81
-    }
81
+	}
82 82
 
83
-    private function add_hooks() {
83
+	private function add_hooks() {
84 84
 
85
-        // Add front-end access to Gravity Forms delete file action
86
-        add_action( 'wp_ajax_nopriv_rg_delete_file', array( 'GFForms', 'delete_file') );
85
+		// Add front-end access to Gravity Forms delete file action
86
+		add_action( 'wp_ajax_nopriv_rg_delete_file', array( 'GFForms', 'delete_file') );
87 87
 
88
-        // Make sure this hook is run for non-admins
89
-        add_action( 'wp_ajax_rg_delete_file', array( 'GFForms', 'delete_file') );
88
+		// Make sure this hook is run for non-admins
89
+		add_action( 'wp_ajax_rg_delete_file', array( 'GFForms', 'delete_file') );
90 90
 
91
-        add_filter( 'gravityview_blacklist_field_types', array( $this, 'modify_field_blacklist' ), 10, 2 );
91
+		add_filter( 'gravityview_blacklist_field_types', array( $this, 'modify_field_blacklist' ), 10, 2 );
92 92
 
93
-        // add template path to check for field
94
-        add_filter( 'gravityview_template_paths', array( $this, 'add_template_path' ) );
93
+		// add template path to check for field
94
+		add_filter( 'gravityview_template_paths', array( $this, 'add_template_path' ) );
95 95
 
96 96
 		add_filter( 'gravityview/field/is_visible', array( $this, 'maybe_not_visible' ), 10, 3 );
97 97
 
98
-    }
98
+	}
99 99
 
100 100
 	/**
101 101
 	 * Trigger hooks that are normally run in the admin for Addons, but need to be triggered manually because we're not in the admin
@@ -153,77 +153,77 @@  discard block
 block discarded – undo
153 153
 		return false;
154 154
 	}
155 155
 
156
-    /**
157
-     * Include this extension templates path
158
-     * @param array $file_paths List of template paths ordered
159
-     */
160
-    public function add_template_path( $file_paths ) {
161
-
162
-        // Index 100 is the default GravityView template path.
163
-        $file_paths[ 110 ] = self::$file;
164
-
165
-        return $file_paths;
166
-    }
167
-
168
-    /**
169
-     *
170
-     * Return a well formatted nonce key according to GravityView Edit Entry protocol
171
-     *
172
-     * @param $view_id int GravityView view id
173
-     * @param $form_id int Gravity Forms form id
174
-     * @param $entry_id int Gravity Forms entry id
175
-     * @return string
176
-     */
177
-    public static function get_nonce_key( $view_id, $form_id, $entry_id ) {
178
-        return sprintf( 'edit_%d_%d_%d', $view_id, $form_id, $entry_id );
179
-    }
180
-
181
-
182
-    /**
183
-     * The edit entry link creates a secure link with a nonce
184
-     *
185
-     * It also mimics the URL structure Gravity Forms expects to have so that
186
-     * it formats the display of the edit form like it does in the backend, like
187
-     * "You can edit this post from the post page" fields, for example.
188
-     *
189
-     * @param $entry array Gravity Forms entry object
190
-     * @param $view_id int GravityView view id
191
-     * @param $post_id int GravityView Post ID where View may be embedded {@since 1.9.2}
192
-     * @param string|array $field_values Parameters to pass in to the Edit Entry form to prefill data. Uses the same format as Gravity Forms "Allow field to be populated dynamically" {@since 1.9.2} {@see https://www.gravityhelp.com/documentation/article/allow-field-to-be-populated-dynamically/ }
193
-     * @return string
194
-     */
195
-    public static function get_edit_link( $entry, $view_id, $post_id = null, $field_values = '' ) {
196
-
197
-        $nonce_key = self::get_nonce_key( $view_id, $entry['form_id'], $entry['id']  );
198
-
199
-        $base = gv_entry_link( $entry, $post_id ? : $view_id  );
200
-
201
-        $url = add_query_arg( array(
202
-            'edit' => wp_create_nonce( $nonce_key )
203
-        ), $base );
204
-
205
-        if( $post_id ) {
206
-	        $url = add_query_arg( array( 'gvid' => $view_id ), $url );
207
-        }
208
-
209
-	    /**
210
-	     * Allow passing params to dynamically populate entry with values
211
-	     * @since 1.9.2
212
-	     */
213
-	    if( !empty( $field_values ) ) {
214
-
215
-		    if( is_array( $field_values ) ) {
216
-			    // If already an array, no parse_str() needed
217
-			    $params = $field_values;
218
-		    } else {
219
-			    parse_str( $field_values, $params );
220
-		    }
221
-
222
-		    $url = add_query_arg( $params, $url );
223
-	    }
224
-
225
-        return $url;
226
-    }
156
+	/**
157
+	 * Include this extension templates path
158
+	 * @param array $file_paths List of template paths ordered
159
+	 */
160
+	public function add_template_path( $file_paths ) {
161
+
162
+		// Index 100 is the default GravityView template path.
163
+		$file_paths[ 110 ] = self::$file;
164
+
165
+		return $file_paths;
166
+	}
167
+
168
+	/**
169
+	 *
170
+	 * Return a well formatted nonce key according to GravityView Edit Entry protocol
171
+	 *
172
+	 * @param $view_id int GravityView view id
173
+	 * @param $form_id int Gravity Forms form id
174
+	 * @param $entry_id int Gravity Forms entry id
175
+	 * @return string
176
+	 */
177
+	public static function get_nonce_key( $view_id, $form_id, $entry_id ) {
178
+		return sprintf( 'edit_%d_%d_%d', $view_id, $form_id, $entry_id );
179
+	}
180
+
181
+
182
+	/**
183
+	 * The edit entry link creates a secure link with a nonce
184
+	 *
185
+	 * It also mimics the URL structure Gravity Forms expects to have so that
186
+	 * it formats the display of the edit form like it does in the backend, like
187
+	 * "You can edit this post from the post page" fields, for example.
188
+	 *
189
+	 * @param $entry array Gravity Forms entry object
190
+	 * @param $view_id int GravityView view id
191
+	 * @param $post_id int GravityView Post ID where View may be embedded {@since 1.9.2}
192
+	 * @param string|array $field_values Parameters to pass in to the Edit Entry form to prefill data. Uses the same format as Gravity Forms "Allow field to be populated dynamically" {@since 1.9.2} {@see https://www.gravityhelp.com/documentation/article/allow-field-to-be-populated-dynamically/ }
193
+	 * @return string
194
+	 */
195
+	public static function get_edit_link( $entry, $view_id, $post_id = null, $field_values = '' ) {
196
+
197
+		$nonce_key = self::get_nonce_key( $view_id, $entry['form_id'], $entry['id']  );
198
+
199
+		$base = gv_entry_link( $entry, $post_id ? : $view_id  );
200
+
201
+		$url = add_query_arg( array(
202
+			'edit' => wp_create_nonce( $nonce_key )
203
+		), $base );
204
+
205
+		if( $post_id ) {
206
+			$url = add_query_arg( array( 'gvid' => $view_id ), $url );
207
+		}
208
+
209
+		/**
210
+		 * Allow passing params to dynamically populate entry with values
211
+		 * @since 1.9.2
212
+		 */
213
+		if( !empty( $field_values ) ) {
214
+
215
+			if( is_array( $field_values ) ) {
216
+				// If already an array, no parse_str() needed
217
+				$params = $field_values;
218
+			} else {
219
+				parse_str( $field_values, $params );
220
+			}
221
+
222
+			$url = add_query_arg( $params, $url );
223
+		}
224
+
225
+		return $url;
226
+	}
227 227
 
228 228
 	/**
229 229
 	 * Edit mode doesn't allow certain field types.
@@ -278,19 +278,19 @@  discard block
 block discarded – undo
278 278
 	}
279 279
 
280 280
 
281
-    /**
282
-     * checks if user has permissions to edit a specific entry
283
-     *
284
-     * Needs to be used combined with GravityView_Edit_Entry::user_can_edit_entry for maximum security!!
285
-     *
286
-     * @param  array $entry Gravity Forms entry array
287
-     * @param \GV\View int $view_id ID of the view you want to check visibility against {@since 1.9.2}. Required since 2.0
288
-     * @return bool
289
-     */
290
-    public static function check_user_cap_edit_entry( $entry, $view = 0 ) {
281
+	/**
282
+	 * checks if user has permissions to edit a specific entry
283
+	 *
284
+	 * Needs to be used combined with GravityView_Edit_Entry::user_can_edit_entry for maximum security!!
285
+	 *
286
+	 * @param  array $entry Gravity Forms entry array
287
+	 * @param \GV\View int $view_id ID of the view you want to check visibility against {@since 1.9.2}. Required since 2.0
288
+	 * @return bool
289
+	 */
290
+	public static function check_user_cap_edit_entry( $entry, $view = 0 ) {
291 291
 
292
-        // No permission by default
293
-        $user_can_edit = false;
292
+		// No permission by default
293
+		$user_can_edit = false;
294 294
 
295 295
 		// get user_edit setting
296 296
 		if ( empty( $view ) ) {
@@ -308,59 +308,59 @@  discard block
 block discarded – undo
308 308
 			$user_edit = GVCommon::get_template_setting( $view_id, 'user_edit' );
309 309
 		}
310 310
 
311
-        // If they can edit any entries (as defined in Gravity Forms)
312
-        // Or if they can edit other people's entries
313
-        // Then we're good.
314
-        if( GVCommon::has_cap( array( 'gravityforms_edit_entries', 'gravityview_edit_others_entries' ), $entry['id'] ) ) {
311
+		// If they can edit any entries (as defined in Gravity Forms)
312
+		// Or if they can edit other people's entries
313
+		// Then we're good.
314
+		if( GVCommon::has_cap( array( 'gravityforms_edit_entries', 'gravityview_edit_others_entries' ), $entry['id'] ) ) {
315 315
 
316
-            gravityview()->log->debug( 'User has ability to edit all entries.' );
316
+			gravityview()->log->debug( 'User has ability to edit all entries.' );
317 317
 
318
-            $user_can_edit = true;
318
+			$user_can_edit = true;
319 319
 
320
-        } else if( !isset( $entry['created_by'] ) ) {
320
+		} else if( !isset( $entry['created_by'] ) ) {
321 321
 
322
-            gravityview()->log->error( 'Entry `created_by` doesn\'t exist.');
322
+			gravityview()->log->error( 'Entry `created_by` doesn\'t exist.');
323 323
 
324
-            $user_can_edit = false;
324
+			$user_can_edit = false;
325 325
 
326
-        } else {
326
+		} else {
327 327
 
328 328
 
329
-            $current_user = wp_get_current_user();
329
+			$current_user = wp_get_current_user();
330 330
 
331
-            // User edit is disabled
332
-            if( empty( $user_edit ) ) {
331
+			// User edit is disabled
332
+			if( empty( $user_edit ) ) {
333 333
 
334
-                gravityview()->log->debug( 'User Edit is disabled. Returning false.' );
334
+				gravityview()->log->debug( 'User Edit is disabled. Returning false.' );
335 335
 
336
-                $user_can_edit = false;
337
-            }
336
+				$user_can_edit = false;
337
+			}
338 338
 
339
-            // User edit is enabled and the logged-in user is the same as the user who created the entry. We're good.
340
-            else if( is_user_logged_in() && intval( $current_user->ID ) === intval( $entry['created_by'] ) ) {
339
+			// User edit is enabled and the logged-in user is the same as the user who created the entry. We're good.
340
+			else if( is_user_logged_in() && intval( $current_user->ID ) === intval( $entry['created_by'] ) ) {
341 341
 
342
-                gravityview()->log->debug( 'User {user_id} created the entry.', array( 'user_id', $current_user->ID ) );
342
+				gravityview()->log->debug( 'User {user_id} created the entry.', array( 'user_id', $current_user->ID ) );
343 343
 
344
-                $user_can_edit = true;
344
+				$user_can_edit = true;
345 345
 
346
-            } else if( ! is_user_logged_in() ) {
346
+			} else if( ! is_user_logged_in() ) {
347 347
 
348
-                gravityview()->log->debug( 'No user defined; edit entry requires logged in user' );
349
-            }
348
+				gravityview()->log->debug( 'No user defined; edit entry requires logged in user' );
349
+			}
350 350
 
351
-        }
351
+		}
352 352
 
353
-        /**
354
-         * @filter `gravityview/edit_entry/user_can_edit_entry` Modify whether user can edit an entry.
355
-         * @since 1.15 Added `$entry` and `$view_id` parameters
356
-         * @param[in,out] boolean $user_can_edit Can the current user edit the current entry? (Default: false)
357
-         * @param[in] array $entry Gravity Forms entry array {@since 1.15}
358
-         * @param[in] int $view_id ID of the view you want to check visibility against {@since 1.15}
359
-         */
360
-        $user_can_edit = apply_filters( 'gravityview/edit_entry/user_can_edit_entry', $user_can_edit, $entry, $view_id );
353
+		/**
354
+		 * @filter `gravityview/edit_entry/user_can_edit_entry` Modify whether user can edit an entry.
355
+		 * @since 1.15 Added `$entry` and `$view_id` parameters
356
+		 * @param[in,out] boolean $user_can_edit Can the current user edit the current entry? (Default: false)
357
+		 * @param[in] array $entry Gravity Forms entry array {@since 1.15}
358
+		 * @param[in] int $view_id ID of the view you want to check visibility against {@since 1.15}
359
+		 */
360
+		$user_can_edit = apply_filters( 'gravityview/edit_entry/user_can_edit_entry', $user_can_edit, $entry, $view_id );
361 361
 
362
-        return (bool)$user_can_edit;
363
-    }
362
+		return (bool)$user_can_edit;
363
+	}
364 364
 
365 365
 
366 366
 
Please login to merge, or discard this patch.
Spacing   +18 added lines, -18 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
 
37 37
         self::$file = plugin_dir_path( __FILE__ );
38 38
 
39
-        if( is_admin() ) {
39
+        if ( is_admin() ) {
40 40
             $this->load_components( 'admin' );
41 41
         }
42 42
 
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
 
58 58
     static function getInstance() {
59 59
 
60
-        if( empty( self::$instance ) ) {
60
+        if ( empty( self::$instance ) ) {
61 61
             self::$instance = new GravityView_Edit_Entry;
62 62
         }
63 63
 
@@ -83,10 +83,10 @@  discard block
 block discarded – undo
83 83
     private function add_hooks() {
84 84
 
85 85
         // Add front-end access to Gravity Forms delete file action
86
-        add_action( 'wp_ajax_nopriv_rg_delete_file', array( 'GFForms', 'delete_file') );
86
+        add_action( 'wp_ajax_nopriv_rg_delete_file', array( 'GFForms', 'delete_file' ) );
87 87
 
88 88
         // Make sure this hook is run for non-admins
89
-        add_action( 'wp_ajax_rg_delete_file', array( 'GFForms', 'delete_file') );
89
+        add_action( 'wp_ajax_rg_delete_file', array( 'GFForms', 'delete_file' ) );
90 90
 
91 91
         add_filter( 'gravityview_blacklist_field_types', array( $this, 'modify_field_blacklist' ), 10, 2 );
92 92
 
@@ -103,8 +103,8 @@  discard block
 block discarded – undo
103 103
 	 */
104 104
 	private function addon_specific_hooks() {
105 105
 
106
-		if( class_exists( 'GFSignature' ) && is_callable( array( 'GFSignature', 'get_instance' ) ) ) {
107
-			add_filter('gform_admin_pre_render', array( GFSignature::get_instance(), 'edit_lead_script'));
106
+		if ( class_exists( 'GFSignature' ) && is_callable( array( 'GFSignature', 'get_instance' ) ) ) {
107
+			add_filter( 'gform_admin_pre_render', array( GFSignature::get_instance(), 'edit_lead_script' ) );
108 108
 		}
109 109
 
110 110
 	}
@@ -194,15 +194,15 @@  discard block
 block discarded – undo
194 194
      */
195 195
     public static function get_edit_link( $entry, $view_id, $post_id = null, $field_values = '' ) {
196 196
 
197
-        $nonce_key = self::get_nonce_key( $view_id, $entry['form_id'], $entry['id']  );
197
+        $nonce_key = self::get_nonce_key( $view_id, $entry[ 'form_id' ], $entry[ 'id' ] );
198 198
 
199
-        $base = gv_entry_link( $entry, $post_id ? : $view_id  );
199
+        $base = gv_entry_link( $entry, $post_id ?: $view_id );
200 200
 
201 201
         $url = add_query_arg( array(
202 202
             'edit' => wp_create_nonce( $nonce_key )
203 203
         ), $base );
204 204
 
205
-        if( $post_id ) {
205
+        if ( $post_id ) {
206 206
 	        $url = add_query_arg( array( 'gvid' => $view_id ), $url );
207 207
         }
208 208
 
@@ -210,9 +210,9 @@  discard block
 block discarded – undo
210 210
 	     * Allow passing params to dynamically populate entry with values
211 211
 	     * @since 1.9.2
212 212
 	     */
213
-	    if( !empty( $field_values ) ) {
213
+	    if ( ! empty( $field_values ) ) {
214 214
 
215
-		    if( is_array( $field_values ) ) {
215
+		    if ( is_array( $field_values ) ) {
216 216
 			    // If already an array, no parse_str() needed
217 217
 			    $params = $field_values;
218 218
 		    } else {
@@ -233,7 +233,7 @@  discard block
 block discarded – undo
233 233
 	 */
234 234
 	public function modify_field_blacklist( $fields = array(), $context = NULL ) {
235 235
 
236
-		if( empty( $context ) || $context !== 'edit' ) {
236
+		if ( empty( $context ) || $context !== 'edit' ) {
237 237
 			return $fields;
238 238
 		}
239 239
 
@@ -311,15 +311,15 @@  discard block
 block discarded – undo
311 311
         // If they can edit any entries (as defined in Gravity Forms)
312 312
         // Or if they can edit other people's entries
313 313
         // Then we're good.
314
-        if( GVCommon::has_cap( array( 'gravityforms_edit_entries', 'gravityview_edit_others_entries' ), $entry['id'] ) ) {
314
+        if ( GVCommon::has_cap( array( 'gravityforms_edit_entries', 'gravityview_edit_others_entries' ), $entry[ 'id' ] ) ) {
315 315
 
316 316
             gravityview()->log->debug( 'User has ability to edit all entries.' );
317 317
 
318 318
             $user_can_edit = true;
319 319
 
320
-        } else if( !isset( $entry['created_by'] ) ) {
320
+        } else if ( ! isset( $entry[ 'created_by' ] ) ) {
321 321
 
322
-            gravityview()->log->error( 'Entry `created_by` doesn\'t exist.');
322
+            gravityview()->log->error( 'Entry `created_by` doesn\'t exist.' );
323 323
 
324 324
             $user_can_edit = false;
325 325
 
@@ -329,7 +329,7 @@  discard block
 block discarded – undo
329 329
             $current_user = wp_get_current_user();
330 330
 
331 331
             // User edit is disabled
332
-            if( empty( $user_edit ) ) {
332
+            if ( empty( $user_edit ) ) {
333 333
 
334 334
                 gravityview()->log->debug( 'User Edit is disabled. Returning false.' );
335 335
 
@@ -337,13 +337,13 @@  discard block
 block discarded – undo
337 337
             }
338 338
 
339 339
             // User edit is enabled and the logged-in user is the same as the user who created the entry. We're good.
340
-            else if( is_user_logged_in() && intval( $current_user->ID ) === intval( $entry['created_by'] ) ) {
340
+            else if ( is_user_logged_in() && intval( $current_user->ID ) === intval( $entry[ 'created_by' ] ) ) {
341 341
 
342 342
                 gravityview()->log->debug( 'User {user_id} created the entry.', array( 'user_id', $current_user->ID ) );
343 343
 
344 344
                 $user_can_edit = true;
345 345
 
346
-            } else if( ! is_user_logged_in() ) {
346
+            } else if ( ! is_user_logged_in() ) {
347 347
 
348 348
                 gravityview()->log->debug( 'No user defined; edit entry requires logged in user' );
349 349
             }
Please login to merge, or discard this patch.
includes/extensions/edit-entry/class-edit-entry-locking.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
 			return;
32 32
 		}
33 33
 
34
-		$min = defined( 'SCRIPT_DEBUG' ) && SCRIPT_DEBUG || isset( $_GET['gform_debug'] ) ? '' : '.min';
34
+		$min = defined( 'SCRIPT_DEBUG' ) && SCRIPT_DEBUG || isset( $_GET[ 'gform_debug' ] ) ? '' : '.min';
35 35
 		$locking_path = GFCommon::get_base_url() . '/includes/locking/';
36 36
 
37 37
 		wp_enqueue_script( 'gforms_locking', $locking_path . "js/locking{$min}.js", array( 'jquery', 'heartbeat' ), GFCommon::$version );
@@ -40,20 +40,20 @@  discard block
 block discarded – undo
40 40
 		$translations = array_map( 'wp_strip_all_tags', $this->get_strings() );
41 41
 
42 42
 		$strings = array(
43
-			'noResponse'    => $translations['no_response'],
44
-			'requestAgain'  => $translations['request_again'],
45
-			'requestError'  => $translations['request_error'],
46
-			'gainedControl' => $translations['gained_control'],
47
-			'rejected'      => $translations['request_rejected'],
48
-			'pending'       => $translations['request_pending'],
43
+			'noResponse'    => $translations[ 'no_response' ],
44
+			'requestAgain'  => $translations[ 'request_again' ],
45
+			'requestError'  => $translations[ 'request_error' ],
46
+			'gainedControl' => $translations[ 'gained_control' ],
47
+			'rejected'      => $translations[ 'request_rejected' ],
48
+			'pending'       => $translations[ 'request_pending' ],
49 49
 		);
50 50
 
51
-		$lock_user_id = $this->check_lock( $entry['id'] );
51
+		$lock_user_id = $this->check_lock( $entry[ 'id' ] );
52 52
 
53 53
 		$vars = array(
54 54
 			'hasLock'    => ! $lock_user_id ? 1 : 0,
55 55
 			'lockUI'     => $this->get_lock_ui( $lock_user_id ),
56
-			'objectID'   => $entry['id'],
56
+			'objectID'   => $entry[ 'id' ],
57 57
 			'objectType' => 'entry',
58 58
 			'strings'    => $strings,
59 59
 		);
@@ -173,11 +173,11 @@  discard block
 block discarded – undo
173 173
 		global $wp;
174 174
 		$current_url = add_query_arg( $wp->query_string, '', home_url( $wp->request ) );
175 175
 
176
-		if ( isset( $_GET['get-edit-lock'] ) ) {
176
+		if ( isset( $_GET[ 'get-edit-lock' ] ) ) {
177 177
 			$this->set_lock( $entry_id );
178 178
 			echo '<script>window.location = ' . json_encode( remove_query_arg( 'get-edit-lock', $current_url ) ) . ';</script>';
179 179
 			exit();
180
-		} else if ( isset( $_GET['release-edit-lock'] ) ) {
180
+		} else if ( isset( $_GET[ 'release-edit-lock' ] ) ) {
181 181
 			$this->delete_lock_meta( $entry_id );
182 182
 			$current_url = remove_query_arg( 'edit', $current_url );
183 183
 			echo '<script>window.location = ' . json_encode( remove_query_arg( 'release-edit-lock', $current_url ) ) . ';</script>';
Please login to merge, or discard this patch.
includes/extensions/edit-entry/class-edit-entry-render.php 1 patch
Spacing   +243 added lines, -243 removed lines patch added patch discarded remove patch
@@ -135,16 +135,16 @@  discard block
 block discarded – undo
135 135
 	function load() {
136 136
 
137 137
 		/** @define "GRAVITYVIEW_DIR" "../../../" */
138
-		include_once( GRAVITYVIEW_DIR .'includes/class-admin-approve-entries.php' );
138
+		include_once( GRAVITYVIEW_DIR . 'includes/class-admin-approve-entries.php' );
139 139
 
140 140
 		// Don't display an embedded form when editing an entry
141 141
 		add_action( 'wp_head', array( $this, 'prevent_render_form' ) );
142 142
 		add_action( 'wp_footer', array( $this, 'prevent_render_form' ) );
143 143
 
144 144
 		// Stop Gravity Forms processing what is ours!
145
-		add_filter( 'wp', array( $this, 'prevent_maybe_process_form'), 8 );
145
+		add_filter( 'wp', array( $this, 'prevent_maybe_process_form' ), 8 );
146 146
 
147
-		add_filter( 'gravityview_is_edit_entry', array( $this, 'is_edit_entry') );
147
+		add_filter( 'gravityview_is_edit_entry', array( $this, 'is_edit_entry' ) );
148 148
 
149 149
 		add_action( 'gravityview_edit_entry', array( $this, 'init' ), 10, 4 );
150 150
 
@@ -155,7 +155,7 @@  discard block
 block discarded – undo
155 155
 		add_filter( 'gform_plupload_settings', array( $this, 'modify_fileupload_settings' ), 10, 3 );
156 156
 
157 157
 		// Add fields expected by GFFormDisplay::validate()
158
-		add_filter( 'gform_pre_validation', array( $this, 'gform_pre_validation') );
158
+		add_filter( 'gform_pre_validation', array( $this, 'gform_pre_validation' ) );
159 159
 
160 160
 		// Fix multiselect value for GF 2.2
161 161
 		add_filter( 'gravityview/edit_entry/field_value_multiselect', array( $this, 'fix_multiselect_value_serialization' ), 10, 3 );
@@ -172,8 +172,8 @@  discard block
 block discarded – undo
172 172
 	 * @return void
173 173
 	 */
174 174
 	public function prevent_render_form() {
175
-		if( $this->is_edit_entry() ) {
176
-			if( 'wp_head' === current_filter() ) {
175
+		if ( $this->is_edit_entry() ) {
176
+			if ( 'wp_head' === current_filter() ) {
177 177
 				add_filter( 'gform_shortcode_form', '__return_empty_string' );
178 178
 			} else {
179 179
 				remove_filter( 'gform_shortcode_form', '__return_empty_string' );
@@ -188,14 +188,14 @@  discard block
 block discarded – undo
188 188
 	 */
189 189
 	public function prevent_maybe_process_form() {
190 190
 
191
-	    if( ! $this->is_edit_entry_submission() ) {
191
+	    if ( ! $this->is_edit_entry_submission() ) {
192 192
 			return;
193 193
 		}
194 194
 
195 195
 		gravityview()->log->debug( 'GravityView_Edit_Entry[prevent_maybe_process_form] Removing GFForms::maybe_process_form() action.' );
196 196
 
197
-		remove_action( 'wp',  array( 'RGForms', 'maybe_process_form'), 9 );
198
-		remove_action( 'wp',  array( 'GFForms', 'maybe_process_form'), 9 );
197
+		remove_action( 'wp', array( 'RGForms', 'maybe_process_form' ), 9 );
198
+		remove_action( 'wp', array( 'GFForms', 'maybe_process_form' ), 9 );
199 199
 	}
200 200
 
201 201
 	/**
@@ -204,7 +204,7 @@  discard block
 block discarded – undo
204 204
 	 */
205 205
 	public function is_edit_entry() {
206 206
 
207
-		$is_edit_entry = GravityView_frontend::is_single_entry() && ! empty( $_GET['edit'] );
207
+		$is_edit_entry = GravityView_frontend::is_single_entry() && ! empty( $_GET[ 'edit' ] );
208 208
 
209 209
 		return ( $is_edit_entry || $this->is_edit_entry_submission() );
210 210
 	}
@@ -215,7 +215,7 @@  discard block
 block discarded – undo
215 215
 	 * @return boolean
216 216
 	 */
217 217
 	public function is_edit_entry_submission() {
218
-		return !empty( $_POST[ self::$nonce_field ] );
218
+		return ! empty( $_POST[ self::$nonce_field ] );
219 219
 	}
220 220
 
221 221
 	/**
@@ -228,16 +228,16 @@  discard block
 block discarded – undo
228 228
 
229 229
 
230 230
 		$entries = $gravityview_view->getEntries();
231
-	    self::$original_entry = $entries[0];
232
-	    $this->entry = $entries[0];
231
+	    self::$original_entry = $entries[ 0 ];
232
+	    $this->entry = $entries[ 0 ];
233 233
 
234 234
 		self::$original_form = $gravityview_view->getForm();
235 235
 		$this->form = $gravityview_view->getForm();
236
-		$this->form_id = $this->entry['form_id'];
236
+		$this->form_id = $this->entry[ 'form_id' ];
237 237
 		$this->view_id = $gravityview_view->getViewId();
238 238
 		$this->post_id = \GV\Utils::get( $post, 'ID', null );
239 239
 
240
-		self::$nonce_key = GravityView_Edit_Entry::get_nonce_key( $this->view_id, $this->form_id, $this->entry['id'] );
240
+		self::$nonce_key = GravityView_Edit_Entry::get_nonce_key( $this->view_id, $this->form_id, $this->entry[ 'id' ] );
241 241
 	}
242 242
 
243 243
 
@@ -296,9 +296,9 @@  discard block
 block discarded – undo
296 296
 	private function print_scripts() {
297 297
 		$gravityview_view = GravityView_View::getInstance();
298 298
 
299
-		wp_register_script( 'gform_gravityforms', GFCommon::get_base_url().'/js/gravityforms.js', array( 'jquery', 'gform_json', 'gform_placeholder', 'sack', 'plupload-all', 'gravityview-fe-view' ) );
299
+		wp_register_script( 'gform_gravityforms', GFCommon::get_base_url() . '/js/gravityforms.js', array( 'jquery', 'gform_json', 'gform_placeholder', 'sack', 'plupload-all', 'gravityview-fe-view' ) );
300 300
 
301
-		GFFormDisplay::enqueue_form_scripts( $gravityview_view->getForm(), false);
301
+		GFFormDisplay::enqueue_form_scripts( $gravityview_view->getForm(), false );
302 302
 
303 303
 		wp_localize_script( 'gravityview-fe-view', 'gvGlobals', array( 'cookiepath' => COOKIEPATH ) );
304 304
 
@@ -314,19 +314,19 @@  discard block
 block discarded – undo
314 314
 	 */
315 315
 	private function process_save( $gv_data ) {
316 316
 
317
-		if ( empty( $_POST ) || ! isset( $_POST['lid'] ) ) {
317
+		if ( empty( $_POST ) || ! isset( $_POST[ 'lid' ] ) ) {
318 318
 			return;
319 319
 		}
320 320
 
321 321
 		// Make sure the entry, view, and form IDs are all correct
322 322
 		$valid = $this->verify_nonce();
323 323
 
324
-		if ( !$valid ) {
324
+		if ( ! $valid ) {
325 325
 			gravityview()->log->error( 'Nonce validation failed.' );
326 326
 			return;
327 327
 		}
328 328
 
329
-		if ( $this->entry['id'] !== $_POST['lid'] ) {
329
+		if ( $this->entry[ 'id' ] !== $_POST[ 'lid' ] ) {
330 330
 			gravityview()->log->error( 'Entry ID did not match posted entry ID.' );
331 331
 			return;
332 332
 		}
@@ -337,7 +337,7 @@  discard block
 block discarded – undo
337 337
 
338 338
 		$this->validate();
339 339
 
340
-		if( $this->is_valid ) {
340
+		if ( $this->is_valid ) {
341 341
 
342 342
 			gravityview()->log->debug( 'Submission is valid.' );
343 343
 
@@ -349,15 +349,15 @@  discard block
 block discarded – undo
349 349
 			/**
350 350
 			 * @hack to avoid the capability validation of the method save_lead for GF 1.9+
351 351
 			 */
352
-			unset( $_GET['page'] );
352
+			unset( $_GET[ 'page' ] );
353 353
 
354
-			$date_created = $this->entry['date_created'];
354
+			$date_created = $this->entry[ 'date_created' ];
355 355
 
356 356
 			/**
357 357
 			 * @hack to force Gravity Forms to use $read_value_from_post in GFFormsModel::save_lead()
358 358
 			 * @since 1.17.2
359 359
 			 */
360
-			unset( $this->entry['date_created'] );
360
+			unset( $this->entry[ 'date_created' ] );
361 361
 
362 362
 			/**
363 363
 			 * @action `gravityview/edit_entry/before_update` Perform an action after the entry has been updated using Edit Entry
@@ -367,14 +367,14 @@  discard block
 block discarded – undo
367 367
 			 * @param GravityView_Edit_Entry_Render $this This object
368 368
 			 * @param GravityView_View_Data $gv_data The View data
369 369
 			 */
370
-			do_action( 'gravityview/edit_entry/before_update', $form, $this->entry['id'], $this, $gv_data );
370
+			do_action( 'gravityview/edit_entry/before_update', $form, $this->entry[ 'id' ], $this, $gv_data );
371 371
 
372 372
 			GFFormsModel::save_lead( $form, $this->entry );
373 373
 
374 374
 	        // Delete the values for hidden inputs
375 375
 	        $this->unset_hidden_field_values();
376 376
 			
377
-			$this->entry['date_created'] = $date_created;
377
+			$this->entry[ 'date_created' ] = $date_created;
378 378
 
379 379
 			// Process calculation fields
380 380
 			$this->update_calculation_fields();
@@ -399,7 +399,7 @@  discard block
 block discarded – undo
399 399
 			 * @param GravityView_Edit_Entry_Render $this This object
400 400
 			 * @param GravityView_View_Data $gv_data The View data
401 401
 			 */
402
-			do_action( 'gravityview/edit_entry/after_update', $this->form, $this->entry['id'], $this, $gv_data );
402
+			do_action( 'gravityview/edit_entry/after_update', $this->form, $this->entry[ 'id' ], $this, $gv_data );
403 403
 
404 404
 		} else {
405 405
 			gravityview()->log->error( 'Submission is NOT valid.', array( 'entry' => $this->entry ) );
@@ -435,10 +435,10 @@  discard block
 block discarded – undo
435 435
 
436 436
 		if ( version_compare( GravityView_GFFormsModel::get_database_version(), '2.3-dev-1', '>=' ) && method_exists( 'GFFormsModel', 'get_entry_meta_table_name' ) ) {
437 437
 			$entry_meta_table = GFFormsModel::get_entry_meta_table_name();
438
-			$current_fields = $wpdb->get_results( $wpdb->prepare( "SELECT * FROM $entry_meta_table WHERE entry_id=%d", $this->entry['id'] ) );
438
+			$current_fields = $wpdb->get_results( $wpdb->prepare( "SELECT * FROM $entry_meta_table WHERE entry_id=%d", $this->entry[ 'id' ] ) );
439 439
 		} else {
440 440
 			$lead_detail_table = GFFormsModel::get_lead_details_table_name();
441
-			$current_fields = $wpdb->get_results( $wpdb->prepare( "SELECT * FROM $lead_detail_table WHERE lead_id=%d", $this->entry['id'] ) );
441
+			$current_fields = $wpdb->get_results( $wpdb->prepare( "SELECT * FROM $lead_detail_table WHERE lead_id=%d", $this->entry[ 'id' ] ) );
442 442
 		}
443 443
 
444 444
 	    foreach ( $this->entry as $input_id => $field_value ) {
@@ -456,11 +456,11 @@  discard block
 block discarded – undo
456 456
 
457 457
 				$empty_value = $field->get_value_save_entry(
458 458
 					is_array( $field->get_entry_inputs() ) ? array() : '',
459
-					$this->form, '', $this->entry['id'], $this->entry
459
+					$this->form, '', $this->entry[ 'id' ], $this->entry
460 460
 				);
461 461
 
462 462
 				if ( $field->has_calculation() ) {
463
-					$this->unset_hidden_calculations[] = $field->id; // Unset
463
+					$this->unset_hidden_calculations[ ] = $field->id; // Unset
464 464
 					$empty_value = '';
465 465
 				}
466 466
 
@@ -489,7 +489,7 @@  discard block
 block discarded – undo
489 489
 	private function preset_approval_fields() {
490 490
 		$has_approved_field = false;
491 491
 
492
-		foreach ( self::$original_form['fields'] as $field ) {
492
+		foreach ( self::$original_form[ 'fields' ] as $field ) {
493 493
 			if ( $field->gravityview_approved ) {
494 494
 				$has_approved_field = true;
495 495
 				break;
@@ -502,7 +502,7 @@  discard block
 block discarded – undo
502 502
 
503 503
 		$is_field_hidden = true;
504 504
 
505
-		foreach ( $this->form['fields'] as $field ) {
505
+		foreach ( $this->form[ 'fields' ] as $field ) {
506 506
 			if ( $field->gravityview_approved ) {
507 507
 				$is_field_hidden = false;
508 508
 				break;
@@ -527,7 +527,7 @@  discard block
 block discarded – undo
527 527
 		
528 528
 		remove_filter( 'gravityview/approve_entries/update_unapproved_meta', array( $this, 'prevent_update_unapproved_meta' ), 9 );
529 529
 		
530
-		if ( ! $value = gform_get_meta( $entry['id'], 'is_approved' ) ) {
530
+		if ( ! $value = gform_get_meta( $entry[ 'id' ], 'is_approved' ) ) {
531 531
 			
532 532
 			$value = GravityView_Entry_Approval_Status::UNAPPROVED;
533 533
 			
@@ -592,7 +592,7 @@  discard block
 block discarded – undo
592 592
 		}
593 593
 
594 594
 		/** No file is being uploaded. */
595
-		if ( empty( $_FILES[ $input_name ]['name'] ) ) {
595
+		if ( empty( $_FILES[ $input_name ][ 'name' ] ) ) {
596 596
 			/** So return the original upload */
597 597
 			return $entry[ $input_id ];
598 598
 		}
@@ -610,11 +610,11 @@  discard block
 block discarded – undo
610 610
 	 * @return mixed
611 611
 	 */
612 612
 	public function modify_fileupload_settings( $plupload_init, $form_id, $instance ) {
613
-		if( ! $this->is_edit_entry() ) {
613
+		if ( ! $this->is_edit_entry() ) {
614 614
 			return $plupload_init;
615 615
 		}
616 616
 
617
-		$plupload_init['gf_vars']['max_files'] = 0;
617
+		$plupload_init[ 'gf_vars' ][ 'max_files' ] = 0;
618 618
 
619 619
 		return $plupload_init;
620 620
 	}
@@ -629,27 +629,27 @@  discard block
 block discarded – undo
629 629
 		$form = $this->filter_conditional_logic( $this->form );
630 630
 
631 631
 	    /** @var GF_Field $field */
632
-		foreach( $form['fields'] as $k => &$field ) {
632
+		foreach ( $form[ 'fields' ] as $k => &$field ) {
633 633
 
634 634
 			/**
635 635
 			 * Remove the fields with calculation formulas before save to avoid conflicts with GF logic
636 636
 			 * @since 1.16.3
637 637
 			 * @var GF_Field $field
638 638
 			 */
639
-			if( $field->has_calculation() ) {
640
-				unset( $form['fields'][ $k ] );
639
+			if ( $field->has_calculation() ) {
640
+				unset( $form[ 'fields' ][ $k ] );
641 641
 			}
642 642
 
643 643
 			$field->adminOnly = false;
644 644
 
645
-			if( isset( $field->inputs ) && is_array( $field->inputs ) ) {
646
-				foreach( $field->inputs as $key => $input ) {
647
-				    $field->inputs[ $key ][ 'id' ] = (string)$input['id'];
645
+			if ( isset( $field->inputs ) && is_array( $field->inputs ) ) {
646
+				foreach ( $field->inputs as $key => $input ) {
647
+				    $field->inputs[ $key ][ 'id' ] = (string)$input[ 'id' ];
648 648
 				}
649 649
 			}
650 650
 		}
651 651
 
652
-		$form['fields'] = array_values( $form['fields'] );
652
+		$form[ 'fields' ] = array_values( $form[ 'fields' ] );
653 653
 
654 654
 		return $form;
655 655
 	}
@@ -661,14 +661,14 @@  discard block
 block discarded – undo
661 661
 		$update = false;
662 662
 
663 663
 		// get the most up to date entry values
664
-		$entry = GFAPI::get_entry( $this->entry['id'] );
664
+		$entry = GFAPI::get_entry( $this->entry[ 'id' ] );
665 665
 
666 666
 		if ( version_compare( GravityView_GFFormsModel::get_database_version(), '2.3-dev-1', '>=' ) && method_exists( 'GFFormsModel', 'get_entry_meta_table_name' ) ) {
667 667
 			$entry_meta_table = GFFormsModel::get_entry_meta_table_name();
668
-			$current_fields = $wpdb->get_results( $wpdb->prepare( "SELECT * FROM $entry_meta_table WHERE entry_id=%d", $entry['id'] ) );
668
+			$current_fields = $wpdb->get_results( $wpdb->prepare( "SELECT * FROM $entry_meta_table WHERE entry_id=%d", $entry[ 'id' ] ) );
669 669
 		} else {
670 670
 			$lead_detail_table = GFFormsModel::get_lead_details_table_name();
671
-			$current_fields = $wpdb->get_results( $wpdb->prepare( "SELECT * FROM $lead_detail_table WHERE lead_id=%d", $entry['id'] ) );
671
+			$current_fields = $wpdb->get_results( $wpdb->prepare( "SELECT * FROM $lead_detail_table WHERE lead_id=%d", $entry[ 'id' ] ) );
672 672
 		}
673 673
 
674 674
 
@@ -685,24 +685,24 @@  discard block
 block discarded – undo
685 685
 				$inputs = $field->get_entry_inputs();
686 686
 				if ( is_array( $inputs ) ) {
687 687
 				    foreach ( $inputs as $input ) {
688
-						list( $field_id, $input_id ) = rgexplode( '.', $input['id'], 2 );
688
+						list( $field_id, $input_id ) = rgexplode( '.', $input[ 'id' ], 2 );
689 689
 
690 690
 						if ( 'product' === $field->type ) {
691
-							$input_name = 'input_' . str_replace( '.', '_', $input['id'] );
691
+							$input_name = 'input_' . str_replace( '.', '_', $input[ 'id' ] );
692 692
 
693 693
 							// Only allow quantity to be set if it's allowed to be edited
694 694
 							if ( in_array( $field_id, $allowed_fields ) && $input_id == 3 ) {
695 695
 							} else { // otherwise set to what it previously was
696
-								$_POST[ $input_name ] = $entry[ $input['id'] ];
696
+								$_POST[ $input_name ] = $entry[ $input[ 'id' ] ];
697 697
 							}
698 698
 						} else {
699 699
 							// Set to what it previously was if it's not editable
700 700
 							if ( ! in_array( $field_id, $allowed_fields ) ) {
701
-								$_POST[ $input_name ] = $entry[ $input['id'] ];
701
+								$_POST[ $input_name ] = $entry[ $input[ 'id' ] ];
702 702
 							}
703 703
 						}
704 704
 
705
-						GFFormsModel::save_input( $form, $field, $entry, $current_fields, $input['id'] );
705
+						GFFormsModel::save_input( $form, $field, $entry, $current_fields, $input[ 'id' ] );
706 706
 				    }
707 707
 				} else {
708 708
 					// Set to what it previously was if it's not editable
@@ -742,19 +742,19 @@  discard block
 block discarded – undo
742 742
 
743 743
 		$input_name = 'input_' . $field_id;
744 744
 
745
-		if ( !empty( $_FILES[ $input_name ]['name'] ) ) {
745
+		if ( ! empty( $_FILES[ $input_name ][ 'name' ] ) ) {
746 746
 
747 747
 			// We have a new image
748 748
 
749
-			$value = RGFormsModel::prepare_value( $form, $field, $value, $input_name, $entry['id'] );
749
+			$value = RGFormsModel::prepare_value( $form, $field, $value, $input_name, $entry[ 'id' ] );
750 750
 
751 751
 			$ary = ! empty( $value ) ? explode( '|:|', $value ) : array();
752 752
 	        $ary = stripslashes_deep( $ary );
753 753
 			$img_url = \GV\Utils::get( $ary, 0 );
754 754
 
755
-			$img_title       = count( $ary ) > 1 ? $ary[1] : '';
756
-			$img_caption     = count( $ary ) > 2 ? $ary[2] : '';
757
-			$img_description = count( $ary ) > 3 ? $ary[3] : '';
755
+			$img_title       = count( $ary ) > 1 ? $ary[ 1 ] : '';
756
+			$img_caption     = count( $ary ) > 2 ? $ary[ 2 ] : '';
757
+			$img_description = count( $ary ) > 3 ? $ary[ 3 ] : '';
758 758
 
759 759
 			$image_meta = array(
760 760
 				'post_excerpt' => $img_caption,
@@ -763,7 +763,7 @@  discard block
 block discarded – undo
763 763
 
764 764
 			//adding title only if it is not empty. It will default to the file name if it is not in the array
765 765
 			if ( ! empty( $img_title ) ) {
766
-				$image_meta['post_title'] = $img_title;
766
+				$image_meta[ 'post_title' ] = $img_title;
767 767
 			}
768 768
 
769 769
 			/**
@@ -821,15 +821,15 @@  discard block
 block discarded – undo
821 821
 	 */
822 822
 	private function maybe_update_post_fields( $form ) {
823 823
 
824
-		if( empty( $this->entry['post_id'] ) ) {
824
+		if ( empty( $this->entry[ 'post_id' ] ) ) {
825 825
 	        gravityview()->log->debug( 'This entry has no post fields. Continuing...' );
826 826
 			return;
827 827
 		}
828 828
 
829
-		$post_id = $this->entry['post_id'];
829
+		$post_id = $this->entry[ 'post_id' ];
830 830
 
831 831
 		// Security check
832
-		if( false === GVCommon::has_cap( 'edit_post', $post_id ) ) {
832
+		if ( false === GVCommon::has_cap( 'edit_post', $post_id ) ) {
833 833
 			gravityview()->log->error( 'The current user does not have the ability to edit Post #{post_id}', array( 'post_id' => $post_id ) );
834 834
 			return;
835 835
 		}
@@ -842,25 +842,25 @@  discard block
 block discarded – undo
842 842
 
843 843
 			$field = RGFormsModel::get_field( $form, $field_id );
844 844
 
845
-			if( ! $field ) {
845
+			if ( ! $field ) {
846 846
 				continue;
847 847
 			}
848 848
 
849
-			if( GFCommon::is_post_field( $field ) && 'post_category' !== $field->type ) {
849
+			if ( GFCommon::is_post_field( $field ) && 'post_category' !== $field->type ) {
850 850
 
851 851
 				// Get the value of the field, including $_POSTed value
852 852
 				$value = RGFormsModel::get_field_value( $field );
853 853
 
854 854
 				// Use temporary entry variable, to make values available to fill_post_template() and update_post_image()
855 855
 				$entry_tmp = $this->entry;
856
-				$entry_tmp["{$field_id}"] = $value;
856
+				$entry_tmp[ "{$field_id}" ] = $value;
857 857
 
858
-				switch( $field->type ) {
858
+				switch ( $field->type ) {
859 859
 
860 860
 				    case 'post_title':
861 861
 				        $post_title = $value;
862 862
 				        if ( \GV\Utils::get( $form, 'postTitleTemplateEnabled' ) ) {
863
-				            $post_title = $this->fill_post_template( $form['postTitleTemplate'], $form, $entry_tmp );
863
+				            $post_title = $this->fill_post_template( $form[ 'postTitleTemplate' ], $form, $entry_tmp );
864 864
 				        }
865 865
 				        $updated_post->post_title = $post_title;
866 866
 				        $updated_post->post_name  = $post_title;
@@ -870,7 +870,7 @@  discard block
 block discarded – undo
870 870
 				    case 'post_content':
871 871
 				        $post_content = $value;
872 872
 				        if ( \GV\Utils::get( $form, 'postContentTemplateEnabled' ) ) {
873
-				            $post_content = $this->fill_post_template( $form['postContentTemplate'], $form, $entry_tmp, true );
873
+				            $post_content = $this->fill_post_template( $form[ 'postContentTemplate' ], $form, $entry_tmp, true );
874 874
 				        }
875 875
 				        $updated_post->post_content = $post_content;
876 876
 				        unset( $post_content );
@@ -888,11 +888,11 @@  discard block
 block discarded – undo
888 888
 							$value = $value[ $field_id ];
889 889
 						}
890 890
 
891
-				        if( ! empty( $field->customFieldTemplateEnabled ) ) {
891
+				        if ( ! empty( $field->customFieldTemplateEnabled ) ) {
892 892
 				            $value = $this->fill_post_template( $field->customFieldTemplate, $form, $entry_tmp, true );
893 893
 				        }
894 894
 
895
-						$value = $field->get_value_save_entry( $value, $form, '', $this->entry['id'], $this->entry );
895
+						$value = $field->get_value_save_entry( $value, $form, '', $this->entry[ 'id' ], $this->entry );
896 896
 
897 897
 				        update_post_meta( $post_id, $field->postCustomFieldName, $value );
898 898
 				        break;
@@ -904,7 +904,7 @@  discard block
 block discarded – undo
904 904
 				}
905 905
 
906 906
 				// update entry after
907
-				$this->entry["{$field_id}"] = $value;
907
+				$this->entry[ "{$field_id}" ] = $value;
908 908
 
909 909
 				$update_entry = true;
910 910
 
@@ -913,11 +913,11 @@  discard block
 block discarded – undo
913 913
 
914 914
 		}
915 915
 
916
-		if( $update_entry ) {
916
+		if ( $update_entry ) {
917 917
 
918 918
 			$return_entry = GFAPI::update_entry( $this->entry );
919 919
 
920
-			if( is_wp_error( $return_entry ) ) {
920
+			if ( is_wp_error( $return_entry ) ) {
921 921
 				gravityview()->log->error( 'Updating the entry post fields failed', array( 'data' => array( '$this->entry' => $this->entry, '$return_entry' => $return_entry ) ) );
922 922
 			} else {
923 923
 				gravityview()->log->debug( 'Updating the entry post fields for post #{post_id} succeeded', array( 'post_id' => $post_id ) );
@@ -927,7 +927,7 @@  discard block
 block discarded – undo
927 927
 
928 928
 		$return_post = wp_update_post( $updated_post, true );
929 929
 
930
-		if( is_wp_error( $return_post ) ) {
930
+		if ( is_wp_error( $return_post ) ) {
931 931
 			$return_post->add_data( $updated_post, '$updated_post' );
932 932
 			gravityview()->log->error( 'Updating the post content failed', array( 'data' => compact( 'updated_post', 'return_post' ) ) );
933 933
 		} else {
@@ -961,7 +961,7 @@  discard block
 block discarded – undo
961 961
 		$output = GFCommon::replace_variables( $output, $form, $entry, false, false, false );
962 962
 
963 963
 		// replace conditional shortcodes
964
-		if( $do_shortcode ) {
964
+		if ( $do_shortcode ) {
965 965
 			$output = do_shortcode( $output );
966 966
 		}
967 967
 
@@ -980,19 +980,19 @@  discard block
 block discarded – undo
980 980
 	 */
981 981
 	private function after_update() {
982 982
 
983
-		do_action( 'gform_after_update_entry', self::$original_form, $this->entry['id'], self::$original_entry );
984
-		do_action( "gform_after_update_entry_{$this->form['id']}", self::$original_form, $this->entry['id'], self::$original_entry );
983
+		do_action( 'gform_after_update_entry', self::$original_form, $this->entry[ 'id' ], self::$original_entry );
984
+		do_action( "gform_after_update_entry_{$this->form[ 'id' ]}", self::$original_form, $this->entry[ 'id' ], self::$original_entry );
985 985
 
986 986
 		// Re-define the entry now that we've updated it.
987
-		$entry = RGFormsModel::get_lead( $this->entry['id'] );
987
+		$entry = RGFormsModel::get_lead( $this->entry[ 'id' ] );
988 988
 
989 989
 		$entry = GFFormsModel::set_entry_meta( $entry, self::$original_form );
990 990
 
991 991
 		if ( version_compare( GFFormsModel::get_database_version(), '2.3-dev-1', '<' ) ) {
992 992
 			// We need to clear the cache because Gravity Forms caches the field values, which
993 993
 			// we have just updated.
994
-			foreach ($this->form['fields'] as $key => $field) {
995
-				GFFormsModel::refresh_lead_field_value( $entry['id'], $field->id );
994
+			foreach ( $this->form[ 'fields' ] as $key => $field ) {
995
+				GFFormsModel::refresh_lead_field_value( $entry[ 'id' ], $field->id );
996 996
 			}
997 997
 		}
998 998
 
@@ -1002,11 +1002,11 @@  discard block
 block discarded – undo
1002 1002
 		 * @since develop
1003 1003
 		 */
1004 1004
 		if ( $allowed_feeds = $this->view->settings->get( 'edit_feeds', array() ) ) {
1005
-			$feeds = GFAPI::get_feeds( null, $entry['form_id'] );
1005
+			$feeds = GFAPI::get_feeds( null, $entry[ 'form_id' ] );
1006 1006
 			if ( ! is_wp_error( $feeds ) ) {
1007 1007
 				$registered_feeds = array();
1008 1008
 				foreach ( GFAddOn::get_registered_addons() as $registered_feed ) {
1009
-					if ( is_subclass_of( $registered_feed,  'GFFeedAddOn' ) ) {
1009
+					if ( is_subclass_of( $registered_feed, 'GFFeedAddOn' ) ) {
1010 1010
 						if ( method_exists( $registered_feed, 'get_instance' ) ) {
1011 1011
 							$registered_feed = call_user_func( array( $registered_feed, 'get_instance' ) );
1012 1012
 							$registered_feeds[ $registered_feed->get_slug() ] = $registered_feed;
@@ -1014,8 +1014,8 @@  discard block
 block discarded – undo
1014 1014
 					}
1015 1015
 				}
1016 1016
 				foreach ( $feeds as $feed ) {
1017
-					if ( in_array( $feed['id'], $allowed_feeds ) ) {
1018
-						if ( $feed_object = \GV\Utils::get( $registered_feeds, $feed['addon_slug'] ) ) {
1017
+					if ( in_array( $feed[ 'id' ], $allowed_feeds ) ) {
1018
+						if ( $feed_object = \GV\Utils::get( $registered_feeds, $feed[ 'addon_slug' ] ) ) {
1019 1019
 							$returned_entry = $feed_object->process_feed( $feed, $entry, self::$original_form );
1020 1020
 							if ( is_array( $returned_entry ) && rgar( $returned_entry, 'id' ) ) {
1021 1021
 								$entry = $returned_entry;
@@ -1042,7 +1042,7 @@  discard block
 block discarded – undo
1042 1042
 	public function edit_entry_form() {
1043 1043
 
1044 1044
 		$locking = new GravityView_Edit_Entry_Locking;
1045
-		$locking->maybe_lock_object( $this->entry['id'] );
1045
+		$locking->maybe_lock_object( $this->entry[ 'id' ] );
1046 1046
 
1047 1047
 		?>
1048 1048
 
@@ -1052,7 +1052,7 @@  discard block
 block discarded – undo
1052 1052
 		</script>
1053 1053
 
1054 1054
 		<div class="gv-edit-entry-wrapper"><?php
1055
-			$javascript = gravityview_ob_include( GravityView_Edit_Entry::$file .'/partials/inline-javascript.php', $this );
1055
+			$javascript = gravityview_ob_include( GravityView_Edit_Entry::$file . '/partials/inline-javascript.php', $this );
1056 1056
 
1057 1057
 			/**
1058 1058
 			 * Fixes weird wpautop() issue
@@ -1068,7 +1068,7 @@  discard block
 block discarded – undo
1068 1068
 				     * @param string $edit_entry_title Modify the "Edit Entry" title
1069 1069
 				     * @param GravityView_Edit_Entry_Render $this This object
1070 1070
 				     */
1071
-				    $edit_entry_title = apply_filters('gravityview_edit_entry_title', __('Edit Entry', 'gravityview'), $this );
1071
+				    $edit_entry_title = apply_filters( 'gravityview_edit_entry_title', __( 'Edit Entry', 'gravityview' ), $this );
1072 1072
 
1073 1073
 				    echo esc_attr( $edit_entry_title );
1074 1074
 			?></span>
@@ -1134,18 +1134,18 @@  discard block
 block discarded – undo
1134 1134
 				*/
1135 1135
 				$labels = apply_filters( 'gravityview/edit_entry/button_labels', $labels, $this->form, $this->entry, $this->view_id );
1136 1136
 
1137
-				$this->is_paged_submitted = \GV\Utils::_POST( 'save' ) === $labels['submit'];
1137
+				$this->is_paged_submitted = \GV\Utils::_POST( 'save' ) === $labels[ 'submit' ];
1138 1138
 			}
1139 1139
 
1140 1140
 			$back_link = remove_query_arg( array( 'page', 'view', 'edit' ) );
1141 1141
 
1142
-			if( ! $this->is_valid ){
1142
+			if ( ! $this->is_valid ) {
1143 1143
 
1144 1144
 				// Keeping this compatible with Gravity Forms.
1145
-				$validation_message = "<div class='validation_error'>" . __('There was a problem with your submission.', 'gravityview') . " " . __('Errors have been highlighted below.', 'gravityview') . "</div>";
1146
-				$message = apply_filters("gform_validation_message_{$this->form['id']}", apply_filters("gform_validation_message", $validation_message, $this->form), $this->form);
1145
+				$validation_message = "<div class='validation_error'>" . __( 'There was a problem with your submission.', 'gravityview' ) . " " . __( 'Errors have been highlighted below.', 'gravityview' ) . "</div>";
1146
+				$message = apply_filters( "gform_validation_message_{$this->form[ 'id' ]}", apply_filters( "gform_validation_message", $validation_message, $this->form ), $this->form );
1147 1147
 
1148
-				echo GVCommon::generate_notice( $message , 'gv-error' );
1148
+				echo GVCommon::generate_notice( $message, 'gv-error' );
1149 1149
 
1150 1150
 			} elseif ( false === $this->is_paged_submitted ) {
1151 1151
 				// Paged form that hasn't been submitted on the last page yet
@@ -1158,7 +1158,7 @@  discard block
 block discarded – undo
1158 1158
 				 * @param int $view_id View ID
1159 1159
 				 * @param array $entry Gravity Forms entry array
1160 1160
 				 */
1161
-				$message = apply_filters( 'gravityview/edit_entry/page/success', $entry_updated_message , $this->view_id, $this->entry );
1161
+				$message = apply_filters( 'gravityview/edit_entry/page/success', $entry_updated_message, $this->view_id, $this->entry );
1162 1162
 
1163 1163
 				echo GVCommon::generate_notice( $message );
1164 1164
 			} else {
@@ -1170,23 +1170,23 @@  discard block
 block discarded – undo
1170 1170
 
1171 1171
                     case '0':
1172 1172
 	                    $redirect_url = $back_link;
1173
-	                    $entry_updated_message = sprintf( esc_attr_x('Entry Updated. %sReturning to Entry%s', 'Replacements are HTML', 'gravityview'), '<a href="'. esc_url( $redirect_url ) .'">', '</a>' );
1173
+	                    $entry_updated_message = sprintf( esc_attr_x( 'Entry Updated. %sReturning to Entry%s', 'Replacements are HTML', 'gravityview' ), '<a href="' . esc_url( $redirect_url ) . '">', '</a>' );
1174 1174
                         break;
1175 1175
 
1176 1176
                     case '1':
1177 1177
 	                    $redirect_url = $directory_link = GravityView_API::directory_link();
1178
-	                    $entry_updated_message = sprintf( esc_attr_x('Entry Updated. %sReturning to %s%s', 'Replacement 1 is HTML. Replacement 2 is the title of the page where the user will be taken. Replacement 3 is HTML.','gravityview'), '<a href="'. esc_url( $redirect_url ) . '">', esc_html( $view->post_title ), '</a>' );
1178
+	                    $entry_updated_message = sprintf( esc_attr_x( 'Entry Updated. %sReturning to %s%s', 'Replacement 1 is HTML. Replacement 2 is the title of the page where the user will be taken. Replacement 3 is HTML.', 'gravityview' ), '<a href="' . esc_url( $redirect_url ) . '">', esc_html( $view->post_title ), '</a>' );
1179 1179
 	                    break;
1180 1180
 
1181 1181
                     case '2':
1182 1182
 	                    $redirect_url = $edit_redirect_url;
1183 1183
 	                    $redirect_url = GFCommon::replace_variables( $redirect_url, $this->form, $this->entry, false, false, false, 'text' );
1184
-	                    $entry_updated_message = sprintf( esc_attr_x('Entry Updated. %sRedirecting to %s%s', 'Replacement 1 is HTML. Replacement 2 is the URL where the user will be taken. Replacement 3 is HTML.','gravityview'), '<a href="'. esc_url( $redirect_url ) . '">', esc_html( $edit_redirect_url ), '</a>' );
1184
+	                    $entry_updated_message = sprintf( esc_attr_x( 'Entry Updated. %sRedirecting to %s%s', 'Replacement 1 is HTML. Replacement 2 is the URL where the user will be taken. Replacement 3 is HTML.', 'gravityview' ), '<a href="' . esc_url( $redirect_url ) . '">', esc_html( $edit_redirect_url ), '</a>' );
1185 1185
                         break;
1186 1186
 
1187 1187
                     case '':
1188 1188
                     default:
1189
-					    $entry_updated_message = sprintf( esc_attr__('Entry Updated. %sReturn to Entry%s', 'gravityview'), '<a href="'. esc_url( $back_link ) .'">', '</a>' );
1189
+					    $entry_updated_message = sprintf( esc_attr__( 'Entry Updated. %sReturn to Entry%s', 'gravityview' ), '<a href="' . esc_url( $back_link ) . '">', '</a>' );
1190 1190
                         break;
1191 1191
 				}
1192 1192
 
@@ -1202,7 +1202,7 @@  discard block
 block discarded – undo
1202 1202
 				 * @param array $entry Gravity Forms entry array
1203 1203
 				 * @param string $back_link URL to return to the original entry. @since 1.6
1204 1204
 				 */
1205
-				$message = apply_filters( 'gravityview/edit_entry/success', $entry_updated_message , $this->view_id, $this->entry, $back_link );
1205
+				$message = apply_filters( 'gravityview/edit_entry/success', $entry_updated_message, $this->view_id, $this->entry, $back_link );
1206 1206
 
1207 1207
 				echo GVCommon::generate_notice( $message );
1208 1208
 			}
@@ -1226,8 +1226,8 @@  discard block
 block discarded – undo
1226 1226
 		 */
1227 1227
 		do_action( 'gravityview/edit-entry/render/before', $this );
1228 1228
 
1229
-		add_filter( 'gform_pre_render', array( $this, 'filter_modify_form_fields'), 5000, 3 );
1230
-		add_filter( 'gform_submit_button', array( $this, 'render_form_buttons') );
1229
+		add_filter( 'gform_pre_render', array( $this, 'filter_modify_form_fields' ), 5000, 3 );
1230
+		add_filter( 'gform_submit_button', array( $this, 'render_form_buttons' ) );
1231 1231
 		add_filter( 'gform_next_button', array( $this, 'render_form_buttons' ) );
1232 1232
 		add_filter( 'gform_previous_button', array( $this, 'render_form_buttons' ) );
1233 1233
 		add_filter( 'gform_disable_view_counter', '__return_true' );
@@ -1236,14 +1236,14 @@  discard block
 block discarded – undo
1236 1236
 		add_filter( 'gform_field_input', array( $this, 'modify_edit_field_input' ), 10, 5 );
1237 1237
 
1238 1238
 		// We need to remove the fake $_GET['page'] arg to avoid rendering form as if in admin.
1239
-		unset( $_GET['page'] );
1239
+		unset( $_GET[ 'page' ] );
1240 1240
 
1241 1241
 		$this->show_next_button = false;
1242 1242
 		$this->show_previous_button = false;
1243 1243
 
1244 1244
 		// TODO: Verify multiple-page forms
1245 1245
 		if ( GFCommon::has_pages( $this->form ) && apply_filters( 'gravityview/features/paged-edit', false ) ) {
1246
-			if ( intval( $page_number = \GV\Utils::_POST( 'gform_source_page_number_' . $this->form['id'], 0 ) ) ) {
1246
+			if ( intval( $page_number = \GV\Utils::_POST( 'gform_source_page_number_' . $this->form[ 'id' ], 0 ) ) ) {
1247 1247
 
1248 1248
 				$labels = array(
1249 1249
 					'cancel'   => __( 'Cancel', 'gravityview' ),
@@ -1262,20 +1262,20 @@  discard block
 block discarded – undo
1262 1262
 				*/
1263 1263
 				$labels = apply_filters( 'gravityview/edit_entry/button_labels', $labels, $this->form, $this->entry, $this->view_id );
1264 1264
 
1265
-				GFFormDisplay::$submission[ $this->form['id'] ][ 'form' ] = $this->form;
1266
-				GFFormDisplay::$submission[ $this->form['id'] ][ 'is_valid' ] = true;
1265
+				GFFormDisplay::$submission[ $this->form[ 'id' ] ][ 'form' ] = $this->form;
1266
+				GFFormDisplay::$submission[ $this->form[ 'id' ] ][ 'is_valid' ] = true;
1267 1267
 
1268
-				if ( \GV\Utils::_POST( 'save' ) === $labels['next'] ) {
1268
+				if ( \GV\Utils::_POST( 'save' ) === $labels[ 'next' ] ) {
1269 1269
 					$last_page = \GFFormDisplay::get_max_page_number( $this->form );
1270 1270
 
1271 1271
 					while ( ++$page_number < $last_page && RGFormsModel::is_page_hidden( $this->form, $page_number, \GV\Utils::_POST( 'gform_field_values' ) ) ) {
1272 1272
 					} // Advance to next visible page
1273
-				} elseif ( \GV\Utils::_POST( 'save' ) === $labels['previous'] ) {
1273
+				} elseif ( \GV\Utils::_POST( 'save' ) === $labels[ 'previous' ] ) {
1274 1274
 					while ( --$page_number > 1 && RGFormsModel::is_page_hidden( $this->form, $page_number, \GV\Utils::_POST( 'gform_field_values' ) ) ) {
1275 1275
 					} // Advance to next visible page
1276 1276
 				}
1277 1277
 
1278
-				GFFormDisplay::$submission[ $this->form['id'] ]['page_number'] = $page_number;
1278
+				GFFormDisplay::$submission[ $this->form[ 'id' ] ][ 'page_number' ] = $page_number;
1279 1279
 			}
1280 1280
 
1281 1281
 			if ( ( $page_number = intval( $page_number ) ) < 2 ) {
@@ -1301,7 +1301,7 @@  discard block
 block discarded – undo
1301 1301
 
1302 1302
 		ob_start(); // Prevent PHP warnings possibly caused by prefilling list fields for conditional logic
1303 1303
 
1304
-		$html = GFFormDisplay::get_form( $this->form['id'], false, false, true, $this->entry );
1304
+		$html = GFFormDisplay::get_form( $this->form[ 'id' ], false, false, true, $this->entry );
1305 1305
 
1306 1306
 		ob_get_clean();
1307 1307
 
@@ -1329,7 +1329,7 @@  discard block
 block discarded – undo
1329 1329
 	 * @return string
1330 1330
 	 */
1331 1331
 	public function render_form_buttons() {
1332
-		return gravityview_ob_include( GravityView_Edit_Entry::$file .'/partials/form-buttons.php', $this );
1332
+		return gravityview_ob_include( GravityView_Edit_Entry::$file . '/partials/form-buttons.php', $this );
1333 1333
 	}
1334 1334
 
1335 1335
 
@@ -1348,15 +1348,15 @@  discard block
 block discarded – undo
1348 1348
 	 */
1349 1349
 	public function filter_modify_form_fields( $form, $ajax = false, $field_values = '' ) {
1350 1350
 
1351
-		if( $form['id'] != $this->form_id ) {
1351
+		if ( $form[ 'id' ] != $this->form_id ) {
1352 1352
 			return $form;
1353 1353
 		}
1354 1354
 
1355 1355
 		// In case we have validated the form, use it to inject the validation results into the form render
1356
-		if( isset( $this->form_after_validation ) && $this->form_after_validation['id'] === $form['id'] ) {
1356
+		if ( isset( $this->form_after_validation ) && $this->form_after_validation[ 'id' ] === $form[ 'id' ] ) {
1357 1357
 			$form = $this->form_after_validation;
1358 1358
 		} else {
1359
-			$form['fields'] = $this->get_configured_edit_fields( $form, $this->view_id );
1359
+			$form[ 'fields' ] = $this->get_configured_edit_fields( $form, $this->view_id );
1360 1360
 		}
1361 1361
 
1362 1362
 		$form = $this->filter_conditional_logic( $form );
@@ -1364,8 +1364,8 @@  discard block
 block discarded – undo
1364 1364
 		$form = $this->prefill_conditional_logic( $form );
1365 1365
 
1366 1366
 		// for now we don't support Save and Continue feature.
1367
-		if( ! self::$supports_save_and_continue ) {
1368
-	        unset( $form['save'] );
1367
+		if ( ! self::$supports_save_and_continue ) {
1368
+	        unset( $form[ 'save' ] );
1369 1369
 		}
1370 1370
 
1371 1371
 		$form = $this->unselect_default_values( $form );
@@ -1388,31 +1388,31 @@  discard block
 block discarded – undo
1388 1388
 	 */
1389 1389
 	public function verify_user_can_edit_post( $field_content = '', $field, $value, $lead_id = 0, $form_id ) {
1390 1390
 
1391
-		if( ! GFCommon::is_post_field( $field ) ) {
1391
+		if ( ! GFCommon::is_post_field( $field ) ) {
1392 1392
 			return $field_content;
1393 1393
 		}
1394 1394
 
1395 1395
         $message = null;
1396 1396
 
1397 1397
         // First, make sure they have the capability to edit the post.
1398
-        if( false === current_user_can( 'edit_post', $this->entry['post_id'] ) ) {
1398
+        if ( false === current_user_can( 'edit_post', $this->entry[ 'post_id' ] ) ) {
1399 1399
 
1400 1400
             /**
1401 1401
              * @filter `gravityview/edit_entry/unsupported_post_field_text` Modify the message when someone isn't able to edit a post
1402 1402
              * @param string $message The existing "You don't have permission..." text
1403 1403
              */
1404
-            $message = apply_filters('gravityview/edit_entry/unsupported_post_field_text', __('You don&rsquo;t have permission to edit this post.', 'gravityview') );
1404
+            $message = apply_filters( 'gravityview/edit_entry/unsupported_post_field_text', __( 'You don&rsquo;t have permission to edit this post.', 'gravityview' ) );
1405 1405
 
1406
-        } elseif( null === get_post( $this->entry['post_id'] ) ) {
1406
+        } elseif ( null === get_post( $this->entry[ 'post_id' ] ) ) {
1407 1407
             /**
1408 1408
              * @filter `gravityview/edit_entry/no_post_text` Modify the message when someone is editing an entry attached to a post that no longer exists
1409 1409
              * @param string $message The existing "This field is not editable; the post no longer exists." text
1410 1410
              */
1411
-            $message = apply_filters('gravityview/edit_entry/no_post_text', __('This field is not editable; the post no longer exists.', 'gravityview' ) );
1411
+            $message = apply_filters( 'gravityview/edit_entry/no_post_text', __( 'This field is not editable; the post no longer exists.', 'gravityview' ) );
1412 1412
         }
1413 1413
 
1414
-        if( $message ) {
1415
-            $field_content = sprintf('<div class="ginput_container ginput_container_' . $field->type . '">%s</div>', wpautop( $message ) );
1414
+        if ( $message ) {
1415
+            $field_content = sprintf( '<div class="ginput_container ginput_container_' . $field->type . '">%s</div>', wpautop( $message ) );
1416 1416
         }
1417 1417
 
1418 1418
         return $field_content;
@@ -1436,8 +1436,8 @@  discard block
 block discarded – undo
1436 1436
 
1437 1437
 		// If the form has been submitted, then we don't need to pre-fill the values,
1438 1438
 		// Except for fileupload type and when a field input is overridden- run always!!
1439
-		if(
1440
-			( $this->is_edit_entry_submission() && !in_array( $field->type, array( 'fileupload', 'post_image' ) ) )
1439
+		if (
1440
+			( $this->is_edit_entry_submission() && ! in_array( $field->type, array( 'fileupload', 'post_image' ) ) )
1441 1441
 			&& false === ( $gv_field && is_callable( array( $gv_field, 'get_field_input' ) ) )
1442 1442
 			&& ! GFCommon::is_product_field( $field->type )
1443 1443
 			|| ! empty( $field_content )
@@ -1457,7 +1457,7 @@  discard block
 block discarded – undo
1457 1457
 	    $return = null;
1458 1458
 
1459 1459
 		/** @var GravityView_Field $gv_field */
1460
-		if( $gv_field && is_callable( array( $gv_field, 'get_field_input' ) ) ) {
1460
+		if ( $gv_field && is_callable( array( $gv_field, 'get_field_input' ) ) ) {
1461 1461
 			$return = $gv_field->get_field_input( $this->form, $field_value, $this->entry, $field );
1462 1462
 		} else {
1463 1463
 	        $return = $field->get_field_input( $this->form, $field_value, $this->entry );
@@ -1466,7 +1466,7 @@  discard block
 block discarded – undo
1466 1466
 	    // If there was output, it's an error
1467 1467
 	    $warnings = ob_get_clean();
1468 1468
 
1469
-	    if( !empty( $warnings ) ) {
1469
+	    if ( ! empty( $warnings ) ) {
1470 1470
 		    gravityview()->log->error( '{warning}', array( 'warning' => $warnings, 'data' => $field_value ) );
1471 1471
 	    }
1472 1472
 
@@ -1491,7 +1491,7 @@  discard block
 block discarded – undo
1491 1491
 		$override_saved_value = apply_filters( 'gravityview/edit_entry/pre_populate/override', false, $field );
1492 1492
 
1493 1493
 		// We're dealing with multiple inputs (e.g. checkbox) but not time or date (as it doesn't store data in input IDs)
1494
-		if( isset( $field->inputs ) && is_array( $field->inputs ) && !in_array( $field->type, array( 'time', 'date' ) ) ) {
1494
+		if ( isset( $field->inputs ) && is_array( $field->inputs ) && ! in_array( $field->type, array( 'time', 'date' ) ) ) {
1495 1495
 
1496 1496
 			$field_value = array();
1497 1497
 
@@ -1500,10 +1500,10 @@  discard block
 block discarded – undo
1500 1500
 
1501 1501
 			foreach ( (array)$field->inputs as $input ) {
1502 1502
 
1503
-				$input_id = strval( $input['id'] );
1503
+				$input_id = strval( $input[ 'id' ] );
1504 1504
 
1505 1505
 				if ( isset( $this->entry[ $input_id ] ) && ! gv_empty( $this->entry[ $input_id ], false, false ) ) {
1506
-				    $field_value[ $input_id ] =  'post_category' === $field->type ? GFCommon::format_post_category( $this->entry[ $input_id ], true ) : $this->entry[ $input_id ];
1506
+				    $field_value[ $input_id ] = 'post_category' === $field->type ? GFCommon::format_post_category( $this->entry[ $input_id ], true ) : $this->entry[ $input_id ];
1507 1507
 				    $allow_pre_populated = false;
1508 1508
 				}
1509 1509
 
@@ -1511,7 +1511,7 @@  discard block
 block discarded – undo
1511 1511
 
1512 1512
 			$pre_value = $field->get_value_submission( array(), false );
1513 1513
 
1514
-			$field_value = ! $allow_pre_populated && ! ( $override_saved_value && !gv_empty( $pre_value, false, false ) ) ? $field_value : $pre_value;
1514
+			$field_value = ! $allow_pre_populated && ! ( $override_saved_value && ! gv_empty( $pre_value, false, false ) ) ? $field_value : $pre_value;
1515 1515
 
1516 1516
 		} else {
1517 1517
 
@@ -1522,13 +1522,13 @@  discard block
 block discarded – undo
1522 1522
 
1523 1523
 			// saved field entry value (if empty, fallback to the pre-populated value, if exists)
1524 1524
 			// or pre-populated value if not empty and set to override saved value
1525
-			$field_value = isset( $this->entry[ $id ] ) && ! gv_empty( $this->entry[ $id ], false, false ) && ! ( $override_saved_value && !gv_empty( $pre_value, false, false ) ) ? $this->entry[ $id ] : $pre_value;
1525
+			$field_value = isset( $this->entry[ $id ] ) && ! gv_empty( $this->entry[ $id ], false, false ) && ! ( $override_saved_value && ! gv_empty( $pre_value, false, false ) ) ? $this->entry[ $id ] : $pre_value;
1526 1526
 
1527 1527
 			// in case field is post_category but inputType is select, multi-select or radio, convert value into array of category IDs.
1528
-			if ( 'post_category' === $field->type && !gv_empty( $field_value, false, false ) ) {
1528
+			if ( 'post_category' === $field->type && ! gv_empty( $field_value, false, false ) ) {
1529 1529
 				$categories = array();
1530 1530
 				foreach ( explode( ',', $field_value ) as $cat_string ) {
1531
-				    $categories[] = GFCommon::format_post_category( $cat_string, true );
1531
+				    $categories[ ] = GFCommon::format_post_category( $cat_string, true );
1532 1532
 				}
1533 1533
 				$field_value = 'multiselect' === $field->get_input_type() ? $categories : implode( '', $categories );
1534 1534
 			}
@@ -1556,7 +1556,7 @@  discard block
 block discarded – undo
1556 1556
 	     * @param GF_Field $field Gravity Forms field object
1557 1557
 	     * @param GravityView_Edit_Entry_Render $this Current object
1558 1558
 	     */
1559
-	    $field_value = apply_filters( 'gravityview/edit_entry/field_value_' . $field->type , $field_value, $field, $this );
1559
+	    $field_value = apply_filters( 'gravityview/edit_entry/field_value_' . $field->type, $field_value, $field, $this );
1560 1560
 
1561 1561
 		return $field_value;
1562 1562
 	}
@@ -1573,12 +1573,12 @@  discard block
 block discarded – undo
1573 1573
 	 */
1574 1574
 	public function gform_pre_validation( $form ) {
1575 1575
 
1576
-		if( ! $this->verify_nonce() ) {
1576
+		if ( ! $this->verify_nonce() ) {
1577 1577
 			return $form;
1578 1578
 		}
1579 1579
 
1580 1580
 		// Fix PHP warning regarding undefined index.
1581
-		foreach ( $form['fields'] as &$field) {
1581
+		foreach ( $form[ 'fields' ] as &$field ) {
1582 1582
 
1583 1583
 			// This is because we're doing admin form pretending to be front-end, so Gravity Forms
1584 1584
 			// expects certain field array items to be set.
@@ -1586,7 +1586,7 @@  discard block
 block discarded – undo
1586 1586
 	            $field->{$key} = isset( $field->{$key} ) ? $field->{$key} : NULL;
1587 1587
 			}
1588 1588
 
1589
-			switch( RGFormsModel::get_input_type( $field ) ) {
1589
+			switch ( RGFormsModel::get_input_type( $field ) ) {
1590 1590
 
1591 1591
 				/**
1592 1592
 				 * 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.
@@ -1600,26 +1600,26 @@  discard block
 block discarded – undo
1600 1600
 				    // Set the previous value
1601 1601
 				    $entry = $this->get_entry();
1602 1602
 
1603
-				    $input_name = 'input_'.$field->id;
1604
-				    $form_id = $form['id'];
1603
+				    $input_name = 'input_' . $field->id;
1604
+				    $form_id = $form[ 'id' ];
1605 1605
 
1606 1606
 				    $value = NULL;
1607 1607
 
1608 1608
 				    // Use the previous entry value as the default.
1609
-				    if( isset( $entry[ $field->id ] ) ) {
1609
+				    if ( isset( $entry[ $field->id ] ) ) {
1610 1610
 				        $value = $entry[ $field->id ];
1611 1611
 				    }
1612 1612
 
1613 1613
 				    // If this is a single upload file
1614
-				    if( !empty( $_FILES[ $input_name ] ) && !empty( $_FILES[ $input_name ]['name'] ) ) {
1615
-				        $file_path = GFFormsModel::get_file_upload_path( $form['id'], $_FILES[ $input_name ]['name'] );
1616
-				        $value = $file_path['url'];
1614
+				    if ( ! empty( $_FILES[ $input_name ] ) && ! empty( $_FILES[ $input_name ][ 'name' ] ) ) {
1615
+				        $file_path = GFFormsModel::get_file_upload_path( $form[ 'id' ], $_FILES[ $input_name ][ 'name' ] );
1616
+				        $value = $file_path[ 'url' ];
1617 1617
 
1618 1618
 				    } else {
1619 1619
 
1620 1620
 				        // Fix PHP warning on line 1498 of form_display.php for post_image fields
1621 1621
 				        // Fix PHP Notice:  Undefined index:  size in form_display.php on line 1511
1622
-				        $_FILES[ $input_name ] = array('name' => '', 'size' => '' );
1622
+				        $_FILES[ $input_name ] = array( 'name' => '', 'size' => '' );
1623 1623
 
1624 1624
 				    }
1625 1625
 
@@ -1627,10 +1627,10 @@  discard block
 block discarded – undo
1627 1627
 
1628 1628
 				        // If there are fresh uploads, process and merge them.
1629 1629
 				        // Otherwise, use the passed values, which should be json-encoded array of URLs
1630
-				        if( isset( GFFormsModel::$uploaded_files[$form_id][$input_name] ) ) {
1630
+				        if ( isset( GFFormsModel::$uploaded_files[ $form_id ][ $input_name ] ) ) {
1631 1631
 				            $value = empty( $value ) ? '[]' : $value;
1632 1632
 				            $value = stripslashes_deep( $value );
1633
-				            $value = GFFormsModel::prepare_value( $form, $field, $value, $input_name, $entry['id'], array());
1633
+				            $value = GFFormsModel::prepare_value( $form, $field, $value, $input_name, $entry[ 'id' ], array() );
1634 1634
 				        }
1635 1635
 
1636 1636
 				    } else {
@@ -1648,8 +1648,8 @@  discard block
 block discarded – undo
1648 1648
 
1649 1649
 				case 'number':
1650 1650
 				    // Fix "undefined index" issue at line 1286 in form_display.php
1651
-				    if( !isset( $_POST['input_'.$field->id ] ) ) {
1652
-				        $_POST['input_'.$field->id ] = NULL;
1651
+				    if ( ! isset( $_POST[ 'input_' . $field->id ] ) ) {
1652
+				        $_POST[ 'input_' . $field->id ] = NULL;
1653 1653
 				    }
1654 1654
 				    break;
1655 1655
 			}
@@ -1686,7 +1686,7 @@  discard block
 block discarded – undo
1686 1686
 		 * You can enter whatever you want!
1687 1687
 		 * We try validating, and customize the results using `self::custom_validation()`
1688 1688
 		 */
1689
-		add_filter( 'gform_validation_'. $this->form_id, array( $this, 'custom_validation' ), 10, 4);
1689
+		add_filter( 'gform_validation_' . $this->form_id, array( $this, 'custom_validation' ), 10, 4 );
1690 1690
 
1691 1691
 		// Needed by the validate funtion
1692 1692
 		$failed_validation_page = NULL;
@@ -1694,14 +1694,14 @@  discard block
 block discarded – undo
1694 1694
 
1695 1695
 		// Prevent entry limit from running when editing an entry, also
1696 1696
 		// prevent form scheduling from preventing editing
1697
-		unset( $this->form['limitEntries'], $this->form['scheduleForm'] );
1697
+		unset( $this->form[ 'limitEntries' ], $this->form[ 'scheduleForm' ] );
1698 1698
 
1699 1699
 		// Hide fields depending on Edit Entry settings
1700
-		$this->form['fields'] = $this->get_configured_edit_fields( $this->form, $this->view_id );
1700
+		$this->form[ 'fields' ] = $this->get_configured_edit_fields( $this->form, $this->view_id );
1701 1701
 
1702 1702
 		$this->is_valid = GFFormDisplay::validate( $this->form, $field_values, 1, $failed_validation_page );
1703 1703
 
1704
-		remove_filter( 'gform_validation_'. $this->form_id, array( $this, 'custom_validation' ), 10 );
1704
+		remove_filter( 'gform_validation_' . $this->form_id, array( $this, 'custom_validation' ), 10 );
1705 1705
 	}
1706 1706
 
1707 1707
 
@@ -1724,7 +1724,7 @@  discard block
 block discarded – undo
1724 1724
 
1725 1725
 		$gv_valid = true;
1726 1726
 
1727
-		foreach ( $validation_results['form']['fields'] as $key => &$field ) {
1727
+		foreach ( $validation_results[ 'form' ][ 'fields' ] as $key => &$field ) {
1728 1728
 
1729 1729
 			$value = RGFormsModel::get_field_value( $field );
1730 1730
 			$field_type = RGFormsModel::get_input_type( $field );
@@ -1737,22 +1737,22 @@  discard block
 block discarded – undo
1737 1737
 				case 'post_image':
1738 1738
 
1739 1739
 				    // in case nothing is uploaded but there are already files saved
1740
-				    if( !empty( $field->failed_validation ) && !empty( $field->isRequired ) && !empty( $value ) ) {
1740
+				    if ( ! empty( $field->failed_validation ) && ! empty( $field->isRequired ) && ! empty( $value ) ) {
1741 1741
 				        $field->failed_validation = false;
1742 1742
 				        unset( $field->validation_message );
1743 1743
 				    }
1744 1744
 
1745 1745
 				    // validate if multi file upload reached max number of files [maxFiles] => 2
1746
-				    if( \GV\Utils::get( $field, 'maxFiles') && \GV\Utils::get( $field, 'multipleFiles') ) {
1746
+				    if ( \GV\Utils::get( $field, 'maxFiles' ) && \GV\Utils::get( $field, 'multipleFiles' ) ) {
1747 1747
 
1748 1748
 				        $input_name = 'input_' . $field->id;
1749 1749
 				        //uploaded
1750
-				        $file_names = isset( GFFormsModel::$uploaded_files[ $validation_results['form']['id'] ][ $input_name ] ) ? GFFormsModel::$uploaded_files[ $validation_results['form']['id'] ][ $input_name ] : array();
1750
+				        $file_names = isset( GFFormsModel::$uploaded_files[ $validation_results[ 'form' ][ 'id' ] ][ $input_name ] ) ? GFFormsModel::$uploaded_files[ $validation_results[ 'form' ][ 'id' ] ][ $input_name ] : array();
1751 1751
 
1752 1752
 				        //existent
1753 1753
 				        $entry = $this->get_entry();
1754 1754
 				        $value = NULL;
1755
-				        if( isset( $entry[ $field->id ] ) ) {
1755
+				        if ( isset( $entry[ $field->id ] ) ) {
1756 1756
 				            $value = json_decode( $entry[ $field->id ], true );
1757 1757
 				        }
1758 1758
 
@@ -1760,13 +1760,13 @@  discard block
 block discarded – undo
1760 1760
 				        $count_files = ( is_array( $file_names ) ? count( $file_names ) : 0 ) +
1761 1761
 						               ( is_array( $value ) ? count( $value ) : 0 );
1762 1762
 
1763
-				        if( $count_files > $field->maxFiles ) {
1763
+				        if ( $count_files > $field->maxFiles ) {
1764 1764
 				            $field->validation_message = __( 'Maximum number of files reached', 'gravityview' );
1765 1765
 				            $field->failed_validation = 1;
1766 1766
 				            $gv_valid = false;
1767 1767
 
1768 1768
 				            // in case of error make sure the newest upload files are removed from the upload input
1769
-				            GFFormsModel::$uploaded_files[ $validation_results['form']['id'] ] = null;
1769
+				            GFFormsModel::$uploaded_files[ $validation_results[ 'form' ][ 'id' ] ] = null;
1770 1770
 				        }
1771 1771
 
1772 1772
 				    }
@@ -1777,7 +1777,7 @@  discard block
 block discarded – undo
1777 1777
 			}
1778 1778
 
1779 1779
 			// This field has failed validation.
1780
-			if( !empty( $field->failed_validation ) ) {
1780
+			if ( ! empty( $field->failed_validation ) ) {
1781 1781
 
1782 1782
 				gravityview()->log->debug( 'GravityView_Edit_Entry[custom_validation] Field is invalid.', array( 'data' => array( 'field' => $field, 'value' => $value ) ) );
1783 1783
 
@@ -1795,19 +1795,19 @@  discard block
 block discarded – undo
1795 1795
 				}
1796 1796
 
1797 1797
 				// You can't continue inside a switch, so we do it after.
1798
-				if( empty( $field->failed_validation ) ) {
1798
+				if ( empty( $field->failed_validation ) ) {
1799 1799
 				    continue;
1800 1800
 				}
1801 1801
 
1802 1802
 				// checks if the No Duplicates option is not validating entry against itself, since
1803 1803
 				// we're editing a stored entry, it would also assume it's a duplicate.
1804
-				if( !empty( $field->noDuplicates ) ) {
1804
+				if ( ! empty( $field->noDuplicates ) ) {
1805 1805
 
1806 1806
 				    $entry = $this->get_entry();
1807 1807
 
1808 1808
 				    // If the value of the entry is the same as the stored value
1809 1809
 				    // Then we can assume it's not a duplicate, it's the same.
1810
-				    if( !empty( $entry ) && $value == $entry[ $field->id ] ) {
1810
+				    if ( ! empty( $entry ) && $value == $entry[ $field->id ] ) {
1811 1811
 				        //if value submitted was not changed, then don't validate
1812 1812
 				        $field->failed_validation = false;
1813 1813
 
@@ -1820,7 +1820,7 @@  discard block
 block discarded – undo
1820 1820
 				}
1821 1821
 
1822 1822
 				// if here then probably we are facing the validation 'At least one field must be filled out'
1823
-				if( GFFormDisplay::is_empty( $field, $this->form_id  ) && empty( $field->isRequired ) ) {
1823
+				if ( GFFormDisplay::is_empty( $field, $this->form_id ) && empty( $field->isRequired ) ) {
1824 1824
 				    unset( $field->validation_message );
1825 1825
 	                $field->validation_message = false;
1826 1826
 				    continue;
@@ -1832,12 +1832,12 @@  discard block
 block discarded – undo
1832 1832
 
1833 1833
 		}
1834 1834
 
1835
-		$validation_results['is_valid'] = $gv_valid;
1835
+		$validation_results[ 'is_valid' ] = $gv_valid;
1836 1836
 
1837 1837
 		gravityview()->log->debug( 'GravityView_Edit_Entry[custom_validation] Validation results.', array( 'data' => $validation_results ) );
1838 1838
 
1839 1839
 		// We'll need this result when rendering the form ( on GFFormDisplay::get_form )
1840
-		$this->form_after_validation = $validation_results['form'];
1840
+		$this->form_after_validation = $validation_results[ 'form' ];
1841 1841
 
1842 1842
 		return $validation_results;
1843 1843
 	}
@@ -1850,7 +1850,7 @@  discard block
 block discarded – undo
1850 1850
 	 */
1851 1851
 	public function get_entry() {
1852 1852
 
1853
-		if( empty( $this->entry ) ) {
1853
+		if ( empty( $this->entry ) ) {
1854 1854
 			// Get the database value of the entry that's being edited
1855 1855
 			$this->entry = gravityview_get_entry( GravityView_frontend::is_single_entry() );
1856 1856
 		}
@@ -1882,10 +1882,10 @@  discard block
 block discarded – undo
1882 1882
 		}
1883 1883
 
1884 1884
 		// If edit tab not yet configured, show all fields
1885
-		$edit_fields = !empty( $properties['edit_edit-fields'] ) ? $properties['edit_edit-fields'] : NULL;
1885
+		$edit_fields = ! empty( $properties[ 'edit_edit-fields' ] ) ? $properties[ 'edit_edit-fields' ] : NULL;
1886 1886
 
1887 1887
 		// Hide fields depending on admin settings
1888
-		$fields = $this->filter_fields( $form['fields'], $edit_fields );
1888
+		$fields = $this->filter_fields( $form[ 'fields' ], $edit_fields );
1889 1889
 
1890 1890
 	    // If Edit Entry fields are configured, remove adminOnly field settings. Otherwise, don't.
1891 1891
 	    $fields = $this->filter_admin_only_fields( $fields, $edit_fields, $form, $view_id );
@@ -1918,7 +1918,7 @@  discard block
 block discarded – undo
1918 1918
 	 */
1919 1919
 	private function filter_fields( $fields, $configured_fields ) {
1920 1920
 
1921
-		if( empty( $fields ) || !is_array( $fields ) ) {
1921
+		if ( empty( $fields ) || ! is_array( $fields ) ) {
1922 1922
 			return $fields;
1923 1923
 		}
1924 1924
 
@@ -1935,12 +1935,12 @@  discard block
 block discarded – undo
1935 1935
 
1936 1936
 			// Remove the fields that have calculation properties and keep them to be used later
1937 1937
 			// @since 1.16.2
1938
-			if( $field->has_calculation() ) {
1939
-				$this->fields_with_calculation[] = $field;
1938
+			if ( $field->has_calculation() ) {
1939
+				$this->fields_with_calculation[ ] = $field;
1940 1940
 				// don't remove the calculation fields on form render.
1941 1941
 			}
1942 1942
 
1943
-			if( in_array( $field->type, $field_type_blacklist ) ) {
1943
+			if ( in_array( $field->type, $field_type_blacklist ) ) {
1944 1944
 				unset( $fields[ $key ] );
1945 1945
 			}
1946 1946
 		}
@@ -1958,7 +1958,7 @@  discard block
 block discarded – undo
1958 1958
 					continue; // Same
1959 1959
 				}
1960 1960
 
1961
-				$out_fields[] = $field;
1961
+				$out_fields[ ] = $field;
1962 1962
 			}
1963 1963
 
1964 1964
 			return array_values( $out_fields );
@@ -1969,8 +1969,8 @@  discard block
 block discarded – undo
1969 1969
 
1970 1970
 	        /** @var GF_Field $field */
1971 1971
 	        foreach ( $fields as $field ) {
1972
-				if( intval( $configured_field['id'] ) === intval( $field->id ) && $this->user_can_edit_field( $configured_field, false ) ) {
1973
-				    $edit_fields[] = $this->merge_field_properties( $field, $configured_field );
1972
+				if ( intval( $configured_field[ 'id' ] ) === intval( $field->id ) && $this->user_can_edit_field( $configured_field, false ) ) {
1973
+				    $edit_fields[ ] = $this->merge_field_properties( $field, $configured_field );
1974 1974
 				    break;
1975 1975
 				}
1976 1976
 
@@ -1993,14 +1993,14 @@  discard block
 block discarded – undo
1993 1993
 
1994 1994
 		$return_field = $field;
1995 1995
 
1996
-		if( empty( $field_setting['show_label'] ) ) {
1996
+		if ( empty( $field_setting[ 'show_label' ] ) ) {
1997 1997
 			$return_field->label = '';
1998
-		} elseif ( !empty( $field_setting['custom_label'] ) ) {
1999
-			$return_field->label = $field_setting['custom_label'];
1998
+		} elseif ( ! empty( $field_setting[ 'custom_label' ] ) ) {
1999
+			$return_field->label = $field_setting[ 'custom_label' ];
2000 2000
 		}
2001 2001
 
2002
-		if( !empty( $field_setting['custom_class'] ) ) {
2003
-			$return_field->cssClass .= ' '. gravityview_sanitize_html_class( $field_setting['custom_class'] );
2002
+		if ( ! empty( $field_setting[ 'custom_class' ] ) ) {
2003
+			$return_field->cssClass .= ' ' . gravityview_sanitize_html_class( $field_setting[ 'custom_class' ] );
2004 2004
 		}
2005 2005
 
2006 2006
 		/**
@@ -2038,16 +2038,16 @@  discard block
 block discarded – undo
2038 2038
 	     */
2039 2039
 	    $use_gf_adminonly_setting = apply_filters( 'gravityview/edit_entry/use_gf_admin_only_setting', empty( $edit_fields ), $form, $view_id );
2040 2040
 
2041
-	    if( $use_gf_adminonly_setting && false === GVCommon::has_cap( 'gravityforms_edit_entries', $this->entry['id'] ) ) {
2042
-			foreach( $fields as $k => $field ) {
2043
-				if( $field->adminOnly ) {
2041
+	    if ( $use_gf_adminonly_setting && false === GVCommon::has_cap( 'gravityforms_edit_entries', $this->entry[ 'id' ] ) ) {
2042
+			foreach ( $fields as $k => $field ) {
2043
+				if ( $field->adminOnly ) {
2044 2044
 				    unset( $fields[ $k ] );
2045 2045
 				}
2046 2046
 			}
2047 2047
 			return array_values( $fields );
2048 2048
 		}
2049 2049
 
2050
-	    foreach( $fields as &$field ) {
2050
+	    foreach ( $fields as &$field ) {
2051 2051
 		    $field->adminOnly = false;
2052 2052
 		}
2053 2053
 
@@ -2068,7 +2068,7 @@  discard block
 block discarded – undo
2068 2068
 	 */
2069 2069
 	private function unselect_default_values( $form ) {
2070 2070
 
2071
-	    foreach ( $form['fields'] as &$field ) {
2071
+	    foreach ( $form[ 'fields' ] as &$field ) {
2072 2072
 
2073 2073
 			if ( empty( $field->choices ) ) {
2074 2074
                 continue;
@@ -2076,7 +2076,7 @@  discard block
 block discarded – undo
2076 2076
 
2077 2077
             foreach ( $field->choices as &$choice ) {
2078 2078
 				if ( \GV\Utils::get( $choice, 'isSelected' ) ) {
2079
-					$choice['isSelected'] = false;
2079
+					$choice[ 'isSelected' ] = false;
2080 2080
 				}
2081 2081
 			}
2082 2082
 		}
@@ -2101,22 +2101,22 @@  discard block
 block discarded – undo
2101 2101
 	 */
2102 2102
 	function prefill_conditional_logic( $form ) {
2103 2103
 
2104
-		if( ! GFFormDisplay::has_conditional_logic( $form ) ) {
2104
+		if ( ! GFFormDisplay::has_conditional_logic( $form ) ) {
2105 2105
 			return $form;
2106 2106
 		}
2107 2107
 
2108 2108
 		// Have Conditional Logic pre-fill fields as if the data were default values
2109 2109
 		/** @var GF_Field $field */
2110
-		foreach ( $form['fields'] as &$field ) {
2110
+		foreach ( $form[ 'fields' ] as &$field ) {
2111 2111
 
2112
-			if( 'checkbox' === $field->type ) {
2112
+			if ( 'checkbox' === $field->type ) {
2113 2113
 				foreach ( $field->get_entry_inputs() as $key => $input ) {
2114
-				    $input_id = $input['id'];
2114
+				    $input_id = $input[ 'id' ];
2115 2115
 				    $choice = $field->choices[ $key ];
2116 2116
 				    $value = \GV\Utils::get( $this->entry, $input_id );
2117 2117
 				    $match = RGFormsModel::choice_value_match( $field, $choice, $value );
2118
-				    if( $match ) {
2119
-				        $field->choices[ $key ]['isSelected'] = true;
2118
+				    if ( $match ) {
2119
+				        $field->choices[ $key ][ 'isSelected' ] = true;
2120 2120
 				    }
2121 2121
 				}
2122 2122
 			} else {
@@ -2124,15 +2124,15 @@  discard block
 block discarded – undo
2124 2124
 				// We need to run through each field to set the default values
2125 2125
 				foreach ( $this->entry as $field_id => $field_value ) {
2126 2126
 
2127
-				    if( floatval( $field_id ) === floatval( $field->id ) ) {
2127
+				    if ( floatval( $field_id ) === floatval( $field->id ) ) {
2128 2128
 
2129
-				        if( 'list' === $field->type ) {
2129
+				        if ( 'list' === $field->type ) {
2130 2130
 				            $list_rows = maybe_unserialize( $field_value );
2131 2131
 
2132 2132
 				            $list_field_value = array();
2133
-				            foreach ( (array) $list_rows as $row ) {
2134
-				                foreach ( (array) $row as $column ) {
2135
-				                    $list_field_value[] = $column;
2133
+				            foreach ( (array)$list_rows as $row ) {
2134
+				                foreach ( (array)$row as $column ) {
2135
+				                    $list_field_value[ ] = $column;
2136 2136
 				                }
2137 2137
 				            }
2138 2138
 
@@ -2165,32 +2165,32 @@  discard block
 block discarded – undo
2165 2165
 		 * @see https://github.com/gravityview/GravityView/issues/840
2166 2166
 		 * @since develop
2167 2167
 		 */
2168
-		$the_form = GFAPI::get_form( $form['id'] );
2168
+		$the_form = GFAPI::get_form( $form[ 'id' ] );
2169 2169
 		$editable_ids = array();
2170
-		foreach ( $form['fields'] as $field ) {
2171
-			$editable_ids[] = $field['id']; // wp_list_pluck is destructive in this context
2170
+		foreach ( $form[ 'fields' ] as $field ) {
2171
+			$editable_ids[ ] = $field[ 'id' ]; // wp_list_pluck is destructive in this context
2172 2172
 		}
2173 2173
 		$remove_conditions_rule = array();
2174
-		foreach ( $the_form['fields'] as $field ) {
2175
-			if ( ! empty( $field->conditionalLogic ) && ! empty( $field->conditionalLogic['rules'] ) ) {
2176
-				foreach ( $field->conditionalLogic['rules'] as $i => $rule ) {
2177
-					if ( ! in_array( $rule['fieldId'], $editable_ids ) ) {
2174
+		foreach ( $the_form[ 'fields' ] as $field ) {
2175
+			if ( ! empty( $field->conditionalLogic ) && ! empty( $field->conditionalLogic[ 'rules' ] ) ) {
2176
+				foreach ( $field->conditionalLogic[ 'rules' ] as $i => $rule ) {
2177
+					if ( ! in_array( $rule[ 'fieldId' ], $editable_ids ) ) {
2178 2178
 						/**
2179 2179
 						 * This conditional field is not editable in this View.
2180 2180
 						 * We need to remove the rule, but only if it matches.
2181 2181
 						 */
2182
-						if ( $_field = GFAPI::get_field( $the_form, $rule['fieldId'] ) ) {
2182
+						if ( $_field = GFAPI::get_field( $the_form, $rule[ 'fieldId' ] ) ) {
2183 2183
 							$value = $_field->get_value_export( $this->entry );
2184
-						} elseif ( isset( $this->entry[ $rule['fieldId'] ] ) ) {
2185
-							$value = $this->entry[ $rule['fieldId'] ];
2184
+						} elseif ( isset( $this->entry[ $rule[ 'fieldId' ] ] ) ) {
2185
+							$value = $this->entry[ $rule[ 'fieldId' ] ];
2186 2186
 						} else {
2187
-							$value = gform_get_meta( $this->entry['id'], $rule['fieldId'] );
2187
+							$value = gform_get_meta( $this->entry[ 'id' ], $rule[ 'fieldId' ] );
2188 2188
 						}
2189 2189
 
2190
-						$match = GFFormsModel::matches_operation( $value, $rule['value'], $rule['operator'] );
2190
+						$match = GFFormsModel::matches_operation( $value, $rule[ 'value' ], $rule[ 'operator' ] );
2191 2191
 						
2192 2192
 						if ( $match ) {
2193
-							$remove_conditions_rule[] = array( $field['id'], $i );
2193
+							$remove_conditions_rule[ ] = array( $field[ 'id' ], $i );
2194 2194
 						}
2195 2195
 					}
2196 2196
 				}
@@ -2198,21 +2198,21 @@  discard block
 block discarded – undo
2198 2198
 		}
2199 2199
 
2200 2200
 		if ( $remove_conditions_rule ) {
2201
-			foreach ( $form['fields'] as &$field ) {
2201
+			foreach ( $form[ 'fields' ] as &$field ) {
2202 2202
 				foreach ( $remove_conditions_rule as $_remove_conditions_r ) {
2203 2203
 
2204 2204
 				    list( $rule_field_id, $rule_i ) = $_remove_conditions_r;
2205 2205
 
2206
-					if ( $field['id'] == $rule_field_id ) {
2207
-						unset( $field->conditionalLogic['rules'][ $rule_i ] );
2208
-						gravityview()->log->debug( 'Removed conditional rule #{rule} for field {field_id}', array( 'rule' => $rule_i, 'field_id' => $field['id'] ) );
2206
+					if ( $field[ 'id' ] == $rule_field_id ) {
2207
+						unset( $field->conditionalLogic[ 'rules' ][ $rule_i ] );
2208
+						gravityview()->log->debug( 'Removed conditional rule #{rule} for field {field_id}', array( 'rule' => $rule_i, 'field_id' => $field[ 'id' ] ) );
2209 2209
 					}
2210 2210
 				}
2211 2211
 			}
2212 2212
 		}
2213 2213
 
2214 2214
 		/** Normalize the indices... */
2215
-		$form['fields'] = array_values( $form['fields'] );
2215
+		$form[ 'fields' ] = array_values( $form[ 'fields' ] );
2216 2216
 
2217 2217
 		/**
2218 2218
 		 * @filter `gravityview/edit_entry/conditional_logic` Should the Edit Entry form use Gravity Forms conditional logic showing/hiding of fields?
@@ -2222,16 +2222,16 @@  discard block
 block discarded – undo
2222 2222
 		 */
2223 2223
 		$use_conditional_logic = apply_filters( 'gravityview/edit_entry/conditional_logic', true, $form );
2224 2224
 
2225
-		if( $use_conditional_logic ) {
2225
+		if ( $use_conditional_logic ) {
2226 2226
 			return $form;
2227 2227
 		}
2228 2228
 
2229
-		foreach( $form['fields'] as &$field ) {
2229
+		foreach ( $form[ 'fields' ] as &$field ) {
2230 2230
 			/* @var GF_Field $field */
2231 2231
 			$field->conditionalLogic = null;
2232 2232
 		}
2233 2233
 
2234
-		unset( $form['button']['conditionalLogic'] );
2234
+		unset( $form[ 'button' ][ 'conditionalLogic' ] );
2235 2235
 
2236 2236
 		return $form;
2237 2237
 
@@ -2248,7 +2248,7 @@  discard block
 block discarded – undo
2248 2248
 	 */
2249 2249
 	public function manage_conditional_logic( $has_conditional_logic, $form ) {
2250 2250
 
2251
-		if( ! $this->is_edit_entry() ) {
2251
+		if ( ! $this->is_edit_entry() ) {
2252 2252
 			return $has_conditional_logic;
2253 2253
 		}
2254 2254
 
@@ -2280,44 +2280,44 @@  discard block
 block discarded – undo
2280 2280
 		 *  2. There are two entries embedded using oEmbed
2281 2281
 		 *  3. One of the entries has just been saved
2282 2282
 		 */
2283
-		if( !empty( $_POST['lid'] ) && !empty( $_GET['entry'] ) && ( $_POST['lid'] !== $_GET['entry'] ) ) {
2283
+		if ( ! empty( $_POST[ 'lid' ] ) && ! empty( $_GET[ 'entry' ] ) && ( $_POST[ 'lid' ] !== $_GET[ 'entry' ] ) ) {
2284 2284
 
2285 2285
 			$error = true;
2286 2286
 
2287 2287
 		}
2288 2288
 
2289
-		if( !empty( $_GET['entry'] ) && (string)$this->entry['id'] !== $_GET['entry'] ) {
2289
+		if ( ! empty( $_GET[ 'entry' ] ) && (string)$this->entry[ 'id' ] !== $_GET[ 'entry' ] ) {
2290 2290
 
2291 2291
 			$error = true;
2292 2292
 
2293
-		} elseif( ! $this->verify_nonce() ) {
2293
+		} elseif ( ! $this->verify_nonce() ) {
2294 2294
 
2295 2295
 			/**
2296 2296
 			 * If the Entry is embedded, there may be two entries on the same page.
2297 2297
 			 * If that's the case, and one is being edited, the other should fail gracefully and not display an error.
2298 2298
 			 */
2299
-			if( GravityView_oEmbed::getInstance()->get_entry_id() ) {
2299
+			if ( GravityView_oEmbed::getInstance()->get_entry_id() ) {
2300 2300
 				$error = true;
2301 2301
 			} else {
2302
-				$error = __( 'The link to edit this entry is not valid; it may have expired.', 'gravityview');
2302
+				$error = __( 'The link to edit this entry is not valid; it may have expired.', 'gravityview' );
2303 2303
 			}
2304 2304
 
2305 2305
 		}
2306 2306
 
2307
-		if( ! GravityView_Edit_Entry::check_user_cap_edit_entry( $this->entry ) ) {
2308
-			$error = __( 'You do not have permission to edit this entry.', 'gravityview');
2307
+		if ( ! GravityView_Edit_Entry::check_user_cap_edit_entry( $this->entry ) ) {
2308
+			$error = __( 'You do not have permission to edit this entry.', 'gravityview' );
2309 2309
 		}
2310 2310
 
2311
-		if( $this->entry['status'] === 'trash' ) {
2312
-			$error = __('You cannot edit the entry; it is in the trash.', 'gravityview' );
2311
+		if ( $this->entry[ 'status' ] === 'trash' ) {
2312
+			$error = __( 'You cannot edit the entry; it is in the trash.', 'gravityview' );
2313 2313
 		}
2314 2314
 
2315 2315
 		// No errors; everything's fine here!
2316
-		if( empty( $error ) ) {
2316
+		if ( empty( $error ) ) {
2317 2317
 			return true;
2318 2318
 		}
2319 2319
 
2320
-		if( $echo && $error !== true ) {
2320
+		if ( $echo && $error !== true ) {
2321 2321
 
2322 2322
 	        $error = esc_html( $error );
2323 2323
 
@@ -2325,10 +2325,10 @@  discard block
 block discarded – undo
2325 2325
 	         * @since 1.9
2326 2326
 	         */
2327 2327
 	        if ( ! empty( $this->entry ) ) {
2328
-		        $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;" ) );
2328
+		        $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;" ) );
2329 2329
 	        }
2330 2330
 
2331
-			echo GVCommon::generate_notice( wpautop( $error ), 'gv-error error');
2331
+			echo GVCommon::generate_notice( wpautop( $error ), 'gv-error error' );
2332 2332
 		}
2333 2333
 
2334 2334
 		gravityview()->log->error( '{error}', array( 'error' => $error ) );
@@ -2348,17 +2348,17 @@  discard block
 block discarded – undo
2348 2348
 
2349 2349
 		$error = NULL;
2350 2350
 
2351
-		if( ! $this->check_user_cap_edit_field( $field ) ) {
2352
-			$error = __( 'You do not have permission to edit this field.', 'gravityview');
2351
+		if ( ! $this->check_user_cap_edit_field( $field ) ) {
2352
+			$error = __( 'You do not have permission to edit this field.', 'gravityview' );
2353 2353
 		}
2354 2354
 
2355 2355
 		// No errors; everything's fine here!
2356
-		if( empty( $error ) ) {
2356
+		if ( empty( $error ) ) {
2357 2357
 			return true;
2358 2358
 		}
2359 2359
 
2360
-		if( $echo ) {
2361
-			echo GVCommon::generate_notice( wpautop( esc_html( $error ) ), 'gv-error error');
2360
+		if ( $echo ) {
2361
+			echo GVCommon::generate_notice( wpautop( esc_html( $error ) ), 'gv-error error' );
2362 2362
 		}
2363 2363
 
2364 2364
 		gravityview()->log->error( '{error}', array( 'error' => $error ) );
@@ -2379,14 +2379,14 @@  discard block
 block discarded – undo
2379 2379
 	private function check_user_cap_edit_field( $field ) {
2380 2380
 
2381 2381
 		// If they can edit any entries (as defined in Gravity Forms), we're good.
2382
-		if( GVCommon::has_cap( array( 'gravityforms_edit_entries', 'gravityview_edit_others_entries' ) ) ) {
2382
+		if ( GVCommon::has_cap( array( 'gravityforms_edit_entries', 'gravityview_edit_others_entries' ) ) ) {
2383 2383
 			return true;
2384 2384
 		}
2385 2385
 
2386
-		$field_cap = isset( $field['allow_edit_cap'] ) ? $field['allow_edit_cap'] : false;
2386
+		$field_cap = isset( $field[ 'allow_edit_cap' ] ) ? $field[ 'allow_edit_cap' ] : false;
2387 2387
 
2388
-		if( $field_cap ) {
2389
-			return GVCommon::has_cap( $field['allow_edit_cap'] );
2388
+		if ( $field_cap ) {
2389
+			return GVCommon::has_cap( $field[ 'allow_edit_cap' ] );
2390 2390
 		}
2391 2391
 
2392 2392
 		return false;
@@ -2400,17 +2400,17 @@  discard block
 block discarded – undo
2400 2400
 	public function verify_nonce() {
2401 2401
 
2402 2402
 		// Verify form submitted for editing single
2403
-		if( $this->is_edit_entry_submission() ) {
2403
+		if ( $this->is_edit_entry_submission() ) {
2404 2404
 			$valid = wp_verify_nonce( $_POST[ self::$nonce_field ], self::$nonce_field );
2405 2405
 		}
2406 2406
 
2407 2407
 		// Verify
2408
-		else if( ! $this->is_edit_entry() ) {
2408
+		else if ( ! $this->is_edit_entry() ) {
2409 2409
 			$valid = false;
2410 2410
 		}
2411 2411
 
2412 2412
 		else {
2413
-			$valid = wp_verify_nonce( $_GET['edit'], self::$nonce_key );
2413
+			$valid = wp_verify_nonce( $_GET[ 'edit' ], self::$nonce_key );
2414 2414
 		}
2415 2415
 
2416 2416
 		/**
Please login to merge, or discard this patch.