eNTiDi\Autotoc\Hacks

Complexity

Total Complexity

2

Size/Duplication

Total Lines	31
Duplicated Lines	54.84 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

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 string $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');

The method hasMethod cannot be called on $instance (of type string).

        self::$extra_methods[get_class($instance)][strtolower($method)] = [
            'wrap'   => $wrap,
            'method' => $method,
        ];
    }

    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,
        ];
    }
}