GravityView_Edit_Entry_Render

Complexity

Total Complexity

240

Size/Duplication

Total Lines	1885
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	8

Importance

Changes	26
Bugs	10	Features	0

43 Methods

Rating	Name	Duplication	Size	Complexity
A	is_edit_entry()	0	6	4
B	load()	0	26	1
A	prevent_render_form()	0	9	3
A	is_edit_entry_submission()	0	3	1
A	setup_vars()	0	13	1
B	process_save()	0	61	6
A	modify_fileupload_settings()	0	9	2
B	form_prepare_for_save()	0	23	6
D	update_calculation_fields()	0	45	9
D	update_post_image()	0	81	16
A	get_configured_edit_fields()	0	22	2
A	fix_survey_fields()	0	9	2
A	__construct()	0	3	1
A	prevent_maybe_process_form()	0	8	3
B	init()	0	24	4
A	print_scripts()	0	10	1
A	process_save_process_files()	0	10	2
A	render_form_buttons()	0	3	1
A	get_entry()	0	9	2
A	fix_hidden_fields()	0	13	3
B	merge_field_properties()	0	23	4
B	filter_conditional_logic()	0	24	3
A	manage_conditional_logic()	0	8	2
C	user_can_edit_entry()	0	64	14
B	user_can_edit_field()	0	22	4
A	check_user_cap_edit_field()	0	16	4
B	verify_nonce()	0	26	3
B	set_post_categories()	0	30	5
F	maybe_update_post_fields()	0	110	21
A	after_update()	0	18	2
A	edit_entry_form()	0	50	1
B	maybe_print_message()	0	32	3
B	render_edit_form()	0	28	1
B	fix_survey_fields_value()	0	29	6
A	filter_modify_form_fields()	0	18	3
B	verify_user_can_edit_post()	0	30	5
C	modify_edit_field_input()	0	65	12
C	get_field_value()	0	60	19
D	gform_pre_validation()	0	97	15
B	validate()	0	36	3
F	custom_validation()	0	124	21
C	filter_fields()	0	72	13
B	filter_admin_only_fields()	0	28	6

<?php

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.

/**
 * GravityView Edit Entry - render frontend
 *
 * @package   GravityView
 * @license   GPL2+
 * @author    Katz Web Services, Inc.
 * @link      http://gravityview.co
 * @copyright Copyright 2014, Katz Web Services, Inc.
 */

if ( ! defined( 'WPINC' ) ) {
    die;
}


class GravityView_Edit_Entry_Render {

    /**
     * @var GravityView_Edit_Entry
     */
    protected $loader;

	/**
	 * @var string String used to generate unique nonce for the entry/form/view combination. Allows access to edit page.
	 */
    static $nonce_key;

The visibility should be declared for property $nonce_key.

	/**
	 * @since 1.9
	 * @var string String used for check valid edit entry form submission. Allows saving edit form values.
	 */
	private static $nonce_field = 'is_gv_edit_entry';

	/**
	 * @since 1.9
	 * @var bool Whether to allow save and continue functionality
	 */
	private static $supports_save_and_continue = false;

	/**
	 * @since 1.9
	 * @var bool Whether to allow editing product fields
	 */
	private static $supports_product_fields = false;

    /**
     * Gravity Forms entry array
     *
     * @var array
     */
    var $entry;

The visibility should be declared for property $entry.

    /**
     * Gravity Forms form array (GravityView modifies the content through this class lifecycle)
     *
     * @var array
     */
    var $form;

The visibility should be declared for property $form.

    /**
     * Gravity Forms form array (it won't get changed during this class lifecycle)
     * @since 1.16.2.1
     * @var array
     */
    var $original_form;

The visibility should be declared for property $original_form.

    /**
     * Gravity Forms form array after the form validation process
     * @since 1.13
     * @var array
     */
    var $form_after_validation = null;

The visibility should be declared for property $form_after_validation.

    /**
     * Hold an array of GF field objects that have calculation rules
     * @var array
     */
    var $fields_with_calculation = array();

The visibility should be declared for property $fields_with_calculation.

    /**
     * Hold an array of GF field objects with type 'total'
     * @var array
     */
    var $total_fields = array();

The visibility should be declared for property $total_fields.

    /**
     * Gravity Forms form id
     *
     * @var int
     */
    var $form_id;

The visibility should be declared for property $form_id.

    /**
     * ID of the current view
     *
     * @var int
     */
    var $view_id;

The visibility should be declared for property $view_id.

    /**
     * Updated entry is valid (GF Validation object)
     *
     * @var array
     */
    var $is_valid = NULL;

The visibility should be declared for property $is_valid.

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

    function __construct( GravityView_Edit_Entry $loader ) {

It is generally recommended to explicitly declare the visibility for methods.

        $this->loader = $loader;
    }

    function load() {

It is generally recommended to explicitly declare the visibility for methods.

        /** @define "GRAVITYVIEW_DIR" "../../../" */
        include_once( GRAVITYVIEW_DIR .'includes/class-admin-approve-entries.php' );

        // Don't display an embedded form when editing an entry
        add_action( 'wp_head', array( $this, 'prevent_render_form' ) );
        add_action( 'wp_footer', array( $this, 'prevent_render_form' ) );

        // Stop Gravity Forms processing what is ours!
        add_filter( 'wp', array( $this, 'prevent_maybe_process_form'), 8 );

No space before closing parenthesis of array is bad style

        add_filter( 'gravityview_is_edit_entry', array( $this, 'is_edit_entry') );

No space before closing parenthesis of array is bad style

        add_action( 'gravityview_edit_entry', array( $this, 'init' ) );

        // Disable conditional logic if needed (since 1.9)
        add_filter( 'gform_has_conditional_logic', array( $this, 'manage_conditional_logic' ), 10, 2 );

        // Make sure GF doesn't validate max files (since 1.9)
        add_filter( 'gform_plupload_settings', array( $this, 'modify_fileupload_settings' ), 10, 3 );

        // Add fields expected by GFFormDisplay::validate()
        add_filter( 'gform_pre_validation', array( $this, 'gform_pre_validation') );

No space before closing parenthesis of array is bad style

    }

    /**
     * Don't show any forms embedded on a page when GravityView is in Edit Entry mode
     *
     * Adds a `__return_empty_string` filter on the Gravity Forms shortcode on the `wp_head` action
     * And then removes it on the `wp_footer` action
     *
     * @since 1.16.1
     *
     * @return void
     */
    function prevent_render_form() {

It is generally recommended to explicitly declare the visibility for methods.

        if( $this->is_edit_entry() ) {
            if( 'wp_head' === current_filter() ) {
                add_filter( 'gform_shortcode_form', '__return_empty_string' );
            } else {
                remove_filter( 'gform_shortcode_form', '__return_empty_string' );
            }
        }
    }

    /**
     * Because we're mimicking being a front-end Gravity Forms form while using a Gravity Forms
     * backend form, we need to prevent them from saving twice.
     * @return void
     */
    function prevent_maybe_process_form() {

It is generally recommended to explicitly declare the visibility for methods.

        do_action('gravityview_log_debug', 'GravityView_Edit_Entry[prevent_maybe_process_form] $_POSTed data (sanitized): ', esc_html( print_r( $_POST, true ) ) );

Expected 1 spaces after opening bracket; 0 found

The use of function print_r() is discouraged

Detected access of super global var $_POST, probably need manual inspection.

        if( $this->is_edit_entry_submission() && $this->verify_nonce() ) {
            remove_action( 'wp',  array( 'RGForms', 'maybe_process_form'), 9 );

No space before closing parenthesis of array is bad style

        }
    }

    /**
     * Is the current page an Edit Entry page?
     * @return boolean
     */
    public function is_edit_entry() {

        $gf_page = function_exists('rgpost') && ( 'entry' === rgget( 'view' ) && isset( $_GET['edit'] ) || rgpost( 'action' ) === 'update' );

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

Detected access of super global var $_GET, probably need manual inspection.

        return $gf_page;
    }

	/**
	 * Is the current page an Edit Entry page?
	 * @since 1.9
	 * @return boolean
	 */
	public function is_edit_entry_submission() {
		return !empty( $_POST[ self::$nonce_field ] );

Expected 1 space after "!"; 0 found

Detected access of super global var $_POST, probably need manual inspection.

	}

    /**
     * When Edit entry view is requested setup the vars
     */
    function setup_vars() {

It is generally recommended to explicitly declare the visibility for methods.

        $gravityview_view = GravityView_View::getInstance();

Functions must not contain multiple empty lines in a row; found 2 empty lines

        $entries = $gravityview_view->getEntries();
        $this->entry = $entries[0];

        $this->original_form = $this->form = $gravityview_view->getForm();
        $this->form_id = $gravityview_view->getFormId();
        $this->view_id = $gravityview_view->getViewId();

        self::$nonce_key = GravityView_Edit_Entry::get_nonce_key( $this->view_id, $this->form_id, $this->entry['id'] );
    }


