Completed
Push — master ( 1b32e3...e32e6e )
by Zack
06:12 queued 14s
created

fix_survey_fields_value()   C

Complexity

Conditions 8
Paths 5

Size

Total Lines 29
Code Lines 14

Duplication

Lines 0
Ratio 0 %

Importance

Changes 3
Bugs 3 Features 0
Metric Value
c 3
b 3
f 0
dl 0
loc 29
rs 5.3846
cc 8
eloc 14
nc 5
nop 3
1
<?php
0 ignored issues
show
Coding Style Compatibility introduced by
For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 17 and the first side effect is on line 13.

The PSR-1: Basic Coding Standard recommends that a file should either introduce new symbols, that is classes, functions, constants or similar, or have side effects. Side effects are anything that executes logic, like for example printing output, changing ini settings or writing to a file.

The idea behind this recommendation is that merely auto-loading a class should not change the state of an application. It also promotes a cleaner style of programming and makes your code less prone to errors, because the logic is not spread out all over the place.

To learn more about the PSR-1, please see the PHP-FIG site on the PSR-1.

Loading history...
2
/**
3
 * GravityView Edit Entry - render frontend
4
 *
5
 * @package   GravityView
6
 * @license   GPL2+
7
 * @author    Katz Web Services, Inc.
8
 * @link      http://gravityview.co
9
 * @copyright Copyright 2014, Katz Web Services, Inc.
10
 */
