Inspection of "2.0.8" - gravityview/GravityView - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#1071)

by Gennady

created 2018-05-28 21:35 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-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' ] );
 
-		if( $has_field_key = $this->_has_field_key( $get ) ) {
+		if ( $has_field_key = $this->_has_field_key( $get ) ) {
 			return true;
 		}
 
 		$get = array_filter( $get, 'gravityview_is_not_empty_string' );
 
-		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.

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

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

@@ -50,8 +50,8 @@
 block discarded – undo
 		/**
 		 * By array.
 		 */
-		if ( is_array( $view ) && ! empty( $view['id'] ) ) {
-			return $this->get( $view['id'] );
+		if ( is_array( $view ) && ! empty( $view[ 'id' ] ) ) {
+			return $this->get( $view[ 'id' ] );
 		}
 
 		/**

Please login to merge, or discard this patch.

		@@ -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 );

		@@ -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		- if( $has_field_key = $this->_has_field_key( $get ) ) {
	146	+ if ( $has_field_key = $this->_has_field_key( $get ) ) {
147	147	return true;
148	148	}
149	149
150	150	$get = array_filter( $get, 'gravityview_is_not_empty_string' );
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

		@@ -50,8 +50,8 @@
		block discarded – undo
50	50	/**
51	51	* By array.
52	52	*/
53		- if ( is_array( $view ) && ! empty( $view['id'] ) ) {
54		- return $this->get( $view['id'] );
	53	+ if ( is_array( $view ) && ! empty( $view[ 'id' ] ) ) {
	54	+ return $this->get( $view[ 'id' ] );
55	55	}
56	56
57	57	/**

gravityview / GravityView

Pull Request — master (#1071)

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

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

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