LifecycleHooks::trigger() - Code Metrics - Inspection of "[POC] Operation hooks" - Laravel-Backpack/CRUD - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Pull Request — main (#5687)

by Cristian

created 2024-10-25 14:20 UTC

LifecycleHooks::trigger() A

↳ Parent: LifecycleHooks

Complexity

Conditions	5
Paths	8

Size

Total Lines	7
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	5
eloc	5
nc	8
nop	2
dl	0
loc	7
rs	9.6111
c	1
b	0
f	0

<?php

namespace Backpack\CRUD\app\Library\CrudPanel\Hooks;

final class LifecycleHooks
{
    public array $hooks = [];

    public function hookInto(string|array $hooks, callable $callback): void
    {
        $hooks = is_array($hooks) ? $hooks : [$hooks];

        foreach ($hooks as $hook) {
            $this->hooks[$hook][] = $callback;
        }
    }

    public function trigger(string|array $hooks, array $parameters): void
    {
        $hooks = is_array($hooks) ? $hooks : [$hooks];

        foreach ($hooks as $hook) {
            if (isset($this->hooks[$hook])) {
                foreach ($this->hooks[$hook] as $callback) {
                    $callback(...$parameters);
                }
            }
        }
    }

    public function has(string $hook): bool
    {
        return isset($this->hooks[$hook]);
    }
}


1			<?php
2
3			namespace Backpack\CRUD\app\Library\CrudPanel\Hooks;
4
5			final class LifecycleHooks
6			{
7			public array $hooks = [];
8
9			public function hookInto(string\|array $hooks, callable $callback): void
10			{
11			$hooks = is_array($hooks) ? $hooks : [$hooks];
			0 ignored issues – show introduced 2024-10-25 14:32 UTC by Report Bug Copy Issue Report The condition `is_array($hooks)` is always `true`. Loading history...
12			foreach ($hooks as $hook) {
13			$this->hooks[$hook][] = $callback;
14			}
15			}
16
17			public function trigger(string\|array $hooks, array $parameters): void
18			{
19			$hooks = is_array($hooks) ? $hooks : [$hooks];
			0 ignored issues – show introduced 2024-10-25 14:32 UTC by Report Bug Copy Issue Report The condition `is_array($hooks)` is always `true`. Loading history...
20			foreach ($hooks as $hook) {
21			if (isset($this->hooks[$hook])) {
22			foreach ($this->hooks[$hook] as $callback) {
23			$callback(...$parameters);
24			}
25			}
26			}
27			}
28
29			public function has(string $hook): bool
30			{
31			return isset($this->hooks[$hook]);
32			}
33			}
34

Laravel-Backpack / CRUD

Scrutinizer GitHub App not installed

Pull Request — main (#5687)

LifecycleHooks::trigger() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like