From 131708409b7a48949838e13bf528f1dca7947fc7 Mon Sep 17 00:00:00 2001
From: oleibman <10341515+oleibman@users.noreply.github.com>
Date: Thu, 25 Aug 2022 23:22:59 -0700
Subject: [PATCH] Correct Very Minor Error / Php8.2 Deprecation (#3021)

When Reader/Xlsx/WorkbookView was split off from Reader/Xlsx.php, one statement accidentally brought `!empty($this->loadSheetsOnly)` with it. That property does not exist in WorkbookView, so the test is useless (it is always empty); and, in fact, the caller passes its own version of loadSheetsOnly as a parameter, so it isn't needed even it did exist. In Php8.2, this might be a deprecation, although it hasn't shown up in the GitHub 8.2 tests. Fix it anyhow.
---
 src/PhpSpreadsheet/Reader/Xlsx/WorkbookView.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/PhpSpreadsheet/Reader/Xlsx/WorkbookView.php b/src/PhpSpreadsheet/Reader/Xlsx/WorkbookView.php
index 9d61e3d3..4743afbf 100644
--- a/src/PhpSpreadsheet/Reader/Xlsx/WorkbookView.php
+++ b/src/PhpSpreadsheet/Reader/Xlsx/WorkbookView.php
@@ -29,7 +29,7 @@ class WorkbookView
         $this->spreadsheet->setActiveSheetIndex(0);
 
         $workbookView = $xmlWorkbook->children($mainNS)->bookViews->workbookView;
-        if (($readDataOnly !== true || !empty($this->loadSheetsOnly)) && !empty($workbookView)) {
+        if ($readDataOnly !== true && !empty($workbookView)) {
             $workbookViewAttributes = self::testSimpleXml(self::getAttributes($workbookView));
             // active sheet index
             $activeTab = (int) $workbookViewAttributes->activeTab; // refers to old sheet index