eNTiDi\Autotoc\Hacks

Complexity

Total Complexity

2

Size/Duplication

Total Lines	41
Duplicated Lines	41.46 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

45.45%

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	addWrapperMethodToInstance()	9	9	1
A	addCallbackMethodToInstance()	8	8	1

<?php

namespace eNTiDi\Autotoc;

use SilverStripe\View\ViewableData;

class Hacks extends ViewableData
{
    /**
     * Add a new wrapper method.
     *
     * Similar to addWrapperMethod() but made public and working on
     * custom instances to allow to inject custom wrappers.
     *
     * @param ViewableData $instance
     * @param string $method
     * @param string $wrap
     */
    public static function addWrapperMethodToInstance($instance, $method, $wrap)

This method seems to be duplicated in your project.

    {
        // hasMethod() trigger the population of $extra_methods
        $instance->hasMethod('UnexistentMethod');
        self::$extra_methods[get_class($instance)][strtolower($method)] = [
            'wrap'   => $wrap,
            'method' => $method,
        ];
    }

    /**
     * Add callback as a method.
     *
     * Similar to addCallbackMethod() but made public and working on
     * custom instances to allow to inject custom callbacks.
     *
     * @param ViewableData $instance
     * @param string $method
     * @param \Closure $callback
     */
    public static function addCallbackMethodToInstance($instance, $method, $callback)

This method seems to be duplicated in your project.

    {
        // hasMethod() trigger the population of $extra_methods
        $instance->hasMethod('UnexistentMethod');
        self::$extra_methods[get_class($instance)][strtolower($method)] = [
            'callback' => $callback,
        ];
    }
}