Passed
Pull Request — main (#1)
by Jonathan
16:29
created
app/Module/Certificates/CertificatesModule.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -102,11 +102,11 @@  discard block
 block discarded – undo
102 102
      */
103 103
     public function loadRoutes($router): void
104 104
     {
105
-        $router->attach('', '', static function (Map $router): void {
105
+        $router->attach('', '', static function(Map $router): void {
106 106
 
107
-            $router->attach('', '/module-maj/certificates', static function (Map $router): void {
107
+            $router->attach('', '/module-maj/certificates', static function(Map $router): void {
108 108
 
109
-                $router->attach('', '/admin', static function (Map $router): void {
109
+                $router->attach('', '/admin', static function(Map $router): void {
110 110
 
111 111
                     $router->get(AdminConfigPage::class, '/config{/tree}', AdminConfigPage::class);
112 112
                     $router->post(AdminConfigAction::class, '/config/{tree}', AdminConfigAction::class)
@@ -129,7 +129,7 @@  discard block
 block discarded – undo
129 129
                         'permission_preference' =>  'MAJ_CERTIF_SHOW_CERT'
130 130
                     ]);
131 131
 
132
-                $router->attach('', '/certificate/{tree}/{cid}', static function (Map $router): void {
132
+                $router->attach('', '/certificate/{tree}/{cid}', static function(Map $router): void {
133 133
 
134 134
                     $router->extras([
135 135
                         'middleware'            =>  [AuthTreePreference::class],
@@ -167,7 +167,7 @@  discard block
 block discarded – undo
167 167
      */
168 168
     public function headContent(): string
169 169
     {
170
-        return '<link rel="stylesheet" href="' . e($this->moduleCssUrl()) . '">';
170
+        return '<link rel="stylesheet" href="'.e($this->moduleCssUrl()).'">';
171 171
     }
172 172
 
173 173
     /**
@@ -176,7 +176,7 @@  discard block
 block discarded – undo
176 176
      */
177 177
     public function listUrl(Tree $tree, array $parameters = []): string
178 178
     {
179
-        return route(CertificatesList::class, ['tree' => $tree->name() ] + $parameters);
179
+        return route(CertificatesList::class, ['tree' => $tree->name()] + $parameters);
180 180
     }
181 181
 
182 182
     /**
@@ -194,7 +194,7 @@  discard block
 block discarded – undo
194 194
      */
195 195
     public function listIsEmpty(Tree $tree): bool
196 196
     {
197
-        return Auth::accessLevel($tree) > (int) $tree->getPreference('MAJ_CERTIF_SHOW_CERT', (string) Auth::PRIV_HIDE);
197
+        return Auth::accessLevel($tree) > (int)$tree->getPreference('MAJ_CERTIF_SHOW_CERT', (string)Auth::PRIV_HIDE);
198 198
     }
199 199
 
200 200
     /**
Please login to merge, or discard this patch.
app/Module/Certificates/Http/RequestHandlers/AdminConfigAction.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -63,12 +63,12 @@  discard block
 block discarded – undo
63 63
             return redirect($admin_config_route);
64 64
         }
65 65
 
66
-        $params = (array) $request->getParsedBody();
66
+        $params = (array)$request->getParsedBody();
67 67
 
68
-        $tree->setPreference('MAJ_CERTIF_SHOW_CERT', $params['MAJ_CERTIF_SHOW_CERT'] ?? (string) Auth::PRIV_HIDE);
68
+        $tree->setPreference('MAJ_CERTIF_SHOW_CERT', $params['MAJ_CERTIF_SHOW_CERT'] ?? (string)Auth::PRIV_HIDE);
69 69
         $tree->setPreference(
70 70
             'MAJ_CERTIF_SHOW_NO_WATERMARK',
71
-            $params['MAJ_CERTIF_SHOW_NO_WATERMARK'] ?? (string) Auth::PRIV_HIDE
71
+            $params['MAJ_CERTIF_SHOW_NO_WATERMARK'] ?? (string)Auth::PRIV_HIDE
72 72
         );
73 73
         $tree->setPreference('MAJ_CERTIF_WM_DEFAULT', $params['MAJ_CERTIF_WM_DEFAULT'] ?? '');
74 74
 
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
         // Only accept valid folders for MAJ_CERT_ROOTDIR
87 87
         $cert_root_dir = $params['MAJ_CERTIF_ROOTDIR'] ?? '';
88 88
         $cert_root_dir = preg_replace('/[:\/\\\\]+/', '/', $cert_root_dir);
89
-        $cert_root_dir = trim($cert_root_dir, '/') . '/';
89
+        $cert_root_dir = trim($cert_root_dir, '/').'/';
90 90
         $tree->setPreference('MAJ_CERTIF_ROOTDIR', $cert_root_dir);
91 91
 
92 92
         FlashMessages::addMessage(
Please login to merge, or discard this patch.
app/Module/Certificates/Http/RequestHandlers/CertificateImage.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
 
105 105
         if ($certificate === null) {
106 106
             return $this->certif_image_factory
107
-            ->replacementImageResponse((string) StatusCodeInterface::STATUS_NOT_FOUND)
107
+            ->replacementImageResponse((string)StatusCodeInterface::STATUS_NOT_FOUND)
108 108
             ->withHeader('X-Image-Exception', I18N::translate('The certificate was not found in this family tree.'))
109 109
             ;
110 110
         }
@@ -132,7 +132,7 @@  discard block
 block discarded – undo
132 132
         $size = $certificate->tree()->getPreference('MAJ_CERTIF_WM_FONT_MAXSIZE');
133 133
         $text = $this->watermarkText($request, $certificate);
134 134
 
135
-        return new Watermark($text, $color, is_numeric($size) ? (int) $size : Watermark::DEFAULT_SIZE);
135
+        return new Watermark($text, $color, is_numeric($size) ? (int)$size : Watermark::DEFAULT_SIZE);
136 136
     }
137 137
 
138 138
     /**
Please login to merge, or discard this patch.
app/Module/Certificates/Http/RequestHandlers/CertificatesList.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
 
80 80
         $title = I18N::translate('Certificates');
81 81
 
82
-        $cities = array_map(function (string $item): array {
82
+        $cities = array_map(function(string $item): array {
83 83
             return [$this->url_obfuscator_service->obfuscate($item), $item];
84 84
         }, $this->certif_filesystem->cities($tree));
85 85
 
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
             $certificates = $this->certif_filesystem->certificatesForCity($tree, $city);
91 91
         }
92 92
 
93
-        return $this->viewResponse($this->module->name() . '::certificates-list', [
93
+        return $this->viewResponse($this->module->name().'::certificates-list', [
94 94
             'title'                     =>  $title,
95 95
             'tree'                      =>  $tree,
96 96
             'module_name'               =>  $this->module->name(),
Please login to merge, or discard this patch.
app/Module/Certificates/Http/RequestHandlers/AdminConfigPage.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -87,7 +87,7 @@
 block discarded – undo
87 87
             throw new HttpAccessDeniedException();
88 88
         }
89 89
 
90
-        return $this->viewResponse($this->module->name() . '::admin/config', [
90
+        return $this->viewResponse($this->module->name().'::admin/config', [
91 91
             'module_name'       =>  $this->module->name(),
92 92
             'title'             =>  $this->module->title(),
93 93
             'tree'              =>  $tree,
Please login to merge, or discard this patch.
app/Module/Certificates/Http/RequestHandlers/CertificatePage.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -100,7 +100,7 @@
 block discarded – undo
100 100
             return redirect(route(TreePage::class, ['tree' => $tree->name()]));
101 101
         }
102 102
 
103
-        return $this->viewResponse($this->module->name() . '::certificate-page', [
103
+        return $this->viewResponse($this->module->name().'::certificate-page', [
104 104
             'title'                     =>  I18N::translate('Certificate - %s', $certificate->name()),
105 105
             'tree'                      =>  $tree,
106 106
             'module_name'               =>  $this->module->name(),
Please login to merge, or discard this patch.
app/Module/Certificates/Http/RequestHandlers/AutoCompleteFile.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -72,10 +72,10 @@
 block discarded – undo
72 72
             return collect();
73 73
         }
74 74
 
75
-        $query  = $request->getAttribute('query') ?? '';
75
+        $query = $request->getAttribute('query') ?? '';
76 76
 
77 77
         /** @var Collection<string> $results */
78
-        $results =  $this->certif_filesystem
78
+        $results = $this->certif_filesystem
79 79
             ->certificatesForCityContaining($tree, $city, $query)
80 80
             ->map(fn(Certificate $certificate): string => $certificate->filename());
81 81
 
Please login to merge, or discard this patch.
app/Module/Certificates/Services/CertificateDataService.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -43,7 +43,7 @@  discard block
 block discarded – undo
43 43
         $tree = $certificate->tree();
44 44
         return DB::table('individuals')
45 45
             ->where('i_file', '=', $tree->id())
46
-            ->where('i_gedcom', 'like', '% _ACT ' . $this->escapedSqlPath($certificate) . '%')
46
+            ->where('i_gedcom', 'like', '% _ACT '.$this->escapedSqlPath($certificate).'%')
47 47
             ->select(['individuals.*'])
48 48
             ->get()
49 49
             ->map(Registry::individualFactory()->mapper($tree))
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
         $tree = $certificate->tree();
65 65
         return DB::table('families')
66 66
             ->where('f_file', '=', $tree->id())
67
-            ->where('f_gedcom', 'like', '% _ACT ' . $this->escapedSqlPath($certificate) . '%')
67
+            ->where('f_gedcom', 'like', '% _ACT '.$this->escapedSqlPath($certificate).'%')
68 68
             ->select(['families.*'])
69 69
             ->get()
70 70
             ->map(Registry::familyFactory()->mapper($tree))
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
85 85
         $tree = $certificate->tree();
86 86
         return DB::table('media')
87 87
             ->where('m_file', '=', $tree->id())
88
-            ->where('m_gedcom', 'like', '% _ACT ' . $this->escapedSqlPath($certificate) . '%')
88
+            ->where('m_gedcom', 'like', '% _ACT '.$this->escapedSqlPath($certificate).'%')
89 89
             ->select(['media.*'])
90 90
             ->get()
91 91
             ->map(Registry::mediaFactory()->mapper($tree))
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
         return DB::table('other')
108 108
             ->where('o_file', '=', $tree->id())
109 109
             ->where('o_type', '=', 'NOTE')
110
-            ->where('o_gedcom', 'like', '% _ACT ' . $this->escapedSqlPath($certificate) . '%')
110
+            ->where('o_gedcom', 'like', '% _ACT '.$this->escapedSqlPath($certificate).'%')
111 111
             ->select(['other.*'])
112 112
             ->get()
113 113
             ->map(Registry::noteFactory()->mapper($tree))
@@ -135,7 +135,7 @@  discard block
 block discarded – undo
135 135
     public function oneLinkedSource(Certificate $certificate): ?Source
136 136
     {
137 137
         $regex_query = preg_quote($certificate->gedcomPath(), '/');
138
-        $regex_pattern = '/[1-9] SOUR @(' . Gedcom::REGEX_XREF . ')@(?:\n[2-9]\s(?:(?!SOUR)\w+)\s.*)*\n[2-9] _ACT ' . $regex_query . '/i'; //phpcs:ignore Generic.Files.LineLength.TooLong
138
+        $regex_pattern = '/[1-9] SOUR @('.Gedcom::REGEX_XREF.')@(?:\n[2-9]\s(?:(?!SOUR)\w+)\s.*)*\n[2-9] _ACT '.$regex_query.'/i'; //phpcs:ignore Generic.Files.LineLength.TooLong
139 139
 
140 140
         foreach ($this->linkedRecordsLists($certificate) as $linked_records) {
141 141
             foreach ($linked_records as $gedcom_record) {
Please login to merge, or discard this patch.
app/Module/Certificates/Services/UrlObfuscatorService.php 1 patch
Spacing   +2 added lines, -3 removed lines patch added patch discarded remove patch
@@ -43,8 +43,7 @@  discard block
 block discarded – undo
43 43
             $server_software = $request->getServerParams()['SERVER_SOFTWARE'] ?? '';
44 44
             $this->encryption_key = str_pad(md5(
45 45
                 $server_name !== '' && $server_software !== '' ?
46
-                $server_name . $server_software :
47
-                'STANDARDKEYIFNOSERVER'
46
+                $server_name.$server_software : 'STANDARDKEYIFNOSERVER'
48 47
             ), SODIUM_CRYPTO_SECRETBOX_KEYBYTES, "1234567890ABCDEF");
49 48
         }
50 49
         return $this->encryption_key;
@@ -78,7 +77,7 @@  discard block
 block discarded – undo
78 77
 
79 78
         \Fisharebest\Webtrees\DebugBar::addMessage($key);
80 79
         $encryted = sodium_crypto_secretbox($cleartext, $nonce, $key);
81
-        return strtr(base64_encode($nonce . $encryted), '+/=', '._-');
80
+        return strtr(base64_encode($nonce.$encryted), '+/=', '._-');
82 81
     }
83 82
 
84 83
     /**
Please login to merge, or discard this patch.