    /**
     * Load required files and trigger edit flow
     *
     * Run when the is_edit_entry returns true.
     *
     * @param GravityView_View_Data $gv_data GravityView Data object
     * @return void
     */
    function init( $gv_data ) {

It is generally recommended to explicitly declare the visibility for methods.

        require_once( GFCommon::get_base_path() . '/form_display.php' );
        require_once( GFCommon::get_base_path() . '/entry_detail.php' );

        $this->setup_vars();

        // Multiple Views embedded, don't proceed if nonce fails
        if( $gv_data->has_multiple_views() && ! wp_verify_nonce( $_GET['edit'], self::$nonce_key ) ) {

Detected access of super global var $_GET, probably need manual inspection.

Detected usage of a non-validated input variable: $_GET

Detected usage of a non-sanitized input variable: $_GET

            return;
        }

        // Sorry, you're not allowed here.
        if( false === $this->user_can_edit_entry( true ) ) {
            return;
        }

        $this->print_scripts();

        $this->process_save();

        $this->edit_entry_form();

    }


    /**
     * Force Gravity Forms to output scripts as if it were in the admin
     * @return void
     */
    function print_scripts() {

It is generally recommended to explicitly declare the visibility for methods.

        $gravityview_view = GravityView_View::getInstance();

        wp_register_script( 'gform_gravityforms', GFCommon::get_base_url().'/js/gravityforms.js', array( 'jquery', 'gform_json', 'gform_placeholder', 'sack', 'plupload-all', 'gravityview-fe-view' ) );

        GFFormDisplay::enqueue_form_scripts($gravityview_view->getForm(), false);

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

        // Sack is required for images
        wp_print_scripts( array( 'sack', 'gform_gravityforms' ) );
    }


    /**
     * Process edit entry form save
     */
    function process_save() {

It is generally recommended to explicitly declare the visibility for methods.

        if( empty( $_POST ) ) {

Detected access of super global var $_POST, probably need manual inspection.

            return;
        }

        // Make sure the entry, view, and form IDs are all correct
        $valid = $this->verify_nonce();

        if( !$valid ) {

Expected 1 space after "!"; 0 found

            do_action('gravityview_log_error', __METHOD__ . ' Nonce validation failed.' );

Expected 1 spaces after opening bracket; 0 found

            return;
        }

        if( $this->entry['id'] !== $_POST['lid'] ) {

Detected access of super global var $_POST, probably need manual inspection.

Detected usage of a non-validated input variable: $_POST

Detected usage of a non-sanitized input variable: $_POST

            do_action('gravityview_log_error', __METHOD__ . ' Entry ID did not match posted entry ID.' );

Expected 1 spaces after opening bracket; 0 found

            return;
        }

        do_action('gravityview_log_debug', 'GravityView_Edit_Entry[process_save] $_POSTed data (sanitized): ', esc_html( print_r( $_POST, true ) ) );

Expected 1 spaces after opening bracket; 0 found

The use of function print_r() is discouraged

Detected access of super global var $_POST, probably need manual inspection.

        $this->process_save_process_files( $this->form_id );

        $this->validate();

        if( $this->is_valid ) {

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.

            do_action('gravityview_log_debug', 'GravityView_Edit_Entry[process_save] Submission is valid.' );

Expected 1 spaces after opening bracket; 0 found

            /**
             * @hack This step is needed to unset the adminOnly from form fields, to add the calculation fields
             */
            $form = $this->form_prepare_for_save();

            /**
             * @hack to avoid the capability validation of the method save_lead for GF 1.9+
             */
            unset( $_GET['page'] );

Detected access of super global var $_GET, probably need manual inspection.

            GFFormsModel::save_lead( $form, $this->entry );

            // If there's a post associated with the entry, process post fields
            if( !empty( $this->entry['post_id'] ) ) {

Expected 1 space after "!"; 0 found

                $this->maybe_update_post_fields( $form );
            }

            // Process calculation fields
            $this->update_calculation_fields();

            // Perform actions normally performed after updating a lead
            $this->after_update();

            /**
             * @action `gravityview/edit_entry/after_update` Perform an action after the entry has been updated using Edit Entry
             * @param array $form Gravity Forms form array
             * @param string $entry_id Numeric ID of the entry that was updated
             */
            do_action( 'gravityview/edit_entry/after_update', $this->form, $this->entry['id'] );
        }

    } // process_save


    /**
     * Have GF handle file uploads
     *
     * Copy of code from GFFormDisplay::process_form()
     *
     * @param int $form_id
     */
    function process_save_process_files( $form_id ) {

It is generally recommended to explicitly declare the visibility for methods.

        //Loading files that have been uploaded to temp folder
        $files = GFCommon::json_decode( stripslashes( RGForms::post( 'gform_uploaded_files' ) ) );
        if ( ! is_array( $files ) ) {
            $files = array();
        }

        RGFormsModel::$uploaded_files[ $form_id ] = $files;
    }

    /**
     * Remove max_files validation (done on gravityforms.js) to avoid conflicts with GravityView
     * Late validation done on self::custom_validation
     *
     * @param $plupload_init array Plupload settings
     * @param $form_id
     * @param $instance
     * @return mixed
     */
    public function modify_fileupload_settings( $plupload_init, $form_id, $instance ) {

The parameter $form_id is not used and could be removed.

The parameter $instance is not used and could be removed.

        if( ! $this->is_edit_entry() ) {
            return $plupload_init;
        }

        $plupload_init['gf_vars']['max_files'] = 0;

        return $plupload_init;
    }


    /**
     * Unset adminOnly and convert field input key to string
     * @return array $form
     */
    private function form_prepare_for_save() {

        $form = $this->form;

        foreach( $form['fields'] as $k => &$field ) {

            // Remove the fields with calculation formulas before save to avoid conflicts with GF logic
            // @since 1.16.3
            if( $field->has_calculation() ) {
                unset( $form['fields'][ $k ] );
            }

            $field->adminOnly = false;

            if( isset( $field->inputs ) && is_array( $field->inputs ) ) {
                foreach( $field->inputs as $key => $input ) {
                    $field->inputs[ $key ][ 'id' ] = (string)$input['id'];

Array keys should NOT be surrounded by spaces if they only contain a string or an integer.

No space after closing casting parenthesis is prohibited

                }
            }
        }

        return $form;
    }

    private function update_calculation_fields() {

        $form = $this->original_form;
        $update = false;

        // get the most up to date entry values
        $entry = GFAPI::get_entry( $this->entry['id'] );

        if( !empty( $this->fields_with_calculation ) ) {

Expected 1 space after "!"; 0 found

            $update = true;
            foreach ( $this->fields_with_calculation as $calc_field ) {
                $inputs = $calc_field->get_entry_inputs();
                if ( is_array( $inputs ) ) {
                    foreach ( $inputs as $input ) {
                        $input_name = 'input_' . str_replace( '.', '_', $input['id'] );
                        $entry[ strval( $input['id'] ) ] = RGFormsModel::prepare_value( $form, $calc_field, '', $input_name, $entry['id'], $entry );
                    }
                } else {
                    $input_name = 'input_' . str_replace( '.', '_', $calc_field->id);

Expected 1 spaces before closing bracket; 0 found

                    $entry[ strval( $calc_field->id ) ] = RGFormsModel::prepare_value( $form, $calc_field, '', $input_name, $entry['id'], $entry );
                }
            }

Blank line found after control structure

        }

        //saving total field as the last field of the form.
        if ( ! empty( $this->total_fields ) ) {
            $update = true;
            foreach ( $this->total_fields as $total_field ) {
                $input_name = 'input_' . str_replace( '.', '_', $total_field->id);

Expected 1 spaces before closing bracket; 0 found

                $entry[ strval( $total_field->id ) ] = RGFormsModel::prepare_value( $form, $total_field, '', $input_name, $entry['id'], $entry );
            }
        }

        if( $update ) {

            $return_entry = GFAPI::update_entry( $entry );

            if( is_wp_error( $return_entry ) ) {
                do_action( 'gravityview_log_error', 'Updating the entry calculation and total fields failed', $return_entry );
            } else {
                do_action( 'gravityview_log_debug', 'Updating the entry calculation and total fields succeeded' );
            }
        }
    }

