@@ -222,7 +222,6 @@ discard block |
||
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 |
||
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 ) { |
@@ -42,12 +42,12 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
@@ -18,84 +18,84 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
@@ -36,7 +36,7 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 | } |
@@ -31,7 +31,7 @@ discard block |
||
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 |
||
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 |
||
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>'; |
@@ -135,16 +135,16 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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’t have permission to edit this post.', 'gravityview') ); |
|
1404 | + $message = apply_filters( 'gravityview/edit_entry/unsupported_post_field_text', __( 'You don’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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 | /** |