e107inc/e107

View on GitHub
e107_handlers/event_class.php

Summary

Maintainability
A
45 mins
Test Coverage
F
45%

Avoid deeply nested control flow statements.
Open

                            switch($data['method'])
                            {
                                //returns array('caption'=>'', 'text'=>'');
                                case 'form':
                                    if(method_exists($class, "event_{$data['method']}"))
Severity: Major
Found in e107_handlers/event_class.php - About 45 mins to fix

    Suspicious array access to null
    Open

                                                $text[$hook] = $ret;
    Severity: Minor
    Found in e107_handlers/event_class.php by phan

    Suspicious array access to null
    Open

                                                $text[$hook][0] = $ret;        
    Severity: Minor
    Found in e107_handlers/event_class.php by phan

    Returning type ?array|?array{0:}[]|?string but triggerHook() is declared to return array|string
    Open

            return $text;
    Severity: Minor
    Found in e107_handlers/event_class.php by phan

    Argument 2 (default) is false but \e107::getRegistry() takes null defined at /code/e107_handlers/e107_class.php:865
    Open

                $called = e107::getRegistry('core/cachedvars/admin_events_called', false);
    Severity: Minor
    Found in e107_handlers/event_class.php by phan

    There are no issues that match your filters.

    Category
    Status