GravityView_Admin_ApproveEntries

Complexity

Total Complexity

92

Size/Duplication

Total Lines	815
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	14
Bugs	2	Features	2

22 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	filter_links_entry_list()	0	50	3
A	tooltips()	0	9	1
B	add_field_buttons()	0	25	1
A	set_defaults()	0	50	1
B	get_gv_bulk_action()	0	18	6
C	process_bulk_action()	0	70	17
B	update_bulk()	0	21	7
B	update_approved()	0	59	8
C	ajax_update_approved()	0	37	7
C	get_approved_column()	0	32	11
A	add_entry_approved_hidden_input()	0	14	4
A	get_form_id()	0	14	4
B	add_scripts_and_styles()	0	37	5
B	get_bulk_actions()	0	34	3
B	show_approve_entry_column()	0	29	3
A	register_gform_noconflict_script()	0	4	1
A	register_gform_noconflict_style()	0	4	1
B	add_hooks()	0	36	1
A	after_submission()	0	3	1
A	after_update_entry_update_approved_meta()	0	16	2
B	update_approved_meta()	0	47	4

<?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 15 and the first side effect is on line 831.

/**
 * @file class-admin-approve-entries.php
 * @package   GravityView
 * @license   GPL2+
 * @author    Katz Web Services, Inc.
 * @link      http://gravityview.co
 * @copyright Copyright 2014, Katz Web Services, Inc.
 *
 * @since 1.0.0
 */



class GravityView_Admin_ApproveEntries {

	// hold notification messages
	public $bulk_update_message = '';

	/**
	 * @var array Set the prefixes here instead of spread across the class
	 * @since 1.17
	 */
	private $bulk_action_prefixes = array(
		'approve' => 'gvapprove',
		'unapprove' => 'gvunapprove',
	);

	function __construct() {

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

		$this->add_hooks();

	}

	private function add_hooks() {
		/** Edit Gravity Form page */

		// Add button to left menu
		add_filter( 'gform_add_field_buttons', array( $this, 'add_field_buttons' ) );
		// Set defaults
		add_action( 'gform_editor_js_set_default_values', array( $this, 'set_defaults' ) );

		/** gf_entries page - entries table screen */

		// capture bulk actions
		add_action( 'gform_loaded', array( $this, 'process_bulk_action') );

No space before closing parenthesis of array is bad style

		// add hidden field with approve status
		add_action( 'gform_entries_first_column_actions', array( $this, 'add_entry_approved_hidden_input' ), 1, 5 );

		// process ajax approve entry requests
		add_action('wp_ajax_gv_update_approved', array( $this, 'ajax_update_approved'));

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

No space before closing parenthesis of array is bad style

		// when using the User opt-in field, check on entry submission
		add_action( 'gform_after_submission', array( $this, 'after_submission' ), 10, 2 );

		// in case entry is edited (on admin or frontend)
		add_action( 'gform_after_update_entry', array( $this, 'after_update_entry_update_approved_meta' ), 10, 2);

Expected 1 spaces before closing bracket; 0 found

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

		add_filter( 'gravityview_tooltips', array( $this, 'tooltips' ) );

		// adding styles and scripts
		add_action( 'admin_enqueue_scripts', array( $this, 'add_scripts_and_styles') );

No space before closing parenthesis of array is bad style

		// bypass Gravity Forms no-conflict mode
		add_filter( 'gform_noconflict_scripts', array( $this, 'register_gform_noconflict_script' ) );
		add_filter( 'gform_noconflict_styles', array( $this, 'register_gform_noconflict_style' ) );

		add_filter( 'gform_filter_links_entry_list', array( $this, 'filter_links_entry_list' ), 10, 3 );
	}

