Remove unused local variables
authorMatthias Schmidt <gravatronics@live.com>
Mon, 10 May 2021 13:50:37 +0000 (15:50 +0200)
committerMatthias Schmidt <gravatronics@live.com>
Mon, 10 May 2021 13:50:37 +0000 (15:50 +0200)
31 files changed:
wcfsetup/install/files/lib/acp/form/ApplicationManagementForm.class.php
wcfsetup/install/files/lib/acp/form/LabelGroupAddForm.class.php
wcfsetup/install/files/lib/acp/form/LabelGroupEditForm.class.php
wcfsetup/install/files/lib/acp/form/NotificationPresetSettingsForm.class.php
wcfsetup/install/files/lib/acp/page/UserListPage.class.php
wcfsetup/install/files/lib/data/cronjob/CronjobAction.class.php
wcfsetup/install/files/lib/data/option/OptionAction.class.php
wcfsetup/install/files/lib/data/package/update/PackageUpdateAction.class.php
wcfsetup/install/files/lib/form/RegisterForm.class.php
wcfsetup/install/files/lib/system/WCFSetup.class.php
wcfsetup/install/files/lib/system/bbcode/HtmlBBCodeParser.class.php
wcfsetup/install/files/lib/system/cache/builder/TemplateListenerCodeCacheBuilder.class.php
wcfsetup/install/files/lib/system/cache/builder/UserGroupPermissionCacheBuilder.class.php
wcfsetup/install/files/lib/system/comment/CommentHandler.class.php
wcfsetup/install/files/lib/system/condition/AbstractSelectCondition.class.php
wcfsetup/install/files/lib/system/devtools/pip/DevtoolsPip.class.php
wcfsetup/install/files/lib/system/form/builder/field/TSelectionFormField.class.php
wcfsetup/install/files/lib/system/html/input/node/HtmlInputNodeProcessor.class.php
wcfsetup/install/files/lib/system/html/input/node/HtmlInputNodeWoltlabMetacodeMarker.class.php
wcfsetup/install/files/lib/system/message/quote/MessageQuoteManager.class.php
wcfsetup/install/files/lib/system/package/FilesFileHandler.class.php
wcfsetup/install/files/lib/system/package/PackageInstallationDispatcher.class.php
wcfsetup/install/files/lib/system/package/PackageInstallationNodeBuilder.class.php
wcfsetup/install/files/lib/system/package/PackageInstallationScheduler.class.php
wcfsetup/install/files/lib/system/package/PackageUpdateDispatcher.class.php
wcfsetup/install/files/lib/system/package/plugin/PagePackageInstallationPlugin.class.php
wcfsetup/install/files/lib/system/request/LinkHandler.class.php
wcfsetup/install/files/lib/system/search/SearchResultTextParser.class.php
wcfsetup/install/files/lib/system/style/StyleCompiler.class.php
wcfsetup/install/files/lib/util/FileUtil.class.php
wcfsetup/install/files/lib/util/HTTPRequest.class.php

index f9c8e0a520c895468a8c6642edc9d05f6decaac2..01180e0a849f63d683331a25fa64ca2f1f95efc7 100644 (file)
@@ -129,7 +129,7 @@ final class ApplicationManagementForm extends AbstractForm
             }
         }
 
