Inspection of "Add an installer for extensions & plugins" - gravityview/GravityView - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — develop (#1130)

by Zack

created 2018-07-27 18:04 UTC

Status

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -25,7 +25,7 @@  discard block
 block discarded – undo
 			printf( '<body style="padding: 0; margin: 0; font-family: -apple-system, BlinkMacSystemFont, \'Segoe UI\', Roboto, Oxygen-Sans, Ubuntu, Cantarell, \'Helvetica Neue\', sans-serif;">' );
 			printf( '<img src="' . plugins_url( 'assets/images/astronaut-200x263.png', GRAVITYVIEW_FILE ) . '" alt="The GravityView Astronaut Says:" style="float: left; height: 60px; margin-right : 10px;" />' );
 			printf( __( "%sGravityView requires PHP Version %s or newer.%s \n\nYou're using Version %s. Please ask your host to upgrade your server's PHP.", 'gravityview' ),
-				'<h3 style="font-size:16px; margin: 0 0 8px 0;">', GV_MIN_PHP_VERSION , "</h3>\n\n", $version );
+				'<h3 style="font-size:16px; margin: 0 0 8px 0;">', GV_MIN_PHP_VERSION, "</h3>\n\n", $version );
 			printf( '</body>' );
 		}
 
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
 }
 
 /** The future branch of GravityView requires PHP 5.3+ namespaces and SPL. */
