blobdir::__construct() - Code Metrics - Inspection of "Make sure we have the necessary permissions, fixes..." - flack/openpsa - Measure and Improve Code Quality continuously with Scrutinizer

Passed
Push — master ( 425a8c...545c74 )

by Andreas
created 2025-10-21 12:59 UTC
blobdir::__construct() A

↳ Parent: blobdir
Complexity

Conditions	1
Paths	1
Size

Total Lines	4
Code Lines	2
Duplication

Lines	0
Ratio	0 %
Code Coverage

Tests	0
CRAP Score	2
Importance

Changes	1
Bugs	1	Features	0
Metric	Value
cc	1
eloc	2
c	1
b	1
f	0
nc	1
nop	1
dl	0
loc	4
ccs	0
cts	3
cp	0
crap	2
rs	10
<?php
/**
 * @package midcom.console
 * @author CONTENT CONTROL http://www.contentcontrol-berlin.de/
 * @copyright CONTENT CONTROL http://www.contentcontrol-berlin.de/
 * @license http://www.gnu.org/licenses/gpl.html GNU General Public License
 */

namespace midcom\console\command\cleanup;

use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;
use midcom_db_attachment;
use Symfony\Component\Console\Attribute\AsCommand;
use midgard\portable\storage\connection;
use Doctrine\ORM\AbstractQuery;
use midcom_services_auth;

/**
 * Cleanup the blobdir
 * Search for corrupted (0 bytes) files
 *
 * @package midcom.console
 */
#[AsCommand(
    name: 'midcom:cleanup:blobdir',
    description: 'Cleanup the blobdir',
    aliases: ['blobdircleanup']
)]
class blobdir extends Command
{
    private int $_file_counter = 0;

    private string $_dir = "";

    private bool $dry = false;

    private array $findings = [
        'corrupted' => [],
        'orphaned' => [],
        'orphaned_attachments' => []
    ];

    private midcom_services_auth $auth;

    public function __construct(midcom_services_auth $auth)
    {
        $this->auth = $auth;
        parent::__construct();
    }

    protected function configure()
    {
        $this->addOption('dry', 'd', InputOption::VALUE_NONE, 'If set, files and attachments will not be deleted');
    }

    public function check_dir(string $outerDir)
    {
        $outerDir = rtrim($outerDir, "/");
        $dirs = array_diff(scandir($outerDir), [".", ".."]);
        foreach ($dirs as $d) {
            if (is_dir($outerDir . "/" . $d)) {
                $this->check_dir($outerDir . "/" . $d);
            } else {
                // got something
                $file = $outerDir . "/" . $d;
                if (filesize($file) == 0) {
                    $this->findings['corrupted'][] = $file;
                } else {
                    $attachment = $this->get_attachment($file);
                    if (!$attachment) {
                        $this->findings['orphaned'][] = $file;
                    } elseif (!$this->get_attachment_parent($attachment)) {
                        $this->findings['orphaned_attachments'][] = $attachment;
                    }
                }
                $this->_file_counter++;
            }
        }
    }

    private function _determine_location(string $path) : string
    {
        return ltrim(str_replace($this->_dir, "", $path), "/");
    }

    private function get_attachment_parent(midcom_db_attachment $attachment) : bool
    {
        $type = connection::get_em()
            ->createQuery('SELECT r.typename from midgard_repligard r WHERE r.guid = ?1')
            ->setParameter(1, $attachment->parentguid)
            ->getOneOrNullResult(AbstractQuery::HYDRATE_SINGLE_SCALAR);

        if (!$type) {
            return false;
        }

        $dba_type = \midcom::get()->dbclassloader->get_midcom_class_name_for_mgdschema_object($type);

        $qb = \midcom::get()->dbfactory->new_query_builder($dba_type);
        $qb = \midcom::get()->dbfactory->new_query_builder(/** @scrutinizer ignore-type */ $dba_type);
        $qb->include_deleted();
        $qb->add_constraint('guid', '=', $attachment->parentguid);
        return $qb->count() > 0;
    }

    private function get_attachment(string $file) : ?midcom_db_attachment
    {
        $location = $this->_determine_location($file);
        // get attachments
        $qb = midcom_db_attachment::new_query_builder();
        $qb->add_constraint("location", "=", $location);
        try {
            $attachments = $qb->execute();
        } catch (\Exception) {
            return null;
        }
        if (empty($attachments)) {
            return null;
        }
        if (count($attachments) > 1) {
            throw new \midcom_error('Multiple attachments share location ' . $location);
        }

        return $attachments[0];
    }

    private function cleanup_corrupted(OutputInterface $output, array $files)
    {
        $i = 0;
        foreach ($files as $file) {
            $i++;
            // cleanup file
            $output->writeln($i . ") " . $file);
            $this->cleanup_file($output, $file);

            if ($attachment = $this->get_attachment($file)) {
                $this->purge_attachment($output, $attachment);
            }
        }
    }

    private function purge_attachment(OutputInterface $output, midcom_db_attachment $attachment)
    {
        if (!$this->dry) {
            $stat = $attachment->purge();
            if (!$stat || $output->getVerbosity() >= OutputInterface::VERBOSITY_VERBOSE) {
                $output->writeln(($stat) ? "<info>Purge OK</info>" : "<comment>Purge FAILED, reason: " . \midcom_connection::get_error_string() . "</comment>");
            }
        }
    }

    private function cleanup_file(OutputInterface $output, string $file)
    {
        if (!$this->dry) {
            $stat = unlink($file);
            if (!$stat || $output->getVerbosity() >= OutputInterface::VERBOSITY_VERBOSE) {
                $output->writeln(($stat) ? "<info>Cleanup OK</info>" : "<comment>Cleanup FAILED</comment>");
            }
        }
    }

    protected function execute(InputInterface $input, OutputInterface $output) : int
    {
        $dir = \midgard_connection::get_instance()->config->blobdir;
        if (!is_dir($dir)) {
            $output->writeln("<comment>Unable to detect blobdir</comment>");
            return Command::FAILURE;
        }
        $this->_dir = $dir;
        $this->dry = $input->getOption("dry");
        if ($this->dry) {
            $output->writeln("<comment>Running in dry mode!</comment>");
        }

        $output->writeln("Start scanning dir: <comment>" . $dir . "</comment>");
        if (!$this->auth->request_sudo('midcom.core')) {
            $output->writeln("<comment>Unable to get sudo</comment>");
            return Command::FAILURE;
        }

        $this->check_dir($dir);

        $output->writeln("Scanned <info>" . $this->_file_counter . "</info> files");
        $output->writeln("Found <info>" . count($this->findings['corrupted']) . "</info> corrupted files");
        $output->writeln("Found <info>" . count($this->findings['orphaned']) . "</info> orphaned files");
        $output->writeln("Found <info>" . count($this->findings['orphaned_attachments']) . "</info> orphaned attachments");

        $this->cleanup_corrupted($output, $this->findings['corrupted']);

        foreach ($this->findings['orphaned'] as $file) {
            $this->cleanup_file($output, $file);
        }

        foreach ($this->findings['orphaned_attachments'] as $attachment) {
            $this->purge_attachment($output, $attachment);
        }

        $this->auth->drop_sudo();
        $output->writeln("<comment>Done</comment>");
        return Command::SUCCESS;
    }
}

flack / openpsa

Push — master ( 425a8c...545c74 )

blobdir::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like