	/**
	 * Add filter links to the Entries page
	 *
	 * Can be disabled by returning false on the `gravityview/approve_entries/show_filter_links_entry_list` filter
	 *
	 * @since 1.17.1
	 *
	 * @param array $filter_links Array of links to include in the subsubsub filter list. Includes `id`, `field_filters`, `count`, and `label` keys
	 * @param array $form GF Form object of current form
	 * @param bool $include_counts Whether to include counts in the output
	 *
	 * @return array Filter links, with GravityView approved/disapproved links added
	 */
	public function filter_links_entry_list( $filter_links = array(), $form = array(), $include_counts = true ) {

		/**
		 * @filter `gravityview/approve_entries/show_filter_links_entry_list` Disable filter links
		 * @since 1.17.1
		 * @param bool $show_filter_links True: show the "approved"/"disapproved" filter links. False: hide them.
		 * @param array $form GF Form object of current form
		 */
		if( false === apply_filters( 'gravityview/approve_entries/show_filter_links_entry_list', true, $form ) ) {
			return $filter_links;
		}

		$field_filters_approved = array(
			array(
				'key' => 'is_approved',
				'value' => 'Approved'

Each line in an array declaration must end in a comma

			),
		);

		$field_filters_disapproved = array(
			array(
				'key'      => 'is_approved',
				'value'    => '0',
			),
		);

		$approved_count = $disapproved_count = 0;

		// Only count if necessary
		if( $include_counts ) {
			$approved_count = count( gravityview_get_entry_ids( $form['id'], array( 'status' => 'active', 'field_filters' => $field_filters_approved ) ) );
			$disapproved_count = count( gravityview_get_entry_ids( $form['id'], array( 'status' => 'active', 'field_filters' => $field_filters_disapproved ) ) );
		}

		$filter_links[] = array(
			'id'            => 'gv_approved',
			'field_filters' => $field_filters_approved,
			'count'         => $approved_count,
			'label'         => esc_html__( 'Approved', 'gravityview' ),
		);

		$filter_links[] = array(
			'id'            => 'gv_disapproved',
			'field_filters' => $field_filters_disapproved,
			'count'         => $disapproved_count,
			'label'         => esc_html__( 'Disapproved', 'gravityview' ),
		);

		return $filter_links;
	}

	/**
	 * Add the GravityView Fields group tooltip
	 *
	 * @param $tooltips
	 *
	 * @return array Tooltips array with GravityView fields tooltip
	 */
	function tooltips( $tooltips ) {

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

		$tooltips['form_gravityview_fields'] = array(
			'title' => __('GravityView Fields', 'gravityview'),

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

			'value' => __( 'Allow administrators to approve or reject entries and users to opt-in or opt-out of their entries being displayed.', 'gravityview'),

Expected 1 spaces before closing bracket; 0 found

		);

		return $tooltips;
	}


	/**
	 * Inject new add field buttons in the gravity form editor page
	 *
	 * @access public
	 * @param mixed $field_groups
	 * @return array Array of fields
	 */
	function add_field_buttons( $field_groups ) {

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

		$gravityview_fields = array(
			'name' => 'gravityview_fields',
			'label' => 'GravityView',
			'fields' => array(
				array(
					'class' => 'button',
					'value' => __( 'Approve/Reject', 'gravityview' ),
					'onclick' => "StartAddField('gravityviewapproved_admin');",
					'data-type' => 'gravityviewapproved_admin'

Each line in an array declaration must end in a comma

				),
				array(
					'class' => 'button',
					'value' => __( 'User Opt-In', 'gravityview' ),
					'onclick' => "StartAddField('gravityviewapproved');",
					'data-type' => 'gravityviewapproved'

Each line in an array declaration must end in a comma

				),
			)
		);

		array_push( $field_groups, $gravityview_fields );

		return $field_groups;
	}