-if ( version_compare( phpversion(), '5.3.0' , '<' ) ) {
+if ( version_compare( phpversion(), '5.3.0', '<' ) ) {
 	require GRAVITYVIEW_DIR . 'future/_stubs.php';
 
 /** All looks fine. */

Please login to merge, or discard this patch.

future/includes/class-gv-renderer.php 1 patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -61,7 +61,7 @@
 block discarded – undo
 		$action_text = sprintf( esc_html__( 'Add fields to %s', 'gravityview' ), $tab );
 		$message = esc_html__( 'You can only see this message because you are able to edit this View.', 'gravityview' );
 
-		$image =  sprintf( '<img alt="%s" src="%s" style="margin-top: 10px;" />', $tab, esc_url( plugins_url( sprintf( 'assets/images/tab-%s.png', $context ), GRAVITYVIEW_FILE ) ) );
+		$image = sprintf( '<img alt="%s" src="%s" style="margin-top: 10px;" />', $tab, esc_url( plugins_url( sprintf( 'assets/images/tab-%s.png', $context ), GRAVITYVIEW_FILE ) ) );
 		$output = sprintf( '<h3>%s <strong><a href="%s">%s</a></strong></h3><p>%s</p>', $title, esc_url( $edit_link ), $action_text, $message );
 
 		echo \GVCommon::generate_notice( $output . $image, 'gv-error error', 'edit_gravityview', $gravityview->view->ID );

Please login to merge, or discard this patch.

future/includes/class-gv-view.php 3 patches

Indentation -1 removed lines patch added patch discarded remove patch

@@ -62,7 +62,6 @@
 block discarded – undo
 	 *
 	 * Internal static cache for gets, and whatnot.
 	 * This is not persistent, resets across requests.
-
 	 * @internal
 	 */
 	private static $cache = array();

Please login to merge, or discard this patch.

Spacing +23 added lines, -24 removed lines patch added patch discarded remove patch

@@ -108,7 +108,7 @@  discard block
 block discarded – undo
 		$supports = array( 'title', 'revisions' );
 
 		if ( $is_hierarchical ) {
-			$supports[] = 'page-attributes';
+			$supports[ ] = 'page-attributes';
 		}
 
 		/**
@@ -288,13 +288,13 @@  discard block
 block discarded – undo
 			return __( 'You are not allowed to view this content.', 'gravityview' );
 		}
 
-		$is_admin_and_can_view = $view->settings->get( 'admin_show_all_statuses' ) && \GVCommon::has_cap('gravityview_moderate_entries', $view->ID );
+		$is_admin_and_can_view = $view->settings->get( 'admin_show_all_statuses' ) && \GVCommon::has_cap( 'gravityview_moderate_entries', $view->ID );
 
 		/**
 		 * Editing a single entry.
 		 */
 		if ( $entry = $request->is_edit_entry() ) {
-			if ( $entry['status'] != 'active' ) {
+			if ( $entry[ 'status' ] != 'active' ) {
 				gravityview()->log->notice( 'Entry ID #{entry_id} is not active', array( 'entry_id' => $entry->ID ) );
 				return __( 'You are not allowed to view this content.', 'gravityview' );
 			}
@@ -305,7 +305,7 @@  discard block
 block discarded – undo
 			}
 
 			if ( $view->settings->get( 'show_only_approved' ) && ! $is_admin_and_can_view ) {
-				if ( ! \GravityView_Entry_Approval_Status::is_approved( gform_get_meta( $entry->ID, \GravityView_Entry_Approval::meta_key ) )  ) {
+				if ( ! \GravityView_Entry_Approval_Status::is_approved( gform_get_meta( $entry->ID, \GravityView_Entry_Approval::meta_key ) ) ) {
 					gravityview()->log->error( 'Entry ID #{entry_id} is not approved for viewing', array( 'entry_id' => $entry->ID ) );
 					return __( 'You are not allowed to view this content.', 'gravityview' );
 				}
@@ -318,7 +318,7 @@  discard block
 block discarded – undo
 		 * Viewing a single entry.
 		 */
 		} else if ( $entry = $request->is_entry() ) {
-			if ( $entry['status'] != 'active' ) {
+			if ( $entry[ 'status' ] != 'active' ) {
 				gravityview()->log->notice( 'Entry ID #{entry_id} is not active', array( 'entry_id' => $entry->ID ) );
 				return __( 'You are not allowed to view this content.', 'gravityview' );
 			}
@@ -329,7 +329,7 @@  discard block
 block discarded – undo
 			}
 
 			if ( $view->settings->get( 'show_only_approved' ) && ! $is_admin_and_can_view ) {
-				if ( ! \GravityView_Entry_Approval_Status::is_approved( gform_get_meta( $entry->ID, \GravityView_Entry_Approval::meta_key ) )  ) {
+				if ( ! \GravityView_Entry_Approval_Status::is_approved( gform_get_meta( $entry->ID, \GravityView_Entry_Approval::meta_key ) ) ) {
 					gravityview()->log->error( 'Entry ID #{entry_id} is not approved for viewing', array( 'entry_id' => $entry->ID ) );
 					return __( 'You are not allowed to view this content.', 'gravityview' );
 				}
@@ -337,7 +337,7 @@  discard block
 block discarded – undo
 
 			$error = \GVCommon::check_entry_display( $entry->as_entry() );
 
-			if( is_wp_error( $error ) ) {
+			if ( is_wp_error( $error ) ) {
 				gravityview()->log->error( 'Entry ID #{entry_id} is not approved for viewing: {message}', array( 'entry_id' => $entry->ID, 'message' => $error->get_error_message() ) );
 				return __( 'You are not allowed to view this content.', 'gravityview' );
 			}
@@ -396,7 +396,7 @@  discard block
 block discarded – undo
 			$join_column    = is_numeric( $join_column ) ? GF_Field::by_id( $join, $join_column ) : Internal_Field( $join_column );
 			$join_on_column = is_numeric( $join_on_column ) ? GF_Field::by_id( $join_on, $join_on_column ) : Internal_Field( $join_on_column );
 
-			$joins [] = new Join( $join, $join_column, $join_on, $join_on_column );
+			$joins [ ] = new Join( $join, $join_column, $join_on, $join_on_column );
 		}
 
 		return $joins;
@@ -431,10 +431,10 @@  discard block
 block discarded – undo
 
 			list( $join, $join_column, $join_on, $join_on_column ) = $meta;
 
-			$forms_ids [] = GF_Form::by_id( $join_on );
+			$forms_ids [ ] = GF_Form::by_id( $join_on );
 		}
 
-		return ( !empty( $forms_ids) ) ? $forms_ids : null;
+		return ( ! empty( $forms_ids ) ) ? $forms_ids : null;
 	}
 
 	/**
@@ -464,7 +464,7 @@  discard block
 block discarded – undo
 		if ( ! $view->form ) {
 			gravityview()->log->error( 'View #{view_id} tried attaching non-existent Form #{form_id} to it.', array(
 				'view_id' => $view->ID,
-				'form_id' => $view->_gravityview_form_id ? : 0,
+				'form_id' => $view->_gravityview_form_id ?: 0,
 			) );
 		}
 
@@ -690,28 +690,27 @@  discard block
 block discarded – undo
 			 * @todo: Stop using _frontend and use something like $request->get_search_criteria() instead
 			 */
 			$parameters = \GravityView_frontend::get_view_entries_parameters( $this->settings->as_atts(), $this->form->ID );
-			$parameters['context_view_id'] = $this->ID;
+			$parameters[ 'context_view_id' ] = $this->ID;
 			$parameters = \GVCommon::calculate_get_entries_criteria( $parameters, $this->form->ID );
 
 			if ( $request instanceof REST\Request ) {
 				$atts = $this->settings->as_atts();
 				$paging_parameters = wp_parse_args( $request->get_paging(), array(
-						'paging' => array( 'page_size' => $atts['page_size'] ),
+						'paging' => array( 'page_size' => $atts[ 'page_size' ] ),
 					) );
-				$parameters['paging'] = $paging_parameters['paging'];
+				$parameters[ 'paging' ] = $paging_parameters[ 'paging' ];
 			}
 
-			$page = Utils::get( $parameters['paging'], 'current_page' ) ?
-				: ( ( ( $parameters['paging']['offset'] - $this->settings->get( 'offset' ) ) / $parameters['paging']['page_size'] ) + 1 );
+			$page = Utils::get( $parameters[ 'paging' ], 'current_page' ) ?: ( ( ( $parameters[ 'paging' ][ 'offset' ] - $this->settings->get( 'offset' ) ) / $parameters[ 'paging' ][ 'page_size' ] ) + 1 );
 
 			if ( gravityview()->plugin->supports( Plugin::FEATURE_GFQUERY ) ) {
 				/**
 				 * New \GF_Query stuff :)
 				 */
-				$query = new \GF_Query( $this->form->ID, $parameters['search_criteria'], $parameters['sorting'] );
+				$query = new \GF_Query( $this->form->ID, $parameters[ 'search_criteria' ], $parameters[ 'sorting' ] );
 
-				$query->limit( $parameters['paging']['page_size'] )
-					->offset( ( ( $page - 1 ) * $parameters['paging']['page_size'] ) + $this->settings->get( 'offset' ) );
+				$query->limit( $parameters[ 'paging' ][ 'page_size' ] )
+					->offset( ( ( $page - 1 ) * $parameters[ 'paging' ][ 'page_size' ] ) + $this->settings->get( 'offset' ) );
 
 				/**
 				 * Any joins?
@@ -751,15 +750,15 @@  discard block
 block discarded – undo
 				} );
 			} else {
 				$entries = $this->form->entries
-					->filter( \GV\GF_Entry_Filter::from_search_criteria( $parameters['search_criteria'] ) )
+					->filter( \GV\GF_Entry_Filter::from_search_criteria( $parameters[ 'search_criteria' ] ) )
 					->offset( $this->settings->get( 'offset' ) )
-					->limit( $parameters['paging']['page_size'] )
+					->limit( $parameters[ 'paging' ][ 'page_size' ] )
 					->page( $page );
 
-				if ( ! empty( $parameters['sorting'] ) && ! empty( $parameters['sorting']['key'] ) ) {
+				if ( ! empty( $parameters[ 'sorting' ] ) && ! empty( $parameters[ 'sorting' ][ 'key' ] ) ) {
 					$field = new \GV\Field();
-					$field->ID = $parameters['sorting']['key'];
-					$direction = strtolower( $parameters['sorting']['direction'] ) == 'asc' ? \GV\Entry_Sort::ASC : \GV\Entry_Sort::DESC;
+					$field->ID = $parameters[ 'sorting' ][ 'key' ];
+					$direction = strtolower( $parameters[ 'sorting' ][ 'direction' ] ) == 'asc' ? \GV\Entry_Sort::ASC : \GV\Entry_Sort::DESC;
 					$entries = $entries->sort( new \GV\Entry_Sort( $field, $direction ) );
 				}
 			}

Please login to merge, or discard this patch.

Doc Comments +1 added lines patch added patch discarded remove patch

@@ -690,6 +690,7 @@
 block discarded – undo
 	 * Retrieve the entries for the current view and request.
 	 *
 	 * @param \GV\Request The request. Usued for now.
+	 * @param Request $request
 	 *
 	 * @return \GV\Entry_Collection The entries.
 	 */

Please login to merge, or discard this patch.

future/includes/class-gv-template-entry-legacy.php 1 patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -34,7 +34,7 @@
 block discarded – undo
 		global $post;
 
 		if ( $post ) {
-			$context['post'] = $post;
+			$context[ 'post' ] = $post;
 		}
 
 		\GV\Mocks\Legacy_Context::push( $context );

Please login to merge, or discard this patch.

future/includes/class-gv-template-view-list.php 2 patches

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

@@ -64,9 +64,9 @@
 block discarded – undo
 		 */
 		$hide_empty = apply_filters( 'gravityview/render/hide-empty-zone', Utils::get( $extras, 'hide_empty', $this->view->settings->get( 'hide_empty', false ) ), $context );
 
-		$extras['hide_empty'] = $hide_empty;
-		$extras['label'] = $label;
-		$extras['value'] = $value;
+		$extras[ 'hide_empty' ] = $hide_empty;
+		$extras[ 'label' ] = $label;
+		$extras[ 'value' ] = $value;
 
 		return \gravityview_field_output( $extras, $context );
 	}

Please login to merge, or discard this patch.

Indentation +20 added lines, -20 removed lines patch added patch discarded remove patch

@@ -151,11 +151,11 @@  discard block
 block discarded – undo
 		do_action( 'gravityview/template/list/body/before', $context );
 
 		/**
-		* @action `gravityview_list_body_before` Inside the `tbody`, before any rows are rendered. Can be used to insert additional rows.
-		* @deprecated Use `gravityview/template/list/body/before`
-		* @since 1.0.7
-		* @param \GravityView_View $gravityview_view Current GravityView_View object.
-		*/
+		 * @action `gravityview_list_body_before` Inside the `tbody`, before any rows are rendered. Can be used to insert additional rows.
+		 * @deprecated Use `gravityview/template/list/body/before`
+		 * @since 1.0.7
+		 * @param \GravityView_View $gravityview_view Current GravityView_View object.
+		 */
 		do_action( 'gravityview_list_body_before', \GravityView_View::getInstance() /** ugh! */ );
 	}
 
@@ -177,11 +177,11 @@  discard block
 block discarded – undo
 		do_action( 'gravityview/template/list/body/after', $context );
 
 		/**
-		* @action `gravityview_list_body_after` Inside the `tbody`, after any rows are rendered. Can be used to insert additional rows.
-		* @deprecated Use `gravityview/template/list/body/after`
-		* @since 1.0.7
-		* @param \GravityView_View $gravityview_view Current GravityView_View object.
-		*/
+		 * @action `gravityview_list_body_after` Inside the `tbody`, after any rows are rendered. Can be used to insert additional rows.
+		 * @deprecated Use `gravityview/template/list/body/after`
+		 * @since 1.0.7
+		 * @param \GravityView_View $gravityview_view Current GravityView_View object.
+		 */
 		do_action( 'gravityview_list_body_after', \GravityView_View::getInstance() /** ugh! */ );
 	}
 
@@ -212,11 +212,11 @@  discard block
 block discarded – undo
 		$zone = str_replace( '/', '_', $zone );
 
 		/**
-		* @action `gravityview_list_entry_$zone_before` Inside the `entry`, before any rows are rendered. Can be used to insert additional rows.
-		* @deprecated Use `gravityview/template/list/entry/$zone/before`
-		* @since 1.0.7
-		* @param \GravityView_View $gravityview_view Current GravityView_View object.
-		*/
+		 * @action `gravityview_list_entry_$zone_before` Inside the `entry`, before any rows are rendered. Can be used to insert additional rows.
+		 * @deprecated Use `gravityview/template/list/entry/$zone/before`
+		 * @since 1.0.7
+		 * @param \GravityView_View $gravityview_view Current GravityView_View object.
+		 */
 		do_action( sprintf( 'gravityview_list_entry%sbefore', $zone ), $entry->as_entry(), \GravityView_View::getInstance() /** ugh! */ );
 	}
 
@@ -247,11 +247,11 @@  discard block
 block discarded – undo
 		$zone = str_replace( '/', '_', $zone );
 
 		/**
-		* @action `gravityview_list_entry_$zone_after` Inside the `entry`, after any rows are rendered. Can be used to insert additional rows.
-		* @deprecated Use `gravityview/template/list/entry/after`
-		* @since 1.0.7
-		* @param \GravityView_View $gravityview_view Current GravityView_View object.
-		*/
+		 * @action `gravityview_list_entry_$zone_after` Inside the `entry`, after any rows are rendered. Can be used to insert additional rows.
+		 * @deprecated Use `gravityview/template/list/entry/after`
+		 * @since 1.0.7
+		 * @param \GravityView_View $gravityview_view Current GravityView_View object.
+		 */
 		do_action( sprintf( 'gravityview_list_entry%safter', $zone ), $entry->as_entry(), \GravityView_View::getInstance() /** ugh! */ );
 	}
 }