    /**
     * Update the post categories based on all post category fields
     *
     * @since 1.17
     *
     * @param WP_Post &$updated_post Post to be updated (passed by reference)
     * @param array $form Form to check post fields
     * @param array $entry
     *
     * @return mixed
     */
    private function set_post_categories( &$updated_post, $form, $entry ) {

The parameter $entry is not used and could be removed.

        $post_category_fields = GFAPI::get_fields_by_type( $form, 'post_category' );
        
        /**
         * @filter `gravityview/edit_entry/post_content/append_categories` Should post categories be added to or replaced?
         * @since 1.17
         * @param bool $append If `true`, don't delete existing categories, just add on. If `false`, replace the categories with the submitted categories. Default: `false`
         */
        $append = apply_filters( 'gravityview/edit_entry/post_content/append_categories', false );

        $updated_categories = array();

        if( $append ) {
            $updated_categories = wp_get_post_categories( $updated_post->ID );
        }

        if( $post_category_fields ) {

            foreach ( $post_category_fields as $field ) {
                // Get the value of the field, including $_POSTed value
                $field_cats = RGFormsModel::get_field_value( $field );
                $field_cats = is_array( $field_cats ) ? array_values( $field_cats ) : (array)$field_cats;

No space after closing casting parenthesis is prohibited

                $field_cats = gv_map_deep( $field_cats, 'intval' );
                $updated_categories = array_merge( $updated_categories, array_values( $field_cats ) );
            }
        }

        $updated_post->post_category = $updated_categories;
    }

    /**
     * Handle updating the Post Image field
     *
     * Sets a new Featured Image if configured in Gravity Forms; otherwise uploads/updates media
     *
     * @since 1.17
     *
     * @uses GFFormsModel::media_handle_upload
     * @uses set_post_thumbnail
     * 
     * @param array $form GF Form array
     * @param GF_Field $field GF Field
     * @param string $field_id Numeric ID of the field
     * @param string $value
     * @param array $entry GF Entry currently being edited
     * @param int $post_id ID of the Post being edited
     *
     * @return mixed|string
     */
    private function update_post_image( $form, $field, $field_id, $value, $entry, $post_id ) {

        $input_name = 'input_' . $field_id;

        if ( !empty( $_FILES[ $input_name ]['name'] ) ) {

Expected 1 space after "!"; 0 found

            // We have a new image

            $value = RGFormsModel::prepare_value( $form, $field, $value, $input_name, $entry['id'] );

            $ary = ! empty( $value ) ? explode( '|:|', $value ) : array();
            $img_url = rgar( $ary, 0 );

            $img_title       = count( $ary ) > 1 ? $ary[1] : '';
            $img_caption     = count( $ary ) > 2 ? $ary[2] : '';
            $img_description = count( $ary ) > 3 ? $ary[3] : '';

            $image_meta = array(
                'post_excerpt' => $img_caption,
                'post_content' => $img_description,
            );

            //adding title only if it is not empty. It will default to the file name if it is not in the array
            if ( ! empty( $img_title ) ) {
                $image_meta['post_title'] = $img_title;
            }

            /**
             * 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.
             * Note: the method became public in GF 1.9.17.7, but we don't require that version yet.
             */
            require_once GRAVITYVIEW_DIR . 'includes/class-gravityview-gfformsmodel.php';
            $media_id = GravityView_GFFormsModel::media_handle_upload( $img_url, $post_id, $image_meta );

            // is this field set as featured image?
            if ( $media_id && $field->postFeaturedImage ) {
                set_post_thumbnail( $post_id, $media_id );
            }

Blank line found after control structure

        } elseif ( !empty( $_POST[ $input_name ] ) && is_array( $value ) ) {

Expected 1 space after "!"; 0 found

            // Same image although the image title, caption or description might have changed

            $ary = ! empty( $entry[ $field_id ] ) ? explode( '|:|', $entry[ $field_id ] ) : array();
            $img_url = rgar( $ary, 0 );

            // is this really the same image or something went wrong ?
            if( $img_url === $_POST[ $input_name ] ) {

                $img_title       = rgar( $value, $field_id .'.1' );
                $img_caption     = rgar( $value, $field_id .'.4' );
                $img_description = rgar( $value, $field_id .'.7' );

                $value = ! empty( $img_url ) ? $img_url . "|:|" . $img_title . "|:|" . $img_caption . "|:|" . $img_description : '';

The string literal |:| does not require double quotes, as per coding-style, please use single quotes.

                if ( $field->postFeaturedImage ) {

                    $image_meta = array(
                        'ID' => get_post_thumbnail_id( $post_id ),
                        'post_title' => $img_title,
                        'post_excerpt' => $img_caption,
                        'post_content' => $img_description,
                    );

                    // update image title, caption or description
                    wp_update_post( $image_meta );
                }
            }

Blank line found after control structure

        } else {

            // if we get here, image was removed or not set.
            $value = '';

            if ( $field->postFeaturedImage ) {
                delete_post_thumbnail( $post_id );
            }
        }

        return $value;
    }

    /**
     * Loop through the fields being edited and if they include Post fields, update the Entry's post object
     *
     * @param array $form Gravity Forms form
     *
     * @return void
     */
    private function maybe_update_post_fields( $form ) {

        $post_id = $this->entry['post_id'];

        // Security check
        if( false === GVCommon::has_cap( 'edit_post', $post_id ) ) {
            do_action( 'gravityview_log_error', 'The current user does not have the ability to edit Post #'.$post_id );
            return;
        }

        $update_entry = false;

        $updated_post = $original_post = get_post( $post_id );

        // get the most up to date entry values
        $entry = GFAPI::get_entry( $this->entry['id'] );

        foreach ( $entry as $field_id => $value ) {

            //todo: only run through the edit entry configured fields

            $field = RGFormsModel::get_field( $form, $field_id );

            if( class_exists('GF_Fields') ) {

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

                $field = GF_Fields::create( $field );
            }

            if( GFCommon::is_post_field( $field ) ) {

                // Get the value of the field, including $_POSTed value
                $value = RGFormsModel::get_field_value( $field );

                switch( $field->type ) {

                    case 'post_title':
                    case 'post_content':
                    case 'post_excerpt':
                        $updated_post->{$field->type} = $value;
                        break;
                    case 'post_tags':
                        wp_set_post_tags( $post_id, $value, false );
                        break;
                    case 'post_category':
                        break;
                    case 'post_custom_field':

                        $input_type = RGFormsModel::get_input_type( $field );
                        $custom_field_name = $field->postCustomFieldName;

                        // Only certain custom field types are supported
                        switch( $input_type ) {
                            case 'fileupload':
                            /** @noinspection PhpMissingBreakStatementInspection */

Line indented incorrectly; expected at least 8 tabs, found 7

                            case 'list':
                                if( ! is_string( $value ) ) {
                                    $value = function_exists('wp_json_encode') ? wp_json_encode( $value ) : json_encode( $value );

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

                                }
                            // break; left intentionally out

Line indented incorrectly; expected at least 8 tabs, found 7

                            default:
                                update_post_meta( $post_id, $custom_field_name, $value );
                        }

                        break;

                    case 'post_image':
                        $value = $this->update_post_image( $form, $field, $field_id, $value, $entry, $post_id );
                        break;

                }

                //ignore fields that have not changed
                if ( $value === rgget( (string) $field_id, $entry ) ) {
                    continue;
                }

                // update entry
                if( 'post_category' !== $field->type ) {
                    $entry[ strval( $field_id ) ] = $value;
                }

                $update_entry = true;

            }

Blank line found after control structure

        }

        if( $update_entry ) {

            $return_entry = GFAPI::update_entry( $entry );

            if( is_wp_error( $return_entry ) ) {
                do_action( 'gravityview_log_error', 'Updating the entry post fields failed', $return_entry );
            } else {
                do_action( 'gravityview_log_debug', 'Updating the entry post fields for post #'.$post_id.' succeeded' );
            }

Blank line found after control structure

        }

        $this->set_post_categories( $updated_post, $form, $entry );

        $return_post = wp_update_post( $updated_post, true );

        if( is_wp_error( $return_post ) ) {
            $return_post->add_data( $updated_post, '$updated_post' );
            do_action( 'gravityview_log_error', 'Updating the post content failed', $return_post );
        } else {
            do_action( 'gravityview_log_debug', 'Updating the post content for post #'.$post_id.' succeeded', $updated_post );
        }

    }


    /**
     * Perform actions normally performed after updating a lead
     *
     * @since 1.8
     *
     * @see GFEntryDetail::lead_detail_page()
     *
     * @return void
     */
    function after_update() {

It is generally recommended to explicitly declare the visibility for methods.

        do_action( 'gform_after_update_entry', $this->form, $this->entry['id'] );
        do_action( "gform_after_update_entry_{$this->form['id']}", $this->form, $this->entry['id'] );

        // Re-define the entry now that we've updated it.
        $entry = RGFormsModel::get_lead( $this->entry['id'] );

        $entry = GFFormsModel::set_entry_meta( $entry, $this->form );

        // We need to clear the cache because Gravity Forms caches the field values, which
        // we have just updated.
        foreach ($this->form['fields'] as $key => $field) {

No space after opening parenthesis is prohibited

No space before closing parenthesis is prohibited

            GFFormsModel::refresh_lead_field_value( $entry['id'], $field->id );
        }

        $this->entry = $entry;
    }


