jon48 / webtrees-lib

app/Module/Hooks/Hooks/SosaFamilyDatatablesExtenderCollector.php

Indentation +37 added lines, -37 removed lines

@@ -25,45 +25,45 @@
  * @extends AbstractHookCollector<SosaFamilyDatatablesExtenderInterface>
  */
 class SosaFamilyDatatablesExtenderCollector extends AbstractHookCollector implements
-    SosaFamilyDatatablesExtenderInterface
+	SosaFamilyDatatablesExtenderInterface
 {
-    /**
-     * {@inheritDoc}
-     * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::title()
-     */
-    public function title(): string
-    {
-        return I18N::translate('Columns extender for tables of ancestors’ families');
-    }
+	/**
+	 * {@inheritDoc}
+	 * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::title()
+	 */
+	public function title(): string
+	{
+		return I18N::translate('Columns extender for tables of ancestors’ families');
+	}
 
-    /**
-     * {@inheritDoc}
-     * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::description()
-     */
-    public function description(): string
-    {
-        return I18N::translate('Add additional columns to tables of ancestors’ families');
-    }
+	/**
+	 * {@inheritDoc}
+	 * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::description()
+	 */
+	public function description(): string
+	{
+		return I18N::translate('Add additional columns to tables of ancestors’ families');
+	}
 
-    /**
-     * {@inheritDoc}
-     * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::hookInterface()
-     */
-    public function hookInterface(): string
-    {
-        return SosaFamilyDatatablesExtenderInterface::class;
-    }
+	/**
+	 * {@inheritDoc}
+	 * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::hookInterface()
+	 */
+	public function hookInterface(): string
+	{
+		return SosaFamilyDatatablesExtenderInterface::class;
+	}
 
-    /**
-     * {@inheritDoc}
-     * @see \MyArtJaub\Webtrees\Contracts\Hooks\SosaFamilyDatatablesExtenderInterface::sosaFamilyColumns()
-     */
-    public function sosaFamilyColumns(iterable $records): array
-    {
-        $result = [];
-        foreach ($this->hooks() as $hook) {
-            $result += $hook->sosaFamilyColumns($records);
-        }
-        return $result;
-    }
+	/**
+	 * {@inheritDoc}
+	 * @see \MyArtJaub\Webtrees\Contracts\Hooks\SosaFamilyDatatablesExtenderInterface::sosaFamilyColumns()
+	 */
+	public function sosaFamilyColumns(iterable $records): array
+	{
+		$result = [];
+		foreach ($this->hooks() as $hook) {
+			$result += $hook->sosaFamilyColumns($records);
+		}
+		return $result;
+	}
 }

app/Module/Hooks/Hooks/SosaIndividualDatatablesExtenderCollector.php

Indentation +37 added lines, -37 removed lines

@@ -25,45 +25,45 @@
  * @extends AbstractHookCollector<SosaIndividualDatatablesExtenderInterface>
  */
 class SosaIndividualDatatablesExtenderCollector extends AbstractHookCollector implements
-    SosaIndividualDatatablesExtenderInterface
+	SosaIndividualDatatablesExtenderInterface
 {
-    /**
-     * {@inheritDoc}
-     * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::title()
-     */
-    public function title(): string
-    {
-        return I18N::translate('Columns extender for tables of ancestors');
-    }
+	/**
+	 * {@inheritDoc}
+	 * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::title()
+	 */
+	public function title(): string
+	{
+		return I18N::translate('Columns extender for tables of ancestors');
+	}
 
-    /**
-     * {@inheritDoc}
-     * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::description()
-     */
-    public function description(): string
-    {
-        return I18N::translate('Add additional columns to tables of ancestors');
-    }
+	/**
+	 * {@inheritDoc}
+	 * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::description()
+	 */
+	public function description(): string
+	{
+		return I18N::translate('Add additional columns to tables of ancestors');
+	}
 
-    /**
-     * {@inheritDoc}
-     * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::hookInterface()
-     */
-    public function hookInterface(): string
-    {
-        return SosaIndividualDatatablesExtenderInterface::class;
-    }
+	/**
+	 * {@inheritDoc}
+	 * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::hookInterface()
+	 */
+	public function hookInterface(): string
+	{
+		return SosaIndividualDatatablesExtenderInterface::class;
+	}
 
-    /**
-     * {@inheritDoc}
-     * @see \MyArtJaub\Webtrees\Contracts\Hooks\SosaIndividualDatatablesExtenderInterface::sosaIndividualColumns()
-     */
-    public function sosaIndividualColumns(iterable $records): array
-    {
-        $result = [];
-        foreach ($this->hooks() as $hook) {
-            $result += $hook->sosaIndividualColumns($records);
-        }
-        return $result;
-    }
+	/**
+	 * {@inheritDoc}
+	 * @see \MyArtJaub\Webtrees\Contracts\Hooks\SosaIndividualDatatablesExtenderInterface::sosaIndividualColumns()
+	 */
+	public function sosaIndividualColumns(iterable $records): array
+	{
+		$result = [];
+		foreach ($this->hooks() as $hook) {
+			$result += $hook->sosaIndividualColumns($records);
+		}
+		return $result;
+	}
 }

