Passed
Push — master ( 169a66...4e557f )
by Alexander
02:27
created

View   D

Complexity

Total Complexity 59

Size/Duplication

Total Lines 601
Duplicated Lines 0 %

Test Coverage

Coverage 49.07%

Importance

Changes 2
Bugs 1 Features 0
Metric Value
wmc 59
eloc 128
c 2
b 1
f 0
dl 0
loc 601
ccs 79
cts 161
cp 0.4907
rs 4.08

37 Methods

Rating   Name   Duplication   Size   Complexity  
A getRequestedViewFile() 0 3 2
A renderFile() 0 40 5
A afterRender() 0 6 1
A getViewFile() 0 3 2
A beginPage() 0 6 2
A setBlock() 0 3 1
A setSourceLanguage() 0 3 1
A addDynamicPlaceholder() 0 7 2
A setDefaultParameters() 0 3 1
A setLanguage() 0 3 1
A getDefaultParameters() 0 3 1
A evaluateDynamicContent() 0 3 1
A renderDynamic() 0 15 3
A setDynamicPlaceholders() 0 3 1
A beforeRender() 0 6 1
A popDynamicContent() 0 3 1
A setDefaultExtension() 0 3 1
A getDynamicContents() 0 3 1
B findTemplateFile() 0 26 7
A getDynamicPlaceholders() 0 3 1
A getBasePath() 0 3 1
A setSourceLocale() 0 5 1
A localize() 0 20 5
A render() 0 5 1
A setRenderers() 0 3 1
A withTheme() 0 5 1
A __construct() 0 6 1
A removeBlock() 0 3 1
A getSourceLocale() 0 7 2
A getPlaceholderSignature() 0 3 1
A hasBlock() 0 3 1
A getBlock() 0 7 2
A pushDynamicContent() 0 3 1
A setContext() 0 3 1
A endPage() 0 4 1
A setPlaceholderSalt() 0 3 1
A getDefaultExtension() 0 3 1

How to fix   Complexity   

Complex Class

Complex classes like View often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes.

Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.

While breaking up the class, it is a good idea to analyze how other classes use View, and based on these observations, apply Extract Interface, too.

1
<?php
2
3
declare(strict_types=1);
4
5
namespace Yiisoft\View;
6
7
use Psr\EventDispatcher\EventDispatcherInterface;
8
use Psr\Log\LoggerInterface;
9
use Yiisoft\I18n\Locale;
10
use Yiisoft\View\Event\AfterRender;
11
use Yiisoft\View\Event\BeforeRender;
12
use Yiisoft\View\Event\PageBegin;
13
use Yiisoft\View\Event\PageEnd;
14
use Yiisoft\View\Exception\ViewNotFoundException;
15
16
/**
17
 * View represents a view object in the MVC pattern.
18
 *
19
 * View provides a set of methods (e.g. {@see View::render()}) for rendering purpose.
20
 *
21
 * For more details and usage information on View, see the [guide article on views](guide:structure-views).
22
 */
