@@ -112,16 +112,16 @@ discard block |
||
112 | 112 | function load() { |
113 | 113 | |
114 | 114 | /** @define "GRAVITYVIEW_DIR" "../../../" */ |
115 | - include_once( GRAVITYVIEW_DIR .'includes/class-admin-approve-entries.php' ); |
|
115 | + include_once( GRAVITYVIEW_DIR . 'includes/class-admin-approve-entries.php' ); |
|
116 | 116 | |
117 | 117 | // Don't display an embedded form when editing an entry |
118 | 118 | add_action( 'wp_head', array( $this, 'prevent_render_form' ) ); |
119 | 119 | add_action( 'wp_footer', array( $this, 'prevent_render_form' ) ); |
120 | 120 | |
121 | 121 | // Stop Gravity Forms processing what is ours! |
122 | - add_filter( 'wp', array( $this, 'prevent_maybe_process_form'), 8 ); |
|
122 | + add_filter( 'wp', array( $this, 'prevent_maybe_process_form' ), 8 ); |
|
123 | 123 | |
124 | - add_filter( 'gravityview_is_edit_entry', array( $this, 'is_edit_entry') ); |
|
124 | + add_filter( 'gravityview_is_edit_entry', array( $this, 'is_edit_entry' ) ); |
|
125 | 125 | |
126 | 126 | add_action( 'gravityview_edit_entry', array( $this, 'init' ) ); |
127 | 127 | |
@@ -132,7 +132,7 @@ discard block |
||
132 | 132 | add_filter( 'gform_plupload_settings', array( $this, 'modify_fileupload_settings' ), 10, 3 ); |
133 | 133 | |
134 | 134 | // Add fields expected by GFFormDisplay::validate() |
135 | - add_filter( 'gform_pre_validation', array( $this, 'gform_pre_validation') ); |
|
135 | + add_filter( 'gform_pre_validation', array( $this, 'gform_pre_validation' ) ); |
|
136 | 136 | |
137 | 137 | } |
138 | 138 | |
@@ -147,8 +147,8 @@ discard block |
||
147 | 147 | * @return void |
148 | 148 | */ |
149 | 149 | function prevent_render_form() { |
150 | - if( $this->is_edit_entry() ) { |
|
151 | - if( 'wp_head' === current_filter() ) { |
|
150 | + if ( $this->is_edit_entry() ) { |
|
151 | + if ( 'wp_head' === current_filter() ) { |
|
152 | 152 | add_filter( 'gform_shortcode_form', '__return_empty_string' ); |
153 | 153 | } else { |
154 | 154 | remove_filter( 'gform_shortcode_form', '__return_empty_string' ); |
@@ -163,10 +163,10 @@ discard block |
||
163 | 163 | */ |
164 | 164 | function prevent_maybe_process_form() { |
165 | 165 | |
166 | - do_action('gravityview_log_debug', 'GravityView_Edit_Entry[prevent_maybe_process_form] $_POSTed data (sanitized): ', esc_html( print_r( $_POST, true ) ) ); |
|
166 | + do_action( 'gravityview_log_debug', 'GravityView_Edit_Entry[prevent_maybe_process_form] $_POSTed data (sanitized): ', esc_html( print_r( $_POST, true ) ) ); |
|
167 | 167 | |
168 | - if( $this->is_edit_entry_submission() && $this->verify_nonce() ) { |
|
169 | - remove_action( 'wp', array( 'RGForms', 'maybe_process_form'), 9 ); |
|
168 | + if ( $this->is_edit_entry_submission() && $this->verify_nonce() ) { |
|
169 | + remove_action( 'wp', array( 'RGForms', 'maybe_process_form' ), 9 ); |
|
170 | 170 | } |
171 | 171 | } |
172 | 172 | |
@@ -178,7 +178,7 @@ discard block |
||
178 | 178 | |
179 | 179 | $gf_page = ( 'entry' === RGForms::get( 'view' ) ); |
180 | 180 | |
181 | - return ( $gf_page && isset( $_GET['edit'] ) || RGForms::post( 'action' ) === 'update' ); |
|
181 | + return ( $gf_page && isset( $_GET[ 'edit' ] ) || RGForms::post( 'action' ) === 'update' ); |
|
182 | 182 | } |
183 | 183 | |
184 | 184 | /** |
@@ -187,7 +187,7 @@ discard block |
||
187 | 187 | * @return boolean |
188 | 188 | */ |
189 | 189 | public function is_edit_entry_submission() { |
190 | - return !empty( $_POST[ self::$nonce_field ] ); |
|
190 | + return ! empty( $_POST[ self::$nonce_field ] ); |
|
191 | 191 | } |
192 | 192 | |
193 | 193 | /** |
@@ -198,13 +198,13 @@ discard block |
||
198 | 198 | |
199 | 199 | |
200 | 200 | $entries = $gravityview_view->getEntries(); |
201 | - $this->entry = $entries[0]; |
|
201 | + $this->entry = $entries[ 0 ]; |
|
202 | 202 | |
203 | 203 | $this->original_form = $this->form = $gravityview_view->getForm(); |
204 | 204 | $this->form_id = $gravityview_view->getFormId(); |
205 | 205 | $this->view_id = $gravityview_view->getViewId(); |
206 | 206 | |
207 | - self::$nonce_key = GravityView_Edit_Entry::get_nonce_key( $this->view_id, $this->form_id, $this->entry['id'] ); |
|
207 | + self::$nonce_key = GravityView_Edit_Entry::get_nonce_key( $this->view_id, $this->form_id, $this->entry[ 'id' ] ); |
|
208 | 208 | } |
209 | 209 | |
210 | 210 | |
@@ -224,12 +224,12 @@ discard block |
||
224 | 224 | $this->setup_vars(); |
225 | 225 | |
226 | 226 | // Multiple Views embedded, don't proceed if nonce fails |
227 | - if( $gv_data->has_multiple_views() && ! wp_verify_nonce( $_GET['edit'], self::$nonce_key ) ) { |
|
227 | + if ( $gv_data->has_multiple_views() && ! wp_verify_nonce( $_GET[ 'edit' ], self::$nonce_key ) ) { |
|
228 | 228 | return; |
229 | 229 | } |
230 | 230 | |
231 | 231 | // Sorry, you're not allowed here. |
232 | - if( false === $this->user_can_edit_entry( true ) ) { |
|
232 | + if ( false === $this->user_can_edit_entry( true ) ) { |
|
233 | 233 | return; |
234 | 234 | } |
235 | 235 | |
@@ -249,9 +249,9 @@ discard block |
||
249 | 249 | function print_scripts() { |
250 | 250 | $gravityview_view = GravityView_View::getInstance(); |
251 | 251 | |
252 | - wp_register_script( 'gform_gravityforms', GFCommon::get_base_url().'/js/gravityforms.js', array( 'jquery', 'gform_json', 'gform_placeholder', 'sack', 'plupload-all', 'gravityview-fe-view' ) ); |
|
252 | + wp_register_script( 'gform_gravityforms', GFCommon::get_base_url() . '/js/gravityforms.js', array( 'jquery', 'gform_json', 'gform_placeholder', 'sack', 'plupload-all', 'gravityview-fe-view' ) ); |
|
253 | 253 | |
254 | - GFFormDisplay::enqueue_form_scripts($gravityview_view->getForm(), false); |
|
254 | + GFFormDisplay::enqueue_form_scripts( $gravityview_view->getForm(), false ); |
|
255 | 255 | |
256 | 256 | // Sack is required for images |
257 | 257 | wp_print_scripts( array( 'sack', 'gform_gravityforms' ) ); |
@@ -263,32 +263,32 @@ discard block |
||
263 | 263 | */ |
264 | 264 | function process_save() { |
265 | 265 | |
266 | - if( empty( $_POST ) ) { |
|
266 | + if ( empty( $_POST ) ) { |
|
267 | 267 | return; |
268 | 268 | } |
269 | 269 | |
270 | 270 | // Make sure the entry, view, and form IDs are all correct |
271 | 271 | $valid = $this->verify_nonce(); |
272 | 272 | |
273 | - if( !$valid ) { |
|
274 | - do_action('gravityview_log_error', __METHOD__ . ' Nonce validation failed.' ); |
|
273 | + if ( ! $valid ) { |
|
274 | + do_action( 'gravityview_log_error', __METHOD__ . ' Nonce validation failed.' ); |
|
275 | 275 | return; |
276 | 276 | } |
277 | 277 | |
278 | - if( $this->entry['id'] !== $_POST['lid'] ) { |
|
279 | - do_action('gravityview_log_error', __METHOD__ . ' Entry ID did not match posted entry ID.' ); |
|
278 | + if ( $this->entry[ 'id' ] !== $_POST[ 'lid' ] ) { |
|
279 | + do_action( 'gravityview_log_error', __METHOD__ . ' Entry ID did not match posted entry ID.' ); |
|
280 | 280 | return; |
281 | 281 | } |
282 | 282 | |
283 | - do_action('gravityview_log_debug', 'GravityView_Edit_Entry[process_save] $_POSTed data (sanitized): ', esc_html( print_r( $_POST, true ) ) ); |
|
283 | + do_action( 'gravityview_log_debug', 'GravityView_Edit_Entry[process_save] $_POSTed data (sanitized): ', esc_html( print_r( $_POST, true ) ) ); |
|
284 | 284 | |
285 | 285 | $this->process_save_process_files( $this->form_id ); |
286 | 286 | |
287 | 287 | $this->validate(); |
288 | 288 | |
289 | - if( $this->is_valid ) { |
|
289 | + if ( $this->is_valid ) { |
|
290 | 290 | |
291 | - do_action('gravityview_log_debug', 'GravityView_Edit_Entry[process_save] Submission is valid.' ); |
|
291 | + do_action( 'gravityview_log_debug', 'GravityView_Edit_Entry[process_save] Submission is valid.' ); |
|
292 | 292 | |
293 | 293 | /** |
294 | 294 | * @hack This step is needed to unset the adminOnly from form fields, to add the calculation fields |
@@ -298,12 +298,12 @@ discard block |
||
298 | 298 | /** |
299 | 299 | * @hack to avoid the capability validation of the method save_lead for GF 1.9+ |
300 | 300 | */ |
301 | - unset( $_GET['page'] ); |
|
301 | + unset( $_GET[ 'page' ] ); |
|
302 | 302 | |
303 | 303 | GFFormsModel::save_lead( $form, $this->entry ); |
304 | 304 | |
305 | 305 | // If there's a post associated with the entry, process post fields |
306 | - if( !empty( $this->entry['post_id'] ) ) { |
|
306 | + if ( ! empty( $this->entry[ 'post_id' ] ) ) { |
|
307 | 307 | $this->maybe_update_post_fields( $form ); |
308 | 308 | } |
309 | 309 | |
@@ -318,7 +318,7 @@ discard block |
||
318 | 318 | * @param array $form Gravity Forms form array |
319 | 319 | * @param string $entry_id Numeric ID of the entry that was updated |
320 | 320 | */ |
321 | - do_action( 'gravityview/edit_entry/after_update', $this->form, $this->entry['id'] ); |
|
321 | + do_action( 'gravityview/edit_entry/after_update', $this->form, $this->entry[ 'id' ] ); |
|
322 | 322 | } |
323 | 323 | |
324 | 324 | } // process_save |
@@ -352,11 +352,11 @@ discard block |
||
352 | 352 | * @return mixed |
353 | 353 | */ |
354 | 354 | public function modify_fileupload_settings( $plupload_init, $form_id, $instance ) { |
355 | - if( ! $this->is_edit_entry() ) { |
|
355 | + if ( ! $this->is_edit_entry() ) { |
|
356 | 356 | return $plupload_init; |
357 | 357 | } |
358 | 358 | |
359 | - $plupload_init['gf_vars']['max_files'] = 0; |
|
359 | + $plupload_init[ 'gf_vars' ][ 'max_files' ] = 0; |
|
360 | 360 | |
361 | 361 | return $plupload_init; |
362 | 362 | } |
@@ -370,19 +370,19 @@ discard block |
||
370 | 370 | |
371 | 371 | $form = $this->form; |
372 | 372 | |
373 | - foreach( $form['fields'] as $k => &$field ) { |
|
373 | + foreach ( $form[ 'fields' ] as $k => &$field ) { |
|
374 | 374 | |
375 | 375 | // Remove the fields with calculation formulas before save to avoid conflicts with GF logic |
376 | 376 | // @since 1.16.3 |
377 | - if( $field->has_calculation() ) { |
|
378 | - unset( $form['fields'][ $k ] ); |
|
377 | + if ( $field->has_calculation() ) { |
|
378 | + unset( $form[ 'fields' ][ $k ] ); |
|
379 | 379 | } |
380 | 380 | |
381 | 381 | $field->adminOnly = false; |
382 | 382 | |
383 | - if( isset( $field->inputs ) && is_array( $field->inputs ) ) { |
|
384 | - foreach( $field->inputs as $key => $input ) { |
|
385 | - $field->inputs[ $key ][ 'id' ] = (string)$input['id']; |
|
383 | + if ( isset( $field->inputs ) && is_array( $field->inputs ) ) { |
|
384 | + foreach ( $field->inputs as $key => $input ) { |
|
385 | + $field->inputs[ $key ][ 'id' ] = (string)$input[ 'id' ]; |
|
386 | 386 | } |
387 | 387 | } |
388 | 388 | } |
@@ -396,20 +396,20 @@ discard block |
||
396 | 396 | $update = false; |
397 | 397 | |
398 | 398 | // get the most up to date entry values |
399 | - $entry = GFAPI::get_entry( $this->entry['id'] ); |
|
399 | + $entry = GFAPI::get_entry( $this->entry[ 'id' ] ); |
|
400 | 400 | |
401 | - if( !empty( $this->fields_with_calculation ) ) { |
|
401 | + if ( ! empty( $this->fields_with_calculation ) ) { |
|
402 | 402 | $update = true; |
403 | 403 | foreach ( $this->fields_with_calculation as $calc_field ) { |
404 | 404 | $inputs = $calc_field->get_entry_inputs(); |
405 | 405 | if ( is_array( $inputs ) ) { |
406 | 406 | foreach ( $inputs as $input ) { |
407 | - $input_name = 'input_' . str_replace( '.', '_', $input['id'] ); |
|
408 | - $entry[ strval( $input['id'] ) ] = RGFormsModel::prepare_value( $form, $calc_field, '', $input_name, $entry['id'], $entry ); |
|
407 | + $input_name = 'input_' . str_replace( '.', '_', $input[ 'id' ] ); |
|
408 | + $entry[ strval( $input[ 'id' ] ) ] = RGFormsModel::prepare_value( $form, $calc_field, '', $input_name, $entry[ 'id' ], $entry ); |
|
409 | 409 | } |
410 | 410 | } else { |
411 | - $input_name = 'input_' . str_replace( '.', '_', $calc_field->id); |
|
412 | - $entry[ strval( $calc_field->id ) ] = RGFormsModel::prepare_value( $form, $calc_field, '', $input_name, $entry['id'], $entry ); |
|
411 | + $input_name = 'input_' . str_replace( '.', '_', $calc_field->id ); |
|
412 | + $entry[ strval( $calc_field->id ) ] = RGFormsModel::prepare_value( $form, $calc_field, '', $input_name, $entry[ 'id' ], $entry ); |
|
413 | 413 | } |
414 | 414 | } |
415 | 415 | |
@@ -419,16 +419,16 @@ discard block |
||
419 | 419 | if ( ! empty( $this->total_fields ) ) { |
420 | 420 | $update = true; |
421 | 421 | foreach ( $this->total_fields as $total_field ) { |
422 | - $input_name = 'input_' . str_replace( '.', '_', $total_field->id); |
|
423 | - $entry[ strval( $total_field->id ) ] = RGFormsModel::prepare_value( $form, $total_field, '', $input_name, $entry['id'], $entry ); |
|
422 | + $input_name = 'input_' . str_replace( '.', '_', $total_field->id ); |
|
423 | + $entry[ strval( $total_field->id ) ] = RGFormsModel::prepare_value( $form, $total_field, '', $input_name, $entry[ 'id' ], $entry ); |
|
424 | 424 | } |
425 | 425 | } |
426 | 426 | |
427 | - if( $update ) { |
|
427 | + if ( $update ) { |
|
428 | 428 | |
429 | 429 | $return_entry = GFAPI::update_entry( $entry ); |
430 | 430 | |
431 | - if( is_wp_error( $return_entry ) ) { |
|
431 | + if ( is_wp_error( $return_entry ) ) { |
|
432 | 432 | do_action( 'gravityview_log_error', 'Updating the entry calculation and total fields failed', $return_entry ); |
433 | 433 | } else { |
434 | 434 | do_action( 'gravityview_log_debug', 'Updating the entry calculation and total fields succeeded' ); |
@@ -446,11 +446,11 @@ discard block |
||
446 | 446 | */ |
447 | 447 | function maybe_update_post_fields( $form ) { |
448 | 448 | |
449 | - $post_id = $this->entry['post_id']; |
|
449 | + $post_id = $this->entry[ 'post_id' ]; |
|
450 | 450 | |
451 | 451 | // Security check |
452 | - if( false === GVCommon::has_cap( 'edit_post', $post_id ) ) { |
|
453 | - do_action( 'gravityview_log_error', 'The current user does not have the ability to edit Post #'.$post_id ); |
|
452 | + if ( false === GVCommon::has_cap( 'edit_post', $post_id ) ) { |
|
453 | + do_action( 'gravityview_log_error', 'The current user does not have the ability to edit Post #' . $post_id ); |
|
454 | 454 | return; |
455 | 455 | } |
456 | 456 | |
@@ -459,7 +459,7 @@ discard block |
||
459 | 459 | $updated_post = $original_post = get_post( $post_id ); |
460 | 460 | |
461 | 461 | // get the most up to date entry values |
462 | - $entry = GFAPI::get_entry( $this->entry['id'] ); |
|
462 | + $entry = GFAPI::get_entry( $this->entry[ 'id' ] ); |
|
463 | 463 | |
464 | 464 | foreach ( $entry as $field_id => $value ) { |
465 | 465 | |
@@ -467,16 +467,16 @@ discard block |
||
467 | 467 | |
468 | 468 | $field = RGFormsModel::get_field( $form, $field_id ); |
469 | 469 | |
470 | - if( class_exists('GF_Fields') ) { |
|
470 | + if ( class_exists( 'GF_Fields' ) ) { |
|
471 | 471 | $field = GF_Fields::create( $field ); |
472 | 472 | } |
473 | 473 | |
474 | - if( GFCommon::is_post_field( $field ) ) { |
|
474 | + if ( GFCommon::is_post_field( $field ) ) { |
|
475 | 475 | |
476 | 476 | // Get the value of the field, including $_POSTed value |
477 | 477 | $value = RGFormsModel::get_field_value( $field ); |
478 | 478 | |
479 | - switch( $field->type ) { |
|
479 | + switch ( $field->type ) { |
|
480 | 480 | |
481 | 481 | case 'post_title': |
482 | 482 | case 'post_content': |
@@ -494,14 +494,14 @@ discard block |
||
494 | 494 | wp_set_post_categories( $post_id, $categories, false ); |
495 | 495 | |
496 | 496 | // if post_category is type checkbox, then value is an array of inputs |
497 | - if( isset( $value[ strval( $field_id ) ] ) ) { |
|
498 | - foreach( $value as $input_id => $val ) { |
|
497 | + if ( isset( $value[ strval( $field_id ) ] ) ) { |
|
498 | + foreach ( $value as $input_id => $val ) { |
|
499 | 499 | $input_name = 'input_' . str_replace( '.', '_', $input_id ); |
500 | - $entry[ strval( $input_id ) ] = RGFormsModel::prepare_value( $form, $field, $val, $input_name, $entry['id'], $entry ); |
|
500 | + $entry[ strval( $input_id ) ] = RGFormsModel::prepare_value( $form, $field, $val, $input_name, $entry[ 'id' ], $entry ); |
|
501 | 501 | } |
502 | 502 | } else { |
503 | 503 | $input_name = 'input_' . str_replace( '.', '_', $field_id ); |
504 | - $entry[ strval( $field_id ) ] = RGFormsModel::prepare_value( $form, $field, $value, $input_name, $entry['id'], $entry ); |
|
504 | + $entry[ strval( $field_id ) ] = RGFormsModel::prepare_value( $form, $field, $value, $input_name, $entry[ 'id' ], $entry ); |
|
505 | 505 | } |
506 | 506 | |
507 | 507 | break; |
@@ -511,12 +511,12 @@ discard block |
||
511 | 511 | $custom_field_name = $field->postCustomFieldName; |
512 | 512 | |
513 | 513 | // Only certain custom field types are supported |
514 | - switch( $input_type ) { |
|
514 | + switch ( $input_type ) { |
|
515 | 515 | case 'fileupload': |
516 | 516 | /** @noinspection PhpMissingBreakStatementInspection */ |
517 | 517 | case 'list': |
518 | - if( ! is_string( $value ) ) { |
|
519 | - $value = function_exists('wp_json_encode') ? wp_json_encode( $value ) : json_encode( $value ); |
|
518 | + if ( ! is_string( $value ) ) { |
|
519 | + $value = function_exists( 'wp_json_encode' ) ? wp_json_encode( $value ) : json_encode( $value ); |
|
520 | 520 | } |
521 | 521 | // break; left intentionally out |
522 | 522 | default: |
@@ -529,11 +529,11 @@ discard block |
||
529 | 529 | |
530 | 530 | $input_name = 'input_' . $field_id; |
531 | 531 | |
532 | - if ( !empty( $_FILES[ $input_name ]['name'] ) ) { |
|
532 | + if ( ! empty( $_FILES[ $input_name ][ 'name' ] ) ) { |
|
533 | 533 | |
534 | 534 | // We have a new image |
535 | 535 | |
536 | - $value = RGFormsModel::prepare_value( $form, $field, $value, $input_name, $entry['id'] ); |
|
536 | + $value = RGFormsModel::prepare_value( $form, $field, $value, $input_name, $entry[ 'id' ] ); |
|
537 | 537 | |
538 | 538 | // is this field set as featured image, if not, leave |
539 | 539 | if ( ! $field->postFeaturedImage ) { |
@@ -543,9 +543,9 @@ discard block |
||
543 | 543 | $ary = ! empty( $value ) ? explode( '|:|', $value ) : array(); |
544 | 544 | $img_url = rgar( $ary, 0 ); |
545 | 545 | |
546 | - $img_title = count( $ary ) > 1 ? $ary[1] : ''; |
|
547 | - $img_caption = count( $ary ) > 2 ? $ary[2] : ''; |
|
548 | - $img_description = count( $ary ) > 3 ? $ary[3] : ''; |
|
546 | + $img_title = count( $ary ) > 1 ? $ary[ 1 ] : ''; |
|
547 | + $img_caption = count( $ary ) > 2 ? $ary[ 2 ] : ''; |
|
548 | + $img_description = count( $ary ) > 3 ? $ary[ 3 ] : ''; |
|
549 | 549 | |
550 | 550 | $image_meta = array( |
551 | 551 | 'post_excerpt' => $img_caption, |
@@ -554,7 +554,7 @@ discard block |
||
554 | 554 | |
555 | 555 | //adding title only if it is not empty. It will default to the file name if it is not in the array |
556 | 556 | if ( ! empty( $img_title ) ) { |
557 | - $image_meta['post_title'] = $img_title; |
|
557 | + $image_meta[ 'post_title' ] = $img_title; |
|
558 | 558 | } |
559 | 559 | |
560 | 560 | //todo: As soon as \GFFormsModel::media_handle_upload becomes a public method, move this call to \GFFormsModel::media_handle_upload and remove the hack from this class. |
@@ -567,7 +567,7 @@ discard block |
||
567 | 567 | |
568 | 568 | break; |
569 | 569 | |
570 | - } elseif ( !empty( $_POST[ $input_name ] ) && is_array( $value ) ) { |
|
570 | + } elseif ( ! empty( $_POST[ $input_name ] ) && is_array( $value ) ) { |
|
571 | 571 | |
572 | 572 | // Same image although the image title, caption or description might have changed |
573 | 573 | |
@@ -575,11 +575,11 @@ discard block |
||
575 | 575 | $img_url = rgar( $ary, 0 ); |
576 | 576 | |
577 | 577 | // is this really the same image or something went wrong ? |
578 | - if( $img_url === $_POST[ $input_name ] ) { |
|
578 | + if ( $img_url === $_POST[ $input_name ] ) { |
|
579 | 579 | |
580 | - $img_title = isset( $value[ $field_id .'.1' ] ) ? $value[ $field_id .'.1' ] : ''; |
|
581 | - $img_caption = isset( $value[ $field_id .'.4' ] ) ? $value[ $field_id .'.4' ] : ''; |
|
582 | - $img_description = isset( $value[ $field_id .'.7' ] ) ? $value[ $field_id .'.7' ] : ''; |
|
580 | + $img_title = isset( $value[ $field_id . '.1' ] ) ? $value[ $field_id . '.1' ] : ''; |
|
581 | + $img_caption = isset( $value[ $field_id . '.4' ] ) ? $value[ $field_id . '.4' ] : ''; |
|
582 | + $img_description = isset( $value[ $field_id . '.7' ] ) ? $value[ $field_id . '.7' ] : ''; |
|
583 | 583 | |
584 | 584 | $value = ! empty( $img_url ) ? $img_url . "|:|" . $img_title . "|:|" . $img_caption . "|:|" . $img_description : ''; |
585 | 585 | |
@@ -613,12 +613,12 @@ discard block |
||
613 | 613 | } |
614 | 614 | |
615 | 615 | //ignore fields that have not changed |
616 | - if ( $value === rgget( (string) $field_id, $entry ) ) { |
|
616 | + if ( $value === rgget( (string)$field_id, $entry ) ) { |
|
617 | 617 | continue; |
618 | 618 | } |
619 | 619 | |
620 | 620 | // update entry |
621 | - if( 'post_category' !== $field->type ) { |
|
621 | + if ( 'post_category' !== $field->type ) { |
|
622 | 622 | $entry[ strval( $field_id ) ] = $value; |
623 | 623 | } |
624 | 624 | |
@@ -628,25 +628,25 @@ discard block |
||
628 | 628 | |
629 | 629 | } |
630 | 630 | |
631 | - if( $update_entry ) { |
|
631 | + if ( $update_entry ) { |
|
632 | 632 | |
633 | 633 | $return_entry = GFAPI::update_entry( $entry ); |
634 | 634 | |
635 | - if( is_wp_error( $return_entry ) ) { |
|
635 | + if ( is_wp_error( $return_entry ) ) { |
|
636 | 636 | do_action( 'gravityview_log_error', 'Updating the entry post fields failed', $return_entry ); |
637 | 637 | } else { |
638 | - do_action( 'gravityview_log_debug', 'Updating the entry post fields for post #'.$post_id.' succeeded' ); |
|
638 | + do_action( 'gravityview_log_debug', 'Updating the entry post fields for post #' . $post_id . ' succeeded' ); |
|
639 | 639 | } |
640 | 640 | |
641 | 641 | } |
642 | 642 | |
643 | 643 | $return_post = wp_update_post( $updated_post, true ); |
644 | 644 | |
645 | - if( is_wp_error( $return_post ) ) { |
|
645 | + if ( is_wp_error( $return_post ) ) { |
|
646 | 646 | $return_post->add_data( $updated_post, '$updated_post' ); |
647 | 647 | do_action( 'gravityview_log_error', 'Updating the post content failed', $return_post ); |
648 | 648 | } else { |
649 | - do_action( 'gravityview_log_debug', 'Updating the post content for post #'.$post_id.' succeeded', $updated_post ); |
|
649 | + do_action( 'gravityview_log_debug', 'Updating the post content for post #' . $post_id . ' succeeded', $updated_post ); |
|
650 | 650 | } |
651 | 651 | |
652 | 652 | } |
@@ -663,18 +663,18 @@ discard block |
||
663 | 663 | */ |
664 | 664 | function after_update() { |
665 | 665 | |
666 | - do_action( 'gform_after_update_entry', $this->form, $this->entry['id'] ); |
|
667 | - do_action( "gform_after_update_entry_{$this->form['id']}", $this->form, $this->entry['id'] ); |
|
666 | + do_action( 'gform_after_update_entry', $this->form, $this->entry[ 'id' ] ); |
|
667 | + do_action( "gform_after_update_entry_{$this->form[ 'id' ]}", $this->form, $this->entry[ 'id' ] ); |
|
668 | 668 | |
669 | 669 | // Re-define the entry now that we've updated it. |
670 | - $entry = RGFormsModel::get_lead( $this->entry['id'] ); |
|
670 | + $entry = RGFormsModel::get_lead( $this->entry[ 'id' ] ); |
|
671 | 671 | |
672 | 672 | $entry = GFFormsModel::set_entry_meta( $entry, $this->form ); |
673 | 673 | |
674 | 674 | // We need to clear the cache because Gravity Forms caches the field values, which |
675 | 675 | // we have just updated. |
676 | - foreach ($this->form['fields'] as $key => $field) { |
|
677 | - GFFormsModel::refresh_lead_field_value( $entry['id'], $field->id ); |
|
676 | + foreach ( $this->form[ 'fields' ] as $key => $field ) { |
|
677 | + GFFormsModel::refresh_lead_field_value( $entry[ 'id' ], $field->id ); |
|
678 | 678 | } |
679 | 679 | |
680 | 680 | $this->entry = $entry; |
@@ -692,7 +692,7 @@ discard block |
||
692 | 692 | |
693 | 693 | <div class="gv-edit-entry-wrapper"><?php |
694 | 694 | |
695 | - $javascript = gravityview_ob_include( GravityView_Edit_Entry::$file .'/partials/inline-javascript.php', $this ); |
|
695 | + $javascript = gravityview_ob_include( GravityView_Edit_Entry::$file . '/partials/inline-javascript.php', $this ); |
|
696 | 696 | |
697 | 697 | /** |
698 | 698 | * Fixes weird wpautop() issue |
@@ -708,7 +708,7 @@ discard block |
||
708 | 708 | * @param string $edit_entry_title Modify the "Edit Entry" title |
709 | 709 | * @param GravityView_Edit_Entry_Render $this This object |
710 | 710 | */ |
711 | - $edit_entry_title = apply_filters('gravityview_edit_entry_title', __('Edit Entry', 'gravityview'), $this ); |
|
711 | + $edit_entry_title = apply_filters( 'gravityview_edit_entry_title', __( 'Edit Entry', 'gravityview' ), $this ); |
|
712 | 712 | |
713 | 713 | echo esc_attr( $edit_entry_title ); |
714 | 714 | ?></span> |
@@ -748,20 +748,20 @@ discard block |
||
748 | 748 | */ |
749 | 749 | private function maybe_print_message() { |
750 | 750 | |
751 | - if( rgpost('action') === 'update' ) { |
|
751 | + if ( rgpost( 'action' ) === 'update' ) { |
|
752 | 752 | |
753 | 753 | $back_link = esc_url( remove_query_arg( array( 'page', 'view', 'edit' ) ) ); |
754 | 754 | |
755 | - if( ! $this->is_valid ){ |
|
755 | + if ( ! $this->is_valid ) { |
|
756 | 756 | |
757 | 757 | // Keeping this compatible with Gravity Forms. |
758 | - $validation_message = "<div class='validation_error'>" . __('There was a problem with your submission.', 'gravityview') . " " . __('Errors have been highlighted below.', 'gravityview') . "</div>"; |
|
759 | - $message = apply_filters("gform_validation_message_{$this->form['id']}", apply_filters("gform_validation_message", $validation_message, $this->form), $this->form); |
|
758 | + $validation_message = "<div class='validation_error'>" . __( 'There was a problem with your submission.', 'gravityview' ) . " " . __( 'Errors have been highlighted below.', 'gravityview' ) . "</div>"; |
|
759 | + $message = apply_filters( "gform_validation_message_{$this->form[ 'id' ]}", apply_filters( "gform_validation_message", $validation_message, $this->form ), $this->form ); |
|
760 | 760 | |
761 | - echo GVCommon::generate_notice( $message , 'gv-error' ); |
|
761 | + echo GVCommon::generate_notice( $message, 'gv-error' ); |
|
762 | 762 | |
763 | 763 | } else { |
764 | - $entry_updated_message = sprintf( esc_attr__('Entry Updated. %sReturn to Entry%s', 'gravityview'), '<a href="'. $back_link .'">', '</a>' ); |
|
764 | + $entry_updated_message = sprintf( esc_attr__( 'Entry Updated. %sReturn to Entry%s', 'gravityview' ), '<a href="' . $back_link . '">', '</a>' ); |
|
765 | 765 | |
766 | 766 | /** |
767 | 767 | * @filter `gravityview/edit_entry/success` Modify the edit entry success message (including the anchor link) |
@@ -771,7 +771,7 @@ discard block |
||
771 | 771 | * @param array $entry Gravity Forms entry array |
772 | 772 | * @param string $back_link URL to return to the original entry. @since 1.6 |
773 | 773 | */ |
774 | - $message = apply_filters( 'gravityview/edit_entry/success', $entry_updated_message , $this->view_id, $this->entry, $back_link ); |
|
774 | + $message = apply_filters( 'gravityview/edit_entry/success', $entry_updated_message, $this->view_id, $this->entry, $back_link ); |
|
775 | 775 | |
776 | 776 | echo GVCommon::generate_notice( $message ); |
777 | 777 | } |
@@ -788,24 +788,24 @@ discard block |
||
788 | 788 | */ |
789 | 789 | private function render_edit_form() { |
790 | 790 | |
791 | - add_filter( 'gform_pre_render', array( $this, 'filter_modify_form_fields'), 5000, 3 ); |
|
792 | - add_filter( 'gform_submit_button', array( $this, 'render_form_buttons') ); |
|
791 | + add_filter( 'gform_pre_render', array( $this, 'filter_modify_form_fields' ), 5000, 3 ); |
|
792 | + add_filter( 'gform_submit_button', array( $this, 'render_form_buttons' ) ); |
|
793 | 793 | add_filter( 'gform_disable_view_counter', '__return_true' ); |
794 | 794 | |
795 | 795 | add_filter( 'gform_field_input', array( $this, 'verify_user_can_edit_post' ), 5, 5 ); |
796 | 796 | add_filter( 'gform_field_input', array( $this, 'modify_edit_field_input' ), 10, 5 ); |
797 | 797 | |
798 | - add_filter( 'gform_field_value', array( $this, 'fix_survey_fields_value'), 10, 3 ); |
|
798 | + add_filter( 'gform_field_value', array( $this, 'fix_survey_fields_value' ), 10, 3 ); |
|
799 | 799 | |
800 | 800 | // We need to remove the fake $_GET['page'] arg to avoid rendering form as if in admin. |
801 | - unset( $_GET['page'] ); |
|
801 | + unset( $_GET[ 'page' ] ); |
|
802 | 802 | |
803 | 803 | // TODO: Verify multiple-page forms |
804 | 804 | // TODO: Product fields are not editable |
805 | 805 | |
806 | - $html = GFFormDisplay::get_form( $this->form['id'], false, false, true, $this->entry ); |
|
806 | + $html = GFFormDisplay::get_form( $this->form[ 'id' ], false, false, true, $this->entry ); |
|
807 | 807 | |
808 | - remove_filter( 'gform_field_value', array( $this, 'fix_survey_fields_value'), 10 ); |
|
808 | + remove_filter( 'gform_field_value', array( $this, 'fix_survey_fields_value' ), 10 ); |
|
809 | 809 | remove_filter( 'gform_pre_render', array( $this, 'filter_modify_form_fields' ), 5000 ); |
810 | 810 | remove_filter( 'gform_submit_button', array( $this, 'render_form_buttons' ) ); |
811 | 811 | remove_filter( 'gform_disable_view_counter', '__return_true' ); |
@@ -829,8 +829,8 @@ discard block |
||
829 | 829 | */ |
830 | 830 | function fix_survey_fields_value( $value, $field, $name ) { |
831 | 831 | |
832 | - if( 'survey' === $field->type && '' === $value ) { |
|
833 | - $value = $this->entry["{$field->id}"]; |
|
832 | + if ( 'survey' === $field->type && '' === $value ) { |
|
833 | + $value = $this->entry[ "{$field->id}" ]; |
|
834 | 834 | } |
835 | 835 | |
836 | 836 | return $value; |
@@ -842,7 +842,7 @@ discard block |
||
842 | 842 | * @return string |
843 | 843 | */ |
844 | 844 | public function render_form_buttons() { |
845 | - return gravityview_ob_include( GravityView_Edit_Entry::$file .'/partials/form-buttons.php', $this ); |
|
845 | + return gravityview_ob_include( GravityView_Edit_Entry::$file . '/partials/form-buttons.php', $this ); |
|
846 | 846 | } |
847 | 847 | |
848 | 848 | |
@@ -862,17 +862,17 @@ discard block |
||
862 | 862 | public function filter_modify_form_fields( $form, $ajax = false, $field_values = '' ) { |
863 | 863 | |
864 | 864 | // In case we have validated the form, use it to inject the validation results into the form render |
865 | - if( isset( $this->form_after_validation ) ) { |
|
865 | + if ( isset( $this->form_after_validation ) ) { |
|
866 | 866 | $form = $this->form_after_validation; |
867 | 867 | } else { |
868 | - $form['fields'] = $this->get_configured_edit_fields( $form, $this->view_id ); |
|
868 | + $form[ 'fields' ] = $this->get_configured_edit_fields( $form, $this->view_id ); |
|
869 | 869 | } |
870 | 870 | |
871 | 871 | $form = $this->filter_conditional_logic( $form ); |
872 | 872 | |
873 | 873 | // for now we don't support Save and Continue feature. |
874 | - if( ! self::$supports_save_and_continue ) { |
|
875 | - unset( $form['save'] ); |
|
874 | + if ( ! self::$supports_save_and_continue ) { |
|
875 | + unset( $form[ 'save' ] ); |
|
876 | 876 | } |
877 | 877 | |
878 | 878 | return $form; |
@@ -893,29 +893,29 @@ discard block |
||
893 | 893 | */ |
894 | 894 | function verify_user_can_edit_post( $field_content = '', $field, $value, $lead_id = 0, $form_id ) { |
895 | 895 | |
896 | - if( GFCommon::is_post_field( $field ) ) { |
|
896 | + if ( GFCommon::is_post_field( $field ) ) { |
|
897 | 897 | |
898 | 898 | $message = null; |
899 | 899 | |
900 | 900 | // First, make sure they have the capability to edit the post. |
901 | - if( false === current_user_can( 'edit_post', $this->entry['post_id'] ) ) { |
|
901 | + if ( false === current_user_can( 'edit_post', $this->entry[ 'post_id' ] ) ) { |
|
902 | 902 | |
903 | 903 | /** |
904 | 904 | * @filter `gravityview/edit_entry/unsupported_post_field_text` Modify the message when someone isn't able to edit a post |
905 | 905 | * @param string $message The existing "You don't have permission..." text |
906 | 906 | */ |
907 | - $message = apply_filters('gravityview/edit_entry/unsupported_post_field_text', __('You don’t have permission to edit this post.', 'gravityview') ); |
|
907 | + $message = apply_filters( 'gravityview/edit_entry/unsupported_post_field_text', __( 'You don’t have permission to edit this post.', 'gravityview' ) ); |
|
908 | 908 | |
909 | - } elseif( null === get_post( $this->entry['post_id'] ) ) { |
|
909 | + } elseif ( null === get_post( $this->entry[ 'post_id' ] ) ) { |
|
910 | 910 | /** |
911 | 911 | * @filter `gravityview/edit_entry/no_post_text` Modify the message when someone is editing an entry attached to a post that no longer exists |
912 | 912 | * @param string $message The existing "This field is not editable; the post no longer exists." text |
913 | 913 | */ |
914 | - $message = apply_filters('gravityview/edit_entry/no_post_text', __('This field is not editable; the post no longer exists.', 'gravityview' ) ); |
|
914 | + $message = apply_filters( 'gravityview/edit_entry/no_post_text', __( 'This field is not editable; the post no longer exists.', 'gravityview' ) ); |
|
915 | 915 | } |
916 | 916 | |
917 | - if( $message ) { |
|
918 | - $field_content = sprintf('<div class="ginput_container ginput_container_' . $field->type . '">%s</div>', wpautop( $message ) ); |
|
917 | + if ( $message ) { |
|
918 | + $field_content = sprintf( '<div class="ginput_container ginput_container_' . $field->type . '">%s</div>', wpautop( $message ) ); |
|
919 | 919 | } |
920 | 920 | } |
921 | 921 | |
@@ -940,8 +940,8 @@ discard block |
||
940 | 940 | |
941 | 941 | // If the form has been submitted, then we don't need to pre-fill the values, |
942 | 942 | // Except for fileupload type and when a field input is overridden- run always!! |
943 | - if( |
|
944 | - ( $this->is_edit_entry_submission() && !in_array( $field->type, array( 'fileupload', 'post_image' ) ) ) |
|
943 | + if ( |
|
944 | + ( $this->is_edit_entry_submission() && ! in_array( $field->type, array( 'fileupload', 'post_image' ) ) ) |
|
945 | 945 | && false === ( $gv_field && is_callable( array( $gv_field, 'get_field_input' ) ) ) |
946 | 946 | || ! empty( $field_content ) |
947 | 947 | || GFCommon::is_product_field( $field->type ) // Prevent product fields from appearing editable |
@@ -950,8 +950,8 @@ discard block |
||
950 | 950 | } |
951 | 951 | |
952 | 952 | // Turn on Admin-style display for file upload fields only |
953 | - if( 'fileupload' === $field->type ) { |
|
954 | - $_GET['page'] = 'gf_entries'; |
|
953 | + if ( 'fileupload' === $field->type ) { |
|
954 | + $_GET[ 'page' ] = 'gf_entries'; |
|
955 | 955 | } |
956 | 956 | |
957 | 957 | // SET SOME FIELD DEFAULTS TO PREVENT ISSUES |
@@ -975,7 +975,7 @@ discard block |
||
975 | 975 | // Prevent any PHP warnings, like undefined index |
976 | 976 | ob_start(); |
977 | 977 | |
978 | - if( $gv_field && is_callable( array( $gv_field, 'get_field_input' ) ) ) { |
|
978 | + if ( $gv_field && is_callable( array( $gv_field, 'get_field_input' ) ) ) { |
|
979 | 979 | /** @var GF_Field $gv_field */ |
980 | 980 | $return = $gv_field->get_field_input( $this->form, $field_value, $this->entry, $field ); |
981 | 981 | } else { |
@@ -986,7 +986,7 @@ discard block |
||
986 | 986 | // If there was output, it's an error |
987 | 987 | $warnings = ob_get_clean(); |
988 | 988 | |
989 | - if( !empty( $warnings ) ) { |
|
989 | + if ( ! empty( $warnings ) ) { |
|
990 | 990 | do_action( 'gravityview_log_error', __METHOD__ . $warnings, $field_value ); |
991 | 991 | } |
992 | 992 | |
@@ -995,7 +995,7 @@ discard block |
||
995 | 995 | * We need the fileupload html field to render with the proper id |
996 | 996 | * ( <li id="field_80_16" ... > ) |
997 | 997 | */ |
998 | - unset( $_GET['page'] ); |
|
998 | + unset( $_GET[ 'page' ] ); |
|
999 | 999 | |
1000 | 1000 | return $return; |
1001 | 1001 | } |
@@ -1018,7 +1018,7 @@ discard block |
||
1018 | 1018 | $override_saved_value = apply_filters( 'gravityview/edit_entry/pre_populate/override', false, $field ); |
1019 | 1019 | |
1020 | 1020 | // We're dealing with multiple inputs (e.g. checkbox) but not time or date (as it doesn't store data in input IDs) |
1021 | - if( isset( $field->inputs ) && is_array( $field->inputs ) && !in_array( $field->type, array( 'time', 'date' ) ) ) { |
|
1021 | + if ( isset( $field->inputs ) && is_array( $field->inputs ) && ! in_array( $field->type, array( 'time', 'date' ) ) ) { |
|
1022 | 1022 | |
1023 | 1023 | $field_value = array(); |
1024 | 1024 | |
@@ -1027,10 +1027,10 @@ discard block |
||
1027 | 1027 | |
1028 | 1028 | foreach ( (array)$field->inputs as $input ) { |
1029 | 1029 | |
1030 | - $input_id = strval( $input['id'] ); |
|
1030 | + $input_id = strval( $input[ 'id' ] ); |
|
1031 | 1031 | |
1032 | 1032 | if ( isset( $this->entry[ $input_id ] ) && ! gv_empty( $this->entry[ $input_id ], false, false ) ) { |
1033 | - $field_value[ $input_id ] = 'post_category' === $field->type ? GFCommon::format_post_category( $this->entry[ $input_id ], true ) : $this->entry[ $input_id ]; |
|
1033 | + $field_value[ $input_id ] = 'post_category' === $field->type ? GFCommon::format_post_category( $this->entry[ $input_id ], true ) : $this->entry[ $input_id ]; |
|
1034 | 1034 | $allow_pre_populated = false; |
1035 | 1035 | } |
1036 | 1036 | |
@@ -1038,7 +1038,7 @@ discard block |
||
1038 | 1038 | |
1039 | 1039 | $pre_value = $field->get_value_submission( array(), false ); |
1040 | 1040 | |
1041 | - $field_value = ! $allow_pre_populated && ! ( $override_saved_value && !gv_empty( $pre_value, false, false ) ) ? $field_value : $pre_value; |
|
1041 | + $field_value = ! $allow_pre_populated && ! ( $override_saved_value && ! gv_empty( $pre_value, false, false ) ) ? $field_value : $pre_value; |
|
1042 | 1042 | |
1043 | 1043 | } else { |
1044 | 1044 | |
@@ -1049,13 +1049,13 @@ discard block |
||
1049 | 1049 | |
1050 | 1050 | // saved field entry value (if empty, fallback to the pre-populated value, if exists) |
1051 | 1051 | // or pre-populated value if not empty and set to override saved value |
1052 | - $field_value = !gv_empty( $this->entry[ $id ], false, false ) && ! ( $override_saved_value && !gv_empty( $pre_value, false, false ) ) ? $this->entry[ $id ] : $pre_value; |
|
1052 | + $field_value = ! gv_empty( $this->entry[ $id ], false, false ) && ! ( $override_saved_value && ! gv_empty( $pre_value, false, false ) ) ? $this->entry[ $id ] : $pre_value; |
|
1053 | 1053 | |
1054 | 1054 | // in case field is post_category but inputType is select, multi-select or radio, convert value into array of category IDs. |
1055 | - if ( 'post_category' === $field->type && !gv_empty( $field_value, false, false ) ) { |
|
1055 | + if ( 'post_category' === $field->type && ! gv_empty( $field_value, false, false ) ) { |
|
1056 | 1056 | $categories = array(); |
1057 | 1057 | foreach ( explode( ',', $field_value ) as $cat_string ) { |
1058 | - $categories[] = GFCommon::format_post_category( $cat_string, true ); |
|
1058 | + $categories[ ] = GFCommon::format_post_category( $cat_string, true ); |
|
1059 | 1059 | } |
1060 | 1060 | $field_value = 'multiselect' === $field->get_input_type() ? $categories : implode( '', $categories ); |
1061 | 1061 | } |
@@ -1080,12 +1080,12 @@ discard block |
||
1080 | 1080 | */ |
1081 | 1081 | function gform_pre_validation( $form ) { |
1082 | 1082 | |
1083 | - if( ! $this->verify_nonce() ) { |
|
1083 | + if ( ! $this->verify_nonce() ) { |
|
1084 | 1084 | return $form; |
1085 | 1085 | } |
1086 | 1086 | |
1087 | 1087 | // Fix PHP warning regarding undefined index. |
1088 | - foreach ( $form['fields'] as &$field) { |
|
1088 | + foreach ( $form[ 'fields' ] as &$field ) { |
|
1089 | 1089 | |
1090 | 1090 | // This is because we're doing admin form pretending to be front-end, so Gravity Forms |
1091 | 1091 | // expects certain field array items to be set. |
@@ -1098,7 +1098,7 @@ discard block |
||
1098 | 1098 | $field['emailConfirmEnabled'] = ''; |
1099 | 1099 | }*/ |
1100 | 1100 | |
1101 | - switch( RGFormsModel::get_input_type( $field ) ) { |
|
1101 | + switch ( RGFormsModel::get_input_type( $field ) ) { |
|
1102 | 1102 | |
1103 | 1103 | /** |
1104 | 1104 | * 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. |
@@ -1112,37 +1112,37 @@ discard block |
||
1112 | 1112 | // Set the previous value |
1113 | 1113 | $entry = $this->get_entry(); |
1114 | 1114 | |
1115 | - $input_name = 'input_'.$field->id; |
|
1116 | - $form_id = $form['id']; |
|
1115 | + $input_name = 'input_' . $field->id; |
|
1116 | + $form_id = $form[ 'id' ]; |
|
1117 | 1117 | |
1118 | 1118 | $value = NULL; |
1119 | 1119 | |
1120 | 1120 | // Use the previous entry value as the default. |
1121 | - if( isset( $entry[ $field->id ] ) ) { |
|
1121 | + if ( isset( $entry[ $field->id ] ) ) { |
|
1122 | 1122 | $value = $entry[ $field->id ]; |
1123 | 1123 | } |
1124 | 1124 | |
1125 | 1125 | // If this is a single upload file |
1126 | - if( !empty( $_FILES[ $input_name ] ) && !empty( $_FILES[ $input_name ]['name'] ) ) { |
|
1127 | - $file_path = GFFormsModel::get_file_upload_path( $form['id'], $_FILES[ $input_name ]['name'] ); |
|
1128 | - $value = $file_path['url']; |
|
1126 | + if ( ! empty( $_FILES[ $input_name ] ) && ! empty( $_FILES[ $input_name ][ 'name' ] ) ) { |
|
1127 | + $file_path = GFFormsModel::get_file_upload_path( $form[ 'id' ], $_FILES[ $input_name ][ 'name' ] ); |
|
1128 | + $value = $file_path[ 'url' ]; |
|
1129 | 1129 | |
1130 | 1130 | } else { |
1131 | 1131 | |
1132 | 1132 | // Fix PHP warning on line 1498 of form_display.php for post_image fields |
1133 | 1133 | // Fix PHP Notice: Undefined index: size in form_display.php on line 1511 |
1134 | - $_FILES[ $input_name ] = array('name' => '', 'size' => '' ); |
|
1134 | + $_FILES[ $input_name ] = array( 'name' => '', 'size' => '' ); |
|
1135 | 1135 | |
1136 | 1136 | } |
1137 | 1137 | |
1138 | - if( rgar($field, "multipleFiles") ) { |
|
1138 | + if ( rgar( $field, "multipleFiles" ) ) { |
|
1139 | 1139 | |
1140 | 1140 | // If there are fresh uploads, process and merge them. |
1141 | 1141 | // Otherwise, use the passed values, which should be json-encoded array of URLs |
1142 | - if( isset( GFFormsModel::$uploaded_files[$form_id][$input_name] ) ) { |
|
1142 | + if ( isset( GFFormsModel::$uploaded_files[ $form_id ][ $input_name ] ) ) { |
|
1143 | 1143 | $value = empty( $value ) ? '[]' : $value; |
1144 | 1144 | $value = stripslashes_deep( $value ); |
1145 | - $value = GFFormsModel::prepare_value( $form, $field, $value, $input_name, $entry['id'], array()); |
|
1145 | + $value = GFFormsModel::prepare_value( $form, $field, $value, $input_name, $entry[ 'id' ], array() ); |
|
1146 | 1146 | } |
1147 | 1147 | |
1148 | 1148 | } else { |
@@ -1160,14 +1160,14 @@ discard block |
||
1160 | 1160 | |
1161 | 1161 | case 'number': |
1162 | 1162 | // Fix "undefined index" issue at line 1286 in form_display.php |
1163 | - if( !isset( $_POST['input_'.$field->id ] ) ) { |
|
1164 | - $_POST['input_'.$field->id ] = NULL; |
|
1163 | + if ( ! isset( $_POST[ 'input_' . $field->id ] ) ) { |
|
1164 | + $_POST[ 'input_' . $field->id ] = NULL; |
|
1165 | 1165 | } |
1166 | 1166 | break; |
1167 | 1167 | case 'captcha': |
1168 | 1168 | // Fix issue with recaptcha_check_answer() on line 1458 in form_display.php |
1169 | - $_POST['recaptcha_challenge_field'] = NULL; |
|
1170 | - $_POST['recaptcha_response_field'] = NULL; |
|
1169 | + $_POST[ 'recaptcha_challenge_field' ] = NULL; |
|
1170 | + $_POST[ 'recaptcha_response_field' ] = NULL; |
|
1171 | 1171 | break; |
1172 | 1172 | } |
1173 | 1173 | |
@@ -1203,7 +1203,7 @@ discard block |
||
1203 | 1203 | * You can enter whatever you want! |
1204 | 1204 | * We try validating, and customize the results using `self::custom_validation()` |
1205 | 1205 | */ |
1206 | - add_filter( 'gform_validation_'. $this->form_id, array( $this, 'custom_validation' ), 10, 4); |
|
1206 | + add_filter( 'gform_validation_' . $this->form_id, array( $this, 'custom_validation' ), 10, 4 ); |
|
1207 | 1207 | |
1208 | 1208 | // Needed by the validate funtion |
1209 | 1209 | $failed_validation_page = NULL; |
@@ -1211,14 +1211,14 @@ discard block |
||
1211 | 1211 | |
1212 | 1212 | // Prevent entry limit from running when editing an entry, also |
1213 | 1213 | // prevent form scheduling from preventing editing |
1214 | - unset( $this->form['limitEntries'], $this->form['scheduleForm'] ); |
|
1214 | + unset( $this->form[ 'limitEntries' ], $this->form[ 'scheduleForm' ] ); |
|
1215 | 1215 | |
1216 | 1216 | // Hide fields depending on Edit Entry settings |
1217 | - $this->form['fields'] = $this->get_configured_edit_fields( $this->form, $this->view_id ); |
|
1217 | + $this->form[ 'fields' ] = $this->get_configured_edit_fields( $this->form, $this->view_id ); |
|
1218 | 1218 | |
1219 | 1219 | $this->is_valid = GFFormDisplay::validate( $this->form, $field_values, 1, $failed_validation_page ); |
1220 | 1220 | |
1221 | - remove_filter( 'gform_validation_'. $this->form_id, array( $this, 'custom_validation' ), 10 ); |
|
1221 | + remove_filter( 'gform_validation_' . $this->form_id, array( $this, 'custom_validation' ), 10 ); |
|
1222 | 1222 | } |
1223 | 1223 | |
1224 | 1224 | |
@@ -1235,13 +1235,13 @@ discard block |
||
1235 | 1235 | */ |
1236 | 1236 | function custom_validation( $validation_results ) { |
1237 | 1237 | |
1238 | - do_action('gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Validation results: ', $validation_results ); |
|
1238 | + do_action( 'gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Validation results: ', $validation_results ); |
|
1239 | 1239 | |
1240 | - do_action('gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] $_POSTed data (sanitized): ', esc_html( print_r( $_POST, true ) ) ); |
|
1240 | + do_action( 'gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] $_POSTed data (sanitized): ', esc_html( print_r( $_POST, true ) ) ); |
|
1241 | 1241 | |
1242 | 1242 | $gv_valid = true; |
1243 | 1243 | |
1244 | - foreach ( $validation_results['form']['fields'] as $key => &$field ) { |
|
1244 | + foreach ( $validation_results[ 'form' ][ 'fields' ] as $key => &$field ) { |
|
1245 | 1245 | |
1246 | 1246 | $value = RGFormsModel::get_field_value( $field ); |
1247 | 1247 | $field_type = RGFormsModel::get_input_type( $field ); |
@@ -1254,35 +1254,35 @@ discard block |
||
1254 | 1254 | case 'post_image': |
1255 | 1255 | |
1256 | 1256 | // in case nothing is uploaded but there are already files saved |
1257 | - if( !empty( $field->failed_validation ) && !empty( $field->isRequired ) && !empty( $value ) ) { |
|
1257 | + if ( ! empty( $field->failed_validation ) && ! empty( $field->isRequired ) && ! empty( $value ) ) { |
|
1258 | 1258 | $field->failed_validation = false; |
1259 | 1259 | unset( $field->validation_message ); |
1260 | 1260 | } |
1261 | 1261 | |
1262 | 1262 | // validate if multi file upload reached max number of files [maxFiles] => 2 |
1263 | - if( rgobj( $field, 'maxFiles') && rgobj( $field, 'multipleFiles') ) { |
|
1263 | + if ( rgobj( $field, 'maxFiles' ) && rgobj( $field, 'multipleFiles' ) ) { |
|
1264 | 1264 | |
1265 | 1265 | $input_name = 'input_' . $field->id; |
1266 | 1266 | //uploaded |
1267 | - $file_names = isset( GFFormsModel::$uploaded_files[ $validation_results['form']['id'] ][ $input_name ] ) ? GFFormsModel::$uploaded_files[ $validation_results['form']['id'] ][ $input_name ] : array(); |
|
1267 | + $file_names = isset( GFFormsModel::$uploaded_files[ $validation_results[ 'form' ][ 'id' ] ][ $input_name ] ) ? GFFormsModel::$uploaded_files[ $validation_results[ 'form' ][ 'id' ] ][ $input_name ] : array(); |
|
1268 | 1268 | |
1269 | 1269 | //existent |
1270 | 1270 | $entry = $this->get_entry(); |
1271 | 1271 | $value = NULL; |
1272 | - if( isset( $entry[ $field->id ] ) ) { |
|
1272 | + if ( isset( $entry[ $field->id ] ) ) { |
|
1273 | 1273 | $value = json_decode( $entry[ $field->id ], true ); |
1274 | 1274 | } |
1275 | 1275 | |
1276 | 1276 | // count uploaded files and existent entry files |
1277 | 1277 | $count_files = count( $file_names ) + count( $value ); |
1278 | 1278 | |
1279 | - if( $count_files > $field->maxFiles ) { |
|
1279 | + if ( $count_files > $field->maxFiles ) { |
|
1280 | 1280 | $field->validation_message = __( 'Maximum number of files reached', 'gravityview' ); |
1281 | 1281 | $field->failed_validation = 1; |
1282 | 1282 | $gv_valid = false; |
1283 | 1283 | |
1284 | 1284 | // in case of error make sure the newest upload files are removed from the upload input |
1285 | - GFFormsModel::$uploaded_files[ $validation_results['form']['id'] ] = null; |
|
1285 | + GFFormsModel::$uploaded_files[ $validation_results[ 'form' ][ 'id' ] ] = null; |
|
1286 | 1286 | } |
1287 | 1287 | |
1288 | 1288 | } |
@@ -1293,7 +1293,7 @@ discard block |
||
1293 | 1293 | } |
1294 | 1294 | |
1295 | 1295 | // This field has failed validation. |
1296 | - if( !empty( $field->failed_validation ) ) { |
|
1296 | + if ( ! empty( $field->failed_validation ) ) { |
|
1297 | 1297 | |
1298 | 1298 | do_action( 'gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Field is invalid.', array( 'field' => $field, 'value' => $value ) ); |
1299 | 1299 | |
@@ -1311,32 +1311,32 @@ discard block |
||
1311 | 1311 | } |
1312 | 1312 | |
1313 | 1313 | // You can't continue inside a switch, so we do it after. |
1314 | - if( empty( $field->failed_validation ) ) { |
|
1314 | + if ( empty( $field->failed_validation ) ) { |
|
1315 | 1315 | continue; |
1316 | 1316 | } |
1317 | 1317 | |
1318 | 1318 | // checks if the No Duplicates option is not validating entry against itself, since |
1319 | 1319 | // we're editing a stored entry, it would also assume it's a duplicate. |
1320 | - if( !empty( $field->noDuplicates ) ) { |
|
1320 | + if ( ! empty( $field->noDuplicates ) ) { |
|
1321 | 1321 | |
1322 | 1322 | $entry = $this->get_entry(); |
1323 | 1323 | |
1324 | 1324 | // If the value of the entry is the same as the stored value |
1325 | 1325 | // Then we can assume it's not a duplicate, it's the same. |
1326 | - if( !empty( $entry ) && $value == $entry[ $field->id ] ) { |
|
1326 | + if ( ! empty( $entry ) && $value == $entry[ $field->id ] ) { |
|
1327 | 1327 | //if value submitted was not changed, then don't validate |
1328 | 1328 | $field->failed_validation = false; |
1329 | 1329 | |
1330 | 1330 | unset( $field->validation_message ); |
1331 | 1331 | |
1332 | - do_action('gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Field not a duplicate; it is the same entry.', $entry ); |
|
1332 | + do_action( 'gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Field not a duplicate; it is the same entry.', $entry ); |
|
1333 | 1333 | |
1334 | 1334 | continue; |
1335 | 1335 | } |
1336 | 1336 | } |
1337 | 1337 | |
1338 | 1338 | // if here then probably we are facing the validation 'At least one field must be filled out' |
1339 | - if( GFFormDisplay::is_empty( $field, $this->form_id ) && empty( $field->isRequired ) ) { |
|
1339 | + if ( GFFormDisplay::is_empty( $field, $this->form_id ) && empty( $field->isRequired ) ) { |
|
1340 | 1340 | unset( $field->validation_message ); |
1341 | 1341 | $field->validation_message = false; |
1342 | 1342 | continue; |
@@ -1348,12 +1348,12 @@ discard block |
||
1348 | 1348 | |
1349 | 1349 | } |
1350 | 1350 | |
1351 | - $validation_results['is_valid'] = $gv_valid; |
|
1351 | + $validation_results[ 'is_valid' ] = $gv_valid; |
|
1352 | 1352 | |
1353 | - do_action('gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Validation results.', $validation_results ); |
|
1353 | + do_action( 'gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Validation results.', $validation_results ); |
|
1354 | 1354 | |
1355 | 1355 | // We'll need this result when rendering the form ( on GFFormDisplay::get_form ) |
1356 | - $this->form_after_validation = $validation_results['form']; |
|
1356 | + $this->form_after_validation = $validation_results[ 'form' ]; |
|
1357 | 1357 | |
1358 | 1358 | return $validation_results; |
1359 | 1359 | } |
@@ -1366,7 +1366,7 @@ discard block |
||
1366 | 1366 | */ |
1367 | 1367 | private function get_entry() { |
1368 | 1368 | |
1369 | - if( empty( $this->entry ) ) { |
|
1369 | + if ( empty( $this->entry ) ) { |
|
1370 | 1370 | // Get the database value of the entry that's being edited |
1371 | 1371 | $this->entry = gravityview_get_entry( GravityView_frontend::is_single_entry() ); |
1372 | 1372 | } |
@@ -1393,7 +1393,7 @@ discard block |
||
1393 | 1393 | $properties = GravityView_View_Data::getInstance()->get_fields( $view_id ); |
1394 | 1394 | |
1395 | 1395 | // If edit tab not yet configured, show all fields |
1396 | - $edit_fields = !empty( $properties['edit_edit-fields'] ) ? $properties['edit_edit-fields'] : NULL; |
|
1396 | + $edit_fields = ! empty( $properties[ 'edit_edit-fields' ] ) ? $properties[ 'edit_edit-fields' ] : NULL; |
|
1397 | 1397 | |
1398 | 1398 | // Show hidden fields as text fields |
1399 | 1399 | $form = $this->fix_hidden_fields( $form ); |
@@ -1402,7 +1402,7 @@ discard block |
||
1402 | 1402 | $form = $this->fix_survey_fields( $form ); |
1403 | 1403 | |
1404 | 1404 | // Hide fields depending on admin settings |
1405 | - $fields = $this->filter_fields( $form['fields'], $edit_fields ); |
|
1405 | + $fields = $this->filter_fields( $form[ 'fields' ], $edit_fields ); |
|
1406 | 1406 | |
1407 | 1407 | // If Edit Entry fields are configured, remove adminOnly field settings. Otherwise, don't. |
1408 | 1408 | $fields = $this->filter_admin_only_fields( $fields, $edit_fields, $form, $view_id ); |
@@ -1422,7 +1422,7 @@ discard block |
||
1422 | 1422 | private function fix_survey_fields( $form ) { |
1423 | 1423 | |
1424 | 1424 | /** @var GF_Field $field */ |
1425 | - foreach( $form['fields'] as &$field ) { |
|
1425 | + foreach ( $form[ 'fields' ] as &$field ) { |
|
1426 | 1426 | $field->allowsPrepopulate = true; |
1427 | 1427 | } |
1428 | 1428 | |
@@ -1439,11 +1439,11 @@ discard block |
||
1439 | 1439 | private function fix_hidden_fields( $form ) { |
1440 | 1440 | |
1441 | 1441 | /** @var GF_Field $field */ |
1442 | - foreach( $form['fields'] as $key => $field ) { |
|
1443 | - if( 'hidden' === $field->type ) { |
|
1442 | + foreach ( $form[ 'fields' ] as $key => $field ) { |
|
1443 | + if ( 'hidden' === $field->type ) { |
|
1444 | 1444 | $text_field = new GF_Field_Text( $field ); |
1445 | 1445 | $text_field->type = 'text'; |
1446 | - $form['fields'][ $key ] = $text_field; |
|
1446 | + $form[ 'fields' ][ $key ] = $text_field; |
|
1447 | 1447 | } |
1448 | 1448 | } |
1449 | 1449 | |
@@ -1464,7 +1464,7 @@ discard block |
||
1464 | 1464 | */ |
1465 | 1465 | private function filter_fields( $fields, $configured_fields ) { |
1466 | 1466 | |
1467 | - if( empty( $fields ) || !is_array( $fields ) ) { |
|
1467 | + if ( empty( $fields ) || ! is_array( $fields ) ) { |
|
1468 | 1468 | return $fields; |
1469 | 1469 | } |
1470 | 1470 | |
@@ -1481,13 +1481,13 @@ discard block |
||
1481 | 1481 | */ |
1482 | 1482 | $hide_product_fields = apply_filters( 'gravityview/edit_entry/hide-product-fields', empty( self::$supports_product_fields ) ); |
1483 | 1483 | |
1484 | - if( $hide_product_fields ) { |
|
1485 | - $field_type_blacklist[] = 'option'; |
|
1486 | - $field_type_blacklist[] = 'quantity'; |
|
1487 | - $field_type_blacklist[] = 'product'; |
|
1488 | - $field_type_blacklist[] = 'total'; |
|
1489 | - $field_type_blacklist[] = 'shipping'; |
|
1490 | - $field_type_blacklist[] = 'calculation'; |
|
1484 | + if ( $hide_product_fields ) { |
|
1485 | + $field_type_blacklist[ ] = 'option'; |
|
1486 | + $field_type_blacklist[ ] = 'quantity'; |
|
1487 | + $field_type_blacklist[ ] = 'product'; |
|
1488 | + $field_type_blacklist[ ] = 'total'; |
|
1489 | + $field_type_blacklist[ ] = 'shipping'; |
|
1490 | + $field_type_blacklist[ ] = 'calculation'; |
|
1491 | 1491 | } |
1492 | 1492 | |
1493 | 1493 | // First, remove blacklist or calculation fields |
@@ -1495,24 +1495,24 @@ discard block |
||
1495 | 1495 | |
1496 | 1496 | // Remove the fields that have calculation properties and keep them to be used later |
1497 | 1497 | // @since 1.16.2 |
1498 | - if( $field->has_calculation() ) { |
|
1499 | - $this->fields_with_calculation[] = $field; |
|
1498 | + if ( $field->has_calculation() ) { |
|
1499 | + $this->fields_with_calculation[ ] = $field; |
|
1500 | 1500 | // don't remove the calculation fields on form render. |
1501 | 1501 | } |
1502 | 1502 | |
1503 | 1503 | // process total field after all fields have been saved |
1504 | 1504 | if ( $field->type == 'total' ) { |
1505 | - $this->total_fields[] = $field; |
|
1505 | + $this->total_fields[ ] = $field; |
|
1506 | 1506 | unset( $fields[ $key ] ); |
1507 | 1507 | } |
1508 | 1508 | |
1509 | - if( in_array( $field->type, $field_type_blacklist ) ) { |
|
1509 | + if ( in_array( $field->type, $field_type_blacklist ) ) { |
|
1510 | 1510 | unset( $fields[ $key ] ); |
1511 | 1511 | } |
1512 | 1512 | } |
1513 | 1513 | |
1514 | 1514 | // The Edit tab has not been configured, so we return all fields by default. |
1515 | - if( empty( $configured_fields ) ) { |
|
1515 | + if ( empty( $configured_fields ) ) { |
|
1516 | 1516 | return $fields; |
1517 | 1517 | } |
1518 | 1518 | |
@@ -1522,8 +1522,8 @@ discard block |
||
1522 | 1522 | /** @var GF_Field $field */ |
1523 | 1523 | foreach ( $fields as $field ) { |
1524 | 1524 | |
1525 | - if( intval( $configured_field['id'] ) === intval( $field->id ) && $this->user_can_edit_field( $configured_field, false ) ) { |
|
1526 | - $edit_fields[] = $this->merge_field_properties( $field, $configured_field ); |
|
1525 | + if ( intval( $configured_field[ 'id' ] ) === intval( $field->id ) && $this->user_can_edit_field( $configured_field, false ) ) { |
|
1526 | + $edit_fields[ ] = $this->merge_field_properties( $field, $configured_field ); |
|
1527 | 1527 | break; |
1528 | 1528 | } |
1529 | 1529 | |
@@ -1546,14 +1546,14 @@ discard block |
||
1546 | 1546 | |
1547 | 1547 | $return_field = $field; |
1548 | 1548 | |
1549 | - if( empty( $field_setting['show_label'] ) ) { |
|
1549 | + if ( empty( $field_setting[ 'show_label' ] ) ) { |
|
1550 | 1550 | $return_field->label = ''; |
1551 | - } elseif ( !empty( $field_setting['custom_label'] ) ) { |
|
1552 | - $return_field->label = $field_setting['custom_label']; |
|
1551 | + } elseif ( ! empty( $field_setting[ 'custom_label' ] ) ) { |
|
1552 | + $return_field->label = $field_setting[ 'custom_label' ]; |
|
1553 | 1553 | } |
1554 | 1554 | |
1555 | - if( !empty( $field_setting['custom_class'] ) ) { |
|
1556 | - $return_field->cssClass .= ' '. gravityview_sanitize_html_class( $field_setting['custom_class'] ); |
|
1555 | + if ( ! empty( $field_setting[ 'custom_class' ] ) ) { |
|
1556 | + $return_field->cssClass .= ' ' . gravityview_sanitize_html_class( $field_setting[ 'custom_class' ] ); |
|
1557 | 1557 | } |
1558 | 1558 | |
1559 | 1559 | /** |
@@ -1591,16 +1591,16 @@ discard block |
||
1591 | 1591 | */ |
1592 | 1592 | $use_gf_adminonly_setting = apply_filters( 'gravityview/edit_entry/use_gf_admin_only_setting', empty( $edit_fields ), $form, $view_id ); |
1593 | 1593 | |
1594 | - if( $use_gf_adminonly_setting && false === GVCommon::has_cap( 'gravityforms_edit_entries', $this->entry['id'] ) ) { |
|
1595 | - foreach( $fields as $k => $field ) { |
|
1596 | - if( $field->adminOnly ) { |
|
1594 | + if ( $use_gf_adminonly_setting && false === GVCommon::has_cap( 'gravityforms_edit_entries', $this->entry[ 'id' ] ) ) { |
|
1595 | + foreach ( $fields as $k => $field ) { |
|
1596 | + if ( $field->adminOnly ) { |
|
1597 | 1597 | unset( $fields[ $k ] ); |
1598 | 1598 | } |
1599 | 1599 | } |
1600 | 1600 | return $fields; |
1601 | 1601 | } |
1602 | 1602 | |
1603 | - foreach( $fields as &$field ) { |
|
1603 | + foreach ( $fields as &$field ) { |
|
1604 | 1604 | $field->adminOnly = false; |
1605 | 1605 | } |
1606 | 1606 | |
@@ -1627,16 +1627,16 @@ discard block |
||
1627 | 1627 | */ |
1628 | 1628 | $use_conditional_logic = apply_filters( 'gravityview/edit_entry/conditional_logic', true, $form ); |
1629 | 1629 | |
1630 | - if( $use_conditional_logic ) { |
|
1630 | + if ( $use_conditional_logic ) { |
|
1631 | 1631 | return $form; |
1632 | 1632 | } |
1633 | 1633 | |
1634 | - foreach( $form['fields'] as &$field ) { |
|
1634 | + foreach ( $form[ 'fields' ] as &$field ) { |
|
1635 | 1635 | /* @var GF_Field $field */ |
1636 | 1636 | $field->conditionalLogic = null; |
1637 | 1637 | } |
1638 | 1638 | |
1639 | - unset( $form['button']['conditionalLogic'] ); |
|
1639 | + unset( $form[ 'button' ][ 'conditionalLogic' ] ); |
|
1640 | 1640 | |
1641 | 1641 | return $form; |
1642 | 1642 | |
@@ -1653,7 +1653,7 @@ discard block |
||
1653 | 1653 | */ |
1654 | 1654 | function manage_conditional_logic( $has_conditional_logic, $form ) { |
1655 | 1655 | |
1656 | - if( ! $this->is_edit_entry() ) { |
|
1656 | + if ( ! $this->is_edit_entry() ) { |
|
1657 | 1657 | return $has_conditional_logic; |
1658 | 1658 | } |
1659 | 1659 | |
@@ -1684,44 +1684,44 @@ discard block |
||
1684 | 1684 | * 2. There are two entries embedded using oEmbed |
1685 | 1685 | * 3. One of the entries has just been saved |
1686 | 1686 | */ |
1687 | - if( !empty( $_POST['lid'] ) && !empty( $_GET['entry'] ) && ( $_POST['lid'] !== $_GET['entry'] ) ) { |
|
1687 | + if ( ! empty( $_POST[ 'lid' ] ) && ! empty( $_GET[ 'entry' ] ) && ( $_POST[ 'lid' ] !== $_GET[ 'entry' ] ) ) { |
|
1688 | 1688 | |
1689 | 1689 | $error = true; |
1690 | 1690 | |
1691 | 1691 | } |
1692 | 1692 | |
1693 | - if( !empty( $_GET['entry'] ) && (string)$this->entry['id'] !== $_GET['entry'] ) { |
|
1693 | + if ( ! empty( $_GET[ 'entry' ] ) && (string)$this->entry[ 'id' ] !== $_GET[ 'entry' ] ) { |
|
1694 | 1694 | |
1695 | 1695 | $error = true; |
1696 | 1696 | |
1697 | - } elseif( ! $this->verify_nonce() ) { |
|
1697 | + } elseif ( ! $this->verify_nonce() ) { |
|
1698 | 1698 | |
1699 | 1699 | /** |
1700 | 1700 | * If the Entry is embedded, there may be two entries on the same page. |
1701 | 1701 | * If that's the case, and one is being edited, the other should fail gracefully and not display an error. |
1702 | 1702 | */ |
1703 | - if( GravityView_oEmbed::getInstance()->get_entry_id() ) { |
|
1703 | + if ( GravityView_oEmbed::getInstance()->get_entry_id() ) { |
|
1704 | 1704 | $error = true; |
1705 | 1705 | } else { |
1706 | - $error = __( 'The link to edit this entry is not valid; it may have expired.', 'gravityview'); |
|
1706 | + $error = __( 'The link to edit this entry is not valid; it may have expired.', 'gravityview' ); |
|
1707 | 1707 | } |
1708 | 1708 | |
1709 | 1709 | } |
1710 | 1710 | |
1711 | - if( ! GravityView_Edit_Entry::check_user_cap_edit_entry( $this->entry ) ) { |
|
1712 | - $error = __( 'You do not have permission to edit this entry.', 'gravityview'); |
|
1711 | + if ( ! GravityView_Edit_Entry::check_user_cap_edit_entry( $this->entry ) ) { |
|
1712 | + $error = __( 'You do not have permission to edit this entry.', 'gravityview' ); |
|
1713 | 1713 | } |
1714 | 1714 | |
1715 | - if( $this->entry['status'] === 'trash' ) { |
|
1716 | - $error = __('You cannot edit the entry; it is in the trash.', 'gravityview' ); |
|
1715 | + if ( $this->entry[ 'status' ] === 'trash' ) { |
|
1716 | + $error = __( 'You cannot edit the entry; it is in the trash.', 'gravityview' ); |
|
1717 | 1717 | } |
1718 | 1718 | |
1719 | 1719 | // No errors; everything's fine here! |
1720 | - if( empty( $error ) ) { |
|
1720 | + if ( empty( $error ) ) { |
|
1721 | 1721 | return true; |
1722 | 1722 | } |
1723 | 1723 | |
1724 | - if( $echo && $error !== true ) { |
|
1724 | + if ( $echo && $error !== true ) { |
|
1725 | 1725 | |
1726 | 1726 | $error = esc_html( $error ); |
1727 | 1727 | |
@@ -1729,13 +1729,13 @@ discard block |
||
1729 | 1729 | * @since 1.9 |
1730 | 1730 | */ |
1731 | 1731 | if ( ! empty( $this->entry ) ) { |
1732 | - $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;" ) ); |
|
1732 | + $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;" ) ); |
|
1733 | 1733 | } |
1734 | 1734 | |
1735 | - echo GVCommon::generate_notice( wpautop( $error ), 'gv-error error'); |
|
1735 | + echo GVCommon::generate_notice( wpautop( $error ), 'gv-error error' ); |
|
1736 | 1736 | } |
1737 | 1737 | |
1738 | - do_action('gravityview_log_error', 'GravityView_Edit_Entry[user_can_edit_entry]' . $error ); |
|
1738 | + do_action( 'gravityview_log_error', 'GravityView_Edit_Entry[user_can_edit_entry]' . $error ); |
|
1739 | 1739 | |
1740 | 1740 | return false; |
1741 | 1741 | } |
@@ -1752,20 +1752,20 @@ discard block |
||
1752 | 1752 | |
1753 | 1753 | $error = NULL; |
1754 | 1754 | |
1755 | - if( ! $this->check_user_cap_edit_field( $field ) ) { |
|
1756 | - $error = __( 'You do not have permission to edit this field.', 'gravityview'); |
|
1755 | + if ( ! $this->check_user_cap_edit_field( $field ) ) { |
|
1756 | + $error = __( 'You do not have permission to edit this field.', 'gravityview' ); |
|
1757 | 1757 | } |
1758 | 1758 | |
1759 | 1759 | // No errors; everything's fine here! |
1760 | - if( empty( $error ) ) { |
|
1760 | + if ( empty( $error ) ) { |
|
1761 | 1761 | return true; |
1762 | 1762 | } |
1763 | 1763 | |
1764 | - if( $echo ) { |
|
1765 | - echo GVCommon::generate_notice( wpautop( esc_html( $error ) ), 'gv-error error'); |
|
1764 | + if ( $echo ) { |
|
1765 | + echo GVCommon::generate_notice( wpautop( esc_html( $error ) ), 'gv-error error' ); |
|
1766 | 1766 | } |
1767 | 1767 | |
1768 | - do_action('gravityview_log_error', 'GravityView_Edit_Entry[user_can_edit_field]' . $error ); |
|
1768 | + do_action( 'gravityview_log_error', 'GravityView_Edit_Entry[user_can_edit_field]' . $error ); |
|
1769 | 1769 | |
1770 | 1770 | return false; |
1771 | 1771 | |
@@ -1783,15 +1783,15 @@ discard block |
||
1783 | 1783 | private function check_user_cap_edit_field( $field ) { |
1784 | 1784 | |
1785 | 1785 | // If they can edit any entries (as defined in Gravity Forms), we're good. |
1786 | - if( GVCommon::has_cap( array( 'gravityforms_edit_entries', 'gravityview_edit_others_entries' ) ) ) { |
|
1786 | + if ( GVCommon::has_cap( array( 'gravityforms_edit_entries', 'gravityview_edit_others_entries' ) ) ) { |
|
1787 | 1787 | return true; |
1788 | 1788 | } |
1789 | 1789 | |
1790 | - $field_cap = isset( $field['allow_edit_cap'] ) ? $field['allow_edit_cap'] : false; |
|
1790 | + $field_cap = isset( $field[ 'allow_edit_cap' ] ) ? $field[ 'allow_edit_cap' ] : false; |
|
1791 | 1791 | |
1792 | 1792 | // If the field has custom editing capaibilities set, check those |
1793 | - if( $field_cap ) { |
|
1794 | - return GVCommon::has_cap( $field['allow_edit_cap'] ); |
|
1793 | + if ( $field_cap ) { |
|
1794 | + return GVCommon::has_cap( $field[ 'allow_edit_cap' ] ); |
|
1795 | 1795 | } |
1796 | 1796 | |
1797 | 1797 | return false; |
@@ -1805,17 +1805,17 @@ discard block |
||
1805 | 1805 | public function verify_nonce() { |
1806 | 1806 | |
1807 | 1807 | // Verify form submitted for editing single |
1808 | - if( $this->is_edit_entry_submission() ) { |
|
1808 | + if ( $this->is_edit_entry_submission() ) { |
|
1809 | 1809 | $valid = wp_verify_nonce( $_POST[ self::$nonce_field ], self::$nonce_field ); |
1810 | 1810 | } |
1811 | 1811 | |
1812 | 1812 | // Verify |
1813 | - else if( ! $this->is_edit_entry() ) { |
|
1813 | + else if ( ! $this->is_edit_entry() ) { |
|
1814 | 1814 | $valid = false; |
1815 | 1815 | } |
1816 | 1816 | |
1817 | 1817 | else { |
1818 | - $valid = wp_verify_nonce( $_GET['edit'], self::$nonce_key ); |
|
1818 | + $valid = wp_verify_nonce( $_GET[ 'edit' ], self::$nonce_key ); |
|
1819 | 1819 | } |
1820 | 1820 | |
1821 | 1821 | /** |