SnifferReport\Controller\MainController::processSniff()

Complexity

Conditions	7
Paths	12

Size

Total Lines	36
Code Lines	24

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public static function processSniff(Request $request)
    {
        $file = $request->files->get('file');
        $git_url = $request->get('git_url');
        $standards = $request->get('standards');
        $options = $request->get('options');

        $valid_git_url = (is_null($git_url) || !Validator::isGitUrl($git_url));
        if ($valid_git_url && is_null($file)) {
            throw new MissingFileOrGitParameterException();
        }

        if (!is_null($file)) {
            $file_name = $file->getClientOriginalName();
            $file->move(FILES_DIRECTORY_ROOT, $file_name);
            $files = FilesHandler::handle($file_name, $file->getClientMimeType());
        } else {
            try {
                $files = GitHandler::handle($git_url);
            } catch (Exception $e) {
                throw new GitCloneException($e);
            }
        }

        if (!Validator::areAllStandardsValid($standards)) {
            throw new InvalidStandardsException();
        }

        $parsed_options = OptionsParser::parseOptions($options);
        $sniff_result = Sniffer::sniffFiles($files, $standards, $parsed_options);

It seems like $parsed_options defined by \SnifferReport\Service\OptionsParser::parseOptions($options) on line 61 can also be of type null; however, SnifferReport\Service\Sniffer::sniffFiles() does only seem to accept array, maybe add an additional type check?

        // Delete all files that were generated in this request.
        $fs = new Filesystem();
        $fs->remove(FILES_DIRECTORY_ROOT);

        return SniffParser::parseSniff($sniff_result);
    }
}