    /**
     * Display the Edit Entry form
     *
     * @return [type] [description]

The doc-type [type] could not be parsed: Unknown type name "" at position 0. [(view supported doc-types)

     */
    public function edit_entry_form() {

        ?>

        <div class="gv-edit-entry-wrapper"><?php

            $javascript = gravityview_ob_include( GravityView_Edit_Entry::$file .'/partials/inline-javascript.php', $this );

The property file cannot be accessed from this context as it is declared private in class GravityView_Edit_Entry.

            /**
             * Fixes weird wpautop() issue
             * @see https://github.com/katzwebservices/GravityView/issues/451
             */
            echo gravityview_strip_whitespace( $javascript );

Expected a sanitizing function (see Codex for 'Data Validation'), but instead saw 'gravityview_strip_whitespace'

            ?><h2 class="gv-edit-entry-title">
                <span><?php

                    /**
                     * @filter `gravityview_edit_entry_title` Modify the edit entry title
                     * @param string $edit_entry_title Modify the "Edit Entry" title
                     * @param GravityView_Edit_Entry_Render $this This object
                     */
                    $edit_entry_title = apply_filters('gravityview_edit_entry_title', __('Edit Entry', 'gravityview'), $this );

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

                    echo esc_attr( $edit_entry_title );
            ?></span>
            </h2>

            <?php $this->maybe_print_message(); ?>

            <?php // The ID of the form needs to be `gform_{form_id}` for the pluploader ?>

            <form method="post" id="gform_<?php echo $this->form_id; ?>" enctype="multipart/form-data">

Expected next thing to be a escaping function, not '$this'

                <?php

                wp_nonce_field( self::$nonce_key, self::$nonce_key );

                wp_nonce_field( self::$nonce_field, self::$nonce_field, false );

                // Print the actual form HTML
                $this->render_edit_form();

                ?>
            </form>

        </div>

    <?php
    }

    /**
     * Display success or error message if the form has been submitted
     *
     * @uses GVCommon::generate_notice
     *
     * @since 1.16.2.2
     *
     * @return void
     */
    private function maybe_print_message() {

        if( rgpost('action') === 'update' ) {

Found "=== '". Use Yoda Condition checks, you must

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

            $back_link = esc_url( remove_query_arg( array( 'page', 'view', 'edit' ) ) );

            if( ! $this->is_valid ){

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.

                // Keeping this compatible with Gravity Forms.
                $validation_message = "<div class='validation_error'>" . __('There was a problem with your submission.', 'gravityview') . " " . __('Errors have been highlighted below.', 'gravityview') . "</div>";

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

The string literal does not require double quotes, as per coding-style, please use single quotes.

Expected a sanitizing function (see Codex for 'Data Validation'), but instead saw '__'

The string literal </div> does not require double quotes, as per coding-style, please use single quotes.

                $message = apply_filters("gform_validation_message_{$this->form['id']}", apply_filters("gform_validation_message", $validation_message, $this->form), $this->form);

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

The string literal gform_validation_message does not require double quotes, as per coding-style, please use single quotes.

                echo GVCommon::generate_notice( $message , 'gv-error' );

Expected a sanitizing function (see Codex for 'Data Validation'), but instead saw 'GVCommon'

            } else {
                $entry_updated_message = sprintf( esc_attr__('Entry Updated. %sReturn to Entry%s', 'gravityview'), '<a href="'. $back_link .'">', '</a>' );

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

                /**
                 * @filter `gravityview/edit_entry/success` Modify the edit entry success message (including the anchor link)
                 * @since 1.5.4
                 * @param string $entry_updated_message Existing message
                 * @param int $view_id View ID
                 * @param array $entry Gravity Forms entry array
                 * @param string $back_link URL to return to the original entry. @since 1.6
                 */
                $message = apply_filters( 'gravityview/edit_entry/success', $entry_updated_message , $this->view_id, $this->entry, $back_link );

                echo GVCommon::generate_notice( $message );

Expected a sanitizing function (see Codex for 'Data Validation'), but instead saw 'GVCommon'

            }

Blank line found after control structure

        }
    }

    /**
     * Display the Edit Entry form in the original Gravity Forms format
     *
     * @since 1.9
     *
     * @return void
     */
    private function render_edit_form() {

        add_filter( 'gform_pre_render', array( $this, 'filter_modify_form_fields'), 5000, 3 );

No space before closing parenthesis of array is bad style

        add_filter( 'gform_submit_button', array( $this, 'render_form_buttons') );

No space before closing parenthesis of array is bad style

        add_filter( 'gform_disable_view_counter', '__return_true' );

        add_filter( 'gform_field_input', array( $this, 'verify_user_can_edit_post' ), 5, 5 );
        add_filter( 'gform_field_input', array( $this, 'modify_edit_field_input' ), 10, 5 );

        add_filter( 'gform_field_value', array( $this, 'fix_survey_fields_value'), 10, 3 );

No space before closing parenthesis of array is bad style

        // We need to remove the fake $_GET['page'] arg to avoid rendering form as if in admin.
        unset( $_GET['page'] );

Detected access of super global var $_GET, probably need manual inspection.

        // TODO: Verify multiple-page forms
        // TODO: Product fields are not editable

        $html = GFFormDisplay::get_form( $this->form['id'], false, false, true, $this->entry );

        remove_filter( 'gform_field_value', array( $this, 'fix_survey_fields_value'), 10 );

No space before closing parenthesis of array is bad style

	    remove_filter( 'gform_pre_render', array( $this, 'filter_modify_form_fields' ), 5000 );
        remove_filter( 'gform_submit_button', array( $this, 'render_form_buttons' ) );
        remove_filter( 'gform_disable_view_counter', '__return_true' );
        remove_filter( 'gform_field_input', array( $this, 'verify_user_can_edit_post' ), 5 );
        remove_filter( 'gform_field_input', array( $this, 'modify_edit_field_input' ), 10 );

        echo $html;

Expected next thing to be a escaping function, not '$html'

    }

    /**
     * Survey fields inject their output using `gform_field_input` filter, but in Edit Entry, the values were empty.
     * We filter the values here because it was the easiest access point: tell the survey field the correct value, GF outputs it.
     *
     * @since 1.16.4
     *
     * @param string $value Existing value
     * @param GF_Field $field
     * @param string $name Field custom parameter name, normally blank.
     *
     * @return mixed
     */
    function fix_survey_fields_value( $value, $field, $name ) {

It is generally recommended to explicitly declare the visibility for methods.

        
        if( 'survey' === $field->type ) {

	        // We need to run through each survey row until we find a match for expected values
	        foreach ( $this->entry as $field_id => $field_value ) {

		        if ( floor( $field_id ) !== floor( $field->id ) ) {
			        continue;
		        }

		        if( rgar( $field, 'gsurveyLikertEnableMultipleRows' ) ) {
			        list( $row_val, $col_val ) = explode( ':', $field_value, 2 );

		            // If the $name matches the $row_val, we are processing the correct row
			        if( $row_val === $name ) {
				        $value = $field_value;
				        break;
			        }
		        } else {
			        // When not processing multiple rows, the value is the $entry[ $field_id ] value.
			        $value = $field_value;
				    break;
		        }
			}
        }

        return $value;
    }

    /**
     * Display the Update/Cancel/Delete buttons for the Edit Entry form
     * @since 1.8
     * @return string
     */
    public function render_form_buttons() {
        return gravityview_ob_include( GravityView_Edit_Entry::$file .'/partials/form-buttons.php', $this );

The property file cannot be accessed from this context as it is declared private in class GravityView_Edit_Entry.

    }


    /**
     * Modify the form fields that are shown when using GFFormDisplay::get_form()
     *
     * By default, all fields will be shown. We only want the Edit Tab configured fields to be shown.
     *
     * @param array $form
     * @param boolean $ajax Whether in AJAX mode
     * @param array|string $field_values Passed parameters to the form
     *
     * @since 1.9
     *
     * @return array Modified form array
     */
    public function filter_modify_form_fields( $form, $ajax = false, $field_values = '' ) {

The parameter $ajax is not used and could be removed.

The parameter $field_values is not used and could be removed.

        // In case we have validated the form, use it to inject the validation results into the form render
        if( isset( $this->form_after_validation ) ) {
            $form = $this->form_after_validation;
        } else {
            $form['fields'] = $this->get_configured_edit_fields( $form, $this->view_id );
        }

        $form = $this->filter_conditional_logic( $form );

        // for now we don't support Save and Continue feature.
        if( ! self::$supports_save_and_continue ) {
	        unset( $form['save'] );
        }

        return $form;
    }

