Remove return types from PIP GUI API
authorMatthias Schmidt <gravatronics@live.com>
Sat, 21 Jul 2018 09:02:33 +0000 (11:02 +0200)
committerMatthias Schmidt <gravatronics@live.com>
Sat, 21 Jul 2018 09:02:33 +0000 (11:02 +0200)
See #2545
See #2720

24 files changed:
wcfsetup/install/files/lib/system/devtools/pip/DevtoolsPipEntryList.class.php
wcfsetup/install/files/lib/system/devtools/pip/IDevtoolsPipEntryList.class.php
wcfsetup/install/files/lib/system/devtools/pip/TXmlGuiPackageInstallationPlugin.class.php
wcfsetup/install/files/lib/system/form/builder/field/OptionFormField.class.php
wcfsetup/install/files/lib/system/form/builder/field/UserGroupOptionFormField.class.php
wcfsetup/install/files/lib/system/package/plugin/ACLOptionPackageInstallationPlugin.class.php
wcfsetup/install/files/lib/system/package/plugin/ACPMenuPackageInstallationPlugin.class.php
wcfsetup/install/files/lib/system/package/plugin/ACPSearchProviderPackageInstallationPlugin.class.php
wcfsetup/install/files/lib/system/package/plugin/AbstractMenuPackageInstallationPlugin.class.php
wcfsetup/install/files/lib/system/package/plugin/ClipboardActionPackageInstallationPlugin.class.php
wcfsetup/install/files/lib/system/package/plugin/CoreObjectPackageInstallationPlugin.class.php
wcfsetup/install/files/lib/system/package/plugin/CronjobPackageInstallationPlugin.class.php
wcfsetup/install/files/lib/system/package/plugin/EventListenerPackageInstallationPlugin.class.php
wcfsetup/install/files/lib/system/package/plugin/MediaProviderPackageInstallationPlugin.class.php
wcfsetup/install/files/lib/system/package/plugin/MenuItemPackageInstallationPlugin.class.php
wcfsetup/install/files/lib/system/package/plugin/MenuPackageInstallationPlugin.class.php
wcfsetup/install/files/lib/system/package/plugin/ObjectTypeDefinitionPackageInstallationPlugin.class.php
wcfsetup/install/files/lib/system/package/plugin/ObjectTypePackageInstallationPlugin.class.php
wcfsetup/install/files/lib/system/package/plugin/PIPPackageInstallationPlugin.class.php
wcfsetup/install/files/lib/system/package/plugin/PagePackageInstallationPlugin.class.php
wcfsetup/install/files/lib/system/package/plugin/TemplateListenerPackageInstallationPlugin.class.php
wcfsetup/install/files/lib/system/package/plugin/UserMenuPackageInstallationPlugin.class.php
wcfsetup/install/files/lib/system/package/plugin/UserNotificationEventPackageInstallationPlugin.class.php
wcfsetup/install/files/lib/system/package/plugin/UserProfileMenuPackageInstallationPlugin.class.php

