From 6a349ccf5ab13851ec6457d253529caab3985111 Mon Sep 17 00:00:00 2001 From: oleibman <10341515+oleibman@users.noreply.github.com> Date: Mon, 21 Mar 2022 14:29:57 -0700 Subject: [PATCH] Scrutinizer Faulty Analysis (#2704) Despite typehint, Scrutinizer assigns union type to a variable, resulting in subsequent faulty analysis of a method call invoked on that variable. Adding a bogus routine that will never actually be called to one module eliminates 8 Scrutinizer errors. --- src/PhpSpreadsheet/Spreadsheet.php | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/src/PhpSpreadsheet/Spreadsheet.php b/src/PhpSpreadsheet/Spreadsheet.php index 2b8c8360..52d7fb55 100644 --- a/src/PhpSpreadsheet/Spreadsheet.php +++ b/src/PhpSpreadsheet/Spreadsheet.php @@ -1602,4 +1602,14 @@ class Spreadsheet } } } + + /** + * Silliness to mollify Scrutinizer. + * + * @codeCoverageIgnore + */ + public function getSharedComponent(): Style + { + return new Style(); + } }