diff options
| author | Greg Roach <fisharebest@webtrees.net> | 2019-10-30 20:39:55 +0000 |
|---|---|---|
| committer | Greg Roach <fisharebest@webtrees.net> | 2019-10-31 15:22:31 +0000 |
| commit | ddeb33548102a68bedfc3a35cbda68303ed365ca (patch) | |
| tree | 27e2327c15d27d855c664cab92ba391cfd3b7d0b /app/Http/RequestHandlers/DeleteRecord.php | |
| parent | 4f8110804df3633753385a12c50a7bd26dbb68e4 (diff) | |
| download | webtrees-ddeb33548102a68bedfc3a35cbda68303ed365ca.tar.gz webtrees-ddeb33548102a68bedfc3a35cbda68303ed365ca.tar.bz2 webtrees-ddeb33548102a68bedfc3a35cbda68303ed365ca.zip | |
Make variable types clearer to static analysis tools
Diffstat (limited to 'app/Http/RequestHandlers/DeleteRecord.php')
| -rw-r--r-- | app/Http/RequestHandlers/DeleteRecord.php | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/app/Http/RequestHandlers/DeleteRecord.php b/app/Http/RequestHandlers/DeleteRecord.php index 82c4d75c93..a3adff4a47 100644 --- a/app/Http/RequestHandlers/DeleteRecord.php +++ b/app/Http/RequestHandlers/DeleteRecord.php @@ -30,6 +30,7 @@ use Psr\Http\Message\ServerRequestInterface; use Psr\Http\Server\RequestHandlerInterface; use function assert; +use function is_string; use function preg_match; use function preg_match_all; use function preg_replace; @@ -52,10 +53,11 @@ class DeleteRecord implements RequestHandlerInterface $tree = $request->getAttribute('tree'); assert($tree instanceof Tree); - $xref = $request->getAttribute('xref'); - $record = GedcomRecord::getInstance($xref, $tree); + $xref = $request->getAttribute('xref'); + assert(is_string($xref)); - Auth::checkRecordAccess($record, true); + $record = GedcomRecord::getInstance($xref, $tree); + $record = Auth::checkRecordAccess($record, true); if ($record && Auth::isEditor($record->tree()) && $record->canShow() && $record->canEdit()) { // Delete links to this record |