Chamilo\CoreBundle\Controller\CreateResourceNodeFileAction::__invoke()

Complexity

Conditions	10
Paths	30

Size

Total Lines	60
Code Lines	38

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function __invoke(Request $request): CDocument
    {
        $document = new CDocument();
        $title = $request->get('title');

        if ('file' === $request->get('filetype')) {
            $fileParsed = false;
            // File upload
            if ($request->files->count() > 0) {
                /** @var UploadedFile $uploadedFile */
                $uploadedFile = $request->files->get('uploadFile');
                if (!$uploadedFile) {

$uploadedFile is of type Symfony\Component\HttpFoundation\File\UploadedFile, thus it always evaluated to true.

                    throw new BadRequestHttpException('"uploadFile" is required');
                }
                $title = $uploadedFile->getClientOriginalName();
                $document->setUploadFile($uploadedFile);
                $fileParsed = true;
            }

            // Get data in content and create a HTML file
            if (false === $fileParsed && $request->request->has('content')) {
                $content = $request->request->get('content');
                $title .= '.html';
                $handle = tmpfile();
                fwrite($handle, $content);

It seems like $handle can also be of type false; however, parameter $handle of fwrite() does only seem to accept resource, maybe add an additional type check?

                $meta = stream_get_meta_data($handle);

It seems like $handle can also be of type false; however, parameter $stream of stream_get_meta_data() does only seem to accept resource, maybe add an additional type check?

                $file = new UploadedFile($meta['uri'], $title, 'text/html', null, true);
                $document->setUploadFile($file);
                $fileParsed = true;
            }

            if (false === $fileParsed) {
                throw new \InvalidArgumentException(
                    'filetype was set to "file" but not upload found'
                );
            }
        }

        if ($request->request->has('resourceLinkList')) {
            $links = $request->get('resourceLinkList');
            if (strpos($links, '[') === false) {
                $links = json_decode('['.$links.']', true);
            } else {
                $links = json_decode($links, true);
            }
            if (empty($links)) {
                throw new \InvalidArgumentException(
                    'resourceLinkList is not a valid json. Example: [{"c_id":1:"visibility":1}]'
                );
            }
            $document->setResourceLinkList($links);
        }

        $document->setTitle($title);
        $document->setComment($request->get('comment'));

        $nodeId = (int) $request->get('parentResourceNodeId');
        $document->setParentResourceNode($nodeId);

        return $document;
    }