GeminiLabs\Pollux\MetaBox\Condition

Complexity

Total Complexity

16

Size/Duplication

Total Lines	134
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

12 Methods

Rating	Name	Duplication	Size	Complexity
A	validate()	0	10	2
A	normalizeCondition()	0	17	4
A	validateClassExists()	0	4	1
A	validateDefined()	0	4	1
A	validateFunctionExists()	0	4	1
A	validateHook()	0	4	1
A	validateIsFrontPage()	0	4	1
A	validateIsHome()	0	4	1
A	validateIsPageTemplate()	0	4	1
A	validateIsPluginActive()	0	4	1
A	validateIsPluginInactive()	0	4	1
A	validateUnknown()	0	4	1

<?php

namespace GeminiLabs\Pollux\MetaBox;

trait Condition
{
	protected static $conditions = [
		'class_exists', 'defined', 'function_exists', 'hook', 'is_front_page', 'is_home',
		'is_page_template', 'is_plugin_active', 'is_plugin_inactive',
	];

	/**
	 * @return bool
	 */
	public function validate( array $conditions )
	{
		array_walk( $conditions, function( &$value, $key ) {
			$method = $this->app->buildMethodName( $key, 'validate' );

The property app does not exist. Did you maybe forget to declare it?

			$value = method_exists( $this, $method )
				? $this->$method( $value )
				: $this->validateUnknown( $key, $value );
		});
		return !in_array( false, $conditions );
	}

	/**
	 * @param mixed $conditions
	 * @return array
	 */
	protected function normalizeCondition( $conditions )
	{
		$conditions = $this->toArray( $conditions );

It seems like toArray() must be provided by classes using this trait. How about adding it as abstract method to this trait?

		if( count( array_filter( array_keys( $conditions ), 'is_string' )) == 0 ) {
			foreach( $conditions as $key ) {
				$conditions[str_replace( '!', '', $key )] = substr( $key, 0, 1 ) == '!' ? 0 : 1;
			}
			$conditions = array_filter( $conditions, function( $key ) {
				return !is_numeric( $key );
			}, ARRAY_FILTER_USE_KEY );
		}
		$hook = sprintf( 'pollux/%s/conditions', $this->getClassname() );

It seems like getClassname() must be provided by classes using this trait. How about adding it as abstract method to this trait?

		return array_intersect_key(
			$conditions,
			array_flip( apply_filters( $hook, static::$conditions ))
		);
	}

	/**
	 * @param string $value
	 * @return bool
	 */
	protected function validateClassExists( $value )
	{
		return class_exists( $value );
	}

	/**
	 * @param string $value
	 * @return bool
	 */
	protected function validateDefined( $value )
	{
		return defined( $value );
	}

	/**
	 * @param string $value
	 * @return bool
	 */
	protected function validateFunctionExists( $value )
	{
		return function_exists( $value );
	}

	/**
	 * @param string $value
	 * @return bool
	 */
	protected function validateHook( $value )
	{
		return apply_filters( $value, true );
	}

	/**
	 * @param bool $value
	 * @return bool
	 */
	protected function validateIsFrontPage( $value )
	{
		return $value == ( $this->getPostId() == get_option( 'page_on_front' ));

It seems like getPostId() must be provided by classes using this trait. How about adding it as abstract method to this trait?

	}

	/**
	 * @param bool $value
	 * @return bool
	 */
	protected function validateIsHome( $value )
	{
		return $value == ( $this->getPostId() == get_option( 'page_for_posts' ));

It seems like getPostId() must be provided by classes using this trait. How about adding it as abstract method to this trait?

	}

	/**
	 * @param string $value
	 * @return bool
	 */
	protected function validateIsPageTemplate( $value )
	{
		return basename( get_page_template_slug( $this->getPostId() )) == $value;

It seems like getPostId() must be provided by classes using this trait. How about adding it as abstract method to this trait?

	}

	/**
	 * @param string $value
	 * @return bool
	 */
	protected function validateIsPluginActive( $value )
	{
		return is_plugin_active( $value );
	}

	/**
	 * @param string $value
	 * @return bool
	 */
	protected function validateIsPluginInactive( $value )
	{
		return is_plugin_inactive( $value );
	}

	/**
	 * @param string $key
	 * @param mixed $value
	 * @return bool
	 */
	protected function validateUnknown( $key, $value )
	{
		return apply_filters( 'pollux/metabox/condition', true, $key, $value );
	}
}