GeminiLabs\Pollux\Helper

Complexity

Total Complexity

12

Size/Duplication

Total Lines	87
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	7
Bugs	1	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	buildClassName()	0	7	2
A	startsWith()	0	3	1
A	buildMethodName()	0	3	1
A	endsWith()	0	6	2
A	getClassname()	0	7	2
A	toArray()	0	3	1
A	getCurrentScreen()	0	12	3

<?php

namespace GeminiLabs\Pollux;

class Helper
{
	/**
	 * @param string $name
	 * @param string $path
	 * @return string
	 */
	public static function buildClassName( $name, $path = '' )
	{
		$className = array_map( 'ucfirst', array_map( 'strtolower', preg_split( '/[-_]/', $name )));

It seems like preg_split('/[-_]/', $name) can also be of type false; however, parameter $arr1 of array_map() does only seem to accept array, maybe add an additional type check?

		$className = implode( '', $className );
		return !empty( $path )
			? str_replace( '\\\\', '\\', sprintf( '%s\%s', $path, $className ))
			: $className;
	}

	/**
	 * @param string $name
	 * @param string $prefix
	 * @return string
	 */
	public static function buildMethodName( $name, $prefix = 'get' )
	{
		return lcfirst( static::buildClassName( $prefix . '-' . $name ));
	}

	/**
	 * @param string $needle
	 * @param string $haystack
	 * @return bool
	 */
	public static function endsWith( $needle, $haystack )
	{
		$length = strlen( $needle );
		return $length != 0
			? substr( $haystack, -$length ) === $needle
			: true;
	}

	/**
	 * @param mixed $fromClass
	 * @return string
	 */
	public static function getClassname( $fromClass )
	{
		$className = is_string( $fromClass )
			? $fromClass
			: get_class( $fromClass );
		$paths = explode( '\\', $className );
		return end( $paths );
	}

	/**
	 * get_current_screen() is unreliable because it is not defined on all admin pages.
	 * @return \WP_Screen|\stdClass
	 */
	public static function getCurrentScreen()
	{
		global $hook_suffix, $pagenow;
		if( function_exists( 'get_current_screen' )) {
			$screen = get_current_screen();

Are you sure the assignment to $screen is correct as get_current_screen() seems to always return null.

		}
		if( empty( $screen )) {
			$screen = new \stdClass();
			$screen->base = $screen->id = $hook_suffix;
		}
		$screen->pagenow = $pagenow;
		return $screen;
	}

	/**
	 * @param string $needle
	 * @param string $haystack
	 * @return bool
	 */
	public static function startsWith( $needle, $haystack )
	{
		return substr( $haystack, 0, strlen( $needle )) === $needle;
	}

	/**
	 * @param mixed $value
	 * @return array
	 */
	public static function toArray( $value )
	{
		return array_filter( (array) $value );
	}
}