    /**
     * 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.
     *
     * @since 1.16.2.2
     *
     * @param string $field_content Always empty. Returning not-empty overrides the input.
     * @param GF_Field $field
     * @param string|array $value If array, it's a field with multiple inputs. If string, single input.
     * @param int $lead_id Lead ID. Always 0 for the `gform_field_input` filter.
     * @param int $form_id Form ID
     *
     * @return string If error, the error message. If no error, blank string (modify_edit_field_input() runs next)
     */
    function verify_user_can_edit_post( $field_content = '', $field, $value, $lead_id = 0, $form_id ) {

The parameter $value is not used and could be removed.

The parameter $lead_id is not used and could be removed.

The parameter $form_id is not used and could be removed.

It is generally recommended to explicitly declare the visibility for methods.

        if( GFCommon::is_post_field( $field ) ) {

            $message = null;

            // First, make sure they have the capability to edit the post.
            if( false === current_user_can( 'edit_post', $this->entry['post_id'] ) ) {

                /**
                 * @filter `gravityview/edit_entry/unsupported_post_field_text` Modify the message when someone isn't able to edit a post
                 * @param string $message The existing "You don't have permission..." text
                 */
                $message = apply_filters('gravityview/edit_entry/unsupported_post_field_text', __('You don’t have permission to edit this post.', 'gravityview') );

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

            } elseif( null === get_post( $this->entry['post_id'] ) ) {
                /**
                 * @filter `gravityview/edit_entry/no_post_text` Modify the message when someone is editing an entry attached to a post that no longer exists
                 * @param string $message The existing "This field is not editable; the post no longer exists." text
                 */
                $message = apply_filters('gravityview/edit_entry/no_post_text', __('This field is not editable; the post no longer exists.', 'gravityview' ) );

Expected 1 spaces after opening bracket; 0 found

            }

            if( $message ) {
                $field_content = sprintf('<div class="ginput_container ginput_container_' . $field->type . '">%s</div>', wpautop( $message ) );

Expected 1 spaces after opening bracket; 0 found

            }
        }

        return $field_content;
    }

    /**
     *
     * Fill-in the saved values into the form inputs
     *
     * @param string $field_content Always empty. Returning not-empty overrides the input.
     * @param GF_Field $field
     * @param string|array $value If array, it's a field with multiple inputs. If string, single input.
     * @param int $lead_id Lead ID. Always 0 for the `gform_field_input` filter.
     * @param int $form_id Form ID
     *
     * @return mixed
     */
    function modify_edit_field_input( $field_content = '', $field, $value, $lead_id = 0, $form_id ) {

The parameter $lead_id is not used and could be removed.

The parameter $form_id is not used and could be removed.

It is generally recommended to explicitly declare the visibility for methods.

        $gv_field = GravityView_Fields::get_associated_field( $field );

        // If the form has been submitted, then we don't need to pre-fill the values,
        // Except for fileupload type and when a field input is overridden- run always!!
        if(
            ( $this->is_edit_entry_submission() && !in_array( $field->type, array( 'fileupload', 'post_image' ) ) )

Expected 1 space after "!"; 0 found

            && false === ( $gv_field && is_callable( array( $gv_field, 'get_field_input' ) ) )
            || ! empty( $field_content )
            || GFCommon::is_product_field( $field->type ) // Prevent product fields from appearing editable
        ) {
	        return $field_content;
        }

        // Turn on Admin-style display for file upload fields only
        if( 'fileupload' === $field->type ) {
            $_GET['page'] = 'gf_entries';
        }

        // SET SOME FIELD DEFAULTS TO PREVENT ISSUES
        $field->adminOnly = false; /** @see GFFormDisplay::get_counter_init_script() need to prevent adminOnly */

        // add categories as choices for Post Category field
        if ( 'post_category' === $field->type ) {
            $field = GFCommon::add_categories_as_choices( $field, $value );
        }

        $field_value = $this->get_field_value( $field );

        /**
         * @filter `gravityview/edit_entry/field_value` Change the value of an Edit Entry field, if needed
         * @since 1.11
         * @param mixed $field_value field value used to populate the input
         * @param object $field Gravity Forms field object ( Class GF_Field )
         */
        $field_value = apply_filters( 'gravityview/edit_entry/field_value', $field_value, $field );

	    // Prevent any PHP warnings, like undefined index
	    ob_start();

        if( $gv_field && is_callable( array( $gv_field, 'get_field_input' ) ) ) {
            /** @var GF_Field $gv_field */
            $return = $gv_field->get_field_input( $this->form, $field_value, $this->entry, $field );
        } else {
	        $return = $field->get_field_input( $this->form, $field_value, $this->entry );
        }

Functions must not contain multiple empty lines in a row; found 2 empty lines

	    // If there was output, it's an error
	    $warnings = ob_get_clean();

	    if( !empty( $warnings ) ) {

Expected 1 space after "!"; 0 found

		    do_action( 'gravityview_log_error', __METHOD__ . $warnings, $field_value );
	    }

        /**
         * Unset hack $_GET['page'] = 'gf_entries'
         * We need the fileupload html field to render with the proper id
         *  ( <li id="field_80_16" ... > )
         */
        unset( $_GET['page'] );

Detected access of super global var $_GET, probably need manual inspection.

        return $return;
    }

    /**
     * Modify the value for the current field input
     *
     * @param GF_Field $field
     *
     * @return array|mixed|string|void
     */
    private function get_field_value( $field ) {

        /**
         * @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.
         * @param boolean True: override saved values; False: don't override (default)
         * @param $field GF_Field object Gravity Forms field object
         * @since 1.13
         */
        $override_saved_value = apply_filters( 'gravityview/edit_entry/pre_populate/override', false, $field );

        // We're dealing with multiple inputs (e.g. checkbox) but not time or date (as it doesn't store data in input IDs)
        if( isset( $field->inputs ) && is_array( $field->inputs ) && !in_array( $field->type, array( 'time', 'date' ) ) ) {

Expected 1 space after "!"; 0 found

            $field_value = array();

            // only accept pre-populated values if the field doesn't have any choice selected.
            $allow_pre_populated = $field->allowsPrepopulate;

            foreach ( (array)$field->inputs as $input ) {

No space after closing casting parenthesis is prohibited

                $input_id = strval( $input['id'] );
                
                if ( isset( $this->entry[ $input_id ] ) && ! gv_empty( $this->entry[ $input_id ], false, false ) ) {
                    $field_value[ $input_id ] =  'post_category' === $field->type ? GFCommon::format_post_category( $this->entry[ $input_id ], true ) : $this->entry[ $input_id ];

Expected 1 space after "="; 2 found

                    $allow_pre_populated = false;
                }

Blank line found after control structure

            }

            $pre_value = $field->get_value_submission( array(), false );

            $field_value = ! $allow_pre_populated && ! ( $override_saved_value && !gv_empty( $pre_value, false, false ) ) ? $field_value : $pre_value;

Expected 1 space after "!"; 0 found

        } else {

            $id = intval( $field->id );

            // get pre-populated value if exists
            $pre_value = $field->allowsPrepopulate ? GFFormsModel::get_parameter_value( $field->inputName, array(), $field ) : '';

            // saved field entry value (if empty, fallback to the pre-populated value, if exists)
            // or pre-populated value if not empty and set to override saved value
            $field_value = !gv_empty( $this->entry[ $id ], false, false ) && ! ( $override_saved_value && !gv_empty( $pre_value, false, false ) ) ? $this->entry[ $id ] : $pre_value;

Expected 1 space after "!"; 0 found

            // in case field is post_category but inputType is select, multi-select or radio, convert value into array of category IDs.
            if ( 'post_category' === $field->type && !gv_empty( $field_value, false, false ) ) {

Expected 1 space after "!"; 0 found

                $categories = array();
                foreach ( explode( ',', $field_value ) as $cat_string ) {
                    $categories[] = GFCommon::format_post_category( $cat_string, true );
                }
                $field_value = 'multiselect' === $field->get_input_type() ? $categories : implode( '', $categories );
            }

Blank line found after control structure

        }

        // if value is empty get the default value if defined
        $field_value = $field->get_value_default_if_empty( $field_value );

        return $field_value;
    }


    // ---- Entry validation

