LifecycleHooks::hookInto() - Code Metrics - Inspection of "[WIP] Next CRUD Version" - Laravel-Backpack/CRUD - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — main (#5715)

by Cristian

created 2025-02-07 08:40 UTC

LifecycleHooks::hookInto() A

↳ Parent: LifecycleHooks

Complexity

Conditions	3
Paths	4

Size

Total Lines	5
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	3
eloc	3
nc	4
nop	2
dl	0
loc	5
rs	10
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) {
                    if ($callback instanceof \Closure) {
                        $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			if ($callback instanceof \Closure) {
24			$callback(...$parameters);
25			}
26			}
27			}
28			}
29			}
30
31			public function has(string $hook): bool
32			{
33			return isset($this->hooks[$hook]);
34			}
35			}
36

Laravel-Backpack / CRUD

Scrutinizer GitHub App not installed

Pull Request — main (#5715)

LifecycleHooks::hookInto() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like