	/**
	 * At edit form page, set the field Approve defaults
	 *
	 * @todo Convert to a partial include file
	 * @access public
	 * @return void
	 */
	function set_defaults() {

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

		?>
		case 'gravityviewapproved_admin':
			field.label = "<?php echo esc_js( __( 'Approved? (Admin-only)', 'gravityview' ) ); ?>";

			field.adminLabel = "<?php echo esc_js( __( 'Approved?', 'gravityview' ) ); ?>";
			field.adminOnly = true;

			field.choices = null;
			field.inputs = null;

			if( !field.choices ) {
				field.choices = new Array( new Choice("<?php _e( 'Approved', 'gravityview' ); ?>") );
			}

			field.inputs = new Array();
			for( var i=1; i<=field.choices.length; i++ ) {
				field.inputs.push(new Input(field.id + (i/10), field.choices[i-1].text));
			}

			field.type = 'checkbox';
			field.gravityview_approved = 1;

			break;
		case 'gravityviewapproved':
			field.label = "<?php echo esc_js( __( 'Show Entry on Website', 'gravityview' ) ); ?>";

			field.adminLabel = "<?php echo esc_js( __( 'Opt-In', 'gravityview' ) ); ?>";
			field.adminOnly = false;

			field.choices = null;
			field.inputs = null;

			if( !field.choices ) {
				field.choices = new Array(
					new Choice("<?php echo esc_js( __( 'Yes, display my entry on the website', 'gravityview' ) ); ?>")
				);
			}

			field.inputs = new Array();
			for( var i=1; i<=field.choices.length; i++ ) {
				field.inputs.push(new Input(field.id + (i/10), field.choices[i-1].text));
			}

			field.type = 'checkbox';
			field.gravityview_approved = 1;

			break;
		<?php
	}

	/**
	 * Get the Bulk Action submitted value if it is a GravityView Approve/Unapprove action
	 *
	 * @since 1.17.1
	 *
	 * @return string|false If the bulk action was GravityView Approve/Unapprove, return the full string (gvapprove-16, gvunapprove-16). Otherwise, return false.
	 */
	private function get_gv_bulk_action() {

		$gv_bulk_action = false;

		if( version_compare( GFForms::$version, '2.0', '>=' ) ) {
			$bulk_action = ( '-1' !== rgpost('action') ) ? rgpost('action') : rgpost('action2');

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

		} else {
			// GF 1.9.x - Bulk action 2 is the bottom bulk action select form.
			$bulk_action = rgpost('bulk_action') ? rgpost('bulk_action') : rgpost('bulk_action2');

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

		}

		// Check the $bulk_action value against GV actions, see if they're the same. I hate strpos().
		if( ! empty( $bulk_action ) && preg_match( '/^('. implode( '|', $this->bulk_action_prefixes ) .')/ism', $bulk_action ) ) {
			$gv_bulk_action = $bulk_action;
		}

		return $gv_bulk_action;
	}