index dd814a3e2dee7ebe0e2f536663693cab3afd0e67..9b7f966d09c5a61de695e4baba197746953e67be 100644 (file)
@@ -56,14 +56,14 @@ class DevtoolsPipEntryList implements IDevtoolsPipEntryList {
        /**
         * @inheritDoc
         */
-       public function getEntries(): array {
+       public function getEntries() {
                return $this->entries;
        }
        
        /**
         * @inheritDoc
         */
-       public function getKeys(): array {
+       public function getKeys() {
                if ($this->keys === null) {
                        throw new \BadMethodCallException("No keys have been set.");
                }
index 2c50d853bab4dc81ceadd980eefa5abb39fab830..320667482ccf861f7d60e139dde71c0e8f4895c1 100644 (file)
@@ -28,7 +28,7 @@ interface IDevtoolsPipEntryList {
         * 
         * @return      array
         */
-       public function getEntries(): array;
+       public function getEntries();
        
        /**
         * Returns the expected keys of the entries that can be used to display the
@@ -40,7 +40,7 @@ interface IDevtoolsPipEntryList {
         * @return      array
         * @throws      \BadMethodCallException         if no keys have been set
         */
-       public function getKeys(): array;
+       public function getKeys();
        
        /**
         * Sets the keys of the entries that can be used to display the entry list
index 8c8bd947c118730a891cb974d2f125a348851484..937763bc03aeb929d9641a365fe62f33b96076d4 100644 (file)
@@ -71,7 +71,7 @@ trait TXmlGuiPackageInstallationPlugin {
         * @param       string                  $identifier
         * @return      string                  new identifier
         */
-       public function editEntry(IFormDocument $form, string $identifier): string {
+       public function editEntry(IFormDocument $form, string $identifier) {
                $xml = $this->getProjectXml();
                $document = $xml->getDocument();
                
@@ -101,7 +101,7 @@ trait TXmlGuiPackageInstallationPlugin {
         * 
         * @return      string
         */
-       public function getAdditionalTemplateCode(): string {
+       public function getAdditionalTemplateCode() {
                return '';
        }
        
@@ -112,7 +112,7 @@ trait TXmlGuiPackageInstallationPlugin {
         * @param       string          $value
         * @return      string
         */
-       protected function getAutoCdataValue(string $value): string {
+       protected function getAutoCdataValue(string $value) {
                if (strpos('<', $value) !== false || strpos('>', $value) !== false || strpos('&', $value) !== false) {
                        $value = '<![CDATA[' . StringUtil::escapeCDATA($value) . ']]>';
                }
@@ -144,7 +144,7 @@ trait TXmlGuiPackageInstallationPlugin {
         * @param       bool            $saveData       is `true` if data is intended to be saved and otherwise `false`
         * @return      array
         */
-       abstract protected function getElementData(\DOMElement $element, bool $saveData = false): array;
+       abstract protected function getElementData(\DOMElement $element, bool $saveData = false);
        
        /**
         * Returns the identifier of the given `import` element.
@@ -152,7 +152,7 @@ trait TXmlGuiPackageInstallationPlugin {
         * @param       \DOMElement     $element
         * @return      string
         */
-       abstract protected function getElementIdentifier(\DOMElement $element): string;
+       abstract protected function getElementIdentifier(\DOMElement $element);
        
        /**
         * Returns the xml code of an empty xml file with the appropriate structure
@@ -161,7 +161,7 @@ trait TXmlGuiPackageInstallationPlugin {
         * 
         * @return      string
         */
-       protected function getEmptyXml(): string {
+       protected function getEmptyXml() {
                $xsdFilename = $this->getXsdFilename();
                
                return <<<XML
@@ -178,7 +178,7 @@ XML;
         * 
         * @return      string
         */
-       protected function getXsdFilename(): string {
+       protected function getXsdFilename() {
                $classNamePieces = explode('\\', get_class($this));
                
                return lcfirst(str_replace('PackageInstallationPlugin', '', array_pop($classNamePieces)));
@@ -189,7 +189,7 @@ XML;
         * 
         * @return      IDevtoolsPipEntryList
         */
-       public function getEntryList(): IDevtoolsPipEntryList {
+       public function getEntryList() {
                $xml = $this->getProjectXml();
                $xpath = $xml->xpath();
                
@@ -216,7 +216,7 @@ XML;
         * 
         * @return      string[]
         */
-       public function getEntryTypes(): array {
+       public function getEntryTypes() {
                return [];
        }
        
@@ -243,7 +243,7 @@ XML;
         * 
         * @return      XML
         */
-       protected function getProjectXml(): XML {
+       protected function getProjectXml() {
                $fileLocation = $this->getXmlFileLocation();
                
                $xml = new XML();
@@ -262,7 +262,7 @@ XML;
         * 
         * @return      string
         */
-       protected function getXmlFileLocation(): string {
+       protected function getXmlFileLocation() {
                /** @var DevtoolsProject $project */
                $project = $this->installation->getProject();
                
@@ -324,7 +324,7 @@ XML;
         * @param       IFormDocument           $document
         * @return      bool
         */
-       public function setEntryData(string $identifier, IFormDocument $document): bool {
+       public function setEntryData(string $identifier, IFormDocument $document) {
                $xml = $this->getProjectXml();
                
                $element = $this->getElementByIdentifier($xml, $identifier);
@@ -471,5 +471,5 @@ XML;
         * @param       IFormDocument           $form
         * @return      \DOMElement
         */
-       abstract protected function writeEntry(\DOMDocument $document, IFormDocument $form): \DOMElement
+       abstract protected function writeEntry(\DOMDocument $document, IFormDocument $form); 
 }
index c1c1ecdf7793786d182d19479fb7cffb4c7761a2..4c46f1c33309f39e4810167ec68c8ca4cbdf841b 100644 (file)
@@ -40,7 +40,7 @@ class OptionFormField extends ItemListFormField {
         * 
         * @return      int[]
         */
-       public function getPackageIDs(): array {
+       public function getPackageIDs() {
                return $this->__packageIDs;
        }
        
@@ -53,7 +53,7 @@ class OptionFormField extends ItemListFormField {
         * 
         * @throws      \InvalidArgumentException       if the given package ids are invalid
         */
-       public function packageIDs(array $packageIDs): OptionFormField {
+       public function packageIDs(array $packageIDs) {
                foreach ($packageIDs as $packageID) {
                        if (PackageCache::getInstance()->getPackage($packageID) === null) {
                                throw new \InvalidArgumentException("Unknown package with id '{$packageID}'.");
@@ -102,7 +102,7 @@ class OptionFormField extends ItemListFormField {
        /**
         * @inheritDoc
         */
-       protected static function getDefaultId(): string {
+       protected static function getDefaultId() {
                return 'options';
        }
 }
index 35d9b605d49086dd358cf4461be8472ae95fcf55..89850e330a52f4a504744f97d497e5a933460241 100644 (file)
@@ -40,7 +40,7 @@ class UserGroupOptionFormField extends ItemListFormField {
         * 
         * @return      int[]
         */
-       public function getPackageIDs(): array {
+       public function getPackageIDs() {
                return $this->__packageIDs;
        }
        
@@ -53,7 +53,7 @@ class UserGroupOptionFormField extends ItemListFormField {
         * 
         * @throws      \InvalidArgumentException       if the given package ids are invalid
         */
-       public function packageIDs(array $packageIDs): UserGroupOptionFormField {
+       public function packageIDs(array $packageIDs) {
                foreach ($packageIDs as $packageID) {
                        if (PackageCache::getInstance()->getPackage($packageID) === null) {
                                throw new \InvalidArgumentException("Unknown package with id '{$packageID}'.");
@@ -102,7 +102,7 @@ class UserGroupOptionFormField extends ItemListFormField {
        /**
         * @inheritDoc
         */
-       protected static function getDefaultId(): string {
+       protected static function getDefaultId() {
                return 'permissions';
        }
 }
index ec9d39875576a2bf7f71a865b0dd329e1c451d7e..aa4b3e898eb3a5ad3b33724132c797c79edfe1de 100644 (file)
@@ -469,7 +469,7 @@ class ACLOptionPackageInstallationPlugin extends AbstractOptionPackageInstallati
         * @inheritDoc
         * @since       3.2
         */
-       public function getEntryTypes(): array {
+       public function getEntryTypes() {
                return ['options', 'categories'];
        }
        
@@ -477,7 +477,7 @@ class ACLOptionPackageInstallationPlugin extends AbstractOptionPackageInstallati
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false): array {
+       protected function getElementData(\DOMElement $element, bool $saveData = false) {
                $data = [
                        'name' => $element->getAttribute('name'),
                        'packageID' => $this->installation->getPackage()->packageID,
@@ -498,7 +498,7 @@ class ACLOptionPackageInstallationPlugin extends AbstractOptionPackageInstallati
         * @inheritDoc
         * @since       3.2
         */
-       public function getElementIdentifier(\DOMElement $element): string {
+       public function getElementIdentifier(\DOMElement $element) {
                $elementData = $this->getElementData($element);
                
                return sha1($elementData['objectType'] . '/' . $elementData['name']);
@@ -508,7 +508,7 @@ class ACLOptionPackageInstallationPlugin extends AbstractOptionPackageInstallati
         * @inheritDoc
         * @since       3.2
         */
-       protected function getXsdFilename(): string {
+       protected function getXsdFilename() {
                return 'aclOption';
        }
        
@@ -660,7 +660,7 @@ class ACLOptionPackageInstallationPlugin extends AbstractOptionPackageInstallati
         * @inheritDoc
         * @since       3.2
         */
-       protected function writeEntry(\DOMDocument $document, IFormDocument $form): \DOMElement {
+       protected function writeEntry(\DOMDocument $document, IFormDocument $form) {
                $formData = $form->getData()['data'];
                
                switch ($this->entryType) {
index 4a2e752e408f5f545e12599367a0e060d7821a1d..0c1f6b75574f4ff5b9530c330c716edc3a30bbdb 100644 (file)
@@ -66,7 +66,7 @@ class ACPMenuPackageInstallationPlugin extends AbstractMenuPackageInstallationPl
                // icons are only available for menu items on the first or fourth level
                // thus the parent menu item must be on zeroth level (no parent menu item)
                // or on the third level
-               $iconParentMenuItems = array_keys(array_filter($menuItemLevels, function(int $value): bool {
+               $iconParentMenuItems = array_keys(array_filter($menuItemLevels, function(int $value) {
                        return $value === 0 || $value == 3;
                }));
                
@@ -110,7 +110,7 @@ class ACPMenuPackageInstallationPlugin extends AbstractMenuPackageInstallationPl
                // menu items on the first and second level do not support links,
                // thus the parent menu item must be at least on the second level
                // for the menu item to support links
-               $menuItemsSupportingLinks = array_keys(array_filter($menuItemLevels, function(int $menuItemLevel): bool {
+               $menuItemsSupportingLinks = array_keys(array_filter($menuItemLevels, function(int $menuItemLevel) {
                        return $menuItemLevel >= 2;
                }));
                
@@ -129,7 +129,7 @@ class ACPMenuPackageInstallationPlugin extends AbstractMenuPackageInstallationPl
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false): array {
+       protected function getElementData(\DOMElement $element, bool $saveData = false) {
                $data = parent::getElementData($element);
                
                $icon = $element->getElementsByTagName('icon')->item(0);
@@ -144,7 +144,7 @@ class ACPMenuPackageInstallationPlugin extends AbstractMenuPackageInstallationPl
         * @inheritDoc
         * @since       3.2
         */
-       protected function writeEntry(\DOMDocument $document, IFormDocument $form): \DOMElement {
+       protected function writeEntry(\DOMDocument $document, IFormDocument $form) {
                $formData = $form->getData()['data'];
                
                $menuItem = parent::writeEntry($document, $form);
index f8a3a3f135e41b832295635692ae7a004b78ee76..32b1adb14d34d423f5e321852f4cc2d7bf32c1fc 100644 (file)
@@ -113,7 +113,7 @@ class ACPSearchProviderPackageInstallationPlugin extends AbstractXMLPackageInsta
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false): array {
+       protected function getElementData(\DOMElement $element, bool $saveData = false) {
                $data = [
                        'className' => $element->getElementsByTagName('classname')->item(0)->nodeValue,
                        'packageID' => $this->installation->getPackage()->packageID,
@@ -132,7 +132,7 @@ class ACPSearchProviderPackageInstallationPlugin extends AbstractXMLPackageInsta
         * @inheritDoc
         * @since       3.2
         */
-       public function getElementIdentifier(\DOMElement $element): string {
+       public function getElementIdentifier(\DOMElement $element) {
                return $element->getAttribute('name');
        }
        
@@ -234,7 +234,7 @@ class ACPSearchProviderPackageInstallationPlugin extends AbstractXMLPackageInsta
         * @inheritDoc
         * @since       3.2
         */
-       protected function writeEntry(\DOMDocument $document, IFormDocument $form): \DOMElement {
+       protected function writeEntry(\DOMDocument $document, IFormDocument $form) {
                $data = $form->getData()['data'];
                
                $acpSearchProvider = $document->createElement('acpsearchprovider');
index 27f30f783fac599a68b32950e435398451cb077f..e7a8a8a7e86bcc4532542813512c5ce295bad77d 100644 (file)
@@ -159,7 +159,7 @@ abstract class AbstractMenuPackageInstallationPlugin extends AbstractXMLPackageI
                                ->objectProperty('parent')
                                ->label('wcf.acp.pip.abstractMenu.parentMenuItem')
                                ->filterable()
-                               ->options(function(): array {
+                               ->options(function() {
                                        $menuStructure = $this->getMenuStructureData()['structure'];
                                        
                                        $options = [[
@@ -168,7 +168,7 @@ abstract class AbstractMenuPackageInstallationPlugin extends AbstractXMLPackageI
                                                'value' => ''
                                        ]];
                                        
-                                       $buildOptions = function(string $parent = '', int $depth = 0) use ($menuStructure, &$buildOptions): array {
+                                       $buildOptions = function(string $parent = '', int $depth = 0) use ($menuStructure, &$buildOptions) {
                                                // only consider menu items until the third level (thus only parent
                                                // menu items until the second level) as potential parent menu items
                                                if ($depth > 2) {
@@ -272,7 +272,7 @@ abstract class AbstractMenuPackageInstallationPlugin extends AbstractXMLPackageI
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false): array {
+       protected function getElementData(\DOMElement $element, bool $saveData = false) {
                $data = [
                        'menuItem' => $element->getAttribute('name'),
                        'packageID' => $this->installation->getPackage()->packageID
@@ -315,7 +315,7 @@ abstract class AbstractMenuPackageInstallationPlugin extends AbstractXMLPackageI
         * @inheritDoc
         * @since       3.2
         */
-       public function getElementIdentifier(\DOMElement $element): string {
+       public function getElementIdentifier(\DOMElement $element) {
                return $element->getAttribute('name');
        }
        
@@ -324,7 +324,7 @@ abstract class AbstractMenuPackageInstallationPlugin extends AbstractXMLPackageI
         * 
         * @return      array
         */
-       protected function getMenuStructureData(): array {
+       protected function getMenuStructureData() {
                // replace `Editor` with `List`
                $listClassName = substr($this->className, 0, -6) . 'List';
                
@@ -396,7 +396,7 @@ abstract class AbstractMenuPackageInstallationPlugin extends AbstractXMLPackageI
                
                // build array containing the ACP menu items saved in the database
                // in the order as they would be displayed in the ACP
-               $buildPositions = function(string $parent = '') use ($menuItemStructure, &$buildPositions): array {
+               $buildPositions = function(string $parent = '') use ($menuItemStructure, &$buildPositions) {
                        $positions = [];
                        foreach ($menuItemStructure[$parent] as $menuItem) {
                                // only consider menu items of the current package for positions
@@ -429,7 +429,7 @@ abstract class AbstractMenuPackageInstallationPlugin extends AbstractXMLPackageI
         * @inheritDoc
         * @since       3.2
         */
-       protected function writeEntry(\DOMDocument $document, IFormDocument $form): \DOMElement {
+       protected function writeEntry(\DOMDocument $document, IFormDocument $form) {
                $formData = $form->getData()['data'];
                
                $menuItem = $document->createElement($this->tagName);
index a74ac2db27104a329ce06dd68800eef660474749..c49c5b8fe87b2136871f5ce78382f6fbbd264c8a 100644 (file)
@@ -245,7 +245,7 @@ class ClipboardActionPackageInstallationPlugin extends AbstractXMLPackageInstall
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false): array {
+       protected function getElementData(\DOMElement $element, bool $saveData = false) {
                $data = [
                        'actionClassName' => $element->getElementsByTagName('actionclassname')->item(0)->nodeValue,
                        'actionName' => $element->getAttribute('name'),
@@ -276,7 +276,7 @@ class ClipboardActionPackageInstallationPlugin extends AbstractXMLPackageInstall
         * @inheritDoc
         * @since       3.2
         */
-       public function getElementIdentifier(\DOMElement $element): string {
+       public function getElementIdentifier(\DOMElement $element) {
                return sha1(
                        $element->getElementsByTagName('actionclassname')->item(0)->nodeValue . '/' .
                        $element->getAttribute('name')
@@ -343,7 +343,7 @@ class ClipboardActionPackageInstallationPlugin extends AbstractXMLPackageInstall
         * @inheritDoc
         * @since       3.2
         */
-       protected function writeEntry(\DOMDocument $document, IFormDocument $form): \DOMElement {
+       protected function writeEntry(\DOMDocument $document, IFormDocument $form) {
                $formData = $form->getData();
                $data = $formData['data'];
                
index f9f82c5c84c82f915935d54c9ed2563074a8b41a..ce5383f148b513f083dab48005896735dfb3dee4 100644 (file)
@@ -127,7 +127,7 @@ class CoreObjectPackageInstallationPlugin extends AbstractXMLPackageInstallation
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false): array {
+       protected function getElementData(\DOMElement $element, bool $saveData = false) {
                return [
                        'objectName' => $element->getElementsByTagName('objectname')->item(0)->nodeValue,
                        'packageID' => $this->installation->getPackage()->packageID
@@ -138,7 +138,7 @@ class CoreObjectPackageInstallationPlugin extends AbstractXMLPackageInstallation
         * @inheritDoc
         * @since       3.2
         */
-       public function getElementIdentifier(\DOMElement $element): string {
+       public function getElementIdentifier(\DOMElement $element) {
                return sha1($element->getElementsByTagName('objectname')->item(0)->nodeValue);
        }
        
@@ -164,7 +164,7 @@ class CoreObjectPackageInstallationPlugin extends AbstractXMLPackageInstallation
         * @inheritDoc
         * @since       3.2
         */
-       protected function writeEntry(\DOMDocument $document, IFormDocument $form): \DOMElement {
+       protected function writeEntry(\DOMDocument $document, IFormDocument $form) {
                $data = $form->getData()['data'];
                
                $coreObject = $document->createElement($this->tagName);
index 48b959ffd56411965075a67c114d8cfea385732b..c5cd1a0ae6c871f4ad4ad94b909c9f8a7fae9171 100644 (file)
@@ -250,7 +250,7 @@ class CronjobPackageInstallationPlugin extends AbstractXMLPackageInstallationPlu
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false): array {
+       protected function getElementData(\DOMElement $element, bool $saveData = false) {
                $data = [
                        'className' => $element->getElementsByTagName('classname')->item(0)->nodeValue,
                        'cronjobName' => $element->getAttribute('name'),
@@ -316,7 +316,7 @@ class CronjobPackageInstallationPlugin extends AbstractXMLPackageInstallationPlu
         * @inheritDoc
         * @since       3.2
         */
-       public function getElementIdentifier(\DOMElement $element): string {
+       public function getElementIdentifier(\DOMElement $element) {
                return $element->getAttribute('name');
        }
        
@@ -367,7 +367,7 @@ class CronjobPackageInstallationPlugin extends AbstractXMLPackageInstallationPlu
         * @inheritDoc
         * @since       3.2
         */
-       protected function writeEntry(\DOMDocument $document, IFormDocument $form): \DOMElement {
+       protected function writeEntry(\DOMDocument $document, IFormDocument $form) {
                $data = $form->getData();
                $formData = $form->getData()['data'];
                
index e6e24ef32ef23ee156bb91f9725e1387477eaf78..7eb92c497f3050dfa0f600384ef318e3956e91f6 100644 (file)
@@ -286,7 +286,7 @@ class EventListenerPackageInstallationPlugin extends AbstractXMLPackageInstallat
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false): array {
+       protected function getElementData(\DOMElement $element, bool $saveData = false) {
                $data = [
                        'eventClassName' => $element->getElementsByTagName('eventclassname')->item(0)->nodeValue,
                        'eventName' => StringUtil::normalizeCsv($element->getElementsByTagName('eventname')->item(0)->nodeValue),
@@ -309,7 +309,7 @@ class EventListenerPackageInstallationPlugin extends AbstractXMLPackageInstallat
         * @inheritDoc
         * @since       3.2
         */
-       public function getElementIdentifier(\DOMElement $element): string {
+       public function getElementIdentifier(\DOMElement $element) {
                return $element->getAttribute('name');
        }
        
@@ -347,7 +347,7 @@ class EventListenerPackageInstallationPlugin extends AbstractXMLPackageInstallat
         * @inheritDoc
         * @since       3.2
         */
-       protected function writeEntry(\DOMDocument $document, IFormDocument $form): \DOMElement {
+       protected function writeEntry(\DOMDocument $document, IFormDocument $form) {
                $data = $form->getData()['data'];
                
                $eventListener = $document->createElement($this->tagName);
index 6a9be2a007c2c07e8e014567580198df719ecafb..91853ce4a6d37216bfefd6005632cda5b9f08b27 100644 (file)
@@ -182,7 +182,7 @@ class MediaProviderPackageInstallationPlugin extends AbstractXMLPackageInstallat
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false): array {
+       protected function getElementData(\DOMElement $element, bool $saveData = false) {
                $data = [
                        'name' => $element->getAttribute('name'),
                        'packageID' => $this->installation->getPackage()->packageID,
@@ -207,7 +207,7 @@ class MediaProviderPackageInstallationPlugin extends AbstractXMLPackageInstallat
         * @inheritDoc
         * @since       3.2
         */
-       public function getElementIdentifier(\DOMElement $element): string {
+       public function getElementIdentifier(\DOMElement $element) {
                return $element->getAttribute('name');
        }
        
@@ -244,7 +244,7 @@ class MediaProviderPackageInstallationPlugin extends AbstractXMLPackageInstallat
         * @inheritDoc
         * @since       3.2
         */
-       protected function writeEntry(\DOMDocument $document, IFormDocument $form): \DOMElement {
+       protected function writeEntry(\DOMDocument $document, IFormDocument $form) {
                $data = $form->getData()['data'];
                
                $provider = $document->createElement($this->tagName);
index e79ad2511339d126bcbb046d2cbfda4998e2ce5c..dae6e4a2d0cbca3d8728f5df5c1e604ba8558d5b 100644 (file)
@@ -304,7 +304,7 @@ class MenuItemPackageInstallationPlugin extends AbstractXMLPackageInstallationPl
                                ->label('wcf.acp.pip.menuItem.menu')
                                ->description('wcf.acp.pip.menuItem.menu.description')
                                ->required()
-                               ->options(function() use ($menuList): array {
+                               ->options(function() use ($menuList) {
                                        $options = [];
                                        foreach ($menuList as $menu) {
                                                $options[$menu->identifier] = $menu->identifier;
@@ -330,7 +330,7 @@ class MenuItemPackageInstallationPlugin extends AbstractXMLPackageInstallationPl
                                ->description('wcf.acp.pip.menuItem.page.description')
                                ->required()
                                ->filterable()
-                               ->options(function(): array {
+                               ->options(function() {
                                        $pageNodeList = (new PageNodeTree())->getNodeList();
                                        
                                        $nestedOptions = [[
@@ -380,7 +380,7 @@ class MenuItemPackageInstallationPlugin extends AbstractXMLPackageInstallationPl
                                SingleSelectionFormField::create('parentMenuItem' . $menu->menuID)
                                        ->objectProperty('parent')
                                        ->label('wcf.acp.pip.menuItem.parentMenuItem')
-                                       ->options(function() use($menu): array {
+                                       ->options(function() use($menu) {
                                                $options = [[
                                                        'depth' => 0,
                                                        'label' => 'wcf.global.noSelection',
@@ -439,7 +439,7 @@ class MenuItemPackageInstallationPlugin extends AbstractXMLPackageInstallationPl
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false): array {
+       protected function getElementData(\DOMElement $element, bool $saveData = false) {
                $data = [
                        'identifier' => $element->getAttribute('identifier'),
                        'packageID' => $this->installation->getPackageID(),
@@ -512,7 +512,7 @@ class MenuItemPackageInstallationPlugin extends AbstractXMLPackageInstallationPl
         * @inheritDoc
         * @since       3.2
         */
-       public function getElementIdentifier(\DOMElement $element): string {
+       public function getElementIdentifier(\DOMElement $element) {
                return $element->getAttribute('identifier');
        }
        
@@ -575,7 +575,7 @@ class MenuItemPackageInstallationPlugin extends AbstractXMLPackageInstallationPl
         * @inheritDoc
         * @since       3.2
         */
-       protected function writeEntry(\DOMDocument $document, IFormDocument $form): \DOMElement {
+       protected function writeEntry(\DOMDocument $document, IFormDocument $form) {
                $formData = $form->getData();
                $data = $formData['data'];
                
index c4161426b22e6c7ab2688cfa060e51855f0f1ca7..d5f48d6b1eeb467cac20c2afa82ddb0d1ea88507 100644 (file)
@@ -298,7 +298,7 @@ class MenuPackageInstallationPlugin extends AbstractXMLPackageInstallationPlugin
         * @inheritDoc
         * @since       3.2
         */
-       public function getAdditionalTemplateCode(): string {
+       public function getAdditionalTemplateCode() {
                return WCF::getTPL()->fetch('__menuPipGui');
        }
        
@@ -364,7 +364,7 @@ class MenuPackageInstallationPlugin extends AbstractXMLPackageInstallationPlugin
                        MultipleSelectionFormField::create('boxVisibilityExceptions')
                                ->label('wcf.acp.pip.menu.boxVisibilityExceptions.hiddenEverywhere')
                                ->filterable()
-                               ->options(function(): array {
+                               ->options(function() {
                                        $pageNodeList = (new PageNodeTree())->getNodeList();
                                        
                                        $nestedOptions = [];
@@ -407,7 +407,7 @@ class MenuPackageInstallationPlugin extends AbstractXMLPackageInstallationPlugin
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false): array {
+       protected function getElementData(\DOMElement $element, bool $saveData = false) {
                $data = [
                        'identifier' => $element->getAttribute('identifier'),
                        'packageID' => $this->installation->getPackageID(),
@@ -503,7 +503,7 @@ class MenuPackageInstallationPlugin extends AbstractXMLPackageInstallationPlugin
         * @inheritDoc
         * @since       3.2
         */
-       public function getElementIdentifier(\DOMElement $element): string {
+       public function getElementIdentifier(\DOMElement $element) {
                return $element->getAttribute('identifier');
        }
        
@@ -539,7 +539,7 @@ class MenuPackageInstallationPlugin extends AbstractXMLPackageInstallationPlugin
         * @inheritDoc
         * @since       3.2
         */
-       protected function writeEntry(\DOMDocument $document, IFormDocument $form): \DOMElement {
+       protected function writeEntry(\DOMDocument $document, IFormDocument $form) {
                $formData = $form->getData();
                
                if ($formData['data']['identifier'] === 'com.woltlab.wcf.MainMenu') {
index d84a522550e673328b234f4230023d0143dfa257..f2d969047ef47ca6b9b0e856a87de97d01894194 100644 (file)
@@ -172,7 +172,7 @@ class ObjectTypeDefinitionPackageInstallationPlugin extends AbstractXMLPackageIn
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false): array {
+       protected function getElementData(\DOMElement $element, bool $saveData = false) {
                $data = [
                        'definitionName' => $element->getElementsByTagName('name')->item(0)->nodeValue,
                        'packageID' => $this->installation->getPackage()->packageID
@@ -190,7 +190,7 @@ class ObjectTypeDefinitionPackageInstallationPlugin extends AbstractXMLPackageIn
         * @inheritDoc
         * @since       3.2
         */
-       public function getElementIdentifier(\DOMElement $element): string {
+       public function getElementIdentifier(\DOMElement $element) {
                return $element->getElementsByTagName('name')->item(0)->nodeValue;
        }
        
@@ -231,7 +231,7 @@ class ObjectTypeDefinitionPackageInstallationPlugin extends AbstractXMLPackageIn
         * @inheritDoc
         * @since       3.2
         */
-       protected function writeEntry(\DOMDocument $document, IFormDocument $form): \DOMElement {
+       protected function writeEntry(\DOMDocument $document, IFormDocument $form) {
                $data = $form->getData()['data'];
                
                $definition = $document->createElement('definition');
index 73f03c694f53ab2d4f6fe2bafa2baa2228af94b6..45d4b2d7acdb03f2334bc7f34f3adab9869e2c71 100644 (file)
@@ -158,7 +158,7 @@ class ObjectTypePackageInstallationPlugin extends AbstractXMLPackageInstallation
         * @inheritDoc
         * @since       3.2
         */
-       public function getAdditionalTemplateCode(): string {
+       public function getAdditionalTemplateCode() {
                return WCF::getTPL()->fetch('__objectTypePipGui', 'wcf', [
                        'definitionNames' => $this->definitionNames,
                        'definitionInterfaces' => $this->definitionInterfaces
@@ -169,7 +169,7 @@ class ObjectTypePackageInstallationPlugin extends AbstractXMLPackageInstallation
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false): array {
+       protected function getElementData(\DOMElement $element, bool $saveData = false) {
                $data = [
                        'definitionID' => $this->getDefinitionID($element->getElementsByTagName('definitionname')->item(0)->nodeValue),
                        'objectType' => $element->getElementsByTagName('name')->item(0)->nodeValue,
@@ -593,7 +593,7 @@ class ObjectTypePackageInstallationPlugin extends AbstractXMLPackageInstallation
         * @inheritDoc
         * @since       3.2
         */
-       public function getElementIdentifier(\DOMElement $element): string {
+       public function getElementIdentifier(\DOMElement $element) {
                return sha1(
                        $element->getElementsByTagName('name')->item(0)->nodeValue . '/' .
                        $element->getElementsByTagName('definitionname')->item(0)->nodeValue
@@ -604,7 +604,7 @@ class ObjectTypePackageInstallationPlugin extends AbstractXMLPackageInstallation
         * @inheritDoc
         * @since       3.2
         */
-       protected function getEmptyXml(): string {
+       protected function getEmptyXml() {
                return <<<XML
 <?xml version="1.0" encoding="UTF-8"?>
 <data xmlns="http://www.woltlab.com" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://www.woltlab.com http://www.woltlab.com/XSD/vortex/objectType.xsd">
@@ -617,7 +617,7 @@ XML;
         * @inheritDoc
         * @since       3.2
         */
-       public function getEntryList(): IDevtoolsPipEntryList {
+       public function getEntryList() {
                $xml = $this->getProjectXml();
                $xpath = $xml->xpath();
                
@@ -659,7 +659,7 @@ XML;
         * @return      FormContainer
         * @since       3.2
         */
-       public function getObjectTypeDefinitionDataContainer(IFormDocument $form, string $definitionName): FormContainer {
+       public function getObjectTypeDefinitionDataContainer(IFormDocument $form, string $definitionName) {
                /** @var SingleSelectionFormField $definitionNameField */
                $definitionIDField = $form->getNodeById('definitionID');
                
@@ -735,7 +735,7 @@ XML;
         * @inheritDoc
         * @since       3.2
         */
-       protected function writeEntry(\DOMDocument $document, IFormDocument $form): \DOMElement {
+       protected function writeEntry(\DOMDocument $document, IFormDocument $form) {
                $type = $document->createElement('type');
                foreach ($form->getData()['data'] as $key => $value) {
                        if ($key === 'definitionID') {
@@ -959,7 +959,7 @@ XML;
         * @param       string          $databaseTableName      name of the database table that stores the conditioned objects
         * @return      TextFormField
         */
-       public function getIntegerConditionPropertyNameField(TextFormField $classNameField, string $conditionClass, string $id, string $languageItemPrefix, string $databaseTableName): TextFormField {
+       public function getIntegerConditionPropertyNameField(TextFormField $classNameField, string $conditionClass, string $id, string $languageItemPrefix, string $databaseTableName) {
                return TextFormField::create($id)
                        ->objectProperty('propertyname')
                        ->label($languageItemPrefix . '.propertyName')
@@ -1002,7 +1002,7 @@ XML;
         * @param       int             $minimumSegmentCount    minimum number of dot-separated segments
         * @return      FormFieldValidator
         */
-       public static function getObjectTypeAlikeValueValidator(string $languageItemPrefix, int $minimumSegmentCount = 4): FormFieldValidator {
+       public static function getObjectTypeAlikeValueValidator(string $languageItemPrefix, int $minimumSegmentCount = 4) {
                return new FormFieldValidator('format', function(TextFormField $formField) use ($languageItemPrefix, $minimumSegmentCount) {
                        if ($formField->getValue()) {
                                $segments = explode('.', $formField->getValue());
index c64dbbb301334e589ea035f42434279317d926b5..1e188829acd83f6a3815f2a3a4f8349d015074c3 100644 (file)
@@ -144,7 +144,7 @@ class PIPPackageInstallationPlugin extends AbstractXMLPackageInstallationPlugin
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false): array {
+       protected function getElementData(\DOMElement $element, bool $saveData = false) {
                return [
                        'className' => $element->nodeValue,
                        'pluginName' => $element->getAttribute('name'),
@@ -156,7 +156,7 @@ class PIPPackageInstallationPlugin extends AbstractXMLPackageInstallationPlugin
         * @inheritDoc
         * @since       3.2
         */
-       public function getElementIdentifier(\DOMElement $element): string {
+       public function getElementIdentifier(\DOMElement $element) {
                return $element->getAttribute('name');
        }
        
@@ -190,7 +190,7 @@ class PIPPackageInstallationPlugin extends AbstractXMLPackageInstallationPlugin
         * @inheritDoc
         * @since       3.2
         */
-       protected function writeEntry(\DOMDocument $document, IFormDocument $form): \DOMElement {
+       protected function writeEntry(\DOMDocument $document, IFormDocument $form) {
                /** @var TextFormField $className */
                $className = $form->getNodeById('className');
                /** @var TextFormField $pluginName */
index 8873e1882a5153201645585a1b735b3900016027..7ed1649b54aec70a3e8df8383af873e907c5013b 100644 (file)
@@ -498,7 +498,7 @@ class PagePackageInstallationPlugin extends AbstractXMLPackageInstallationPlugin
                                ->label('wcf.acp.pip.page.parent')
                                ->required()
                                ->filterable()
-                               ->options(function(): array {
+                               ->options(function() {
                                        $pageNodeList = (new PageNodeTree())->getNodeList();
                                        
                                        $nestedOptions = [[
@@ -636,7 +636,7 @@ class PagePackageInstallationPlugin extends AbstractXMLPackageInstallationPlugin
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false): array {
+       protected function getElementData(\DOMElement $element, bool $saveData = false) {
                $data = [
                        'identifier' => $element->getAttribute('identifier'),
                        'originIsSystem' => 1,
@@ -732,7 +732,7 @@ class PagePackageInstallationPlugin extends AbstractXMLPackageInstallationPlugin
         * @inheritDoc
         * @since       3.2
         */
-       public function getElementIdentifier(\DOMElement $element): string {
+       public function getElementIdentifier(\DOMElement $element) {
                return $element->getAttribute('identifier');
        }
        
@@ -769,7 +769,7 @@ class PagePackageInstallationPlugin extends AbstractXMLPackageInstallationPlugin
         * @inheritDoc
         * @since       3.2
         */
-       protected function writeEntry(\DOMDocument $document, IFormDocument $form): \DOMElement {
+       protected function writeEntry(\DOMDocument $document, IFormDocument $form) {
                $formData = $form->getData();
                $data = $formData['data'];
                
index 37d8e9058dee483ed64a1ab418fa1e2dddf5ee2a..2b786d45a5952c8fa6bfeebd2581253c0b82ac20 100644 (file)
@@ -324,7 +324,7 @@ class TemplateListenerPackageInstallationPlugin extends AbstractXMLPackageInstal
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false): array {
+       protected function getElementData(\DOMElement $element, bool $saveData = false) {
                return [
                        'environment' => $element->getElementsByTagName('environment')->item(0)->nodeValue,
                        'eventName' => $element->getElementsByTagName('eventname')->item(0)->nodeValue,
@@ -339,7 +339,7 @@ class TemplateListenerPackageInstallationPlugin extends AbstractXMLPackageInstal
         * @inheritDoc
         * @since       3.2
         */
-       public function getElementIdentifier(\DOMElement $element): string {
+       public function getElementIdentifier(\DOMElement $element) {
                return sha1(
                        $element->getElementsByTagName('templatename')->item(0)->nodeValue . '/' .
                        $element->getElementsByTagName('eventname')->item(0)->nodeValue . '/' .
@@ -397,7 +397,7 @@ class TemplateListenerPackageInstallationPlugin extends AbstractXMLPackageInstal
         * @inheritDoc
         * @since       3.2
         */
-       protected function writeEntry(\DOMDocument $document, IFormDocument $form): \DOMElement {
+       protected function writeEntry(\DOMDocument $document, IFormDocument $form) {
                $data = $form->getData()['data'];
                
                $listener = $document->createElement($this->tagName);
index e43a0ae0abbc0d8cab3cd7a38aecc88ea73b9de3..53afd19c2470d7e91a65e7222da61e83c76a7afe 100644 (file)
@@ -107,7 +107,7 @@ class UserMenuPackageInstallationPlugin extends AbstractMenuPackageInstallationP
                // menu items on the first and second level do not support links,
                // thus the parent menu item must be at least on the second level
                // for the menu item to support links
-               $menuItemsSupportingLinks = array_keys(array_filter($menuItemLevels, function(int $menuItemLevel): bool {
+               $menuItemsSupportingLinks = array_keys(array_filter($menuItemLevels, function(int $menuItemLevel) {
                        return $menuItemLevel >= 2;
                }));
                
@@ -126,7 +126,7 @@ class UserMenuPackageInstallationPlugin extends AbstractMenuPackageInstallationP
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false): array {
+       protected function getElementData(\DOMElement $element, bool $saveData = false) {
                $data = parent::getElementData($element);
                
                $className = $element->getElementsByTagName('classname')->item(0);
@@ -146,7 +146,7 @@ class UserMenuPackageInstallationPlugin extends AbstractMenuPackageInstallationP
         * @inheritDoc
         * @since       3.2
         */
-       protected function writeEntry(\DOMDocument $document, IFormDocument $form): \DOMElement {
+       protected function writeEntry(\DOMDocument $document, IFormDocument $form) {
                $formData = $form->getData()['data'];
                
                $menuItem = parent::writeEntry($document, $form);
index 8cea9da1b79b13f3be35984003d68f18393ae4b7..2ae02cc09055d28ca3a2c45327c94ed94dcf6ec6 100644 (file)
@@ -206,7 +206,7 @@ class UserNotificationEventPackageInstallationPlugin extends AbstractXMLPackageI
                                ->label('wcf.acp.pip.userNotificationEvent.objectType')
                                ->description('wcf.acp.pip.userNotificationEvent.objectType.description')
                                ->required()
-                               ->options(function(): array {
+                               ->options(function() {
                                        $options = [];
                                        foreach (ObjectTypeCache::getInstance()->getObjectTypes('com.woltlab.wcf.notification.objectType') as $objectType) {
                                                $options[$objectType->objectType] = $objectType->objectType;
@@ -280,7 +280,7 @@ class UserNotificationEventPackageInstallationPlugin extends AbstractXMLPackageI
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false): array {
+       protected function getElementData(\DOMElement $element, bool $saveData = false) {
                $data = [
                        'className' => $element->getElementsByTagName('classname')->item(0)->nodeValue,
                        'objectTypeID' => $this->getObjectTypeID($element->getElementsByTagName('objecttype')->item(0)->nodeValue),
@@ -316,7 +316,7 @@ class UserNotificationEventPackageInstallationPlugin extends AbstractXMLPackageI
         * @inheritDoc
         * @since       3.2
         */
-       public function getElementIdentifier(\DOMElement $element): string {
+       public function getElementIdentifier(\DOMElement $element) {
                return sha1(
                        $element->getElementsByTagName('name')->item(0)->nodeValue . '/' .
                        $element->getElementsByTagName('objecttype')->item(0)->nodeValue
@@ -363,7 +363,7 @@ class UserNotificationEventPackageInstallationPlugin extends AbstractXMLPackageI
         * @inheritDoc
         * @since       3.2
         */
-       protected function writeEntry(\DOMDocument $document, IFormDocument $form): \DOMElement {
+       protected function writeEntry(\DOMDocument $document, IFormDocument $form) {
                $data = $form->getData()['data'];
                
                $event = $document->createElement($this->tagName);
index 2532a3fa9d94de82de22cb669c7d1a83b1acb779..5879b9e6881ac6c0121a251038d98589d0bc8530 100644 (file)
@@ -178,7 +178,7 @@ class UserProfileMenuPackageInstallationPlugin extends AbstractXMLPackageInstall
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false): array {
+       protected function getElementData(\DOMElement $element, bool $saveData = false) {
                $data = [
                        'className' => $element->getElementsByTagName('classname')->item(0)->nodeValue,
                        'menuItem' => $element->getAttribute('name'),
@@ -210,7 +210,7 @@ class UserProfileMenuPackageInstallationPlugin extends AbstractXMLPackageInstall
         * @inheritDoc
         * @since       3.2
         */
-       public function getElementIdentifier(\DOMElement $element): string {
+       public function getElementIdentifier(\DOMElement $element) {
                return $element->getAttribute('name');
        }
        
@@ -261,7 +261,7 @@ class UserProfileMenuPackageInstallationPlugin extends AbstractXMLPackageInstall
         * @inheritDoc
         * @since       3.2
         */
-       protected function writeEntry(\DOMDocument $document, IFormDocument $form): \DOMElement {
+       protected function writeEntry(\DOMDocument $document, IFormDocument $form) {
                $data = $form->getData()['data'];
                
                $userProfileMenuItem = $document->createElement('userprofilemenuitem');