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

Complexity

Conditions	7
Paths	36

Size

Total Lines	33
Code Lines	20

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	56

Importance

Changes	4
Bugs	0	Features	0

<?php
    public function run(Runner $runner, Service $service, Event $event, Framework $framework, Filesystem $fs)
    {
        $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']);

        foreach ($framework->listFiles('resources/defaults') as $file) {
            if (!$fs->exists($file)) {
                $source = $framework->getPath("resources/defaults/$file");
                $destination = $fs->getPath($file);
                file_put_contents($destination, file_get_contents($source));
            }
        }
    }
}