Fisharebest\Webtrees\Http\RequestHandlers\ExportGedcomClient::handle()

Complexity

Conditions	11
Paths	16

Size

Total Lines	101
Code Lines	61

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function handle(ServerRequestInterface $request): ResponseInterface
    {
        $tree = $request->getAttribute('tree');
        assert($tree instanceof Tree);

        $convert          = (bool) ($request->getParsedBody()['convert'] ?? false);
        $zip              = (bool) ($request->getParsedBody()['zip'] ?? false);
        $media            = (bool) ($request->getParsedBody()['media'] ?? false);
        $media_path       = $request->getParsedBody()['media-path'] ?? '';
        $privatize_export = $request->getParsedBody()['privatize_export'];

        $access_levels = [
            'gedadmin' => Auth::PRIV_NONE,
            'user'     => Auth::PRIV_USER,
            'visitor'  => Auth::PRIV_PRIVATE,
            'none'     => Auth::PRIV_HIDE,
        ];

        $access_level = $access_levels[$privatize_export];
        $encoding     = $convert ? 'ANSI' : 'UTF-8';

        // What to call the downloaded file
        $download_filename = $tree->name();

        // Force a ".ged" suffix
        if (strtolower(pathinfo($download_filename, PATHINFO_EXTENSION)) !== 'ged') {
            $download_filename .= '.ged';
        }

        if ($zip || $media) {
            // Export the GEDCOM to an in-memory stream.
            $tmp_stream = tmpfile();
            FunctionsExport::exportGedcom($tree, $tmp_stream, $access_level, $media_path, $encoding);

It seems like $tmp_stream can also be of type false; however, parameter $stream of Fisharebest\Webtrees\Functions\FunctionsExport::exportGedcom() does only seem to accept resource, maybe add an additional type check?

            rewind($tmp_stream);

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

            $path = $tree->getPreference('MEDIA_DIRECTORY', 'media/');

            // Create a new/empty .ZIP file
            $temp_zip_file  = tempnam(sys_get_temp_dir(), 'webtrees-zip-');
            $zip_adapter    = new ZipArchiveAdapter($temp_zip_file);
            $zip_filesystem = new Filesystem($zip_adapter);
            $zip_filesystem->writeStream($download_filename, $tmp_stream);
            fclose($tmp_stream);

            if ($media) {
                $manager = new MountManager([
                    'media' => $tree->mediaFilesystem(),
                    'zip'   => $zip_filesystem,
                ]);

                $records = DB::table('media')
                    ->where('m_file', '=', $tree->id())
                    ->get()
                    ->map(Media::rowMapper())
                    ->filter(GedcomRecord::accessFilter());

                foreach ($records as $record) {
                    foreach ($record->mediaFiles() as $media_file) {
                        $from = 'media://' . $media_file->filename();
                        $to   = 'zip://' . $path . $media_file->filename();
                        if (!$media_file->isExternal() && $manager->has($from)) {
                            $manager->copy($from, $to);
                        }
                    }
                }
            }

            // Need to force-close ZipArchive filesystems.
            $zip_adapter->getArchive()->close();

            // Use a stream, so that we do not have to load the entire file into memory.
            $stream   = app(StreamFactoryInterface::class)->createStreamFromFile($temp_zip_file);
            $filename = addcslashes($download_filename, '"') . '.zip';

            /** @var ResponseFactoryInterface $response_factory */
            $response_factory = app(ResponseFactoryInterface::class);

            return $response_factory->createResponse()
                ->withBody($stream)
                ->withHeader('Content-Type', 'application/zip')
                ->withHeader('Content-Disposition', 'attachment; filename="' . $filename . '"');
        }

        $resource = fopen('php://temp', 'wb+');
        FunctionsExport::exportGedcom($tree, $resource, $access_level, $media_path, $encoding);
        rewind($resource);

        $charset = $convert ? 'ISO-8859-1' : 'UTF-8';

        /** @var StreamFactoryInterface $response_factory */
        $stream_factory = app(StreamFactoryInterface::class);

        $stream = $stream_factory->createStreamFromResource($resource);

It seems like $resource can also be of type false; however, parameter $resource of Psr\Http\Message\StreamFactoryInterface::createStreamFromResource() does only seem to accept resource, maybe add an additional type check?

        /** @var ResponseFactoryInterface $response_factory */
        $response_factory = app(ResponseFactoryInterface::class);

        return $response_factory->createResponse()
            ->withBody($stream)
            ->withHeader('Content-Type', 'text/x-gedcom; charset=' . $charset)
            ->withHeader('Content-Disposition', 'attachment; filename="' . addcslashes($download_filename, '"') . '"');
    }