Please login to merge, or discard this patch.

future/includes/class-gv-core.php 1 patch

Spacing +4 added lines, -4 removed lines patch added patch discarded remove patch

@@ -113,14 +113,14 @@
 block discarded – undo
 
 		/** Require critical legacy core files. @todo Deprecate */
 		require_once $this->plugin->dir( 'includes/helper-functions.php' );
-		require_once $this->plugin->dir( 'includes/class-common.php');
-		require_once $this->plugin->dir( 'includes/connector-functions.php');
+		require_once $this->plugin->dir( 'includes/class-common.php' );
+		require_once $this->plugin->dir( 'includes/connector-functions.php' );
 		require_once $this->plugin->dir( 'includes/class-gravityview-compatibility.php' );
 		require_once $this->plugin->dir( 'includes/class-gravityview-roles-capabilities.php' );
 		require_once $this->plugin->dir( 'includes/class-gravityview-admin-notices.php' );
 		require_once $this->plugin->dir( 'includes/class-admin.php' );
-		require_once $this->plugin->dir( 'includes/class-post-types.php');
-		require_once $this->plugin->dir( 'includes/class-cache.php');
+		require_once $this->plugin->dir( 'includes/class-post-types.php' );
+		require_once $this->plugin->dir( 'includes/class-cache.php' );
 
 		/**
 		 * GravityView extensions and widgets.

Please login to merge, or discard this patch.

future/includes/class-gv-request.php 2 patches

Indentation +4 added lines, -4 removed lines patch added patch discarded remove patch

@@ -112,10 +112,10 @@
 block discarded – undo
 	 */
 	public function is_edit_entry() {
 		/**
-		* @filter `gravityview_is_edit_entry` Whether we're currently on the Edit Entry screen \n
-		* The Edit Entry functionality overrides this value.
-		* @param boolean $is_edit_entry
-		*/
+		 * @filter `gravityview_is_edit_entry` Whether we're currently on the Edit Entry screen \n
+		 * The Edit Entry functionality overrides this value.
+		 * @param boolean $is_edit_entry
+		 */
 		if ( ( $entry = $this->is_entry() ) && apply_filters( 'gravityview_is_edit_entry', false ) ) {
 			return $entry;
 		}

Please login to merge, or discard this patch.

Spacing +8 added lines, -8 removed lines patch added patch discarded remove patch

@@ -45,7 +45,7 @@  discard block
 block discarded – undo
 	 */
 	public static function is_add_oembed_preview() {
 		/** The preview request is a parse-embed AJAX call without a type set. */
-		return ( self::is_ajax() && ! empty( $_POST['action'] ) && $_POST['action'] == 'parse-embed' && ! isset( $_POST['type'] ) );
+		return ( self::is_ajax() && ! empty( $_POST[ 'action' ] ) && $_POST[ 'action' ] == 'parse-embed' && ! isset( $_POST[ 'type' ] ) );
 	}
 
 	/**
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
 	 * @return boolean
 	 */
 	public static function is_rest() {
-		return ! empty( $GLOBALS['wp']->query_vars['rest_route'] );
+		return ! empty( $GLOBALS[ 'wp' ]->query_vars[ 'rest_route' ] );
 	}
 
 	/**
@@ -141,15 +141,15 @@  discard block
 block discarded – undo
 			$get = $_GET;
 		}
 
-		unset( $get['mode'] );
+		unset( $get[ 'mode' ] );
 
 		$get = array_filter( $get, 'gravityview_is_not_empty_string' );
 
-		if( $has_field_key = $this->_has_field_key( $get ) ) {
+		if ( $has_field_key = $this->_has_field_key( $get ) ) {
 			return true;
 		}
 
-		return isset( $get['gv_search'] ) || isset( $get['gv_start'] ) || isset( $get['gv_end'] ) || isset( $get['gv_by'] ) || isset( $get['gv_id'] );
+		return isset( $get[ 'gv_search' ] ) || isset( $get[ 'gv_start' ] ) || isset( $get[ 'gv_end' ] ) || isset( $get[ 'gv_by' ] ) || isset( $get[ 'gv_id' ] );
 	}
 
 	/**
@@ -172,13 +172,13 @@  discard block
 block discarded – undo
 
 		$meta = array();
 		foreach ( $fields as $field ) {
-			if( empty( $field->_gf_field_class_name ) ) {
-				$meta[] = preg_quote( $field->name );
+			if ( empty( $field->_gf_field_class_name ) ) {
+				$meta[ ] = preg_quote( $field->name );
 			}
 		}
 
 		foreach ( $get as $key => $value ) {
-			if ( preg_match('/^filter_(([0-9_]+)|'. implode( '|', $meta ) .')$/sm', $key ) ) {
+			if ( preg_match( '/^filter_(([0-9_]+)|' . implode( '|', $meta ) . ')$/sm', $key ) ) {
 				$has_field_key = true;
 				break;
 			}

Please login to merge, or discard this patch.

future/includes/class-gv-collection-entry-sort.php 1 patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -78,7 +78,7 @@
 block discarded – undo
 		if ( $this->field ) {
 			return array(
 				'key' => $this->field->ID,
-				'direction' => $this->direction ? : self::ASC,
+				'direction' => $this->direction ?: self::ASC,
 				'is_numeric' => self::ALPHA ? true : false,
 			);
 		}

Please login to merge, or discard this patch.

future/includes/class-gv-entry-multi.php 1 patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -44,7 +44,7 @@  discard block
 block discarded – undo
 			if ( ! $entry instanceof Entry ) {
 				continue;
 			}
-			$_entry->entries[ $entry['form_id'] ]  = &$entry;
+			$_entry->entries[ $entry[ 'form_id' ] ] = &$entry;
 		}
 		return $_entry;
 	}
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
 
 			foreach ( $this->entries as $entry ) {
 				$entry = $entry->as_entry();
-				$_entry['_multi'][ $entry['form_id'] ] = $entry;
+				$_entry[ '_multi' ][ $entry[ 'form_id' ] ] = $entry;
 			}
 		}
 

Please login to merge, or discard this patch.

		@@ -108,7 +108,7 @@ discard block
		block discarded – undo
108	108	$supports = array( 'title', 'revisions' );
109	109
110	110	if ( $is_hierarchical ) {
111		- $supports[] = 'page-attributes';
	111	+ $supports[ ] = 'page-attributes';
112	112	}
113	113
114	114	/**
		@@ -288,13 +288,13 @@ discard block
		block discarded – undo
288	288	return __( 'You are not allowed to view this content.', 'gravityview' );
289	289	}
290	290
291		- $is_admin_and_can_view = $view->settings->get( 'admin_show_all_statuses' ) && \GVCommon::has_cap('gravityview_moderate_entries', $view->ID );
	291	+ $is_admin_and_can_view = $view->settings->get( 'admin_show_all_statuses' ) && \GVCommon::has_cap( 'gravityview_moderate_entries', $view->ID );
292	292
293	293	/**
294	294	* Editing a single entry.
295	295	*/
296	296	if ( $entry = $request->is_edit_entry() ) {
297		- if ( $entry['status'] != 'active' ) {
	297	+ if ( $entry[ 'status' ] != 'active' ) {
298	298	gravityview()->log->notice( 'Entry ID #{entry_id} is not active', array( 'entry_id' => $entry->ID ) );
299	299	return __( 'You are not allowed to view this content.', 'gravityview' );
300	300	}
		@@ -305,7 +305,7 @@ discard block
		block discarded – undo
305	305	}
306	306
307	307	if ( $view->settings->get( 'show_only_approved' ) && ! $is_admin_and_can_view ) {
308		- if ( ! \GravityView_Entry_Approval_Status::is_approved( gform_get_meta( $entry->ID, \GravityView_Entry_Approval::meta_key ) ) ) {
	308	+ if ( ! \GravityView_Entry_Approval_Status::is_approved( gform_get_meta( $entry->ID, \GravityView_Entry_Approval::meta_key ) ) ) {
309	309	gravityview()->log->error( 'Entry ID #{entry_id} is not approved for viewing', array( 'entry_id' => $entry->ID ) );
310	310	return __( 'You are not allowed to view this content.', 'gravityview' );
311	311	}
		@@ -318,7 +318,7 @@ discard block
		block discarded – undo
318	318	* Viewing a single entry.
319	319	*/
320	320	} else if ( $entry = $request->is_entry() ) {
321		- if ( $entry['status'] != 'active' ) {
	321	+ if ( $entry[ 'status' ] != 'active' ) {
322	322	gravityview()->log->notice( 'Entry ID #{entry_id} is not active', array( 'entry_id' => $entry->ID ) );
323	323	return __( 'You are not allowed to view this content.', 'gravityview' );
324	324	}
		@@ -329,7 +329,7 @@ discard block
		block discarded – undo
329	329	}
330	330
331	331	if ( $view->settings->get( 'show_only_approved' ) && ! $is_admin_and_can_view ) {
332		- if ( ! \GravityView_Entry_Approval_Status::is_approved( gform_get_meta( $entry->ID, \GravityView_Entry_Approval::meta_key ) ) ) {
	332	+ if ( ! \GravityView_Entry_Approval_Status::is_approved( gform_get_meta( $entry->ID, \GravityView_Entry_Approval::meta_key ) ) ) {
333	333	gravityview()->log->error( 'Entry ID #{entry_id} is not approved for viewing', array( 'entry_id' => $entry->ID ) );
334	334	return __( 'You are not allowed to view this content.', 'gravityview' );
335	335	}
		@@ -337,7 +337,7 @@ discard block
		block discarded – undo
337	337
338	338	$error = \GVCommon::check_entry_display( $entry->as_entry() );
339	339
340		- if( is_wp_error( $error ) ) {
	340	+ if ( is_wp_error( $error ) ) {
341	341	gravityview()->log->error( 'Entry ID #{entry_id} is not approved for viewing: {message}', array( 'entry_id' => $entry->ID, 'message' => $error->get_error_message() ) );
342	342	return __( 'You are not allowed to view this content.', 'gravityview' );
343	343	}
		@@ -396,7 +396,7 @@ discard block
		block discarded – undo
396	396	$join_column = is_numeric( $join_column ) ? GF_Field::by_id( $join, $join_column ) : Internal_Field( $join_column );
397	397	$join_on_column = is_numeric( $join_on_column ) ? GF_Field::by_id( $join_on, $join_on_column ) : Internal_Field( $join_on_column );
398	398
399		- $joins [] = new Join( $join, $join_column, $join_on, $join_on_column );
	399	+ $joins [ ] = new Join( $join, $join_column, $join_on, $join_on_column );
400	400	}
401	401
402	402	return $joins;
		@@ -431,10 +431,10 @@ discard block
		block discarded – undo
431	431
432	432	list( $join, $join_column, $join_on, $join_on_column ) = $meta;
433	433
434		- $forms_ids [] = GF_Form::by_id( $join_on );
	434	+ $forms_ids [ ] = GF_Form::by_id( $join_on );
435	435	}
436	436
437		- return ( !empty( $forms_ids) ) ? $forms_ids : null;
	437	+ return ( ! empty( $forms_ids ) ) ? $forms_ids : null;
438	438	}
439	439
440	440	/**
		@@ -464,7 +464,7 @@ discard block
		block discarded – undo
464	464	if ( ! $view->form ) {
465	465	gravityview()->log->error( 'View #{view_id} tried attaching non-existent Form #{form_id} to it.', array(
466	466	'view_id' => $view->ID,
467		- 'form_id' => $view->_gravityview_form_id ? : 0,
	467	+ 'form_id' => $view->_gravityview_form_id ?: 0,
468	468	) );
469	469	}
470	470
		@@ -690,28 +690,27 @@ discard block
		block discarded – undo
690	690	* @todo: Stop using _frontend and use something like $request->get_search_criteria() instead
691	691	*/
692	692	$parameters = \GravityView_frontend::get_view_entries_parameters( $this->settings->as_atts(), $this->form->ID );
693		- $parameters['context_view_id'] = $this->ID;
	693	+ $parameters[ 'context_view_id' ] = $this->ID;
694	694	$parameters = \GVCommon::calculate_get_entries_criteria( $parameters, $this->form->ID );
695	695
696	696	if ( $request instanceof REST\Request ) {
697	697	$atts = $this->settings->as_atts();
698	698	$paging_parameters = wp_parse_args( $request->get_paging(), array(
699		- 'paging' => array( 'page_size' => $atts['page_size'] ),
	699	+ 'paging' => array( 'page_size' => $atts[ 'page_size' ] ),
700	700	) );
701		- $parameters['paging'] = $paging_parameters['paging'];
	701	+ $parameters[ 'paging' ] = $paging_parameters[ 'paging' ];
702	702	}
703	703
704		- $page = Utils::get( $parameters['paging'], 'current_page' ) ?
705		- : ( ( ( $parameters['paging']['offset'] - $this->settings->get( 'offset' ) ) / $parameters['paging']['page_size'] ) + 1 );
	704	+ $page = Utils::get( $parameters[ 'paging' ], 'current_page' ) ?: ( ( ( $parameters[ 'paging' ][ 'offset' ] - $this->settings->get( 'offset' ) ) / $parameters[ 'paging' ][ 'page_size' ] ) + 1 );
706	705
707	706	if ( gravityview()->plugin->supports( Plugin::FEATURE_GFQUERY ) ) {
708	707	/**
709	708	* New \GF_Query stuff :)
710	709	*/
711		- $query = new \GF_Query( $this->form->ID, $parameters['search_criteria'], $parameters['sorting'] );
	710	+ $query = new \GF_Query( $this->form->ID, $parameters[ 'search_criteria' ], $parameters[ 'sorting' ] );
712	711
713		- $query->limit( $parameters['paging']['page_size'] )
714		- ->offset( ( ( $page - 1 ) * $parameters['paging']['page_size'] ) + $this->settings->get( 'offset' ) );
	712	+ $query->limit( $parameters[ 'paging' ][ 'page_size' ] )
	713	+ ->offset( ( ( $page - 1 ) * $parameters[ 'paging' ][ 'page_size' ] ) + $this->settings->get( 'offset' ) );
715	714
716	715	/**
717	716	* Any joins?
		@@ -751,15 +750,15 @@ discard block
		block discarded – undo
751	750	} );
752	751	} else {
753	752	$entries = $this->form->entries
754		- ->filter( \GV\GF_Entry_Filter::from_search_criteria( $parameters['search_criteria'] ) )
	753	+ ->filter( \GV\GF_Entry_Filter::from_search_criteria( $parameters[ 'search_criteria' ] ) )
755	754	->offset( $this->settings->get( 'offset' ) )
756		- ->limit( $parameters['paging']['page_size'] )
	755	+ ->limit( $parameters[ 'paging' ][ 'page_size' ] )
757	756	->page( $page );
758	757
759		- if ( ! empty( $parameters['sorting'] ) && ! empty( $parameters['sorting']['key'] ) ) {
	758	+ if ( ! empty( $parameters[ 'sorting' ] ) && ! empty( $parameters[ 'sorting' ][ 'key' ] ) ) {
760	759	$field = new \GV\Field();
761		- $field->ID = $parameters['sorting']['key'];
762		- $direction = strtolower( $parameters['sorting']['direction'] ) == 'asc' ? \GV\Entry_Sort::ASC : \GV\Entry_Sort::DESC;
	760	+ $field->ID = $parameters[ 'sorting' ][ 'key' ];
	761	+ $direction = strtolower( $parameters[ 'sorting' ][ 'direction' ] ) == 'asc' ? \GV\Entry_Sort::ASC : \GV\Entry_Sort::DESC;
763	762	$entries = $entries->sort( new \GV\Entry_Sort( $field, $direction ) );
764	763	}
765	764	}

		@@ -690,6 +690,7 @@
		block discarded – undo
690	690	* Retrieve the entries for the current view and request.
691	691	*
692	692	* @param \GV\Request The request. Usued for now.
	693	+ * @param Request $request
693	694	*
694	695	* @return \GV\Entry_Collection The entries.
695	696	*/

		@@ -151,11 +151,11 @@ discard block
		block discarded – undo
151	151	do_action( 'gravityview/template/list/body/before', $context );
152	152
153	153	/**
154		- * @action `gravityview_list_body_before` Inside the `tbody`, before any rows are rendered. Can be used to insert additional rows.
155		- * @deprecated Use `gravityview/template/list/body/before`
156		- * @since 1.0.7
157		- * @param \GravityView_View $gravityview_view Current GravityView_View object.
158		- */
	154	+ * @action `gravityview_list_body_before` Inside the `tbody`, before any rows are rendered. Can be used to insert additional rows.
	155	+ * @deprecated Use `gravityview/template/list/body/before`
	156	+ * @since 1.0.7
	157	+ * @param \GravityView_View $gravityview_view Current GravityView_View object.
	158	+ */
159	159	do_action( 'gravityview_list_body_before', \GravityView_View::getInstance() /** ugh! */ );
160	160	}
161	161
		@@ -177,11 +177,11 @@ discard block
		block discarded – undo
177	177	do_action( 'gravityview/template/list/body/after', $context );
178	178
179	179	/**
180		- * @action `gravityview_list_body_after` Inside the `tbody`, after any rows are rendered. Can be used to insert additional rows.
181		- * @deprecated Use `gravityview/template/list/body/after`
182		- * @since 1.0.7
183		- * @param \GravityView_View $gravityview_view Current GravityView_View object.
184		- */
	180	+ * @action `gravityview_list_body_after` Inside the `tbody`, after any rows are rendered. Can be used to insert additional rows.
	181	+ * @deprecated Use `gravityview/template/list/body/after`
	182	+ * @since 1.0.7
	183	+ * @param \GravityView_View $gravityview_view Current GravityView_View object.
	184	+ */
185	185	do_action( 'gravityview_list_body_after', \GravityView_View::getInstance() /** ugh! */ );
186	186	}
187	187
		@@ -212,11 +212,11 @@ discard block
		block discarded – undo
212	212	$zone = str_replace( '/', '_', $zone );
213	213
214	214	/**
215		- * @action `gravityview_list_entry_$zone_before` Inside the `entry`, before any rows are rendered. Can be used to insert additional rows.
216		- * @deprecated Use `gravityview/template/list/entry/$zone/before`
217		- * @since 1.0.7
218		- * @param \GravityView_View $gravityview_view Current GravityView_View object.
219		- */
	215	+ * @action `gravityview_list_entry_$zone_before` Inside the `entry`, before any rows are rendered. Can be used to insert additional rows.
	216	+ * @deprecated Use `gravityview/template/list/entry/$zone/before`
	217	+ * @since 1.0.7
	218	+ * @param \GravityView_View $gravityview_view Current GravityView_View object.
	219	+ */
220	220	do_action( sprintf( 'gravityview_list_entry%sbefore', $zone ), $entry->as_entry(), \GravityView_View::getInstance() /** ugh! */ );
221	221	}
222	222
		@@ -247,11 +247,11 @@ discard block
		block discarded – undo
247	247	$zone = str_replace( '/', '_', $zone );
248	248
249	249	/**
250		- * @action `gravityview_list_entry_$zone_after` Inside the `entry`, after any rows are rendered. Can be used to insert additional rows.
251		- * @deprecated Use `gravityview/template/list/entry/after`
252		- * @since 1.0.7
253		- * @param \GravityView_View $gravityview_view Current GravityView_View object.
254		- */
	250	+ * @action `gravityview_list_entry_$zone_after` Inside the `entry`, after any rows are rendered. Can be used to insert additional rows.
	251	+ * @deprecated Use `gravityview/template/list/entry/after`
	252	+ * @since 1.0.7
	253	+ * @param \GravityView_View $gravityview_view Current GravityView_View object.
	254	+ */
255	255	do_action( sprintf( 'gravityview_list_entry%safter', $zone ), $entry->as_entry(), \GravityView_View::getInstance() /** ugh! */ );
256	256	}
257	257	}

		@@ -113,14 +113,14 @@
		block discarded – undo