	/**
	 * Capture bulk actions - gf_entries table
	 *
	 * @uses  GravityView_frontend::get_search_criteria() Convert the $_POST search request into a properly formatted request.
	 * @access public
	 * @return void|boolean
	 */
	public function process_bulk_action() {

		if ( ! is_admin() || ! class_exists( 'GFForms' ) || empty( $_POST ) ) {

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

			return false;
		}

		// The action is formatted like: gvapprove-16 or gvunapprove-16, where the first word is the name of the action and the second is the ID of the form.
		$bulk_action = $this->get_gv_bulk_action();
		
		// gforms_entry_list is the nonce that confirms we're on the right page
		// gforms_update_note is sent when bulk editing entry notes. We don't want to process then.
		if ( $bulk_action && rgpost('gforms_entry_list') && empty( $_POST['gforms_update_note'] ) ) {

The expression $bulk_action of type string|false is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== false instead.

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

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

			check_admin_referer( 'gforms_entry_list', 'gforms_entry_list' );

			/**
			 * The extra '-' is to make sure that there are at *least* two items in array.
			 * @see https://github.com/katzwebservices/GravityView/issues/370
			 */
			$bulk_action .= '-';

			list( $approved_status, $form_id ) = explode( '-', $bulk_action );

			if ( empty( $form_id ) ) {
				do_action( 'gravityview_log_error', '[process_bulk_action] Form ID is empty from parsing bulk action.', $bulk_action );
				return false;
			}

			// All entries are set to be updated, not just the visible ones
			if ( ! empty( $_POST['all_entries'] ) ) {

				// Convert the current entry search into GF-formatted search criteria
				$search = array(
					'search_field' => isset( $_POST['f'] ) ? $_POST['f'][0] : 0,

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

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

					'search_value' => isset( $_POST['v'][0] ) ? $_POST['v'][0] : '',

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

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

					'search_operator' => isset( $_POST['o'][0] ) ? $_POST['o'][0] : 'contains',

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

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

				);

				$search_criteria = GravityView_frontend::get_search_criteria( $search, $form_id );

				// Get all the entry IDs for the form
				$entries = gravityview_get_entry_ids( $form_id, $search_criteria );

			} else {

				// Changed from 'lead' to 'entry' in 2.0
				$entries = isset( $_POST['lead'] ) ? $_POST['lead'] : $_POST['entry'];

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

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

			}

			if ( empty( $entries ) ) {
				do_action( 'gravityview_log_error', '[process_bulk_action] Entries are empty' );
				return false;
			}

			$entry_count = count( $entries ) > 1 ? sprintf( __( '%d entries', 'gravityview' ), count( $entries ) ) : __( '1 entry', 'gravityview' );

			switch ( $approved_status ) {
				case $this->bulk_action_prefixes['approve']:
					self::update_bulk( $entries, 1, $form_id );
					$this->bulk_update_message = sprintf( __( '%s approved.', 'gravityview' ), $entry_count );
					break;

				case $this->bulk_action_prefixes['unapprove']:
					self::update_bulk( $entries, 0, $form_id );
					$this->bulk_update_message = sprintf( __( '%s disapproved.', 'gravityview' ), $entry_count );
					break;
			}
		}
	}





	/**
	 * Process a bulk of entries to update the approve field/property
	 *
	 * @access private
	 * @static
	 * @param array|boolean $entries If array, array of entry IDs that are to be updated. If true: update all entries.
	 * @param int $approved Approved status. If `0`: unapproved, if not empty, `Approved`
	 * @param int $form_id The Gravity Forms Form ID
	 * @return boolean|void
	 */
	private static function update_bulk( $entries, $approved, $form_id ) {

		if( empty($entries) || ( $entries !== true && !is_array($entries) ) ) {

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

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

Expected 1 space after "!"; 0 found

			do_action( 'gravityview_log_error', __METHOD__ . ' Entries were empty or malformed.', $entries );
			return false;
		}

		if( ! GVCommon::has_cap( 'gravityview_moderate_entries' ) ) {
			do_action( 'gravityview_log_error', __METHOD__ . ' User does not have the `gravityview_moderate_entries` capability.' );
			return false;
		}

		$approved = empty( $approved ) ? 0 : 'Approved';

		// calculate approved field id once instead of looping through in the update_approved() method
		$approved_column_id = self::get_approved_column( $form_id );

		foreach( $entries as $entry_id ) {

The expression $entries of type boolean|array is not guaranteed to be traversable. How about adding an additional type check?

			self::update_approved( (int)$entry_id, $approved, $form_id, $approved_column_id );

No space after closing casting parenthesis is prohibited

		}
	}