-        foreach ($this->landingPageID as $packageID => $landingPageID) {
+        foreach ($this->landingPageID as $landingPageID) {
             if (!$landingPageID) {
                 continue;
             }
index ce9abbddfbf3a826094326095283a837921254fa..dc56be22757058ac81cbd171b8219a7726798112 100644 (file)
@@ -208,7 +208,7 @@ class LabelGroupAddForm extends AbstractForm
         // save object type relations
         $this->saveObjectTypeRelations($returnValues['returnValues']->groupID);
 
-        foreach ($this->labelObjectTypes as $objectTypeID => $labelObjectType) {
+        foreach ($this->labelObjectTypes as $labelObjectType) {
             $labelObjectType->save();
         }
 
index b907a3eab84edaee8d7c8eab17a9b236272f7383..e7f278fbb1990d1b5a19fc00d3821256f0c85a3a 100644 (file)
@@ -95,7 +95,7 @@ class LabelGroupEditForm extends LabelGroupAddForm
         // update object type relations
         $this->saveObjectTypeRelations($this->groupID);
 
-        foreach ($this->labelObjectTypes as $objectTypeID => $labelObjectType) {
+        foreach ($this->labelObjectTypes as $labelObjectType) {
             $labelObjectType->save();
         }
 
index d1bc76cfcd8e97023d82064857832cb750203491..dbf2770516899acc0a047b0fd9fe1f0cbc748e76 100644 (file)
@@ -184,7 +184,7 @@ class NotificationPresetSettingsForm extends AbstractForm
     {
         parent::save();
 
-        foreach ($this->events as $objectType => $events) {
+        foreach ($this->events as $events) {
             foreach ($events as $event) {
                 $preset = 0;
                 $presetMailNotificationType = 'none';
index 0592ee48923a0f2a27aa79c3d14ef96dad39ebad..2fef97bcabcf0a7b6f08ad7e888007c466dd23f0 100755 (executable)
@@ -313,7 +313,7 @@ class UserListPage extends SortablePage
             }
 
             // get special columns
-            foreach ($this->users as $key => $user) {
+            foreach ($this->users as $user) {
                 foreach ($this->columns as $column) {
                     switch ($column) {
                         case 'email':
index 00a316f1f7d586371a517284c067d1758aba4d44..fdc3e6678c917ca1e45c8406b186b62d04ca4568 100644 (file)
@@ -122,7 +122,7 @@ class CronjobAction extends AbstractDatabaseObjectAction implements IToggleActio
         WCF::getSession()->disableUpdate();
 
         try {
-            foreach ($this->getObjects() as $key => $cronjob) {
+            foreach ($this->getObjects() as $cronjob) {
                 // mark them as pending
                 $cronjob->update(['state' => Cronjob::PENDING]);
             }
index 661dd51d38ee925ee2d4eb4b2d01193088d4bbc4..8a48a9e17d265bf76cdec3f299fd8b90ac70ed56 100644 (file)
@@ -189,7 +189,7 @@ class OptionAction extends AbstractDatabaseObjectAction
         foreach ($dirs as $dir => $domainPaths) {
             \krsort($domainPaths);
 
-            foreach ($domainPaths as $domainPath => $value) {
+            foreach ($domainPaths as $value) {
                 $htaccess = "{$dir}.htaccess";
                 $path = FileUtil::addTrailingSlash(\substr($value, \strlen($dir)));
                 if ($path == '/') {
index 9d1bd4d3eb6299660219fd2e72178e4bea07bd46..ba28c6178779e0c1be95e95c9748652839e3ded7 100644 (file)
@@ -203,7 +203,7 @@ class PackageUpdateAction extends AbstractDatabaseObjectAction
         // remove duplicates when there are both versions from trusted and untrusted servers
         foreach ($possiblePackages as $identifier => $packageSources) {
             $hasTrustedSource = false;
-            foreach ($packageSources as $packageUpdateID => $packageUpdateServerID) {
+            foreach ($packageSources as $packageUpdateServerID) {
                 if (\in_array($packageUpdateServerID, $trustedServerIDs)) {
                     $hasTrustedSource = true;
                     break;
@@ -222,7 +222,7 @@ class PackageUpdateAction extends AbstractDatabaseObjectAction
 
         // Sort by the highest version and return all other sources for the same package.
         $validPackageUpdateIDs = [];
-        foreach ($possiblePackages as $identifier => $packageSources) {
+        foreach ($possiblePackages as $packageSources) {
             if (\count($packageSources) > 1) {
                 $packageUpdateVersionIDs = [];
                 foreach (\array_keys($packageSources) as $packageUpdateID) {
@@ -395,7 +395,7 @@ class PackageUpdateAction extends AbstractDatabaseObjectAction
 
         // determine highest versions
         $packageUpdates = [];
-        foreach ($packageVersions as $package => $versionData) {
+        foreach ($packageVersions as $versionData) {
             $accessible = $existing = $versions = [];
 
             foreach ($versionData as $packageUpdateID => $versionTypes) {
@@ -550,7 +550,7 @@ class PackageUpdateAction extends AbstractDatabaseObjectAction
          * @var int $packageUpdateID
          * @var ViewablePackageUpdate $packageUpdate
          */
-        foreach ($packageUpdates as $packageUpdateID => $packageUpdate) {
+        foreach ($packageUpdates as $packageUpdate) {
             $versionIDs = $updateData[$packageUpdate->packageUpdateID];
             $packageUpdate->setAccessibleVersion($updateVersions[$versionIDs['accessible']]);
             $packageUpdate->setLatestVersion($updateVersions[$versionIDs['existing']]);
index f339920f507bf8a2db6741f2e9d29b48ba993cf2..69bea557abd5bfb98857a17cf1433d7bd2b73353 100644 (file)
@@ -452,7 +452,6 @@ class RegisterForm extends UserAddForm
         $result = $this->objectAction->executeAction();
         /** @var User $user */
         $user = $result['returnValues'];
-        $userEditor = new UserEditor($user);
 
         // update session
         WCF::getSession()->changeUser($user);
index 5aaafda3fecc15986124d45f7051d55f61194018..5d6bc90e76c82decf70930dba27fc5e152c5eb2e 100644 (file)
@@ -199,7 +199,7 @@ class WCFSetup extends WCF
      */
     protected static function getAvailableLanguages()
     {
-        $languages = $match = [];
+        $languages = [];
         foreach (\glob(TMP_DIR . 'setup/lang/*.xml') as $file) {
             $xml = new XML();
             $xml->load($file);
index 7289e84dabc48db56fd95a158cecc201442c5843..02d864eec55bff0cb4d2a9dd302a9b9ce7696136 100644 (file)
@@ -482,7 +482,7 @@ class HtmlBBCodeParser extends BBCodeParser
             return $tag['source'];
         }
 
-        $index = (isset($tag['bufferPlaceholder'])) ? $index = $tag['bufferPlaceholder'] : \count($this->openTagIdentifiers);
+        $index = (isset($tag['bufferPlaceholder'])) ? $tag['bufferPlaceholder'] : \count($this->openTagIdentifiers);
 
         $uuid = StringUtil::getUUID();
         $this->openTagIdentifiers[$index] = [
index 20b03b97103b7f79664e3560d7ea39d36a78fad4..6fc42e2b6f1093175145899d653a5815cf53605f 100644 (file)
@@ -40,7 +40,7 @@ class TemplateListenerCodeCacheBuilder extends AbstractCacheBuilder
             if ($templateListener->options || $templateListener->permissions) {
                 $templateCode = '{if ';
 
-                $options = $permissions = [];
+                $options = [];
                 if ($templateListener->options) {
                     $options = \explode(',', \strtoupper($templateListener->options));
 
index 02db02e7b02c013d01dd736bb4252e3d388a21eb..80d91ca3c396db9b54c44e1e3e57111de858a05b 100644 (file)
@@ -69,7 +69,7 @@ class UserGroupPermissionCacheBuilder extends AbstractCacheBuilder
             $optionData[$row['groupID']][$row['optionName']] = $row;
         }
 
-        foreach ($optionData as $groupID => $options) {
+        foreach ($optionData as $options) {
             $optionBlacklist = [];
 
             foreach ($options as $option) {
index 8d62ed74a227c7be09efed834f2e47e866b64ce4..288c63280bc5f3a0a944678011300c0ab8ddf527 100644 (file)
@@ -493,7 +493,7 @@ class CommentHandler extends SingletonFactory
         }
 
         if (!empty($commentEvents)) {
-            foreach ($commentEvents as $eventID => $eventData) {
+            foreach ($commentEvents as $eventData) {
                 UserNotificationHandler::getInstance()->markAsConfirmed(
                     $eventData['eventName'],
                     $eventData['objectType'],
@@ -572,7 +572,7 @@ class CommentHandler extends SingletonFactory
             }
 
             if (!empty($responseEvents)) {
-                foreach ($responseEvents as $eventID => $eventData) {
+                foreach ($responseEvents as $eventData) {
                     UserNotificationHandler::getInstance()->markAsConfirmed(
                         $eventData['eventName'],
                         $eventData['objectType'],
@@ -672,7 +672,7 @@ class CommentHandler extends SingletonFactory
         }
 
         if (!empty($responseEvents)) {
-            foreach ($responseEvents as $eventID => $eventData) {
+            foreach ($responseEvents as $eventData) {
                 UserNotificationHandler::getInstance()->markAsConfirmed(
                     $eventData['eventName'],
                     $eventData['objectType'],
index c358662aa31e46588072015453bc70bf9ebfd757..c92cffae6e4696da3f8ced410911653fdbb032e5 100644 (file)
@@ -149,7 +149,7 @@ abstract class AbstractSelectCondition extends AbstractSingleFieldCondition
             $options = $this->getOptions();
 
             if (!isset($options[$this->fieldValue])) {
-                foreach ($options as $key => $value) {
+                foreach ($options as $value) {
                     if (\is_array($value) && isset($value[$this->fieldValue])) {
                         return;
                     }
index da8f2c4dabf54857126d1f31c61acb1a9cb63b6b..a1a0bd3b29e6f8d33b26903b66bd28fab9ad1714 100644 (file)
@@ -331,7 +331,7 @@ class DevtoolsPip extends DatabaseObjectDecorator
                         });
 
                         $iterator = new \RecursiveIteratorIterator($filter, \RecursiveIteratorIterator::SELF_FIRST);
-                        foreach ($iterator as $value => $item) {
+                        foreach ($iterator as $item) {
                             /** @var \SplFileInfo $item */
                             $itemPath = $item->getRealPath();
                             if (\is_dir($itemPath)) {
@@ -410,7 +410,7 @@ class DevtoolsPip extends DatabaseObjectDecorator
                         });
 
                         $iterator = new \RecursiveIteratorIterator($filter, \RecursiveIteratorIterator::SELF_FIRST);
-                        foreach ($iterator as $value => $item) {
+                        foreach ($iterator as $item) {
                             /** @var \SplFileInfo $item */
                             $itemPath = $item->getRealPath();
                             if (\is_dir($itemPath)) {
index 6b54b6ca03be63589eac9cb881db51e2155ed8c7..07a7276ca0d2a691a15a599798b01d49d50bd0b6 100644 (file)
@@ -139,7 +139,7 @@ trait TSelectionFormField
 
             if ($nestedOptions) {
                 $collectedOptions = [];
-                foreach ($options as $key => $object) {
+                foreach ($options as $object) {
                     if (!($object instanceof IObjectTreeNode)) {
                         throw new \InvalidArgumentException(
                             "Nested traversable options must implement '" . IObjectTreeNode::class . "'."
index e7ac9b3169af60f96a356bf0f5ee3cdd7f14f931..14c910f22fb9c4d62de9245be0090c5ce9115894 100644 (file)
@@ -540,7 +540,6 @@ class HtmlInputNodeProcessor extends AbstractHtmlNodeProcessor
         }
 
         if (!empty($inlineStyles)) {
-            $styles = [];
             /** @var \DOMElement $element */
             foreach ($this->getXPath()->query('//*[@style]') as $element) {
                 $tmp = \array_filter(\explode(';', $element->getAttribute('style')));
index f91c128c69463fd2581554df11a5b254be9cea0e..4c77a5975059d7d739d9d38fd3730772134339ff 100644 (file)
@@ -253,7 +253,7 @@ class HtmlInputNodeWoltlabMetacodeMarker extends AbstractHtmlInputNode
     protected function groupPairsByName(array $pairs)
     {
         $groups = [];
-        foreach ($pairs as $uuid => $data) {
+        foreach ($pairs as $data) {
             $name = $data['name'];
 
             if (!isset($groups[$name])) {
index b1523403dcc15555d838e17b90c2197a7f5c772a..cbb4201836fe532296b33fa27f8f006afffbef1d 100644 (file)
@@ -521,7 +521,7 @@ class MessageQuoteManager extends SingletonFactory
             $objectIDs[$objectType] = [];
             if (isset($this->quotes[$objectType])) {
                 foreach ($this->quotes[$objectType] as $objectID => $quotes) {
-                    foreach ($quotes as $quoteID => $isFullQuote) {
+                    foreach ($quotes as $isFullQuote) {
                         if ($isFullQuote) {
                             $objectIDs[$objectType][] = $objectID;
                             break;
index f21b85d660110f395aac2539aacd979ebc114997..2133591bd0cf84ea12e77b6c74a54568ad6f92e0 100644 (file)
@@ -40,7 +40,7 @@ class FilesFileHandler extends PackageInstallationFileHandler
 
                 // check delivered files
                 if (!empty($lockedFiles)) {
-                    foreach ($files as $key => $file) {
+                    foreach ($files as $file) {
                         if (isset($lockedFiles[$file])) {
                             $owningPackage = new Package($lockedFiles[$file]);
 
index 1fcc166f74b5cfc93de1e6a053f5504eef004c9f..edb123972f7de14d59e2788cf0713e5c9d3ae752 100644 (file)
@@ -606,7 +606,7 @@ class PackageInstallationDispatcher
                     VALUES      (?, ?)";
             $statement = WCF::getDB()->prepareStatement($sql);
 
-            foreach ($requirements as $identifier => $possibleRequirements) {
+            foreach ($requirements as $possibleRequirements) {
                 $requirement = \array_shift($possibleRequirements);
 
                 $statement->execute([
index 8ddd35d2554510482f067d5634aaff2e6320e9f3..833326405bc37fc2a55aa58aecd1f87386ca200f 100644 (file)
@@ -645,7 +645,7 @@ class PackageInstallationNodeBuilder
                     VALUES      (?, ?, ?, ?, ?, ?, ?)";
             $statement = WCF::getDB()->prepareStatement($sql);
 
-            foreach ($pluginNodes as $index => $nodeData) {
+            foreach ($pluginNodes as $nodeData) {
                 $statement->execute([
                     $this->installation->queue->queueID,
                     $this->installation->queue->processNo,
index b4fd2af82d221f716097afc832f16aa78407dd48..fd177776b9e168df3025d0cc27796bd0af4664e5 100644 (file)
@@ -450,7 +450,7 @@ class PackageInstallationScheduler
             $statement = WCF::getDB()->prepareStatement($sql);
             $statement->execute($conditions->getParameters());
             while ($row = $statement->fetchArray()) {
-                foreach ($packageInstallations as $key => $packageInstallation) {
+                foreach ($packageInstallations as $packageInstallation) {
                     if ($packageInstallation['package'] == $row['excludedPackage']) {
                         if (!empty($row['excludedPackageVersion'])) {
                             // check version
index e52383e9586ccd1c3fc14a3daf84ad9a910a096b..bf1e454cfc9a9bd41ad7a71cfcfefd0dd15e7f00 100644 (file)
@@ -841,7 +841,7 @@ class PackageUpdateDispatcher extends SingletonFactory
 
         // remove requirements of application packages
         if ($removeRequirements) {
-            foreach ($existingPackages as $identifier => $instances) {
+            foreach ($existingPackages as $instances) {
                 foreach ($instances as $instance) {
                     if ($instance['isApplication'] && isset($updates[$instance['packageID']])) {
                         $updates = $this->removeUpdateRequirements(
index 77abb43879fd8838b5bdbb6c1fb7fa0a7304fd93..76d6050df7b45abc57ad510134274ad96fe20012 100644 (file)
@@ -401,7 +401,7 @@ class PagePackageInstallationPlugin extends AbstractXMLPackageInstallationPlugin
             SearchIndexManager::getInstance()->createSearchIndices();
 
             // update search index
-            foreach ($this->pages as $pageID => $page) {
+            foreach ($this->pages as $page) {
                 if ($page->pageType == 'text' || $page->pageType == 'html') {
                     foreach ($page->getPageContents() as $languageID => $pageContent) {
                         SearchIndexManager::getInstance()->set(
index 295c02cb87593c4453abd0801bfa9bfb3c9a72ec..dd6ca13080e023d158c3ee6df671602c1f1c7d28 100644 (file)
@@ -116,7 +116,7 @@ class LinkHandler extends SingletonFactory
     {
         $abbreviation = 'wcf';
         $anchor = '';
-        $isACP = $originIsACP = RequestHandler::getInstance()->isACPRequest();
+        $isACP = RequestHandler::getInstance()->isACPRequest();
         $isRaw = false;
         $encodeTitle = true;
 
index 061f1991feedbcd6519ff09897e853b201f6b9f8..d355d686231db79579c26532d2fe1177c6bcba88 100644 (file)
@@ -142,7 +142,7 @@ class SearchResultTextParser extends SingletonFactory
                     $newText = '';
                     $i = 0;
                     $length = \count($matches);
-                    foreach ($matches as $keyword => $position) {
+                    foreach ($matches as $position) {
                         if ($position['start'] < $start) {
                             $position['end'] += $start - $position['start'];
                             $position['start'] = $start;
index c1d925c2133f5a13ccf29c791e79f8ffcfed5594..cc236cc284bb174cabf0d70204d015d97cf5c4ca 100644 (file)
@@ -98,8 +98,6 @@ final class StyleCompiler extends SingletonFactory
      */
     public static function getDefaultVariables()
     {
-        $variables = [];
-
         $sql = "SELECT      variable.variableName, variable.defaultValue
                 FROM        wcf" . WCF_N . "_style_variable variable
                 ORDER BY    variable.variableID ASC";
index 7fe477a814a225caec756a692fc9fb6c4f3eaf6a..05b8eab8ea5394a8f822329f7037daea627c6084 100644 (file)
@@ -339,7 +339,7 @@ final class FileUtil
             $result[] = '';
         }
 
-        foreach ($pathA as $key => $dir) {
+        foreach ($pathA as $dir) {
             if ($dir == '..') {
                 if (\end($result) == '..') {
                     $result[] = '..';
index 2b14b8e73502b3f86874fc8cc072c8c56a7f6015..b4a1284de3c71895a32e9c100754f331f320a68b 100644 (file)
@@ -129,7 +129,7 @@ final class HTTPRequest
                         new \RecursiveArrayIterator($this->postParameters),
                         \RecursiveIteratorIterator::SELF_FIRST
                     );
-                    foreach ($iterator as $k => $v) {
+                    foreach ($iterator as $v) {
                         /** @noinspection PhpUndefinedMethodInspection */
                         if (!$iterator->hasChildren()) {
                             $key = '';