23
class View implements DynamicContentAwareInterface
24
{
25
    /**
26
     * @var string view path
27
     */
28
    private string $basePath;
29
30
    /**
31
     * @var array a list of named output blocks. The keys are the block names and the values are the corresponding block
32
     * content. You can call {@see beginBlock()} and {@see endBlock()} to capture small fragments of a view.
33
     * They can be later accessed somewhere else through this property.
34
     */
35
    private array $blocks;
36
37
    /**
38
     * @var ViewContextInterface|null the context under which the {@see renderFile()} method is being invoked.
39
     */
40
    private ?ViewContextInterface $context = null;
41
42
    /**
43
     * @var string the default view file extension. This will be appended to view file names if they don't have file
44
     * extensions.
45
     */
46
    private string $defaultExtension = 'php';
47
48
    /**
49
     * @var array custom parameters that are shared among view templates.
50
     */
51
    private array $defaultParameters = [];
52
53
    /**
54
     * @var EventDispatcherInterface
55
     */
56
    protected EventDispatcherInterface $eventDispatcher;
57
58
    /**
59
     * @var array a list of available renderers indexed by their corresponding supported file extensions. Each renderer
60
     * may be a view renderer object or the configuration for creating the renderer object. For example, the
61
     * following configuration enables the Twig view renderer:
62
     *
63
     * ```php
64
     * [
65
     *     'twig' => ['__class' => \Yiisoft\Yii\Twig\ViewRenderer::class],
66
     * ]
67
     * ```
68
     *
69
     * If no renderer is available for the given view file, the view file will be treated as a normal PHP and rendered
70
     * via PhpTemplateRenderer.
71
     */
72
    protected array $renderers = [];
73
74
    /**
75
     * @var Theme|null The theme object.
76
     */
77
    protected ?Theme $theme = null;
78
79
    /**
80
     * @var DynamicContentAwareInterface[] a list of currently active dynamic content class instances.
81
     */
82
    private array $cacheStack = [];
83
84
    /**
85
     * @var array a list of placeholders for embedding dynamic contents.
86
     */
87
    private array $dynamicPlaceholders = [];
88
89
    /**
90
     * @var string
91
     */
92
    private string $language = 'en';
93
94
    /**
95
     * @var LoggerInterface
96
     */
97
    private LoggerInterface $logger;
98
99
    /**
100
     * @var string
101
     */
102
    private string $sourceLanguage = 'en';
103
104
    /**
105
     * @var Locale|null source locale used to find localized view file.
106
     */
107
    private ?Locale $sourceLocale = null;
108
109
    private string $placeholderSignature;
110
111
    /**
112
     * @var array the view files currently being rendered. There may be multiple view files being
113
     * rendered at a moment because one view may be rendered within another.
114
     */
115
    private array $viewFiles = [];
116
117 21
    public function __construct(string $basePath, EventDispatcherInterface $eventDispatcher, LoggerInterface $logger)
118
    {
119 21
        $this->basePath = $basePath;
120 21
        $this->eventDispatcher = $eventDispatcher;
121 21
        $this->logger = $logger;
122 21
        $this->setPlaceholderSalt(__DIR__);
123 21
    }
124
125
    /**
126
     * @return static
127
     */
128 1
    public function withTheme(Theme $theme): self
129
    {
130 1
        $new = clone $this;
131 1
        $new->theme = $theme;
132 1
        return $new;
133
    }
134
135 21
    public function setPlaceholderSalt(string $salt): void
136
    {
137 21
        $this->placeholderSignature = dechex(crc32($salt));
138 21
    }
139
140 8
    public function getPlaceholderSignature(): string
141
    {
142 8
        return $this->placeholderSignature;
143
    }
144
145
    public function getBasePath(): string
146
    {
147
        return $this->basePath;
148
    }
149
150
    public function setRenderers(array $renderers): void
151
    {
152
        $this->renderers = $renderers;
153
    }
154
155
    public function setSourceLanguage(string $language): void
156
    {
157
        $this->sourceLanguage = $language;
158
    }
159
160
    public function setLanguage(string $language): void
161
    {
162
        $this->language = $language;
163
    }
164
165
    public function setContext(ViewContextInterface $context): void
166
    {
167
        $this->context = $context;
168
    }
169
170
    public function getDefaultExtension(): string
171
    {
172
        return $this->defaultExtension;
173
    }
174
175
    public function setDefaultExtension(string $defaultExtension): void
176
    {
177
        $this->defaultExtension = $defaultExtension;
178
    }
179
180
    public function getDefaultParameters(): array
181
    {
182
        return $this->defaultParameters;
183
    }
184
185 2
    public function setDefaultParameters(array $defaultParameters): void
186
    {
187 2
        $this->defaultParameters = $defaultParameters;
188 2
    }
189
190
    /**
191
     * {@see blocks}
192
     *
193
     * @param string $id
194
     * @param string $value
195
     */
196
    public function setBlock(string $id, string $value): void
197
    {
198
        $this->blocks[$id] = $value;
199
    }
200
201
    /**
202
     * {@see blocks}
203
     *
204
     * @param string $id
205
     */
206
    public function removeBlock(string $id): void
207
    {
208
        unset($this->blocks[$id]);
209
    }
210
211
    /**
212
     * {@see blocks}
213
     *
214
     * @param string $id
215
     *
216
     * @return string
217
     */
218
    public function getBlock(string $id): string
219
    {
220
        if (isset($this->blocks[$id])) {
221
            return $this->blocks[$id];
222
        }
223
224
        throw new \InvalidArgumentException('Block: "' . $id . '" not found.');
225
    }
226
227
    /**
228
     * {@see blocks}
229
     *
230
     * @param string $id
231
     *
232
     * @return bool
233
     */
234
    public function hasBlock(string $id): bool
235
    {
236
        return isset($this->blocks[$id]);
237
    }
238
239
    /**
240
     * Renders a view.
241
     *
242
     * The view to be rendered can be specified in one of the following formats:
243
     *
244
     * - [path alias](guide:concept-aliases) (e.g. "@app/views/site/index");
245
     * - absolute path within application (e.g. "//site/index"): the view name starts with double slashes. The actual
246
     *   view file will be looked for under the [[Application::viewPath|view path]] of the application.
247
     * - absolute path within current module (e.g. "/site/index"): the view name starts with a single slash. The actual
248
     *   view file will be looked for under the [[Module::viewPath|view path]] of the [[Controller::module|current module]].
249
     * - relative view (e.g. "index"): the view name does not start with `@` or `/`. The corresponding view file will be
250
     *   looked for under the {@see ViewContextInterface::getViewPath()} of the view `$context`.
251
     *   If `$context` is not given, it will be looked for under the directory containing the view currently
252
     *   being rendered (i.e., this happens when rendering a view within another view).
253
     *
254
     * @param string $view the view name.
255
     * @param array $parameters the parameters (name-value pairs) that will be extracted and made available in the view
256
     * file.
257
     * @param ViewContextInterface|null $context the context to be assigned to the view and can later be accessed via
258
     * {@see context} in the view. If the context implements {@see ViewContextInterface}, it may also be used to locate
259
     * the view file corresponding to a relative view name.
260
     *
261
     * @throws \RuntimeException if the view cannot be resolved.
262
     * @throws ViewNotFoundException if the view file does not exist.
263
     * @throws \Throwable
264
     *
265
     * {@see renderFile()}
266
     *
267
     * @return string the rendering result
268
     */
269 4
    public function render(string $view, array $parameters = [], ?ViewContextInterface $context = null): string
270
    {
271 4
        $viewFile = $this->findTemplateFile($view, $context);
272
273 4
        return $this->renderFile($viewFile, $parameters, $context);
274
    }
275
276
    /**
277
     * Finds the view file based on the given view name.
278
     *
279
     * @param string $view the view name or the [path alias](guide:concept-aliases) of the view file. Please refer to
280
     * {@see render()} on how to specify this parameter.
281
     * @param ViewContextInterface|null $context the context to be assigned to the view and can later be accessed via
282
     * {@see context} in the view. If the context implements {@see ViewContextInterface}, it may also be used to locate the
283
     * view file corresponding to a relative view name.
284
     *
285
     * @throws \RuntimeException if a relative view name is given while there is no active context to determine the
286
     * corresponding view file.
287
     *
288
     * @return string the view file path. Note that the file may not exist.
289
     */
290 4
    protected function findTemplateFile(string $view, ?ViewContextInterface $context = null): string
291
    {
292 4
        if (strncmp($view, '//', 2) === 0) {
293
            // path relative to basePath e.g. "//layouts/main"
294 4
            $file = $this->basePath . '/' . ltrim($view, '/');
295 1
        } elseif ($context instanceof ViewContextInterface) {
296
            // path provided by context
297
            $file = $context->getViewPath() . '/' . $view;
298 1
        } elseif (($currentViewFile = $this->getRequestedViewFile()) !== false) {
299
            // path relative to currently rendered view
300 1
            $file = dirname($currentViewFile) . '/' . $view;
0 ignored issues
show
Bug introduced by
It seems like $currentViewFile can also be of type true; however, parameter $path of dirname() does only seem to accept string, maybe add an additional type check? ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-type  annotation

300
            $file = dirname(/** @scrutinizer ignore-type */ $currentViewFile) . '/' . $view;
Loading history...
301
        } else {
302
            throw new \RuntimeException("Unable to resolve view file for view '$view': no active view context.");
303
        }
304
305 4
        if (pathinfo($file, PATHINFO_EXTENSION) !== '') {
306 1
            return $file;
307
        }
308
309 3
        $path = $file . '.' . $this->defaultExtension;
310
311 3
        if ($this->defaultExtension !== 'php' && !is_file($path)) {
312
            $path = $file . '.php';
313
        }
314
315 3
        return $path;
316
    }
317
318
    /**
319
     * Renders a view file.
320
     *
321
     * If {@see theme} is enabled (not null), it will try to render the themed version of the view file as long as it
322
     * is available.
323
     *
324
     * If {@see renderers} is enabled (not null), the method will use it to render the view file. Otherwise,
325
     * it will simply include the view file as a normal PHP file, capture its output and
326
     * return it as a string.
327
     *
328
     * @param string $viewFile the view file. This can be either an absolute file path or an alias of it.
329
     * @param array $parameters the parameters (name-value pairs) that will be extracted and made available in the view
330
     * file.
331
     * @param ViewContextInterface|null $context the context that the view should use for rendering the view. If null,
332
     * existing {@see context} will be used.
333
     *
334
     * @throws \Throwable
335
     * @throws ViewNotFoundException if the view file does not exist
336
     *
337
     * @return string the rendering result
338
     */
339 11
    public function renderFile(string $viewFile, array $parameters = [], ?ViewContextInterface $context = null): string
340
    {
341 11
        $parameters = array_merge($this->defaultParameters, $parameters);
342
343
        // TODO: these two match now
344 11
        $requestedFile = $viewFile;
345
346 11
        if ($this->theme !== null) {
347 1
            $viewFile = $this->theme->applyTo($viewFile);
348
        }
349
350 11
        if (is_file($viewFile)) {
351 11
            $viewFile = $this->localize($viewFile);
352
        } else {
353
            throw new ViewNotFoundException("The view file does not exist: $viewFile");
354
        }
355
356 11
        $oldContext = $this->context;
357 11
        if ($context !== null) {
358
            $this->context = $context;
359
        }
360 11
        $output = '';
361 11
        $this->viewFiles[] = [
362 11
            'resolved' => $viewFile,
363 11
            'requested' => $requestedFile,
364
        ];
365
366 11
        if ($this->beforeRender($viewFile, $parameters)) {
367 11
            $this->logger->debug("Rendering view file: $viewFile");
368 11
            $ext = pathinfo($viewFile, PATHINFO_EXTENSION);
369 11
            $renderer = $this->renderers[$ext] ?? new PhpTemplateRenderer();
370 11
            $output = $renderer->render($this, $viewFile, $parameters);
371
372 11
            $output = $this->afterRender($viewFile, $parameters, $output);
373
        }
374
375 11
        array_pop($this->viewFiles);
376 11
        $this->context = $oldContext;
377
378 11
        return $output;
379
    }
380
381
    /**
382
     * Returns the localized version of a specified file.
383
     *
384
     * The searching is based on the specified language code. In particular, a file with the same name will be looked
385
     * for under the subdirectory whose name is the same as the language code. For example, given the file
386
     * "path/to/view.php" and language code "zh-CN", the localized file will be looked for as path/to/zh-CN/view.php".
387
     * If the file is not found, it will try a fallback with just a language code that is "zh" i.e. "path/to/zh/view.php".
388
     * If it is not found as well the original file will be returned.
389
     *
390
     * If the target and the source language codes are the same, the original file will be returned.
391
     *
392
     * @param string $file the original file
393
     * @param string|null $language the target language that the file should be localized to.
394
     * @param string|null $sourceLanguage the language that the original file is in.
395
     *
396
     * @return string the matching localized file, or the original file if the localized version is not found.
397
     * If the target and the source language codes are the same, the original file will be returned.
398
     */
399 12
    public function localize(string $file, ?string $language = null, ?string $sourceLanguage = null): string
400
    {
401 12
        $language = $language ?? $this->language;
402 12
        $sourceLanguage = $sourceLanguage ?? $this->sourceLanguage;
403
404 12
        if ($language === $sourceLanguage) {
405 12
            return $file;
406
        }
407 1
        $desiredFile = dirname($file) . DIRECTORY_SEPARATOR . $language . DIRECTORY_SEPARATOR . basename($file);
408 1
        if (is_file($desiredFile)) {
409 1
            return $desiredFile;
410
        }
411
412
        $language = substr($language, 0, 2);
413
        if ($language === $sourceLanguage) {
414
            return $file;
415
        }
416
        $desiredFile = dirname($file) . DIRECTORY_SEPARATOR . $language . DIRECTORY_SEPARATOR . basename($file);
417
418
        return is_file($desiredFile) ? $desiredFile : $file;
419
    }
420
421
    /**
422
     * @return bool|string the view file currently being rendered. False if no view file is being rendered.
423
     */
424 7
    public function getViewFile()
425
    {
426 7
        return empty($this->viewFiles) ? false : end($this->viewFiles)['resolved'];
427
    }
428
429
    /**
430
     * @return bool|string the requested view currently being rendered. False if no view file is being rendered.
431
     */
432 1
    protected function getRequestedViewFile()
433
    {
434 1
        return empty($this->viewFiles) ? false : end($this->viewFiles)['requested'];
435
    }
436
437
    /**
438
     * This method is invoked right before {@see renderFile()} renders a view file.
439
     *
440
     * The default implementation will trigger the {@see BeforeRender()} event. If you override this method, make sure
441
     * you call the parent implementation first.
442
     *
443
     * @param string $viewFile the view file to be rendered.
444
     * @param array $parameters the parameter array passed to the {@see render()} method.
445
     *
446
     * @return bool whether to continue rendering the view file.
447
     */
448 11
    public function beforeRender(string $viewFile, array $parameters): bool
449
    {
450 11
        $event = new BeforeRender($viewFile, $parameters);
451 11
        $event = $this->eventDispatcher->dispatch($event);
452
453 11
        return !$event->isPropagationStopped();
454
    }
455
456
    /**
457
     * This method is invoked right after {@see renderFile()} renders a view file.
458
     *
459
     * The default implementation will trigger the {@see AfterRender} event. If you override this method, make sure you
460
     * call the parent implementation first.
461
     *
462
     * @param string $viewFile the view file being rendered.
463
     * @param array $parameters the parameter array passed to the {@see render()} method.
464
     * @param string $output the rendering result of the view file.
465
     *
466
     * @return string Updated output. It will be passed to {@see renderFile()} and returned.
467
     */
468 11
    public function afterRender(string $viewFile, array $parameters, string $output): string
469
    {
470 11
        $event = new AfterRender($viewFile, $parameters, $output);
471 11
        $event = $this->eventDispatcher->dispatch($event);
472
473 11
        return $event->getResult();
474
    }
475
476
    /**
477
     * Renders dynamic content returned by the given PHP statements.
478
     *
479
     * This method is mainly used together with content caching (fragment caching and page caching) when some portions
480
     * of the content (called *dynamic content*) should not be cached. The dynamic content must be returned by some PHP
481
     * statements. You can optionally pass additional parameters that will be available as variables in the PHP
482
     * statement:.
483
     *
484
     * ```php
485
     * <?= $this->renderDynamic('return foo($myVar);', [
486
     *     'myVar' => $model->getMyComplexVar(),
487
     * ]) ?>
488
     * ```
489
     *
490
     * @param string $statements the PHP statements for generating the dynamic content.
491
     * @param array $parameters the parameters (name-value pairs) that will be extracted and made
492
     * available in the $statement context. The parameters will be stored in the cache and be reused
493
     * each time $statement is executed. You should make sure, that these are safely serializable.
494
     *
495
     * @return string the placeholder of the dynamic content, or the dynamic content if there is no active content
496
     *                cache currently.
497
     */
498
    public function renderDynamic(string $statements, array $parameters = []): string
499
    {
500
        if (!empty($parameters)) {
501
            $statements = 'extract(unserialize(\'' . str_replace(['\\', '\''], ['\\\\', '\\\''], serialize($parameters)) . '\'));' . $statements;
502
        }
503
504
        if (!empty($this->cacheStack)) {
505
            $n = count($this->dynamicPlaceholders);
506
            $placeholder = "<![CDATA[YII-DYNAMIC-$n-{$this->getPlaceholderSignature()}]]>";
507
            $this->addDynamicPlaceholder($placeholder, $statements);
508
509
            return $placeholder;
510
        }
511
512
        return $this->evaluateDynamicContent($statements);
513
    }
514
515
    /**
516
     * Get source locale.
517
     *
518
     * @return Locale
519
     */
520
    public function getSourceLocale(): Locale
521
    {
522
        if ($this->sourceLocale === null) {
523
            $this->sourceLocale = new Locale('en-US');
524
        }
525
526
        return $this->sourceLocale;
0 ignored issues
show
Bug Best Practice introduced by
The expression return $this->sourceLocale could return the type null which is incompatible with the type-hinted return Yiisoft\I18n\Locale. Consider adding an additional type-check to rule them out.
Loading history...
527
    }
528
529
    /**
530
     * Set source locale.
531
     *
532
     * @param string $locale
533
     *
534
     * @return self
535
     */
536
    public function setSourceLocale(string $locale): self
537
    {
538
        $this->sourceLocale = new Locale($locale);
539
540
        return $this;
541
    }
542
543
    public function getDynamicPlaceholders(): array
544
    {
545
        return $this->dynamicPlaceholders;
546
    }
547
548
    public function setDynamicPlaceholders(array $placeholders): void
549
    {
550
        $this->dynamicPlaceholders = $placeholders;
551
    }
552
553
    public function addDynamicPlaceholder(string $name, string $statements): void
554
    {
555
        foreach ($this->cacheStack as $cache) {
556
            $cache->addDynamicPlaceholder($name, $statements);
557
        }
558
559
        $this->dynamicPlaceholders[$name] = $statements;
560
    }
561
562
    /**
563
     * Evaluates the given PHP statements.
564
     *
565
     * This method is mainly used internally to implement dynamic content feature.
566
     *
567
     * @param string $statements the PHP statements to be evaluated.
568
     *
569
     * @return mixed the return value of the PHP statements.
570
     */
571
    public function evaluateDynamicContent(string $statements)
572
    {
573
        return eval($statements);
0 ignored issues
show
introduced by
The use of eval() is discouraged.
Loading history...
574
    }
575
576
    /**
577
     * Returns a list of currently active dynamic content class instances.
578
     *
579
     * @return DynamicContentAwareInterface[] class instances supporting dynamic contents.
580
     */
581
    public function getDynamicContents(): array
582
    {
583
        return $this->cacheStack;
584
    }
585
586
    /**
587
     * Adds a class instance supporting dynamic contents to the end of a list of currently active dynamic content class
588
     * instances.
589
     *
590
     * @param DynamicContentAwareInterface $instance class instance supporting dynamic contents.
591
     */
592
    public function pushDynamicContent(DynamicContentAwareInterface $instance): void
593
    {
594
        $this->cacheStack[] = $instance;
595
    }
596
597
    /**
598
     * Removes a last class instance supporting dynamic contents from a list of currently active dynamic content class
599
     * instances.
600
     */
601
    public function popDynamicContent(): void
602
    {
603
        array_pop($this->cacheStack);
604
    }
605
606
    /**
607
     * Marks the beginning of a page.
608
     */
609 7
    public function beginPage(): void
610
    {
611 7
        ob_start();
612 7
        PHP_VERSION_ID >= 80000 ? ob_implicit_flush(false) : ob_implicit_flush(0);
613
614 7
        $this->eventDispatcher->dispatch(new PageBegin($this->getViewFile()));
0 ignored issues
show
Bug introduced by
It seems like $this->getViewFile() can also be of type boolean; however, parameter $file of Yiisoft\View\Event\PageBegin::__construct() does only seem to accept string, maybe add an additional type check? ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-type  annotation

614
        $this->eventDispatcher->dispatch(new PageBegin(/** @scrutinizer ignore-type */ $this->getViewFile()));
Loading history...
615 7
    }
616
617
    /**
618
     * Marks the ending of a page.
619
     */
620
    public function endPage(): void
621
    {
622
        $this->eventDispatcher->dispatch(new PageEnd($this->getViewFile()));
0 ignored issues
show
Bug introduced by
It seems like $this->getViewFile() can also be of type boolean; however, parameter $file of Yiisoft\View\Event\PageEnd::__construct() does only seem to accept string, maybe add an additional type check? ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-type  annotation

622
        $this->eventDispatcher->dispatch(new PageEnd(/** @scrutinizer ignore-type */ $this->getViewFile()));
Loading history...
623
        ob_end_flush();
624
    }
625
}
626