Completed
Pull Request — master (#836)
by Zack
18:58
created
includes/plugin-and-theme-hooks/class-gravityview-theme-hooks-wpml.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
 
63 63
 		add_filter( 'icl_ls_languages', array( $this, 'wpml_ls_filter' ) );
64 64
 
65
-		add_filter( 'gravityview_directory_link', array( $this, 'filter_gravityview_back_link') );
65
+		add_filter( 'gravityview_directory_link', array( $this, 'filter_gravityview_back_link' ) );
66 66
 	}
67 67
 
68 68
 	/**
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
 	function filter_gravityview_back_link( $link ) {
83 83
 		global $wpml_url_filters;
84 84
 
85
-		if( $wpml_url_filters ) {
85
+		if ( $wpml_url_filters ) {
86 86
 			$link = $wpml_url_filters->permalink_filter( $link, GravityView_frontend::getInstance()->getPostId() );
87 87
 		}
88 88
 
@@ -99,7 +99,7 @@  discard block
 block discarded – undo
99 99
 	private function remove_url_hooks() {
100 100
 		global $wpml_url_filters;
101 101
 
102
-		if( ! $wpml_url_filters ) {
102
+		if ( ! $wpml_url_filters ) {
103 103
 			return;
104 104
 		}
105 105
 
@@ -122,7 +122,7 @@  discard block
 block discarded – undo
122 122
 	private function add_url_hooks() {
123 123
 		global $wpml_url_filters;
124 124
 
125
-		if( ! $wpml_url_filters ) {
125
+		if ( ! $wpml_url_filters ) {
126 126
 			return;
127 127
 		}
128 128
 
@@ -161,7 +161,7 @@  discard block
 block discarded – undo
161 161
 
162 162
 			$this->remove_url_hooks();
163 163
 
164
-			if( $translations ) {
164
+			if ( $translations ) {
165 165
 				foreach ( $languages as $lang_code => $language ) {
166 166
 
167 167
 					$lang_post_id = $translations[ $lang_code ]->element_id;
@@ -194,7 +194,7 @@  discard block
 block discarded – undo
194 194
 							break;
195 195
 					}
196 196
 
197
-					$languages[ $lang_code ]['url'] = $entry_link;
197
+					$languages[ $lang_code ][ 'url' ] = $entry_link;
198 198
 				}
199 199
 			}
200 200
 
Please login to merge, or discard this patch.
includes/fields/class-gravityview-field-total.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -50,13 +50,13 @@  discard block
 block discarded – undo
50 50
 	 *
51 51
 	 * @return array Blacklist, with "total" added. If not edit context, original field blacklist. Otherwise, blacklist including total.
52 52
 	 */
