Entry::get_permalink() - Code Metrics - Inspection of "Merge pull request #1064 from gravityview/develop" - gravityview/GravityView - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 397f5b...27d97b )

by Zack

created 2018-05-24 07:30 UTC

Entry::get_permalink() F

↳ Parent: Entry

Complexity

Conditions	15
Paths	360

Size

Total Lines	77
Code Lines	30

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	29
CRAP Score	15

Importance

Changes

Metric	Value
cc	15
eloc	30
nc	360
nop	3
dl	0
loc	77
ccs	29
cts	29
cp	1
crap	15
rs	3.9271
c	0
b	0
f	0

How to fix Long Method Complexity

<?php

namespace GV;

/** If this file is called directly, abort. */
if ( ! defined( 'GRAVITYVIEW_DIR' ) ) {
	die();
}

/**
 * The base \GV\Entry class.
 *
 * Contains all entry data and some processing and logic rules.
 */
abstract class Entry {

	/**
	 * @var string The identifier of the backend used for this entry.
	 * @api
	 * @since 2.0
	 */
	public static $backend = null;

	/**
	 * @var int The ID for this entry.
	 *
	 * @api
	 * @since 2.0
	 */
	public $ID = null;

	/**
	 * @var mixed The backing entry.
	 */
	protected $entry;
	
	/**
	 * Adds the necessary rewrites for single Entries.
	 *
	 * @internal
	 * @return void
	 */
	public static function add_rewrite_endpoint() {
		global $wp_rewrite;
function myFunction($a, $b) {
    // Do something
}

		$endpoint = self::get_endpoint_name();

		/** Let's make sure the endpoint array is not polluted. */
		if ( in_array( array( EP_ALL, $endpoint, $endpoint ), $wp_rewrite->endpoints ) ) {
			return;
		}

		add_rewrite_endpoint( $endpoint, EP_ALL );
	}

	/**
	 * Return the endpoint name for a single Entry.
	 *
	 * Also used as the query_var for the time being.
	 *
	 * @internal
	 * @return string The name. Default: "entry"
	 */
	public static function get_endpoint_name() {
		/**
		 * @filter `gravityview_directory_endpoint` Change the slug used for single entries
		 * @param[in,out] string $endpoint Slug to use when accessing single entry. Default: `entry`
		 */
		$endpoint = apply_filters( 'gravityview_directory_endpoint', 'entry' );

		return sanitize_title( $endpoint );
	}

	/**
	 * Construct a \GV\Entry instance by ID.
	 *
	 * @param int|string $entry_id The internal entry ID.
	 *
	 * @api
	 * @since 2.0
	 * @return \GV\Entry|null An instance of this entry or null if not found.
	 */
	public static function by_id( $entry_id ) {
		return null;
	}

	/**
	 * Return the backing entry object.
	 *
	 * @return array The backing entry object.
	 */
	public function as_entry() {
		return $this->entry;
	}

	/**
	 * Return the link to this entry in the supplied context.
	 *
	 * @api
	 * @since 2.0
	 *
	 * @param \GV\View|null $view The View context.
	 * @param \GV\Request $request The Request (current if null).
	 * @param boolean $track_directory Keep the housing directory arguments intact (used for breadcrumbs, for example). Default: true.
	 *
	 * @return string The permalink to this entry.
	 */
	public function get_permalink( \GV\View $view = null, \GV\Request $request = null, $track_directory = true ) {
		if ( is_null( $request ) ) {
			$request = &gravityview()->request;
		}

		global $post;
function myFunction($a, $b) {
    // Do something
}

		$args = array();

		$view_id = is_null ( $view ) ? null : $view->ID;
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}

		$permalink = null;

		/** This is not a regular view. */
		if ( ! $request->is_view() ) {

			/** Must be an embed of some sort. */
			if ( is_object( $post ) && is_numeric( $post->ID ) ) {
				$permalink = get_permalink( $post->ID );
				$args['gvid'] = $view_id;
			}
		}
		
		/** Fallback to regular view base. */
		if ( is_null( $permalink ) ) {
			$permalink = get_permalink( $view_id );
		}

		/**
		 * @filter `gravityview_directory_link` Modify the URL to the View "directory" context
		 * @since 1.19.4
		 * @param string $link URL to the View's "directory" context (Multiple Entries screen)
		 * @param int $post_id ID of the post to link to. If the View is embedded, it is the post or page ID
		 */
		$permalink = apply_filters( 'gravityview_directory_link', $permalink, $request->is_view() ? $view_id : ( $post ? $post->ID : null ) );

		$entry_endpoint_name = \GV\Entry::get_endpoint_name();
		$entry_slug = \GravityView_API::get_entry_slug( $this->ID, $this->as_entry() );

		/** Assemble the permalink. */
		if ( get_option( 'permalink_structure' ) && ! is_preview() ) {
			/**
			 * Make sure the $directory_link doesn't contain any query otherwise it will break when adding the entry slug.
			 * @since 1.16.5
			 */
			$link_parts = explode( '?', $permalink );

			$query = ! empty( $link_parts[1] ) ? '?' . $link_parts[1] : '';

			$permalink = trailingslashit( $link_parts[0] ) . $entry_endpoint_name . '/'. $entry_slug .'/' . $query;
		} else {
			$args[ $entry_endpoint_name ] = $entry_slug;
		}

		if ( $track_directory ) {
			if ( ! empty( $_GET['pagenum'] ) ) {
				$args['pagenum'] = intval( $_GET['pagenum'] );

			}

			if ( $sort = Utils::_GET( 'sort' ) ) {
				$args['sort'] = $sort;
				$args['dir'] = Utils::_GET( 'dir' );
			}
		}

		$permalink = add_query_arg( $args, $permalink );

		/**
		 * @filter `gravityview/entry/permalink` The permalink of this entry.
		 * @since 2.0
		 * @param string $permalink The permalink.
		 * @param \GV\Entry $entry The entry we're retrieving it for.
		 * @param \GV\View|null $view The view context.
		 * @param \GV\Request $reqeust The request context.
		 */
		return apply_filters( 'gravityview/entry/permalink', $permalink, $this, $view, $request );
	}
}


1		<?php
		0 ignored issues – show Coding Style Compatibility introduced 2017-01-22 20:12 UTC by Report Bug Copy Issue Report For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 14 and the first side effect is on line 6. The PSR-1: Basic Coding Standard recommends that a file should either introduce new symbols, that is classes, functions, constants or similar, or have side effects. Side effects are anything that executes logic, like for example printing output, changing ini settings or writing to a file. The idea behind this recommendation is that merely auto-loading a class should not change the state of an application. It also promotes a cleaner style of programming and makes your code less prone to errors, because the logic is not spread out all over the place. To learn more about the PSR-1, please see the PHP-FIG site on the PSR-1. Loading history...
