gravityview / GravityView

includes/plugin-and-theme-hooks/class-gravityview-plugin-hooks-gravity-flow.php

Spacing +10 added lines, -10 removed lines

@@ -33,7 +33,7 @@ 
 
 		parent::add_hooks();
 
-		add_filter( 'gravityview/search/searchable_fields', array( $this, 'modify_search_bar_fields_dropdown'), 10, 2 );
+		add_filter( 'gravityview/search/searchable_fields', array( $this, 'modify_search_bar_fields_dropdown' ), 10, 2 );
 
 		add_filter( 'gravityview/admin/available_fields', array( $this, 'maybe_add_non_default_fields' ), 10, 3 );
 
@@ -53,7 +53,7 @@ 
 	 * @return void
 	 */
 	public function clear_cache_after_workflow( $form, $entry_id, $step_id, $starting_step_id ) {
-		do_action( 'gravityview_clear_form_cache', $form['id'] );
+		do_action( 'gravityview_clear_form_cache', $form[ 'id' ] );
 	}
 
 	/**
@@ -70,13 +70,13 @@ 
 	 */
 	public static function get_status_options( $form_id = 0, $status_key = 'workflow_final_status' ) {
 
-		if( empty( $form_id ) ) {
+		if ( empty( $form_id ) ) {
 			$form_id = GravityView_View::getInstance()->getFormId();
 		}
 
 		$entry_meta = gravity_flow()->get_entry_meta( array(), $form_id );
 
-		return (array) \GV\Utils::get( $entry_meta, $status_key . '/filter/choices' );
+		return (array)\GV\Utils::get( $entry_meta, $status_key . '/filter/choices' );
 	}
 
 
@@ -98,7 +98,7 @@ 
 
 		$workflow_steps = $GFlow->get_steps();
 
-		if( $workflow_steps ) {
+		if ( $workflow_steps ) {
 
 			foreach ( $workflow_steps as $step ) {
 
@@ -110,12 +110,12 @@ 
 				);
 			}
 
-			$fields['workflow_step'] = array(
+			$fields[ 'workflow_step' ] = array(
 				'label' => esc_html__( 'Workflow Step', 'gravityview' ),
 				'type'  => 'select',
 			);
 
-			$fields['workflow_final_status'] = array(
+			$fields[ 'workflow_final_status' ] = array(
 				'label' => esc_html__( 'Workflow Status', 'gravityview' ),
 				'type'  => 'select',
 			);
@@ -131,7 +131,7 @@ 
 		if ( ( $insert_at = array_search( 'workflow_final_status', wp_list_pluck( $fields, 'key' ) ) ) !== false ) {
 			$fields_end = array_splice( $fields, $insert_at + 1 );
 
-			$fields[] = array(
+			$fields[ ] = array(
 				'text' => __( 'Workflow Current Status Timestamp', 'gravityview' ),
 				'operators' => array( '>', '<' ),
 				'placeholder' => 'yyyy-mm-dd',
@@ -157,8 +157,8 @@ 
 				$keys_end = array_splice( $keys, $insert_at + 1 );
 				$values_end = array_splice( $values, $insert_at + 1 );
 
-				$keys[] = 'workflow_current_status_timestamp';
-				$values[] = array(
+				$keys[ ] = 'workflow_current_status_timestamp';
+				$values[ ] = array(
 					'label' => __( 'Workflow Current Status Timestamp', 'gravityview' ),
 					'type' => 'workflow_current_status_timestamp',
 				);

includes/extensions/lightbox/class-gravityview-lightbox-provider.php

Spacing +4 added lines, -4 removed lines

@@ -23,8 +23,8 @@ 
 		add_filter( 'gravityview/fields/fileupload/link_atts', array( $this, 'fileupload_link_atts' ), 10, 3 );
 		add_filter( 'gravityview/get_link/allowed_atts', array( $this, 'allowed_atts' ) );
 
-		add_action( 'wp_enqueue_scripts', array( $this, 'enqueue_scripts') );
-		add_action( 'wp_enqueue_scripts', array( $this, 'enqueue_styles') );
+		add_action( 'wp_enqueue_scripts', array( $this, 'enqueue_scripts' ) );
+		add_action( 'wp_enqueue_scripts', array( $this, 'enqueue_styles' ) );
 
 		add_action( 'wp_footer', array( $this, 'output_footer' ) );
 	}
@@ -44,8 +44,8 @@ 
 		remove_filter( 'gravityview/fields/fileupload/link_atts', array( $this, 'fileupload_link_atts' ), 10 );
 		remove_filter( 'gravityview/get_link/allowed_atts', array( $this, 'allowed_atts' ) );
 
-		remove_action( 'wp_enqueue_scripts', array( $this, 'enqueue_scripts') );
-		remove_action( 'wp_enqueue_scripts', array( $this, 'enqueue_styles') );
+		remove_action( 'wp_enqueue_scripts', array( $this, 'enqueue_scripts' ) );
+		remove_action( 'wp_enqueue_scripts', array( $this, 'enqueue_styles' ) );
 
 		remove_action( 'wp_footer', array( $this, 'output_footer' ) );
 	}

includes/extensions/lightbox/class-gravityview-lightbox.php

Spacing +2 added lines, -2 removed lines

@@ -38,7 +38,7 @@ 
 	 */
 	public function set_provider( $provider = null ) {
 
-		if( gravityview()->request->is_admin() ) {
+		if ( gravityview()->request->is_admin() ) {
 			return;
 		}
 
@@ -57,7 +57,7 @@ 
 		}
 
 		// We're switching providers; remove the hooks that were added.
-		if( self::$active_provider ) {
+		if ( self::$active_provider ) {
 			self::$active_provider->remove_hooks();
 		}
 

includes/extensions/lightbox/fancybox/class-gravityview-lightbox-provider-fancybox.php

Spacing +14 added lines, -14 removed lines

@@ -102,17 +102,17 @@ 
 	 */
 	public function allowed_atts( $atts = array() ) {
 
-		$atts['data-fancybox']         = null;
-		$atts['data-fancybox-trigger'] = null;
-		$atts['data-fancybox-index']   = null;
-		$atts['data-src']              = null;
-		$atts['data-type']             = null;
-		$atts['data-width']            = null;
-		$atts['data-height']           = null;
-		$atts['data-srcset']           = null;
-		$atts['data-caption']          = null;
-		$atts['data-options']          = null;
-		$atts['data-filter']           = null;
+		$atts[ 'data-fancybox' ]         = null;
+		$atts[ 'data-fancybox-trigger' ] = null;
+		$atts[ 'data-fancybox-index' ]   = null;
+		$atts[ 'data-src' ]              = null;
+		$atts[ 'data-type' ]             = null;
+		$atts[ 'data-width' ]            = null;
+		$atts[ 'data-height' ]           = null;
+		$atts[ 'data-srcset' ]           = null;
+		$atts[ 'data-caption' ]          = null;
+		$atts[ 'data-options' ]          = null;
+		$atts[ 'data-filter' ]           = null;
 
 		return $atts;
 	}
@@ -126,14 +126,14 @@ 
 			return $link_atts;
 		}
 
-		$link_atts['class'] = \GV\Utils::get( $link_atts, 'class' ) . ' gravityview-fancybox';
+		$link_atts[ 'class' ] = \GV\Utils::get( $link_atts, 'class' ) . ' gravityview-fancybox';
 
-		$link_atts['class'] = sanitize_html_class( $link_atts['class'] );
+		$link_atts[ 'class' ] = sanitize_html_class( $link_atts[ 'class' ] );
 
 		if ( $context && ! empty( $context->field->field ) ) {
 			if ( $context->field->field->multipleFiles ) {
 				$entry = $context->entry->as_entry();
-				$link_atts['data-fancybox'] = 'gallery-' . sprintf( "%s-%s-%s", $entry['form_id'], $context->field->ID, $context->entry->get_slug() );
+				$link_atts[ 'data-fancybox' ] = 'gallery-' . sprintf( "%s-%s-%s", $entry[ 'form_id' ], $context->field->ID, $context->entry->get_slug() );
 			}
 		}
 

future/lib/EDD_SL_Plugin_Updater.php

Doc Comments +1 added lines, -1 removed lines

@@ -97,7 +97,7 @@
 	 * @uses api_request()
 	 *
 	 * @param array   $_transient_data Update array build by WordPress.
-	 * @return array Modified update array with custom plugin data.
+	 * @return stdClass Modified update array with custom plugin data.
 	 */
 	public function check_update( $_transient_data ) {
 

Braces +3 added lines, -1 removed lines

@@ -10,7 +10,9 @@
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) exit;
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
 
 /**
  * Allows plugins to use their own update API.

Spacing +52 added lines, -52 removed lines

@@ -48,10 +48,10 @@ 
 		$this->api_data    = $_api_data;
 		$this->name        = plugin_basename( $_plugin_file );
 		$this->slug        = basename( $_plugin_file, '.php' );
-		$this->version     = $_api_data['version'];
-		$this->wp_override = isset( $_api_data['wp_override'] ) ? (bool) $_api_data['wp_override'] : false;
-		$this->beta        = ! empty( $this->api_data['beta'] ) ? true : false;
-		$this->cache_key   = 'edd_sl_' . md5( serialize( $this->slug . $this->api_data['license'] . $this->beta ) );
+		$this->version     = $_api_data[ 'version' ];
+		$this->wp_override = isset( $_api_data[ 'wp_override' ] ) ? (bool)$_api_data[ 'wp_override' ] : false;
+		$this->beta        = ! empty( $this->api_data[ 'beta' ] ) ? true : false;
+		$this->cache_key   = 'edd_sl_' . md5( serialize( $this->slug . $this->api_data[ 'license' ] . $this->beta ) );
 
 		$edd_plugin_data[ $this->slug ] = $this->api_data;
 
@@ -170,11 +170,11 @@ 
 			return;
 		}
 
-		if( ! current_user_can( 'update_plugins' ) ) {
+		if ( ! current_user_can( 'update_plugins' ) ) {
 			return;
 		}
 
-		if( ! is_multisite() ) {
+		if ( ! is_multisite() ) {
 			return;
 		}
 
@@ -281,7 +281,7 @@ 
 					'<a target="_blank" class="thickbox" href="' . esc_url( $changelog_link ) . '">',
 					esc_html( $version_info->new_version ),
 					'</a>',
-					'<a href="' . esc_url( wp_nonce_url( self_admin_url( 'update.php?action=upgrade-plugin&plugin=' ) . $this->name, 'upgrade-plugin_' . $this->name ) ) .'">',
+					'<a href="' . esc_url( wp_nonce_url( self_admin_url( 'update.php?action=upgrade-plugin&plugin=' ) . $this->name, 'upgrade-plugin_' . $this->name ) ) . '">',
 					'</a>'
 				);
 			}
@@ -365,7 +365,7 @@ 
 			$_data->contributors = $this->convert_object_to_array( $_data->contributors );
 		}
 
-		if( ! isset( $_data->plugin ) ) {
+		if ( ! isset( $_data->plugin ) ) {
 			$_data->plugin = $this->name;
 		}
 
@@ -404,7 +404,7 @@ 
 
 		$verify_ssl = $this->verify_ssl();
 		if ( strpos( $url, 'https://' ) !== false && strpos( $url, 'edd_action=package_download' ) ) {
-			$args['sslverify'] = $verify_ssl;
+			$args[ 'sslverify' ] = $verify_ssl;
 		}
 		return $args;
 
@@ -432,9 +432,9 @@ 
 		if ( ! is_array( $edd_plugin_url_available ) || ! isset( $edd_plugin_url_available[ $store_hash ] ) ) {
 			$test_url_parts = parse_url( $this->api_url );
 
-			$scheme = ! empty( $test_url_parts['scheme'] ) ? $test_url_parts['scheme']     : 'http';
-			$host   = ! empty( $test_url_parts['host'] )   ? $test_url_parts['host']       : '';
-			$port   = ! empty( $test_url_parts['port'] )   ? ':' . $test_url_parts['port'] : '';
+			$scheme = ! empty( $test_url_parts[ 'scheme' ] ) ? $test_url_parts[ 'scheme' ] : 'http';
+			$host   = ! empty( $test_url_parts[ 'host' ] ) ? $test_url_parts[ 'host' ] : '';
+			$port   = ! empty( $test_url_parts[ 'port' ] ) ? ':' . $test_url_parts[ 'port' ] : '';
 
 			if ( empty( $host ) ) {
 				$edd_plugin_url_available[ $store_hash ] = false;
@@ -451,27 +451,27 @@ 
 
 		$data = array_merge( $this->api_data, $_data );
 
-		if ( $data['slug'] != $this->slug ) {
+		if ( $data[ 'slug' ] != $this->slug ) {
 			return;
 		}
 
-		if( $this->api_url == trailingslashit ( home_url() ) ) {
+		if ( $this->api_url == trailingslashit( home_url() ) ) {
 			return false; // Don't allow a plugin to ping itself
 		}
 
 		$api_params = array(
 			'edd_action' => 'get_version',
-			'license'    => ! empty( $data['license'] ) ? $data['license'] : '',
-			'item_name'  => isset( $data['item_name'] ) ? $data['item_name'] : false,
-			'item_id'    => isset( $data['item_id'] ) ? $data['item_id'] : false,
-			'version'    => isset( $data['version'] ) ? $data['version'] : false,
-			'slug'       => $data['slug'],
-			'author'     => $data['author'],
+			'license'    => ! empty( $data[ 'license' ] ) ? $data[ 'license' ] : '',
+			'item_name'  => isset( $data[ 'item_name' ] ) ? $data[ 'item_name' ] : false,
+			'item_id'    => isset( $data[ 'item_id' ] ) ? $data[ 'item_id' ] : false,
+			'version'    => isset( $data[ 'version' ] ) ? $data[ 'version' ] : false,
+			'slug'       => $data[ 'slug' ],
+			'author'     => $data[ 'author' ],
 			'url'        => home_url(),
-			'beta'       => ! empty( $data['beta'] ),
+			'beta'       => ! empty( $data[ 'beta' ] ),
 		);
 
-		$request    = wp_remote_post( $this->api_url, array( 'timeout' => 15, 'sslverify' => $verify_ssl, 'body' => $api_params ) );
+		$request = wp_remote_post( $this->api_url, array( 'timeout' => 15, 'sslverify' => $verify_ssl, 'body' => $api_params ) );
 
 		if ( ! is_wp_error( $request ) ) {
 			$request = json_decode( wp_remote_retrieve_body( $request ) );
@@ -491,9 +491,9 @@ 
 			$request->icons = maybe_unserialize( $request->icons );
 		}
 
-		if( ! empty( $request->sections ) ) {
-			foreach( $request->sections as $key => $section ) {
-				$request->$key = (array) $section;
+		if ( ! empty( $request->sections ) ) {
+			foreach ( $request->sections as $key => $section ) {
+				$request->$key = (array)$section;
 			}
 		}
 
@@ -507,35 +507,35 @@ 
 
 		global $edd_plugin_data;
 
-		if( empty( $_REQUEST['edd_sl_action'] ) || 'view_plugin_changelog' != $_REQUEST['edd_sl_action'] ) {
+		if ( empty( $_REQUEST[ 'edd_sl_action' ] ) || 'view_plugin_changelog' != $_REQUEST[ 'edd_sl_action' ] ) {
 			return;
 		}
 
-		if( empty( $_REQUEST['plugin'] ) ) {
+		if ( empty( $_REQUEST[ 'plugin' ] ) ) {
 			return;
 		}
 
-		if( empty( $_REQUEST['slug'] ) ) {
+		if ( empty( $_REQUEST[ 'slug' ] ) ) {
 			return;
 		}
 
-		if( ! current_user_can( 'update_plugins' ) ) {
+		if ( ! current_user_can( 'update_plugins' ) ) {
 			wp_die( __( 'You do not have permission to install plugin updates', 'gravityview' ), __( 'Error', 'gravityview' ), array( 'response' => 403 ) );
 		}
 
-		$data         = $edd_plugin_data[ $_REQUEST['slug'] ];
+		$data         = $edd_plugin_data[ $_REQUEST[ 'slug' ] ];
 		$version_info = $this->get_cached_version_info();
 
-		if( false === $version_info ) {
+		if ( false === $version_info ) {
 
 			$api_params = array(
 				'edd_action' => 'get_version',
-				'item_name'  => isset( $data['item_name'] ) ? $data['item_name'] : false,
-				'item_id'    => isset( $data['item_id'] ) ? $data['item_id'] : false,
-				'slug'       => $_REQUEST['slug'],
-				'author'     => $data['author'],
+				'item_name'  => isset( $data[ 'item_name' ] ) ? $data[ 'item_name' ] : false,
+				'item_id'    => isset( $data[ 'item_id' ] ) ? $data[ 'item_id' ] : false,
+				'slug'       => $_REQUEST[ 'slug' ],
+				'author'     => $data[ 'author' ],
 				'url'        => home_url(),
-				'beta'       => ! empty( $data['beta'] )
+				'beta'       => ! empty( $data[ 'beta' ] )
 			);
 
 			$verify_ssl = $this->verify_ssl();
@@ -551,22 +551,22 @@ 
 				$version_info = false;
 			}
 
-			if( ! empty( $version_info ) ) {
-				foreach( $version_info->sections as $key => $section ) {
-					$version_info->$key = (array) $section;
+			if ( ! empty( $version_info ) ) {
+				foreach ( $version_info->sections as $key => $section ) {
+					$version_info->$key = (array)$section;
 				}
 			}
 
 			$this->set_version_info_cache( $version_info );
 
 			// Delete the unneeded option
-			delete_option( md5( 'edd_plugin_' . sanitize_key( $_REQUEST['plugin'] ) . '_' . $this->beta . '_version_info' ) );
+			delete_option( md5( 'edd_plugin_' . sanitize_key( $_REQUEST[ 'plugin' ] ) . '_' . $this->beta . '_version_info' ) );
 		}
 
 		if ( isset( $version_info->sections ) ) {
 			$sections = $this->convert_object_to_array( $version_info->sections );
-			if ( ! empty( $sections['changelog'] ) ) {
-				echo '<div style="background:#fff;padding:10px;">' . wp_kses_post( $sections['changelog'] ) . '</div>';
+			if ( ! empty( $sections[ 'changelog' ] ) ) {
+				echo '<div style="background:#fff;padding:10px;">' . wp_kses_post( $sections[ 'changelog' ] ) . '</div>';
 			}
 		}
 
@@ -581,25 +581,25 @@ 
 	 */
 	public function get_cached_version_info( $cache_key = '' ) {
 
-		if( empty( $cache_key ) ) {
+		if ( empty( $cache_key ) ) {
 			$cache_key = $this->cache_key;
 		}
 
 		$cache = get_option( $cache_key );
 
-		if( empty( $cache['timeout'] ) || time() > $cache['timeout'] ) {
+		if ( empty( $cache[ 'timeout' ] ) || time() > $cache[ 'timeout' ] ) {
 			return false; // Cache is expired
 		}
 
 		// We need to turn the icons into an array, thanks to WP Core forcing these into an object at some point.
-		$cache['value'] = json_decode( $cache['value'] );
-		if ( ! empty( $cache['value']->icons ) ) {
-			$cache['value']->icons = (array) $cache['value']->icons;
+		$cache[ 'value' ] = json_decode( $cache[ 'value' ] );
+		if ( ! empty( $cache[ 'value' ]->icons ) ) {
+			$cache[ 'value' ]->icons = (array)$cache[ 'value' ]->icons;
 		} else {
-			$cache['value']->icons = array();
+			$cache[ 'value' ]->icons = array();
 		}
 
-		return $cache['value'];
+		return $cache[ 'value' ];
 
 	}
 
@@ -611,7 +611,7 @@ 
 	 */
 	public function set_version_info_cache( $value = '', $cache_key = '' ) {
 
-		if( empty( $cache_key ) ) {
+		if ( empty( $cache_key ) ) {
 			$cache_key = $this->cache_key;
 		}
 
@@ -623,7 +623,7 @@ 
 		update_option( $cache_key, $data, 'no' );
 
 		// Delete the duplicate option
-		delete_option( 'edd_api_request_' . md5( serialize( $this->slug . $this->api_data['license'] . $this->beta ) ) );
+		delete_option( 'edd_api_request_' . md5( serialize( $this->slug . $this->api_data[ 'license' ] . $this->beta ) ) );
 	}
 
 	/**
@@ -633,7 +633,7 @@ 
 	 * @return bool
 	 */
 	private function verify_ssl() {
-		return (bool) apply_filters( 'edd_sl_api_request_verify_ssl', true, $this );
+		return (bool)apply_filters( 'edd_sl_api_request_verify_ssl', true, $this );
 	}
 
 }

includes/load-plugin-and-theme-hooks.php

Spacing +2 added lines, -2 removed lines

@@ -21,13 +21,13 @@
 $plugin_hooks_files = glob( $include_path . 'class-gravityview-plugin-hooks-*.php' );
 
 // Load all plugin files automatically
-foreach ( (array) $plugin_hooks_files as $plugin_hooks_file ) {
+foreach ( (array)$plugin_hooks_files as $plugin_hooks_file ) {
 	include $plugin_hooks_file;
 }
 
 $theme_hooks_files = glob( $include_path . 'class-gravityview-theme-hooks-*.php' );
 
 // Load all theme files automatically
-foreach ( (array) $theme_hooks_files as $theme_hooks_file ) {
+foreach ( (array)$theme_hooks_files as $theme_hooks_file ) {
 	include $theme_hooks_file;
 }

includes/plugin-and-theme-hooks/class-gravityview-theme-hooks-wpml.php

Spacing +5 added lines, -5 removed lines

@@ -64,7 +64,7 @@ 
 
 		add_filter( 'icl_ls_languages', array( $this, 'wpml_ls_filter' ) );
 
-		add_filter( 'gravityview_directory_link', array( $this, 'filter_gravityview_back_link') );
+		add_filter( 'gravityview_directory_link', array( $this, 'filter_gravityview_back_link' ) );
 	}
 
 	/**
@@ -84,7 +84,7 @@ 
 	function filter_gravityview_back_link( $link ) {
 		global $wpml_url_filters;
 
-		if( $wpml_url_filters ) {
+		if ( $wpml_url_filters ) {
 			$link = $wpml_url_filters->permalink_filter( $link, GravityView_frontend::getInstance()->getPostId() );
 		}
 
@@ -101,7 +101,7 @@ 
 	private function remove_url_hooks() {
 		global $wpml_url_filters;
 
-		if( ! $wpml_url_filters ) {
+		if ( ! $wpml_url_filters ) {
 			return;
 		}
 
@@ -129,7 +129,7 @@ 
 	private function add_url_hooks() {
 		global $wpml_url_filters;
 
-		if( ! $wpml_url_filters ) {
+		if ( ! $wpml_url_filters ) {
 			return;
 		}
 
@@ -226,7 +226,7 @@ 
 					break;
 			}
 
-			$languages[ $lang_code ]['url'] = $entry_link;
+			$languages[ $lang_code ][ 'url' ] = $entry_link;
 		}
 
 		$this->add_url_hooks();

future/includes/class-gv-settings-addon.php

Doc Comments +1 added lines, -1 removed lines

@@ -140,7 +140,7 @@
 	 *
 	 * @param array $items Styles to exclude from no-conflict
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	public function register_no_conflict( $items ) {
 

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -421,7 +421,7 @@
     </header>
     <div class="gform-settings-panel__content" style="padding: 0 1rem 1.25rem">
 
-HTML;
+html;
 		} else {
 			echo '<div class="gv-uninstall-form-wrapper" style="font-size: 110%; padding: 15px 0;">';
 		}

Spacing +49 added lines, -49 removed lines

@@ -118,7 +118,7 @@ 
 		$setting_tabs = parent::get_app_settings_tabs();
 
 		foreach ( $setting_tabs as &$tab ) {
-			if ( 'uninstall' !== $tab['name'] ) {
+			if ( 'uninstall' !== $tab[ 'name' ] ) {
 				continue;
 			}
 
@@ -129,7 +129,7 @@ 
 			}
 
 			// Add trash can icon to resemble the look and feel of the GF Settings page
-			$tab['icon'] = 'dashicons-trash';
+			$tab[ 'icon' ] = 'dashicons-trash';
 		}
 
 		return array_filter( $setting_tabs );
@@ -144,8 +144,8 @@ 
 	 */
 	public function register_no_conflict( $items ) {
 
-		$items[] = 'gform_settings';
-		$items[] = 'gv-admin-edd-license';
+		$items[ ] = 'gform_settings';
+		$items[ ] = 'gv-admin-edd-license';
 
 		return $items;
 	}
@@ -196,7 +196,7 @@ 
 
 		$href = add_query_arg( array( 'post_type' => 'gravityview', 'page' => 'gravityview_settings', 'view' => 'uninstall' ), admin_url( 'edit.php' ) );
 
-		$uninstall_button = '<a href="' . esc_url( $href ) . '" class="' . gravityview_sanitize_html_class( $html_class ). '">' . esc_html__( 'Uninstall GravityView', 'gravityview' ) . '</a>';
+		$uninstall_button = '<a href="' . esc_url( $href ) . '" class="' . gravityview_sanitize_html_class( $html_class ) . '">' . esc_html__( 'Uninstall GravityView', 'gravityview' ) . '</a>';
 
 		$html .= $uninstall_button;
 
@@ -215,10 +215,10 @@ 
 	 */
 	public function settings_save( $field, $echo = true ) {
 
-		$field['type']  = 'submit';
-		$field['name']  = 'gform-settings-save';
-		$field['class'] = 'button button-primary primary button-hero';
-		$field['value'] = Utils::get( $field, 'value', __( 'Update Settings', 'gravityview' ) );
+		$field[ 'type' ]  = 'submit';
+		$field[ 'name' ]  = 'gform-settings-save';
+		$field[ 'class' ] = 'button button-primary primary button-hero';
+		$field[ 'value' ] = Utils::get( $field, 'value', __( 'Update Settings', 'gravityview' ) );
 
 		$html = $this->as_html( $field, false );
 
@@ -240,8 +240,8 @@ 
 	 */
 	public function modify_app_settings_menu_title( $setting_tabs ) {
 
-		$setting_tabs[0]['label'] = __( 'GravityView Settings', 'gravityview' );
-		$setting_tabs[0]['icon']  = 'dashicons-admin-settings';
+		$setting_tabs[ 0 ][ 'label' ] = __( 'GravityView Settings', 'gravityview' );
+		$setting_tabs[ 0 ][ 'icon' ]  = 'dashicons-admin-settings';
 
 		return $setting_tabs;
 	}
@@ -489,7 +489,7 @@ 
 					$i = 0;
 					while ( $i < 11 ) {
 						echo '<li class="inline number-scale"><label><input name="likely_to_refer" id="likely_to_refer_' . $i . '" value="' . $i . '" type="radio"> ' . $i . '</label></li>';
-						$i ++;
+						$i++;
 					}
 					?>
 				</ul>
@@ -694,14 +694,14 @@ 
 	 */
 	public function as_html( $field, $echo = true ) {
 
-		$field['type'] = ( isset( $field['type'] ) && in_array( $field['type'], array( 'submit', 'reset', 'button' ) ) ) ? $field['type'] : 'submit';
+		$field[ 'type' ] = ( isset( $field[ 'type' ] ) && in_array( $field[ 'type' ], array( 'submit', 'reset', 'button' ) ) ) ? $field[ 'type' ] : 'submit';
 
 		$attributes    = $this->get_field_attributes( $field );
 		$default_value = Utils::get( $field, 'value', Utils::get( $field, 'default_value' ) );
-		$value         = $this->get( $field['name'], $default_value );
+		$value         = $this->get( $field[ 'name' ], $default_value );
 
-		$attributes['class'] = isset( $attributes['class'] ) ? esc_attr( $attributes['class'] ) : 'button-primary primary gfbutton';
-		$name                = ( $field['name'] === 'gform-settings-save' ) ? $field['name'] : '_gaddon_setting_' . $field['name'];
+		$attributes[ 'class' ] = isset( $attributes[ 'class' ] ) ? esc_attr( $attributes[ 'class' ] ) : 'button-primary primary gfbutton';
+		$name                = ( $field[ 'name' ] === 'gform-settings-save' ) ? $field[ 'name' ] : '_gaddon_setting_' . $field[ 'name' ];
 
 		if ( empty( $value ) ) {
 			$value = __( 'Update Settings', 'gravityview' );
@@ -710,7 +710,7 @@ 
 		$attributes = $this->get_field_attributes( $field );
 
 		$html = '<input
-                    type="' . $field['type'] . '"
+                    type="' . $field[ 'type' ] . '"
                     name="' . esc_attr( $name ) . '"
                     value="' . $value . '" ' .
 				implode( ' ', $attributes ) .
@@ -744,7 +744,7 @@ 
 	 */
 	public function is_save_postback() {
 
-		return isset( $_POST['gform-settings-save'] ) && isset( $_POST['_gravityview_save_settings_nonce'] );
+		return isset( $_POST[ 'gform-settings-save' ] ) && isset( $_POST[ '_gravityview_save_settings_nonce' ] );
 	}
 
 	/**
@@ -856,7 +856,7 @@ 
 
 		$scripts = parent::scripts();
 
-		$scripts[] = array(
+		$scripts[ ] = array(
 				'handle'  => 'gform_tooltip_init',
 				'enqueue' => array(
 						array(
@@ -884,11 +884,11 @@ 
 		);
 
 		// This file was removed from 2.5
-		if( ! gravityview()->plugin->is_GF_25() ) {
-			$deps[] = 'gform_tooltip';
+		if ( ! gravityview()->plugin->is_GF_25() ) {
+			$deps[ ] = 'gform_tooltip';
 		}
 
-		$styles[] = array(
+		$styles[ ] = array(
 			'handle'  => 'gravityview_settings',
 			'src'     => plugins_url( 'assets/css/admin-settings.css', GRAVITYVIEW_FILE ),
 			'version' => Plugin::$version,
@@ -975,7 +975,7 @@ 
 						'name'          => 'support-email',
 						'type'          => 'text',
 						'validate'      => 'email',
-						'default_value' => $default_settings['support-email'],
+						'default_value' => $default_settings[ 'support-email' ],
 						'label'         => __( 'Support Email', 'gravityview' ),
 						'description'   => __( 'In order to provide responses to your support requests, please provide your email address.', 'gravityview' ),
 						'class'         => 'code regular-text',
@@ -987,7 +987,7 @@ 
 						'name'          => 'support_port',
 						'type'          => 'radio',
 						'label'         => __( 'Show Support Port?', 'gravityview' ),
-						'default_value' => $default_settings['support_port'],
+						'default_value' => $default_settings[ 'support_port' ],
 						'horizontal'    => 1,
 						'choices'       => array(
 								array(
@@ -1006,7 +1006,7 @@ 
 						'name'          => 'no-conflict-mode',
 						'type'          => 'radio',
 						'label'         => __( 'No-Conflict Mode', 'gravityview' ),
-						'default_value' => $default_settings['no-conflict-mode'],
+						'default_value' => $default_settings[ 'no-conflict-mode' ],
 						'horizontal'    => 1,
 						'choices'       => array(
 								array(
@@ -1028,7 +1028,7 @@ 
 								'name'          => 'rest_api',
 								'type'          => 'radio',
 								'label'         => __( 'REST API', 'gravityview' ),
-								'default_value' => $default_settings['rest_api'],
+								'default_value' => $default_settings[ 'rest_api' ],
 								'horizontal'    => 1,
 								'choices'       => array(
 										array(
@@ -1047,7 +1047,7 @@ 
 					'name' => 'powered_by',
 					'type' => 'checkbox',
 					'label' => __( 'Display "Powered By" Link', 'gravityview' ),
-					'default_value' => $default_settings['powered_by'],
+					'default_value' => $default_settings[ 'powered_by' ],
 					'choices' => array(
 						array(
 							'label' => esc_html__( 'Display a "Powered by GravityView" link', 'gravityview' ),
@@ -1072,7 +1072,7 @@ 
 						'name'          => 'beta',
 						'type'          => 'checkbox',
 						'label'         => __( 'Become a Beta Tester', 'gravityview' ),
-						'default_value' => $default_settings['beta'],
+						'default_value' => $default_settings[ 'beta' ],
 						'horizontal'    => 1,
 						'choices'       => array(
 								array(
@@ -1106,17 +1106,17 @@ 
 		 * @since 1.7.4
 		 */
 		foreach ( $fields as &$field ) {
-			$field['name']          = isset( $field['name'] ) ? $field['name'] : Utils::get( $field, 'id' );
-			$field['label']         = isset( $field['label'] ) ? $field['label'] : Utils::get( $field, 'title' );
-			$field['default_value'] = isset( $field['default_value'] ) ? $field['default_value'] : Utils::get( $field, 'default' );
-			$field['description']   = isset( $field['description'] ) ? $field['description'] : Utils::get( $field, 'subtitle' );
+			$field[ 'name' ]          = isset( $field[ 'name' ] ) ? $field[ 'name' ] : Utils::get( $field, 'id' );
+			$field[ 'label' ]         = isset( $field[ 'label' ] ) ? $field[ 'label' ] : Utils::get( $field, 'title' );
+			$field[ 'default_value' ] = isset( $field[ 'default_value' ] ) ? $field[ 'default_value' ] : Utils::get( $field, 'default' );
+			$field[ 'description' ]   = isset( $field[ 'description' ] ) ? $field[ 'description' ] : Utils::get( $field, 'subtitle' );
 
 			if ( $disabled_attribute ) {
-				$field['disabled'] = $disabled_attribute;
+				$field[ 'disabled' ] = $disabled_attribute;
 			}
 
-			if ( empty( $field['disabled'] ) ) {
-				unset( $field['disabled'] );
+			if ( empty( $field[ 'disabled' ] ) ) {
+				unset( $field[ 'disabled' ] );
 			}
 		}
 
@@ -1128,23 +1128,23 @@ 
 					'description' => __( 'Enter the license key that was sent to you on purchase. This enables plugin updates &amp; support.', 'gravityview' ),
 					'type' => 'edd_license',
 					'data-pending-text' => __( 'Verifying license&hellip;', 'gravityview' ),
-					'default_value' => $default_settings['license_key'],
+					'default_value' => $default_settings[ 'license_key' ],
 					'class' => ( '' == $this->get( 'license_key' ) ) ? 'activate code regular-text edd-license-key' : 'deactivate code regular-text edd-license-key',
 			),
 			array(
 					'name' => 'license_key_response',
-					'default_value' => $default_settings['license_key_response'],
+					'default_value' => $default_settings[ 'license_key_response' ],
 					'type' => 'hidden',
 			),
 			array(
 					'name' => 'license_key_status',
-					'default_value' => $default_settings['license_key_status'],
+					'default_value' => $default_settings[ 'license_key_status' ],
 					'type' => 'hidden',
 			),
 		);
 
 		if ( defined( 'GRAVITYVIEW_LICENSE_KEY' ) && GRAVITYVIEW_LICENSE_KEY ) {
-			$license_fields[0] = array_merge( $license_fields[0], array(
+			$license_fields[ 0 ] = array_merge( $license_fields[ 0 ], array(
 				'disabled' => true,
 				'title'    => __( 'The license key is defined by your site\'s configuration file.', 'gravityview' ),
 			) );
@@ -1155,7 +1155,7 @@ 
 
 		if ( \gravityview()->plugin->is_GF_25() ) {
 
-			$sections[] = array(
+			$sections[ ] = array(
 					'title'       => __( 'GravityView License', 'gravityview' ),
 					'class'       => 'gform-settings-panel--full gv-settings-panel--license',
 					'description' => $version_info,
@@ -1173,7 +1173,7 @@ 
 			) );
 		}
 
-		$sections[] = array(
+		$sections[ ] = array(
 			'title' => ( gravityview()->plugin->is_GF_25() ? __( 'GravityView Settings', 'gravityview' ) : null ),
 			'class' => 'gform-settings-panel--full gv-settings-panel--core',
 			'fields'      => $fields,
@@ -1197,8 +1197,8 @@ 
 
 			if ( $disabled_attribute ) {
 				foreach ( $extension_sections as &$section ) {
-					foreach ( $section['fields'] as &$field ) {
-						$field['disabled'] = $disabled_attribute;
+					foreach ( $section[ 'fields' ] as &$field ) {
+						$field[ 'disabled' ] = $disabled_attribute;
 					}
 				}
 			}
@@ -1264,7 +1264,7 @@ 
 	public function settings_edd_license( $field, $echo = true ) {
 
 		if ( defined( 'GRAVITYVIEW_LICENSE_KEY' ) && GRAVITYVIEW_LICENSE_KEY ) {
-			$field['input_type'] = 'password';
+			$field[ 'input_type' ] = 'password';
 		}
 
 		$text = $this->settings_text( $field, false );
@@ -1316,7 +1316,7 @@ 
 
 		?>
 
-		<tr id="gaddon-setting-row-<?php echo esc_attr( $field['name'] ); ?>">
+		<tr id="gaddon-setting-row-<?php echo esc_attr( $field[ 'name' ] ); ?>">
 			<td colspan="2">
 				<?php $this->single_setting( $field ); ?>
 			</td>
@@ -1342,8 +1342,8 @@ 
 	 */
 	public function single_setting_row( $field ) {
 
-		$field['gv_description'] = Utils::get( $field, 'description' );
-		$field['description']    = Utils::get( $field, 'subtitle' );
+		$field[ 'gv_description' ] = Utils::get( $field, 'description' );
+		$field[ 'description' ]    = Utils::get( $field, 'subtitle' );
 		parent::single_setting_row( $field );
 	}
 
@@ -1404,8 +1404,8 @@ 
 		// then we assume it's changed. If it's changed, unset the status and the previous response.
 		if ( ! $added_message && ( $local_key !== $response_key ) ) {
 
-			unset( $posted_settings['license_key_response'] );
-			unset( $posted_settings['license_key_status'] );
+			unset( $posted_settings[ 'license_key_response' ] );
+			unset( $posted_settings[ 'license_key_status' ] );
 
 			\GFCommon::add_error_message( __( 'The license key you entered has been saved, but not activated. Please activate the license.', 'gravityview' ) );
 

vendor/phpcompatibility/php-compatibility/PHPCompatibility/AbstractFunctionCallParameterSniff.php

Doc Comments +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
     /**
      * Returns an array of tokens this test wants to listen for.
      *
-     * @return array
+     * @return integer[]
      */
     public function register()
     {

Indentation +153 added lines, -153 removed lines

@@ -24,157 +24,157 @@
  */
 abstract class AbstractFunctionCallParameterSniff extends Sniff
 {
-    /**
-     * Is the sniff looking for a function call or a method call ?
-     *
-     * Note: the child class may need to do additional checks to make sure that
-     * the method called is of the right class/object.
-     * Checking that is outside of the scope of this abstract sniff.
-     *
-     * @var bool False (default) if the sniff is looking for function calls.
-     *           True if the sniff is looking for method calls.
-     */
-    protected $isMethod = false;
-
-    /**
-     * Functions the sniff is looking for. Should be defined in the child class.
-     *
-     * @var array The only requirement for this array is that the top level
-     *            array keys are the names of the functions you're looking for.
-     *            Other than that, the array can have arbitrary content
-     *            depending on your needs.
-     */
-    protected $targetFunctions = array();
-
-    /**
-     * List of tokens which when they preceed the $stackPtr indicate that this
-     * is not a function call.
-     *
-     * @var array
-     */
-    private $ignoreTokens = array(
-        \T_DOUBLE_COLON    => true,
-        \T_OBJECT_OPERATOR => true,
-        \T_FUNCTION        => true,
-        \T_NEW             => true,
-        \T_CONST           => true,
-        \T_USE             => true,
-    );
-
-
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        // Handle case-insensitivity of function names.
-        $this->targetFunctions = $this->arrayKeysToLowercase($this->targetFunctions);
-
-        return array(\T_STRING);
-    }
-
-
-    /**
-     * Processes this test, when one of its tokens is encountered.
-     *
-     * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
-     * @param int                   $stackPtr  The position of the current token in
-     *                                         the stack passed in $tokens.
-     *
-     * @return int|void Integer stack pointer to skip forward or void to continue
-     *                  normal file processing.
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        if ($this->bowOutEarly() === true) {
-            return;
-        }
-
-        $tokens     = $phpcsFile->getTokens();
-        $function   = $tokens[$stackPtr]['content'];
-        $functionLc = strtolower($function);
-
-        if (isset($this->targetFunctions[$functionLc]) === false) {
-            return;
-        }
-
-        $prevNonEmpty = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($stackPtr - 1), null, true);
-
-        if ($this->isMethod === true) {
-            if ($tokens[$prevNonEmpty]['code'] !== \T_DOUBLE_COLON
-                && $tokens[$prevNonEmpty]['code'] !== \T_OBJECT_OPERATOR
-            ) {
-                // Not a call to a PHP method.
-                return;
-            }
-        } else {
-            if (isset($this->ignoreTokens[$tokens[$prevNonEmpty]['code']]) === true) {
-                // Not a call to a PHP function.
-                return;
-            }
-
-            if ($tokens[$prevNonEmpty]['code'] === \T_NS_SEPARATOR
-                && $tokens[$prevNonEmpty - 1]['code'] === \T_STRING
-            ) {
-                // Namespaced function.
-                return;
-            }
-        }
-
-        $parameters = $this->getFunctionCallParameters($phpcsFile, $stackPtr);
-
-        if (empty($parameters)) {
-            return $this->processNoParameters($phpcsFile, $stackPtr, $function);
-        } else {
-            return $this->processParameters($phpcsFile, $stackPtr, $function, $parameters);
-        }
-    }
-
-
-    /**
-     * Do a version check to determine if this sniff needs to run at all.
-     *
-     * If the check done in a child class is not specific to one PHP version,
-     * this function should return `false`.
-     *
-     * @return bool
-     */
-    abstract protected function bowOutEarly();
-
-
-    /**
-     * Process the parameters of a matched function.
-     *
-     * This method has to be made concrete in child classes.
-     *
-     * @param \PHP_CodeSniffer_File $phpcsFile    The file being scanned.
-     * @param int                   $stackPtr     The position of the current token in the stack.
-     * @param string                $functionName The token content (function name) which was matched.
-     * @param array                 $parameters   Array with information about the parameters.
-     *
-     * @return int|void Integer stack pointer to skip forward or void to continue
-     *                  normal file processing.
-     */
-    abstract public function processParameters(File $phpcsFile, $stackPtr, $functionName, $parameters);
-
-
-    /**
-     * Process the function if no parameters were found.
-     *
-     * Defaults to doing nothing. Can be overloaded in child classes to handle functions
-     * were parameters are expected, but none found.
-     *
-     * @param \PHP_CodeSniffer_File $phpcsFile    The file being scanned.
-     * @param int                   $stackPtr     The position of the current token in the stack.
-     * @param string                $functionName The token content (function name) which was matched.
-     *
-     * @return int|void Integer stack pointer to skip forward or void to continue
-     *                  normal file processing.
-     */
-    public function processNoParameters(File $phpcsFile, $stackPtr, $functionName)
-    {
-        return;
-    }
+	/**
+	 * Is the sniff looking for a function call or a method call ?
+	 *
+	 * Note: the child class may need to do additional checks to make sure that
+	 * the method called is of the right class/object.
+	 * Checking that is outside of the scope of this abstract sniff.
+	 *
+	 * @var bool False (default) if the sniff is looking for function calls.
+	 *           True if the sniff is looking for method calls.
+	 */
+	protected $isMethod = false;
+
+	/**
+	 * Functions the sniff is looking for. Should be defined in the child class.
+	 *
+	 * @var array The only requirement for this array is that the top level
+	 *            array keys are the names of the functions you're looking for.
+	 *            Other than that, the array can have arbitrary content
+	 *            depending on your needs.
+	 */
+	protected $targetFunctions = array();
+
+	/**
+	 * List of tokens which when they preceed the $stackPtr indicate that this
+	 * is not a function call.
+	 *
+	 * @var array
+	 */
+	private $ignoreTokens = array(
+		\T_DOUBLE_COLON    => true,
+		\T_OBJECT_OPERATOR => true,
+		\T_FUNCTION        => true,
+		\T_NEW             => true,
+		\T_CONST           => true,
+		\T_USE             => true,
+	);
+
+
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		// Handle case-insensitivity of function names.
+		$this->targetFunctions = $this->arrayKeysToLowercase($this->targetFunctions);
+
+		return array(\T_STRING);
+	}
+
+
+	/**
+	 * Processes this test, when one of its tokens is encountered.
+	 *
+	 * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
+	 * @param int                   $stackPtr  The position of the current token in
+	 *                                         the stack passed in $tokens.
+	 *
+	 * @return int|void Integer stack pointer to skip forward or void to continue
+	 *                  normal file processing.
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		if ($this->bowOutEarly() === true) {
+			return;
+		}
+
+		$tokens     = $phpcsFile->getTokens();
+		$function   = $tokens[$stackPtr]['content'];
+		$functionLc = strtolower($function);
+
+		if (isset($this->targetFunctions[$functionLc]) === false) {
+			return;
+		}
+
+		$prevNonEmpty = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($stackPtr - 1), null, true);
+
+		if ($this->isMethod === true) {
+			if ($tokens[$prevNonEmpty]['code'] !== \T_DOUBLE_COLON
+				&& $tokens[$prevNonEmpty]['code'] !== \T_OBJECT_OPERATOR
+			) {
+				// Not a call to a PHP method.
+				return;
+			}
+		} else {
+			if (isset($this->ignoreTokens[$tokens[$prevNonEmpty]['code']]) === true) {
+				// Not a call to a PHP function.
+				return;
+			}
+
+			if ($tokens[$prevNonEmpty]['code'] === \T_NS_SEPARATOR
+				&& $tokens[$prevNonEmpty - 1]['code'] === \T_STRING
+			) {
+				// Namespaced function.
+				return;
+			}
+		}
+
+		$parameters = $this->getFunctionCallParameters($phpcsFile, $stackPtr);
+
+		if (empty($parameters)) {
+			return $this->processNoParameters($phpcsFile, $stackPtr, $function);
+		} else {
+			return $this->processParameters($phpcsFile, $stackPtr, $function, $parameters);
+		}
+	}
+
+
+	/**
+	 * Do a version check to determine if this sniff needs to run at all.
+	 *
+	 * If the check done in a child class is not specific to one PHP version,
+	 * this function should return `false`.
+	 *
+	 * @return bool
+	 */
+	abstract protected function bowOutEarly();
+
+
+	/**
+	 * Process the parameters of a matched function.
+	 *
+	 * This method has to be made concrete in child classes.
+	 *
+	 * @param \PHP_CodeSniffer_File $phpcsFile    The file being scanned.
+	 * @param int                   $stackPtr     The position of the current token in the stack.
+	 * @param string                $functionName The token content (function name) which was matched.
+	 * @param array                 $parameters   Array with information about the parameters.
+	 *
+	 * @return int|void Integer stack pointer to skip forward or void to continue
+	 *                  normal file processing.
+	 */
+	abstract public function processParameters(File $phpcsFile, $stackPtr, $functionName, $parameters);
+
+
+	/**
+	 * Process the function if no parameters were found.
+	 *
+	 * Defaults to doing nothing. Can be overloaded in child classes to handle functions
+	 * were parameters are expected, but none found.
+	 *
+	 * @param \PHP_CodeSniffer_File $phpcsFile    The file being scanned.
+	 * @param int                   $stackPtr     The position of the current token in the stack.
+	 * @param string                $functionName The token content (function name) which was matched.
+	 *
+	 * @return int|void Integer stack pointer to skip forward or void to continue
+	 *                  normal file processing.
+	 */
+	public function processNoParameters(File $phpcsFile, $stackPtr, $functionName)
+	{
+		return;
+	}
 }

Spacing +20 added lines, -20 removed lines

@@ -70,9 +70,9 @@ 
     public function register()
     {
         // Handle case-insensitivity of function names.
-        $this->targetFunctions = $this->arrayKeysToLowercase($this->targetFunctions);
+        $this->targetFunctions = $this->arrayKeysToLowercase( $this->targetFunctions );
 
-        return array(\T_STRING);
+        return array( \T_STRING );
     }
 
 
@@ -86,49 +86,49 @@ 
      * @return int|void Integer stack pointer to skip forward or void to continue
      *                  normal file processing.
      */
-    public function process(File $phpcsFile, $stackPtr)
+    public function process( File $phpcsFile, $stackPtr )
     {
-        if ($this->bowOutEarly() === true) {
+        if ( $this->bowOutEarly() === true ) {
             return;
         }
 
         $tokens     = $phpcsFile->getTokens();
-        $function   = $tokens[$stackPtr]['content'];
-        $functionLc = strtolower($function);
+        $function   = $tokens[ $stackPtr ][ 'content' ];
+        $functionLc = strtolower( $function );
 
-        if (isset($this->targetFunctions[$functionLc]) === false) {
+        if ( isset( $this->targetFunctions[ $functionLc ] ) === false ) {
             return;
         }
 
-        $prevNonEmpty = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($stackPtr - 1), null, true);
+        $prevNonEmpty = $phpcsFile->findPrevious( Tokens::$emptyTokens, ( $stackPtr - 1 ), null, true );
 
-        if ($this->isMethod === true) {
-            if ($tokens[$prevNonEmpty]['code'] !== \T_DOUBLE_COLON
-                && $tokens[$prevNonEmpty]['code'] !== \T_OBJECT_OPERATOR
+        if ( $this->isMethod === true ) {
+            if ( $tokens[ $prevNonEmpty ][ 'code' ] !== \T_DOUBLE_COLON
+                && $tokens[ $prevNonEmpty ][ 'code' ] !== \T_OBJECT_OPERATOR
             ) {
                 // Not a call to a PHP method.
                 return;
             }
         } else {
-            if (isset($this->ignoreTokens[$tokens[$prevNonEmpty]['code']]) === true) {
+            if ( isset( $this->ignoreTokens[ $tokens[ $prevNonEmpty ][ 'code' ] ] ) === true ) {
                 // Not a call to a PHP function.
                 return;
             }
 
-            if ($tokens[$prevNonEmpty]['code'] === \T_NS_SEPARATOR
-                && $tokens[$prevNonEmpty - 1]['code'] === \T_STRING
+            if ( $tokens[ $prevNonEmpty ][ 'code' ] === \T_NS_SEPARATOR
+                && $tokens[ $prevNonEmpty - 1 ][ 'code' ] === \T_STRING
             ) {
                 // Namespaced function.
                 return;
             }
         }
 
-        $parameters = $this->getFunctionCallParameters($phpcsFile, $stackPtr);
+        $parameters = $this->getFunctionCallParameters( $phpcsFile, $stackPtr );
 
-        if (empty($parameters)) {
-            return $this->processNoParameters($phpcsFile, $stackPtr, $function);
+        if ( empty( $parameters ) ) {
+            return $this->processNoParameters( $phpcsFile, $stackPtr, $function );
         } else {
-            return $this->processParameters($phpcsFile, $stackPtr, $function, $parameters);
+            return $this->processParameters( $phpcsFile, $stackPtr, $function, $parameters );
         }
     }
 
@@ -157,7 +157,7 @@ 
      * @return int|void Integer stack pointer to skip forward or void to continue
      *                  normal file processing.
      */
-    abstract public function processParameters(File $phpcsFile, $stackPtr, $functionName, $parameters);
+    abstract public function processParameters( File $phpcsFile, $stackPtr, $functionName, $parameters );
 
 
     /**
@@ -173,7 +173,7 @@ 
      * @return int|void Integer stack pointer to skip forward or void to continue
      *                  normal file processing.
      */
-    public function processNoParameters(File $phpcsFile, $stackPtr, $functionName)
+    public function processNoParameters( File $phpcsFile, $stackPtr, $functionName )
     {
         return;
     }

Braces +4 added lines, -8 removed lines

@@ -22,8 +22,7 @@ 
  * @package  PHPCompatibility
  * @author   Juliette Reinders Folmer <phpcompatibility_nospam@adviesenzo.nl>
  */
-abstract class AbstractFunctionCallParameterSniff extends Sniff
-{
+abstract class AbstractFunctionCallParameterSniff extends Sniff {
     /**
      * Is the sniff looking for a function call or a method call ?
      *
@@ -67,8 +66,7 @@ 
      *
      * @return array
      */
-    public function register()
-    {
+    public function register() {
         // Handle case-insensitivity of function names.
         $this->targetFunctions = $this->arrayKeysToLowercase($this->targetFunctions);
 
@@ -86,8 +84,7 @@ 
      * @return int|void Integer stack pointer to skip forward or void to continue
      *                  normal file processing.
      */
-    public function process(File $phpcsFile, $stackPtr)
-    {
+    public function process(File $phpcsFile, $stackPtr) {
         if ($this->bowOutEarly() === true) {
             return;
         }
@@ -173,8 +170,7 @@ 
      * @return int|void Integer stack pointer to skip forward or void to continue
      *                  normal file processing.
      */
-    public function processNoParameters(File $phpcsFile, $stackPtr, $functionName)
-    {
+    public function processNoParameters(File $phpcsFile, $stackPtr, $functionName) {
         return;
     }
 }