hipanel\rbac\console\converter\PhpToNodeFilesConverter::convert()

Complexity

Conditions	3
Paths	9

Size

Total Lines	24
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

<?php
    public function convert()
    {
        $traverser = new NodeTraverser();
        $visitor = new Visitor();
        $traverser->addVisitor($visitor);

        try {
            $stmts = $this->parser->parse(file_get_contents($this->convertFile));
            $traverser->traverse($stmts);

It seems like $stmts can also be of type null; however, parameter $nodes of PhpParser\NodeTraverser::traverse() does only seem to accept array, maybe add an additional type check?

            $output = TsFileGenerator::fromParts(
                (new TsObjectGenerator('ROLES_AND_PERMISSIONS', require $this->convertFile))->generate(),
                (new TsEnumGenerator('ROLES_ENUM', $visitor->getRoles()))->generate(),
                (new TsEnumGenerator('PERMISSIONS_ENUM', $visitor->getPermissions()))->generate(),
            );

            if (!is_dir($this->storageDir)) {
                FileHelper::createDirectory($this->storageDir);
            }

            $targetFile = $this->storageDir . '/' . self::EXPORT_FILE_NAME;
            file_put_contents($targetFile, $output, LOCK_EX);
        } catch (Throwable $e) {
            echo 'Falied to parse file: ' . $e->getMessage();
        }