app/Module/Hooks/Hooks/FactSourceTextExtenderCollector.php

Indentation +48 added lines, -48 removed lines

@@ -27,56 +27,56 @@
  */
 class FactSourceTextExtenderCollector extends AbstractHookCollector implements FactSourceTextExtenderInterface
 {
-    /**
-     * {@inheritDoc}
-     * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::title()
-     */
-    public function title(): string
-    {
-        return I18N::translate('Text extender for source citations’ title');
-    }
+	/**
+	 * {@inheritDoc}
+	 * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::title()
+	 */
+	public function title(): string
+	{
+		return I18N::translate('Text extender for source citations’ title');
+	}
 
-    /**
-     * {@inheritDoc}
-     * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::description()
-     */
-    public function description(): string
-    {
-        return I18N::translate('Extends the title of source citations with additional text or icons.');
-    }
+	/**
+	 * {@inheritDoc}
+	 * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::description()
+	 */
+	public function description(): string
+	{
+		return I18N::translate('Extends the title of source citations with additional text or icons.');
+	}
 
-    /**
-     * {@inheritDoc}
-     * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::hookInterface()
-     */
-    public function hookInterface(): string
-    {
-        return FactSourceTextExtenderInterface::class;
-    }
+	/**
+	 * {@inheritDoc}
+	 * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::hookInterface()
+	 */
+	public function hookInterface(): string
+	{
+		return FactSourceTextExtenderInterface::class;
+	}
 
-    /**
-     * {@inheritDoc}
-     * @see \MyArtJaub\Webtrees\Contracts\Hooks\FactSourceTextExtenderInterface::factSourcePrepend()
-     */
-    public function factSourcePrepend(Tree $tree, $fact): string
-    {
-        return $this->hooks()
-            ->map(
-                fn(FactSourceTextExtenderInterface $hook) =>
-                    $hook->factSourcePrepend($tree, $fact)
-            )->implode('');
-    }
+	/**
+	 * {@inheritDoc}
+	 * @see \MyArtJaub\Webtrees\Contracts\Hooks\FactSourceTextExtenderInterface::factSourcePrepend()
+	 */
+	public function factSourcePrepend(Tree $tree, $fact): string
+	{
+		return $this->hooks()
+			->map(
+				fn(FactSourceTextExtenderInterface $hook) =>
+					$hook->factSourcePrepend($tree, $fact)
+			)->implode('');
+	}
 
-    /**
-     * {@inheritDoc}
-     * @see \MyArtJaub\Webtrees\Contracts\Hooks\FactSourceTextExtenderInterface::factSourceAppend()
-     */
-    public function factSourceAppend(Tree $tree, $fact): string
-    {
-        return $this->hooks()
-            ->map(
-                fn(FactSourceTextExtenderInterface $hook) =>
-                    $hook->factSourcePrepend($tree, $fact)
-            )->implode('');
-    }
+	/**
+	 * {@inheritDoc}
+	 * @see \MyArtJaub\Webtrees\Contracts\Hooks\FactSourceTextExtenderInterface::factSourceAppend()
+	 */
+	public function factSourceAppend(Tree $tree, $fact): string
+	{
+		return $this->hooks()
+			->map(
+				fn(FactSourceTextExtenderInterface $hook) =>
+					$hook->factSourcePrepend($tree, $fact)
+			)->implode('');
+	}
 }

app/Module/Hooks/Hooks/FamilyDatatablesExtenderCollector.php

Indentation +37 added lines, -37 removed lines

@@ -25,45 +25,45 @@
  * @extends AbstractHookCollector<FamilyDatatablesExtenderInterface>
  */
 class FamilyDatatablesExtenderCollector extends AbstractHookCollector implements
-    FamilyDatatablesExtenderInterface
+	FamilyDatatablesExtenderInterface
 {
-    /**
-     * {@inheritDoc}
-     * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::title()
-     */
-    public function title(): string
-    {
-        return I18N::translate('Columns extender for tables of families');
-    }
+	/**
+	 * {@inheritDoc}
+	 * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::title()
+	 */
+	public function title(): string
+	{
+		return I18N::translate('Columns extender for tables of families');
+	}
 
-    /**
-     * {@inheritDoc}
-     * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::description()
-     */
-    public function description(): string
-    {
-        return I18N::translate('Add additional columns to tables of families');
-    }
+	/**
+	 * {@inheritDoc}
+	 * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::description()
+	 */
+	public function description(): string
+	{
+		return I18N::translate('Add additional columns to tables of families');
+	}
 
-    /**
-     * {@inheritDoc}
-     * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::hookInterface()
-     */
-    public function hookInterface(): string
-    {
-        return FamilyDatatablesExtenderInterface::class;
-    }
+	/**
+	 * {@inheritDoc}
+	 * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::hookInterface()
+	 */
+	public function hookInterface(): string
+	{
+		return FamilyDatatablesExtenderInterface::class;
+	}
 
-    /**
-     * {@inheritDoc}
-     * @see \MyArtJaub\Webtrees\Contracts\Hooks\FamilyDatatablesExtenderInterface::familyColumns()
-     */
-    public function familyColumns(iterable $records): array
-    {
-        $result = [];
-        foreach ($this->hooks() as $hook) {
-            $result += $hook->familyColumns($records);
-        }
-        return $result;
-    }
+	/**
+	 * {@inheritDoc}
+	 * @see \MyArtJaub\Webtrees\Contracts\Hooks\FamilyDatatablesExtenderInterface::familyColumns()
+	 */
+	public function familyColumns(iterable $records): array
+	{
+		$result = [];
+		foreach ($this->hooks() as $hook) {
+			$result += $hook->familyColumns($records);
+		}
+		return $result;
+	}
 }

app/Module/Hooks/Hooks/IndividualDatatablesExtenderCollector.php

Indentation +37 added lines, -37 removed lines

@@ -25,45 +25,45 @@
  * @extends AbstractHookCollector<IndividualDatatablesExtenderInterface>
  */
 class IndividualDatatablesExtenderCollector extends AbstractHookCollector implements
-    IndividualDatatablesExtenderInterface
+	IndividualDatatablesExtenderInterface
 {
-    /**
-     * {@inheritDoc}
-     * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::title()
-     */
-    public function title(): string
-    {
-        return I18N::translate('Columns extender for tables of individuals');
-    }
+	/**
+	 * {@inheritDoc}
+	 * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::title()
+	 */
+	public function title(): string
+	{
+		return I18N::translate('Columns extender for tables of individuals');
+	}
 
-    /**
-     * {@inheritDoc}
-     * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::description()
-     */
-    public function description(): string
-    {
-        return I18N::translate('Add additional columns to tables of individuals');
-    }
+	/**
+	 * {@inheritDoc}
+	 * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::description()
+	 */
+	public function description(): string
+	{
+		return I18N::translate('Add additional columns to tables of individuals');
+	}
 
-    /**
-     * {@inheritDoc}
-     * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::hookInterface()
-     */
-    public function hookInterface(): string
-    {
-        return IndividualDatatablesExtenderInterface::class;
-    }
+	/**
+	 * {@inheritDoc}
+	 * @see \MyArtJaub\Webtrees\Common\Hooks\AbstractHookCollector::hookInterface()
+	 */
+	public function hookInterface(): string
+	{
+		return IndividualDatatablesExtenderInterface::class;
+	}
 
-    /**
-     * {@inheritDoc}
-     * @see \MyArtJaub\Webtrees\Contracts\Hooks\IndividualDatatablesExtenderInterface::individualColumns()
-     */
-    public function individualColumns(iterable $records): array
-    {
-        $result = [];
-        foreach ($this->hooks() as $hook) {
-            $result += $hook->individualColumns($records);
-        }
-        return $result;
-    }
+	/**
+	 * {@inheritDoc}
+	 * @see \MyArtJaub\Webtrees\Contracts\Hooks\IndividualDatatablesExtenderInterface::individualColumns()
+	 */
+	public function individualColumns(iterable $records): array
+	{
+		$result = [];
+		foreach ($this->hooks() as $hook) {
+			$result += $hook->individualColumns($records);
+		}
+		return $result;
+	}
 }

