@@ -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 | /** |