Basis\Jobs\Module\Bootstrap::run()

Complexity

Conditions	5
Paths	12

Size

Total Lines	25
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	30

Importance

Changes	4
Bugs	0	Features	0

<?php
    public function run(Runner $runner, Service $service, Event $event)
    {
        $runner->dispatch('tarantool.migrate');

        $meta = $runner->dispatch('module.meta');
        foreach ($meta['jobs'] as $job) {
            $class = new ReflectionClass($runner->getJobClass($job));
            $params = [];
            foreach ($class->getProperties(ReflectionProperty::IS_PUBLIC) as $property) {
                $params[] = $property->getName();
            }
            $service->registerJob($job, $params);
        }

        foreach ($meta['routes'] as $route) {
            $service->registerRoute($route);
        }

        foreach ($event->getSubscription() as $event => $listeners) {

It seems like $event is not always an object, but can also be of type integer|string. Maybe add an additional type check?

            $service->subscribe($event);
        }

        $assets = $runner->dispatch('module.assets');
        $service->updateAssetsVersion($assets['hash']);
    }
}