app/Module/Hooks/Schema/Migration0.php

Indentation +8 added lines, -8 removed lines

@@ -21,13 +21,13 @@
  */
 class Migration0 implements MigrationInterface
 {
-    /**
-     * {@inheritDoc}
-     * @see \Fisharebest\Webtrees\Schema\MigrationInterface::upgrade()
-     */
-    public function upgrade(): void
-    {
+	/**
+	 * {@inheritDoc}
+	 * @see \Fisharebest\Webtrees\Schema\MigrationInterface::upgrade()
+	 */
+	public function upgrade(): void
+	{
 
-        // This migration has been superseded by migration 1.
-    }
+		// This migration has been superseded by migration 1.
+	}
 }

app/Module/Hooks/Schema/Migration1.php

Indentation +24 added lines, -24 removed lines

@@ -23,28 +23,28 @@
  */
 class Migration1 implements MigrationInterface
 {
-    /**
-     * {@inheritDoc}
-     * @see \Fisharebest\Webtrees\Schema\MigrationInterface::upgrade()
-     */
-    public function upgrade(): void
-    {
-        $in_transaction = DB::connection()->getPdo()->inTransaction();
-
-        if (DB::schema()->hasTable('maj_hooks')) {
-            DB::schema()->drop('maj_hooks');
-        }
-
-        DB::schema()->create('maj_hook_order', static function (Blueprint $table): void {
-            $table->string('majho_module_name', 32);
-            $table->string('majho_hook_name', 64);
-            $table->integer('majho_hook_order')->nullable();
-
-            $table->primary(['majho_module_name', 'majho_hook_name']);
-        });
-
-        if ($in_transaction && !DB::connection()->getPdo()->inTransaction()) {
-            DB::connection()->beginTransaction();
-        }
-    }
+	/**
+	 * {@inheritDoc}
+	 * @see \Fisharebest\Webtrees\Schema\MigrationInterface::upgrade()
+	 */
+	public function upgrade(): void
+	{
+		$in_transaction = DB::connection()->getPdo()->inTransaction();
+
+		if (DB::schema()->hasTable('maj_hooks')) {
+			DB::schema()->drop('maj_hooks');
+		}
+
+		DB::schema()->create('maj_hook_order', static function (Blueprint $table): void {
+			$table->string('majho_module_name', 32);
+			$table->string('majho_hook_name', 64);
+			$table->integer('majho_hook_order')->nullable();
+
+			$table->primary(['majho_module_name', 'majho_hook_name']);
+		});
+
+		if ($in_transaction && !DB::connection()->getPdo()->inTransaction()) {
+			DB::connection()->beginTransaction();
+		}
+	}
 }

app/Module/Hooks/Http/RequestHandlers/ModulesHooksPage.php

Indentation +34 added lines, -34 removed lines

