| Conditions | 10 | 
| Paths | 4 | 
| Total Lines | 38 | 
| Code Lines | 31 | 
| Lines | 0 | 
| Ratio | 0 % | 
| Changes | 3 | ||
| Bugs | 0 | Features | 0 | 
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
| 1 | <?php  | 
            ||
| 18 | public static function eventCoreIncludeCommonAuthSuccess()  | 
            ||
| 19 |     { | 
            ||
| 20 | $autoloader = dirname(__DIR__) . '/vendor/autoload.php';  | 
            ||
| 21 |         if (!file_exists($autoloader)) { | 
            ||
| 22 |             trigger_error("xwhoops25/vendor/autoload.php not found, was 'composer install' done?"); | 
            ||
| 23 | return;  | 
            ||
| 24 | }  | 
            ||
| 25 |         xoops_loadLanguage('logger'); | 
            ||
| 26 | require_once $autoloader;  | 
            ||
| 27 |         $permissionHelper = new Permission('xwhoops25'); | 
            ||
| 28 |         if ($permissionHelper) { | 
            ||
| 29 | $permissionName = 'use_xwhoops';  | 
            ||
| 30 | $permissionItemId = 0;  | 
            ||
| 31 | |||
| 32 |             if ($permissionHelper->checkPermission($permissionName, $permissionItemId, false)) { | 
            ||
| 33 | $whoops = new \Whoops\Run;  | 
            ||
| 34 | $handler = new \Whoops\Handler\PrettyPageHandler;  | 
            ||
| 35 | $whoops->pushHandler($handler);  | 
            ||
| 36 | $whoops->register();  | 
            ||
| 37 | $handler->addDataTableCallback(  | 
            ||
| 38 | _LOGGER_QUERIES,  | 
            ||
| 39 |                     function () { | 
            ||
| 40 | $logger = XoopsLogger::getInstance();  | 
            ||
| 41 |                         if (false == $logger->renderingEnabled) { | 
            ||
| 42 | return ['XoopsLogger' => 'off']; // logger is off so data is incomplete  | 
            ||
| 43 | }  | 
            ||
| 44 | $queries = [];  | 
            ||
| 45 | $count=1;  | 
            ||
| 46 |                         foreach($logger->queries as $key => $q) { | 
            ||
| 47 | $error = (null===$q['errno'] ? '' : $q['errno'].' ') . (null===$q['error'] ? '' : $q['error']);  | 
            ||
| 48 |                             $queryTime = isset($q['query_time']) ? sprintf('%0.6f', $q['query_time']) : ''; | 
            ||
| 49 | $queryKey = (string) $count++ . ' - ' . $queryTime;  | 
            ||
| 50 |                             if (null !== $q['errno']) { | 
            ||
| 51 | $queryKey = (string) $count .' - Error' ;  | 
            ||
| 52 | }  | 
            ||
| 53 | $queries[$queryKey] = htmlentities($q['sql']) . ' ' . $error;  | 
            ||
| 54 | }  | 
            ||
| 55 | return ($queries);  | 
            ||
| 56 | }  | 
            ||
| 62 | 
The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g.
excluded_paths: ["lib/*"], you can move it to the dependency path list as follows:For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths