TimberFunctionWrapper

Complexity

Total Complexity

15

Size/Duplication

Total Lines	90
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__toString()	0	7	2
B	__construct()	0	16	6
A	add_to_twig()	0	9	1
A	call()	0	10	3
A	_parse_args()	0	10	3

<?php

class TimberFunctionWrapper {

	private $_class;
	private $_function;
	private $_args;
	private $_use_ob;

	public function __toString() {
		 try {

Line indented incorrectly; expected 2 tabs, found 2

			return (string)$this->call();

No space after closing casting parenthesis is prohibited

		 } catch (Exception $e) {

Line indented incorrectly; expected 3 tabs, found 2

		 	return 'Caught exception: ' . $e->getMessage() . "\n";

Line indented incorrectly; expected at least 4 tabs, found 3

		 }

Line indented incorrectly; expected 3 tabs, found 2

	}

	/**
	 *
	 *
	 * @param callable $function
	 * @param array   $args
	 * @param bool    $return_output_buffer
	 */
	public function __construct( $function, $args = array(), $return_output_buffer = false ) {
		if( is_array( $function ) ) {

Space after opening control structure is required

No space before opening parenthesis is prohibited

			if( (is_string( $function[0] ) && class_exists( $function[0] ) ) || gettype( $function[0] ) === 'object' ) {

Space after opening control structure is required

No space before opening parenthesis is prohibited

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

				$this->_class = $function[0];
			}
			
			if( is_string( $function[1] ) ) $this->_function = $function[1];

Space after opening control structure is required

It is generally a best practice to always use braces with control structures.

		} else {
			$this->_function = $function;
		}

		$this->_args = $args;
		$this->_use_ob = $return_output_buffer;

		add_filter( 'get_twig', array( &$this, 'add_to_twig' ) );
	}

	/**
	 *
	 *
	 * @param Twig_Environment $twig
	 * @return Twig_Environment
	 */
	public function add_to_twig( $twig ) {
		$wrapper = $this;

		$twig->addFunction( new Twig_SimpleFunction( $this->_function, function () use ( $wrapper ) {
					return call_user_func_array( array( $wrapper, 'call' ), func_get_args() );
				} ) );

Line indented incorrectly; expected 2 tabs, found 4

This line of the multi-line function call does not seem to be indented correctly. Expected 8 spaces, but found 16.

		return $twig;
	}

	/**
	 *
	 *
	 * @return string
	 */
	public function call() {
		$args = $this->_parse_args( func_get_args(), $this->_args );
		$callable = ( isset( $this->_class ) ) ? array( $this->_class, $this->_function ) : $this->_function;

		if ( $this->_use_ob ) {
			return TimberHelper::ob_function( $callable, $args );
		} else {
			return call_user_func_array( $callable, $args );
		}
	}

	/**
	 *
	 *
	 * @param array   $args
	 * @param array   $defaults
	 * @return array
	 */
	private function _parse_args( $args, $defaults ) {
		$_arg = reset( $defaults );

		foreach ( $args as $index => $arg ) {
			$defaults[$index] = is_null( $arg ) ? $_arg : $arg;

Array keys should be surrounded by spaces unless they contain a string or an integer.

			$_arg = next( $defaults );
		}

		return $defaults;
	}

}