@@ -29,44 +29,44 @@
  */
 class ModulesHooksPage implements RequestHandlerInterface
 {
-    use ViewResponseTrait;
+	use ViewResponseTrait;
 
-    protected HookService $hook_service;
+	protected HookService $hook_service;
 
-    /**
-     * Constructor for ModulesHooksPage request handler
-     *
-     * @param HookService $hook_service
-     */
-    public function __construct(HookService $hook_service)
-    {
-        $this->hook_service = $hook_service;
-    }
+	/**
+	 * Constructor for ModulesHooksPage request handler
+	 *
+	 * @param HookService $hook_service
+	 */
+	public function __construct(HookService $hook_service)
+	{
+		$this->hook_service = $hook_service;
+	}
 
-    /**
-     * {@inheritDoc}
-     * @see \Psr\Http\Server\RequestHandlerInterface::handle()
-     */
-    public function handle(ServerRequestInterface $request): ResponseInterface
-    {
-        $this->layout = 'layouts/administration';
+	/**
+	 * {@inheritDoc}
+	 * @see \Psr\Http\Server\RequestHandlerInterface::handle()
+	 */
+	public function handle(ServerRequestInterface $request): ResponseInterface
+	{
+		$this->layout = 'layouts/administration';
 
-        $hook_name = Validator::attributes($request)->string('hook_name', '');
-        $hook = $this->hook_service->find($hook_name, true);
-        if ($hook === null) {
-            throw new HttpNotFoundException(I18N::translate('The hook with name “%s” does not exist.', $hook_name));
-        }
+		$hook_name = Validator::attributes($request)->string('hook_name', '');
+		$hook = $this->hook_service->find($hook_name, true);
+		if ($hook === null) {
+			throw new HttpNotFoundException(I18N::translate('The hook with name “%s” does not exist.', $hook_name));
+		}
 
-        $modules = $hook->hooks()
-            ->sortKeys()
-            ->mapWithKeys(fn(HookInterface $hook) => [$hook->module()->name() => $hook->module()]);
+		$modules = $hook->hooks()
+			->sortKeys()
+			->mapWithKeys(fn(HookInterface $hook) => [$hook->module()->name() => $hook->module()]);
 
-        return $this->viewResponse('admin/components', [
-            'description'    => $hook->description(),
-            'modules'        => $modules,
-            'title'          => $hook->title(),
-            'uses_access'    => false,
-            'uses_sorting'   => true
-        ]);
-    }
+		return $this->viewResponse('admin/components', [
+			'description'    => $hook->description(),
+			'modules'        => $modules,
+			'title'          => $hook->title(),
+			'uses_access'    => false,
+			'uses_sorting'   => true
+		]);
+	}
 }

app/Module/Hooks/Services/HookService.php

Indentation +88 added lines, -88 removed lines