    /**
     * Add field keys that Gravity Forms expects.
     *
     * @see GFFormDisplay::validate()
     * @param  array $form GF Form
     * @return array       Modified GF Form
     */
    function gform_pre_validation( $form ) {

It is generally recommended to explicitly declare the visibility for methods.

        if( ! $this->verify_nonce() ) {
            return $form;
        }

        // Fix PHP warning regarding undefined index.
        foreach ( $form['fields'] as &$field) {

No space before closing parenthesis is prohibited

            // This is because we're doing admin form pretending to be front-end, so Gravity Forms
            // expects certain field array items to be set.
            foreach ( array( 'noDuplicates', 'adminOnly', 'inputType', 'isRequired', 'enablePrice', 'inputs', 'allowedExtensions' ) as $key ) {
	            $field->{$key} = isset( $field->{$key} ) ? $field->{$key} : NULL;

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

            }

            // unset emailConfirmEnabled for email type fields
           /* if( 'email' === $field['type'] && !empty( $field['emailConfirmEnabled'] ) ) {

61% of this comment could be valid code. Did you maybe forget this after debugging?

Line indented incorrectly; expected at least 3 tabs, found 2

                $field['emailConfirmEnabled'] = '';
            }*/

            switch( RGFormsModel::get_input_type( $field ) ) {

                /**
                 * 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.
                 *
                 * 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.
                 *
                 * @hack
                 */
                case 'fileupload':

                    // Set the previous value
                    $entry = $this->get_entry();

                    $input_name = 'input_'.$field->id;
                    $form_id = $form['id'];

                    $value = NULL;

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

                    // Use the previous entry value as the default.
                    if( isset( $entry[ $field->id ] ) ) {
                        $value = $entry[ $field->id ];
                    }

                    // If this is a single upload file
                    if( !empty( $_FILES[ $input_name ] ) && !empty( $_FILES[ $input_name ]['name'] ) ) {

Expected 1 space after "!"; 0 found

                        $file_path = GFFormsModel::get_file_upload_path( $form['id'], $_FILES[ $input_name ]['name'] );
                        $value = $file_path['url'];

                    } else {

                        // Fix PHP warning on line 1498 of form_display.php for post_image fields
                        // Fix PHP Notice:  Undefined index:  size in form_display.php on line 1511
                        $_FILES[ $input_name ] = array('name' => '', 'size' => '' );

No space after opening parenthesis of array is bad style

                    }

                    if( rgar($field, "multipleFiles") ) {

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

The string literal multipleFiles does not require double quotes, as per coding-style, please use single quotes.

                        // If there are fresh uploads, process and merge them.
                        // Otherwise, use the passed values, which should be json-encoded array of URLs
                        if( isset( GFFormsModel::$uploaded_files[$form_id][$input_name] ) ) {

Array keys should be surrounded by spaces unless they contain a string or an integer.

                            $value = empty( $value ) ? '[]' : $value;
                            $value = stripslashes_deep( $value );
                            $value = GFFormsModel::prepare_value( $form, $field, $value, $input_name, $entry['id'], array());

Expected 1 spaces before closing bracket; 0 found

                        }

Blank line found after control structure

                    } else {

                        // A file already exists when editing an entry
                        // We set this to solve issue when file upload fields are required.
                        GFFormsModel::$uploaded_files[ $form_id ][ $input_name ] = $value;

                    }

                    $this->entry[ $input_name ] = $value;
                    $_POST[ $input_name ] = $value;

                    break;

                case 'number':
                    // Fix "undefined index" issue at line 1286 in form_display.php
                    if( !isset( $_POST['input_'.$field->id ] ) ) {

Expected 1 space after "!"; 0 found

Array keys should be surrounded by spaces unless they contain a string or an integer.

                        $_POST['input_'.$field->id ] = NULL;

Array keys should be surrounded by spaces unless they contain a string or an integer.

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

                    }
                    break;
                case 'captcha':
                    // Fix issue with recaptcha_check_answer() on line 1458 in form_display.php
                    $_POST['recaptcha_challenge_field'] = NULL;

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

                    $_POST['recaptcha_response_field'] = NULL;

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

                    break;
            }

Blank line found after control structure

        }

        return $form;
    }


    /**
     * Process validation for a edit entry submission
     *
     * Sets the `is_valid` object var
     *
     * @return void
     */
    function validate() {

It is generally recommended to explicitly declare the visibility for methods.

        /**
         * If using GF User Registration Add-on, remove the validation step, otherwise generates error when updating the entry
         * GF User Registration Add-on version > 3.x has a different class name
         * @since 1.16.2
         */
        if ( class_exists( 'GF_User_Registration' ) ) {
            remove_filter( 'gform_validation', array( GF_User_Registration::get_instance(), 'validate' ) );
        } else  if ( class_exists( 'GFUser' ) ) {
            remove_filter( 'gform_validation', array( 'GFUser', 'user_registration_validation' ) );
        }

Functions must not contain multiple empty lines in a row; found 2 empty lines

        /**
         * For some crazy reason, Gravity Forms doesn't validate Edit Entry form submissions.
         * You can enter whatever you want!
         * We try validating, and customize the results using `self::custom_validation()`
         */
        add_filter( 'gform_validation_'. $this->form_id, array( $this, 'custom_validation' ), 10, 4);

Expected 1 spaces before closing bracket; 0 found

        // Needed by the validate funtion
        $failed_validation_page = NULL;

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

        $field_values = RGForms::post( 'gform_field_values' );

        // Prevent entry limit from running when editing an entry, also
        // prevent form scheduling from preventing editing
        unset( $this->form['limitEntries'], $this->form['scheduleForm'] );

        // Hide fields depending on Edit Entry settings
        $this->form['fields'] = $this->get_configured_edit_fields( $this->form, $this->view_id );

        $this->is_valid = GFFormDisplay::validate( $this->form, $field_values, 1, $failed_validation_page );

        remove_filter( 'gform_validation_'. $this->form_id, array( $this, 'custom_validation' ), 10 );
    }


    /**
     * Make validation work for Edit Entry
     *
     * Because we're calling the GFFormDisplay::validate() in an unusual way (as a front-end
     * form pretending to be a back-end form), validate() doesn't know we _can't_ edit post
     * fields. This goes through all the fields and if they're an invalid post field, we
     * set them as valid. If there are still issues, we'll return false.
     *
     * @param  [type] $validation_results [description]

The doc-type [type] could not be parsed: Unknown type name "" at position 0. [(view supported doc-types)

     * @return [type]                     [description]

The doc-type [type] could not be parsed: Unknown type name "" at position 0. [(view supported doc-types)

     */
    function custom_validation( $validation_results ) {

It is generally recommended to explicitly declare the visibility for methods.

        do_action('gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Validation results: ', $validation_results );

Expected 1 spaces after opening bracket; 0 found

        do_action('gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] $_POSTed data (sanitized): ', esc_html( print_r( $_POST, true ) ) );

Expected 1 spaces after opening bracket; 0 found

The use of function print_r() is discouraged

Detected access of super global var $_POST, probably need manual inspection.

        $gv_valid = true;

        foreach ( $validation_results['form']['fields'] as $key => &$field ) {

            $value = RGFormsModel::get_field_value( $field );
            $field_type = RGFormsModel::get_input_type( $field );

            // Validate always
            switch ( $field_type ) {

Functions must not contain multiple empty lines in a row; found 2 empty lines

                case 'fileupload' :
                case 'post_image':

                    // in case nothing is uploaded but there are already files saved
                    if( !empty( $field->failed_validation ) && !empty( $field->isRequired ) && !empty( $value ) ) {

Expected 1 space after "!"; 0 found

                        $field->failed_validation = false;
                        unset( $field->validation_message );
                    }

                    // validate if multi file upload reached max number of files [maxFiles] => 2
                    if( rgobj( $field, 'maxFiles') && rgobj( $field, 'multipleFiles') ) {

Expected 1 spaces before closing bracket; 0 found

                        $input_name = 'input_' . $field->id;
                        //uploaded
                        $file_names = isset( GFFormsModel::$uploaded_files[ $validation_results['form']['id'] ][ $input_name ] ) ? GFFormsModel::$uploaded_files[ $validation_results['form']['id'] ][ $input_name ] : array();

                        //existent
                        $entry = $this->get_entry();
                        $value = NULL;

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

                        if( isset( $entry[ $field->id ] ) ) {
                            $value = json_decode( $entry[ $field->id ], true );
                        }

                        // count uploaded files and existent entry files
                        $count_files = count( $file_names ) + count( $value );

                        if( $count_files > $field->maxFiles ) {
                            $field->validation_message = __( 'Maximum number of files reached', 'gravityview' );
                            $field->failed_validation = 1;
                            $gv_valid = false;

                            // in case of error make sure the newest upload files are removed from the upload input
                            GFFormsModel::$uploaded_files[ $validation_results['form']['id'] ] = null;
                        }

Blank line found after control structure

                    }

Functions must not contain multiple empty lines in a row; found 2 empty lines

                    break;

            }

            // This field has failed validation.
            if( !empty( $field->failed_validation ) ) {

Expected 1 space after "!"; 0 found

                do_action( 'gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Field is invalid.', array( 'field' => $field, 'value' => $value ) );

                switch ( $field_type ) {

                    // Captchas don't need to be re-entered.
                    case 'captcha':

                        // Post Image fields aren't editable, so we un-fail them.
                    case 'post_image':
                        $field->failed_validation = false;
                        unset( $field->validation_message );
                        break;

                }

                // You can't continue inside a switch, so we do it after.
                if( empty( $field->failed_validation ) ) {
                    continue;
                }

                // checks if the No Duplicates option is not validating entry against itself, since
                // we're editing a stored entry, it would also assume it's a duplicate.
                if( !empty( $field->noDuplicates ) ) {

Expected 1 space after "!"; 0 found

                    $entry = $this->get_entry();

                    // If the value of the entry is the same as the stored value
                    // Then we can assume it's not a duplicate, it's the same.
                    if( !empty( $entry ) && $value == $entry[ $field->id ] ) {

Expected 1 space after "!"; 0 found

                        //if value submitted was not changed, then don't validate
                        $field->failed_validation = false;

                        unset( $field->validation_message );

                        do_action('gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Field not a duplicate; it is the same entry.', $entry );

Expected 1 spaces after opening bracket; 0 found

                        continue;
                    }
                }

                // if here then probably we are facing the validation 'At least one field must be filled out'
                if( GFFormDisplay::is_empty( $field, $this->form_id  ) && empty( $field->isRequired ) ) {

Expected 1 spaces before closing bracket; 2 found

                    unset( $field->validation_message );
	                $field->validation_message = false;
                    continue;
                }

                $gv_valid = false;

            }

Blank line found after control structure

        }

        $validation_results['is_valid'] = $gv_valid;

        do_action('gravityview_log_debug', 'GravityView_Edit_Entry[custom_validation] Validation results.', $validation_results );

Expected 1 spaces after opening bracket; 0 found

        // We'll need this result when rendering the form ( on GFFormDisplay::get_form )
        $this->form_after_validation = $validation_results['form'];

        return $validation_results;
    }