	/**
	 * update_approved function.
	 *
	 * @access public
	 * @static
	 * @param int $entry_id (default: 0)
	 * @param int $approved (default: 0)
	 * @param int $form_id (default: 0)
	 * @param int $approvedcolumn (default: 0)
	 * @return boolean True: It worked; False: it failed
	 */
	public static function update_approved( $entry_id = 0, $approved = 0, $form_id = 0, $approvedcolumn = 0) {

		if( !class_exists( 'GFAPI' ) ) {

Expected 1 space after "!"; 0 found

			do_action( 'gravityview_log_error', __METHOD__ . 'GFAPI does not exist' );
			return false;
		}

		if( empty( $approvedcolumn ) ) {
			$approvedcolumn = self::get_approved_column( $form_id );
		}

		//get the entry
		$entry = GFAPI::get_entry( $entry_id );

		//update entry
		$entry[ (string)$approvedcolumn ] = $approved;

No space after closing casting parenthesis is prohibited

		/** @var bool|WP_Error $result */
		$result = GFAPI::update_entry( $entry );

		/**
		 * GFAPI::update_entry() doesn't trigger `gform_after_update_entry`, so we trigger updating the meta ourselves.
		 */
		self::update_approved_meta( $entry_id, $approved, $form_id );

		// add note to entry
		if( $result === true ) {

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

			$note = empty( $approved ) ? __( 'Disapproved the Entry for GravityView', 'gravityview' ) : __( 'Approved the Entry for GravityView', 'gravityview' );

			/**
			 * @filter `gravityview/approve_entries/add-note` Add a note when the entry has been approved or disapproved?
			 * @since 1.16.3
			 * @param bool $add_note True: Yep, add that note! False: Do not, under any circumstances, add that note!
			 */
			$add_note = apply_filters( 'gravityview/approve_entries/add-note', true );

			if( $add_note && class_exists( 'GravityView_Entry_Notes' ) ) {
				$current_user = wp_get_current_user();
				GravityView_Entry_Notes::add_note( $entry_id, $current_user->ID, $current_user->display_name, $note );
			}

			/**
			 * Destroy the cache for this form
			 * @see class-cache.php
			 * @since 1.5.1
			 */
			do_action( 'gravityview_clear_form_cache', $form_id );

		} else if( is_wp_error( $result ) ) {

			do_action( 'gravityview_log_error', __METHOD__ . sprintf( ' - Entry approval not updated: %s', $result->get_error_message() ) );

			$result = false;
		}

		return $result;

	}


	/**
	 * Update the is_approved meta whenever the entry is submitted (and it contains a User Opt-in field)
	 *
	 * @since 1.16.6
	 *
	 * @param $entry array Gravity Forms entry object
	 * @param $form array Gravity Forms form object
	 */
	public function after_submission( $entry, $form ) {
		$this->after_update_entry_update_approved_meta( $form , $entry['id'] );
	}



	/**
	 * Update the is_approved meta whenever the entry is updated
	 *
	 * @since 1.7.6.1 Was previously named `update_approved_meta`
	 *
	 * @param  array $form     Gravity Forms form array
	 * @param  int $entry_id ID of the Gravity Forms entry
	 * @return void
	 */
	public function after_update_entry_update_approved_meta( $form, $entry_id = NULL ) {

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

		$approvedcolumn = self::get_approved_column( $form['id'] );

        /**
         * If the form doesn't contain the approve field, don't assume anything.
         */
        if( empty( $approvedcolumn ) ) {
            return;
        }

		$entry = GFAPI::get_entry( $entry_id );

		self::update_approved_meta( $entry_id, $entry[ (string)$approvedcolumn ], $form['id'] );

No space after closing casting parenthesis is prohibited

	}

