Passed
Push — feature/code-analysis ( e964aa...4fe35d )
by Jonathan
14:33
created

GeoAnalysisViewEditAction::handle()   B

Complexity

Conditions 9
Paths 11

Size

Total Lines 62
Code Lines 39

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
cc 9
eloc 39
nc 11
nop 1
dl 0
loc 62
rs 7.7404
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-2022, 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\Validator;
22
use Fisharebest\Webtrees\Services\ModuleService;
23
use Illuminate\Contracts\Container\BindingResolutionException;
24
use MyArtJaub\Webtrees\Contracts\GeoDispersion\GeoAnalysisInterface;
25
use MyArtJaub\Webtrees\Module\GeoDispersion\GeoDispersionModule;
26
use MyArtJaub\Webtrees\Module\GeoDispersion\Services\GeoAnalysisViewDataService;
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 = Validator::attributes($request)->tree();
59
60
        $admin_config_route = route(AdminConfigPage::class, ['tree' => $tree->name()]);
61
62
        if ($this->module === null) {
63
            FlashMessages::addMessage(
64
                I18N::translate('The attached module could not be found.'),
65
                'danger'
66
            );
67
            return redirect($admin_config_route);
68
        }
69
70
71
        $view_id = Validator::attributes($request)->integer('view_id', -1);
72
        $view = $this->geoview_data_service->find($tree, $view_id, true);
73
74
        $description    = Validator::parsedBody($request)->string('view_description', '');
75
        $place_depth    = Validator::parsedBody($request)->integer('view_depth', 1);
76
        $top_places     = Validator::parsedBody($request)->integer('view_top_places', 0);
77
78
        $analysis = null;
0 ignored issues
show
Unused Code introduced by
The assignment to $analysis is dead and can be removed.
Loading history...
79
        try {
80
            $analysis = app(Validator::parsedBody($request)->string('view_analysis', ''));
81
        } catch (BindingResolutionException $ex) {
0 ignored issues
show
Coding Style Comprehensibility introduced by
Consider adding a comment why this CATCH block is empty.
Loading history...
82
        }
83
84
        if (
85
            $view === null
86
            || $analysis === null || !($analysis instanceof GeoAnalysisInterface)
87
            || $place_depth <= 0 && $top_places < 0
88
        ) {
89
            FlashMessages::addMessage(
90
                I18N::translate('The parameters for view with ID “%s” are not valid.', I18N::number($view_id)),
91
                'danger'
92
            );
93
            return redirect($admin_config_route);
94
        }
95
96
        $new_view = $view
97
            ->with($view->isEnabled(), $description, $analysis, $place_depth, $top_places)
98
            ->withGlobalSettingsUpdate($request);
99
100
        try {
101
            $this->geoview_data_service->update($new_view);
102
            FlashMessages::addMessage(
103
                I18N::translate('The geographical dispersion analysis view has been successfully updated.'),
104
                'success'
105
            );
106
            //phpcs:ignore Generic.Files.LineLength.TooLong
107
            Log::addConfigurationLog('Module ' . $this->module->title() . ' : View “' . $view->id() . '” has been updated.');
108
        } catch (Throwable $ex) {
109
            FlashMessages::addMessage(
110
                I18N::translate('An error occured while updating the geographical dispersion analysis view.'),
111
                'danger'
112
            );
113
            //phpcs:ignore Generic.Files.LineLength.TooLong
114
            Log::addErrorLog('Module ' . $this->module->title() . ' : Error when updating view “' . $view->id() . '”: ' . $ex->getMessage());
115
        }
116
117
        return redirect($admin_config_route);
118
    }
119
}
120