@@ -30,101 +30,101 @@
  */
 class HookService implements HookServiceInterface
 {
-    private ModuleService $module_service;
+	private ModuleService $module_service;
 
-    /**
-     * Constructor for HookService
-     *
-     * @param ModuleService $module_service
-     */
-    public function __construct(ModuleService $module_service)
-    {
-        $this->module_service = $module_service;
-    }
+	/**
+	 * Constructor for HookService
+	 *
+	 * @param ModuleService $module_service
+	 */
+	public function __construct(ModuleService $module_service)
+	{
+		$this->module_service = $module_service;
+	}
 
-    /**
-     * {@inheritDoc}
-     * @see \MyArtJaub\Webtrees\Contracts\Hooks\HookServiceInterface::use()
-     */
-    public function use(string $hook_interface): ?HookCollectorInterface
-    {
-        return $this->all()->get($hook_interface);
-    }
+	/**
+	 * {@inheritDoc}
+	 * @see \MyArtJaub\Webtrees\Contracts\Hooks\HookServiceInterface::use()
+	 */
+	public function use(string $hook_interface): ?HookCollectorInterface
+	{
+		return $this->all()->get($hook_interface);
+	}
 
 
-    /**
-     * Find a hook collector by its name, with or without the disabled ones.
-     *
-     * @param string $hook_name
-     * @return HookCollectorInterface|null
-     */
-    public function find(string $hook_name, bool $include_disabled = false): ?HookCollectorInterface
-    {
-        return $this->all($include_disabled)
-            ->first(fn(HookCollectorInterface $hook_collector) => $hook_collector->name() === $hook_name);
-    }
+	/**
+	 * Find a hook collector by its name, with or without the disabled ones.
+	 *
+	 * @param string $hook_name
+	 * @return HookCollectorInterface|null
+	 */
+	public function find(string $hook_name, bool $include_disabled = false): ?HookCollectorInterface
+	{
+		return $this->all($include_disabled)
+			->first(fn(HookCollectorInterface $hook_collector) => $hook_collector->name() === $hook_name);
+	}
 
-    /**
-     * Get all hook collectors subscribed by modules, with hooks ordered, with or without the disabled ones.
-     *
-     * @param bool $include_disabled
-     * @return Collection<string, HookCollectorInterface>
-     */
-    public function all(bool $include_disabled = false): Collection
-    {
-        return Registry::cache()->array()->remember('all-hooks', function () use ($include_disabled): Collection {
-            $hooks_info = DB::table('maj_hook_order')
-                ->get()
-                ->groupBy(['majho_hook_name', 'majho_module_name']);
+	/**
+	 * Get all hook collectors subscribed by modules, with hooks ordered, with or without the disabled ones.
+	 *
+	 * @param bool $include_disabled
+	 * @return Collection<string, HookCollectorInterface>
+	 */
+	public function all(bool $include_disabled = false): Collection
+	{
+		return Registry::cache()->array()->remember('all-hooks', function () use ($include_disabled): Collection {
+			$hooks_info = DB::table('maj_hook_order')
+				->get()
+				->groupBy(['majho_hook_name', 'majho_module_name']);
 
-            $hooks = $this->module_service
-                ->findByInterface(ModuleHookSubscriberInterface::class, $include_disabled)
-                ->flatMap(fn(ModuleHookSubscriberInterface $module) => $module->listSubscribedHooks());
+			$hooks = $this->module_service
+				->findByInterface(ModuleHookSubscriberInterface::class, $include_disabled)
+				->flatMap(fn(ModuleHookSubscriberInterface $module) => $module->listSubscribedHooks());
 
-            $hook_collectors = collect();
-            $hook_instances = collect();
-            foreach ($hooks as $hook) {
-                if (!($hook instanceof HookInterface)) {
-                    continue;
-                }
-                if ($hook instanceof HookCollectorInterface) {
-                    $hook_collectors->put($hook->hookInterface(), $hook);
-                } else {
-                    $hook_instances->add($hook);
-                }
-            }
+			$hook_collectors = collect();
+			$hook_instances = collect();
+			foreach ($hooks as $hook) {
+				if (!($hook instanceof HookInterface)) {
+					continue;
+				}
+				if ($hook instanceof HookCollectorInterface) {
+					$hook_collectors->put($hook->hookInterface(), $hook);
+				} else {
+					$hook_instances->add($hook);
+				}
+			}
 
-            foreach ($hook_collectors as $hook_interface => $hook_collector) {
-                $hook_info = $hooks_info->get($hook_collector->name()) ?? collect();
-                foreach (
-                    $hook_instances->filter(
-                        fn(HookInterface $hook): bool => $hook instanceof $hook_interface
-                    ) as $hook_instance
-                ) {
-                    $hook_module_info = $hook_info->get($hook_instance->module()->name(), collect())->first();
-                    $hook_order = $hook_module_info instanceof stdClass ? (int) $hook_module_info->majho_hook_order : 0;
-                    $hook_collector->register($hook_instance, $hook_order);
-                }
-            }
-            return $hook_collectors;
-        });
-    }
+			foreach ($hook_collectors as $hook_interface => $hook_collector) {
+				$hook_info = $hooks_info->get($hook_collector->name()) ?? collect();
+				foreach (
+					$hook_instances->filter(
+						fn(HookInterface $hook): bool => $hook instanceof $hook_interface
+					) as $hook_instance
+				) {
+					$hook_module_info = $hook_info->get($hook_instance->module()->name(), collect())->first();
+					$hook_order = $hook_module_info instanceof stdClass ? (int) $hook_module_info->majho_hook_order : 0;
+					$hook_collector->register($hook_instance, $hook_order);
+				}
+			}
+			return $hook_collectors;
+		});
+	}
 
-    /**
-     * Update the order of the modules implementing a hook in the database.
-     *
-     * @param HookCollectorInterface $hook_collector
-     * @param ModuleInterface $module
-     * @param int $order
-     * @return int
-     */
-    public function updateOrder(HookCollectorInterface $hook_collector, ModuleInterface $module, int $order): int
-    {
-        return DB::table('maj_hook_order')
-            ->upsert([
-                'majho_module_name' =>  $module->name(),
-                'majho_hook_name'   =>  $hook_collector->name(),
-                'majho_hook_order'  =>  $order
-            ], ['majho_module_name', 'majho_hook_name'], ['majho_hook_order']);
-    }
+	/**
+	 * Update the order of the modules implementing a hook in the database.
+	 *
+	 * @param HookCollectorInterface $hook_collector
+	 * @param ModuleInterface $module
+	 * @param int $order
+	 * @return int
+	 */
+	public function updateOrder(HookCollectorInterface $hook_collector, ModuleInterface $module, int $order): int
+	{
+		return DB::table('maj_hook_order')
+			->upsert([
+				'majho_module_name' =>  $module->name(),
+				'majho_hook_name'   =>  $hook_collector->name(),
+				'majho_hook_order'  =>  $order
+			], ['majho_module_name', 'majho_hook_name'], ['majho_hook_order']);
+	}
 }