    /**
     * TODO: This seems to be hacky... we should remove it. Entry is set when updating the form using setup_vars()!
     * Get the current entry and set it if it's not yet set.
     * @return array Gravity Forms entry array
     */
    private function get_entry() {

        if( empty( $this->entry ) ) {
            // Get the database value of the entry that's being edited
            $this->entry = gravityview_get_entry( GravityView_frontend::is_single_entry() );
        }

        return $this->entry;
    }



    // --- Filters

    /**
     * Get the Edit Entry fields as configured in the View
     *
     * @since 1.8
     *
     * @param int $view_id
     *
     * @return array Array of fields that are configured in the Edit tab in the Admin
     */
    private function get_configured_edit_fields( $form, $view_id ) {

        // Get all fields for form
        $properties = GravityView_View_Data::getInstance()->get_fields( $view_id );

        // If edit tab not yet configured, show all fields
        $edit_fields = !empty( $properties['edit_edit-fields'] ) ? $properties['edit_edit-fields'] : NULL;

Expected 1 space after "!"; 0 found

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

	    // Show hidden fields as text fields
	    $form = $this->fix_hidden_fields( $form );

        // Show hidden fields as text fields
        $form = $this->fix_survey_fields( $form );

        // Hide fields depending on admin settings
        $fields = $this->filter_fields( $form['fields'], $edit_fields );

	    // If Edit Entry fields are configured, remove adminOnly field settings. Otherwise, don't.
	    $fields = $this->filter_admin_only_fields( $fields, $edit_fields, $form, $view_id );

        return $fields;
    }

    /**
     * Make sure Survey fields accept pre-populating values; otherwise existing values won't be filled-in
     *
     * @since 1.16.4
     *
     * @param array $form
     *
     * @return array Form, with all fields set to `allowsPrepopulate => true`
     */
    private function fix_survey_fields( $form ) {

        /** @var GF_Field $field */
        foreach( $form['fields'] as &$field ) {
            $field->allowsPrepopulate = true;
        }

        return $form;
    }

	/**
	 * @since 1.9.2
	 *
	 * @param $fields
	 *
	 * @return mixed
	 */
	private function fix_hidden_fields( $form ) {

		/** @var GF_Field $field */
		foreach( $form['fields'] as $key => $field ) {
			if( 'hidden' === $field->type ) {
				$text_field = new GF_Field_Text( $field );
				$text_field->type = 'text';
				$form['fields'][ $key ] = $text_field;
			}
		}

		return $form;
	}


    /**
     * Filter area fields based on specified conditions
     *  - This filter removes the fields that have calculation configured
     *
     * @uses GravityView_Edit_Entry::user_can_edit_field() Check caps
     * @access private
     * @param GF_Field[] $fields
     * @param array $configured_fields
     * @since  1.5
     * @return array $fields
     */
    private function filter_fields( $fields, $configured_fields ) {

        if( empty( $fields ) || !is_array( $fields ) ) {

Expected 1 space after "!"; 0 found

            return $fields;
        }

        $edit_fields = array();

        $field_type_blacklist = array(
            'page',
        );

	    /**
	     * @filter `gravityview/edit_entry/hide-product-fields` Hide product fields from being editable.
	     * @since 1.9.1
         * @param boolean $hide_product_fields Whether to hide product fields in the editor.  Default: false
	     */
	    $hide_product_fields = apply_filters( 'gravityview/edit_entry/hide-product-fields', empty( self::$supports_product_fields ) );

	    if( $hide_product_fields ) {
		    $field_type_blacklist[] = 'option';
		    $field_type_blacklist[] = 'quantity';
            $field_type_blacklist[] = 'product';
            $field_type_blacklist[] = 'total';
            $field_type_blacklist[] = 'shipping';
            $field_type_blacklist[] = 'calculation';
	    }

        // First, remove blacklist or calculation fields
        foreach ( $fields as $key => $field ) {

            // Remove the fields that have calculation properties and keep them to be used later
            // @since 1.16.2
            if( $field->has_calculation() ) {
                $this->fields_with_calculation[] = $field;
                // don't remove the calculation fields on form render.
            }

            // process total field after all fields have been saved
            if ( $field->type == 'total' ) {

Found "== '". Use Yoda Condition checks, you must

                $this->total_fields[] = $field;
                unset( $fields[ $key ] );
            }

            if( in_array( $field->type, $field_type_blacklist ) ) {
                unset( $fields[ $key ] );
            }
        }

        // The Edit tab has not been configured, so we return all fields by default.
        if( empty( $configured_fields ) ) {
            return $fields;
        }

        // The edit tab has been configured, so we loop through to configured settings
        foreach ( $configured_fields as $configured_field ) {

	        /** @var GF_Field $field */
	        foreach ( $fields as $field ) {

                if( intval( $configured_field['id'] ) === intval( $field->id ) && $this->user_can_edit_field( $configured_field, false ) ) {
                    $edit_fields[] = $this->merge_field_properties( $field, $configured_field );
                    break;
                }

Blank line found after control structure

            }

Blank line found after control structure

        }

        return $edit_fields;

    }

    /**
     * Override GF Form field properties with the ones defined on the View
     * @param  GF_Field $field GF Form field object
     * @param  array $setting  GV field options

There is no parameter named $setting. Did you maybe mean $field_setting?

     * @since  1.5
     * @return array
     */
    private function merge_field_properties( $field, $field_setting ) {

        $return_field = $field;

        if( empty( $field_setting['show_label'] ) ) {
            $return_field->label = '';
        } elseif ( !empty( $field_setting['custom_label'] ) ) {

Expected 1 space after "!"; 0 found

            $return_field->label = $field_setting['custom_label'];
        }

        if( !empty( $field_setting['custom_class'] ) ) {

Expected 1 space after "!"; 0 found

            $return_field->cssClass .= ' '. gravityview_sanitize_html_class( $field_setting['custom_class'] );
        }

        /**
         * Normalize page numbers - avoid conflicts with page validation
         * @since 1.6
         */
        $return_field->pageNumber = 1;

        return $return_field;

    }