	/**
	 * Update the `is_approved` entry meta value
	 *
	 * @since 1.7.6.1 `after_update_entry_update_approved_meta` was previously to be named `update_approved_meta`
	 * @since 1.17.1 Added $form_id parameter
	 *
	 * @param  int $entry_id ID of the Gravity Forms entry
	 * @param  string $is_approved String whether entry is approved or not. `0` for not approved, `Approved` for approved.
	 * @param int $form_id ID of the form of the entry being updated. Improves query performance.
	 *
	 * @return void
	 */
	private static function update_approved_meta( $entry_id, $is_approved, $form_id = 0 ) {

		/**
		 * Make sure that the "User Opt-in" and the Admin Approve/Reject entry set the same meta value
		 * @since 1.16.6
		 */
		$is_approved = empty( $is_approved ) ? 0 : 'Approved';

		// update entry meta
		if( function_exists('gform_update_meta') ) {

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

			gform_update_meta( $entry_id, 'is_approved', $is_approved, $form_id );

			/**
			 * @action `gravityview/approve_entries/updated` Triggered when an entry approval is updated
			 * @since 1.7.6.1
			 * @param  int $entry_id ID of the Gravity Forms entry
			 * @param  string $is_approved String whether entry is approved or not. `0` for not approved, `Approved` for approved.
			 */
			do_action( 'gravityview/approve_entries/updated', $entry_id, $is_approved );

			if( empty( $is_approved ) ) {

				/**
				 * @action `gravityview/approve_entries/disapproved` Triggered when an entry is rejected
				 * @since 1.7.6.1
				 * @param  int $entry_id ID of the Gravity Forms entry
				 */
				do_action( 'gravityview/approve_entries/disapproved', $entry_id );

			} else {

				/**
				 * @action `gravityview/approve_entries/approved` Triggered when an entry is approved
				 * @since 1.7.6.1
				 * @param  int $entry_id ID of the Gravity Forms entry
				 */
				do_action( 'gravityview/approve_entries/approved', $entry_id );

			}

Blank line found after control structure

		} else {

			do_action('gravityview_log_error', __METHOD__ . ' - `gform_update_meta` does not exist.' );

Expected 1 spaces after opening bracket; 0 found

		}
	}


	/**
	 * Approve/Disapprove entries using the × or ✓ icons in the GF Entries screen
	 * @return void
	 */
	public function ajax_update_approved() {

		if( empty( $_POST['entry_id'] ) || empty( $_POST['form_id'] ) ) {

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

			do_action( 'gravityview_log_error', __METHOD__ . ' entry_id or form_id are empty.', $_POST );

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

			$result = false;
		}

		else if ( ! isset( $_POST['nonce'] ) || ! wp_verify_nonce( $_POST['nonce'], 'gravityview_ajaxgfentries' ) ) {

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

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

			do_action( 'gravityview_log_error', __METHOD__ . ' Security check failed.', $_POST );

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

			$result = false;
		}

		else if( ! GVCommon::has_cap( 'gravityview_moderate_entries', $_POST['entry_id'] ) ) {

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

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

			do_action( 'gravityview_log_error', __METHOD__ . ' User does not have the `gravityview_moderate_entries` capability.' );

			$result = false;
		}

		else {

			$result = self::update_approved( $_POST['entry_id'], $_POST['approved'], $_POST['form_id'] );

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

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

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

			if( is_wp_error( $result ) ) {
				/** @var WP_Error $result */
				do_action( 'gravityview_log_error', __METHOD__ .' Error updating approval: ' . $result->get_error_message() );
				$result = false;
			}

Blank line found after control structure

		}

		exit( $result );

The method ajax_update_approved() contains an exit expression.

	}


	/**
	 * Calculate the approve field.input id
	 *
	 * @access public
	 * @static
	 * @param mixed $form GF Form or Form ID
	 * @return false|null|string Returns the input ID of the approved field. Returns NULL if no approved fields were found. Returns false if $form_id wasn't set.
	 */
	static public function get_approved_column( $form ) {

As per PSR2, the static declaration should come after the visibility declaration.

        if( empty( $form ) ) {
            return null;
        }

        if( !is_array( $form ) ) {

Expected 1 space after "!"; 0 found

            $form = GVCommon::get_form( $form );
        }

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

            $field = (array) $field;

			if( !empty( $field['gravityview_approved'] ) ) {

Expected 1 space after "!"; 0 found

				if( !empty($field['inputs'][0]['id']) ) {

Expected 1 space after "!"; 0 found

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

					return $field['inputs'][0]['id'];
				}
			}

            // Note: This is just for backward compatibility from GF Directory plugin and old GV versions - when using i18n it may not work..
            if( 'checkbox' == $field['type'] && isset( $field['inputs'] ) && is_array( $field['inputs'] ) ) {
                foreach ( $field['inputs'] as $key2 => $input ) {
                    if ( strtolower( $input['label'] ) == 'approved' ) {

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

                        return $input['id'];
                    }
                }
            }
		}