53
-	public function add_to_blacklist( $blacklist = array(), $context = NULL  ){
53
+	public function add_to_blacklist( $blacklist = array(), $context = NULL ) {
54 54
 
55
-		if( empty( $context ) || $context !== 'edit' ) {
55
+		if ( empty( $context ) || $context !== 'edit' ) {
56 56
 			return $fields;
57 57
 		}
58 58
 
59
-		$blacklist[] = 'total';
59
+		$blacklist[ ] = 'total';
60 60
 
61 61
 		return $blacklist;
62 62
 	}
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
 	 */
75 75
 	function edit_entry_recalculate_totals( $form = array(), $entry_id = 0, $Edit_Entry_Render = null ) {
76 76
 
77
-		$original_form = GFAPI::get_form( $form['id'] );
77
+		$original_form = GFAPI::get_form( $form[ 'id' ] );
78 78
 
79 79
 		$total_fields = GFCommon::get_fields_by_type( $original_form, 'total' );
80 80
 
@@ -85,12 +85,12 @@  discard block
 block discarded – undo
85 85
 
86 86
 			/** @var GF_Field_Total $total_field */
87 87
 			foreach ( $total_fields as $total_field ) {
88
-				$entry["{$total_field->id}"] = GFCommon::get_order_total( $original_form, $entry );
88
+				$entry[ "{$total_field->id}" ] = GFCommon::get_order_total( $original_form, $entry );
89 89
 			}
90 90
 
91 91
 			$return_entry = GFAPI::update_entry( $entry );
92 92
 
93
-			if( is_wp_error( $return_entry ) ) {
93
+			if ( is_wp_error( $return_entry ) ) {
94 94
 				do_action( 'gravityview_log_error', __METHOD__ . ': Updating the entry total fields failed', $return_entry );
95 95
 			} else {
96 96
 				do_action( 'gravityview_log_debug', __METHOD__ . ': Updating the entry total fields succeeded' );
Please login to merge, or discard this patch.
includes/extensions/edit-entry/class-edit-entry-render.php 1 patch
Spacing   +226 added lines, -226 removed lines patch added patch discarded remove patch
@@ -106,16 +106,16 @@  discard block
 block discarded – undo
106 106
     function load() {
107 107
 
108 108
         /** @define "GRAVITYVIEW_DIR" "../../../" */
109
-        include_once( GRAVITYVIEW_DIR .'includes/class-admin-approve-entries.php' );
109
+        include_once( GRAVITYVIEW_DIR . 'includes/class-admin-approve-entries.php' );
110 110
 
111 111
         // Don't display an embedded form when editing an entry
112 112
         add_action( 'wp_head', array( $this, 'prevent_render_form' ) );
113 113
         add_action( 'wp_footer', array( $this, 'prevent_render_form' ) );
114 114
 
115 115
         // Stop Gravity Forms processing what is ours!
116
-        add_filter( 'wp', array( $this, 'prevent_maybe_process_form'), 8 );
116
+        add_filter( 'wp', array( $this, 'prevent_maybe_process_form' ), 8 );
117 117
 
118
-        add_filter( 'gravityview_is_edit_entry', array( $this, 'is_edit_entry') );
118
+        add_filter( 'gravityview_is_edit_entry', array( $this, 'is_edit_entry' ) );
119 119
 
120 120
         add_action( 'gravityview_edit_entry', array( $this, 'init' ) );
121 121
 
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
         add_filter( 'gform_plupload_settings', array( $this, 'modify_fileupload_settings' ), 10, 3 );
127 127
 
128 128
         // Add fields expected by GFFormDisplay::validate()
129
-        add_filter( 'gform_pre_validation', array( $this, 'gform_pre_validation') );
129
+        add_filter( 'gform_pre_validation', array( $this, 'gform_pre_validation' ) );
130 130
 
131 131
     }
132 132
 
@@ -141,8 +141,8 @@  discard block
 block discarded – undo
141 141
      * @return void
142 142
      */
143 143
     public function prevent_render_form() {
144
-        if( $this->is_edit_entry() ) {
145
-            if( 'wp_head' === current_filter() ) {
144
+        if ( $this->is_edit_entry() ) {
145
+            if ( 'wp_head' === current_filter() ) {
146 146
                 add_filter( 'gform_shortcode_form', '__return_empty_string' );
147 147
             } else {
148 148
                 remove_filter( 'gform_shortcode_form', '__return_empty_string' );
@@ -157,10 +157,10 @@  discard block
 block discarded – undo
157 157
      */
158 158
     public function prevent_maybe_process_form() {
159 159
 
160
-        do_action('gravityview_log_debug', 'GravityView_Edit_Entry[prevent_maybe_process_form] $_POSTed data (sanitized): ', esc_html( print_r( $_POST, true ) ) );
160
+        do_action( 'gravityview_log_debug', 'GravityView_Edit_Entry[prevent_maybe_process_form] $_POSTed data (sanitized): ', esc_html( print_r( $_POST, true ) ) );
161 161
 
162
-        if( $this->is_edit_entry_submission() ) {
163
-            remove_action( 'wp',  array( 'RGForms', 'maybe_process_form'), 9 );
162
+        if ( $this->is_edit_entry_submission() ) {
163
+            remove_action( 'wp', array( 'RGForms', 'maybe_process_form' ), 9 );
164 164
         }
165 165
     }
166 166
 
@@ -170,7 +170,7 @@  discard block
 block discarded – undo
170 170
      */
171 171
     public function is_edit_entry() {
172 172
 
173
-        $is_edit_entry = GravityView_frontend::is_single_entry() && ! empty( $_GET['edit'] );
173
+        $is_edit_entry = GravityView_frontend::is_single_entry() && ! empty( $_GET[ 'edit' ] );
174 174
 
175 175
         return ( $is_edit_entry || $this->is_edit_entry_submission() );
176 176
     }
@@ -181,7 +181,7 @@  discard block
 block discarded – undo
181 181
 	 * @return boolean
182 182
 	 */
183 183
 	public function is_edit_entry_submission() {
184
-		return !empty( $_POST[ self::$nonce_field ] );
184
+		return ! empty( $_POST[ self::$nonce_field ] );
185 185
 	}
186 186
 
187 187
     /**
@@ -192,15 +192,15 @@  discard block
 block discarded – undo
192 192
 
193 193
 
194 194
         $entries = $gravityview_view->getEntries();
195
-	    self::$original_entry = $entries[0];
196
-	    $this->entry = $entries[0];
195
+	    self::$original_entry = $entries[ 0 ];
196
+	    $this->entry = $entries[ 0 ];
197 197
 
198 198
         self::$original_form = $gravityview_view->getForm();
199 199
         $this->form = $gravityview_view->getForm();
200 200
         $this->form_id = $gravityview_view->getFormId();
201 201
         $this->view_id = $gravityview_view->getViewId();
202 202
 
203
-        self::$nonce_key = GravityView_Edit_Entry::get_nonce_key( $this->view_id, $this->form_id, $this->entry['id'] );
203
+        self::$nonce_key = GravityView_Edit_Entry::get_nonce_key( $this->view_id, $this->form_id, $this->entry[ 'id' ] );
204 204
     }
205 205
 
206 206
 
@@ -220,14 +220,14 @@  discard block
 block discarded – undo
220 220
         $this->setup_vars();
221 221
 
222 222
         // Multiple Views embedded, don't proceed if nonce fails
223
-        if( $gv_data->has_multiple_views() && ! wp_verify_nonce( $_GET['edit'], self::$nonce_key ) ) {
224
-            do_action('gravityview_log_error', __METHOD__ . ': Nonce validation failed for the Edit Entry request; returning' );
223
+        if ( $gv_data->has_multiple_views() && ! wp_verify_nonce( $_GET[ 'edit' ], self::$nonce_key ) ) {
224
+            do_action( 'gravityview_log_error', __METHOD__ . ': Nonce validation failed for the Edit Entry request; returning' );
225 225
             return;
226 226
         }
227 227
 
228 228
         // Sorry, you're not allowed here.
229
-        if( false === $this->user_can_edit_entry( true ) ) {
230
-            do_action('gravityview_log_error', __METHOD__ . ': User is not allowed to edit this entry; returning', $this->entry );
229
+        if ( false === $this->user_can_edit_entry( true ) ) {
230
+            do_action( 'gravityview_log_error', __METHOD__ . ': User is not allowed to edit this entry; returning', $this->entry );
231 231
             return;
232 232
         }
233 233
 
@@ -247,9 +247,9 @@  discard block
 block discarded – undo
247 247
     private function print_scripts() {
248 248
         $gravityview_view = GravityView_View::getInstance();
249 249
 
250
-        wp_register_script( 'gform_gravityforms', GFCommon::get_base_url().'/js/gravityforms.js', array( 'jquery', 'gform_json', 'gform_placeholder', 'sack', 'plupload-all', 'gravityview-fe-view' ) );
250
+        wp_register_script( 'gform_gravityforms', GFCommon::get_base_url() . '/js/gravityforms.js', array( 'jquery', 'gform_json', 'gform_placeholder', 'sack', 'plupload-all', 'gravityview-fe-view' ) );
251 251
 
252
-        GFFormDisplay::enqueue_form_scripts($gravityview_view->getForm(), false);
252
+        GFFormDisplay::enqueue_form_scripts( $gravityview_view->getForm(), false );
253 253
 
254 254
         // Sack is required for images
255 255
         wp_print_scripts( array( 'sack', 'gform_gravityforms' ) );
@@ -261,32 +261,32 @@  discard block
 block discarded – undo
261 261
      */
262 262
     private function process_save() {
263 263
 
264
-        if( empty( $_POST ) || ! isset( $_POST['lid'] ) ) {
264
+        if ( empty( $_POST ) || ! isset( $_POST[ 'lid' ] ) ) {
265 265
             return;
266 266
         }
267 267
 
268 268
         // Make sure the entry, view, and form IDs are all correct
269 269
         $valid = $this->verify_nonce();
270 270
 
271
-        if( !$valid ) {
272
-            do_action('gravityview_log_error', __METHOD__ . ' Nonce validation failed.' );
271
+        if ( ! $valid ) {
272
+            do_action( 'gravityview_log_error', __METHOD__ . ' Nonce validation failed.' );
273 273
             return;
274 274
         }
275 275
 
276
-        if( $this->entry['id'] !== $_POST['lid'] ) {
277
-            do_action('gravityview_log_error', __METHOD__ . ' Entry ID did not match posted entry ID.' );
276
+        if ( $this->entry[ 'id' ] !== $_POST[ 'lid' ] ) {
277
+            do_action( 'gravityview_log_error', __METHOD__ . ' Entry ID did not match posted entry ID.' );
278 278
             return;
279 279
         }
280 280
 
281
-        do_action('gravityview_log_debug', __METHOD__ . ': $_POSTed data (sanitized): ', esc_html( print_r( $_POST, true ) ) );
281
+        do_action( 'gravityview_log_debug', __METHOD__ . ': $_POSTed data (sanitized): ', esc_html( print_r( $_POST, true ) ) );
282 282
 
283 283
         $this->process_save_process_files( $this->form_id );
284 284
 
285 285
         $this->validate();
286 286
 
287
-        if( $this->is_valid ) {
287
+        if ( $this->is_valid ) {
288 288
 
289
-            do_action('gravityview_log_debug', __METHOD__ . ': Submission is valid.' );
289
+            do_action( 'gravityview_log_debug', __METHOD__ . ': Submission is valid.' );
290 290
 
291 291
             /**
292 292
              * @hack This step is needed to unset the adminOnly from form fields, to add the calculation fields
@@ -296,22 +296,22 @@  discard block
 block discarded – undo
296 296
             /**
297 297
              * @hack to avoid the capability validation of the method save_lead for GF 1.9+
298 298
              */
299
-            unset( $_GET['page'] );
299
+            unset( $_GET[ 'page' ] );
300 300
 
301
-            $date_created = $this->entry['date_created'];
301
+            $date_created = $this->entry[ 'date_created' ];
302 302
 
303 303
             /**
304 304
              * @hack to force Gravity Forms to use $read_value_from_post in GFFormsModel::save_lead()
305 305
              * @since 1.17.2
306 306
              */
307
-            unset( $this->entry['date_created'] );
307
+            unset( $this->entry[ 'date_created' ] );
308 308
 
309 309
             GFFormsModel::save_lead( $form, $this->entry );
310 310
 
311 311
 	        // Delete the values for hidden inputs
312 312
 	        $this->unset_hidden_field_values();
313 313
             
314
-            $this->entry['date_created'] = $date_created;
314
+            $this->entry[ 'date_created' ] = $date_created;
315 315
 
316 316
             // Process calculation fields
317 317
             $this->update_calculation_fields();
@@ -331,10 +331,10 @@  discard block
 block discarded – undo
331 331
              * @param string $entry_id Numeric ID of the entry that was updated
332 332
              * @param GravityView_Edit_Entry_Render $this This object
333 333
              */
334
-            do_action( 'gravityview/edit_entry/after_update', $this->form, $this->entry['id'], $this );
334
+            do_action( 'gravityview/edit_entry/after_update', $this->form, $this->entry[ 'id' ], $this );
335 335
 
336 336
         } else {
337
-            do_action('gravityview_log_error', __METHOD__ . ': Submission is NOT valid.', $this->entry );
337
+            do_action( 'gravityview_log_error', __METHOD__ . ': Submission is NOT valid.', $this->entry );
338 338
         }
339 339
 
340 340
     } // process_save
@@ -351,8 +351,8 @@  discard block
 block discarded – undo
351 351
     private function unset_hidden_field_values() {
352 352
 	    global $wpdb;
353 353
 
354
-	    $lead_detail_table      = GFFormsModel::get_lead_details_table_name();
355
-	    $current_fields   = $wpdb->get_results( $wpdb->prepare( "SELECT id, field_number FROM $lead_detail_table WHERE lead_id=%d", $this->entry['id'] ) );
354
+	    $lead_detail_table = GFFormsModel::get_lead_details_table_name();
355
+	    $current_fields = $wpdb->get_results( $wpdb->prepare( "SELECT id, field_number FROM $lead_detail_table WHERE lead_id=%d", $this->entry[ 'id' ] ) );
356 356
 
357 357
 	    foreach ( $this->entry as $input_id => $field_value ) {
358 358
 
@@ -405,11 +405,11 @@  discard block
 block discarded – undo
405 405
      * @return mixed
406 406
      */
407 407
     public function modify_fileupload_settings( $plupload_init, $form_id, $instance ) {
408
-        if( ! $this->is_edit_entry() ) {
408
+        if ( ! $this->is_edit_entry() ) {
409 409
             return $plupload_init;
410 410
         }
411 411
 
412
-        $plupload_init['gf_vars']['max_files'] = 0;
412
+        $plupload_init[ 'gf_vars' ][ 'max_files' ] = 0;
413 413
 
414 414
         return $plupload_init;
415 415
     }
@@ -424,22 +424,22 @@  discard block
 block discarded – undo
424 424
         $form = $this->form;
425 425
 
426 426
 	    /** @var GF_Field $field */
427
-        foreach( $form['fields'] as $k => &$field ) {
427
+        foreach ( $form[ 'fields' ] as $k => &$field ) {
428 428
 
429 429
             /**
430 430
              * Remove the fields with calculation formulas before save to avoid conflicts with GF logic
431 431
              * @since 1.16.3
432 432
              * @var GF_Field $field
433 433
              */
434
-            if( $field->has_calculation() ) {
435
-                unset( $form['fields'][ $k ] );
434
+            if ( $field->has_calculation() ) {
435
+                unset( $form[ 'fields' ][ $k ] );
436 436
             }
437 437
 
438 438
             $field->adminOnly = false;
439 439
 
440
-            if( isset( $field->inputs ) && is_array( $field->inputs ) ) {
441
-                foreach( $field->inputs as $key => $input ) {
442
-                    $field->inputs[ $key ][ 'id' ] = (string)$input['id'];
440
+            if ( isset( $field->inputs ) && is_array( $field->inputs ) ) {
441
+                foreach ( $field->inputs as $key => $input ) {
442
+                    $field->inputs[ $key ][ 'id' ] = (string)$input[ 'id' ];
443 443
                 }
444 444
             }
445 445
         }
@@ -453,30 +453,30 @@  discard block
 block discarded – undo
453 453
         $update = false;
454 454
 
455 455
         // get the most up to date entry values
456
-        $entry = GFAPI::get_entry( $this->entry['id'] );
456
+        $entry = GFAPI::get_entry( $this->entry[ 'id' ] );
457 457
 
458
-        if( !empty( $this->fields_with_calculation ) ) {
458
+        if ( ! empty( $this->fields_with_calculation ) ) {
459 459
             $update = true;
460 460
             foreach ( $this->fields_with_calculation as $calc_field ) {
461 461
                 $inputs = $calc_field->get_entry_inputs();
462 462
                 if ( is_array( $inputs ) ) {
463 463
                     foreach ( $inputs as $input ) {
464
-                        $input_name = 'input_' . str_replace( '.', '_', $input['id'] );
465
-                        $entry[ strval( $input['id'] ) ] = RGFormsModel::prepare_value( $form, $calc_field, '', $input_name, $entry['id'], $entry );
464
+                        $input_name = 'input_' . str_replace( '.', '_', $input[ 'id' ] );
465
+                        $entry[ strval( $input[ 'id' ] ) ] = RGFormsModel::prepare_value( $form, $calc_field, '', $input_name, $entry[ 'id' ], $entry );
466 466
                     }
467 467
                 } else {
468
-                    $input_name = 'input_' . str_replace( '.', '_', $calc_field->id);
469
-                    $entry[ strval( $calc_field->id ) ] = RGFormsModel::prepare_value( $form, $calc_field, '', $input_name, $entry['id'], $entry );
468
+                    $input_name = 'input_' . str_replace( '.', '_', $calc_field->id );
469
+                    $entry[ strval( $calc_field->id ) ] = RGFormsModel::prepare_value( $form, $calc_field, '', $input_name, $entry[ 'id' ], $entry );
470 470
                 }
471 471
             }
472 472
 
473 473
         }
474 474
 
475
-        if( $update ) {
475
+        if ( $update ) {
476 476
 
477 477
             $return_entry = GFAPI::update_entry( $entry );
478 478
 
479
-            if( is_wp_error( $return_entry ) ) {
479
+            if ( is_wp_error( $return_entry ) ) {
480 480
                 do_action( 'gravityview_log_error', 'Updating the entry calculation fields failed', $return_entry );
481 481
             } else {
482 482
                 do_action( 'gravityview_log_debug', 'Updating the entry calculation fields succeeded' );
@@ -507,18 +507,18 @@  discard block
 block discarded – undo
507 507
 
508 508
         $input_name = 'input_' . $field_id;
509 509
 
510
-        if ( !empty( $_FILES[ $input_name ]['name'] ) ) {
510
+        if ( ! empty( $_FILES[ $input_name ][ 'name' ] ) ) {
511 511
 
512 512
             // We have a new image
513 513
 
514
-            $value = RGFormsModel::prepare_value( $form, $field, $value, $input_name, $entry['id'] );
514
+            $value = RGFormsModel::prepare_value( $form, $field, $value, $input_name, $entry[ 'id' ] );
515 515
 
516 516
             $ary = ! empty( $value ) ? explode( '|:|', $value ) : array();
517 517
             $img_url = rgar( $ary, 0 );
518 518
 
519
-            $img_title       = count( $ary ) > 1 ? $ary[1] : '';
520
-            $img_caption     = count( $ary ) > 2 ? $ary[2] : '';
521
-            $img_description = count( $ary ) > 3 ? $ary[3] : '';
519
+            $img_title       = count( $ary ) > 1 ? $ary[ 1 ] : '';
520
+            $img_caption     = count( $ary ) > 2 ? $ary[ 2 ] : '';
521
+            $img_description = count( $ary ) > 3 ? $ary[ 3 ] : '';
522 522
 
523 523
             $image_meta = array(
524 524
                 'post_excerpt' => $img_caption,
@@ -527,7 +527,7 @@  discard block
 block discarded – undo
527 527
 
528 528
             //adding title only if it is not empty. It will default to the file name if it is not in the array
529 529
             if ( ! empty( $img_title ) ) {
530
-                $image_meta['post_title'] = $img_title;
530
+                $image_meta[ 'post_title' ] = $img_title;
531 531
             }
532 532
 
533 533
             /**
@@ -542,22 +542,22 @@  discard block
 block discarded – undo
542 542
                 set_post_thumbnail( $post_id, $media_id );
543 543
             }
544 544
 
545
-        } elseif ( !empty( $_POST[ $input_name ] ) && is_array( $value ) ) {
545
+        } elseif ( ! empty( $_POST[ $input_name ] ) && is_array( $value ) ) {
546 546
 
547 547
             // Same image although the image title, caption or description might have changed
548 548
 
549 549
             $ary = array();
550
-            if( ! empty( $entry[ $field_id ] ) ) {
550
+            if ( ! empty( $entry[ $field_id ] ) ) {
551 551
                 $ary = is_array( $entry[ $field_id ] ) ? $entry[ $field_id ] : explode( '|:|', $entry[ $field_id ] );
552 552
             }
553 553
             $img_url = rgar( $ary, 0 );
554 554
 
555 555
             // is this really the same image or something went wrong ?
556
-            if( $img_url === $_POST[ $input_name ] ) {
556
+            if ( $img_url === $_POST[ $input_name ] ) {
557 557
 
558
-                $img_title       = rgar( $value, $field_id .'.1' );
559
-                $img_caption     = rgar( $value, $field_id .'.4' );
560
-                $img_description = rgar( $value, $field_id .'.7' );
558
+                $img_title       = rgar( $value, $field_id . '.1' );
559
+                $img_caption     = rgar( $value, $field_id . '.4' );
560
+                $img_description = rgar( $value, $field_id . '.7' );
561 561
 
562 562
                 $value = ! empty( $img_url ) ? $img_url . "|:|" . $img_title . "|:|" . $img_caption . "|:|" . $img_description : '';
563 563
 
@@ -597,16 +597,16 @@  discard block
 block discarded – undo
597 597
      */
598 598
     private function maybe_update_post_fields( $form ) {
599 599
 
600
-        if( empty( $this->entry['post_id'] ) ) {
600
+        if ( empty( $this->entry[ 'post_id' ] ) ) {
601 601
 	        do_action( 'gravityview_log_debug', __METHOD__ . ': This entry has no post fields. Continuing...' );
602 602
             return;
603 603
         }
604 604
 
605
-        $post_id = $this->entry['post_id'];
605
+        $post_id = $this->entry[ 'post_id' ];
606 606
 
607 607
         // Security check
608
-        if( false === GVCommon::has_cap( 'edit_post', $post_id ) ) {
609
-            do_action( 'gravityview_log_error', 'The current user does not have the ability to edit Post #'.$post_id );
608
+        if ( false === GVCommon::has_cap( 'edit_post', $post_id ) ) {
609
+            do_action( 'gravityview_log_error', 'The current user does not have the ability to edit Post #' . $post_id );
610 610
             return;
611 611
         }
612 612
 
@@ -618,25 +618,25 @@  discard block
 block discarded – undo
618 618
 
619 619
             $field = RGFormsModel::get_field( $form, $field_id );
620 620
 
621
-            if( ! $field ) {
621
+            if ( ! $field ) {
622 622
                 continue;
623 623
             }
624 624
 
625
-            if( GFCommon::is_post_field( $field ) && 'post_category' !== $field->type ) {
625
+            if ( GFCommon::is_post_field( $field ) && 'post_category' !== $field->type ) {
626 626
 
627 627
                 // Get the value of the field, including $_POSTed value
628 628
                 $value = RGFormsModel::get_field_value( $field );
629 629
 
630 630
                 // Use temporary entry variable, to make values available to fill_post_template() and update_post_image()
631 631
                 $entry_tmp = $this->entry;
632
-                $entry_tmp["{$field_id}"] = $value;
632
+                $entry_tmp[ "{$field_id}" ] = $value;
633 633
 
634
-                switch( $field->type ) {
634
+                switch ( $field->type ) {
635 635
 
636 636
                     case 'post_title':
637 637
                         $post_title = $value;
638
-                        if( rgar( $form, 'postTitleTemplateEnabled' ) ) {
639
-                            $post_title = $this->fill_post_template( $form['postTitleTemplate'], $form, $entry_tmp );
638
+                        if ( rgar( $form, 'postTitleTemplateEnabled' ) ) {
639
+                            $post_title = $this->fill_post_template( $form[ 'postTitleTemplate' ], $form, $entry_tmp );
640 640
                         }
641 641
                         $updated_post->post_title = $post_title;
642 642
                         $updated_post->post_name  = $post_title;
@@ -645,8 +645,8 @@  discard block
 block discarded – undo
645 645
 
646 646
                     case 'post_content':
647 647
                         $post_content = $value;
648
-                        if( rgar( $form, 'postContentTemplateEnabled' ) ) {
649
-                            $post_content = $this->fill_post_template( $form['postContentTemplate'], $form, $entry_tmp, true );
648
+                        if ( rgar( $form, 'postContentTemplateEnabled' ) ) {
649
+                            $post_content = $this->fill_post_template( $form[ 'postContentTemplate' ], $form, $entry_tmp, true );
650 650
                         }
651 651
                         $updated_post->post_content = $post_content;
652 652
                         unset( $post_content );
@@ -660,12 +660,12 @@  discard block
 block discarded – undo
660 660
                     case 'post_category':
661 661
                         break;
662 662
                     case 'post_custom_field':
663
-                        if( ! empty( $field->customFieldTemplateEnabled ) ) {
663
+                        if ( ! empty( $field->customFieldTemplateEnabled ) ) {
664 664
                             $value = $this->fill_post_template( $field->customFieldTemplate, $form, $entry_tmp, true );
665 665
                         }
666 666
 
667 667
 	                    if ( $this->is_field_json_encoded( $field ) && ! is_string( $value ) ) {
668
-		                    $value = function_exists('wp_json_encode') ? wp_json_encode( $value ) : json_encode( $value );
668
+		                    $value = function_exists( 'wp_json_encode' ) ? wp_json_encode( $value ) : json_encode( $value );
669 669
 	                    }
670 670
 
671 671
                         update_post_meta( $post_id, $field->postCustomFieldName, $value );
@@ -678,7 +678,7 @@  discard block
 block discarded – undo
678 678
                 }
679 679
 
680 680
                 // update entry after
681
-                $this->entry["{$field_id}"] = $value;
681
+                $this->entry[ "{$field_id}" ] = $value;
682 682
 
683 683
                 $update_entry = true;
684 684
 
@@ -687,25 +687,25 @@  discard block
 block discarded – undo
687 687
 
688 688
         }
689 689
 
690
-        if( $update_entry ) {
690
+        if ( $update_entry ) {
691 691
 
692 692
             $return_entry = GFAPI::update_entry( $this->entry );
693 693
 
694
-            if( is_wp_error( $return_entry ) ) {
694
+            if ( is_wp_error( $return_entry ) ) {
695 695
                do_action( 'gravityview_log_error', 'Updating the entry post fields failed', array( '$this->entry' => $this->entry, '$return_entry' => $return_entry ) );
696 696
             } else {
697
-                do_action( 'gravityview_log_debug', 'Updating the entry post fields for post #'.$post_id.' succeeded' );
697
+                do_action( 'gravityview_log_debug', 'Updating the entry post fields for post #' . $post_id . ' succeeded' );
698 698
             }
699 699
 
700 700
         }
701 701
 
702 702
         $return_post = wp_update_post( $updated_post, true );
703 703
 
704
-        if( is_wp_error( $return_post ) ) {
704
+        if ( is_wp_error( $return_post ) ) {
705 705
             $return_post->add_data( $updated_post, '$updated_post' );
706 706
             do_action( 'gravityview_log_error', 'Updating the post content failed', compact( 'updated_post', 'return_post' ) );
707 707
         } else {
708
-            do_action( 'gravityview_log_debug', 'Updating the post content for post #'.$post_id.' succeeded', $updated_post );
708
+            do_action( 'gravityview_log_debug', 'Updating the post content for post #' . $post_id . ' succeeded', $updated_post );
709 709
         }
710 710
     }
711 711
 
@@ -723,7 +723,7 @@  discard block
 block discarded – undo
723 723
         $input_type = RGFormsModel::get_input_type( $field );
724 724
 
725 725
 	    // Only certain custom field types are supported
726
-	    switch( $input_type ) {
726
+	    switch ( $input_type ) {
727 727
 		    case 'fileupload':
728 728
 		    case 'list':
729 729
 		    case 'multiselect':
@@ -760,7 +760,7 @@  discard block
 block discarded – undo
760 760
         $output = GFCommon::replace_variables( $output, $form, $entry, false, false, false );
761 761
 
762 762
         // replace conditional shortcodes
763
-        if( $do_shortcode ) {
763
+        if ( $do_shortcode ) {
764 764
             $output = do_shortcode( $output );
765 765
         }
766 766
 
@@ -779,18 +779,18 @@  discard block
 block discarded – undo
779 779
      */
780 780
     private function after_update() {
781 781
 
782
-        do_action( 'gform_after_update_entry', $this->form, $this->entry['id'], self::$original_entry );
783
-        do_action( "gform_after_update_entry_{$this->form['id']}", $this->form, $this->entry['id'], self::$original_entry );
782
+        do_action( 'gform_after_update_entry', $this->form, $this->entry[ 'id' ], self::$original_entry );
783
+        do_action( "gform_after_update_entry_{$this->form[ 'id' ]}", $this->form, $this->entry[ 'id' ], self::$original_entry );
784 784
 
785 785
         // Re-define the entry now that we've updated it.
786
-        $entry = RGFormsModel::get_lead( $this->entry['id'] );
786
+        $entry = RGFormsModel::get_lead( $this->entry[ 'id' ] );
787 787
 
788 788
         $entry = GFFormsModel::set_entry_meta( $entry, $this->form );
789 789
 
790 790
         // We need to clear the cache because Gravity Forms caches the field values, which
791 791
         // we have just updated.
792
-        foreach ($this->form['fields'] as $key => $field) {
793
-            GFFormsModel::refresh_lead_field_value( $entry['id'], $field->id );
792
+        foreach ( $this->form[ 'fields' ] as $key => $field ) {
793
+            GFFormsModel::refresh_lead_field_value( $entry[ 'id' ], $field->id );
794 794
         }
795 795
 
796 796
         $this->entry = $entry;
@@ -808,7 +808,7 @@  discard block
 block discarded – undo
808 808
 
809 809
         <div class="gv-edit-entry-wrapper"><?php
810 810
 
811
-            $javascript = gravityview_ob_include( GravityView_Edit_Entry::$file .'/partials/inline-javascript.php', $this );
811
+            $javascript = gravityview_ob_include( GravityView_Edit_Entry::$file . '/partials/inline-javascript.php', $this );
812 812
 
813 813
             /**
814 814
              * Fixes weird wpautop() issue
@@ -824,7 +824,7 @@  discard block
 block discarded – undo
824 824
                      * @param string $edit_entry_title Modify the "Edit Entry" title
825 825
                      * @param GravityView_Edit_Entry_Render $this This object
826 826
                      */
827
-                    $edit_entry_title = apply_filters('gravityview_edit_entry_title', __('Edit Entry', 'gravityview'), $this );
827
+                    $edit_entry_title = apply_filters( 'gravityview_edit_entry_title', __( 'Edit Entry', 'gravityview' ), $this );
828 828
 
829 829
                     echo esc_attr( $edit_entry_title );
830 830
             ?></span>
@@ -870,20 +870,20 @@  discard block
 block discarded – undo
870 870
      */
871 871
     private function maybe_print_message() {
872 872
 
873
-        if( rgpost('action') === 'update' ) {
873
+        if ( rgpost( 'action' ) === 'update' ) {
874 874
 
875 875
             $back_link = esc_url( remove_query_arg( array( 'page', 'view', 'edit' ) ) );
876 876
 
877
-            if( ! $this->is_valid ){
877
+            if ( ! $this->is_valid ) {
878 878
 
879 879
                 // Keeping this compatible with Gravity Forms.
880
-                $validation_message = "<div class='validation_error'>" . __('There was a problem with your submission.', 'gravityview') . " " . __('Errors have been highlighted below.', 'gravityview') . "</div>";
881
-                $message = apply_filters("gform_validation_message_{$this->form['id']}", apply_filters("gform_validation_message", $validation_message, $this->form), $this->form);
880
+                $validation_message = "<div class='validation_error'>" . __( 'There was a problem with your submission.', 'gravityview' ) . " " . __( 'Errors have been highlighted below.', 'gravityview' ) . "</div>";
881
+                $message = apply_filters( "gform_validation_message_{$this->form[ 'id' ]}", apply_filters( "gform_validation_message", $validation_message, $this->form ), $this->form );
882 882
 
883
-                echo GVCommon::generate_notice( $message , 'gv-error' );
883
+                echo GVCommon::generate_notice( $message, 'gv-error' );
884 884
 
885 885
             } else {
886
-                $entry_updated_message = sprintf( esc_attr__('Entry Updated. %sReturn to Entry%s', 'gravityview'), '<a href="'. $back_link .'">', '</a>' );
886
+                $entry_updated_message = sprintf( esc_attr__( 'Entry Updated. %sReturn to Entry%s', 'gravityview' ), '<a href="' . $back_link . '">', '</a>' );
887 887
 
888 888
                 /**
889 889
                  * @filter `gravityview/edit_entry/success` Modify the edit entry success message (including the anchor link)
@@ -893,7 +893,7 @@  discard block
 block discarded – undo
893 893
                  * @param array $entry Gravity Forms entry array
894 894
                  * @param string $back_link URL to return to the original entry. @since 1.6
895 895
                  */
896
-                $message = apply_filters( 'gravityview/edit_entry/success', $entry_updated_message , $this->view_id, $this->entry, $back_link );
896
+                $message = apply_filters( 'gravityview/edit_entry/success', $entry_updated_message, $this->view_id, $this->entry, $back_link );
897 897
 
898 898
                 echo GVCommon::generate_notice( $message );
899 899
             }
@@ -917,21 +917,21 @@  discard block
 block discarded – undo
917 917
          */
918 918
         do_action( 'gravityview/edit-entry/render/before', $this );
919 919
 
920
-        add_filter( 'gform_pre_render', array( $this, 'filter_modify_form_fields'), 5000, 3 );
921
-        add_filter( 'gform_submit_button', array( $this, 'render_form_buttons') );
920
+        add_filter( 'gform_pre_render', array( $this, 'filter_modify_form_fields' ), 5000, 3 );
921
+        add_filter( 'gform_submit_button', array( $this, 'render_form_buttons' ) );
922 922
         add_filter( 'gform_disable_view_counter', '__return_true' );
923 923
 
924 924
         add_filter( 'gform_field_input', array( $this, 'verify_user_can_edit_post' ), 5, 5 );
925 925
         add_filter( 'gform_field_input', array( $this, 'modify_edit_field_input' ), 10, 5 );
926 926
 
927 927
         // We need to remove the fake $_GET['page'] arg to avoid rendering form as if in admin.
928
-        unset( $_GET['page'] );
928
+        unset( $_GET[ 'page' ] );
929 929
 
930 930
         // TODO: Verify multiple-page forms
931 931
 
932 932
         ob_start(); // Prevent PHP warnings possibly caused by prefilling list fields for conditional logic
933 933
 
934
-        $html = GFFormDisplay::get_form( $this->form['id'], false, false, true, $this->entry );
934
+        $html = GFFormDisplay::get_form( $this->form[ 'id' ], false, false, true, $this->entry );
935 935
 
936 936
         ob_get_clean();
937 937
 
@@ -957,7 +957,7 @@  discard block
 block discarded – undo
957 957
      * @return string
958 958
      */
959 959
     public function render_form_buttons() {
960
-        return gravityview_ob_include( GravityView_Edit_Entry::$file .'/partials/form-buttons.php', $this );
960
+        return gravityview_ob_include( GravityView_Edit_Entry::$file . '/partials/form-buttons.php', $this );
961 961
     }
962 962
 
963 963
 
@@ -977,10 +977,10 @@  discard block
 block discarded – undo
977 977
     public function filter_modify_form_fields( $form, $ajax = false, $field_values = '' ) {
978 978
 
979 979
         // In case we have validated the form, use it to inject the validation results into the form render
980
-        if( isset( $this->form_after_validation ) ) {
980
+        if ( isset( $this->form_after_validation ) ) {
981 981
             $form = $this->form_after_validation;
982 982
         } else {
983
-            $form['fields'] = $this->get_configured_edit_fields( $form, $this->view_id );
983
+            $form[ 'fields' ] = $this->get_configured_edit_fields( $form, $this->view_id );
984 984
         }
985 985
 
986 986
         $form = $this->filter_conditional_logic( $form );
@@ -988,8 +988,8 @@  discard block
 block discarded – undo
988 988
         $form = $this->prefill_conditional_logic( $form );
989 989
 
990 990
         // for now we don't support Save and Continue feature.
991
-        if( ! self::$supports_save_and_continue ) {
992
-	        unset( $form['save'] );
991
+        if ( ! self::$supports_save_and_continue ) {
992
+	        unset( $form[ 'save' ] );
993 993
         }
994 994
 
995 995
         return $form;
@@ -1010,29 +1010,29 @@  discard block
 block discarded – undo
1010 1010
      */
1011 1011
     public function verify_user_can_edit_post( $field_content = '', $field, $value, $lead_id = 0, $form_id ) {
1012 1012
 
1013
-        if( GFCommon::is_post_field( $field ) ) {
1013
+        if ( GFCommon::is_post_field( $field ) ) {
1014 1014
 
1015 1015
             $message = null;
1016 1016
 
1017 1017
             // First, make sure they have the capability to edit the post.
1018
-            if( false === current_user_can( 'edit_post', $this->entry['post_id'] ) ) {
1018
+            if ( false === current_user_can( 'edit_post', $this->entry[ 'post_id' ] ) ) {
1019 1019
 
1020 1020
                 /**
1021 1021
                  * @filter `gravityview/edit_entry/unsupported_post_field_text` Modify the message when someone isn't able to edit a post
1022 1022
                  * @param string $message The existing "You don't have permission..." text
1023 1023
                  */
1024
-                $message = apply_filters('gravityview/edit_entry/unsupported_post_field_text', __('You don&rsquo;t have permission to edit this post.', 'gravityview') );
1024
+                $message = apply_filters( 'gravityview/edit_entry/unsupported_post_field_text', __( 'You don&rsquo;t have permission to edit this post.', 'gravityview' ) );
1025 1025
 
1026
-            } elseif( null === get_post( $this->entry['post_id'] ) ) {
1026
+            } elseif ( null === get_post( $this->entry[ 'post_id' ] ) ) {
1027 1027
                 /**
1028 1028
                  * @filter `gravityview/edit_entry/no_post_text` Modify the message when someone is editing an entry attached to a post that no longer exists
1029 1029
                  * @param string $message The existing "This field is not editable; the post no longer exists." text
1030 1030
                  */
1031
-                $message = apply_filters('gravityview/edit_entry/no_post_text', __('This field is not editable; the post no longer exists.', 'gravityview' ) );
1031
+                $message = apply_filters( 'gravityview/edit_entry/no_post_text', __( 'This field is not editable; the post no longer exists.', 'gravityview' ) );
1032 1032
             }
1033 1033
 
1034
-            if( $message ) {
1035
-                $field_content = sprintf('<div class="ginput_container ginput_container_' . $field->type . '">%s</div>', wpautop( $message ) );
1034
+            if ( $message ) {
1035
+                $field_content = sprintf( '<div class="ginput_container ginput_container_' . $field->type . '">%s</div>', wpautop( $message ) );
1036 1036
             }
1037 1037
         }
1038 1038
 
@@ -1057,8 +1057,8 @@  discard block
 block discarded – undo
1057 1057
 
1058 1058
         // If the form has been submitted, then we don't need to pre-fill the values,
1059 1059
         // Except for fileupload type and when a field input is overridden- run always!!
1060
-        if(
1061
-            ( $this->is_edit_entry_submission() && !in_array( $field->type, array( 'fileupload', 'post_image' ) ) )
1060
+        if (
1061
+            ( $this->is_edit_entry_submission() && ! in_array( $field->type, array( 'fileupload', 'post_image' ) ) )
1062 1062
             && false === ( $gv_field && is_callable( array( $gv_field, 'get_field_input' ) ) )
1063 1063
             && ! GFCommon::is_product_field( $field->type )
1064 1064
             || ! empty( $field_content )
@@ -1078,7 +1078,7 @@  discard block
 block discarded – undo
1078 1078
 	    $return = null;
1079 1079
 
1080 1080
         /** @var GravityView_Field $gv_field */
1081
-        if( $gv_field && is_callable( array( $gv_field, 'get_field_input' ) ) ) {
1081
+        if ( $gv_field && is_callable( array( $gv_field, 'get_field_input' ) ) ) {
1082 1082
             $return = $gv_field->get_field_input( $this->form, $field_value, $this->entry, $field );
1083 1083
         } else {
1084 1084
 	        $return = $field->get_field_input( $this->form, $field_value, $this->entry );
@@ -1087,7 +1087,7 @@  discard block
 block discarded – undo
1087 1087
 	    // If there was output, it's an error
1088 1088
 	    $warnings = ob_get_clean();
1089 1089
 
1090
-	    if( !empty( $warnings ) ) {
1090
+	    if ( ! empty( $warnings ) ) {
1091 1091
 		    do_action( 'gravityview_log_error', __METHOD__ . $warnings, $field_value );
1092 1092
 	    }
1093 1093
 
@@ -1112,7 +1112,7 @@  discard block
 block discarded – undo
1112 1112
         $override_saved_value = apply_filters( 'gravityview/edit_entry/pre_populate/override', false, $field );
1113 1113
 
1114 1114
         // We're dealing with multiple inputs (e.g. checkbox) but not time or date (as it doesn't store data in input IDs)
1115
-        if( isset( $field->inputs ) && is_array( $field->inputs ) && !in_array( $field->type, array( 'time', 'date' ) ) ) {
1115
+        if ( isset( $field->inputs ) && is_array( $field->inputs ) && ! in_array( $field->type, array( 'time', 'date' ) ) ) {
1116 1116
 
1117 1117
             $field_value = array();
1118 1118
 
@@ -1121,10 +1121,10 @@  discard block
 block discarded – undo
1121 1121
 
1122 1122
             foreach ( (array)$field->inputs as $input ) {
1123 1123
 
1124
-                $input_id = strval( $input['id'] );
1124
+                $input_id = strval( $input[ 'id' ] );
1125 1125
                 
1126 1126
                 if ( isset( $this->entry[ $input_id ] ) && ! gv_empty( $this->entry[ $input_id ], false, false ) ) {
1127
-                    $field_value[ $input_id ] =  'post_category' === $field->type ? GFCommon::format_post_category( $this->entry[ $input_id ], true ) : $this->entry[ $input_id ];
1127
+                    $field_value[ $input_id ] = 'post_category' === $field->type ? GFCommon::format_post_category( $this->entry[ $input_id ], true ) : $this->entry[ $input_id ];
1128 1128
                     $allow_pre_populated = false;
1129 1129
                 }
1130 1130
 
@@ -1132,7 +1132,7 @@  discard block
 block discarded – undo
1132 1132
 
1133 1133
             $pre_value = $field->get_value_submission( array(), false );
1134 1134
 
1135
-            $field_value = ! $allow_pre_populated && ! ( $override_saved_value && !gv_empty( $pre_value, false, false ) ) ? $field_value : $pre_value;
1135
+            $field_value = ! $allow_pre_populated && ! ( $override_saved_value && ! gv_empty( $pre_value, false, false ) ) ? $field_value : $pre_value;
1136 1136
 
1137 1137
         } else {
1138 1138
 
@@ -1143,13 +1143,13 @@  discard block
 block discarded – undo
1143 1143
 
1144 1144
             // saved field entry value (if empty, fallback to the pre-populated value, if exists)
1145 1145
             // or pre-populated value if not empty and set to override saved value
1146
-            $field_value = !gv_empty( $this->entry[ $id ], false, false ) && ! ( $override_saved_value && !gv_empty( $pre_value, false, false ) ) ? $this->entry[ $id ] : $pre_value;
1146
+            $field_value = ! gv_empty( $this->entry[ $id ], false, false ) && ! ( $override_saved_value && ! gv_empty( $pre_value, false, false ) ) ? $this->entry[ $id ] : $pre_value;
1147 1147
 
1148 1148
             // in case field is post_category but inputType is select, multi-select or radio, convert value into array of category IDs.
1149
-            if ( 'post_category' === $field->type && !gv_empty( $field_value, false, false ) ) {
1149
+            if ( 'post_category' === $field->type && ! gv_empty( $field_value, false, false ) ) {
1150 1150
                 $categories = array();
1151 1151
                 foreach ( explode( ',', $field_value ) as $cat_string ) {
1152
-                    $categories[] = GFCommon::format_post_category( $cat_string, true );
1152
+                    $categories[ ] = GFCommon::format_post_category( $cat_string, true );
1153 1153
                 }
1154 1154
                 $field_value = 'multiselect' === $field->get_input_type() ? $categories : implode( '', $categories );
1155 1155
             }
@@ -1177,7 +1177,7 @@  discard block
 block discarded – undo
1177 1177
 	     * @param GF_Field $field Gravity Forms field object
1178 1178
 	     * @param GravityView_Edit_Entry_Render $this Current object
1179 1179
 	     */
1180
-	    $field_value = apply_filters( 'gravityview/edit_entry/field_value_' . $field->type , $field_value, $field, $this );
1180
+	    $field_value = apply_filters( 'gravityview/edit_entry/field_value_' . $field->type, $field_value, $field, $this );
1181 1181
 
1182 1182
         return $field_value;
1183 1183
     }
@@ -1194,12 +1194,12 @@  discard block
 block discarded – undo
1194 1194
      */
1195 1195
     public function gform_pre_validation( $form ) {
1196 1196
 
1197
-        if( ! $this->verify_nonce() ) {
1197
+        if ( ! $this->verify_nonce() ) {
1198 1198
             return $form;
1199 1199
         }
1200 1200
 
1201 1201
         // Fix PHP warning regarding undefined index.
1202
-        foreach ( $form['fields'] as &$field) {
1202
+        foreach ( $form[ 'fields' ] as &$field ) {
1203 1203
 
1204 1204
             // This is because we're doing admin form pretending to be front-end, so Gravity Forms
1205 1205
             // expects certain field array items to be set.
@@ -1207,7 +1207,7 @@  discard block
 block discarded – undo
1207 1207
 	            $field->{$key} = isset( $field->{$key} ) ? $field->{$key} : NULL;
1208 1208
             }
1209 1209
 
1210
-            switch( RGFormsModel::get_input_type( $field ) ) {
1210
+            switch ( RGFormsModel::get_input_type( $field ) ) {
1211 1211
 
1212 1212
                 /**
1213 1213
                  * 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.
@@ -1221,37 +1221,37 @@  discard block
 block discarded – undo
1221 1221
                     // Set the previous value
1222 1222
                     $entry = $this->get_entry();
1223 1223
 
1224
-                    $input_name = 'input_'.$field->id;
1225
-                    $form_id = $form['id'];
1224
+                    $input_name = 'input_' . $field->id;
1225
+                    $form_id = $form[ 'id' ];
1226 1226
 
1227 1227
                     $value = NULL;
1228 1228
 
1229 1229
                     // Use the previous entry value as the default.
1230
-                    if( isset( $entry[ $field->id ] ) ) {
1230
+                    if ( isset( $entry[ $field->id ] ) ) {
1231 1231
                         $value = $entry[ $field->id ];
1232 1232
                     }
1233 1233
 
1234 1234
                     // If this is a single upload file
1235
-                    if( !empty( $_FILES[ $input_name ] ) && !empty( $_FILES[ $input_name ]['name'] ) ) {
1236
-                        $file_path = GFFormsModel::get_file_upload_path( $form['id'], $_FILES[ $input_name ]['name'] );
1237
-                        $value = $file_path['url'];
1235
+                    if ( ! empty( $_FILES[ $input_name ] ) && ! empty( $_FILES[ $input_name ][ 'name' ] ) ) {
1236
+                        $file_path = GFFormsModel::get_file_upload_path( $form[ 'id' ], $_FILES[ $input_name ][ 'name' ] );
1237
+                        $value = $file_path[ 'url' ];
1238 1238
 
1239 1239
                     } else {
1240 1240
 
1241 1241
                         // Fix PHP warning on line 1498 of form_display.php for post_image fields
1242 1242
                         // Fix PHP Notice:  Undefined index:  size in form_display.php on line 1511
1243
-                        $_FILES[ $input_name ] = array('name' => '', 'size' => '' );
1243
+                        $_FILES[ $input_name ] = array( 'name' => '', 'size' => '' );
1244 1244
 
1245 1245
                     }
1246 1246
 
1247
-                    if( rgar($field, "multipleFiles") ) {
1247
+                    if ( rgar( $field, "multipleFiles" ) ) {
1248 1248
 
1249 1249
                         // If there are fresh uploads, process and merge them.
1250 1250
                         // Otherwise, use the passed values, which should be json-encoded array of URLs
1251
-                        if( isset( GFFormsModel::$uploaded_files[$form_id][$input_name] ) ) {
1251
+                        if ( isset( GFFormsModel::$uploaded_files[ $form_id ][ $input_name ] ) ) {
1252 1252
                             $value = empty( $value ) ? '[]' : $value;
1253 1253
                             $value = stripslashes_deep( $value );
1254
-                            $value = GFFormsModel::prepare_value( $form, $field, $value, $input_name, $entry['id'], array());
1254
+                            $value = GFFormsModel::prepare_value( $form, $field, $value, $input_name, $entry[ 'id' ], array() );
1255 1255
                         }
1256 1256
 
1257 1257
                     } else {
@@ -1269,8 +1269,8 @@  discard block
 block discarded – undo
1269 1269
 
1270 1270
                 case 'number':
1271 1271
                     // Fix "undefined index" issue at line 1286 in form_display.php
1272
-                    if( !isset( $_POST['input_'.$field->id ] ) ) {
1273
-                        $_POST['input_'.$field->id ] = NULL;
1272
+                    if ( ! isset( $_POST[ 'input_' . $field->id ] ) ) {
1273
+                        $_POST[ 'input_' . $field->id ] = NULL;
1274 1274
                     }
1275 1275
                     break;
1276 1276
             }
@@ -1307,7 +1307,7 @@  discard block
 block discarded – undo
1307 1307
          * You can enter whatever you want!
1308 1308
          * We try validating, and customize the results using `self::custom_validation()`
1309 1309
          */
1310
-        add_filter( 'gform_validation_'. $this->form_id, array( $this, 'custom_validation' ), 10, 4);
1310
+        add_filter( 'gform_validation_' . $this->form_id, array( $this, 'custom_validation' ), 10, 4 );
1311 1311
 
1312 1312
         // Needed by the validate funtion
1313 1313
         $failed_validation_page = NULL;
@@ -1315,14 +1315,14 @@  discard block
 block discarded – undo
1315 1315
 
1316 1316
         // Prevent entry limit from running when editing an entry, also
1317 1317
         // prevent form scheduling from preventing editing
1318
-        unset( $this->form['limitEntries'], $this->form['scheduleForm'] );
1318
+        unset( $this->form[ 'limitEntries' ], $this->form[ 'scheduleForm' ] );
1319 1319
 
1320 1320
         // Hide fields depending on Edit Entry settings
1321
-        $this->form['fields'] = $this->get_configured_edit_fields( $this->form, $this->view_id );
1321
+        $this->form[ 'fields' ] = $this->get_configured_edit_fields( $this->form, $this->view_id );
1322 1322
 
1323 1323
         $this->is_valid = GFFormDisplay::validate( $this->form, $field_values, 1, $failed_validation_page );
1324 1324
 
1325
-        remove_filter( 'gform_validation_'. $this->form_id, array( $this, 'custom_validation' ), 10 );
1325
+        remove_filter( 'gform_validation_' . $this->form_id, array( $this, 'custom_validation' ), 10 );
1326 1326
     }
1327 1327
 
1328 1328
 
@@ -1339,13 +1339,13 @@  discard block
 block discarded – undo
1339 1339
      */
1340 1340
     public function custom_validation( $validation_results ) {
1341 1341
 
1342
-        do_action('gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Validation results: ', $validation_results );
1342
+        do_action( 'gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Validation results: ', $validation_results );
1343 1343
 
1344
-        do_action('gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] $_POSTed data (sanitized): ', esc_html( print_r( $_POST, true ) ) );
1344
+        do_action( 'gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] $_POSTed data (sanitized): ', esc_html( print_r( $_POST, true ) ) );
1345 1345
 
1346 1346
         $gv_valid = true;
1347 1347
 
1348
-        foreach ( $validation_results['form']['fields'] as $key => &$field ) {
1348
+        foreach ( $validation_results[ 'form' ][ 'fields' ] as $key => &$field ) {
1349 1349
 
1350 1350
             $value = RGFormsModel::get_field_value( $field );
1351 1351
             $field_type = RGFormsModel::get_input_type( $field );
@@ -1358,35 +1358,35 @@  discard block
 block discarded – undo
1358 1358
                 case 'post_image':
1359 1359
 
1360 1360
                     // in case nothing is uploaded but there are already files saved
1361
-                    if( !empty( $field->failed_validation ) && !empty( $field->isRequired ) && !empty( $value ) ) {
1361
+                    if ( ! empty( $field->failed_validation ) && ! empty( $field->isRequired ) && ! empty( $value ) ) {
1362 1362
                         $field->failed_validation = false;
1363 1363
                         unset( $field->validation_message );
1364 1364
                     }
1365 1365
 
1366 1366
                     // validate if multi file upload reached max number of files [maxFiles] => 2
1367
-                    if( rgobj( $field, 'maxFiles') && rgobj( $field, 'multipleFiles') ) {
1367
+                    if ( rgobj( $field, 'maxFiles' ) && rgobj( $field, 'multipleFiles' ) ) {
1368 1368
 
1369 1369
                         $input_name = 'input_' . $field->id;
1370 1370
                         //uploaded
1371
-                        $file_names = isset( GFFormsModel::$uploaded_files[ $validation_results['form']['id'] ][ $input_name ] ) ? GFFormsModel::$uploaded_files[ $validation_results['form']['id'] ][ $input_name ] : array();
1371
+                        $file_names = isset( GFFormsModel::$uploaded_files[ $validation_results[ 'form' ][ 'id' ] ][ $input_name ] ) ? GFFormsModel::$uploaded_files[ $validation_results[ 'form' ][ 'id' ] ][ $input_name ] : array();
1372 1372
 
1373 1373
                         //existent
1374 1374
                         $entry = $this->get_entry();
1375 1375
                         $value = NULL;
1376
-                        if( isset( $entry[ $field->id ] ) ) {
1376
+                        if ( isset( $entry[ $field->id ] ) ) {
1377 1377
                             $value = json_decode( $entry[ $field->id ], true );
1378 1378
                         }
1379 1379
 
1380 1380
                         // count uploaded files and existent entry files
1381 1381
                         $count_files = count( $file_names ) + count( $value );
1382 1382
 
1383
-                        if( $count_files > $field->maxFiles ) {
1383
+                        if ( $count_files > $field->maxFiles ) {
1384 1384
                             $field->validation_message = __( 'Maximum number of files reached', 'gravityview' );
1385 1385
                             $field->failed_validation = 1;
1386 1386
                             $gv_valid = false;
1387 1387
 
1388 1388
                             // in case of error make sure the newest upload files are removed from the upload input
1389
-                            GFFormsModel::$uploaded_files[ $validation_results['form']['id'] ] = null;
1389
+                            GFFormsModel::$uploaded_files[ $validation_results[ 'form' ][ 'id' ] ] = null;
1390 1390
                         }
1391 1391
 
1392 1392
                     }
@@ -1397,7 +1397,7 @@  discard block
 block discarded – undo
1397 1397
             }
1398 1398
 
1399 1399
             // This field has failed validation.
1400
-            if( !empty( $field->failed_validation ) ) {
1400
+            if ( ! empty( $field->failed_validation ) ) {
1401 1401
 
1402 1402
                 do_action( 'gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Field is invalid.', array( 'field' => $field, 'value' => $value ) );
1403 1403
 
@@ -1415,32 +1415,32 @@  discard block
 block discarded – undo
1415 1415
                 }
1416 1416
 
1417 1417
                 // You can't continue inside a switch, so we do it after.
1418
-                if( empty( $field->failed_validation ) ) {
1418
+                if ( empty( $field->failed_validation ) ) {
1419 1419
                     continue;
1420 1420
                 }
1421 1421
 
1422 1422
                 // checks if the No Duplicates option is not validating entry against itself, since
1423 1423
                 // we're editing a stored entry, it would also assume it's a duplicate.
1424
-                if( !empty( $field->noDuplicates ) ) {
1424
+                if ( ! empty( $field->noDuplicates ) ) {
1425 1425
 
1426 1426
                     $entry = $this->get_entry();
1427 1427
 
1428 1428
                     // If the value of the entry is the same as the stored value
1429 1429
                     // Then we can assume it's not a duplicate, it's the same.
1430
-                    if( !empty( $entry ) && $value == $entry[ $field->id ] ) {
1430
+                    if ( ! empty( $entry ) && $value == $entry[ $field->id ] ) {
1431 1431
                         //if value submitted was not changed, then don't validate
1432 1432
                         $field->failed_validation = false;
1433 1433
 
1434 1434
                         unset( $field->validation_message );
1435 1435
 
1436
-                        do_action('gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Field not a duplicate; it is the same entry.', $entry );
1436
+                        do_action( 'gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Field not a duplicate; it is the same entry.', $entry );
1437 1437
 
1438 1438
                         continue;
1439 1439
                     }
1440 1440
                 }
1441 1441
 
1442 1442
                 // if here then probably we are facing the validation 'At least one field must be filled out'
1443
-                if( GFFormDisplay::is_empty( $field, $this->form_id  ) && empty( $field->isRequired ) ) {
1443
+                if ( GFFormDisplay::is_empty( $field, $this->form_id ) && empty( $field->isRequired ) ) {
1444 1444
                     unset( $field->validation_message );
1445 1445
 	                $field->validation_message = false;
1446 1446
                     continue;
@@ -1452,12 +1452,12 @@  discard block
 block discarded – undo
1452 1452
 
1453 1453
         }
1454 1454
 
1455
-        $validation_results['is_valid'] = $gv_valid;
1455
+        $validation_results[ 'is_valid' ] = $gv_valid;
1456 1456
 
1457
-        do_action('gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Validation results.', $validation_results );
1457
+        do_action( 'gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Validation results.', $validation_results );
1458 1458
 
1459 1459
         // We'll need this result when rendering the form ( on GFFormDisplay::get_form )
1460
-        $this->form_after_validation = $validation_results['form'];
1460
+        $this->form_after_validation = $validation_results[ 'form' ];
1461 1461
 
1462 1462
         return $validation_results;
1463 1463
     }
@@ -1470,7 +1470,7 @@  discard block
 block discarded – undo
1470 1470
      */
1471 1471
     public function get_entry() {
1472 1472
 
1473
-        if( empty( $this->entry ) ) {
1473
+        if ( empty( $this->entry ) ) {
1474 1474
             // Get the database value of the entry that's being edited
1475 1475
             $this->entry = gravityview_get_entry( GravityView_frontend::is_single_entry() );
1476 1476
         }
@@ -1497,10 +1497,10 @@  discard block
 block discarded – undo
1497 1497
         $properties = GravityView_View_Data::getInstance()->get_fields( $view_id );
1498 1498
 
1499 1499
         // If edit tab not yet configured, show all fields
1500
-        $edit_fields = !empty( $properties['edit_edit-fields'] ) ? $properties['edit_edit-fields'] : NULL;
1500
+        $edit_fields = ! empty( $properties[ 'edit_edit-fields' ] ) ? $properties[ 'edit_edit-fields' ] : NULL;
1501 1501
 
1502 1502
         // Hide fields depending on admin settings
1503
-        $fields = $this->filter_fields( $form['fields'], $edit_fields );
1503
+        $fields = $this->filter_fields( $form[ 'fields' ], $edit_fields );
1504 1504
 
1505 1505
 	    // If Edit Entry fields are configured, remove adminOnly field settings. Otherwise, don't.
1506 1506
 	    $fields = $this->filter_admin_only_fields( $fields, $edit_fields, $form, $view_id );
@@ -1532,7 +1532,7 @@  discard block
 block discarded – undo
1532 1532
      */
1533 1533
     private function filter_fields( $fields, $configured_fields ) {
1534 1534
 
1535
-        if( empty( $fields ) || !is_array( $fields ) ) {
1535
+        if ( empty( $fields ) || ! is_array( $fields ) ) {
1536 1536
             return $fields;
1537 1537
         }
1538 1538
 
@@ -1545,18 +1545,18 @@  discard block
 block discarded – undo
1545 1545
 
1546 1546
             // Remove the fields that have calculation properties and keep them to be used later
1547 1547
             // @since 1.16.2
1548
-            if( $field->has_calculation() ) {
1549
-                $this->fields_with_calculation[] = $field;
1548
+            if ( $field->has_calculation() ) {
1549
+                $this->fields_with_calculation[ ] = $field;
1550 1550
                 // don't remove the calculation fields on form render.
1551 1551
             }
1552 1552
 
1553
-            if( in_array( $field->type, $field_type_blacklist ) ) {
1553
+            if ( in_array( $field->type, $field_type_blacklist ) ) {
1554 1554
                 unset( $fields[ $key ] );
1555 1555
             }
1556 1556
         }
1557 1557
 
1558 1558
         // The Edit tab has not been configured, so we return all fields by default.
1559
-        if( empty( $configured_fields ) ) {
1559
+        if ( empty( $configured_fields ) ) {
1560 1560
             return $fields;
1561 1561
         }
1562 1562
 
@@ -1566,8 +1566,8 @@  discard block
 block discarded – undo
1566 1566
 	        /** @var GF_Field $field */
1567 1567
 	        foreach ( $fields as $field ) {
1568 1568
 
1569
-                if( intval( $configured_field['id'] ) === intval( $field->id ) && $this->user_can_edit_field( $configured_field, false ) ) {
1570
-                    $edit_fields[] = $this->merge_field_properties( $field, $configured_field );
1569
+                if ( intval( $configured_field[ 'id' ] ) === intval( $field->id ) && $this->user_can_edit_field( $configured_field, false ) ) {
1570
+                    $edit_fields[ ] = $this->merge_field_properties( $field, $configured_field );
1571 1571
                     break;
1572 1572
                 }
1573 1573
 
@@ -1590,14 +1590,14 @@  discard block
 block discarded – undo
1590 1590
 
1591 1591
         $return_field = $field;
1592 1592
 
1593
-        if( empty( $field_setting['show_label'] ) ) {
1593
+        if ( empty( $field_setting[ 'show_label' ] ) ) {
1594 1594
             $return_field->label = '';
1595
-        } elseif ( !empty( $field_setting['custom_label'] ) ) {
1596
-            $return_field->label = $field_setting['custom_label'];
1595
+        } elseif ( ! empty( $field_setting[ 'custom_label' ] ) ) {
1596
+            $return_field->label = $field_setting[ 'custom_label' ];
1597 1597
         }
1598 1598
 
1599
-        if( !empty( $field_setting['custom_class'] ) ) {
1600
-            $return_field->cssClass .= ' '. gravityview_sanitize_html_class( $field_setting['custom_class'] );
1599
+        if ( ! empty( $field_setting[ 'custom_class' ] ) ) {
1600
+            $return_field->cssClass .= ' ' . gravityview_sanitize_html_class( $field_setting[ 'custom_class' ] );
1601 1601
         }
1602 1602
 
1603 1603
         /**
@@ -1635,16 +1635,16 @@  discard block
 block discarded – undo
1635 1635
 	     */
1636 1636
 	    $use_gf_adminonly_setting = apply_filters( 'gravityview/edit_entry/use_gf_admin_only_setting', empty( $edit_fields ), $form, $view_id );
1637 1637
 
1638
-	    if( $use_gf_adminonly_setting && false === GVCommon::has_cap( 'gravityforms_edit_entries', $this->entry['id'] ) ) {
1639
-            foreach( $fields as $k => $field ) {
1640
-                if( $field->adminOnly ) {
1638
+	    if ( $use_gf_adminonly_setting && false === GVCommon::has_cap( 'gravityforms_edit_entries', $this->entry[ 'id' ] ) ) {
1639
+            foreach ( $fields as $k => $field ) {
1640
+                if ( $field->adminOnly ) {
1641 1641
                     unset( $fields[ $k ] );
1642 1642
                 }
1643 1643
             }
1644 1644
             return $fields;
1645 1645
         }
1646 1646
 
1647
-	    foreach( $fields as &$field ) {
1647
+	    foreach ( $fields as &$field ) {
1648 1648
 		    $field->adminOnly = false;
1649 1649
         }
1650 1650
 
@@ -1668,22 +1668,22 @@  discard block
 block discarded – undo
1668 1668
      */
1669 1669
     function prefill_conditional_logic( $form ) {
1670 1670
 
1671
-        if( ! GFFormDisplay::has_conditional_logic( $form ) ) {
1671
+        if ( ! GFFormDisplay::has_conditional_logic( $form ) ) {
1672 1672
             return $form;
1673 1673
         }
1674 1674
 
1675 1675
         // Have Conditional Logic pre-fill fields as if the data were default values
1676 1676
         /** @var GF_Field $field */
1677
-        foreach ( $form['fields'] as &$field ) {
1677
+        foreach ( $form[ 'fields' ] as &$field ) {
1678 1678
 
1679
-            if( 'checkbox' === $field->type ) {
1679
+            if ( 'checkbox' === $field->type ) {
1680 1680
                 foreach ( $field->get_entry_inputs() as $key => $input ) {
1681
-                    $input_id = $input['id'];
1681
+                    $input_id = $input[ 'id' ];
1682 1682
                     $choice = $field->choices[ $key ];
1683 1683
                     $value = rgar( $this->entry, $input_id );
1684 1684
                     $match = RGFormsModel::choice_value_match( $field, $choice, $value );
1685
-                    if( $match ) {
1686
-                        $field->choices[ $key ]['isSelected'] = true;
1685
+                    if ( $match ) {
1686
+                        $field->choices[ $key ][ 'isSelected' ] = true;
1687 1687
                     }
1688 1688
                 }
1689 1689
             } else {
@@ -1691,15 +1691,15 @@  discard block
 block discarded – undo
1691 1691
                 // We need to run through each field to set the default values
1692 1692
                 foreach ( $this->entry as $field_id => $field_value ) {
1693 1693
 
1694
-                    if( floatval( $field_id ) === floatval( $field->id ) ) {
1694
+                    if ( floatval( $field_id ) === floatval( $field->id ) ) {
1695 1695
 
1696
-                        if( 'list' === $field->type ) {
1696
+                        if ( 'list' === $field->type ) {
1697 1697
                             $list_rows = maybe_unserialize( $field_value );
1698 1698
 
1699 1699
                             $list_field_value = array();
1700
-                            foreach ( (array) $list_rows as $row ) {
1701
-                                foreach ( (array) $row as $column ) {
1702
-                                    $list_field_value[] = $column;
1700
+                            foreach ( (array)$list_rows as $row ) {
1701
+                                foreach ( (array)$row as $column ) {
1702
+                                    $list_field_value[ ] = $column;
1703 1703
                                 }
1704 1704
                             }
1705 1705
 
@@ -1734,16 +1734,16 @@  discard block
 block discarded – undo
1734 1734
          */
1735 1735
         $use_conditional_logic = apply_filters( 'gravityview/edit_entry/conditional_logic', true, $form );
1736 1736
 
1737
-        if( $use_conditional_logic ) {
1737
+        if ( $use_conditional_logic ) {
1738 1738
             return $form;
1739 1739
         }
1740 1740
 
1741
-        foreach( $form['fields'] as &$field ) {
1741
+        foreach ( $form[ 'fields' ] as &$field ) {
1742 1742
             /* @var GF_Field $field */
1743 1743
             $field->conditionalLogic = null;
1744 1744
         }
1745 1745
 
1746
-        unset( $form['button']['conditionalLogic'] );
1746
+        unset( $form[ 'button' ][ 'conditionalLogic' ] );
1747 1747
 
1748 1748
         return $form;
1749 1749
 
@@ -1760,7 +1760,7 @@  discard block
 block discarded – undo
1760 1760
      */
1761 1761
     public function manage_conditional_logic( $has_conditional_logic, $form ) {
1762 1762
 
1763
-        if( ! $this->is_edit_entry() ) {
1763
+        if ( ! $this->is_edit_entry() ) {
1764 1764
             return $has_conditional_logic;
1765 1765
         }
1766 1766
 
@@ -1792,44 +1792,44 @@  discard block
 block discarded – undo
1792 1792
          *  2. There are two entries embedded using oEmbed
1793 1793
          *  3. One of the entries has just been saved
1794 1794
          */
1795
-        if( !empty( $_POST['lid'] ) && !empty( $_GET['entry'] ) && ( $_POST['lid'] !== $_GET['entry'] ) ) {
1795
+        if ( ! empty( $_POST[ 'lid' ] ) && ! empty( $_GET[ 'entry' ] ) && ( $_POST[ 'lid' ] !== $_GET[ 'entry' ] ) ) {
1796 1796
 
1797 1797
             $error = true;
1798 1798
 
1799 1799
         }
1800 1800
 
1801
-        if( !empty( $_GET['entry'] ) && (string)$this->entry['id'] !== $_GET['entry'] ) {
1801
+        if ( ! empty( $_GET[ 'entry' ] ) && (string)$this->entry[ 'id' ] !== $_GET[ 'entry' ] ) {
1802 1802
 
1803 1803
             $error = true;
1804 1804
 
1805
-        } elseif( ! $this->verify_nonce() ) {
1805
+        } elseif ( ! $this->verify_nonce() ) {
1806 1806
 
1807 1807
             /**
1808 1808
              * If the Entry is embedded, there may be two entries on the same page.
1809 1809
              * If that's the case, and one is being edited, the other should fail gracefully and not display an error.
1810 1810
              */
1811
-            if( GravityView_oEmbed::getInstance()->get_entry_id() ) {
1811
+            if ( GravityView_oEmbed::getInstance()->get_entry_id() ) {
1812 1812
                 $error = true;
1813 1813
             } else {
1814
-                $error = __( 'The link to edit this entry is not valid; it may have expired.', 'gravityview');
1814
+                $error = __( 'The link to edit this entry is not valid; it may have expired.', 'gravityview' );
1815 1815
             }
1816 1816
 
1817 1817
         }
1818 1818
 
1819
-        if( ! GravityView_Edit_Entry::check_user_cap_edit_entry( $this->entry ) ) {
1820
-            $error = __( 'You do not have permission to edit this entry.', 'gravityview');
1819
+        if ( ! GravityView_Edit_Entry::check_user_cap_edit_entry( $this->entry ) ) {
1820
+            $error = __( 'You do not have permission to edit this entry.', 'gravityview' );
1821 1821
         }
1822 1822
 
1823
-        if( $this->entry['status'] === 'trash' ) {
1824
-            $error = __('You cannot edit the entry; it is in the trash.', 'gravityview' );
1823
+        if ( $this->entry[ 'status' ] === 'trash' ) {
1824
+            $error = __( 'You cannot edit the entry; it is in the trash.', 'gravityview' );
1825 1825
         }
1826 1826
 
1827 1827
         // No errors; everything's fine here!
1828
-        if( empty( $error ) ) {
1828
+        if ( empty( $error ) ) {
1829 1829
             return true;
1830 1830
         }
1831 1831
 
1832
-        if( $echo && $error !== true ) {
1832
+        if ( $echo && $error !== true ) {
1833 1833
 
1834 1834
 	        $error = esc_html( $error );
1835 1835
 
@@ -1837,13 +1837,13 @@  discard block
 block discarded – undo
1837 1837
 	         * @since 1.9
1838 1838
 	         */
1839 1839
 	        if ( ! empty( $this->entry ) ) {
1840
-		        $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;" ) );
1840
+		        $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;" ) );
1841 1841
 	        }
1842 1842
 
1843
-            echo GVCommon::generate_notice( wpautop( $error ), 'gv-error error');
1843
+            echo GVCommon::generate_notice( wpautop( $error ), 'gv-error error' );
1844 1844
         }
1845 1845
 
1846
-        do_action('gravityview_log_error', 'GravityView_Edit_Entry[user_can_edit_entry]' . $error );
1846
+        do_action( 'gravityview_log_error', 'GravityView_Edit_Entry[user_can_edit_entry]' . $error );
1847 1847
 
1848 1848
         return false;
1849 1849
     }
@@ -1860,20 +1860,20 @@  discard block
 block discarded – undo
1860 1860
 
1861 1861
         $error = NULL;
1862 1862
 
1863
-        if( ! $this->check_user_cap_edit_field( $field ) ) {
1864
-            $error = __( 'You do not have permission to edit this field.', 'gravityview');
1863
+        if ( ! $this->check_user_cap_edit_field( $field ) ) {
1864
+            $error = __( 'You do not have permission to edit this field.', 'gravityview' );
1865 1865
         }
1866 1866
 
1867 1867
         // No errors; everything's fine here!
1868
-        if( empty( $error ) ) {
1868
+        if ( empty( $error ) ) {
1869 1869
             return true;
1870 1870
         }
1871 1871
 
1872
-        if( $echo ) {
1873
-            echo GVCommon::generate_notice( wpautop( esc_html( $error ) ), 'gv-error error');
1872
+        if ( $echo ) {
1873
+            echo GVCommon::generate_notice( wpautop( esc_html( $error ) ), 'gv-error error' );
1874 1874
         }
1875 1875
 
1876
-        do_action('gravityview_log_error', 'GravityView_Edit_Entry[user_can_edit_field]' . $error );
1876
+        do_action( 'gravityview_log_error', 'GravityView_Edit_Entry[user_can_edit_field]' . $error );
1877 1877
 
1878 1878
         return false;
1879 1879
 
@@ -1891,15 +1891,15 @@  discard block
 block discarded – undo
1891 1891
     private function check_user_cap_edit_field( $field ) {
1892 1892
 
1893 1893
         // If they can edit any entries (as defined in Gravity Forms), we're good.
1894
-        if( GVCommon::has_cap( array( 'gravityforms_edit_entries', 'gravityview_edit_others_entries' ) ) ) {
1894
+        if ( GVCommon::has_cap( array( 'gravityforms_edit_entries', 'gravityview_edit_others_entries' ) ) ) {
1895 1895
             return true;
1896 1896
         }
1897 1897
 
1898
-        $field_cap = isset( $field['allow_edit_cap'] ) ? $field['allow_edit_cap'] : false;
1898
+        $field_cap = isset( $field[ 'allow_edit_cap' ] ) ? $field[ 'allow_edit_cap' ] : false;
1899 1899
 
1900 1900
         // If the field has custom editing capaibilities set, check those
1901
-        if( $field_cap ) {
1902
-            return GVCommon::has_cap( $field['allow_edit_cap'] );
1901
+        if ( $field_cap ) {
1902
+            return GVCommon::has_cap( $field[ 'allow_edit_cap' ] );
1903 1903
         }
1904 1904
 
1905 1905
         return false;
@@ -1913,17 +1913,17 @@  discard block
 block discarded – undo
1913 1913
     public function verify_nonce() {
1914 1914
 
1915 1915
         // Verify form submitted for editing single
1916
-        if( $this->is_edit_entry_submission() ) {
1916
+        if ( $this->is_edit_entry_submission() ) {
1917 1917
             $valid = wp_verify_nonce( $_POST[ self::$nonce_field ], self::$nonce_field );
1918 1918
         }
1919 1919
 
1920 1920
         // Verify
1921
-        else if( ! $this->is_edit_entry() ) {
1921
+        else if ( ! $this->is_edit_entry() ) {
1922 1922
             $valid = false;
1923 1923
         }
1924 1924
 
1925 1925
         else {
1926
-            $valid = wp_verify_nonce( $_GET['edit'], self::$nonce_key );
1926
+            $valid = wp_verify_nonce( $_GET[ 'edit' ], self::$nonce_key );
1927 1927
         }
1928 1928
 
1929 1929
         /**
Please login to merge, or discard this patch.
includes/class-frontend-views.php 1 patch
Spacing   +113 added lines, -113 removed lines patch added patch discarded remove patch
@@ -85,9 +85,9 @@  discard block
 block discarded – undo
85 85
 	private function __construct() {}
86 86
 
87 87
 	private function initialize() {
88
-		add_action( 'wp', array( $this, 'parse_content'), 11 );
88
+		add_action( 'wp', array( $this, 'parse_content' ), 11 );
89 89
 		add_filter( 'parse_query', array( $this, 'parse_query_fix_frontpage' ), 10 );
90
-		add_action( 'template_redirect', array( $this, 'set_entry_data'), 1 );
90
+		add_action( 'template_redirect', array( $this, 'set_entry_data' ), 1 );
91 91
 
92 92
 		// Enqueue scripts and styles after GravityView_Template::register_styles()
93 93
 		add_action( 'wp_enqueue_scripts', array( $this, 'add_scripts_and_styles' ), 20 );
@@ -236,15 +236,15 @@  discard block
 block discarded – undo
236 236
 
237 237
 			$this->context_view_id = $view_id;
238 238
 
239
-		} elseif ( isset( $_GET['gvid'] ) && $this->getGvOutputData()->has_multiple_views() ) {
239
+		} elseif ( isset( $_GET[ 'gvid' ] ) && $this->getGvOutputData()->has_multiple_views() ) {
240 240
 			/**
241 241
 			 * used on a has_multiple_views context
242 242
 			 * @see GravityView_API::entry_link
243 243
 			 * @see GravityView_View_Data::getInstance()->has_multiple_views()
244 244
 			 */
245
-			$this->context_view_id = $_GET['gvid'];
245
+			$this->context_view_id = $_GET[ 'gvid' ];
246 246
 
247
-		} elseif ( ! $this->getGvOutputData()->has_multiple_views() )  {
247
+		} elseif ( ! $this->getGvOutputData()->has_multiple_views() ) {
248 248
 			$array_keys = array_keys( $this->getGvOutputData()->get_views() );
249 249
 			$this->context_view_id = array_pop( $array_keys );
250 250
 			unset( $array_keys );
@@ -279,24 +279,24 @@  discard block
 block discarded – undo
279 279
 		global $wp_rewrite;
280 280
 
281 281
 		$is_front_page = ( $query->is_home || $query->is_page );
282
-		$show_on_front = ( 'page' === get_option('show_on_front') );
283
-		$front_page_id = get_option('page_on_front');
282
+		$show_on_front = ( 'page' === get_option( 'show_on_front' ) );
283
+		$front_page_id = get_option( 'page_on_front' );
284 284
 
285
-		if (  $is_front_page && $show_on_front && $front_page_id ) {
285
+		if ( $is_front_page && $show_on_front && $front_page_id ) {
286 286
 
287 287
 			// Force to be an array, potentially a query string ( entry=16 )
288 288
 			$_query = wp_parse_args( $query->query );
289 289
 
290 290
 			// pagename can be set and empty depending on matched rewrite rules. Ignore an empty pagename.
291
-			if ( isset( $_query['pagename'] ) && '' === $_query['pagename'] ) {
292
-				unset( $_query['pagename'] );
291
+			if ( isset( $_query[ 'pagename' ] ) && '' === $_query[ 'pagename' ] ) {
292
+				unset( $_query[ 'pagename' ] );
293 293
 			}
294 294
 
295 295
 			// this is where will break from core wordpress
296 296
 			$ignore = array( 'preview', 'page', 'paged', 'cpage' );
297 297
 			$endpoints = rgobj( $wp_rewrite, 'endpoints' );
298
-			foreach ( (array) $endpoints as $endpoint ) {
299
-				$ignore[] = $endpoint[1];
298
+			foreach ( (array)$endpoints as $endpoint ) {
299
+				$ignore[ ] = $endpoint[ 1 ];
300 300
 			}
301 301
 			unset( $endpoints );
302 302
 
@@ -306,21 +306,21 @@  discard block
 block discarded – undo
306 306
 			// - The query includes keys that are associated with registered endpoints. `entry`, for example.
307 307
 			if ( empty( $_query ) || ! array_diff( array_keys( $_query ), $ignore ) ) {
308 308
 
309
-				$qv =& $query->query_vars;
309
+				$qv = & $query->query_vars;
310 310
 
311 311
 				// Prevent redirect when on the single entry endpoint
312
-				if( self::is_single_entry() ) {
312
+				if ( self::is_single_entry() ) {
313 313
 					add_filter( 'redirect_canonical', '__return_false' );
314 314
 				}
315 315
 
316 316
 				$query->is_page = true;
317 317
 				$query->is_home = false;
318
-				$qv['page_id']  = $front_page_id;
318
+				$qv[ 'page_id' ]  = $front_page_id;
319 319
 
320 320
 				// Correct <!--nextpage--> for page_on_front
321
-				if ( ! empty( $qv['paged'] ) ) {
322
-					$qv['page'] = $qv['paged'];
323
-					unset( $qv['paged'] );
321
+				if ( ! empty( $qv[ 'paged' ] ) ) {
322
+					$qv[ 'page' ] = $qv[ 'paged' ];
323
+					unset( $qv[ 'paged' ] );
324 324
 				}
325 325
 			}
326 326
 
@@ -353,7 +353,7 @@  discard block
 block discarded – undo
353 353
 
354 354
 		$this->setIsGravityviewPostType( get_post_type( $post ) === 'gravityview' );
355 355
 
356
-		$post_id = $this->getPostId() ? $this->getPostId() : (isset( $post ) ? $post->ID : null );
356
+		$post_id = $this->getPostId() ? $this->getPostId() : ( isset( $post ) ? $post->ID : null );
357 357
 		$this->setPostId( $post_id );
358 358
 		$post_has_shortcode = ! empty( $post->post_content ) ? gravityview_has_shortcode_r( $post->post_content, 'gravityview' ) : false;
359 359
 		$this->setPostHasShortcode( $this->isGravityviewPostType() ? null : ! empty( $post_has_shortcode ) );
@@ -389,7 +389,7 @@  discard block
 block discarded – undo
389 389
 
390 390
 		$search_method = GravityView_Widget_Search::getInstance()->get_search_method();
391 391
 
392
-		if( 'post' === $search_method ) {
392
+		if ( 'post' === $search_method ) {
393 393
 			$get = $_POST;
394 394
 		} else {
395 395
 			$get = $_GET;
@@ -445,20 +445,20 @@  discard block
 block discarded – undo
445 445
 		 * @param boolean $in_the_loop Whether to apply the filter to the menu title and the meta tag <title> - outside the loop
446 446
 		 * @param array $entry Current entry
447 447
 		 */
448
-		$apply_outside_loop = apply_filters( 'gravityview/single/title/out_loop' , in_the_loop(), $entry );
448
+		$apply_outside_loop = apply_filters( 'gravityview/single/title/out_loop', in_the_loop(), $entry );
449 449
 
450 450
 		if ( ! $apply_outside_loop ) {
451 451
 			return $title;
452 452
 		}
453 453
 
454 454
 		// User reported WooCommerce doesn't pass two args.
455
-		if ( empty( $passed_post_id ) )  {
455
+		if ( empty( $passed_post_id ) ) {
456 456
 			return $title;
457 457
 		}
458 458
 
459 459
 		// Don't modify the title for anything other than the current view/post.
460 460
 		// This is true for embedded shortcodes and Views.
461
-		if ( is_object( $post ) && (int) $post->ID !== (int) $passed_post_id ) {
461
+		if ( is_object( $post ) && (int)$post->ID !== (int)$passed_post_id ) {
462 462
 			return $title;
463 463
 		}
464 464
 
@@ -468,19 +468,19 @@  discard block
 block discarded – undo
468 468
 			$view_meta = $this->getGvOutputData()->get_view( $context_view_id );
469 469
 		} else {
470 470
 			foreach ( $this->getGvOutputData()->get_views() as $view_id => $view_data ) {
471
-				if ( intval( $view_data['form_id'] ) === intval( $entry['form_id'] ) ) {
471
+				if ( intval( $view_data[ 'form_id' ] ) === intval( $entry[ 'form_id' ] ) ) {
472 472
 					$view_meta = $view_data;
473 473
 					break;
474 474
 				}
475 475
 			}
476 476
 		}
477 477
 
478
-		if ( ! empty( $view_meta['atts']['single_title'] ) ) {
478
+		if ( ! empty( $view_meta[ 'atts' ][ 'single_title' ] ) ) {
479 479
 
480
-			$title = $view_meta['atts']['single_title'];
480
+			$title = $view_meta[ 'atts' ][ 'single_title' ];
481 481
 
482 482
 			// We are allowing HTML in the fields, so no escaping the output
483
-			$title = GravityView_API::replace_variables( $title, $view_meta['form'], $entry );
483
+			$title = GravityView_API::replace_variables( $title, $view_meta[ 'form' ], $entry );
484 484
 
485 485
 			$title = do_shortcode( $title );
486 486
 		}
@@ -516,7 +516,7 @@  discard block
 block discarded – undo
516 516
 		}
517 517
 
518 518
 		// Only render in the loop. Fixes issues with the_content filter being applied in places like the sidebar
519
-		if( ! in_the_loop() ) {
519
+		if ( ! in_the_loop() ) {
520 520
 			return $content;
521 521
 		}
522 522
 
@@ -584,7 +584,7 @@  discard block
 block discarded – undo
584 584
 
585 585
 		$context = GravityView_View::getInstance()->getContext();
586 586
 
587
-		switch( $context ) {
587
+		switch ( $context ) {
588 588
 			case 'directory':
589 589
 				$tab = __( 'Multiple Entries', 'gravityview' );
590 590
 				break;
@@ -598,9 +598,9 @@  discard block
 block discarded – undo
598 598
 		}
599 599
 
600 600
 
601
-		$title = sprintf( esc_html_x('The %s layout has not been configured.', 'Displayed when a View is not configured. %s is replaced by the tab label', 'gravityview' ), $tab );
601
+		$title = sprintf( esc_html_x( 'The %s layout has not been configured.', 'Displayed when a View is not configured. %s is replaced by the tab label', 'gravityview' ), $tab );
602 602
 		$edit_link = admin_url( sprintf( 'post.php?post=%d&action=edit#%s-view', $view_id, $context ) );
603
-		$action_text = sprintf( esc_html__('Add fields to %s', 'gravityview' ), $tab );
603
+		$action_text = sprintf( esc_html__( 'Add fields to %s', 'gravityview' ), $tab );
604 604
 		$message = esc_html__( 'You can only see this message because you are able to edit this View.', 'gravityview' );
605 605
 
606 606
 		$output = sprintf( '<h3>%s <strong><a href="%s">%s</a></strong></h3><p>%s</p>', $title, esc_url( $edit_link ), $action_text, $message );
@@ -633,7 +633,7 @@  discard block
 block discarded – undo
633 633
 	public function render_view( $passed_args ) {
634 634
 
635 635
 		// validate attributes
636
-		if ( empty( $passed_args['id'] ) ) {
636
+		if ( empty( $passed_args[ 'id' ] ) ) {
637 637
 			do_action( 'gravityview_log_error', '[render_view] Returning; no ID defined.', $passed_args );
638 638
 			return null;
639 639
 		}
@@ -655,7 +655,7 @@  discard block
 block discarded – undo
655 655
 			return null;
656 656
 		}
657 657
 
658
-		$view_id = $passed_args['id'];
658
+		$view_id = $passed_args[ 'id' ];
659 659
 
660 660
 		$view_data = $this->getGvOutputData()->get_view( $view_id, $passed_args );
661 661
 
@@ -668,7 +668,7 @@  discard block
 block discarded – undo
668 668
 		$passed_args = array_filter( $passed_args, 'strlen' );
669 669
 
670 670
 		//Override shortcode args over View template settings
671
-		$atts = wp_parse_args( $passed_args, $view_data['atts'] );
671
+		$atts = wp_parse_args( $passed_args, $view_data[ 'atts' ] );
672 672
 
673 673
 		do_action( 'gravityview_log_debug', '[render_view] Arguments after merging with View settings: ', $atts );
674 674
 
@@ -691,14 +691,14 @@  discard block
 block discarded – undo
691 691
 		 * @since 1.15
692 692
 		 * @since 1.17.2 Added check for if a user has no caps but is logged in (member of multisite, but not any site). Treat as if logged-out.
693 693
 		 */
694
-		if( is_user_logged_in() && ! ( empty( wp_get_current_user()->caps ) && empty( wp_get_current_user()->roles ) ) && false === GVCommon::has_cap( 'read_gravityview', $view_id ) ) {
694
+		if ( is_user_logged_in() && ! ( empty( wp_get_current_user()->caps ) && empty( wp_get_current_user()->roles ) ) && false === GVCommon::has_cap( 'read_gravityview', $view_id ) ) {
695 695
 
696 696
 			do_action( 'gravityview_log_debug', sprintf( '%s Returning: View %d is not visible by current user.', __METHOD__, $view_id ) );
697 697
 
698 698
 			return null;
699 699
 		}
700 700
 
701
-		if( $this->isGravityviewPostType() ) {
701
+		if ( $this->isGravityviewPostType() ) {
702 702
 
703 703
 			/**
704 704
 			 * @filter `gravityview_direct_access` Should Views be directly accessible, or only visible using the shortcode?
@@ -710,9 +710,9 @@  discard block
 block discarded – undo
710 710
 			 */
711 711
 			$direct_access = apply_filters( 'gravityview_direct_access', true, $view_id );
712 712
 
713
-			$embed_only = ! empty( $atts['embed_only'] );
713
+			$embed_only = ! empty( $atts[ 'embed_only' ] );
714 714
 
715
-			if( ! $direct_access || ( $embed_only && ! GVCommon::has_cap( 'read_private_gravityviews' ) ) ) {
715
+			if ( ! $direct_access || ( $embed_only && ! GVCommon::has_cap( 'read_private_gravityviews' ) ) ) {
716 716
 				return __( 'You are not allowed to view this content.', 'gravityview' );
717 717
 			}
718 718
 		}
@@ -727,7 +727,7 @@  discard block
 block discarded – undo
727 727
 
728 728
 		$gravityview_view = new GravityView_View( $view_data );
729 729
 
730
-		$post_id = ! empty( $atts['post_id'] ) ? intval( $atts['post_id'] ) : get_the_ID();
730
+		$post_id = ! empty( $atts[ 'post_id' ] ) ? intval( $atts[ 'post_id' ] ) : get_the_ID();
731 731
 
732 732
 		$gravityview_view->setPostId( $post_id );
733 733
 
@@ -737,20 +737,20 @@  discard block
 block discarded – undo
737 737
 			do_action( 'gravityview_log_debug', '[render_view] Executing Directory View' );
738 738
 
739 739
 			//fetch template and slug
740
-			$view_slug = apply_filters( 'gravityview_template_slug_'. $view_data['template_id'], 'table', 'directory' );
740
+			$view_slug = apply_filters( 'gravityview_template_slug_' . $view_data[ 'template_id' ], 'table', 'directory' );
741 741
 
742 742
 			do_action( 'gravityview_log_debug', '[render_view] View template slug: ', $view_slug );
743 743
 
744 744
 			/**
745 745
 			 * Disable fetching initial entries for views that don't need it (DataTables)
746 746
 			 */
747
-			$get_entries = apply_filters( 'gravityview_get_view_entries_'.$view_slug, true );
747
+			$get_entries = apply_filters( 'gravityview_get_view_entries_' . $view_slug, true );
748 748
 
749 749
 			/**
750 750
 			 * Hide View data until search is performed
751 751
 			 * @since 1.5.4
752 752
 			 */
753
-			if ( ! empty( $atts['hide_until_searched'] ) && ! $this->isSearch() ) {
753
+			if ( ! empty( $atts[ 'hide_until_searched' ] ) && ! $this->isSearch() ) {
754 754
 				$gravityview_view->setHideUntilSearched( true );
755 755
 				$get_entries = false;
756 756
 			}
@@ -758,23 +758,23 @@  discard block
 block discarded – undo
758 758
 
759 759
 			if ( $get_entries ) {
760 760
 
761
-				if ( ! empty( $atts['sort_columns'] ) ) {
761
+				if ( ! empty( $atts[ 'sort_columns' ] ) ) {
762 762
 					// add filter to enable column sorting
763
-					add_filter( 'gravityview/template/field_label', array( $this, 'add_columns_sort_links' ) , 100, 3 );
763
+					add_filter( 'gravityview/template/field_label', array( $this, 'add_columns_sort_links' ), 100, 3 );
764 764
 				}
765 765
 
766
-				$view_entries = self::get_view_entries( $atts, $view_data['form_id'] );
766
+				$view_entries = self::get_view_entries( $atts, $view_data[ 'form_id' ] );
767 767
 
768
-				do_action( 'gravityview_log_debug', sprintf( '[render_view] Get Entries. Found %s entries total, showing %d entries', $view_entries['count'], sizeof( $view_entries['entries'] ) ) );
768
+				do_action( 'gravityview_log_debug', sprintf( '[render_view] Get Entries. Found %s entries total, showing %d entries', $view_entries[ 'count' ], sizeof( $view_entries[ 'entries' ] ) ) );
769 769
 
770 770
 			} else {
771 771
 
772 772
 				$view_entries = array( 'count' => null, 'entries' => null, 'paging' => null );
773 773
 
774
-				do_action( 'gravityview_log_debug', '[render_view] Not fetching entries because `gravityview_get_view_entries_'.$view_slug.'` is false' );
774
+				do_action( 'gravityview_log_debug', '[render_view] Not fetching entries because `gravityview_get_view_entries_' . $view_slug . '` is false' );
775 775
 			}
776 776
 
777
-			$gravityview_view->setPaging( $view_entries['paging'] );
777
+			$gravityview_view->setPaging( $view_entries[ 'paging' ] );
778 778
 			$gravityview_view->setContext( 'directory' );
779 779
 			$sections = array( 'header', 'body', 'footer' );
780 780
 
@@ -788,7 +788,7 @@  discard block
 block discarded – undo
788 788
 			 * @action `gravityview_render_entry_{View ID}` Before rendering a single entry for a specific View ID
789 789
 			 * @since 1.17
790 790
 			 */
791
-			do_action( 'gravityview_render_entry_'.$view_data['id'] );
791
+			do_action( 'gravityview_render_entry_' . $view_data[ 'id' ] );
792 792
 
793 793
 			$entry = $this->getEntry();
794 794
 
@@ -798,7 +798,7 @@  discard block
 block discarded – undo
798 798
 				do_action( 'gravityview_log_debug', '[render_view] Entry does not exist. This may be because of View filters limiting access.' );
799 799
 
800 800
 				// Only display warning once when multiple Views are embedded
801
-				if( $view_id !== (int) GravityView_frontend::get_context_view_id() ) {
801
+				if ( $view_id !== (int)GravityView_frontend::get_context_view_id() ) {
802 802
 					ob_end_clean();
803 803
 					return null;
804 804
 				}
@@ -822,21 +822,21 @@  discard block
 block discarded – undo
822 822
 			// We're in single view, but the view being processed is not the same view the single entry belongs to.
823 823
 			// important: do not remove this as it prevents fake attempts of displaying entries from other views/forms
824 824
 			if ( $this->getGvOutputData()->has_multiple_views() && $view_id != $this->get_context_view_id() ) {
825
-				do_action( 'gravityview_log_debug', '[render_view] In single entry view, but the entry does not belong to this View. Perhaps there are multiple views on the page. View ID: '. $view_id );
825
+				do_action( 'gravityview_log_debug', '[render_view] In single entry view, but the entry does not belong to this View. Perhaps there are multiple views on the page. View ID: ' . $view_id );
826 826
 				ob_end_clean();
827 827
 				return null;
828 828
 			}
829 829
 
830 830
 			//fetch template and slug
831
-			$view_slug = apply_filters( 'gravityview_template_slug_' . $view_data['template_id'], 'table', 'single' );
831
+			$view_slug = apply_filters( 'gravityview_template_slug_' . $view_data[ 'template_id' ], 'table', 'single' );
832 832
 			do_action( 'gravityview_log_debug', '[render_view] View single template slug: ', $view_slug );
833 833
 
834 834
 			//fetch entry detail
835
-			$view_entries['count'] = 1;
836
-			$view_entries['entries'][] = $entry;
837
-			do_action( 'gravityview_log_debug', '[render_view] Get single entry: ', $view_entries['entries'] );
835
+			$view_entries[ 'count' ] = 1;
836
+			$view_entries[ 'entries' ][ ] = $entry;
837
+			do_action( 'gravityview_log_debug', '[render_view] Get single entry: ', $view_entries[ 'entries' ] );
838 838
 
839
-			$back_link_label = isset( $atts['back_link_label'] ) ? $atts['back_link_label'] : null;
839
+			$back_link_label = isset( $atts[ 'back_link_label' ] ) ? $atts[ 'back_link_label' ] : null;
840 840
 
841 841
 			// set back link label
842 842
 			$gravityview_view->setBackLinkLabel( $back_link_label );
@@ -846,11 +846,11 @@  discard block
 block discarded – undo
846 846
 		}
847 847
 
848 848
 		// add template style
849
-		self::add_style( $view_data['template_id'] );
849
+		self::add_style( $view_data[ 'template_id' ] );
850 850
 
851 851
 		// Prepare to render view and set vars
852
-		$gravityview_view->setEntries( $view_entries['entries'] );
853
-		$gravityview_view->setTotalEntries( $view_entries['count'] );
852
+		$gravityview_view->setEntries( $view_entries[ 'entries' ] );
853
+		$gravityview_view->setTotalEntries( $view_entries[ 'count' ] );
854 854
 
855 855
 		// If Edit
856 856
 		if ( 'edit' === gravityview_get_context() ) {
@@ -863,11 +863,11 @@  discard block
 block discarded – undo
863 863
 
864 864
 		} else {
865 865
 			// finaly we'll render some html
866
-			$sections = apply_filters( 'gravityview_render_view_sections', $sections, $view_data['template_id'] );
866
+			$sections = apply_filters( 'gravityview_render_view_sections', $sections, $view_data[ 'template_id' ] );
867 867
 
868 868
 			do_action( 'gravityview_log_debug', '[render_view] Sections to render: ', $sections );
869 869
 			foreach ( $sections as $section ) {
870
-				do_action( 'gravityview_log_debug', '[render_view] Rendering '. $section . ' section.' );
870
+				do_action( 'gravityview_log_debug', '[render_view] Rendering ' . $section . ' section.' );
871 871
 				$gravityview_view->render( $view_slug, $section, false );
872 872
 			}
873 873
 		}
@@ -921,7 +921,7 @@  discard block
 block discarded – undo
921 921
 				$datetime_format = 'Y-m-d H:i:s';
922 922
 				$search_is_outside_view_bounds = false;
923 923
 
924
-				if( ! empty( $search_criteria[ $key ] ) ) {
924
+				if ( ! empty( $search_criteria[ $key ] ) ) {
925 925
 
926 926
 					$search_date = strtotime( $search_criteria[ $key ] );
927 927
 
@@ -949,14 +949,14 @@  discard block
 block discarded – undo
949 949
 				if ( empty( $search_criteria[ $key ] ) || $search_is_outside_view_bounds ) {
950 950
 
951 951
 					// Then we override the search and re-set the start date
952
-					$return_search_criteria[ $key ] = date_i18n( $datetime_format , $date, true );
952
+					$return_search_criteria[ $key ] = date_i18n( $datetime_format, $date, true );
953 953
 				}
954 954
 			}
955 955
 		}
956 956
 
957
-		if( isset( $return_search_criteria['start_date'] ) && isset( $return_search_criteria['end_date'] ) ) {
957
+		if ( isset( $return_search_criteria[ 'start_date' ] ) && isset( $return_search_criteria[ 'end_date' ] ) ) {
958 958
 			// The start date is AFTER the end date. This will result in no results, but let's not force the issue.
959
-			if ( strtotime( $return_search_criteria['start_date'] ) > strtotime( $return_search_criteria['end_date'] ) ) {
959
+			if ( strtotime( $return_search_criteria[ 'start_date' ] ) > strtotime( $return_search_criteria[ 'end_date' ] ) ) {
960 960
 				do_action( 'gravityview_log_error', __METHOD__ . ' Invalid search: the start date is after the end date.', $return_search_criteria );
961 961
 			}
962 962
 		}
@@ -975,19 +975,19 @@  discard block
 block discarded – undo
975 975
 	public static function process_search_only_approved( $args, $search_criteria ) {
976 976
 
977 977
 		/** @since 1.19 */
978
-		if( ! empty( $args['admin_show_all_statuses'] ) && GVCommon::has_cap('gravityview_moderate_entries') ) {
978
+		if ( ! empty( $args[ 'admin_show_all_statuses' ] ) && GVCommon::has_cap( 'gravityview_moderate_entries' ) ) {
979 979
 			do_action( 'gravityview_log_debug', __METHOD__ . ': User can moderate entries; showing all approval statuses' );
980 980
 			return $search_criteria;
981 981
 		}
982 982
 
983
-		if ( ! empty( $args['show_only_approved'] ) ) {
983
+		if ( ! empty( $args[ 'show_only_approved' ] ) ) {
984 984
 
985
-			$search_criteria['field_filters'][] = array(
985
+			$search_criteria[ 'field_filters' ][ ] = array(
986 986
 				'key' => GravityView_Entry_Approval::meta_key,
987 987
 				'value' => GravityView_Entry_Approval_Status::APPROVED
988 988
 			);
989 989
 
990
-			$search_criteria['field_filters']['mode'] = 'all'; // force all the criterias to be met
990
+			$search_criteria[ 'field_filters' ][ 'mode' ] = 'all'; // force all the criterias to be met
991 991
 
992 992
 			do_action( 'gravityview_log_debug', '[process_search_only_approved] Search Criteria if show only approved: ', $search_criteria );
993 993
 		}
@@ -1014,18 +1014,18 @@  discard block
 block discarded – undo
1014 1014
 	 */
1015 1015
 	public static function is_entry_approved( $entry, $args = array() ) {
1016 1016
 
1017
-		if ( empty( $entry['id'] ) || ( array_key_exists( 'show_only_approved', $args ) && ! $args['show_only_approved'] ) ) {
1017
+		if ( empty( $entry[ 'id' ] ) || ( array_key_exists( 'show_only_approved', $args ) && ! $args[ 'show_only_approved' ] ) ) {
1018 1018
 			// is implicitly approved if entry is null or View settings doesn't require to check for approval
1019 1019
 			return true;
1020 1020
 		}
1021 1021
 
1022 1022
 		/** @since 1.19 */
1023
-		if( ! empty( $args['admin_show_all_statuses'] ) && GVCommon::has_cap('gravityview_moderate_entries') ) {
1023
+		if ( ! empty( $args[ 'admin_show_all_statuses' ] ) && GVCommon::has_cap( 'gravityview_moderate_entries' ) ) {
1024 1024
 			do_action( 'gravityview_log_debug', __METHOD__ . ': User can moderate entries, so entry is approved for viewing' );
1025 1025
 			return true;
1026 1026
 		}
1027 1027
 
1028
-		$is_approved = gform_get_meta( $entry['id'], GravityView_Entry_Approval::meta_key );
1028
+		$is_approved = gform_get_meta( $entry[ 'id' ], GravityView_Entry_Approval::meta_key );
1029 1029
 
1030 1030
 		return GravityView_Entry_Approval_Status::is_approved( $is_approved );
1031 1031
 	}
@@ -1059,26 +1059,26 @@  discard block
 block discarded – undo
1059 1059
 		do_action( 'gravityview_log_debug', '[get_search_criteria] Search Criteria after hook gravityview_fe_search_criteria: ', $search_criteria );
1060 1060
 
1061 1061
 		// implicity search
1062
-		if ( ! empty( $args['search_value'] ) ) {
1062
+		if ( ! empty( $args[ 'search_value' ] ) ) {
1063 1063
 
1064 1064
 			// Search operator options. Options: `is` or `contains`
1065
-			$operator = ! empty( $args['search_operator'] ) && in_array( $args['search_operator'], array( 'is', 'isnot', '>', '<', 'contains' ) ) ? $args['search_operator'] : 'contains';
1065
+			$operator = ! empty( $args[ 'search_operator' ] ) && in_array( $args[ 'search_operator' ], array( 'is', 'isnot', '>', '<', 'contains' ) ) ? $args[ 'search_operator' ] : 'contains';
1066 1066
 
1067
-			$search_criteria['field_filters'][] = array(
1067
+			$search_criteria[ 'field_filters' ][ ] = array(
1068 1068
 				'key' => rgget( 'search_field', $args ), // The field ID to search
1069
-				'value' => _wp_specialchars( $args['search_value'] ), // The value to search. Encode ampersands but not quotes.
1069
+				'value' => _wp_specialchars( $args[ 'search_value' ] ), // The value to search. Encode ampersands but not quotes.
1070 1070
 				'operator' => $operator,
1071 1071
 			);
1072 1072
 		}
1073 1073
 
1074
-		if( $search_criteria !== $original_search_criteria ) {
1074
+		if ( $search_criteria !== $original_search_criteria ) {
1075 1075
 			do_action( 'gravityview_log_debug', '[get_search_criteria] Search Criteria after implicity search: ', $search_criteria );
1076 1076
 		}
1077 1077
 
1078 1078
 		// Handle setting date range
1079 1079
 		$search_criteria = self::process_search_dates( $args, $search_criteria );
1080 1080
 
1081
-		if( $search_criteria !== $original_search_criteria ) {
1081
+		if ( $search_criteria !== $original_search_criteria ) {
1082 1082
 			do_action( 'gravityview_log_debug', '[get_search_criteria] Search Criteria after date params: ', $search_criteria );
1083 1083
 		}
1084 1084
 
@@ -1089,7 +1089,7 @@  discard block
 block discarded – undo
1089 1089
 		 * @filter `gravityview_status` Modify entry status requirements to be included in search results.
1090 1090
 		 * @param string $status Default: `active`. Accepts all Gravity Forms entry statuses, including `spam` and `trash`
1091 1091
 		 */
1092
-		$search_criteria['status'] = apply_filters( 'gravityview_status', 'active', $args );
1092
+		$search_criteria[ 'status' ] = apply_filters( 'gravityview_status', 'active', $args );
1093 1093
 
1094 1094
 		return $search_criteria;
1095 1095
 	}
@@ -1201,7 +1201,7 @@  discard block
 block discarded – undo
1201 1201
 			'search_criteria' => $search_criteria,
1202 1202
 			'sorting' => self::updateViewSorting( $args, $form_id ),
1203 1203
 			'paging' => $paging,
1204
-			'cache' => isset( $args['cache'] ) ? $args['cache'] : true,
1204
+			'cache' => isset( $args[ 'cache' ] ) ? $args[ 'cache' ] : true,
1205 1205
 		);
1206 1206
 
1207 1207
 		/**
@@ -1226,7 +1226,7 @@  discard block
 block discarded – undo
1226 1226
 		 * @param array $parameters Array with `search_criteria`, `sorting` and `paging` keys.
1227 1227
 		 * @param array $args View configuration args.
1228 1228
 		 */
1229
-		$parameters = apply_filters( 'gravityview_get_entries_'.$args['id'], $parameters, $args, $form_id );
1229
+		$parameters = apply_filters( 'gravityview_get_entries_' . $args[ 'id' ], $parameters, $args, $form_id );
1230 1230
 
1231 1231
 		do_action( 'gravityview_log_debug', __METHOD__ . ': $parameters passed to gravityview_get_entries(): ', $parameters );
1232 1232
 
@@ -1251,16 +1251,16 @@  discard block
 block discarded – undo
1251 1251
 		$default_page_size = apply_filters( 'gravityview_default_page_size', 25 );
1252 1252
 
1253 1253
 		// Paging & offset
1254
-		$page_size = ! empty( $args['page_size'] ) ? intval( $args['page_size'] ) : $default_page_size;
1254
+		$page_size = ! empty( $args[ 'page_size' ] ) ? intval( $args[ 'page_size' ] ) : $default_page_size;
1255 1255
 
1256 1256
 		if ( -1 === $page_size ) {
1257 1257
 			$page_size = PHP_INT_MAX;
1258 1258
 		}
1259 1259
 
1260
-		if ( isset( $args['offset'] ) ) {
1261
-			$offset = intval( $args['offset'] );
1260
+		if ( isset( $args[ 'offset' ] ) ) {
1261
+			$offset = intval( $args[ 'offset' ] );
1262 1262
 		} else {
1263
-			$curr_page = empty( $_GET['pagenum'] ) ? 1 : intval( $_GET['pagenum'] );
1263
+			$curr_page = empty( $_GET[ 'pagenum' ] ) ? 1 : intval( $_GET[ 'pagenum' ] );
1264 1264
 			$offset = ( $curr_page - 1 ) * $page_size;
1265 1265
 		}
1266 1266
 
@@ -1285,8 +1285,8 @@  discard block
 block discarded – undo
1285 1285
 	 */
1286 1286
 	public static function updateViewSorting( $args, $form_id ) {
1287 1287
 		$sorting = array();
1288
-		$sort_field_id = isset( $_GET['sort'] ) ? $_GET['sort'] : rgar( $args, 'sort_field' );
1289
-		$sort_direction = isset( $_GET['dir'] ) ? $_GET['dir'] : rgar( $args, 'sort_direction' );
1288
+		$sort_field_id = isset( $_GET[ 'sort' ] ) ? $_GET[ 'sort' ] : rgar( $args, 'sort_field' );
1289
+		$sort_direction = isset( $_GET[ 'dir' ] ) ? $_GET[ 'dir' ] : rgar( $args, 'sort_direction' );
1290 1290
 
1291 1291
 		$sort_field_id = self::_override_sorting_id_by_field_type( $sort_field_id, $form_id );
1292 1292
 
@@ -1326,11 +1326,11 @@  discard block
 block discarded – undo
1326 1326
 
1327 1327
 		$sort_field = GFFormsModel::get_field( $form, $sort_field_id );
1328 1328
 
1329
-		if( ! $sort_field ) {
1329
+		if ( ! $sort_field ) {
1330 1330
 			return $sort_field_id;
1331 1331
 		}
1332 1332
 
1333
-		switch ( $sort_field['type'] ) {
1333
+		switch ( $sort_field[ 'type' ] ) {
1334 1334
 
1335 1335
 			case 'address':
1336 1336
 				// Sorting by full address
@@ -1347,7 +1347,7 @@  discard block
 block discarded – undo
1347 1347
 					 */
1348 1348
 					$address_part = apply_filters( 'gravityview/sorting/address', 'city', $sort_field_id, $form_id );
1349 1349
 
1350
-					switch( strtolower( $address_part ) ){
1350
+					switch ( strtolower( $address_part ) ) {
1351 1351
 						case 'street':
1352 1352
 							$sort_field_id .= '.1';
1353 1353
 							break;
@@ -1432,7 +1432,7 @@  discard block
 block discarded – undo
1432 1432
 		 */
1433 1433
 		$single_entry = apply_filters( 'gravityview/is_single_entry', $single_entry );
1434 1434
 
1435
-		if ( empty( $single_entry ) ){
1435
+		if ( empty( $single_entry ) ) {
1436 1436
 			return false;
1437 1437
 		} else {
1438 1438
 			return $single_entry;
@@ -1459,7 +1459,7 @@  discard block
 block discarded – undo
1459 1459
 				 * Don't enqueue the scripts or styles if it's not going to be displayed.
1460 1460
 				 * @since 1.15
1461 1461
 				 */
1462
-				if( is_user_logged_in() && false === GVCommon::has_cap( 'read_gravityview', $view_id ) ) {
1462
+				if ( is_user_logged_in() && false === GVCommon::has_cap( 'read_gravityview', $view_id ) ) {
1463 1463
 					continue;
1464 1464
 				}
1465 1465
 
@@ -1468,19 +1468,19 @@  discard block
 block discarded – undo
1468 1468
 				$css_dependencies = array();
1469 1469
 
1470 1470
 				// If the thickbox is enqueued, add dependencies
1471
-				if ( ! empty( $data['atts']['lightbox'] ) ) {
1471
+				if ( ! empty( $data[ 'atts' ][ 'lightbox' ] ) ) {
1472 1472
 
1473 1473
 					/**
1474 1474
 					 * @filter `gravity_view_lightbox_script` Override the lightbox script to enqueue. Default: `thickbox`
1475 1475
 					 * @param string $script_slug If you want to use a different lightbox script, return the name of it here.
1476 1476
 					 */
1477
-					$js_dependencies[] = apply_filters( 'gravity_view_lightbox_script', 'thickbox' );
1477
+					$js_dependencies[ ] = apply_filters( 'gravity_view_lightbox_script', 'thickbox' );
1478 1478
 
1479 1479
 					/**
1480 1480
 					 * @filter `gravity_view_lightbox_style` Modify the lightbox CSS slug. Default: `thickbox`
1481 1481
 					 * @param string $script_slug If you want to use a different lightbox script, return the name of its CSS file here.
1482 1482
 					 */
1483
-					$css_dependencies[] = apply_filters( 'gravity_view_lightbox_style', 'thickbox' );
1483
+					$css_dependencies[ ] = apply_filters( 'gravity_view_lightbox_style', 'thickbox' );
1484 1484
 				}
1485 1485
 
1486 1486
 				/**
@@ -1488,25 +1488,25 @@  discard block
 block discarded – undo
1488 1488
 				 * @see https://github.com/katzwebservices/GravityView/issues/536
1489 1489
 				 * @since 1.15
1490 1490
 				 */
1491
-				if( gravityview_view_has_single_checkbox_or_radio( $data['form'], $data['fields'] ) ) {
1492
-					$css_dependencies[] = 'dashicons';
1491
+				if ( gravityview_view_has_single_checkbox_or_radio( $data[ 'form' ], $data[ 'fields' ] ) ) {
1492
+					$css_dependencies[ ] = 'dashicons';
1493 1493
 				}
1494 1494
 
1495 1495
 				wp_register_script( 'gravityview-jquery-cookie', plugins_url( 'assets/lib/jquery.cookie/jquery.cookie.min.js', GRAVITYVIEW_FILE ), array( 'jquery' ), GravityView_Plugin::version, true );
1496 1496
 
1497 1497
 				$script_debug = ( defined( 'SCRIPT_DEBUG' ) && SCRIPT_DEBUG ) ? '' : '.min';
1498 1498
 
1499
-				wp_register_script( 'gravityview-fe-view', plugins_url( 'assets/js/fe-views' . $script_debug . '.js', GRAVITYVIEW_FILE ), apply_filters( 'gravityview_js_dependencies', $js_dependencies ) , GravityView_Plugin::version, true );
1499
+				wp_register_script( 'gravityview-fe-view', plugins_url( 'assets/js/fe-views' . $script_debug . '.js', GRAVITYVIEW_FILE ), apply_filters( 'gravityview_js_dependencies', $js_dependencies ), GravityView_Plugin::version, true );
1500 1500
 
1501 1501
 				wp_enqueue_script( 'gravityview-fe-view' );
1502 1502
 
1503
-				if ( ! empty( $data['atts']['sort_columns'] ) ) {
1503
+				if ( ! empty( $data[ 'atts' ][ 'sort_columns' ] ) ) {
1504 1504
 					wp_enqueue_style( 'gravityview_font', plugins_url( 'assets/css/font.css', GRAVITYVIEW_FILE ), $css_dependencies, GravityView_Plugin::version, 'all' );
1505 1505
 				}
1506 1506
 
1507 1507
 				$this->enqueue_default_style( $css_dependencies );
1508 1508
 
1509
-				self::add_style( $data['template_id'] );
1509
+				self::add_style( $data[ 'template_id' ] );
1510 1510
 			}
1511 1511
 
1512 1512
 			if ( 'wp_print_footer_scripts' === current_filter() ) {
@@ -1568,7 +1568,7 @@  discard block
 block discarded – undo
1568 1568
 		} elseif ( empty( $template_id ) ) {
1569 1569
 			do_action( 'gravityview_log_error', '[add_style] Cannot add template style; template_id is empty' );
1570 1570
 		} else {
1571
-			do_action( 'gravityview_log_error', sprintf( '[add_style] Cannot add template style; %s is not registered', 'gravityview_style_'.$template_id ) );
1571
+			do_action( 'gravityview_log_error', sprintf( '[add_style] Cannot add template style; %s is not registered', 'gravityview_style_' . $template_id ) );
1572 1572
 		}
1573 1573
 
1574 1574
 	}
@@ -1593,11 +1593,11 @@  discard block
 block discarded – undo
1593 1593
 		 * Not a table-based template; don't add sort icons
1594 1594
 		 * @since 1.12
1595 1595
 		 */
1596
-		if( ! preg_match( '/table/ism', GravityView_View::getInstance()->getTemplatePartSlug() ) ) {
1596
+		if ( ! preg_match( '/table/ism', GravityView_View::getInstance()->getTemplatePartSlug() ) ) {
1597 1597
 			return $label;
1598 1598
 		}
1599 1599
 
1600
-		if ( ! $this->is_field_sortable( $field['id'], $form ) ) {
1600
+		if ( ! $this->is_field_sortable( $field[ 'id' ], $form ) ) {
1601 1601
 			return $label;
1602 1602
 		}
1603 1603
 
@@ -1605,29 +1605,29 @@  discard block
 block discarded – undo
1605 1605
 
1606 1606
 		$class = 'gv-sort';
1607 1607
 
1608
-		$sort_field_id = self::_override_sorting_id_by_field_type( $field['id'], $form['id'] );
1608
+		$sort_field_id = self::_override_sorting_id_by_field_type( $field[ 'id' ], $form[ 'id' ] );
1609 1609
 
1610 1610
 		$sort_args = array(
1611
-			'sort' => $field['id'],
1611
+			'sort' => $field[ 'id' ],
1612 1612
 			'dir' => 'asc',
1613 1613
 		);
1614 1614
 
1615
-		if ( ! empty( $sorting['key'] ) && (string) $sort_field_id === (string) $sorting['key'] ) {
1615
+		if ( ! empty( $sorting[ 'key' ] ) && (string)$sort_field_id === (string)$sorting[ 'key' ] ) {
1616 1616
 			//toggle sorting direction.
1617
-			if ( 'asc' === $sorting['direction'] ) {
1618
-				$sort_args['dir'] = 'desc';
1617
+			if ( 'asc' === $sorting[ 'direction' ] ) {
1618
+				$sort_args[ 'dir' ] = 'desc';
1619 1619
 				$class .= ' gv-icon-sort-desc';
1620 1620
 			} else {
1621
-				$sort_args['dir'] = 'asc';
1621
+				$sort_args[ 'dir' ] = 'asc';
1622 1622
 				$class .= ' gv-icon-sort-asc';
1623 1623
 			}
1624 1624
 		} else {
1625 1625
 			$class .= ' gv-icon-caret-up-down';
1626 1626
 		}
1627 1627
 
1628
-		$url = add_query_arg( $sort_args, remove_query_arg( array('pagenum') ) );
1628
+		$url = add_query_arg( $sort_args, remove_query_arg( array( 'pagenum' ) ) );
1629 1629
 
1630
-		return '<a href="'. esc_url_raw( $url ) .'" class="'. $class .'" ></a>&nbsp;'. $label;
1630
+		return '<a href="' . esc_url_raw( $url ) . '" class="' . $class . '" ></a>&nbsp;' . $label;
1631 1631
 
1632 1632
 	}
1633 1633
 
@@ -1645,7 +1645,7 @@  discard block
 block discarded – undo
1645 1645
 
1646 1646
 		$field_type = $field_id;
1647 1647
 
1648
-		if( is_numeric( $field_id ) ) {
1648
+		if ( is_numeric( $field_id ) ) {
1649 1649
 			$field = GFFormsModel::get_field( $form, $field_id );
1650 1650
 			$field_type = $field->type;
1651 1651
 		}
@@ -1668,7 +1668,7 @@  discard block
 block discarded – undo
1668 1668
 			return false;
1669 1669
 		}
1670 1670
 
1671
-		return apply_filters( "gravityview/sortable/formfield_{$form['id']}_{$field_id}", apply_filters( "gravityview/sortable/field_{$field_id}", true, $form ) );
1671
+		return apply_filters( "gravityview/sortable/formfield_{$form[ 'id' ]}_{$field_id}", apply_filters( "gravityview/sortable/field_{$field_id}", true, $form ) );
1672 1672
 
1673 1673
 	}
1674 1674
 
Please login to merge, or discard this patch.