gravityview / GravityView

includes/widgets/class-gravityview-widget.php

Spacing +26 added lines, -26 removed lines

@@ -51,14 +51,14 @@ 
 	// hold widget View options
 	private $widget_options;
 
-	function __construct( $widget_label , $widget_id , $defaults = array(), $settings = array() ) {
+	function __construct( $widget_label, $widget_id, $defaults = array(), $settings = array() ) {
 
 
 		/**
 		 * The shortcode name is set to the lowercase name of the widget class, unless overridden by the class specifying a different value for $shortcode_name
 		 * @var string
 		 */
-		$this->shortcode_name = !isset( $this->shortcode_name ) ? strtolower( get_class($this) ) : $this->shortcode_name;
+		$this->shortcode_name = ! isset( $this->shortcode_name ) ? strtolower( get_class( $this ) ) : $this->shortcode_name;
 
 		$this->widget_label = $widget_label;
 		$this->widget_id = $widget_id;
@@ -69,7 +69,7 @@ 
 		$this->settings = wp_parse_args( $settings, $this->settings );
 
 		// register widgets to be listed in the View Configuration
-		add_filter( 'gravityview_register_directory_widgets', array( $this, 'register_widget') );
+		add_filter( 'gravityview_register_directory_widgets', array( $this, 'register_widget' ) );
 
 		// widget options
 		add_filter( 'gravityview_template_widget_options', array( $this, 'assign_widget_options' ), 10, 3 );
@@ -78,10 +78,10 @@ 
 		add_action( "gravityview_render_widget_{$widget_id}", array( $this, 'render_frontend' ), 10, 1 );
 
 		// register shortcodes
-		add_action( 'wp', array( $this, 'add_shortcode') );
+		add_action( 'wp', array( $this, 'add_shortcode' ) );
 
 		// Use shortcodes in text widgets.
-		add_filter('widget_text', array( $this, 'maybe_do_shortcode' ) );
+		add_filter( 'widget_text', array( $this, 'maybe_do_shortcode' ) );
 	}
 
 
@@ -99,14 +99,14 @@ 
 		 * @param boolean $enable_custom_class False by default. Return true if you want to enable.
 		 * @param GravityView_Widget $this Current instance of GravityView_Widget
 		 */
-		$enable_custom_class = apply_filters('gravityview/widget/enable_custom_class', false, $this );
+		$enable_custom_class = apply_filters( 'gravityview/widget/enable_custom_class', false, $this );
 
-		if( $enable_custom_class ) {
+		if ( $enable_custom_class ) {
 
-			$settings['custom_class'] = array(
+			$settings[ 'custom_class' ] = array(
 				'type' => 'text',
 				'label' => __( 'Custom CSS Class:', 'gravityview' ),
-				'desc' => __( 'This class will be added to the widget container', 'gravityview'),
+				'desc' => __( 'This class will be added to the widget container', 'gravityview' ),
 				'value' => '',
 				'merge_tags' => true,
 			);
@@ -128,7 +128,7 @@ 
 	 * @return array|null   Settings array; NULL if not set
 	 */
 	public function get_settings() {
-		return !empty( $this->settings ) ? $this->settings : NULL;
+		return ! empty( $this->settings ) ? $this->settings : NULL;
 	}
 
 	/**
@@ -139,7 +139,7 @@ 
 	public function get_setting( $key ) {
 		$setting = NULL;
 
-		if( isset( $this->settings ) && is_array( $this->settings ) ) {
+		if ( isset( $this->settings ) && is_array( $this->settings ) ) {
 			$setting = isset( $this->settings[ $key ] ) ? $this->settings[ $key ] : NULL;
 		}
 
@@ -154,7 +154,7 @@ 
 	 */
 	function maybe_do_shortcode( $text, $widget = NULL ) {
 
-		if( !empty( $this->shortcode_name ) && has_shortcode( $text, $this->shortcode_name ) ) {
+		if ( ! empty( $this->shortcode_name ) && has_shortcode( $text, $this->shortcode_name ) ) {
 			return do_shortcode( $text );
 		}
 
@@ -183,26 +183,26 @@ 
 			return;
 		}
 
-		if( empty( $this->shortcode_name ) ) { return; }
+		if ( empty( $this->shortcode_name ) ) { return; }
 
 		// If the widget shouldn't output on single entries, don't show it
-		if( empty( $this->show_on_single ) && class_exists('GravityView_frontend') && GravityView_frontend::is_single_entry() ) {
-			do_action('gravityview_log_debug', sprintf( '%s[add_shortcode]: Skipping; set to not run on single entry.', get_class($this)) );
+		if ( empty( $this->show_on_single ) && class_exists( 'GravityView_frontend' ) && GravityView_frontend::is_single_entry() ) {
+			do_action( 'gravityview_log_debug', sprintf( '%s[add_shortcode]: Skipping; set to not run on single entry.', get_class( $this ) ) );
 
 			add_shortcode( $this->shortcode_name, '__return_null' );
 			return;
 		}
 
 
-		if( !has_gravityview_shortcode( $post ) ) {
+		if ( ! has_gravityview_shortcode( $post ) ) {
 
-			do_action('gravityview_log_debug', sprintf( '%s[add_shortcode]: No shortcode present; not adding render_frontend shortcode.', get_class($this)) );
+			do_action( 'gravityview_log_debug', sprintf( '%s[add_shortcode]: No shortcode present; not adding render_frontend shortcode.', get_class( $this ) ) );
 
 			add_shortcode( $this->shortcode_name, '__return_null' );
 			return;
 		}
 
-		add_shortcode( $this->shortcode_name, array( $this, 'render_shortcode') );
+		add_shortcode( $this->shortcode_name, array( $this, 'render_shortcode' ) );
 	}
 
 	/**
@@ -212,7 +212,7 @@ 
 	 */
 	function register_widget( $widgets ) {
 		$widgets[ $this->widget_id ] = array(
-			'label' => $this->widget_label ,
+			'label' => $this->widget_label,
 			'description' => $this->widget_description,
 			'subtitle' => $this->widget_subtitle,
 		);
@@ -229,7 +229,7 @@ 
 	 */
 	public function assign_widget_options( $options = array(), $template = '', $widget = '' ) {
 
-		if( $this->widget_id === $widget ) {
+		if ( $this->widget_id === $widget ) {
 			$options = array_merge( $options, $this->settings );
 		}
 
@@ -242,9 +242,9 @@ 
 	 *
 	 * @return void
 	 */
-	public function render_frontend( $widget_args, $content = '', $context = '') {
+	public function render_frontend( $widget_args, $content = '', $context = '' ) {
 		// to be defined by child class
-		if( !$this->pre_render_frontend() ) {
+		if ( ! $this->pre_render_frontend() ) {
 			return;
 		}
 	}
@@ -256,8 +256,8 @@ 
 	public function pre_render_frontend() {
 		$gravityview_view = GravityView_View::getInstance();
 
-		if( empty( $gravityview_view ) ) {
-			do_action('gravityview_log_debug', sprintf( '%s[render_frontend]: $gravityview_view not instantiated yet.', get_class($this)) );
+		if ( empty( $gravityview_view ) ) {
+			do_action( 'gravityview_log_debug', sprintf( '%s[render_frontend]: $gravityview_view not instantiated yet.', get_class( $this ) ) );
 			return false;
 		}
 
@@ -268,8 +268,8 @@ 
 		 */
 		$hide_until_search = apply_filters( 'gravityview/widget/hide_until_searched', $gravityview_view->hide_until_searched, $this );
 
-		if( $hide_until_search ) {
-			do_action('gravityview_log_debug', sprintf( '%s[render_frontend]: Hide View data until search is performed', get_class($this)) );
+		if ( $hide_until_search ) {
+			do_action( 'gravityview_log_debug', sprintf( '%s[render_frontend]: Hide View data until search is performed', get_class( $this ) ) );
 			return false;
 		}
 

includes/class-gvlogic-shortcode.php

Doc Comments +1 added lines, -1 removed lines

@@ -232,7 +232,7 @@
 	 * Does the if and the comparison match?
 	 * @uses GVCommon::matches_operation
 	 *
-	 * @return boolean True: yep; false: nope
+	 * @return boolean|null True: yep; false: nope
 	 */
 	private function set_is_match() {
 		$this->is_match = GVCommon::matches_operation( $this->if, $this->comparison, $this->operation );

includes/fields/class-gravityview-field.php

Doc Comments +1 added lines, -1 removed lines

@@ -245,7 +245,7 @@
 	 * @param bool $url_encode Whether to URL-encode output
 	 * @param bool $esc_html Whether to apply `esc_html()` to output
 	 *
-	 * @return mixed
+	 * @return string
 	 */
 	public function replace_merge_tag( $matches = array(), $text = '', $form = array(), $entry = array(), $url_encode = false, $esc_html = false ) {
 

Spacing +22 added lines, -22 removed lines

@@ -134,17 +134,17 @@ 
 
 		add_filter( 'gravityview/sortable/field_blacklist', array( $this, '_filter_sortable_fields' ), 1 );
 
-		if( $this->entry_meta_key ) {
+		if ( $this->entry_meta_key ) {
 			add_filter( 'gform_entry_meta', array( $this, 'add_entry_meta' ) );
 			add_filter( 'gravityview/common/sortable_fields', array( $this, 'add_sortable_field' ), 10, 2 );
 		}
 
-		if( $this->_custom_merge_tag ) {
+		if ( $this->_custom_merge_tag ) {
 			add_filter( 'gform_custom_merge_tags', array( $this, '_filter_gform_custom_merge_tags' ), 10, 4 );
 			add_filter( 'gform_replace_merge_tags', array( $this, '_filter_gform_replace_merge_tags' ), 10, 7 );
 		}
 
-		if( 'meta' === $this->group || '' !== $this->default_search_label ) {
+		if ( 'meta' === $this->group || '' !== $this->default_search_label ) {
 			add_filter( 'gravityview_search_field_label', array( $this, 'set_default_search_label' ), 10, 3 );
 		}
 
@@ -152,7 +152,7 @@ 
 		 * Auto-assign label from Gravity Forms label, if exists
 		 * @since 1.20
 		 */
-		if( empty( $this->label ) && ! empty( $this->_gf_field_class_name ) && class_exists( $this->_gf_field_class_name ) ) {
+		if ( empty( $this->label ) && ! empty( $this->_gf_field_class_name ) && class_exists( $this->_gf_field_class_name ) ) {
 			$this->label = ucfirst( GF_Fields::get( $this->name )->get_form_editor_field_title() );
 		}
 
@@ -175,7 +175,7 @@ 
 			'type'  => $this->name
 		);
 
-		$fields["{$this->entry_meta_key}"] = $added_field;
+		$fields[ "{$this->entry_meta_key}" ] = $added_field;
 
 		return $fields;
 	}
@@ -195,7 +195,7 @@ 
 	 */
 	function set_default_search_label( $label = '', $gf_field = null, $field = array() ) {
 
-		if( $this->name === $field['field'] && '' === $label ) {
+		if ( $this->name === $field[ 'field' ] && '' === $label ) {
 			$label = esc_html( $this->default_search_label );
 		}
 
@@ -216,7 +216,7 @@ 
 	 *
 	 * @return string Original text if {_custom_merge_tag} isn't found. Otherwise, replaced text.
 	 */
-	public function _filter_gform_replace_merge_tags( $text, $form = array(), $entry = array(), $url_encode = false, $esc_html = false  ) {
+	public function _filter_gform_replace_merge_tags( $text, $form = array(), $entry = array(), $url_encode = false, $esc_html = false ) {
 
 		// Is there is field merge tag? Strip whitespace off the ned, too.
 		preg_match_all( '/{' . preg_quote( $this->_custom_merge_tag ) . ':?(.*?)(?:\s)?}/ism', $text, $matches, PREG_SET_ORDER );
@@ -247,19 +247,19 @@ 
 	 */
 	public function replace_merge_tag( $matches = array(), $text = '', $form = array(), $entry = array(), $url_encode = false, $esc_html = false ) {
 
-		foreach( $matches as $match ) {
+		foreach ( $matches as $match ) {
 
-			$full_tag = $match[0];
+			$full_tag = $match[ 0 ];
 
 			// Strip the Merge Tags
-			$tag = str_replace( array( '{', '}'), '', $full_tag );
+			$tag = str_replace( array( '{', '}' ), '', $full_tag );
 
 			// Replace the value from the entry, if exists
-			if( isset( $entry[ $tag ] ) ) {
+			if ( isset( $entry[ $tag ] ) ) {
 
 				$value = $entry[ $tag ];
 
-				if( is_callable( array( $this, 'get_content') ) ) {
+				if ( is_callable( array( $this, 'get_content' ) ) ) {
 					$value = $this->get_content( $value );
 				}
 
@@ -332,8 +332,8 @@ 
 	 */
 	public function _filter_sortable_fields( $not_sortable ) {
 
-		if( ! $this->is_sortable ) {
-			$not_sortable[] = $this->name;
+		if ( ! $this->is_sortable ) {
+			$not_sortable[ ] = $this->name;
 		}
 
 		return $not_sortable;
@@ -350,7 +350,7 @@ 
 	 */
 	function add_entry_meta( $entry_meta ) {
 
-		if( ! isset( $entry_meta["{$this->entry_meta_key}"] ) ) {
+		if ( ! isset( $entry_meta[ "{$this->entry_meta_key}" ] ) ) {
 
 			$added_meta = array(
 				'label'             => $this->label,
@@ -359,13 +359,13 @@ 
 			);
 
 			if ( $this->entry_meta_update_callback && is_callable( $this->entry_meta_update_callback ) ) {
-				$added_meta['update_entry_meta_callback'] = $this->entry_meta_update_callback;
+				$added_meta[ 'update_entry_meta_callback' ] = $this->entry_meta_update_callback;
 			}
 
-			$entry_meta["{$this->entry_meta_key}"] = $added_meta;
+			$entry_meta[ "{$this->entry_meta_key}" ] = $added_meta;
 
 		} else {
-			do_action( 'gravityview_log_error', __METHOD__ . ' Entry meta already set: ' . $this->entry_meta_key, $entry_meta["{$this->entry_meta_key}"] );
+			do_action( 'gravityview_log_error', __METHOD__ . ' Entry meta already set: ' . $this->entry_meta_key, $entry_meta[ "{$this->entry_meta_key}" ] );
 		}
 
 		return $entry_meta;
@@ -394,7 +394,7 @@ 
 			'date_display' => array(
 				'type' => 'text',
 				'label' => __( 'Override Date Format', 'gravityview' ),
-				'desc' => sprintf( __( 'Define how the date is displayed (using %sthe PHP date format%s)', 'gravityview'), '<a href="https://codex.wordpress.org/Formatting_Date_and_Time">', '</a>' ),
+				'desc' => sprintf( __( 'Define how the date is displayed (using %sthe PHP date format%s)', 'gravityview' ), '<a href="https://codex.wordpress.org/Formatting_Date_and_Time">', '</a>' ),
 				/**
 				 * @filter `gravityview_date_format` Override the date format with a [PHP date format](https://codex.wordpress.org/Formatting_Date_and_Time)
 				 * @param[in,out] null|string $date_format Date Format (default: null)
@@ -419,7 +419,7 @@ 
 
 		$options = $this->field_support_options();
 
-		if( isset( $options[ $key ] ) ) {
+		if ( isset( $options[ $key ] ) ) {
 			$field_options[ $key ] = $options[ $key ];
 		}
 
@@ -483,11 +483,11 @@ 
 		$connected_form = rgpost( 'form_id' );
 
 		// Otherwise, get the Form ID from the Post page
-		if( empty( $connected_form ) ) {
+		if ( empty( $connected_form ) ) {
 			$connected_form = gravityview_get_form_id( get_the_ID() );
 		}
 
-		if( empty( $connected_form ) ) {
+		if ( empty( $connected_form ) ) {
 			do_action( 'gravityview_log_error', sprintf( '%s: Form not found for form ID "%s"', __METHOD__, $connected_form ) );
 			return false;
 		}

includes/fields/class-gravityview-fields.php

Doc Comments +1 added lines, -1 removed lines

@@ -77,7 +77,7 @@
 	/**
 	 * Alias for get_instance()
 	 *
-	 * @param $field_name
+	 * @param string $field_name
 	 *
 	 * @return GravityView_Field|false
 	 */

includes/fields/class-gravityview-field-date.php

Indentation -1 removed lines

@@ -40,7 +40,6 @@
 	 * Get the default date format for a field based on the field ID and the time format setting
 	 *
 	 * @since 1.16.4
-
 	 * @param string $date_format The Gravity Forms date format for the field. Default: "mdy"
 	 * @param int $field_id The ID of the field. Used to figure out full date/day/month/year
 	 *

includes/fields/class-gravityview-field-multiselect.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
 	 * @see GFCommon::get_field_filter_settings Gravity Forms suggests checkboxes should just be "contains"
 	 * @var array
 	 */
-	var $search_operators = array( 'is', 'in', 'not in', 'isnot', 'contains');
+	var $search_operators = array( 'is', 'in', 'not in', 'isnot', 'contains' );
 
 	var $is_searchable = true;
 

includes/fields/class-gravityview-field-radio.php

Spacing +3 added lines, -3 removed lines

@@ -11,7 +11,7 @@ 
 
 	var $is_searchable = true;
 
-	var $search_operators = array( 'is', 'in', 'not in', 'isnot', 'contains');
+	var $search_operators = array( 'is', 'in', 'not in', 'isnot', 'contains' );
 
 	var $_gf_field_class_name = 'GF_Field_Radio';
 
@@ -40,8 +40,8 @@ 
 		// Set the $_field_id var
 		$field_options = parent::field_options( $field_options, $template_id, $field_id, $context, $input_type );
 		
-		if( $this->is_choice_value_enabled() ) {
-			$field_options['choice_display'] = array(
+		if ( $this->is_choice_value_enabled() ) {
+			$field_options[ 'choice_display' ] = array(
 				'type'    => 'radio',
 				'value'   => 'value',
 				'label'   => __( 'What should be displayed:', 'gravityview' ),

includes/fields/class-gravityview-field-post-category.php

Spacing +10 added lines, -10 removed lines

@@ -11,7 +11,7 @@ 
 
 	var $is_searchable = true;
 
-	var $search_operators = array( 'is', 'in', 'not in', 'isnot', 'contains');
+	var $search_operators = array( 'is', 'in', 'not in', 'isnot', 'contains' );
 
 	var $_gf_field_class_name = 'GF_Field_Post_Category';
 
@@ -44,7 +44,7 @@ 
 		$entry = GFAPI::get_entry( $entry_id );
 		$post_id = rgar( $entry, 'post_id' );
 
-		if( empty( $post_id ) ) {
+		if ( empty( $post_id ) ) {
 			return false;
 		}
 
@@ -52,7 +52,7 @@ 
 
 		$post_category_fields = GFAPI::get_fields_by_type( $form, 'post_category' );
 
-		if( $post_category_fields ) {
+		if ( $post_category_fields ) {
 
 			$updated_categories = array();
 
@@ -119,18 +119,18 @@ 
 	 *
 	 * @return mixed
 	 */
-	function edit_entry_post_category_choices( $choices, $field, $form_id  ) {
+	function edit_entry_post_category_choices( $choices, $field, $form_id ) {
 
-		$entry = GravityView_Edit_Entry::getInstance()->instances['render']->get_entry();
+		$entry = GravityView_Edit_Entry::getInstance()->instances[ 'render' ]->get_entry();
 
 		// $entry['post_id'] should always be set, but we check to make sure.
-		if( $entry && isset( $entry['post_id'] ) && $post_id = $entry['post_id'] ) {
+		if ( $entry && isset( $entry[ 'post_id' ] ) && $post_id = $entry[ 'post_id' ] ) {
 
 			$post_categories = wp_get_post_categories( $post_id, array( 'fields' => 'ids' ) );
 
 			// Always use the live value
 			foreach ( $choices as &$choice ) {
-				$choice['isSelected'] = in_array( $choice['value'], array_values( $post_categories ) );
+				$choice[ 'isSelected' ] = in_array( $choice[ 'value' ], array_values( $post_categories ) );
 			}
 		}
 
@@ -139,12 +139,12 @@ 
 
 	function field_options( $field_options, $template_id, $field_id, $context, $input_type ) {
 
-		if( 'edit' === $context ) {
+		if ( 'edit' === $context ) {
 			return $field_options;
 		}
 
-		$this->add_field_support('dynamic_data', $field_options );
-		$this->add_field_support('link_to_term', $field_options );
+		$this->add_field_support( 'dynamic_data', $field_options );
+		$this->add_field_support( 'link_to_term', $field_options );
 
 		return $field_options;
 	}

includes/fields/class-gravityview-field-post-tags.php

Spacing +4 added lines, -4 removed lines

@@ -14,7 +14,7 @@ 
 
 	var $is_searchable = true;
 
-	var $search_operators = array( 'is', 'in', 'not in', 'isnot', 'contains');
+	var $search_operators = array( 'is', 'in', 'not in', 'isnot', 'contains' );
 
 	var $_gf_field_class_name = 'GF_Field_Post_Tags';
 
@@ -27,12 +27,12 @@ 
 
 	function field_options( $field_options, $template_id, $field_id, $context, $input_type ) {
 
-		if( 'edit' === $context ) {
+		if ( 'edit' === $context ) {
 			return $field_options;
 		}
 
-		$this->add_field_support('dynamic_data', $field_options );
-		$this->add_field_support('link_to_term', $field_options );
+		$this->add_field_support( 'dynamic_data', $field_options );
+		$this->add_field_support( 'link_to_term', $field_options );
 
 		return $field_options;
 	}