		return null;
	}

	/**
	 * Add a hidden input that is used in the Javascript to show approved/disapproved entries checkbox
	 *
	 * See the /assets/js/admin-entries-list.js setInitialApprovedEntries method
	 *
	 * @param $form_id
	 * @param $field_id
	 * @param $value
	 * @param $entry
	 * @param $query_string
	 *
	 * @return void
	 */
	static public function add_entry_approved_hidden_input(  $form_id, $field_id, $value, $entry, $query_string ) {

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

As per PSR2, the static declaration should come after the visibility declaration.

		if( ! GVCommon::has_cap( 'gravityview_moderate_entries', $entry['id'] ) ) {
			return;
		}

		if( empty( $entry['id'] ) ) {
			return;
		}

		if( gform_get_meta( $entry['id'], 'is_approved' ) ) {
			echo '<input type="hidden" class="entry_approved" id="entry_approved_'. $entry['id'] .'" value="true" />';

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

		}
	}

	/**
	 * Get the form ID of the form currently being displayed
	 *
	 * @since 1.17.1
	 *
	 * @return int ID of the current form being displayed
	 */
	private function get_form_id() {

		$form_id = GFForms::get('id');

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

		// If there are no forms identified, use the first form. That's how GF does it.
		if( empty( $form_id ) && class_exists('RGFormsModel') ) {

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

			$forms = gravityview_get_forms();
			if( !empty( $forms ) ) {

Expected 1 space after "!"; 0 found

				$form_id = $forms[0]['id'];
			}
		}

		return absint( $form_id );
	}


	function add_scripts_and_styles( $hook ) {

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

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

		if( ! class_exists( 'RGForms' ) ) {

			do_action( 'gravityview_log_error', 'GravityView_Admin_ApproveEntries[add_scripts_and_styles] RGForms does not exist.' );

			return;
		}

		// enqueue styles & scripts gf_entries
		// But only if we're on the main Entries page, not on reports pages
		if( GFForms::get_page() !== 'entry_list' ) {

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

			return;
		}

		$form_id = $this->get_form_id();

		wp_enqueue_style( 'gravityview_entries_list', plugins_url('assets/css/admin-entries-list.css', GRAVITYVIEW_FILE), array(), GravityView_Plugin::version );

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

		$script_debug = (defined('SCRIPT_DEBUG') && SCRIPT_DEBUG) ? '' : '.min';

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

		wp_enqueue_script( 'gravityview_gf_entries_scripts', plugins_url('assets/js/admin-entries-list'.$script_debug.'.js', GRAVITYVIEW_FILE), array( 'jquery' ), GravityView_Plugin::version );

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

		wp_localize_script( 'gravityview_gf_entries_scripts', 'gvGlobals', array(
			'nonce' => wp_create_nonce( 'gravityview_ajaxgfentries'),

Expected 1 spaces before closing bracket; 0 found

			'form_id' => $form_id,
			'show_column' => (int)$this->show_approve_entry_column( $form_id ),

No space after closing casting parenthesis is prohibited

			'add_bulk_action' => (int)GVCommon::has_cap( 'gravityview_moderate_entries' ),

No space after closing casting parenthesis is prohibited

			'bulk_actions' => $this->get_bulk_actions( $form_id ),
			'bulk_message' => $this->bulk_update_message,
			'approve_title' => __( 'Entry not approved for directory viewing. Click to approve this entry.', 'gravityview'),

Expected 1 spaces before closing bracket; 0 found

			'unapprove_title' => __( 'Entry approved for directory viewing. Click to disapprove this entry.', 'gravityview'),

Expected 1 spaces before closing bracket; 0 found

			'column_title' => __( 'Show entry in directory view?', 'gravityview'),

Expected 1 spaces before closing bracket; 0 found

			'column_link' => esc_url( add_query_arg( array('sort' => self::get_approved_column( $form_id ) ) ) ),

No space after opening parenthesis of array is bad style

		) );

	}

	/**
	 * Get an array of options to be added to the Gravity Forms "Bulk action" dropdown in a "GravityView" option group
	 *
	 * @since 1.16.3
	 *
	 * @param int $form_id  ID of the form currently being displayed
	 *
	 * @return array Array of actions to be added to the GravityView option group
	 */
	private function get_bulk_actions( $form_id ) {

		$bulk_actions = array(
			'GravityView' => array(
				array(
					'label' => __( 'Approve', 'gravityview' ),
					'value' => sprintf( '%s-%d', $this->bulk_action_prefixes['approve'], $form_id ),
				),
				array(
					'label' => __( 'Disapprove', 'gravityview' ),
					'value' => sprintf( '%s-%d', $this->bulk_action_prefixes['unapprove'], $form_id ),
				),
			),
		);

		/**
		 * @filter `gravityview/approve_entries/bulk_actions` Modify the GravityView "Bulk action" dropdown list. Return an empty array to hide.
		 * @see https://gist.github.com/zackkatz/82785402c996b51b4dc9 for an example of how to use this filter
		 * @since 1.16.3
		 * @param array $bulk_actions Associative array of actions to be added to "Bulk action" dropdown inside GravityView `<optgroup>`. Parent array key is the `<optgroup>` label, then each child array must have `label` (displayed text) and `value` (input value) keys
		 * @param int $form_id ID of the form currently being displayed
		 */
		$bulk_actions = apply_filters( 'gravityview/approve_entries/bulk_actions', $bulk_actions, $form_id );

		// Sanitize the values, just to be sure.
		foreach ( $bulk_actions as $key => $group ) {
			foreach ( $group as $i => $action ) {
				$bulk_actions[ $key ][ $i ]['label'] = esc_html( $bulk_actions[ $key ][ $i ]['label'] );
				$bulk_actions[ $key ][ $i ]['value'] = esc_attr( $bulk_actions[ $key ][ $i ]['value'] );
			}
		}

		return $bulk_actions;
	}

	/**
	 * Should the Approve/Reject Entry column be shown in the GF Entries page?
	 *
	 * @since 1.7.2
	 *
	 * @param int $form_id The ID of the Gravity Forms form for which entries are being shown
	 *
	 * @return bool True: Show column; False: hide column
	 */
	private function show_approve_entry_column( $form_id ) {

		$show_approve_column = GVCommon::has_cap( 'gravityview_moderate_entries' );

		/**
		 * @filter `gravityview/approve_entries/hide-if-no-connections` Return true to hide reject/approve if there are no connected Views
		 * @since 1.7.2
		 * @param boolean $hide_if_no_connections
		 */
		$hide_if_no_connections = apply_filters('gravityview/approve_entries/hide-if-no-connections', false );

Expected 1 spaces after opening bracket; 0 found

		if( $hide_if_no_connections ) {

			$connected_views = gravityview_get_connected_views( $form_id );

			if( empty( $connected_views ) ) {
				$show_approve_column = false;
			}
		}

		/**
		 * @filter `gravityview/approve_entries/show-column` Override whether the column is shown
		 * @param boolean $show_approve_column Whether the column will be shown
		 * @param int $form_id The ID of the Gravity Forms form for which entries are being shown
		 */
		$show_approve_column = apply_filters('gravityview/approve_entries/show-column', $show_approve_column, $form_id );

Expected 1 spaces after opening bracket; 0 found

		return $show_approve_column;
	}

	function register_gform_noconflict_script( $scripts ) {

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

		$scripts[] = 'gravityview_gf_entries_scripts';
		return $scripts;
	}

	function register_gform_noconflict_style( $styles ) {

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

		$styles[] = 'gravityview_entries_list';
		return $styles;
	}

}

new GravityView_Admin_ApproveEntries;