2		namespace GV;
3
4		/** If this file is called directly, abort. */
5		if ( ! defined( 'GRAVITYVIEW_DIR' ) ) {
6		die();
7		}
8
9		/**
10		* The base \GV\Entry class.
11		*
12		* Contains all entry data and some processing and logic rules.
13		*/
14		abstract class Entry {
15
16		/**
17		* @var string The identifier of the backend used for this entry.
18		* @api
19		* @since 2.0
20		*/
21		public static $backend = null;
22
23		/**
24		* @var int The ID for this entry.
25		*
26		* @api
27		* @since 2.0
28		*/
29		public $ID = null;
30
31		/**
32		* @var mixed The backing entry.
33		*/
34		protected $entry;
35
36		/**
37		* Adds the necessary rewrites for single Entries.
38		*
39		* @internal
40		* @return void
41		*/
42	1	public static function add_rewrite_endpoint() {
43	1	global $wp_rewrite;
		0 ignored issues – show Compatibility Best Practice introduced 2017-01-22 20:12 UTC by Report Bug Copy Issue Report Use of `global` functionality is not recommended; it makes your code harder to test, and less reusable. Instead of relying on `global` state, we recommend one of these alternatives: 1. Pass all data via parameters function myFunction($a, $b) { // Do something } 2. Create a class that maintains your state class MyClass { private $a; private $b; public function __construct($a, $b) { $this->a = $a; $this->b = $b; } public function myFunction() { // Do something } } Loading history...
44
45	1	$endpoint = self::get_endpoint_name();
46
47		/** Let's make sure the endpoint array is not polluted. */
48	1	if ( in_array( array( EP_ALL, $endpoint, $endpoint ), $wp_rewrite->endpoints ) ) {
49	1	return;
50		}
51
52		add_rewrite_endpoint( $endpoint, EP_ALL );
53		}
54
55		/**
56		* Return the endpoint name for a single Entry.
57		*
58		* Also used as the query_var for the time being.
59		*
60		* @internal
61		* @return string The name. Default: "entry"
62		*/
63	25	public static function get_endpoint_name() {
64		/**
65		* @filter `gravityview_directory_endpoint` Change the slug used for single entries
66		* @param[in,out] string $endpoint Slug to use when accessing single entry. Default: `entry`
67		*/
68	25	$endpoint = apply_filters( 'gravityview_directory_endpoint', 'entry' );
69
70	25	return sanitize_title( $endpoint );
71		}
72
73		/**
74		* Construct a \GV\Entry instance by ID.
75		*
76		* @param int\|string $entry_id The internal entry ID.
77		*
78		* @api
79		* @since 2.0
80		* @return \GV\Entry\|null An instance of this entry or null if not found.
81		*/
82		public static function by_id( $entry_id ) {
83		return null;
84		}
85
86		/**
87		* Return the backing entry object.
88		*
89		* @return array The backing entry object.
90		*/
91	61	public function as_entry() {
92	61	return $this->entry;
93		}
94
95		/**
96		* Return the link to this entry in the supplied context.
97		*
98		* @api
99		* @since 2.0
100		*
101		* @param \GV\View\|null $view The View context.
102		* @param \GV\Request $request The Request (current if null).
103		* @param boolean $track_directory Keep the housing directory arguments intact (used for breadcrumbs, for example). Default: true.
104		*
105		* @return string The permalink to this entry.
106		*/
107	7	public function get_permalink( \GV\View $view = null, \GV\Request $request = null, $track_directory = true ) {
108	7	if ( is_null( $request ) ) {
109	1	$request = &gravityview()->request;
110		}
111
112	7	global $post;
		0 ignored issues – show Compatibility Best Practice introduced 2018-05-08 22:40 UTC by Report Bug Copy Issue Report Use of `global` functionality is not recommended; it makes your code harder to test, and less reusable. Instead of relying on `global` state, we recommend one of these alternatives: 1. Pass all data via parameters function myFunction($a, $b) { // Do something } 2. Create a class that maintains your state class MyClass { private $a; private $b; public function __construct($a, $b) { $this->a = $a; $this->b = $b; } public function myFunction() { // Do something } } Loading history...
113
114	7	$args = array();
115
116	7	$view_id = is_null ( $view ) ? null : $view->ID;
		0 ignored issues – show Documentation introduced 2018-05-21 22:31 UTC by Report Bug Copy Issue Report The property `ID` does not exist on `object<GV\View>`. Since you implemented `__get`, maybe consider adding a @property annotation. Since your code implements the magic getter `_get`, this function will be called for any read access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } If the property has read access only, you can use the @property-read annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history... Coding Style introduced 2018-05-21 22:31 UTC by Report Bug Copy Issue Report Space before opening parenthesis of function call prohibited Loading history...
117
118	7	$permalink = null;
119
120		/** This is not a regular view. */
121	7	if ( ! $request->is_view() ) {
122
123		/** Must be an embed of some sort. */
124	5	if ( is_object( $post ) && is_numeric( $post->ID ) ) {
125	1	$permalink = get_permalink( $post->ID );
126	1	$args['gvid'] = $view_id;
127		}
128		}
129
130		/** Fallback to regular view base. */
131	7	if ( is_null( $permalink ) ) {
132	7	$permalink = get_permalink( $view_id );
133		}
134
135		/**
136		* @filter `gravityview_directory_link` Modify the URL to the View "directory" context
137		* @since 1.19.4
138		* @param string $link URL to the View's "directory" context (Multiple Entries screen)
139		* @param int $post_id ID of the post to link to. If the View is embedded, it is the post or page ID
140		*/
141	7	$permalink = apply_filters( 'gravityview_directory_link', $permalink, $request->is_view() ? $view_id : ( $post ? $post->ID : null ) );
142
143	7	$entry_endpoint_name = \GV\Entry::get_endpoint_name();
144	7	$entry_slug = \GravityView_API::get_entry_slug( $this->ID, $this->as_entry() );
145
146		/** Assemble the permalink. */
147	7	if ( get_option( 'permalink_structure' ) && ! is_preview() ) {
148		/**
149		* Make sure the $directory_link doesn't contain any query otherwise it will break when adding the entry slug.
150		* @since 1.16.5
151		*/
152	1	$link_parts = explode( '?', $permalink );
153
154	1	$query = ! empty( $link_parts[1] ) ? '?' . $link_parts[1] : '';
155
156	1	$permalink = trailingslashit( $link_parts[0] ) . $entry_endpoint_name . '/'. $entry_slug .'/' . $query;
157		} else {
158	7	$args[ $entry_endpoint_name ] = $entry_slug;
159		}
160
161	7	if ( $track_directory ) {
162	7	if ( ! empty( $_GET['pagenum'] ) ) {
163	1	$args['pagenum'] = intval( $_GET['pagenum'] );
		0 ignored issues – show introduced 2018-05-08 22:40 UTC by Report Bug Copy Issue Report Detected access of super global var $_GET, probably need manual inspection. Loading history...
164		}
165
166	7	if ( $sort = Utils::_GET( 'sort' ) ) {
167	1	$args['sort'] = $sort;
168	1	$args['dir'] = Utils::_GET( 'dir' );
169		}
170		}
171
172	7	$permalink = add_query_arg( $args, $permalink );
173
174		/**
175		* @filter `gravityview/entry/permalink` The permalink of this entry.
176		* @since 2.0
177		* @param string $permalink The permalink.
178		* @param \GV\Entry $entry The entry we're retrieving it for.
179		* @param \GV\View\|null $view The view context.
180		* @param \GV\Request $reqeust The request context.
181		*/
182	7	return apply_filters( 'gravityview/entry/permalink', $permalink, $this, $view, $request );
183		}
184		}
185

gravityview / GravityView

Push — master ( 397f5b...27d97b )

Entry::get_permalink() F

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Long Method Complexity

Long Method

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

Duplication Side-by-Side

Filter issues like