    /**
     * Remove fields that shouldn't be visible based on the Gravity Forms adminOnly field property
     *
     * @since 1.9.1
     *
     * @param array|GF_Field[] $fields Gravity Forms form fields
     * @param array|null $edit_fields Fields for the Edit Entry tab configured in the View Configuration
     * @param array $form GF Form array
     * @param int $view_id View ID
     *
     * @return array Possibly modified form array
     */
    function filter_admin_only_fields( $fields = array(), $edit_fields = null, $form = array(), $view_id = 0 ) {

It is generally recommended to explicitly declare the visibility for methods.

	    /**
         * @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
	     * If the Edit Entry tab is not configured, adminOnly fields will not be shown to non-administrators.
	     * If the Edit Entry tab *is* configured, adminOnly fields will be shown to non-administrators, using the configured GV permissions
	     * @since 1.9.1
	     * @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.
	     * @param array $form GF Form array
	     * @param int $view_id View ID
	     */
	    $use_gf_adminonly_setting = apply_filters( 'gravityview/edit_entry/use_gf_admin_only_setting', empty( $edit_fields ), $form, $view_id );

	    if( $use_gf_adminonly_setting && false === GVCommon::has_cap( 'gravityforms_edit_entries', $this->entry['id'] ) ) {
            foreach( $fields as $k => $field ) {
                if( $field->adminOnly ) {
                    unset( $fields[ $k ] );
                }
            }
            return $fields;
        }

	    foreach( $fields as &$field ) {
		    $field->adminOnly = false;
        }

        return $fields;
    }

    // --- Conditional Logic

    /**
     * Remove the conditional logic rules from the form button and the form fields, if needed.
     *
     * @since 1.9
     *
     * @param array $form Gravity Forms form
     * @return array Modified form, if not using Conditional Logic
     */
    function filter_conditional_logic( $form ) {

It is generally recommended to explicitly declare the visibility for methods.

        /**
         * @filter `gravityview/edit_entry/conditional_logic` Should the Edit Entry form use Gravity Forms conditional logic showing/hiding of fields?
         * @since 1.9
         * @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
         * @param array $form Gravity Forms form
         */
        $use_conditional_logic = apply_filters( 'gravityview/edit_entry/conditional_logic', true, $form );

        if( $use_conditional_logic ) {
            return $form;
        }

        foreach( $form['fields'] as &$field ) {
            /* @var GF_Field $field */
            $field->conditionalLogic = null;
        }

        unset( $form['button']['conditionalLogic'] );

        return $form;

    }

    /**
     * Disable the Gravity Forms conditional logic script and features on the Edit Entry screen
     *
     * @since 1.9
     *
     * @param $has_conditional_logic
     * @param $form
     * @return mixed|void
     */
    function manage_conditional_logic( $has_conditional_logic, $form ) {

It is generally recommended to explicitly declare the visibility for methods.

        if( ! $this->is_edit_entry() ) {
            return $has_conditional_logic;
        }

        return apply_filters( 'gravityview/edit_entry/conditional_logic', $has_conditional_logic, $form );
    }


    // --- User checks and nonces

    /**
     * Check if the user can edit the entry
     *
     * - Is the nonce valid?
     * - Does the user have the right caps for the entry
     * - Is the entry in the trash?
     *
     * @todo Move to GVCommon
     *
     * @param  boolean $echo Show error messages in the form?
     * @return boolean        True: can edit form. False: nope.
     */
    function user_can_edit_entry( $echo = false ) {

It is generally recommended to explicitly declare the visibility for methods.

        $error = NULL;

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

        /**
         *  1. Permalinks are turned off
         *  2. There are two entries embedded using oEmbed
         *  3. One of the entries has just been saved
         */
        if( !empty( $_POST['lid'] ) && !empty( $_GET['entry'] ) && ( $_POST['lid'] !== $_GET['entry'] ) ) {

Expected 1 space after "!"; 0 found

            $error = true;

        }

        if( !empty( $_GET['entry'] ) && (string)$this->entry['id'] !== $_GET['entry'] ) {

Expected 1 space after "!"; 0 found

No space after closing casting parenthesis is prohibited

            $error = true;

        } elseif( ! $this->verify_nonce() ) {

            /**
             * If the Entry is embedded, there may be two entries on the same page.
             * If that's the case, and one is being edited, the other should fail gracefully and not display an error.
             */
            if( GravityView_oEmbed::getInstance()->get_entry_id() ) {

The expression \GravityView_oEmbed::getInstance()->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.

                $error = true;
            } else {
                $error = __( 'The link to edit this entry is not valid; it may have expired.', 'gravityview');

Expected 1 spaces before closing bracket; 0 found

            }

Blank line found after control structure

        }

        if( ! GravityView_Edit_Entry::check_user_cap_edit_entry( $this->entry ) ) {
            $error = __( 'You do not have permission to edit this entry.', 'gravityview');

Expected 1 spaces before closing bracket; 0 found

        }

        if( $this->entry['status'] === 'trash' ) {

Found "=== '". Use Yoda Condition checks, you must

            $error = __('You cannot edit the entry; it is in the trash.', 'gravityview' );

Expected 1 spaces after opening bracket; 0 found

        }

        // No errors; everything's fine here!
        if( empty( $error ) ) {
            return true;
        }

        if( $echo && $error !== true ) {

Found "!== true". Use Yoda Condition checks, you must

	        $error = esc_html( $error );

	        /**
	         * @since 1.9
	         */
	        if ( ! empty( $this->entry ) ) {
		        $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;" ) );

Expected 1 spaces after opening bracket; 0 found

The string literal window.history.go(-1); return false; does not require double quotes, as per coding-style, please use single quotes.

	        }

            echo GVCommon::generate_notice( wpautop( $error ), 'gv-error error');

Expected a sanitizing function (see Codex for 'Data Validation'), but instead saw 'GVCommon'

Expected 1 spaces before closing bracket; 0 found

        }

        do_action('gravityview_log_error', 'GravityView_Edit_Entry[user_can_edit_entry]' . $error );

Expected 1 spaces after opening bracket; 0 found

        return false;
    }


    /**
     * Check whether a field is editable by the current user, and optionally display an error message
     * @uses  GravityView_Edit_Entry->check_user_cap_edit_field() Check user capabilities
     * @param  array  $field Field or field settings array
     * @param  boolean $echo  Whether to show error message telling user they aren't allowed
     * @return boolean         True: user can edit the current field; False: nope, they can't.
     */
    private function user_can_edit_field( $field, $echo = false ) {

        $error = NULL;

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

        if( ! $this->check_user_cap_edit_field( $field ) ) {
            $error = __( 'You do not have permission to edit this field.', 'gravityview');

Expected 1 spaces before closing bracket; 0 found

        }

        // No errors; everything's fine here!
        if( empty( $error ) ) {
            return true;
        }

        if( $echo ) {
            echo GVCommon::generate_notice( wpautop( esc_html( $error ) ), 'gv-error error');

Expected a sanitizing function (see Codex for 'Data Validation'), but instead saw 'GVCommon'

Expected 1 spaces before closing bracket; 0 found

        }

        do_action('gravityview_log_error', 'GravityView_Edit_Entry[user_can_edit_field]' . $error );

Expected 1 spaces after opening bracket; 0 found

        return false;

    }


    /**
     * checks if user has permissions to edit a specific field
     *
     * Needs to be used combined with GravityView_Edit_Entry::user_can_edit_field for maximum security!!
     *
     * @param  [type] $field [description]

The doc-type [type] could not be parsed: Unknown type name "" at position 0. [(view supported doc-types)

     * @return bool
     */
    private function check_user_cap_edit_field( $field ) {

        // If they can edit any entries (as defined in Gravity Forms), we're good.
        if( GVCommon::has_cap( array( 'gravityforms_edit_entries', 'gravityview_edit_others_entries' ) ) ) {
            return true;
        }

        $field_cap = isset( $field['allow_edit_cap'] ) ? $field['allow_edit_cap'] : false;

        // If the field has custom editing capaibilities set, check those
        if( $field_cap ) {
            return GVCommon::has_cap( $field['allow_edit_cap'] );
        }

        return false;
    }


    /**
     * Is the current nonce valid for editing the entry?
     * @return boolean
     */
    public function verify_nonce() {

        // Verify form submitted for editing single
        if( $this->is_edit_entry_submission() ) {
            $valid = wp_verify_nonce( $_POST[ self::$nonce_field ], self::$nonce_field );

Detected access of super global var $_POST, probably need manual inspection.

Detected usage of a non-validated input variable: $_POST

Detected usage of a non-sanitized input variable: $_POST

        }

        // Verify
        else if( ! $this->is_edit_entry() ) {
            $valid = false;
        }

        else {
            $valid = wp_verify_nonce( $_GET['edit'], self::$nonce_key );

Detected access of super global var $_GET, probably need manual inspection.

Detected usage of a non-validated input variable: $_GET

Detected usage of a non-sanitized input variable: $_GET

        }

        /**
         * @filter `gravityview/edit_entry/verify_nonce` Override Edit Entry nonce validation. Return true to declare nonce valid.
         * @since 1.13
         * @param int|boolean $valid False if invalid; 1 or 2 when nonce was generated
         * @param string $nonce_field Key used when validating submissions. Default: is_gv_edit_entry
         */
        $valid = apply_filters( 'gravityview/edit_entry/verify_nonce', $valid, self::$nonce_field );

        return $valid;
    }



} //end class