gravityview / GravityView

future/includes/class-gv-logger-wp-action.php

Spacing +3 added lines, -3 removed lines

@@ -28,7 +28,7 @@ 
 	protected function log( $level, $message, $context ) {
 
 		$backtrace = debug_backtrace();
-		$location = $this->interpolate( "{class}{type}{function}", $backtrace[2] );
+		$location = $this->interpolate( "{class}{type}{function}", $backtrace[ 2 ] );
 
 		$message = $this->interpolate( "[$level, $location] $message", $context );
 
@@ -52,14 +52,14 @@ 
 			do_action(
 				sprintf( 'gravityview_log_%s_test', $action ),
 				$this->interpolate( $message, $context ),
-				empty( $context['data'] ) ? array() : $context['data']
+				empty( $context[ 'data' ] ) ? array() : $context[ 'data' ]
 			);
 		}
 		
 		do_action(
 			sprintf( 'gravityview_log_%s', $action ),
 			$this->interpolate( $message, $context ),
-			empty( $context['data'] ) ? array() : $context['data']
+			empty( $context[ 'data' ] ) ? array() : $context[ 'data' ]
 		);
 	}
 }

future/includes/class-gv-logger.php

Indentation +119 added lines, -119 removed lines

@@ -10,14 +10,14 @@ 
  * Describes log levels.
  */
 class LogLevel {
-    const EMERGENCY = 'emergency';
-    const ALERT     = 'alert';
-    const CRITICAL  = 'critical';
-    const ERROR     = 'error';
-    const WARNING   = 'warning';
-    const NOTICE    = 'notice';
-    const INFO      = 'info';
-    const DEBUG     = 'debug';
+	const EMERGENCY = 'emergency';
+	const ALERT     = 'alert';
+	const CRITICAL  = 'critical';
+	const ERROR     = 'error';
+	const WARNING   = 'warning';
+	const NOTICE    = 'notice';
+	const INFO      = 'info';
+	const DEBUG     = 'debug';
 }
 
 /**
@@ -28,127 +28,127 @@ 
  * @see https://github.com/php-fig/log/blob/master/Psr/Log/AbstractLogger.php
  */
 abstract class Logger /** @todo extends Psr\Log\AbstractLogger */ {
-    /**
-     * System is unusable.
-     *
-     * @param string $message
-     * @param array  $context
-     *
-     * @return void
-     */
-    public function emergency($message, array $context = array())
-    {
-        $this->log(LogLevel::EMERGENCY, $message, $context);
-    }
+	/**
+	 * System is unusable.
+	 *
+	 * @param string $message
+	 * @param array  $context
+	 *
+	 * @return void
+	 */
+	public function emergency($message, array $context = array())
+	{
+		$this->log(LogLevel::EMERGENCY, $message, $context);
+	}
 
-    /**
-     * Action must be taken immediately.
-     *
-     * Example: Entire website down, database unavailable, etc. This should
-     * trigger the SMS alerts and wake you up.
-     *
-     * @param string $message
-     * @param array  $context
-     *
-     * @return void
-     */
-    public function alert($message, array $context = array())
-    {
-        $this->log(LogLevel::ALERT, $message, $context);
-    }
+	/**
+	 * Action must be taken immediately.
+	 *
+	 * Example: Entire website down, database unavailable, etc. This should
+	 * trigger the SMS alerts and wake you up.
+	 *
+	 * @param string $message
+	 * @param array  $context
+	 *
+	 * @return void
+	 */
+	public function alert($message, array $context = array())
+	{
+		$this->log(LogLevel::ALERT, $message, $context);
+	}
 
-    /**
-     * Critical conditions.
-     *
-     * Example: Application component unavailable, unexpected exception.
-     *
-     * @param string $message
-     * @param array  $context
-     *
-     * @return void
-     */
-    public function critical($message, array $context = array())
-    {
-        $this->log(LogLevel::CRITICAL, $message, $context);
-    }
+	/**
+	 * Critical conditions.
+	 *
+	 * Example: Application component unavailable, unexpected exception.
+	 *
+	 * @param string $message
+	 * @param array  $context
+	 *
+	 * @return void
+	 */
+	public function critical($message, array $context = array())
+	{
+		$this->log(LogLevel::CRITICAL, $message, $context);
+	}
 
-    /**
-     * Runtime errors that do not require immediate action but should typically
-     * be logged and monitored.
-     *
-     * @param string $message
-     * @param array  $context
-     *
-     * @return void
-     */
-    public function error($message, array $context = array())
-    {
-        $this->log(LogLevel::ERROR, $message, $context);
-    }
+	/**
+	 * Runtime errors that do not require immediate action but should typically
+	 * be logged and monitored.
+	 *
+	 * @param string $message
+	 * @param array  $context
+	 *
+	 * @return void
+	 */
+	public function error($message, array $context = array())
+	{
+		$this->log(LogLevel::ERROR, $message, $context);
+	}
 
-    /**
-     * Exceptional occurrences that are not errors.
-     *
-     * Example: Use of deprecated APIs, poor use of an API, undesirable things
-     * that are not necessarily wrong.
-     *
-     * @param string $message
-     * @param array  $context
-     *
-     * @return void
-     */
-    public function warning($message, array $context = array())
-    {
-        $this->log(LogLevel::WARNING, $message, $context);
-    }
+	/**
+	 * Exceptional occurrences that are not errors.
+	 *
+	 * Example: Use of deprecated APIs, poor use of an API, undesirable things
+	 * that are not necessarily wrong.
+	 *
+	 * @param string $message
+	 * @param array  $context
+	 *
+	 * @return void
+	 */
+	public function warning($message, array $context = array())
+	{
+		$this->log(LogLevel::WARNING, $message, $context);
+	}
 
-    /**
-     * Normal but significant events.
-     *
-     * @param string $message
-     * @param array  $context
-     *
-     * @return void
-     */
-    public function notice($message, array $context = array())
-    {
-        $this->log(LogLevel::NOTICE, $message, $context);
-    }
+	/**
+	 * Normal but significant events.
+	 *
+	 * @param string $message
+	 * @param array  $context
+	 *
+	 * @return void
+	 */
+	public function notice($message, array $context = array())
+	{
+		$this->log(LogLevel::NOTICE, $message, $context);
+	}
 
-    /**
-     * Interesting events.
-     *
-     * Example: User logs in, SQL logs.
-     *
-     * @param string $message
-     * @param array  $context
-     *
-     * @return void
-     */
-    public function info($message, array $context = array())
-    {
-        $this->log(LogLevel::INFO, $message, $context);
-    }
+	/**
+	 * Interesting events.
+	 *
+	 * Example: User logs in, SQL logs.
+	 *
+	 * @param string $message
+	 * @param array  $context
+	 *
+	 * @return void
+	 */
+	public function info($message, array $context = array())
+	{
+		$this->log(LogLevel::INFO, $message, $context);
+	}
 
-    /**
-     * Detailed debug information.
-     *
-     * @param string $message
-     * @param array  $context
-     *
-     * @return void
-     */
-    public function debug($message, array $context = array())
-    {
-        $this->log(LogLevel::DEBUG, $message, $context);
-    }
+	/**
+	 * Detailed debug information.
+	 *
+	 * @param string $message
+	 * @param array  $context
+	 *
+	 * @return void
+	 */
+	public function debug($message, array $context = array())
+	{
+		$this->log(LogLevel::DEBUG, $message, $context);
+	}
 
 	/**
 	 * Bake the context into { } placeholders in the message.
-     * @param string $message
-     * @param array  $context
-     *
-     * @return string The baked message;
+	 * @param string $message
+	 * @param array  $context
+	 *
+	 * @return string The baked message;
 	 */
 	protected function interpolate( $message, $context ) {
 		foreach ( $context as $key => $val ) {

Spacing +16 added lines, -16 removed lines

@@ -36,9 +36,9 @@ 
      *
      * @return void
      */
-    public function emergency($message, array $context = array())
+    public function emergency( $message, array $context = array() )
     {
-        $this->log(LogLevel::EMERGENCY, $message, $context);
+        $this->log( LogLevel::EMERGENCY, $message, $context );
     }
 
     /**
@@ -52,9 +52,9 @@ 
      *
      * @return void
      */
-    public function alert($message, array $context = array())
+    public function alert( $message, array $context = array() )
     {
-        $this->log(LogLevel::ALERT, $message, $context);
+        $this->log( LogLevel::ALERT, $message, $context );
     }
 
     /**
@@ -67,9 +67,9 @@ 
      *
      * @return void
      */
-    public function critical($message, array $context = array())
+    public function critical( $message, array $context = array() )
     {
-        $this->log(LogLevel::CRITICAL, $message, $context);
+        $this->log( LogLevel::CRITICAL, $message, $context );
     }
 
     /**
@@ -81,9 +81,9 @@ 
      *
      * @return void
      */
-    public function error($message, array $context = array())
+    public function error( $message, array $context = array() )
     {
-        $this->log(LogLevel::ERROR, $message, $context);
+        $this->log( LogLevel::ERROR, $message, $context );
     }
 
     /**
@@ -97,9 +97,9 @@ 
      *
      * @return void
      */
-    public function warning($message, array $context = array())
+    public function warning( $message, array $context = array() )
     {
-        $this->log(LogLevel::WARNING, $message, $context);
+        $this->log( LogLevel::WARNING, $message, $context );
     }
 
     /**
@@ -110,9 +110,9 @@ 
      *
      * @return void
      */
-    public function notice($message, array $context = array())
+    public function notice( $message, array $context = array() )
     {
-        $this->log(LogLevel::NOTICE, $message, $context);
+        $this->log( LogLevel::NOTICE, $message, $context );
     }
 
     /**
@@ -125,9 +125,9 @@ 
      *
      * @return void
      */
-    public function info($message, array $context = array())
+    public function info( $message, array $context = array() )
     {
-        $this->log(LogLevel::INFO, $message, $context);
+        $this->log( LogLevel::INFO, $message, $context );
     }
 
     /**
@@ -138,9 +138,9 @@ 
      *
      * @return void
      */
-    public function debug($message, array $context = array())
+    public function debug( $message, array $context = array() )
     {
-        $this->log(LogLevel::DEBUG, $message, $context);
+        $this->log( LogLevel::DEBUG, $message, $context );
     }
 
 	/**

Braces +8 added lines, -16 removed lines

@@ -36,8 +36,7 @@ 
      *
      * @return void
      */
-    public function emergency($message, array $context = array())
-    {
+    public function emergency($message, array $context = array()) {
         $this->log(LogLevel::EMERGENCY, $message, $context);
     }
 
@@ -52,8 +51,7 @@ 
      *
      * @return void
      */
-    public function alert($message, array $context = array())
-    {
+    public function alert($message, array $context = array()) {
         $this->log(LogLevel::ALERT, $message, $context);
     }
 
@@ -67,8 +65,7 @@ 
      *
      * @return void
      */
-    public function critical($message, array $context = array())
-    {
+    public function critical($message, array $context = array()) {
         $this->log(LogLevel::CRITICAL, $message, $context);
     }
 
@@ -81,8 +78,7 @@ 
      *
      * @return void
      */
-    public function error($message, array $context = array())
-    {
+    public function error($message, array $context = array()) {
         $this->log(LogLevel::ERROR, $message, $context);
     }
 
@@ -97,8 +93,7 @@ 
      *
      * @return void
      */
-    public function warning($message, array $context = array())
-    {
+    public function warning($message, array $context = array()) {
         $this->log(LogLevel::WARNING, $message, $context);
     }
 
@@ -110,8 +105,7 @@ 
      *
      * @return void
      */
-    public function notice($message, array $context = array())
-    {
+    public function notice($message, array $context = array()) {
         $this->log(LogLevel::NOTICE, $message, $context);
     }
 
@@ -125,8 +119,7 @@ 
      *
      * @return void
      */
-    public function info($message, array $context = array())
-    {
+    public function info($message, array $context = array()) {
         $this->log(LogLevel::INFO, $message, $context);
     }
 
@@ -138,8 +131,7 @@ 
      *
      * @return void
      */
-    public function debug($message, array $context = array())
-    {
+    public function debug($message, array $context = array()) {
         $this->log(LogLevel::DEBUG, $message, $context);
     }
 

future/includes/class-gv-collection-entry-filter-gravityforms.php

Spacing +4 added lines, -4 removed lines

@@ -55,10 +55,10 @@
 						$search_criteria[ $key ] = $field_filters;
 					}
 
-					if ( ! empty( $b[ $key ]['mode'] ) ) {
-						$search_criteria[ $key ]['mode' ] = $b[ $key ]['mode'];
-					} else if ( ! empty( $a[ $key ]['mode'] ) ) {
-						$search_criteria[ $key ]['mode' ] = $a[ $key ]['mode'];
+					if ( ! empty( $b[ $key ][ 'mode' ] ) ) {
+						$search_criteria[ $key ][ 'mode' ] = $b[ $key ][ 'mode' ];
+					} else if ( ! empty( $a[ $key ][ 'mode' ] ) ) {
+						$search_criteria[ $key ][ 'mode' ] = $a[ $key ][ 'mode' ];
 					}
 					break;
 				case 'start_date':

future/includes/class-gv-settings.php

Spacing +2 added lines, -2 removed lines

@@ -26,7 +26,7 @@ 
 	 * @return void
 	 */
 	public function set( $key, $value ) {
-		$this->settings[$key] = $value;
+		$this->settings[ $key ] = $value;
 	}
 
 	/**
@@ -40,7 +40,7 @@ 
 	 * @return mixed|null
 	 */
 	public function get( $key, $default = null ) {
-		return isset( $this->settings[$key] ) ? $this->settings[$key] : $default;
+		return isset( $this->settings[ $key ] ) ? $this->settings[ $key ] : $default;
 	}
 
 	/**

future/includes/class-gv-field.php

Spacing +9 added lines, -9 removed lines

@@ -144,19 +144,19 @@
 	public function from_configuration( $configuration ) {
 		$configuration = wp_parse_args( $configuration, $this->as_configuration() );
 
-		$this->ID = $configuration['id'];
-		$this->label = $configuration['label'];
-		$this->show_label = $configuration['show_label'] == '1';
-		$this->custom_label = $configuration['custom_label'];
-		$this->custom_class = $configuration['custom_class'];
-		$this->cap = $configuration['only_loggedin'] == '1' ? $configuration['only_loggedin_cap'] : '';
-		$this->search_filter = $configuration['search_filter'] == '1';
-		$this->show_as_link = $configuration['show_as_link'] == '1';
+		$this->ID = $configuration[ 'id' ];
+		$this->label = $configuration[ 'label' ];
+		$this->show_label = $configuration[ 'show_label' ] == '1';
+		$this->custom_label = $configuration[ 'custom_label' ];
+		$this->custom_class = $configuration[ 'custom_class' ];
+		$this->cap = $configuration[ 'only_loggedin' ] == '1' ? $configuration[ 'only_loggedin_cap' ] : '';
+		$this->search_filter = $configuration[ 'search_filter' ] == '1';
+		$this->show_as_link = $configuration[ 'show_as_link' ] == '1';
 
 		/** Shared among all field types (sort of). */
 		$shared_configuration_keys = array(
 			'id', 'label', 'show_label', 'custom_label', 'custom_class',
-			'only_loggedin' ,'only_loggedin_cap', 'search_filter', 'show_as_link',
+			'only_loggedin', 'only_loggedin_cap', 'search_filter', 'show_as_link',
 		);
 
 		/** Everything else goes into the properties for now. @todo subclasses! */

future/lib/class-gamajo-template-loader.php

Spacing +7 added lines, -7 removed lines

@@ -117,7 +117,7 @@ 
 		public function set_template_data( array $data, $var_name = 'data' ) {
 			global $wp_query;
 
-			$wp_query->query_vars[ $var_name ] = (object) $data;
+			$wp_query->query_vars[ $var_name ] = (object)$data;
 		}
 
 		/**
@@ -130,8 +130,8 @@ 
 		public function unset_template_data() {
 			global $wp_query;
 
-			if ( isset( $wp_query->query_vars['data'] ) ) {
-				unset( $wp_query->query_vars['data'] );
+			if ( isset( $wp_query->query_vars[ 'data' ] ) ) {
+				unset( $wp_query->query_vars[ 'data' ] );
 			}
 		}
 
@@ -148,9 +148,9 @@ 
 		protected function get_template_file_names( $slug, $name ) {
 			$templates = array();
 			if ( isset( $name ) ) {
-				$templates[] = $slug . '-' . $name . '.php';
+				$templates[ ] = $slug . '-' . $name . '.php';
 			}
-			$templates[] = $slug . '.php';
+			$templates[ ] = $slug . '.php';
 
 			/**
 			 * Allow template choices to be filtered.
@@ -188,7 +188,7 @@ 
 			$located = false;
 
 			// Remove empty entries.
-			$template_names = array_filter( (array) $template_names );
+			$template_names = array_filter( (array)$template_names );
 			$template_paths = $this->get_template_paths();
 
 			// Try to find a template file.
@@ -233,7 +233,7 @@ 
 
 			// Only add this conditionally, so non-child themes don't redundantly check active theme twice.
 			if ( is_child_theme() ) {
-				$file_paths[1] = trailingslashit( get_stylesheet_directory() ) . $theme_directory;
+				$file_paths[ 1 ] = trailingslashit( get_stylesheet_directory() ) . $theme_directory;
 			}
 
 			/**

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

Spacing +4 added lines, -4 removed lines

@@ -48,7 +48,7 @@ 
 			return $meta_keys;
 		}
 
-		if( $acf_keys = get_field_objects( $post_id, array( 'load_value' => false ) ) ) {
+		if ( $acf_keys = get_field_objects( $post_id, array( 'load_value' => false ) ) ) {
 			return array_merge( array_keys( $acf_keys ), $meta_keys );
 		}
 
@@ -63,9 +63,9 @@ 
 	 * @return void
 	 */
 	private function fix_posted_fields() {
-		if( is_admin() && isset( $_POST['action'] ) && isset( $_POST['post_type'] ) ) {
-			if( 'editpost' === $_POST['action'] && 'gravityview' === $_POST['post_type'] ) {
-				$_POST['fields'] = _gravityview_process_posted_fields();
+		if ( is_admin() && isset( $_POST[ 'action' ] ) && isset( $_POST[ 'post_type' ] ) ) {
+			if ( 'editpost' === $_POST[ 'action' ] && 'gravityview' === $_POST[ 'post_type' ] ) {
+				$_POST[ 'fields' ] = _gravityview_process_posted_fields();
 			}
 		}
 	}

future/_mocks.php

Doc Comments +2 added lines

@@ -62,6 +62,8 @@
  * @internal
  * @since future
  *
+ * @param integer $form_id
+ * @param integer $count
  * @return array The old associative array data as returned by
  *  \GravityView_frontend::get_view_entries(), the paging parameters
  *  and a total count of all entries.

Spacing +12 added lines, -12 removed lines

@@ -96,19 +96,19 @@ 
 		$paging = rgar( $parameters, 'paging' );
 	} else {
 		$entries = $form->entries
-			->filter( \GV\GF_Entry_Filter::from_search_criteria( $criteria['search_criteria'] ) )
-			->offset( $args['offset'] )
-			->limit( $criteria['paging']['page_size'] );
+			->filter( \GV\GF_Entry_Filter::from_search_criteria( $criteria[ 'search_criteria' ] ) )
+			->offset( $args[ 'offset' ] )
+			->limit( $criteria[ 'paging' ][ 'page_size' ] );
 
-		if ( $criteria['paging']['page_size'] ) {
-			$entries = $entries->page( ( ( $criteria['paging']['offset'] - $args['offset'] ) / $criteria['paging']['page_size'] ) + 1 );
+		if ( $criteria[ 'paging' ][ 'page_size' ] ) {
+			$entries = $entries->page( ( ( $criteria[ 'paging' ][ 'offset' ] - $args[ 'offset' ] ) / $criteria[ 'paging' ][ 'page_size' ] ) + 1 );
 		}
 
-		if ( ! empty( $criteria['sorting'] ) ) {
+		if ( ! empty( $criteria[ 'sorting' ] ) ) {
 			$field = new \GV\Field();
-			$field->ID = $criteria['sorting']['key'];
-			$direction = strtolower( $criteria['sorting']['direction'] ) == 'asc' ? \GV\Entry_Sort::ASC : \GV\Entry_Sort::DESC;
-			$mode = $criteria['sorting']['is_numeric'] ? \GV\Entry_Sort::NUMERIC : \GV\Entry_Sort::ALPHA;
+			$field->ID = $criteria[ 'sorting' ][ 'key' ];
+			$direction = strtolower( $criteria[ 'sorting' ][ 'direction' ] ) == 'asc' ? \GV\Entry_Sort::ASC : \GV\Entry_Sort::DESC;
+			$mode = $criteria[ 'sorting' ][ 'is_numeric' ] ? \GV\Entry_Sort::NUMERIC : \GV\Entry_Sort::ALPHA;
 			$entries = $entries->sort( new \GV\Entry_Sort( $field, $direction, $mode ) );
 		}
 
@@ -134,7 +134,7 @@ 
 
 /** Add some global fix for field capability discrepancies. */
 add_filter( 'gravityview/configuration/fields', function( $fields ) {
-	if ( empty( $fields  ) ) {
+	if ( empty( $fields ) ) {
 		return $fields;
 	}
 
@@ -161,11 +161,11 @@ 
 		}
 
 		foreach ( $_fields as $uid => &$_field ) {
-			if ( ! isset( $_field['only_loggedin'] ) ) {
+			if ( ! isset( $_field[ 'only_loggedin' ] ) ) {
 				continue;
 			}
 			/** If we do not require login, we don't require a cap. */
-			$_field['only_loggedin'] != '1' && ( $_field['only_loggedin_cap'] = '' );
+			$_field[ 'only_loggedin' ] != '1' && ( $_field[ 'only_loggedin_cap' ] = '' );
 		}
 	}
 	return $fields;

future/loader.php

Spacing +1 added lines, -1 removed lines

@@ -5,7 +5,7 @@
 }
 
 /** The future branch of GravityView requires PHP 5.3+ namespaces and SPL. */
-if ( version_compare( phpversion(), '5.3' , '<' ) ) {
+if ( version_compare( phpversion(), '5.3', '<' ) ) {
 	return false;
 
 /** Tests with a suppressed future. */