+global\GravityView_Admin::is_admin_page()

Complexity

Conditions	23
Paths	577

Size

Total Lines	46
Code Lines	23

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
	static function is_admin_page( $hook = '', $page = NULL ) {

It is generally recommended to explicitly declare the visibility for methods.

TRUE, FALSE and NULL must be lowercase; expected null, but found NULL.

		global $current_screen, $plugin_page, $pagenow, $post;

Use of global functionality is not recommended; it makes your code harder to test, and less reusable.

		if( ! is_admin() ) { return false; }

		$is_page = false;

		$is_gv_screen = (!empty($current_screen) && isset($current_screen->post_type) && $current_screen->post_type === 'gravityview');

Expected 1 space before "!"; 0 found

Expected 1 space after "!"; 0 found

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

		$is_gv_post_type_get = (isset($_GET['post_type']) && $_GET['post_type'] === 'gravityview');

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

Detected access of super global var $_GET, probably need manual inspection.

Detected usage of a non-sanitized input variable: $_GET

		$is_gv_settings_get = isset( $_GET['page'] ) && $_GET['page'] === 'gravityview_settings';

Detected access of super global var $_GET, probably need manual inspection.

Detected usage of a non-sanitized input variable: $_GET

		if( empty( $post ) && $pagenow === 'post.php' && !empty( $_GET['post'] ) ) {

Found "=== '". Use Yoda Condition checks, you must

Expected 1 space after "!"; 0 found

			$gv_post = get_post( intval( $_GET['post'] ) );

Detected access of super global var $_GET, probably need manual inspection.

			$is_gv_post_type = (!empty($gv_post) && !empty($gv_post->post_type) && $gv_post->post_type === 'gravityview');

Expected 1 space before "!"; 0 found

Expected 1 space after "!"; 0 found

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

		} else {
			$is_gv_post_type = (!empty($post) && !empty($post->post_type) && $post->post_type === 'gravityview');

Expected 1 space before "!"; 0 found

Expected 1 space after "!"; 0 found

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

		}

		if( $is_gv_screen || $is_gv_post_type || $is_gv_post_type || $is_gv_post_type_get || $is_gv_settings_get ) {

			// $_GET `post_type` variable

38% of this comment could be valid code. Did you maybe forget this after debugging?

			if(in_array($pagenow, array( 'post.php' , 'post-new.php' )) ) {

No space after opening parenthesis is prohibited

Expected 1 spaces after opening bracket; 0 found

Expected 1 spaces before closing bracket; 0 found

Expected 0 spaces between "'post.php'" and comma; 1 found

				$is_page = 'single';
			} else if ( in_array( $plugin_page, array( 'gravityview_settings', 'gravityview_page_gravityview_settings' ) ) || ( !empty( $_GET['page'] ) && $_GET['page'] === 'gravityview_settings' ) ) {

Found "=== '". Use Yoda Condition checks, you must

Expected 1 space after "!"; 0 found

				$is_page = 'settings';
			} else {
				$is_page = 'views';
			}
		}

		/**
		 * @filter `gravityview_is_admin_page` Is the current admin page a GravityView-related page?
		 * @param[in,out] string|bool $is_page If false, no. If string, the name of the page (`single`, `settings`, or `views`)
		 * @param[in] string $hook The name of the page to check against. Is passed to the method.
		 */
		$is_page = apply_filters( 'gravityview_is_admin_page', $is_page, $hook );

		// If the current page is the same as the compared page
		if( !empty( $page ) ) {

Expected 1 space after "!"; 0 found

			return $is_page === $page;
		}

		return $is_page;
	}