Passed
Branch main (f9aaf7)
by Jonathan
14:43
created

GeoAnalysisViewEditAction::handle()   B

Complexity

Conditions 9
Paths 11

Size

Total Lines 65
Code Lines 41

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
cc 9
eloc 41
nc 11
nop 1
dl 0
loc 65
rs 7.7084
c 0
b 0
f 0

How to fix   Long Method   

Long Method

Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.

For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.

Commonly applied refactorings include:

1
<?php
2
3
/**
4
 * webtrees-lib: MyArtJaub library for webtrees
5
 *
6
 * @package MyArtJaub\Webtrees
7
 * @subpackage GeoDispersion
8
 * @author Jonathan Jaubart <[email protected]>
9
 * @copyright Copyright (c) 2009-2021, Jonathan Jaubart
10
 * @license http://www.gnu.org/licenses/gpl.html GNU General Public License, version 3
11
 */
12
13
declare(strict_types=1);
14
15
namespace MyArtJaub\Webtrees\Module\GeoDispersion\Http\RequestHandlers;
16
17
use Fisharebest\Webtrees\FlashMessages;
18
use Fisharebest\Webtrees\I18N;
19
use Fisharebest\Webtrees\Log;
20
use Fisharebest\Webtrees\Tree;
21
use Fisharebest\Webtrees\Services\ModuleService;
22
use Illuminate\Contracts\Container\BindingResolutionException;
23
use MyArtJaub\Webtrees\Contracts\GeoDispersion\GeoAnalysisInterface;
24
use MyArtJaub\Webtrees\Module\GeoDispersion\GeoDispersionModule;
25
use MyArtJaub\Webtrees\Module\GeoDispersion\Services\GeoAnalysisViewDataService;
26
use MyArtJaub\Webtrees\Module\GeoDispersion\Views\AbstractGeoAnalysisView;
27
use Psr\Http\Message\ResponseInterface;
28
use Psr\Http\Message\ServerRequestInterface;
29
use Psr\Http\Server\RequestHandlerInterface;
30
use Throwable;
31
32
/**
33
 * Request handler for editing a geographical analysis view.
34
 */
35
class GeoAnalysisViewEditAction implements RequestHandlerInterface
36
{
37
    private ?GeoDispersionModule $module;
38
    private GeoAnalysisViewDataService $geoview_data_service;
39
40
    /**
41
     * Constructor for GeoAnalysisViewEditAction Request Handler
42
     *
43
     * @param ModuleService $module_service
44
     * @param GeoAnalysisViewDataService $geoview_data_service
45
     */
46
    public function __construct(ModuleService $module_service, GeoAnalysisViewDataService $geoview_data_service)
47
    {
48
        $this->module = $module_service->findByInterface(GeoDispersionModule::class)->first();
49
        $this->geoview_data_service = $geoview_data_service;
50
    }
51
52
    /**
53
     * {@inheritDoc}
54
     * @see \Psr\Http\Server\RequestHandlerInterface::handle()
55
     */
56
    public function handle(ServerRequestInterface $request): ResponseInterface
57
    {
58
        $tree = $request->getAttribute('tree');
59
        assert($tree instanceof Tree);
60
61
        $admin_config_route = route(AdminConfigPage::class, ['tree' => $tree->name()]);
62
63
        if ($this->module === null) {
64
            FlashMessages::addMessage(
65
                I18N::translate('The attached module could not be found.'),
66
                'danger'
67
            );
68
            return redirect($admin_config_route);
69
        }
70
71
72
        $view_id = (int) $request->getAttribute('view_id');
73
        $view = $this->geoview_data_service->find($tree, $view_id, true);
74
75
        $params = (array) $request->getParsedBody();
76
77
        $description    = $params['view_description'] ?? '';
78
        $place_depth    = (int) ($params['view_depth'] ?? 1);
79
        $top_places     = (int) ($params['view_top_places'] ?? 0);
80
81
        $analysis = null;
0 ignored issues
show
Unused Code introduced by
The assignment to $analysis is dead and can be removed.
Loading history...
82
        try {
83
            $analysis = app($params['view_analysis'] ?? '');
84
        } catch (BindingResolutionException $ex) {
0 ignored issues
show
Coding Style Comprehensibility introduced by
Consider adding a comment why this CATCH block is empty.
Loading history...
85
        }
86
87
        if (
88
            $view === null
89
            || $analysis === null || !($analysis instanceof GeoAnalysisInterface)
90
            || $place_depth <= 0 && $top_places < 0
91
        ) {
92
            FlashMessages::addMessage(
93
                I18N::translate('The parameters for view with ID “%s” are not valid.', I18N::number($view_id)),
94
                'danger'
95
            );
96
            return redirect($admin_config_route);
97
        }
98
99
        $new_view = $view
100
            ->with($view->isEnabled(), $description, $analysis, $place_depth, $top_places)
101
            ->withGlobalSettingsUpdate($request);
102
103
        try {
104
            $this->geoview_data_service->update($new_view);
105
            FlashMessages::addMessage(
106
                I18N::translate('The geographical dispersion analysis view has been successfully updated.'),
107
                'success'
108
            );
109
            //phpcs:ignore Generic.Files.LineLength.TooLong
110
            Log::addConfigurationLog('Module ' . $this->module->title() . ' : View “' . $view->id() . '” has been updated.');
111
        } catch (Throwable $ex) {
112
            FlashMessages::addMessage(
113
                I18N::translate('An error occured while updating the geographical dispersion analysis view.'),
114
                'danger'
115
            );
116
            //phpcs:ignore Generic.Files.LineLength.TooLong
117
            Log::addErrorLog('Module ' . $this->module->title() . ' : Error when updating view “' . $view->id() . '”: ' . $ex->getMessage());
118
        }
119
120
        return redirect($admin_config_route);
121
    }
122
}
123