113	113
114	114	/** Require critical legacy core files. @todo Deprecate */
115	115	require_once $this->plugin->dir( 'includes/helper-functions.php' );
116		- require_once $this->plugin->dir( 'includes/class-common.php');
117		- require_once $this->plugin->dir( 'includes/connector-functions.php');
	116	+ require_once $this->plugin->dir( 'includes/class-common.php' );
	117	+ require_once $this->plugin->dir( 'includes/connector-functions.php' );
118	118	require_once $this->plugin->dir( 'includes/class-gravityview-compatibility.php' );
119	119	require_once $this->plugin->dir( 'includes/class-gravityview-roles-capabilities.php' );
120	120	require_once $this->plugin->dir( 'includes/class-gravityview-admin-notices.php' );
121	121	require_once $this->plugin->dir( 'includes/class-admin.php' );
122		- require_once $this->plugin->dir( 'includes/class-post-types.php');
123		- require_once $this->plugin->dir( 'includes/class-cache.php');
	122	+ require_once $this->plugin->dir( 'includes/class-post-types.php' );
	123	+ require_once $this->plugin->dir( 'includes/class-cache.php' );
124	124
125	125	/**
126	126	* GravityView extensions and widgets.

		@@ -78,7 +78,7 @@
		block discarded – undo
78	78	if ( $this->field ) {
79	79	return array(
80	80	'key' => $this->field->ID,
81		- 'direction' => $this->direction ? : self::ASC,
	81	+ 'direction' => $this->direction ?: self::ASC,
82	82	'is_numeric' => self::ALPHA ? true : false,
83	83	);
84	84	}

		@@ -25,7 +25,7 @@ discard block
		block discarded – undo
25	25	printf( '<body style="padding: 0; margin: 0; font-family: -apple-system, BlinkMacSystemFont, \'Segoe UI\', Roboto, Oxygen-Sans, Ubuntu, Cantarell, \'Helvetica Neue\', sans-serif;">' );
26	26	printf( '<img src="' . plugins_url( 'assets/images/astronaut-200x263.png', GRAVITYVIEW_FILE ) . '" alt="The GravityView Astronaut Says:" style="float: left; height: 60px; margin-right : 10px;" />' );
27	27	printf( __( "%sGravityView requires PHP Version %s or newer.%s \n\nYou're using Version %s. Please ask your host to upgrade your server's PHP.", 'gravityview' ),
28		- '<h3 style="font-size:16px; margin: 0 0 8px 0;">', GV_MIN_PHP_VERSION , "</h3>\n\n", $version );
	28	+ '<h3 style="font-size:16px; margin: 0 0 8px 0;">', GV_MIN_PHP_VERSION, "</h3>\n\n", $version );
29	29	printf( '</body>' );
30	30	}
31	31
		@@ -34,7 +34,7 @@ discard block
		block discarded – undo
34	34	}
35	35
36	36	/** The future branch of GravityView requires PHP 5.3+ namespaces and SPL. */
37		-if ( version_compare( phpversion(), '5.3.0' , '<' ) ) {
	37	+if ( version_compare( phpversion(), '5.3.0', '<' ) ) {
38	38	require GRAVITYVIEW_DIR . 'future/_stubs.php';
39	39
40	40	/** All looks fine. */

		@@ -61,7 +61,7 @@
		block discarded – undo
61	61	$action_text = sprintf( esc_html__( 'Add fields to %s', 'gravityview' ), $tab );
62	62	$message = esc_html__( 'You can only see this message because you are able to edit this View.', 'gravityview' );
63	63
64		- $image = sprintf( '<img alt="%s" src="%s" style="margin-top: 10px;" />', $tab, esc_url( plugins_url( sprintf( 'assets/images/tab-%s.png', $context ), GRAVITYVIEW_FILE ) ) );
	64	+ $image = sprintf( '<img alt="%s" src="%s" style="margin-top: 10px;" />', $tab, esc_url( plugins_url( sprintf( 'assets/images/tab-%s.png', $context ), GRAVITYVIEW_FILE ) ) );
65	65	$output = sprintf( '<h3>%s <strong><a href="%s">%s</a></strong></h3><p>%s</p>', $title, esc_url( $edit_link ), $action_text, $message );
66	66
67	67	echo \GVCommon::generate_notice( $output . $image, 'gv-error error', 'edit_gravityview', $gravityview->view->ID );

		@@ -62,7 +62,6 @@
		block discarded – undo
62	62	*
63	63	* Internal static cache for gets, and whatnot.
64	64	* This is not persistent, resets across requests.
65		-
66	65	* @internal
67	66	*/
68	67	private static $cache = array();

		@@ -34,7 +34,7 @@
		block discarded – undo
34	34	global $post;
35	35
36	36	if ( $post ) {
37		- $context['post'] = $post;
	37	+ $context[ 'post' ] = $post;
38	38	}
39	39
40	40	\GV\Mocks\Legacy_Context::push( $context );

		@@ -64,9 +64,9 @@
		block discarded – undo
64	64	*/
65	65	$hide_empty = apply_filters( 'gravityview/render/hide-empty-zone', Utils::get( $extras, 'hide_empty', $this->view->settings->get( 'hide_empty', false ) ), $context );
66	66
67		- $extras['hide_empty'] = $hide_empty;
68		- $extras['label'] = $label;
69		- $extras['value'] = $value;
	67	+ $extras[ 'hide_empty' ] = $hide_empty;
	68	+ $extras[ 'label' ] = $label;
	69	+ $extras[ 'value' ] = $value;
70	70
71	71	return \gravityview_field_output( $extras, $context );
72	72	}

		@@ -112,10 +112,10 @@
		block discarded – undo
112	112	*/
113	113	public function is_edit_entry() {
114	114	/**
115		- * @filter `gravityview_is_edit_entry` Whether we're currently on the Edit Entry screen \n
116		- * The Edit Entry functionality overrides this value.
117		- * @param boolean $is_edit_entry
118		- */
	115	+ * @filter `gravityview_is_edit_entry` Whether we're currently on the Edit Entry screen \n
	116	+ * The Edit Entry functionality overrides this value.
	117	+ * @param boolean $is_edit_entry
	118	+ */
119	119	if ( ( $entry = $this->is_entry() ) && apply_filters( 'gravityview_is_edit_entry', false ) ) {
120	120	return $entry;
121	121	}

		@@ -45,7 +45,7 @@ discard block
		block discarded – undo
45	45	*/
46	46	public static function is_add_oembed_preview() {
47	47	/** The preview request is a parse-embed AJAX call without a type set. */
48		- return ( self::is_ajax() && ! empty( $_POST['action'] ) && $_POST['action'] == 'parse-embed' && ! isset( $_POST['type'] ) );
	48	+ return ( self::is_ajax() && ! empty( $_POST[ 'action' ] ) && $_POST[ 'action' ] == 'parse-embed' && ! isset( $_POST[ 'type' ] ) );
49	49	}
50	50
51	51	/**
		@@ -63,7 +63,7 @@ discard block
		block discarded – undo
63	63	* @return boolean
64	64	*/
65	65	public static function is_rest() {
66		- return ! empty( $GLOBALS['wp']->query_vars['rest_route'] );
	66	+ return ! empty( $GLOBALS[ 'wp' ]->query_vars[ 'rest_route' ] );
67	67	}
68	68
69	69	/**
		@@ -141,15 +141,15 @@ discard block
		block discarded – undo
141	141	$get = $_GET;
142	142	}
143	143
144		- unset( $get['mode'] );
	144	+ unset( $get[ 'mode' ] );
145	145
146	146	$get = array_filter( $get, 'gravityview_is_not_empty_string' );
147	147
148		- if( $has_field_key = $this->_has_field_key( $get ) ) {
	148	+ if ( $has_field_key = $this->_has_field_key( $get ) ) {
149	149	return true;
150	150	}
151	151
152		- return isset( $get['gv_search'] ) \|\| isset( $get['gv_start'] ) \|\| isset( $get['gv_end'] ) \|\| isset( $get['gv_by'] ) \|\| isset( $get['gv_id'] );
	152	+ return isset( $get[ 'gv_search' ] ) \|\| isset( $get[ 'gv_start' ] ) \|\| isset( $get[ 'gv_end' ] ) \|\| isset( $get[ 'gv_by' ] ) \|\| isset( $get[ 'gv_id' ] );
153	153	}
154	154
155	155	/**
		@@ -172,13 +172,13 @@ discard block
		block discarded – undo
172	172
173	173	$meta = array();
174	174	foreach ( $fields as $field ) {
175		- if( empty( $field->_gf_field_class_name ) ) {
176		- $meta[] = preg_quote( $field->name );
	175	+ if ( empty( $field->_gf_field_class_name ) ) {
	176	+ $meta[ ] = preg_quote( $field->name );
177	177	}
178	178	}
179	179
180	180	foreach ( $get as $key => $value ) {
181		- if ( preg_match('/^filter_(([0-9_]+)\|'. implode( '\|', $meta ) .')$/sm', $key ) ) {
	181	+ if ( preg_match( '/^filter_(([0-9_]+)\|' . implode( '\|', $meta ) . ')$/sm', $key ) ) {
182	182	$has_field_key = true;
183	183	break;
184	184	}

		@@ -44,7 +44,7 @@ discard block
		block discarded – undo
44	44	if ( ! $entry instanceof Entry ) {
45	45	continue;
46	46	}
47		- $_entry->entries[ $entry['form_id'] ] = &$entry;
	47	+ $_entry->entries[ $entry[ 'form_id' ] ] = &$entry;
48	48	}
49	49	return $_entry;
50	50	}
		@@ -65,7 +65,7 @@ discard block
		block discarded – undo
65	65
66	66	foreach ( $this->entries as $entry ) {
67	67	$entry = $entry->as_entry();
68		- $_entry['_multi'][ $entry['form_id'] ] = $entry;
	68	+ $_entry[ '_multi' ][ $entry[ 'form_id' ] ] = $entry;
69	69	}
70	70	}
71	71

gravityview / GravityView

Pull Request — develop (#1130)

Status

Category

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Indentation -1 removed lines patch added patch discarded remove patch

Spacing +23 added lines, -24 removed lines patch added patch discarded remove patch

Doc Comments +1 added lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

Indentation +20 added lines, -20 removed lines patch added patch discarded remove patch

Spacing +4 added lines, -4 removed lines patch added patch discarded remove patch

Indentation +4 added lines, -4 removed lines patch added patch discarded remove patch

Spacing +8 added lines, -8 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch