MyArtJaub\Webtrees\Module\GeoDispersion\Http\RequestHandlers\MapAdapterEditAction::handle()

Complexity

Conditions	8
Paths	11

Size

Total Lines	54
Code Lines	35

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function handle(ServerRequestInterface $request): ResponseInterface
    {
        $tree = Validator::attributes($request)->tree();

        if ($this->module === null) {
            FlashMessages::addMessage(
                I18N::translate('The attached module could not be found.'),
                'danger'
            );
            return Registry::responseFactory()->redirect(AdminConfigPage::class, ['tree' => $tree->name()]);
        }

        $adapter_id = Validator::attributes($request)->integer('adapter_id', -1);
        $map_adapter = $this->mapadapter_data_service->find($adapter_id);

        $map = $this->map_definition_service->find(Validator::parsedBody($request)->string('map_adapter_map', ''));
        $mapping_property   = Validator::parsedBody($request)->string('map_adapter_property_selected', '');

        $mapper = null;

The assignment to $mapper is dead and can be removed.

        try {
            $mapper = app(Validator::parsedBody($request)->string('map_adapter_mapper', ''));
        } catch (BindingResolutionException $ex) {

Consider adding a comment why this CATCH block is empty.

        }

        if ($map_adapter === null || $map === null || $mapper === null || !($mapper instanceof PlaceMapperInterface)) {
            FlashMessages::addMessage(
                I18N::translate('The parameters for the map configuration are not valid.'),
                'danger'
            );
            return Registry::responseFactory()->redirect(AdminConfigPage::class, ['tree' => $tree->name()]);
        }

        $mapper->setConfig($mapper->config()->withConfigUpdate($request));
        $new_map_adapter = $map_adapter->with($map, $mapper, $mapping_property);
        try {
            $this->mapadapter_data_service->update($new_map_adapter);
            FlashMessages::addMessage(
                I18N::translate('The map configuration has been successfully updated.'),
                'success'
            );
            //phpcs:ignore Generic.Files.LineLength.TooLong
            Log::addConfigurationLog('Module ' . $this->module->title() . ' : Map Adapter “' . $map_adapter->id() . '” has been updated.');
        } catch (Throwable $ex) {
            FlashMessages::addMessage(
                I18N::translate('An error occured while updating the map configuration.'),
                'danger'
            );
            //phpcs:ignore Generic.Files.LineLength.TooLong
            Log::addErrorLog('Module ' . $this->module->title() . ' : Error when updating Map Adapter “' . $map_adapter->id() . '”: ' . $ex->getMessage());
        }

        return Registry::responseFactory()->redirect(MapAdapterEditPage::class, [
            'tree' => $tree->name(),
            'adapter_id' => $map_adapter->id()
        ]);