11
12
if ( ! defined( 'WPINC' ) ) {
13
    die;
14
}
15
16
17
class GravityView_Edit_Entry_Render {
18
19
    /**
20
     * @var GravityView_Edit_Entry
21
     */
22
    protected $loader;
23
24
	/**
25
	 * @var string String used to generate unique nonce for the entry/form/view combination. Allows access to edit page.
26
	 */
27
    static $nonce_key;
0 ignored issues
show
Coding Style introduced by
The visibility should be declared for property $nonce_key.

The PSR-2 coding standard requires that all properties in a class have their visibility explicitly declared. If you declare a property using

class A {
    var $property;
}

the property is implicitly global.

To learn more about the PSR-2, please see the PHP-FIG site on the PSR-2.

Loading history...
28
29
	/**
30
	 * @since 1.9
31
	 * @var string String used for check valid edit entry form submission. Allows saving edit form values.
32
	 */
33
	private static $nonce_field = 'is_gv_edit_entry';
34
35
	/**
36
	 * @since 1.9
37
	 * @var bool Whether to allow save and continue functionality
38
	 */
39
	private static $supports_save_and_continue = false;
40
41
	/**
42
	 * @since 1.9
43
	 * @var bool Whether to allow editing product fields
44
	 */
45
	private static $supports_product_fields = false;
46
47
    /**
48
     * Gravity Forms entry array
49
     *
50
     * @var array
51
     */
52
    var $entry;
0 ignored issues
show
Coding Style introduced by
The visibility should be declared for property $entry.

The PSR-2 coding standard requires that all properties in a class have their visibility explicitly declared. If you declare a property using

class A {
    var $property;
}

the property is implicitly global.

To learn more about the PSR-2, please see the PHP-FIG site on the PSR-2.

Loading history...
53
54
    /**
55
     * Gravity Forms form array (GravityView modifies the content through this class lifecycle)
56
     *
57
     * @var array
58
     */
59
    var $form;
0 ignored issues
show
Coding Style introduced by
The visibility should be declared for property $form.

The PSR-2 coding standard requires that all properties in a class have their visibility explicitly declared. If you declare a property using

class A {
    var $property;
}

the property is implicitly global.

To learn more about the PSR-2, please see the PHP-FIG site on the PSR-2.

Loading history...
60
61
    /**
62
     * Gravity Forms form array (it won't get changed during this class lifecycle)
63
     * @since 1.16.2.1
64
     * @var array
65
     */
66
    var $original_form;
0 ignored issues
show
Coding Style introduced by
The visibility should be declared for property $original_form.

The PSR-2 coding standard requires that all properties in a class have their visibility explicitly declared. If you declare a property using

class A {
    var $property;
}

the property is implicitly global.

To learn more about the PSR-2, please see the PHP-FIG site on the PSR-2.

Loading history...
67
68
    /**
69
     * Gravity Forms form array after the form validation process
70
     * @since 1.13
71
     * @var array
72
     */
73
    var $form_after_validation = null;
0 ignored issues
show
Coding Style introduced by
The visibility should be declared for property $form_after_validation.

The PSR-2 coding standard requires that all properties in a class have their visibility explicitly declared. If you declare a property using

class A {
    var $property;
}

the property is implicitly global.

To learn more about the PSR-2, please see the PHP-FIG site on the PSR-2.

Loading history...
74
75
    /**
76
     * Hold an array of GF field objects that have calculation rules
77
     * @var array
78
     */
79
    var $fields_with_calculation = array();
0 ignored issues
show
Coding Style introduced by
The visibility should be declared for property $fields_with_calculation.

The PSR-2 coding standard requires that all properties in a class have their visibility explicitly declared. If you declare a property using

class A {
    var $property;
}

the property is implicitly global.

To learn more about the PSR-2, please see the PHP-FIG site on the PSR-2.

Loading history...
80
81
    /**
82
     * Hold an array of GF field objects with type 'total'
83
     * @var array
84
     */
85
    var $total_fields = array();
0 ignored issues
show
Coding Style introduced by
The visibility should be declared for property $total_fields.

The PSR-2 coding standard requires that all properties in a class have their visibility explicitly declared. If you declare a property using

class A {
    var $property;
}

the property is implicitly global.

To learn more about the PSR-2, please see the PHP-FIG site on the PSR-2.

Loading history...
86
87
    /**
88
     * Gravity Forms form id
89
     *
90
     * @var int
91
     */
92
    var $form_id;
0 ignored issues
show
Coding Style introduced by
The visibility should be declared for property $form_id.

The PSR-2 coding standard requires that all properties in a class have their visibility explicitly declared. If you declare a property using

class A {
    var $property;
}

the property is implicitly global.

To learn more about the PSR-2, please see the PHP-FIG site on the PSR-2.

Loading history...
93
94
    /**
95
     * ID of the current view
96
     *
97
     * @var int
98
     */
99
    var $view_id;
0 ignored issues
show
Coding Style introduced by
The visibility should be declared for property $view_id.

The PSR-2 coding standard requires that all properties in a class have their visibility explicitly declared. If you declare a property using

class A {
    var $property;
}

the property is implicitly global.

To learn more about the PSR-2, please see the PHP-FIG site on the PSR-2.

Loading history...
100
101
    /**
102
     * Updated entry is valid (GF Validation object)
103
     *
104
     * @var array
105
     */
106
    var $is_valid = NULL;
0 ignored issues
show
Coding Style introduced by
The visibility should be declared for property $is_valid.

The PSR-2 coding standard requires that all properties in a class have their visibility explicitly declared. If you declare a property using

class A {
    var $property;
}

the property is implicitly global.

To learn more about the PSR-2, please see the PHP-FIG site on the PSR-2.

Loading history...
Coding Style introduced by
TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.
Loading history...
107
108
    function __construct( GravityView_Edit_Entry $loader ) {
0 ignored issues
show
Best Practice introduced by
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
109
        $this->loader = $loader;
110
    }
111
112
    function load() {
0 ignored issues
show
Best Practice introduced by
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
113
114
        /** @define "GRAVITYVIEW_DIR" "../../../" */
115
        include_once( GRAVITYVIEW_DIR .'includes/class-admin-approve-entries.php' );
116
117
        // Don't display an embedded form when editing an entry
118
        add_action( 'wp_head', array( $this, 'prevent_render_form' ) );
119
        add_action( 'wp_footer', array( $this, 'prevent_render_form' ) );
120
121
        // Stop Gravity Forms processing what is ours!
122
        add_filter( 'wp', array( $this, 'prevent_maybe_process_form'), 8 );
0 ignored issues
show
introduced by
No space before closing parenthesis of array is bad style
Loading history...
123
124
        add_filter( 'gravityview_is_edit_entry', array( $this, 'is_edit_entry') );
0 ignored issues
show
introduced by
No space before closing parenthesis of array is bad style
Loading history...
125
126
        add_action( 'gravityview_edit_entry', array( $this, 'init' ) );
127
128
        // Disable conditional logic if needed (since 1.9)
129
        add_filter( 'gform_has_conditional_logic', array( $this, 'manage_conditional_logic' ), 10, 2 );
130
131
        // Make sure GF doesn't validate max files (since 1.9)
132
        add_filter( 'gform_plupload_settings', array( $this, 'modify_fileupload_settings' ), 10, 3 );
133
134
        // Add fields expected by GFFormDisplay::validate()
135
        add_filter( 'gform_pre_validation', array( $this, 'gform_pre_validation') );
0 ignored issues
show
introduced by
No space before closing parenthesis of array is bad style
Loading history...
136
137
    }
138
139
    /**
140
     * Don't show any forms embedded on a page when GravityView is in Edit Entry mode
141
     *
142
     * Adds a `__return_empty_string` filter on the Gravity Forms shortcode on the `wp_head` action
143
     * And then removes it on the `wp_footer` action
144
     *
145
     * @since 1.16.1
146
     *
147
     * @return void
148
     */
149
    function prevent_render_form() {
0 ignored issues
show
Best Practice introduced by
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
150
        if( $this->is_edit_entry() ) {
151
            if( 'wp_head' === current_filter() ) {
152
                add_filter( 'gform_shortcode_form', '__return_empty_string' );
153
            } else {
154
                remove_filter( 'gform_shortcode_form', '__return_empty_string' );
155
            }
156
        }
157
    }
158
159
    /**
160
     * Because we're mimicking being a front-end Gravity Forms form while using a Gravity Forms
161
     * backend form, we need to prevent them from saving twice.
162
     * @return void
163
     */
164
    function prevent_maybe_process_form() {
0 ignored issues
show
Best Practice introduced by
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
165
166
        do_action('gravityview_log_debug', 'GravityView_Edit_Entry[prevent_maybe_process_form] $_POSTed data (sanitized): ', esc_html( print_r( $_POST, true ) ) );
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
introduced by
The use of function print_r() is discouraged
Loading history...
introduced by
Detected access of super global var $_POST, probably need manual inspection.
Loading history...
167
168
        if( $this->is_edit_entry_submission() && $this->verify_nonce() ) {
169
            remove_action( 'wp',  array( 'RGForms', 'maybe_process_form'), 9 );
0 ignored issues
show
introduced by
No space before closing parenthesis of array is bad style
Loading history...
170
        }
171
    }
172
173
    /**
174
     * Is the current page an Edit Entry page?
175
     * @return boolean
176
     */
177
    public function is_edit_entry() {
178
179
        $gf_page = class_exists('rgpost') && ( 'entry' === rgget( 'view' ) && isset( $_GET['edit'] ) || rgpost( 'action' ) === 'update' );
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
Coding Style introduced by
Expected 1 spaces before closing bracket; 0 found
Loading history...
introduced by
Detected access of super global var $_GET, probably need manual inspection.
Loading history...
180
        
181
        return $gf_page;
182
    }
183
184
	/**
185
	 * Is the current page an Edit Entry page?
186
	 * @since 1.9
187
	 * @return boolean
188
	 */
189
	public function is_edit_entry_submission() {
190
		return !empty( $_POST[ self::$nonce_field ] );
0 ignored issues
show
introduced by
Expected 1 space after "!"; 0 found
Loading history...
introduced by
Detected access of super global var $_POST, probably need manual inspection.
Loading history...
191
	}
192
193
    /**
194
     * When Edit entry view is requested setup the vars
195
     */
196
    function setup_vars() {
0 ignored issues
show
Best Practice introduced by
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
197
        $gravityview_view = GravityView_View::getInstance();
198
0 ignored issues
show
Coding Style introduced by
Functions must not contain multiple empty lines in a row; found 2 empty lines
Loading history...
199
200
        $entries = $gravityview_view->getEntries();
201
        $this->entry = $entries[0];
202
203
        $this->original_form = $this->form = $gravityview_view->getForm();
204
        $this->form_id = $gravityview_view->getFormId();
205
        $this->view_id = $gravityview_view->getViewId();
206
207
        self::$nonce_key = GravityView_Edit_Entry::get_nonce_key( $this->view_id, $this->form_id, $this->entry['id'] );
208
    }
209
210
211
    /**
212
     * Load required files and trigger edit flow
213
     *
214
     * Run when the is_edit_entry returns true.
215
     *
216
     * @param GravityView_View_Data $gv_data GravityView Data object
217
     * @return void
218
     */
219
    function init( $gv_data ) {
0 ignored issues
show
Best Practice introduced by
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
220
221
        require_once( GFCommon::get_base_path() . '/form_display.php' );
222
        require_once( GFCommon::get_base_path() . '/entry_detail.php' );
223
224
        $this->setup_vars();
225
226
        // Multiple Views embedded, don't proceed if nonce fails
227
        if( $gv_data->has_multiple_views() && ! wp_verify_nonce( $_GET['edit'], self::$nonce_key ) ) {
0 ignored issues
show
introduced by
Detected access of super global var $_GET, probably need manual inspection.
Loading history...
introduced by
Detected usage of a non-validated input variable: $_GET
Loading history...
introduced by
Detected usage of a non-sanitized input variable: $_GET
Loading history...
228
            return;
229
        }
230
231
        // Sorry, you're not allowed here.
232
        if( false === $this->user_can_edit_entry( true ) ) {
233
            return;
234
        }
235
236
        $this->print_scripts();
237
238
        $this->process_save();
239
240
        $this->edit_entry_form();
241
242
    }
243
244
245
    /**
246
     * Force Gravity Forms to output scripts as if it were in the admin
247
     * @return void
248
     */
249
    function print_scripts() {
0 ignored issues
show
Best Practice introduced by
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
250
        $gravityview_view = GravityView_View::getInstance();
251
252
        wp_register_script( 'gform_gravityforms', GFCommon::get_base_url().'/js/gravityforms.js', array( 'jquery', 'gform_json', 'gform_placeholder', 'sack', 'plupload-all', 'gravityview-fe-view' ) );
253
254
        GFFormDisplay::enqueue_form_scripts($gravityview_view->getForm(), false);
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
Coding Style introduced by
Expected 1 spaces before closing bracket; 0 found
Loading history...
255
256
        // Sack is required for images
257
        wp_print_scripts( array( 'sack', 'gform_gravityforms' ) );
258
    }
259
260
261
    /**
262
     * Process edit entry form save
263
     */
264
    function process_save() {
0 ignored issues
show
Best Practice introduced by
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
265
266
        if( empty( $_POST ) ) {
0 ignored issues
show
introduced by
Detected access of super global var $_POST, probably need manual inspection.
Loading history...
267
            return;
268
        }
269
270
        // Make sure the entry, view, and form IDs are all correct
271
        $valid = $this->verify_nonce();
272
273
        if( !$valid ) {
0 ignored issues
show
introduced by
Expected 1 space after "!"; 0 found
Loading history...
274
            do_action('gravityview_log_error', __METHOD__ . ' Nonce validation failed.' );
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
275
            return;
276
        }
277
278
        if( $this->entry['id'] !== $_POST['lid'] ) {
0 ignored issues
show
introduced by
Detected access of super global var $_POST, probably need manual inspection.
Loading history...
introduced by
Detected usage of a non-validated input variable: $_POST
Loading history...
introduced by
Detected usage of a non-sanitized input variable: $_POST
Loading history...
279
            do_action('gravityview_log_error', __METHOD__ . ' Entry ID did not match posted entry ID.' );
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
280
            return;
281
        }
282
283
        do_action('gravityview_log_debug', 'GravityView_Edit_Entry[process_save] $_POSTed data (sanitized): ', esc_html( print_r( $_POST, true ) ) );
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
introduced by
The use of function print_r() is discouraged
Loading history...
introduced by
Detected access of super global var $_POST, probably need manual inspection.
Loading history...
284
285
        $this->process_save_process_files( $this->form_id );
286
287
        $this->validate();
288
289
        if( $this->is_valid ) {
0 ignored issues
show
Bug Best Practice introduced by
The expression $this->is_valid of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent.

Consider making the comparison explicit by using empty(..) or ! empty(...) instead.

Loading history...
290
291
            do_action('gravityview_log_debug', 'GravityView_Edit_Entry[process_save] Submission is valid.' );
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
292
293
            /**
294
             * @hack This step is needed to unset the adminOnly from form fields, to add the calculation fields
295
             */
296
            $form = $this->form_prepare_for_save();
297
298
            /**
299
             * @hack to avoid the capability validation of the method save_lead for GF 1.9+
300
             */
301
            unset( $_GET['page'] );
0 ignored issues
show
introduced by
Detected access of super global var $_GET, probably need manual inspection.
Loading history...
302
303
            GFFormsModel::save_lead( $form, $this->entry );
304
305
            // If there's a post associated with the entry, process post fields
306
            if( !empty( $this->entry['post_id'] ) ) {
0 ignored issues
show
introduced by
Expected 1 space after "!"; 0 found
Loading history...
307
                $this->maybe_update_post_fields( $form );
308
            }
309
310
            // Process calculation fields
311
            $this->update_calculation_fields();
312
313
            // Perform actions normally performed after updating a lead
314
            $this->after_update();
315
316
            /**
317
             * @action `gravityview/edit_entry/after_update` Perform an action after the entry has been updated using Edit Entry
318
             * @param array $form Gravity Forms form array
319
             * @param string $entry_id Numeric ID of the entry that was updated
320
             */
321
            do_action( 'gravityview/edit_entry/after_update', $this->form, $this->entry['id'] );
322
        }
323
324
    } // process_save
325
326
327
    /**
328
     * Have GF handle file uploads
329
     *
330
     * Copy of code from GFFormDisplay::process_form()
331
     *
332
     * @param int $form_id
333
     */
334
    function process_save_process_files( $form_id ) {
0 ignored issues
show
Best Practice introduced by
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
335
336
        //Loading files that have been uploaded to temp folder
337
        $files = GFCommon::json_decode( stripslashes( RGForms::post( 'gform_uploaded_files' ) ) );
338
        if ( ! is_array( $files ) ) {
339
            $files = array();
340
        }
341
342
        RGFormsModel::$uploaded_files[ $form_id ] = $files;
343
    }
344
345
    /**
346
     * Remove max_files validation (done on gravityforms.js) to avoid conflicts with GravityView
347
     * Late validation done on self::custom_validation
348
     *
349
     * @param $plupload_init array Plupload settings
350
     * @param $form_id
351
     * @param $instance
352
     * @return mixed
353
     */
354
    public function modify_fileupload_settings( $plupload_init, $form_id, $instance ) {
0 ignored issues
show
Unused Code introduced by
The parameter $form_id is not used and could be removed.

This check looks from parameters that have been defined for a function or method, but which are not used in the method body.

Loading history...
Unused Code introduced by
The parameter $instance is not used and could be removed.

This check looks from parameters that have been defined for a function or method, but which are not used in the method body.

Loading history...
355
        if( ! $this->is_edit_entry() ) {
356
            return $plupload_init;
357
        }
358
359
        $plupload_init['gf_vars']['max_files'] = 0;
360
361
        return $plupload_init;
362
    }
363
364
365
    /**
366
     * Unset adminOnly and convert field input key to string
367
     * @return array $form
368
     */
369
    private function form_prepare_for_save() {
370
371
        $form = $this->form;
372
373
        foreach( $form['fields'] as $k => &$field ) {
374
375
            // Remove the fields with calculation formulas before save to avoid conflicts with GF logic
376
            // @since 1.16.3
377
            if( $field->has_calculation() ) {
378
                unset( $form['fields'][ $k ] );
379
            }
380
381
            $field->adminOnly = false;
382
383
            if( isset( $field->inputs ) && is_array( $field->inputs ) ) {
384
                foreach( $field->inputs as $key => $input ) {
385
                    $field->inputs[ $key ][ 'id' ] = (string)$input['id'];
0 ignored issues
show
introduced by
Array keys should NOT be surrounded by spaces if they only contain a string or an integer.
Loading history...
introduced by
No space after closing casting parenthesis is prohibited
Loading history...
386
                }
387
            }
388
        }
389
390
        return $form;
391
    }
392
393
    private function update_calculation_fields() {
394
395
        $form = $this->original_form;
396
        $update = false;
397
398
        // get the most up to date entry values
399
        $entry = GFAPI::get_entry( $this->entry['id'] );
400
401
        if( !empty( $this->fields_with_calculation ) ) {
0 ignored issues
show
introduced by
Expected 1 space after "!"; 0 found
Loading history...
402
            $update = true;
403
            foreach ( $this->fields_with_calculation as $calc_field ) {
404
                $inputs = $calc_field->get_entry_inputs();
405
                if ( is_array( $inputs ) ) {
406
                    foreach ( $inputs as $input ) {
407
                        $input_name = 'input_' . str_replace( '.', '_', $input['id'] );
408
                        $entry[ strval( $input['id'] ) ] = RGFormsModel::prepare_value( $form, $calc_field, '', $input_name, $entry['id'], $entry );
409
                    }
410
                } else {
411
                    $input_name = 'input_' . str_replace( '.', '_', $calc_field->id);
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces before closing bracket; 0 found
Loading history...
412
                    $entry[ strval( $calc_field->id ) ] = RGFormsModel::prepare_value( $form, $calc_field, '', $input_name, $entry['id'], $entry );
413
                }
414
            }
1 ignored issue
show
introduced by
Blank line found after control structure
Loading history...
415
416
        }
417
418
        //saving total field as the last field of the form.
419
        if ( ! empty( $this->total_fields ) ) {
420
            $update = true;
421
            foreach ( $this->total_fields as $total_field ) {
422
                $input_name = 'input_' . str_replace( '.', '_', $total_field->id);
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces before closing bracket; 0 found
Loading history...
423
                $entry[ strval( $total_field->id ) ] = RGFormsModel::prepare_value( $form, $total_field, '', $input_name, $entry['id'], $entry );
424
            }
425
        }
426
427
        if( $update ) {
428
429
            $return_entry = GFAPI::update_entry( $entry );
430
431
            if( is_wp_error( $return_entry ) ) {
432
                do_action( 'gravityview_log_error', 'Updating the entry calculation and total fields failed', $return_entry );
433
            } else {
434
                do_action( 'gravityview_log_debug', 'Updating the entry calculation and total fields succeeded' );
435
            }
436
        }
437
    }
438
439
440
    /**
441
     * Loop through the fields being edited and if they include Post fields, update the Entry's post object
442
     *
443
     * @param array $form Gravity Forms form
444
     *
445
     * @return void
446
     */
447
    function maybe_update_post_fields( $form ) {
0 ignored issues
show
Best Practice introduced by
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
448
449
        $post_id = $this->entry['post_id'];
450
451
        // Security check
452
        if( false === GVCommon::has_cap( 'edit_post', $post_id ) ) {
453
            do_action( 'gravityview_log_error', 'The current user does not have the ability to edit Post #'.$post_id );
454
            return;
455
        }
456
457
        $update_entry = false;
458
459
        $updated_post = $original_post = get_post( $post_id );
460
461
        // get the most up to date entry values
462
        $entry = GFAPI::get_entry( $this->entry['id'] );
463
464
        foreach ( $entry as $field_id => $value ) {
465
466
            //todo: only run through the edit entry configured fields
467
468
            $field = RGFormsModel::get_field( $form, $field_id );
469
470
            if( class_exists('GF_Fields') ) {
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
Coding Style introduced by
Expected 1 spaces before closing bracket; 0 found
Loading history...
471
                $field = GF_Fields::create( $field );
472
            }
473
474
            if( GFCommon::is_post_field( $field ) ) {
475
476
                // Get the value of the field, including $_POSTed value
477
                $value = RGFormsModel::get_field_value( $field );
478
479
                switch( $field->type ) {
480
481
                    case 'post_title':
482
                    case 'post_content':
483
                    case 'post_excerpt':
484
                        $updated_post->{$field->type} = $value;
485
                        break;
486
                    case 'post_tags':
487
                        wp_set_post_tags( $post_id, $value, false );
488
                        break;
489
                    case 'post_category':
490
491
                        $categories = is_array( $value ) ? array_values( $value ) : (array)$value;
0 ignored issues
show
introduced by
No space after closing casting parenthesis is prohibited
Loading history...
492
                        $categories = array_filter( $categories );
493
494
                        wp_set_post_categories( $post_id, $categories, false );
495
496
                        // if post_category is type checkbox, then value is an array of inputs
497
                        if( isset( $value[ strval( $field_id ) ] ) ) {
498
                            foreach( $value as $input_id => $val ) {
499
                                $input_name = 'input_' . str_replace( '.', '_', $input_id );
500
                                $entry[ strval( $input_id ) ] = RGFormsModel::prepare_value( $form, $field, $val, $input_name, $entry['id'], $entry );
501
                            }
502
                        } else {
503
                            $input_name = 'input_' . str_replace( '.', '_', $field_id );
504
                            $entry[ strval( $field_id ) ] = RGFormsModel::prepare_value( $form, $field, $value, $input_name, $entry['id'], $entry );
505
                        }
506
507
                        break;
508
                    case 'post_custom_field':
509
510
                        $input_type = RGFormsModel::get_input_type( $field );
511
                        $custom_field_name = $field->postCustomFieldName;
512
513
                        // Only certain custom field types are supported
514
                        switch( $input_type ) {
515
                            case 'fileupload':
516
                            /** @noinspection PhpMissingBreakStatementInspection */
0 ignored issues
show
Coding Style introduced by
Line indented incorrectly; expected at least 8 tabs, found 7
Loading history...
517
                            case 'list':
518
                                if( ! is_string( $value ) ) {
519
                                    $value = function_exists('wp_json_encode') ? wp_json_encode( $value ) : json_encode( $value );
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
Coding Style introduced by
Expected 1 spaces before closing bracket; 0 found
Loading history...
520
                                }
521
                            // break; left intentionally out
1 ignored issue
show
Coding Style introduced by
Line indented incorrectly; expected at least 8 tabs, found 7
Loading history...
522
                            default:
523
                                update_post_meta( $post_id, $custom_field_name, $value );
524
                        }
525
526
                        break;
527
528
                    case 'post_image':
529
530
                        $input_name = 'input_' . $field_id;
531
532
                        if ( !empty( $_FILES[ $input_name ]['name'] ) ) {
0 ignored issues
show
introduced by
Expected 1 space after "!"; 0 found
Loading history...
533
534
                            // We have a new image
535
536
                            $value = RGFormsModel::prepare_value( $form, $field, $value, $input_name, $entry['id'] );
537
538
                            // is this field set as featured image, if not, leave
539
                            if ( ! $field->postFeaturedImage ) {
540
                                break;
541
                            }
542
543
                            $ary = ! empty( $value ) ? explode( '|:|', $value ) : array();
544
                            $img_url = rgar( $ary, 0 );
545
546
                            $img_title       = count( $ary ) > 1 ? $ary[1] : '';
547
                            $img_caption     = count( $ary ) > 2 ? $ary[2] : '';
548
                            $img_description = count( $ary ) > 3 ? $ary[3] : '';
549
550
                            $image_meta = array(
551
                                'post_excerpt' => $img_caption,
552
                                'post_content' => $img_description,
553
                            );
554
555
                            //adding title only if it is not empty. It will default to the file name if it is not in the array
556
                            if ( ! empty( $img_title ) ) {
557
                                $image_meta['post_title'] = $img_title;
558
                            }
559
560
                            //todo: As soon as \GFFormsModel::media_handle_upload becomes a public method, move this call to \GFFormsModel::media_handle_upload and remove the hack from this class.
561
                            require_once GRAVITYVIEW_DIR . 'includes/class-gravityview-gfformsmodel.php';
562
                            $media_id = GravityView_GFFormsModel::media_handle_upload( $img_url, $post_id, $image_meta );
563
564
                            if ( $media_id ) {
565
                                set_post_thumbnail( $post_id, $media_id );
566
                            }
567
568
                            break;
569
570
                        } elseif ( !empty( $_POST[ $input_name ] ) && is_array( $value ) ) {
0 ignored issues
show
introduced by
Expected 1 space after "!"; 0 found
Loading history...
571
572
                            // Same image although the image title, caption or description might have changed
573
574
                            $ary = ! empty( $entry[ $field_id ] ) ? explode( '|:|', $entry[ $field_id ] ) : array();
575
                            $img_url = rgar( $ary, 0 );
576
577
                            // is this really the same image or something went wrong ?
578
                            if( $img_url === $_POST[ $input_name ] ) {
579
580
                                $img_title       = isset( $value[ $field_id .'.1' ] ) ? $value[ $field_id .'.1' ] : '';
581
                                $img_caption     = isset( $value[ $field_id .'.4' ] ) ? $value[ $field_id .'.4' ] : '';
582
                                $img_description = isset( $value[ $field_id .'.7' ] ) ? $value[ $field_id .'.7' ] : '';
583
584
                                $value = ! empty( $img_url ) ? $img_url . "|:|" . $img_title . "|:|" . $img_caption . "|:|" . $img_description : '';
0 ignored issues
show
Coding Style Comprehensibility introduced by
The string literal |:| does not require double quotes, as per coding-style, please use single quotes.

PHP provides two ways to mark string literals. Either with single quotes 'literal' or with double quotes "literal". The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences.

String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (\') and the backslash (\\). Every other character is displayed as is.

Double quoted string literals may contain other variables or more complex escape sequences.

<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;

will print an indented: Single is Value

If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear.

For more information on PHP string literals and available escape sequences see the PHP core documentation.

Loading history...
585
586
                                if ( $field->postFeaturedImage ) {
587
588
                                    $image_meta = array(
589
                                        'ID' => get_post_thumbnail_id( $post_id ),
590
                                        'post_title' => $img_title,
591
                                        'post_excerpt' => $img_caption,
592
                                        'post_content' => $img_description,
593
                                    );
594
595
                                    // update image title, caption or description
596
                                    wp_update_post( $image_meta );
597
                                }
598
599
                                break;
600
                            }
1 ignored issue
show
introduced by
Blank line found after control structure
Loading history...
601
602
                        }
603
604
                        // if we get here, image was removed or not set.
605
606
                        $value = '';
607
                        if ( $field->postFeaturedImage ) {
608
                            delete_post_thumbnail( $post_id );
609
                        }
610
611
                        break;
612
613
                }
614
615
                //ignore fields that have not changed
616
                if ( $value === rgget( (string) $field_id, $entry ) ) {
617
                    continue;
618
                }
619
620
                // update entry
621
                if( 'post_category' !== $field->type ) {
622
                    $entry[ strval( $field_id ) ] = $value;
623
                }
624
625
                $update_entry = true;
626
627
            }
1 ignored issue
show
introduced by
Blank line found after control structure
Loading history...
628
629
        }
630
631
        if( $update_entry ) {
632
633
            $return_entry = GFAPI::update_entry( $entry );
634
635
            if( is_wp_error( $return_entry ) ) {
636
                do_action( 'gravityview_log_error', 'Updating the entry post fields failed', $return_entry );
637
            } else {
638
                do_action( 'gravityview_log_debug', 'Updating the entry post fields for post #'.$post_id.' succeeded' );
639
            }
1 ignored issue
show
introduced by
Blank line found after control structure
Loading history...
640
641
        }
642
643
        $return_post = wp_update_post( $updated_post, true );
644
645
        if( is_wp_error( $return_post ) ) {
646
            $return_post->add_data( $updated_post, '$updated_post' );
647
            do_action( 'gravityview_log_error', 'Updating the post content failed', $return_post );
648
        } else {
649
            do_action( 'gravityview_log_debug', 'Updating the post content for post #'.$post_id.' succeeded', $updated_post );
650
        }
651
652
    }
653
654
655
    /**
656
     * Perform actions normally performed after updating a lead
657
     *
658
     * @since 1.8
659
     *
660
     * @see GFEntryDetail::lead_detail_page()
661
     *
662
     * @return void
663
     */
664
    function after_update() {
0 ignored issues
show
Best Practice introduced by
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
665
666
        do_action( 'gform_after_update_entry', $this->form, $this->entry['id'] );
667
        do_action( "gform_after_update_entry_{$this->form['id']}", $this->form, $this->entry['id'] );
668
669
        // Re-define the entry now that we've updated it.
670
        $entry = RGFormsModel::get_lead( $this->entry['id'] );
671
672
        $entry = GFFormsModel::set_entry_meta( $entry, $this->form );
673
674
        // We need to clear the cache because Gravity Forms caches the field values, which
675
        // we have just updated.
676
        foreach ($this->form['fields'] as $key => $field) {
0 ignored issues
show
introduced by
No space after opening parenthesis is prohibited
Loading history...
introduced by
No space before closing parenthesis is prohibited
Loading history...
677
            GFFormsModel::refresh_lead_field_value( $entry['id'], $field->id );
678
        }
679
680
        $this->entry = $entry;
681
    }
682
683
684
    /**
685
     * Display the Edit Entry form
686
     *
687
     * @return [type] [description]
0 ignored issues
show
Documentation introduced by
The doc-type [type] could not be parsed: Unknown type name "" at position 0. [(view supported doc-types)

This check marks PHPDoc comments that could not be parsed by our parser. To see which comment annotations we can parse, please refer to our documentation on supported doc-types.

Loading history...
688
     */
689
    public function edit_entry_form() {
690
691
        ?>
692
693
        <div class="gv-edit-entry-wrapper"><?php
694
695
            $javascript = gravityview_ob_include( GravityView_Edit_Entry::$file .'/partials/inline-javascript.php', $this );
0 ignored issues
show
Bug introduced by
The property file cannot be accessed from this context as it is declared private in class GravityView_Edit_Entry.

This check looks for access to properties that are not accessible from the current context.

If you need to make a property accessible to another context you can either raise its visibility level or provide an accessible getter in the defining class.

Loading history...
696
697
            /**
698
             * Fixes weird wpautop() issue
699
             * @see https://github.com/katzwebservices/GravityView/issues/451
700
             */
701
            echo gravityview_strip_whitespace( $javascript );
0 ignored issues
show
introduced by
Expected a sanitizing function (see Codex for 'Data Validation'), but instead saw 'gravityview_strip_whitespace'
Loading history...
702
703
            ?><h2 class="gv-edit-entry-title">
704
                <span><?php
705
706
                    /**
707
                     * @filter `gravityview_edit_entry_title` Modify the edit entry title
708
                     * @param string $edit_entry_title Modify the "Edit Entry" title
709
                     * @param GravityView_Edit_Entry_Render $this This object
710
                     */
711
                    $edit_entry_title = apply_filters('gravityview_edit_entry_title', __('Edit Entry', 'gravityview'), $this );
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
Coding Style introduced by
Expected 1 spaces before closing bracket; 0 found
Loading history...
712
713
                    echo esc_attr( $edit_entry_title );
714
            ?></span>
715
            </h2>
716
717
            <?php $this->maybe_print_message(); ?>
718
719
            <?php // The ID of the form needs to be `gform_{form_id}` for the pluploader ?>
720
721
            <form method="post" id="gform_<?php echo $this->form_id; ?>" enctype="multipart/form-data">
0 ignored issues
show
introduced by
Expected next thing to be a escaping function, not '$this'
Loading history...
722
723
                <?php
724
725
                wp_nonce_field( self::$nonce_key, self::$nonce_key );
726
727
                wp_nonce_field( self::$nonce_field, self::$nonce_field, false );
728
729
                // Print the actual form HTML
730
                $this->render_edit_form();
731
732
                ?>
733
            </form>
734
735
        </div>
736
737
    <?php
738
    }
739
740
    /**
741
     * Display success or error message if the form has been submitted
742
     *
743
     * @uses GVCommon::generate_notice
744
     *
745
     * @since 1.16.2.2
746
     *
747
     * @return void
748
     */
749
    private function maybe_print_message() {
750
751
        if( rgpost('action') === 'update' ) {
0 ignored issues
show
introduced by
Found "=== '". Use Yoda Condition checks, you must
Loading history...
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
Coding Style introduced by
Expected 1 spaces before closing bracket; 0 found
Loading history...
752
753
            $back_link = esc_url( remove_query_arg( array( 'page', 'view', 'edit' ) ) );
754
755
            if( ! $this->is_valid ){
0 ignored issues
show
Bug Best Practice introduced by
The expression $this->is_valid of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent.

Consider making the comparison explicit by using empty(..) or ! empty(...) instead.

Loading history...
756
757
                // Keeping this compatible with Gravity Forms.
758
                $validation_message = "<div class='validation_error'>" . __('There was a problem with your submission.', 'gravityview') . " " . __('Errors have been highlighted below.', 'gravityview') . "</div>";
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
Coding Style introduced by
Expected 1 spaces before closing bracket; 0 found
Loading history...
Coding Style Comprehensibility introduced by
The string literal does not require double quotes, as per coding-style, please use single quotes.

PHP provides two ways to mark string literals. Either with single quotes 'literal' or with double quotes "literal". The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences.

String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (\') and the backslash (\\). Every other character is displayed as is.

Double quoted string literals may contain other variables or more complex escape sequences.

<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;

will print an indented: Single is Value

If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear.

For more information on PHP string literals and available escape sequences see the PHP core documentation.

Loading history...
introduced by
Expected a sanitizing function (see Codex for 'Data Validation'), but instead saw '__'
Loading history...
Coding Style Comprehensibility introduced by
The string literal </div> does not require double quotes, as per coding-style, please use single quotes.

PHP provides two ways to mark string literals. Either with single quotes 'literal' or with double quotes "literal". The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences.

String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (\') and the backslash (\\). Every other character is displayed as is.

Double quoted string literals may contain other variables or more complex escape sequences.

<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;

will print an indented: Single is Value

If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear.

For more information on PHP string literals and available escape sequences see the PHP core documentation.

Loading history...
759
                $message = apply_filters("gform_validation_message_{$this->form['id']}", apply_filters("gform_validation_message", $validation_message, $this->form), $this->form);
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
Coding Style introduced by
Expected 1 spaces before closing bracket; 0 found
Loading history...
Coding Style Comprehensibility introduced by
The string literal gform_validation_message does not require double quotes, as per coding-style, please use single quotes.

PHP provides two ways to mark string literals. Either with single quotes 'literal' or with double quotes "literal". The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences.

String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (\') and the backslash (\\). Every other character is displayed as is.

Double quoted string literals may contain other variables or more complex escape sequences.

<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;

will print an indented: Single is Value

If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear.

For more information on PHP string literals and available escape sequences see the PHP core documentation.

Loading history...
760
761
                echo GVCommon::generate_notice( $message , 'gv-error' );
0 ignored issues
show
introduced by
Expected a sanitizing function (see Codex for 'Data Validation'), but instead saw 'GVCommon'
Loading history...
762
763
            } else {
764
                $entry_updated_message = sprintf( esc_attr__('Entry Updated. %sReturn to Entry%s', 'gravityview'), '<a href="'. $back_link .'">', '</a>' );
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
Coding Style introduced by
Expected 1 spaces before closing bracket; 0 found
Loading history...
765
766
                /**
767
                 * @filter `gravityview/edit_entry/success` Modify the edit entry success message (including the anchor link)
768
                 * @since 1.5.4
769
                 * @param string $entry_updated_message Existing message
770
                 * @param int $view_id View ID
771
                 * @param array $entry Gravity Forms entry array
772
                 * @param string $back_link URL to return to the original entry. @since 1.6
773
                 */
774
                $message = apply_filters( 'gravityview/edit_entry/success', $entry_updated_message , $this->view_id, $this->entry, $back_link );
775
776
                echo GVCommon::generate_notice( $message );
0 ignored issues
show
introduced by
Expected a sanitizing function (see Codex for 'Data Validation'), but instead saw 'GVCommon'
Loading history...
777
            }
1 ignored issue
show
introduced by
Blank line found after control structure
Loading history...
778
779
        }
780
    }
781
782
    /**
783
     * Display the Edit Entry form in the original Gravity Forms format
784
     *
785
     * @since 1.9
786
     *
787
     * @return void
788
     */
789
    private function render_edit_form() {
790
791
        add_filter( 'gform_pre_render', array( $this, 'filter_modify_form_fields'), 5000, 3 );
0 ignored issues
show
introduced by
No space before closing parenthesis of array is bad style
Loading history...
792
        add_filter( 'gform_submit_button', array( $this, 'render_form_buttons') );
0 ignored issues
show
introduced by
No space before closing parenthesis of array is bad style
Loading history...
793
        add_filter( 'gform_disable_view_counter', '__return_true' );
794
795
        add_filter( 'gform_field_input', array( $this, 'verify_user_can_edit_post' ), 5, 5 );
796
        add_filter( 'gform_field_input', array( $this, 'modify_edit_field_input' ), 10, 5 );
797
798
        add_filter( 'gform_field_value', array( $this, 'fix_survey_fields_value'), 10, 3 );
0 ignored issues
show
introduced by
No space before closing parenthesis of array is bad style
Loading history...
799
800
        // We need to remove the fake $_GET['page'] arg to avoid rendering form as if in admin.
801
        unset( $_GET['page'] );
0 ignored issues
show
introduced by
Detected access of super global var $_GET, probably need manual inspection.
Loading history...
802
803
        // TODO: Verify multiple-page forms
804
        // TODO: Product fields are not editable
805
806
        $html = GFFormDisplay::get_form( $this->form['id'], false, false, true, $this->entry );
807
808
        remove_filter( 'gform_field_value', array( $this, 'fix_survey_fields_value'), 10 );
0 ignored issues
show
introduced by
No space before closing parenthesis of array is bad style
Loading history...
809
	    remove_filter( 'gform_pre_render', array( $this, 'filter_modify_form_fields' ), 5000 );
810
        remove_filter( 'gform_submit_button', array( $this, 'render_form_buttons' ) );
811
        remove_filter( 'gform_disable_view_counter', '__return_true' );
812
        remove_filter( 'gform_field_input', array( $this, 'verify_user_can_edit_post' ), 5 );
813
        remove_filter( 'gform_field_input', array( $this, 'modify_edit_field_input' ), 10 );
814
815
        echo $html;
0 ignored issues
show
introduced by
Expected next thing to be a escaping function, not '$html'
Loading history...
816
    }
817
818
    /**
819
     * Survey fields inject their output using `gform_field_input` filter, but in Edit Entry, the values were empty.
820
     * We filter the values here because it was the easiest access point: tell the survey field the correct value, GF outputs it.
821
     *
822
     * @since 1.16.4
823
     *
824
     * @param string $value Existing value
825
     * @param GF_Field $field
826
     * @param string $name Field custom parameter name, normally blank.
827
     *
828
     * @return mixed
829
     */
830
    function fix_survey_fields_value( $value, $field, $name ) {
0 ignored issues
show
Best Practice introduced by
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
831
        
832
        if( 'survey' === $field->type && '' === $value && 'likert' === rgar( $field, 'inputType' ) ) {
833
834
	        // We need to run through each survey row until we find a match for expected values
835
	        foreach ( $this->entry as $field_id => $field_value ) {
836
837
		        if ( floor( $field_id ) !== floor( $field->id ) ) {
838
			        continue;
839
		        }
840
841
		        if( rgar( $field, 'gsurveyLikertEnableMultipleRows' ) ) {
842
			        list( $row_val, $col_val ) = explode( ':', $field_value, 2 );
843
844
		            // If the $name matches the $row_val, we are processing the correct row
845
			        if( $row_val === $name ) {
846
				        $value = $field_value;
847
				        break;
848
			        }
849
		        } else {
850
			        // When not processing multiple rows, the value is the $entry[ $field_id ] value.
851
			        $value = $field_value;
852
				    break;
853
		        }
854
			}
855
        }
856
857
        return $value;
858
    }
859
860
    /**
861
     * Display the Update/Cancel/Delete buttons for the Edit Entry form
862
     * @since 1.8
863
     * @return string
864
     */
865
    public function render_form_buttons() {
866
        return gravityview_ob_include( GravityView_Edit_Entry::$file .'/partials/form-buttons.php', $this );
0 ignored issues
show
Bug introduced by
The property file cannot be accessed from this context as it is declared private in class GravityView_Edit_Entry.

This check looks for access to properties that are not accessible from the current context.

If you need to make a property accessible to another context you can either raise its visibility level or provide an accessible getter in the defining class.

Loading history...
867
    }
868
869
870
    /**
871
     * Modify the form fields that are shown when using GFFormDisplay::get_form()
872
     *
873
     * By default, all fields will be shown. We only want the Edit Tab configured fields to be shown.
874
     *
875
     * @param array $form
876
     * @param boolean $ajax Whether in AJAX mode
877
     * @param array|string $field_values Passed parameters to the form
878
     *
879
     * @since 1.9
880
     *
881
     * @return array Modified form array
882
     */
883
    public function filter_modify_form_fields( $form, $ajax = false, $field_values = '' ) {
0 ignored issues
show
Unused Code introduced by
The parameter $ajax is not used and could be removed.

This check looks from parameters that have been defined for a function or method, but which are not used in the method body.

Loading history...
Unused Code introduced by
The parameter $field_values is not used and could be removed.

This check looks from parameters that have been defined for a function or method, but which are not used in the method body.

Loading history...
884
885
        // In case we have validated the form, use it to inject the validation results into the form render
886
        if( isset( $this->form_after_validation ) ) {
887
            $form = $this->form_after_validation;
888
        } else {
889
            $form['fields'] = $this->get_configured_edit_fields( $form, $this->view_id );
890
        }
891
892
        $form = $this->filter_conditional_logic( $form );
893
894
        // for now we don't support Save and Continue feature.
895
        if( ! self::$supports_save_and_continue ) {
896
	        unset( $form['save'] );
897
        }
898
899
        return $form;
900
    }
901
902
    /**
903
     * When displaying a field, check if it's a Post Field, and if so, make sure the post exists and current user has edit rights.
904
     *
905
     * @since 1.16.2.2
906
     *
907
     * @param string $field_content Always empty. Returning not-empty overrides the input.
908
     * @param GF_Field $field
909
     * @param string|array $value If array, it's a field with multiple inputs. If string, single input.
910
     * @param int $lead_id Lead ID. Always 0 for the `gform_field_input` filter.
911
     * @param int $form_id Form ID
912
     *
913
     * @return string If error, the error message. If no error, blank string (modify_edit_field_input() runs next)
914
     */
915
    function verify_user_can_edit_post( $field_content = '', $field, $value, $lead_id = 0, $form_id ) {
0 ignored issues
show
Unused Code introduced by
The parameter $value is not used and could be removed.

This check looks from parameters that have been defined for a function or method, but which are not used in the method body.

Loading history...
Unused Code introduced by
The parameter $lead_id is not used and could be removed.

This check looks from parameters that have been defined for a function or method, but which are not used in the method body.

Loading history...
Unused Code introduced by
The parameter $form_id is not used and could be removed.

This check looks from parameters that have been defined for a function or method, but which are not used in the method body.

Loading history...
Best Practice introduced by
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
916
917
        if( GFCommon::is_post_field( $field ) ) {
918
919
            $message = null;
920
921
            // First, make sure they have the capability to edit the post.
922
            if( false === current_user_can( 'edit_post', $this->entry['post_id'] ) ) {
923
924
                /**
925
                 * @filter `gravityview/edit_entry/unsupported_post_field_text` Modify the message when someone isn't able to edit a post
926
                 * @param string $message The existing "You don't have permission..." text
927
                 */
928
                $message = apply_filters('gravityview/edit_entry/unsupported_post_field_text', __('You don&rsquo;t have permission to edit this post.', 'gravityview') );
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
Coding Style introduced by
Expected 1 spaces before closing bracket; 0 found
Loading history...
929
930
            } elseif( null === get_post( $this->entry['post_id'] ) ) {
931
                /**
932
                 * @filter `gravityview/edit_entry/no_post_text` Modify the message when someone is editing an entry attached to a post that no longer exists
933
                 * @param string $message The existing "This field is not editable; the post no longer exists." text
934
                 */
935
                $message = apply_filters('gravityview/edit_entry/no_post_text', __('This field is not editable; the post no longer exists.', 'gravityview' ) );
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
936
            }
937
938
            if( $message ) {
939
                $field_content = sprintf('<div class="ginput_container ginput_container_' . $field->type . '">%s</div>', wpautop( $message ) );
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
940
            }
941
        }
942
943
        return $field_content;
944
    }
945
946
    /**
947
     *
948
     * Fill-in the saved values into the form inputs
949
     *
950
     * @param string $field_content Always empty. Returning not-empty overrides the input.
951
     * @param GF_Field $field
952
     * @param string|array $value If array, it's a field with multiple inputs. If string, single input.
953
     * @param int $lead_id Lead ID. Always 0 for the `gform_field_input` filter.
954
     * @param int $form_id Form ID
955
     *
956
     * @return mixed
957
     */
958
    function modify_edit_field_input( $field_content = '', $field, $value, $lead_id = 0, $form_id ) {
0 ignored issues
show
Unused Code introduced by
The parameter $lead_id is not used and could be removed.

This check looks from parameters that have been defined for a function or method, but which are not used in the method body.

Loading history...
Unused Code introduced by
The parameter $form_id is not used and could be removed.

This check looks from parameters that have been defined for a function or method, but which are not used in the method body.

Loading history...
Best Practice introduced by
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
959
960
        $gv_field = GravityView_Fields::get_associated_field( $field );
961
962
        // If the form has been submitted, then we don't need to pre-fill the values,
963
        // Except for fileupload type and when a field input is overridden- run always!!
964
        if(
965
            ( $this->is_edit_entry_submission() && !in_array( $field->type, array( 'fileupload', 'post_image' ) ) )
0 ignored issues
show
introduced by
Expected 1 space after "!"; 0 found
Loading history...
966
            && false === ( $gv_field && is_callable( array( $gv_field, 'get_field_input' ) ) )
967
            || ! empty( $field_content )
968
            || GFCommon::is_product_field( $field->type ) // Prevent product fields from appearing editable
969
        ) {
970
	        return $field_content;
971
        }
972
973
        // Turn on Admin-style display for file upload fields only
974
        if( 'fileupload' === $field->type ) {
975
            $_GET['page'] = 'gf_entries';
976
        }
977
978
        // SET SOME FIELD DEFAULTS TO PREVENT ISSUES
979
        $field->adminOnly = false; /** @see GFFormDisplay::get_counter_init_script() need to prevent adminOnly */
980
981
        // add categories as choices for Post Category field
982
        if ( 'post_category' === $field->type ) {
983
            $field = GFCommon::add_categories_as_choices( $field, $value );
984
        }
985
986
        $field_value = $this->get_field_value( $field );
987
988
        /**
989
         * @filter `gravityview/edit_entry/field_value` Change the value of an Edit Entry field, if needed
990
         * @since 1.11
991
         * @param mixed $field_value field value used to populate the input
992
         * @param object $field Gravity Forms field object ( Class GF_Field )
993
         */
994
        $field_value = apply_filters( 'gravityview/edit_entry/field_value', $field_value, $field );
995
996
	    // Prevent any PHP warnings, like undefined index
997
	    ob_start();
998
999
        if( $gv_field && is_callable( array( $gv_field, 'get_field_input' ) ) ) {
1000
            /** @var GF_Field $gv_field */
1001
            $return = $gv_field->get_field_input( $this->form, $field_value, $this->entry, $field );
1002
        } else {
1003
	        $return = $field->get_field_input( $this->form, $field_value, $this->entry );
1004
        }
1005
0 ignored issues
show
Coding Style introduced by
Functions must not contain multiple empty lines in a row; found 2 empty lines
Loading history...
1006
1007
	    // If there was output, it's an error
1008
	    $warnings = ob_get_clean();
1009
1010
	    if( !empty( $warnings ) ) {
0 ignored issues
show
introduced by
Expected 1 space after "!"; 0 found
Loading history...
1011
		    do_action( 'gravityview_log_error', __METHOD__ . $warnings, $field_value );
1012
	    }
1013
1014
        /**
1015
         * Unset hack $_GET['page'] = 'gf_entries'
1016
         * We need the fileupload html field to render with the proper id
1017
         *  ( <li id="field_80_16" ... > )
1018
         */
1019
        unset( $_GET['page'] );
0 ignored issues
show
introduced by
Detected access of super global var $_GET, probably need manual inspection.
Loading history...
1020
1021
        return $return;
1022
    }
1023
1024
    /**
1025
     * Modify the value for the current field input
1026
     *
1027
     * @param GF_Field $field
1028
     *
1029
     * @return array|mixed|string|void
1030
     */
1031
    private function get_field_value( $field ) {
1032
1033
        /**
1034
         * @filter `gravityview/edit_entry/pre_populate/override` Allow the pre-populated value to override saved value in Edit Entry form. By default, pre-populate mechanism only kicks on empty fields.
1035
         * @param boolean True: override saved values; False: don't override (default)
1036
         * @param $field GF_Field object Gravity Forms field object
1037
         * @since 1.13
1038
         */
1039
        $override_saved_value = apply_filters( 'gravityview/edit_entry/pre_populate/override', false, $field );
1040
1041
        // We're dealing with multiple inputs (e.g. checkbox) but not time or date (as it doesn't store data in input IDs)
1042
        if( isset( $field->inputs ) && is_array( $field->inputs ) && !in_array( $field->type, array( 'time', 'date' ) ) ) {
0 ignored issues
show
introduced by
Expected 1 space after "!"; 0 found
Loading history...
1043
1044
            $field_value = array();
1045
1046
            // only accept pre-populated values if the field doesn't have any choice selected.
1047
            $allow_pre_populated = $field->allowsPrepopulate;
1048
1049
            foreach ( (array)$field->inputs as $input ) {
0 ignored issues
show
introduced by
No space after closing casting parenthesis is prohibited
Loading history...
1050
1051
                $input_id = strval( $input['id'] );
1052
                
1053
                if ( isset( $this->entry[ $input_id ] ) && ! gv_empty( $this->entry[ $input_id ], false, false ) ) {
1054
                    $field_value[ $input_id ] =  'post_category' === $field->type ? GFCommon::format_post_category( $this->entry[ $input_id ], true ) : $this->entry[ $input_id ];
0 ignored issues
show
introduced by
Expected 1 space after "="; 2 found
Loading history...
1055
                    $allow_pre_populated = false;
1056
                }
1 ignored issue
show
introduced by
Blank line found after control structure
Loading history...
1057
1058
            }
1059
1060
            $pre_value = $field->get_value_submission( array(), false );
1061
1062
            $field_value = ! $allow_pre_populated && ! ( $override_saved_value && !gv_empty( $pre_value, false, false ) ) ? $field_value : $pre_value;
0 ignored issues
show
introduced by
Expected 1 space after "!"; 0 found
Loading history...
1063
1064
        } else {
1065
1066
            $id = intval( $field->id );
1067
1068
            // get pre-populated value if exists
1069
            $pre_value = $field->allowsPrepopulate ? GFFormsModel::get_parameter_value( $field->inputName, array(), $field ) : '';
1070
1071
            // saved field entry value (if empty, fallback to the pre-populated value, if exists)
1072
            // or pre-populated value if not empty and set to override saved value
1073
            $field_value = !gv_empty( $this->entry[ $id ], false, false ) && ! ( $override_saved_value && !gv_empty( $pre_value, false, false ) ) ? $this->entry[ $id ] : $pre_value;
0 ignored issues
show
introduced by
Expected 1 space after "!"; 0 found
Loading history...
1074
1075
            // in case field is post_category but inputType is select, multi-select or radio, convert value into array of category IDs.
1076
            if ( 'post_category' === $field->type && !gv_empty( $field_value, false, false ) ) {
0 ignored issues
show
introduced by
Expected 1 space after "!"; 0 found
Loading history...
1077
                $categories = array();
1078
                foreach ( explode( ',', $field_value ) as $cat_string ) {
1079
                    $categories[] = GFCommon::format_post_category( $cat_string, true );
1080
                }
1081
                $field_value = 'multiselect' === $field->get_input_type() ? $categories : implode( '', $categories );
1082
            }
1 ignored issue
show
introduced by
Blank line found after control structure
Loading history...
1083
1084
        }
1085
1086
        // if value is empty get the default value if defined
1087
        $field_value = $field->get_value_default_if_empty( $field_value );
1088
1089
        return $field_value;
1090
    }
1091
1092
1093
    // ---- Entry validation
1094
1095
    /**
1096
     * Add field keys that Gravity Forms expects.
1097
     *
1098
     * @see GFFormDisplay::validate()
1099
     * @param  array $form GF Form
1100
     * @return array       Modified GF Form
1101
     */
1102
    function gform_pre_validation( $form ) {
0 ignored issues
show
Best Practice introduced by
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
1103
1104
        if( ! $this->verify_nonce() ) {
1105
            return $form;
1106
        }
1107
1108
        // Fix PHP warning regarding undefined index.
1109
        foreach ( $form['fields'] as &$field) {
0 ignored issues
show
introduced by
No space before closing parenthesis is prohibited
Loading history...
1110
1111
            // This is because we're doing admin form pretending to be front-end, so Gravity Forms
1112
            // expects certain field array items to be set.
1113
            foreach ( array( 'noDuplicates', 'adminOnly', 'inputType', 'isRequired', 'enablePrice', 'inputs', 'allowedExtensions' ) as $key ) {
1114
	            $field->{$key} = isset( $field->{$key} ) ? $field->{$key} : NULL;
0 ignored issues
show
Coding Style introduced by
TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.
Loading history...
1115
            }
1116
1117
            // unset emailConfirmEnabled for email type fields
1118
           /* if( 'email' === $field['type'] && !empty( $field['emailConfirmEnabled'] ) ) {
1 ignored issue
show
Unused Code Comprehensibility introduced by
61% of this comment could be valid code. Did you maybe forget this after debugging?

Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it.

The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production.

This check looks for comments that seem to be mostly valid code and reports them.

Loading history...
Coding Style introduced by
Line indented incorrectly; expected at least 3 tabs, found 2
Loading history...
1119
                $field['emailConfirmEnabled'] = '';
1120
            }*/
1121
1122
            switch( RGFormsModel::get_input_type( $field ) ) {
1123
1124
                /**
1125
                 * 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.
1126
                 *
1127
                 * What we have to do is set the value so that it doesn't get overwritten as empty on save and appears immediately in the Edit Entry screen again.
1128
                 *
1129
                 * @hack
1130
                 */
1131
                case 'fileupload':
1132
1133
                    // Set the previous value
1134
                    $entry = $this->get_entry();
1135
1136
                    $input_name = 'input_'.$field->id;
1137
                    $form_id = $form['id'];
1138
1139
                    $value = NULL;
0 ignored issues
show
Coding Style introduced by
TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.
Loading history...
1140
1141
                    // Use the previous entry value as the default.
1142
                    if( isset( $entry[ $field->id ] ) ) {
1143
                        $value = $entry[ $field->id ];
1144
                    }
1145
1146
                    // If this is a single upload file
1147
                    if( !empty( $_FILES[ $input_name ] ) && !empty( $_FILES[ $input_name ]['name'] ) ) {
0 ignored issues
show
introduced by
Expected 1 space after "!"; 0 found
Loading history...
1148
                        $file_path = GFFormsModel::get_file_upload_path( $form['id'], $_FILES[ $input_name ]['name'] );
1149
                        $value = $file_path['url'];
1150
1151
                    } else {
1152
1153
                        // Fix PHP warning on line 1498 of form_display.php for post_image fields
1154
                        // Fix PHP Notice:  Undefined index:  size in form_display.php on line 1511
1155
                        $_FILES[ $input_name ] = array('name' => '', 'size' => '' );
0 ignored issues
show
introduced by
No space after opening parenthesis of array is bad style
Loading history...
1156
1157
                    }
1158
1159
                    if( rgar($field, "multipleFiles") ) {
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
Coding Style introduced by
Expected 1 spaces before closing bracket; 0 found
Loading history...
Coding Style Comprehensibility introduced by
The string literal multipleFiles does not require double quotes, as per coding-style, please use single quotes.

PHP provides two ways to mark string literals. Either with single quotes 'literal' or with double quotes "literal". The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences.

String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (\') and the backslash (\\). Every other character is displayed as is.

Double quoted string literals may contain other variables or more complex escape sequences.

<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;

will print an indented: Single is Value

If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear.

For more information on PHP string literals and available escape sequences see the PHP core documentation.

Loading history...
1160
1161
                        // If there are fresh uploads, process and merge them.
1162
                        // Otherwise, use the passed values, which should be json-encoded array of URLs
1163
                        if( isset( GFFormsModel::$uploaded_files[$form_id][$input_name] ) ) {
0 ignored issues
show
introduced by
Array keys should be surrounded by spaces unless they contain a string or an integer.
Loading history...
1164
                            $value = empty( $value ) ? '[]' : $value;
1165
                            $value = stripslashes_deep( $value );
1166
                            $value = GFFormsModel::prepare_value( $form, $field, $value, $input_name, $entry['id'], array());
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces before closing bracket; 0 found
Loading history...
1167
                        }
1 ignored issue
show
introduced by
Blank line found after control structure
Loading history...
1168
1169
                    } else {
1170
1171
                        // A file already exists when editing an entry
1172
                        // We set this to solve issue when file upload fields are required.
1173
                        GFFormsModel::$uploaded_files[ $form_id ][ $input_name ] = $value;
1174
1175
                    }
1176
1177
                    $this->entry[ $input_name ] = $value;
1178
                    $_POST[ $input_name ] = $value;
1179
1180
                    break;
1181
1182
                case 'number':
1183
                    // Fix "undefined index" issue at line 1286 in form_display.php
1184
                    if( !isset( $_POST['input_'.$field->id ] ) ) {
0 ignored issues
show
introduced by
Expected 1 space after "!"; 0 found
Loading history...
introduced by
Array keys should be surrounded by spaces unless they contain a string or an integer.
Loading history...
1185
                        $_POST['input_'.$field->id ] = NULL;
0 ignored issues
show
introduced by
Array keys should be surrounded by spaces unless they contain a string or an integer.
Loading history...
Coding Style introduced by
TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.
Loading history...
1186
                    }
1187
                    break;
1188
                case 'captcha':
1189
                    // Fix issue with recaptcha_check_answer() on line 1458 in form_display.php
1190
                    $_POST['recaptcha_challenge_field'] = NULL;
0 ignored issues
show
Coding Style introduced by
TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.
Loading history...
1191
                    $_POST['recaptcha_response_field'] = NULL;
0 ignored issues
show
Coding Style introduced by
TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.
Loading history...
1192
                    break;
1193
            }
1 ignored issue
show
introduced by
Blank line found after control structure
Loading history...
1194
1195
        }
1196
1197
        return $form;
1198
    }
1199
1200
1201
    /**
1202
     * Process validation for a edit entry submission
1203
     *
1204
     * Sets the `is_valid` object var
1205
     *
1206
     * @return void
1207
     */
1208
    function validate() {
0 ignored issues
show
Best Practice introduced by
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
1209
1210
        /**
1211
         * If using GF User Registration Add-on, remove the validation step, otherwise generates error when updating the entry
1212
         * GF User Registration Add-on version > 3.x has a different class name
1213
         * @since 1.16.2
1214
         */
1215
        if ( class_exists( 'GF_User_Registration' ) ) {
1216
            remove_filter( 'gform_validation', array( GF_User_Registration::get_instance(), 'validate' ) );
1217
        } else  if ( class_exists( 'GFUser' ) ) {
1218
            remove_filter( 'gform_validation', array( 'GFUser', 'user_registration_validation' ) );
1219
        }
1220
0 ignored issues
show
Coding Style introduced by
Functions must not contain multiple empty lines in a row; found 2 empty lines
Loading history...
1221
1222
        /**
1223
         * For some crazy reason, Gravity Forms doesn't validate Edit Entry form submissions.
1224
         * You can enter whatever you want!
1225
         * We try validating, and customize the results using `self::custom_validation()`
1226
         */
1227
        add_filter( 'gform_validation_'. $this->form_id, array( $this, 'custom_validation' ), 10, 4);
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces before closing bracket; 0 found
Loading history...
1228
1229
        // Needed by the validate funtion
1230
        $failed_validation_page = NULL;
0 ignored issues
show
Coding Style introduced by
TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.
Loading history...
1231
        $field_values = RGForms::post( 'gform_field_values' );
1232
1233
        // Prevent entry limit from running when editing an entry, also
1234
        // prevent form scheduling from preventing editing
1235
        unset( $this->form['limitEntries'], $this->form['scheduleForm'] );
1236
1237
        // Hide fields depending on Edit Entry settings
1238
        $this->form['fields'] = $this->get_configured_edit_fields( $this->form, $this->view_id );
1239
1240
        $this->is_valid = GFFormDisplay::validate( $this->form, $field_values, 1, $failed_validation_page );
1241
1242
        remove_filter( 'gform_validation_'. $this->form_id, array( $this, 'custom_validation' ), 10 );
1243
    }
1244
1245
1246
    /**
1247
     * Make validation work for Edit Entry
1248
     *
1249
     * Because we're calling the GFFormDisplay::validate() in an unusual way (as a front-end
1250
     * form pretending to be a back-end form), validate() doesn't know we _can't_ edit post
1251
     * fields. This goes through all the fields and if they're an invalid post field, we
1252
     * set them as valid. If there are still issues, we'll return false.
1253
     *
1254
     * @param  [type] $validation_results [description]
0 ignored issues
show
Documentation introduced by
The doc-type [type] could not be parsed: Unknown type name "" at position 0. [(view supported doc-types)

This check marks PHPDoc comments that could not be parsed by our parser. To see which comment annotations we can parse, please refer to our documentation on supported doc-types.

Loading history...
1255
     * @return [type]                     [description]
0 ignored issues
show
Documentation introduced by
The doc-type [type] could not be parsed: Unknown type name "" at position 0. [(view supported doc-types)

This check marks PHPDoc comments that could not be parsed by our parser. To see which comment annotations we can parse, please refer to our documentation on supported doc-types.

Loading history...
1256
     */
1257
    function custom_validation( $validation_results ) {
0 ignored issues
show
Best Practice introduced by
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
1258
1259
        do_action('gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Validation results: ', $validation_results );
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
1260
1261
        do_action('gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] $_POSTed data (sanitized): ', esc_html( print_r( $_POST, true ) ) );
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
introduced by
The use of function print_r() is discouraged
Loading history...
introduced by
Detected access of super global var $_POST, probably need manual inspection.
Loading history...
1262
1263
        $gv_valid = true;
1264
1265
        foreach ( $validation_results['form']['fields'] as $key => &$field ) {
1266
1267
            $value = RGFormsModel::get_field_value( $field );
1268
            $field_type = RGFormsModel::get_input_type( $field );
1269
1270
            // Validate always
1271
            switch ( $field_type ) {
1272
0 ignored issues
show
Coding Style introduced by
Functions must not contain multiple empty lines in a row; found 2 empty lines
Loading history...
1273
1274
                case 'fileupload' :
1275
                case 'post_image':
1276
1277
                    // in case nothing is uploaded but there are already files saved
1278
                    if( !empty( $field->failed_validation ) && !empty( $field->isRequired ) && !empty( $value ) ) {
0 ignored issues
show
introduced by
Expected 1 space after "!"; 0 found
Loading history...
1279
                        $field->failed_validation = false;
1280
                        unset( $field->validation_message );
1281
                    }
1282
1283
                    // validate if multi file upload reached max number of files [maxFiles] => 2
1284
                    if( rgobj( $field, 'maxFiles') && rgobj( $field, 'multipleFiles') ) {
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces before closing bracket; 0 found
Loading history...
1285
1286
                        $input_name = 'input_' . $field->id;
1287
                        //uploaded
1288
                        $file_names = isset( GFFormsModel::$uploaded_files[ $validation_results['form']['id'] ][ $input_name ] ) ? GFFormsModel::$uploaded_files[ $validation_results['form']['id'] ][ $input_name ] : array();
1289
1290
                        //existent
1291
                        $entry = $this->get_entry();
1292
                        $value = NULL;
0 ignored issues
show
Coding Style introduced by
TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.
Loading history...
1293
                        if( isset( $entry[ $field->id ] ) ) {
1294
                            $value = json_decode( $entry[ $field->id ], true );
1295
                        }
1296
1297
                        // count uploaded files and existent entry files
1298
                        $count_files = count( $file_names ) + count( $value );
1299
1300
                        if( $count_files > $field->maxFiles ) {
1301
                            $field->validation_message = __( 'Maximum number of files reached', 'gravityview' );
1302
                            $field->failed_validation = 1;
1303
                            $gv_valid = false;
1304
1305
                            // in case of error make sure the newest upload files are removed from the upload input
1306
                            GFFormsModel::$uploaded_files[ $validation_results['form']['id'] ] = null;
1307
                        }
1 ignored issue
show
introduced by
Blank line found after control structure
Loading history...
1308
1309
                    }
1310
0 ignored issues
show
Coding Style introduced by
Functions must not contain multiple empty lines in a row; found 2 empty lines
Loading history...
1311
1312
                    break;
1313
1314
            }
1315
1316
            // This field has failed validation.
1317
            if( !empty( $field->failed_validation ) ) {
0 ignored issues
show
introduced by
Expected 1 space after "!"; 0 found
Loading history...
1318
1319
                do_action( 'gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Field is invalid.', array( 'field' => $field, 'value' => $value ) );
1320
1321
                switch ( $field_type ) {
1322
1323
                    // Captchas don't need to be re-entered.
1324
                    case 'captcha':
1325
1326
                        // Post Image fields aren't editable, so we un-fail them.
1327
                    case 'post_image':
1328
                        $field->failed_validation = false;
1329
                        unset( $field->validation_message );
1330
                        break;
1331
1332
                }
1333
1334
                // You can't continue inside a switch, so we do it after.
1335
                if( empty( $field->failed_validation ) ) {
1336
                    continue;
1337
                }
1338
1339
                // checks if the No Duplicates option is not validating entry against itself, since
1340
                // we're editing a stored entry, it would also assume it's a duplicate.
1341
                if( !empty( $field->noDuplicates ) ) {
0 ignored issues
show
introduced by
Expected 1 space after "!"; 0 found
Loading history...
1342
1343
                    $entry = $this->get_entry();
1344
1345
                    // If the value of the entry is the same as the stored value
1346
                    // Then we can assume it's not a duplicate, it's the same.
1347
                    if( !empty( $entry ) && $value == $entry[ $field->id ] ) {
0 ignored issues
show
introduced by
Expected 1 space after "!"; 0 found
Loading history...
1348
                        //if value submitted was not changed, then don't validate
1349
                        $field->failed_validation = false;
1350
1351
                        unset( $field->validation_message );
1352
1353
                        do_action('gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Field not a duplicate; it is the same entry.', $entry );
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
1354
1355
                        continue;
1356
                    }
1357
                }
1358
1359
                // if here then probably we are facing the validation 'At least one field must be filled out'
1360
                if( GFFormDisplay::is_empty( $field, $this->form_id  ) && empty( $field->isRequired ) ) {
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces before closing bracket; 2 found
Loading history...
1361
                    unset( $field->validation_message );
1362
	                $field->validation_message = false;
1363
                    continue;
1364
                }
1365
1366
                $gv_valid = false;
1367
1368
            }
1 ignored issue
show
introduced by
Blank line found after control structure
Loading history...
1369
1370
        }
1371
1372
        $validation_results['is_valid'] = $gv_valid;
1373
1374
        do_action('gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Validation results.', $validation_results );
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
1375
1376
        // We'll need this result when rendering the form ( on GFFormDisplay::get_form )
1377
        $this->form_after_validation = $validation_results['form'];
1378
1379
        return $validation_results;
1380
    }
1381
1382
1383
    /**
1384
     * TODO: This seems to be hacky... we should remove it. Entry is set when updating the form using setup_vars()!
1385
     * Get the current entry and set it if it's not yet set.
1386
     * @return array Gravity Forms entry array
1387
     */
1388
    private function get_entry() {
1389
1390
        if( empty( $this->entry ) ) {
1391
            // Get the database value of the entry that's being edited
1392
            $this->entry = gravityview_get_entry( GravityView_frontend::is_single_entry() );
1393
        }
1394
1395
        return $this->entry;
1396
    }
1397
1398
1399
1400
    // --- Filters
1401
1402
    /**
1403
     * Get the Edit Entry fields as configured in the View
1404
     *
1405
     * @since 1.8
1406
     *
1407
     * @param int $view_id
1408
     *
1409
     * @return array Array of fields that are configured in the Edit tab in the Admin
1410
     */
1411
    private function get_configured_edit_fields( $form, $view_id ) {
1412
1413
        // Get all fields for form
1414
        $properties = GravityView_View_Data::getInstance()->get_fields( $view_id );
1415
1416
        // If edit tab not yet configured, show all fields
1417
        $edit_fields = !empty( $properties['edit_edit-fields'] ) ? $properties['edit_edit-fields'] : NULL;
0 ignored issues
show
introduced by
Expected 1 space after "!"; 0 found
Loading history...
Coding Style introduced by
TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.
Loading history...
1418
1419
	    // Show hidden fields as text fields
1420
	    $form = $this->fix_hidden_fields( $form );
1421
1422
        // Show hidden fields as text fields
1423
        $form = $this->fix_survey_fields( $form );
1424
1425
        // Hide fields depending on admin settings
1426
        $fields = $this->filter_fields( $form['fields'], $edit_fields );
1427
1428
	    // If Edit Entry fields are configured, remove adminOnly field settings. Otherwise, don't.
1429
	    $fields = $this->filter_admin_only_fields( $fields, $edit_fields, $form, $view_id );
1430
1431
        return $fields;
1432
    }
1433
1434
    /**
1435
     * Make sure Survey fields accept pre-populating values; otherwise existing values won't be filled-in
1436
     *
1437
     * @since 1.16.4
1438
     *
1439
     * @param array $form
1440
     *
1441
     * @return array Form, with all fields set to `allowsPrepopulate => true`
1442
     */
1443
    private function fix_survey_fields( $form ) {
1444
1445
        /** @var GF_Field $field */
1446
        foreach( $form['fields'] as &$field ) {
1447
            $field->allowsPrepopulate = true;
1448
        }
1449
1450
        return $form;
1451
    }
1452
1453
	/**
1454
	 * @since 1.9.2
1455
	 *
1456
	 * @param $fields
1457
	 *
1458
	 * @return mixed
1459
	 */
1460
	private function fix_hidden_fields( $form ) {
1461
1462
		/** @var GF_Field $field */
1463
		foreach( $form['fields'] as $key => $field ) {
1464
			if( 'hidden' === $field->type ) {
1465
				$text_field = new GF_Field_Text( $field );
1466
				$text_field->type = 'text';
1467
				$form['fields'][ $key ] = $text_field;
1468
			}
1469
		}
1470
1471
		return $form;
1472
	}
1473
1474
1475
    /**
1476
     * Filter area fields based on specified conditions
1477
     *  - This filter removes the fields that have calculation configured
1478
     *
1479
     * @uses GravityView_Edit_Entry::user_can_edit_field() Check caps
1480
     * @access private
1481
     * @param GF_Field[] $fields
1482
     * @param array $configured_fields
1483
     * @since  1.5
1484
     * @return array $fields
1485
     */
1486
    private function filter_fields( $fields, $configured_fields ) {
1487
1488
        if( empty( $fields ) || !is_array( $fields ) ) {
0 ignored issues
show
introduced by
Expected 1 space after "!"; 0 found
Loading history...
1489
            return $fields;
1490
        }
1491
1492
        $edit_fields = array();
1493
1494
        $field_type_blacklist = array(
1495
            'page',
1496
        );
1497
1498
	    /**
1499
	     * @filter `gravityview/edit_entry/hide-product-fields` Hide product fields from being editable.
1500
	     * @since 1.9.1
1501
         * @param boolean $hide_product_fields Whether to hide product fields in the editor.  Default: false
1502
	     */
1503
	    $hide_product_fields = apply_filters( 'gravityview/edit_entry/hide-product-fields', empty( self::$supports_product_fields ) );
1504
1505
	    if( $hide_product_fields ) {
1506
		    $field_type_blacklist[] = 'option';
1507
		    $field_type_blacklist[] = 'quantity';
1508
            $field_type_blacklist[] = 'product';
1509
            $field_type_blacklist[] = 'total';
1510
            $field_type_blacklist[] = 'shipping';
1511
            $field_type_blacklist[] = 'calculation';
1512
	    }
1513
1514
        // First, remove blacklist or calculation fields
1515
        foreach ( $fields as $key => $field ) {
1516
1517
            // Remove the fields that have calculation properties and keep them to be used later
1518
            // @since 1.16.2
1519
            if( $field->has_calculation() ) {
1520
                $this->fields_with_calculation[] = $field;
1521
                // don't remove the calculation fields on form render.
1522
            }
1523
1524
            // process total field after all fields have been saved
1525
            if ( $field->type == 'total' ) {
0 ignored issues
show
introduced by
Found "== '". Use Yoda Condition checks, you must
Loading history...
1526
                $this->total_fields[] = $field;
1527
                unset( $fields[ $key ] );
1528
            }
1529
1530
            if( in_array( $field->type, $field_type_blacklist ) ) {
1531
                unset( $fields[ $key ] );
1532
            }
1533
        }
1534
1535
        // The Edit tab has not been configured, so we return all fields by default.
1536
        if( empty( $configured_fields ) ) {
1537
            return $fields;
1538
        }
1539
1540
        // The edit tab has been configured, so we loop through to configured settings
1541
        foreach ( $configured_fields as $configured_field ) {
1542
1543
	        /** @var GF_Field $field */
1544
	        foreach ( $fields as $field ) {
1545
1546
                if( intval( $configured_field['id'] ) === intval( $field->id ) && $this->user_can_edit_field( $configured_field, false ) ) {
1547
                    $edit_fields[] = $this->merge_field_properties( $field, $configured_field );
1548
                    break;
1549
                }
1 ignored issue
show
introduced by
Blank line found after control structure
Loading history...
1550
1551
            }
1 ignored issue
show
introduced by
Blank line found after control structure
Loading history...
1552
1553
        }
1554
1555
        return $edit_fields;
1556
1557
    }
1558
1559
    /**
1560
     * Override GF Form field properties with the ones defined on the View
1561
     * @param  GF_Field $field GF Form field object
1562
     * @param  array $setting  GV field options
0 ignored issues
show
Documentation introduced by
There is no parameter named $setting. Did you maybe mean $field_setting?

This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. It has, however, found a similar but not annotated parameter which might be a good fit.

Consider the following example. The parameter $ireland is not defined by the method finale(...).

/**
 * @param array $germany
 * @param array $ireland
 */
function finale($germany, $island) {
    return "2:1";
}

The most likely cause is that the parameter was changed, but the annotation was not.

Loading history...
1563
     * @since  1.5
1564
     * @return array
1565
     */
1566
    private function merge_field_properties( $field, $field_setting ) {
1567
1568
        $return_field = $field;
1569
1570
        if( empty( $field_setting['show_label'] ) ) {
1571
            $return_field->label = '';
1572
        } elseif ( !empty( $field_setting['custom_label'] ) ) {
0 ignored issues
show
introduced by
Expected 1 space after "!"; 0 found
Loading history...
1573
            $return_field->label = $field_setting['custom_label'];
1574
        }
1575
1576
        if( !empty( $field_setting['custom_class'] ) ) {
0 ignored issues
show
introduced by
Expected 1 space after "!"; 0 found
Loading history...
1577
            $return_field->cssClass .= ' '. gravityview_sanitize_html_class( $field_setting['custom_class'] );
1578
        }
1579
1580
        /**
1581
         * Normalize page numbers - avoid conflicts with page validation
1582
         * @since 1.6
1583
         */
1584
        $return_field->pageNumber = 1;
1585
1586
        return $return_field;
1587
1588
    }
1589
1590
    /**
1591
     * Remove fields that shouldn't be visible based on the Gravity Forms adminOnly field property
1592
     *
1593
     * @since 1.9.1
1594
     *
1595
     * @param array|GF_Field[] $fields Gravity Forms form fields
1596
     * @param array|null $edit_fields Fields for the Edit Entry tab configured in the View Configuration
1597
     * @param array $form GF Form array
1598
     * @param int $view_id View ID
1599
     *
1600
     * @return array Possibly modified form array
1601
     */
1602
    function filter_admin_only_fields( $fields = array(), $edit_fields = null, $form = array(), $view_id = 0 ) {
0 ignored issues
show
Best Practice introduced by
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
1603
1604
	    /**
1605
         * @filter `gravityview/edit_entry/use_gf_admin_only_setting` When Edit tab isn't configured, should the Gravity Forms "Admin Only" field settings be used to control field display to non-admins? Default: true
1606
	     * If the Edit Entry tab is not configured, adminOnly fields will not be shown to non-administrators.
1607
	     * If the Edit Entry tab *is* configured, adminOnly fields will be shown to non-administrators, using the configured GV permissions
1608
	     * @since 1.9.1
1609
	     * @param boolean $use_gf_adminonly_setting True: Hide field if set to Admin Only in GF and the user is not an admin. False: show field based on GV permissions, ignoring GF permissions.
1610
	     * @param array $form GF Form array
1611
	     * @param int $view_id View ID
1612
	     */
1613
	    $use_gf_adminonly_setting = apply_filters( 'gravityview/edit_entry/use_gf_admin_only_setting', empty( $edit_fields ), $form, $view_id );
1614
1615
	    if( $use_gf_adminonly_setting && false === GVCommon::has_cap( 'gravityforms_edit_entries', $this->entry['id'] ) ) {
1616
            foreach( $fields as $k => $field ) {
1617
                if( $field->adminOnly ) {
1618
                    unset( $fields[ $k ] );
1619
                }
1620
            }
1621
            return $fields;
1622
        }
1623
1624
	    foreach( $fields as &$field ) {
1625
		    $field->adminOnly = false;
1626
        }
1627
1628
        return $fields;
1629
    }
1630
1631
    // --- Conditional Logic
1632
1633
    /**
1634
     * Remove the conditional logic rules from the form button and the form fields, if needed.
1635
     *
1636
     * @since 1.9
1637
     *
1638
     * @param array $form Gravity Forms form
1639
     * @return array Modified form, if not using Conditional Logic
1640
     */
1641
    function filter_conditional_logic( $form ) {
0 ignored issues
show
Best Practice introduced by
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
1642
1643
        /**
1644
         * @filter `gravityview/edit_entry/conditional_logic` Should the Edit Entry form use Gravity Forms conditional logic showing/hiding of fields?
1645
         * @since 1.9
1646
         * @param bool $use_conditional_logic True: Gravity Forms will show/hide fields just like in the original form; False: conditional logic will be disabled and fields will be shown based on configuration. Default: true
1647
         * @param array $form Gravity Forms form
1648
         */
1649
        $use_conditional_logic = apply_filters( 'gravityview/edit_entry/conditional_logic', true, $form );
1650
1651
        if( $use_conditional_logic ) {
1652
            return $form;
1653
        }
1654
1655
        foreach( $form['fields'] as &$field ) {
1656
            /* @var GF_Field $field */
1657
            $field->conditionalLogic = null;
1658
        }
1659
1660
        unset( $form['button']['conditionalLogic'] );
1661
1662
        return $form;
1663
1664
    }
1665
1666
    /**
1667
     * Disable the Gravity Forms conditional logic script and features on the Edit Entry screen
1668
     *
1669
     * @since 1.9
1670
     *
1671
     * @param $has_conditional_logic
1672
     * @param $form
1673
     * @return mixed|void
1674
     */
1675
    function manage_conditional_logic( $has_conditional_logic, $form ) {
0 ignored issues
show
Best Practice introduced by
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
1676
1677
        if( ! $this->is_edit_entry() ) {
1678
            return $has_conditional_logic;
1679
        }
1680
1681
        return apply_filters( 'gravityview/edit_entry/conditional_logic', $has_conditional_logic, $form );
1682
    }
1683
1684
1685
    // --- User checks and nonces
1686
1687
    /**
1688
     * Check if the user can edit the entry
1689
     *
1690
     * - Is the nonce valid?
1691
     * - Does the user have the right caps for the entry
1692
     * - Is the entry in the trash?
1693
     *
1694
     * @todo Move to GVCommon
1695
     *
1696
     * @param  boolean $echo Show error messages in the form?
1697
     * @return boolean        True: can edit form. False: nope.
1698
     */
1699
    function user_can_edit_entry( $echo = false ) {
0 ignored issues
show
Best Practice introduced by
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
1700
1701
        $error = NULL;
0 ignored issues
show
Coding Style introduced by
TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.
Loading history...
1702
1703
        /**
1704
         *  1. Permalinks are turned off
1705
         *  2. There are two entries embedded using oEmbed
1706
         *  3. One of the entries has just been saved
1707
         */
1708
        if( !empty( $_POST['lid'] ) && !empty( $_GET['entry'] ) && ( $_POST['lid'] !== $_GET['entry'] ) ) {
0 ignored issues
show
introduced by
Expected 1 space after "!"; 0 found
Loading history...
1709
1710
            $error = true;
1711
1712
        }
1713
1714
        if( !empty( $_GET['entry'] ) && (string)$this->entry['id'] !== $_GET['entry'] ) {
0 ignored issues
show
introduced by
Expected 1 space after "!"; 0 found
Loading history...
introduced by
No space after closing casting parenthesis is prohibited
Loading history...
1715
1716
            $error = true;
1717
1718
        } elseif( ! $this->verify_nonce() ) {
1719
1720
            /**
1721
             * If the Entry is embedded, there may be two entries on the same page.
1722
             * If that's the case, and one is being edited, the other should fail gracefully and not display an error.
1723
             */
1724
            if( GravityView_oEmbed::getInstance()->get_entry_id() ) {
0 ignored issues
show
Bug Best Practice introduced by
The expression \GravityView_oEmbed::get...tance()->get_entry_id() of type integer|null is loosely compared to true; this is ambiguous if the integer can be zero. You might want to explicitly use !== null instead.

In PHP, under loose comparison (like ==, or !=, or switch conditions), values of different types might be equal.

For integer values, zero is a special case, in particular the following results might be unexpected:

0   == false // true
0   == null  // true
123 == false // false
123 == null  // false

// It is often better to use strict comparison
0 === false // false
0 === null  // false
Loading history...
1725
                $error = true;
1726
            } else {
1727
                $error = __( 'The link to edit this entry is not valid; it may have expired.', 'gravityview');
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces before closing bracket; 0 found
Loading history...
1728
            }
1 ignored issue
show
introduced by
Blank line found after control structure
Loading history...
1729
1730
        }
1731
1732
        if( ! GravityView_Edit_Entry::check_user_cap_edit_entry( $this->entry ) ) {
1733
            $error = __( 'You do not have permission to edit this entry.', 'gravityview');
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces before closing bracket; 0 found
Loading history...
1734
        }
1735
1736
        if( $this->entry['status'] === 'trash' ) {
0 ignored issues
show
introduced by
Found "=== '". Use Yoda Condition checks, you must
Loading history...
1737
            $error = __('You cannot edit the entry; it is in the trash.', 'gravityview' );
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
1738
        }
1739
1740
        // No errors; everything's fine here!
1741
        if( empty( $error ) ) {
1742
            return true;
1743
        }
1744
1745
        if( $echo && $error !== true ) {
0 ignored issues
show
introduced by
Found "!== true". Use Yoda Condition checks, you must
Loading history...
1746
1747
	        $error = esc_html( $error );
1748
1749
	        /**
1750
	         * @since 1.9
1751
	         */
1752
	        if ( ! empty( $this->entry ) ) {
1753
		        $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;" ) );
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
Coding Style Comprehensibility introduced by
The string literal window.history.go(-1); return false; does not require double quotes, as per coding-style, please use single quotes.

PHP provides two ways to mark string literals. Either with single quotes 'literal' or with double quotes "literal". The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences.

String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (\') and the backslash (\\). Every other character is displayed as is.

Double quoted string literals may contain other variables or more complex escape sequences.

<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;

will print an indented: Single is Value

If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear.

For more information on PHP string literals and available escape sequences see the PHP core documentation.

Loading history...
1754
	        }
1755
1756
            echo GVCommon::generate_notice( wpautop( $error ), 'gv-error error');
0 ignored issues
show
introduced by
Expected a sanitizing function (see Codex for 'Data Validation'), but instead saw 'GVCommon'
Loading history...
Coding Style introduced by
Expected 1 spaces before closing bracket; 0 found
Loading history...
1757
        }
1758
1759
        do_action('gravityview_log_error', 'GravityView_Edit_Entry[user_can_edit_entry]' . $error );
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
1760
1761
        return false;
1762
    }
1763
1764
1765
    /**
1766
     * Check whether a field is editable by the current user, and optionally display an error message
1767
     * @uses  GravityView_Edit_Entry->check_user_cap_edit_field() Check user capabilities
1768
     * @param  array  $field Field or field settings array
1769
     * @param  boolean $echo  Whether to show error message telling user they aren't allowed
1770
     * @return boolean         True: user can edit the current field; False: nope, they can't.
1771
     */
1772
    private function user_can_edit_field( $field, $echo = false ) {
1773
1774
        $error = NULL;
0 ignored issues
show
Coding Style introduced by
TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.
Loading history...
1775
1776
        if( ! $this->check_user_cap_edit_field( $field ) ) {
1777
            $error = __( 'You do not have permission to edit this field.', 'gravityview');
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces before closing bracket; 0 found
Loading history...
1778
        }
1779
1780
        // No errors; everything's fine here!
1781
        if( empty( $error ) ) {
1782
            return true;
1783
        }
1784
1785
        if( $echo ) {
1786
            echo GVCommon::generate_notice( wpautop( esc_html( $error ) ), 'gv-error error');
0 ignored issues
show
introduced by
Expected a sanitizing function (see Codex for 'Data Validation'), but instead saw 'GVCommon'
Loading history...
Coding Style introduced by
Expected 1 spaces before closing bracket; 0 found
Loading history...
1787
        }
1788
1789
        do_action('gravityview_log_error', 'GravityView_Edit_Entry[user_can_edit_field]' . $error );
0 ignored issues
show
Coding Style introduced by
Expected 1 spaces after opening bracket; 0 found
Loading history...
1790
1791
        return false;
1792
1793
    }
1794
1795
1796
    /**
1797
     * checks if user has permissions to edit a specific field
1798
     *
1799
     * Needs to be used combined with GravityView_Edit_Entry::user_can_edit_field for maximum security!!
1800
     *
1801
     * @param  [type] $field [description]
0 ignored issues
show
Documentation introduced by
The doc-type [type] could not be parsed: Unknown type name "" at position 0. [(view supported doc-types)

This check marks PHPDoc comments that could not be parsed by our parser. To see which comment annotations we can parse, please refer to our documentation on supported doc-types.

Loading history...
1802
     * @return bool
1803
     */
1804
    private function check_user_cap_edit_field( $field ) {
1805
1806
        // If they can edit any entries (as defined in Gravity Forms), we're good.
1807
        if( GVCommon::has_cap( array( 'gravityforms_edit_entries', 'gravityview_edit_others_entries' ) ) ) {
1808
            return true;
1809
        }
1810
1811
        $field_cap = isset( $field['allow_edit_cap'] ) ? $field['allow_edit_cap'] : false;
1812
1813
        // If the field has custom editing capaibilities set, check those
1814
        if( $field_cap ) {
1815
            return GVCommon::has_cap( $field['allow_edit_cap'] );
1816
        }
1817
1818
        return false;
1819
    }
1820
1821
1822
    /**
1823
     * Is the current nonce valid for editing the entry?
1824
     * @return boolean
1825
     */
1826
    public function verify_nonce() {
1827
1828
        // Verify form submitted for editing single
1829
        if( $this->is_edit_entry_submission() ) {
1830
            $valid = wp_verify_nonce( $_POST[ self::$nonce_field ], self::$nonce_field );
0 ignored issues
show
introduced by
Detected access of super global var $_POST, probably need manual inspection.
Loading history...
introduced by
Detected usage of a non-validated input variable: $_POST
Loading history...
introduced by
Detected usage of a non-sanitized input variable: $_POST
Loading history...
1831
        }
1832
1833
        // Verify
1834
        else if( ! $this->is_edit_entry() ) {
1835
            $valid = false;
1836
        }
1837
1838
        else {
1839
            $valid = wp_verify_nonce( $_GET['edit'], self::$nonce_key );
0 ignored issues
show
introduced by
Detected access of super global var $_GET, probably need manual inspection.
Loading history...
introduced by
Detected usage of a non-validated input variable: $_GET
Loading history...
introduced by
Detected usage of a non-sanitized input variable: $_GET
Loading history...
1840
        }
1841
1842
        /**
1843
         * @filter `gravityview/edit_entry/verify_nonce` Override Edit Entry nonce validation. Return true to declare nonce valid.
1844
         * @since 1.13
1845
         * @param int|boolean $valid False if invalid; 1 or 2 when nonce was generated
1846
         * @param string $nonce_field Key used when validating submissions. Default: is_gv_edit_entry
1847
         */
1848
        $valid = apply_filters( 'gravityview/edit_entry/verify_nonce', $valid, self::$nonce_field );
1849
1850
        return $valid;
1851
    }
1852
1853
1854
1855
} //end class