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

23 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/IGuiPackageInstallationPlugin.class.php
wcfsetup/install/files/lib/system/devtools/pip/TXmlGuiPackageInstallationPlugin.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 9b7f966d09c5a61de695e4baba197746953e67be..bfc43d0cb383a95376c4dc943038775bc45b4cd0 100644 (file)
@@ -29,7 +29,7 @@ class DevtoolsPipEntryList implements IDevtoolsPipEntryList {
        /**
         * @inheritDoc
         */
-       public function addEntry(string $id, array $entry) {
+       public function addEntry($id, array $entry) {
                if ($this->keys === null) {
                        throw new \BadMethodCallException("No keys have been set.");
                }
index 320667482ccf861f7d60e139dde71c0e8f4895c1..15aec49c37bfb3e0160acd8d41eb2418f86d25ac 100644 (file)
@@ -17,11 +17,11 @@ interface IDevtoolsPipEntryList {
         * 
         * Before adding entries, the keys must be set.
         * 
-        * @param       string          $identifier     unique entry ident
+        * @param       string          $id             unique entry identifier
         * @param       array           $entry          entry data
         * @throws      \BadMethodCallException         if no keys have been set
         */
-       public function addEntry(string $id, array $entry);
+       public function addEntry($id, array $entry);
        
        /**
         * Returns all entries in the list.
index 569ac1996c4c52b82d7c5dd05071a85602c545ed..f02fde32829ffab040e80d35783029fc33e39779 100644 (file)
@@ -38,21 +38,21 @@ interface IGuiPackageInstallationPlugin extends IIdempotentPackageInstallationPl
         * @param       string                  $identifier
         * @return      string                  new identifier
         */
-       public function editEntry(IFormDocument $form, string $identifier): string;
+       public function editEntry(IFormDocument $form, $identifier);
        
        /**
         * Returns additional template code for the form to add and edit entries.
         * 
         * @return      string
         */
-       public function getAdditionalTemplateCode(): string;
+       public function getAdditionalTemplateCode();
        
        /**
         * Returns a list of all pip entries of this pip. 
         * 
         * @return      IDevtoolsPipEntryList
         */
-       public function getEntryList(): IDevtoolsPipEntryList;
+       public function getEntryList();
        
        /**
         * Informs the pip of the identifier of the edited entry if the form to
@@ -62,7 +62,7 @@ interface IGuiPackageInstallationPlugin extends IIdempotentPackageInstallationPl
         * 
         * @throws      \InvalidArgumentException       if no such entry exists
         */
-       public function setEditedEntryIdentifier(string $identifier);
+       public function setEditedEntryIdentifier($identifier);
        
        /**
         * Adds the data of the pip entry with the given identifier into the
@@ -73,7 +73,7 @@ interface IGuiPackageInstallationPlugin extends IIdempotentPackageInstallationPl
         * @param       IFormDocument           $document
         * @return      bool
         */
-       public function setEntryData(string $identifier, IFormDocument $document): bool;
+       public function setEntryData($identifier, IFormDocument $document);
        
        /**
         * Returns the list of available entry types. If only one entry type is
@@ -84,7 +84,7 @@ interface IGuiPackageInstallationPlugin extends IIdempotentPackageInstallationPl
         * 
         * @return      string[]
         */
-       public function getEntryTypes(): array;
+       public function getEntryTypes();
        
        /**
         * Sets the type of the currently handled pip entries.
@@ -93,5 +93,5 @@ interface IGuiPackageInstallationPlugin extends IIdempotentPackageInstallationPl
         *
         * @throws      \InvalidArgumentException       if the given entry type is invalid (see `getEntryTypes()` method) 
         */
-       public function setEntryType(string $entryType);
+       public function setEntryType($entryType);
 }
index 937763bc03aeb929d9641a365fe62f33b96076d4..9256e7d7754941be728034a9a141c61cc958e508 100644 (file)
@@ -71,7 +71,7 @@ trait TXmlGuiPackageInstallationPlugin {
         * @param       string                  $identifier
         * @return      string                  new identifier
         */
-       public function editEntry(IFormDocument $form, string $identifier) {
+       public function editEntry(IFormDocument $form, $identifier) {
                $xml = $this->getProjectXml();
                $document = $xml->getDocument();
                
@@ -112,7 +112,7 @@ trait TXmlGuiPackageInstallationPlugin {
         * @param       string          $value
         * @return      string
         */
-       protected function getAutoCdataValue(string $value) {
+       protected function getAutoCdataValue($value) {
                if (strpos('<', $value) !== false || strpos('>', $value) !== false || strpos('&', $value) !== false) {
                        $value = '<![CDATA[' . StringUtil::escapeCDATA($value) . ']]>';
                }
@@ -127,7 +127,7 @@ trait TXmlGuiPackageInstallationPlugin {
         * @param       string  $identifier
         * @return      \DOMElement|null
         */
-       protected function getElementByIdentifier(XML $xml, string $identifier) {
+       protected function getElementByIdentifier(XML $xml, $identifier) {
                foreach ($this->getImportElements($xml->xpath()) as $element) {
                        if ($this->getElementIdentifier($element) === $identifier) {
                                return $element;
@@ -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);
+       abstract protected function getElementData(\DOMElement $element, $saveData = false);
        
        /**
         * Returns the identifier of the given `import` element.
@@ -307,7 +307,7 @@ XML;
         * 
         * @throws      \InvalidArgumentException       if no such entry exists
         */
-       public function setEditedEntryIdentifier(string $identifier) {
+       public function setEditedEntryIdentifier($identifier) {
                $this->editedEntry = $this->getElementByIdentifier($this->getProjectXml(), $identifier);
                
                if ($this->editedEntry === null) {
@@ -324,7 +324,7 @@ XML;
         * @param       IFormDocument           $document
         * @return      bool
         */
-       public function setEntryData(string $identifier, IFormDocument $document) {
+       public function setEntryData($identifier, IFormDocument $document) {
                $xml = $this->getProjectXml();
                
                $element = $this->getElementByIdentifier($xml, $identifier);
@@ -377,7 +377,7 @@ XML;
         * 
         * @throws      \InvalidArgumentException       if the given entry type is invalid (see `getEntryTypes()` method)
         */
-       public function setEntryType(string $entryType) {
+       public function setEntryType($entryType) {
                if (!in_array($entryType, $this->getEntryTypes())) {
                        throw new \InvalidArgumentException("Unknown entry type '{$entryType}'.");
                }
index aa4b3e898eb3a5ad3b33724132c797c79edfe1de..58a6322909ee524e3d13d4888ba810c04b43f770 100644 (file)
@@ -477,7 +477,7 @@ class ACLOptionPackageInstallationPlugin extends AbstractOptionPackageInstallati
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false) {
+       protected function getElementData(\DOMElement $element, $saveData = false) {
                $data = [
                        'name' => $element->getAttribute('name'),
                        'packageID' => $this->installation->getPackage()->packageID,
index 0c1f6b75574f4ff5b9530c330c716edc3a30bbdb..172d793e5992a0c9e1712d2df343131eb7938e1a 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) {
+               $iconParentMenuItems = array_keys(array_filter($menuItemLevels, function($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) {
+               $menuItemsSupportingLinks = array_keys(array_filter($menuItemLevels, function($menuItemLevel) {
                        return $menuItemLevel >= 2;
                }));
                
@@ -129,7 +129,7 @@ class ACPMenuPackageInstallationPlugin extends AbstractMenuPackageInstallationPl
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false) {
+       protected function getElementData(\DOMElement $element, $saveData = false) {
                $data = parent::getElementData($element);
                
                $icon = $element->getElementsByTagName('icon')->item(0);
index 32b1adb14d34d423f5e321852f4cc2d7bf32c1fc..3feb3ab649efa9e2761c1e21a4d6892a78dcdea1 100644 (file)
@@ -113,7 +113,7 @@ class ACPSearchProviderPackageInstallationPlugin extends AbstractXMLPackageInsta
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false) {
+       protected function getElementData(\DOMElement $element, $saveData = false) {
                $data = [
                        'className' => $element->getElementsByTagName('classname')->item(0)->nodeValue,
                        'packageID' => $this->installation->getPackage()->packageID,
index e7a8a8a7e86bcc4532542813512c5ce295bad77d..8bd346e5a3ef160e5c0d3fc129d24bbc410e36c4 100644 (file)
@@ -168,7 +168,7 @@ abstract class AbstractMenuPackageInstallationPlugin extends AbstractXMLPackageI
                                                'value' => ''
                                        ]];
                                        
-                                       $buildOptions = function(string $parent = '', int $depth = 0) use ($menuStructure, &$buildOptions) {
+                                       $buildOptions = function($parent = '', $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) {
+       protected function getElementData(\DOMElement $element, $saveData = false) {
                $data = [
                        'menuItem' => $element->getAttribute('name'),
                        'packageID' => $this->installation->getPackage()->packageID
@@ -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) {
+               $buildPositions = function($parent = '') use ($menuItemStructure, &$buildPositions) {
                        $positions = [];
                        foreach ($menuItemStructure[$parent] as $menuItem) {
                                // only consider menu items of the current package for positions
index c49c5b8fe87b2136871f5ce78382f6fbbd264c8a..a8cf996d9fb251b6ff767867213bd54380aa0064 100644 (file)
@@ -245,7 +245,7 @@ class ClipboardActionPackageInstallationPlugin extends AbstractXMLPackageInstall
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false) {
+       protected function getElementData(\DOMElement $element, $saveData = false) {
                $data = [
                        'actionClassName' => $element->getElementsByTagName('actionclassname')->item(0)->nodeValue,
                        'actionName' => $element->getAttribute('name'),
index ce5383f148b513f083dab48005896735dfb3dee4..5ce39cde380d05276febfb91528480b512475f2a 100644 (file)
@@ -127,7 +127,7 @@ class CoreObjectPackageInstallationPlugin extends AbstractXMLPackageInstallation
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false) {
+       protected function getElementData(\DOMElement $element, $saveData = false) {
                return [
                        'objectName' => $element->getElementsByTagName('objectname')->item(0)->nodeValue,
                        'packageID' => $this->installation->getPackage()->packageID
index c5cd1a0ae6c871f4ad4ad94b909c9f8a7fae9171..fc4c84a9d480d680f93f8a6063608b1da90bdf67 100644 (file)
@@ -250,7 +250,7 @@ class CronjobPackageInstallationPlugin extends AbstractXMLPackageInstallationPlu
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false) {
+       protected function getElementData(\DOMElement $element, $saveData = false) {
                $data = [
                        'className' => $element->getElementsByTagName('classname')->item(0)->nodeValue,
                        'cronjobName' => $element->getAttribute('name'),
index 7eb92c497f3050dfa0f600384ef318e3956e91f6..ab70324a85ffe714664089afbb38b706c92d8d47 100644 (file)
@@ -286,7 +286,7 @@ class EventListenerPackageInstallationPlugin extends AbstractXMLPackageInstallat
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false) {
+       protected function getElementData(\DOMElement $element, $saveData = false) {
                $data = [
                        'eventClassName' => $element->getElementsByTagName('eventclassname')->item(0)->nodeValue,
                        'eventName' => StringUtil::normalizeCsv($element->getElementsByTagName('eventname')->item(0)->nodeValue),
index 91853ce4a6d37216bfefd6005632cda5b9f08b27..e515682e572e78d8b38897519412c182ac24a952 100644 (file)
@@ -182,7 +182,7 @@ class MediaProviderPackageInstallationPlugin extends AbstractXMLPackageInstallat
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false) {
+       protected function getElementData(\DOMElement $element, $saveData = false) {
                $data = [
                        'name' => $element->getAttribute('name'),
                        'packageID' => $this->installation->getPackage()->packageID,
index dae6e4a2d0cbca3d8728f5df5c1e604ba8558d5b..cffba3373f2cabc579bbc0d57ecea78bd81b50e6 100644 (file)
@@ -166,7 +166,7 @@ class MenuItemPackageInstallationPlugin extends AbstractXMLPackageInstallationPl
         * @param       string  $identifier
         * @return      null|int
         */
-       protected function getMenuID(string $identifier) {
+       protected function getMenuID($identifier) {
                $sql = "SELECT  menuID
                        FROM    wcf" . WCF_N . "_menu
                        WHERE   identifier = ?";
@@ -183,7 +183,7 @@ class MenuItemPackageInstallationPlugin extends AbstractXMLPackageInstallationPl
         * @param       string  $identifier
         * @return      null|int
         */
-       protected function getPageID(string $identifier) {
+       protected function getPageID($identifier) {
                $sql = "SELECT  pageID
                        FROM    wcf" . WCF_N . "_page
                        WHERE   identifier = ?";
@@ -439,7 +439,7 @@ class MenuItemPackageInstallationPlugin extends AbstractXMLPackageInstallationPl
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false) {
+       protected function getElementData(\DOMElement $element, $saveData = false) {
                $data = [
                        'identifier' => $element->getAttribute('identifier'),
                        'packageID' => $this->installation->getPackageID(),
index d5f48d6b1eeb467cac20c2afa82ddb0d1ea88507..9df987dee5cfa72ef0f120e54aa566145abdc18c 100644 (file)
@@ -407,7 +407,7 @@ class MenuPackageInstallationPlugin extends AbstractXMLPackageInstallationPlugin
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false) {
+       protected function getElementData(\DOMElement $element, $saveData = false) {
                $data = [
                        'identifier' => $element->getAttribute('identifier'),
                        'packageID' => $this->installation->getPackageID(),
index f2d969047ef47ca6b9b0e856a87de97d01894194..2728d5fa5406680f72562daf570c8bea6978a97f 100644 (file)
@@ -172,7 +172,7 @@ class ObjectTypeDefinitionPackageInstallationPlugin extends AbstractXMLPackageIn
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false) {
+       protected function getElementData(\DOMElement $element, $saveData = false) {
                $data = [
                        'definitionName' => $element->getElementsByTagName('name')->item(0)->nodeValue,
                        'packageID' => $this->installation->getPackage()->packageID
index 45d4b2d7acdb03f2334bc7f34f3adab9869e2c71..a670181d125af4a921e148d3827560e3ccec4d2c 100644 (file)
@@ -169,7 +169,7 @@ class ObjectTypePackageInstallationPlugin extends AbstractXMLPackageInstallation
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false) {
+       protected function getElementData(\DOMElement $element, $saveData = false) {
                $data = [
                        'definitionID' => $this->getDefinitionID($element->getElementsByTagName('definitionname')->item(0)->nodeValue),
                        'objectType' => $element->getElementsByTagName('name')->item(0)->nodeValue,
@@ -659,7 +659,7 @@ XML;
         * @return      FormContainer
         * @since       3.2
         */
-       public function getObjectTypeDefinitionDataContainer(IFormDocument $form, string $definitionName) {
+       public function getObjectTypeDefinitionDataContainer(IFormDocument $form, $definitionName) {
                /** @var SingleSelectionFormField $definitionNameField */
                $definitionIDField = $form->getNodeById('definitionID');
                
@@ -765,7 +765,7 @@ XML;
         * @param       IFormDocument   $form
         * @param       string          $objectTypeDefinition
         */
-       public function addBulkProcessingActionFields(IFormDocument $form, string $objectTypeDefinition) {
+       public function addBulkProcessingActionFields(IFormDocument $form, $objectTypeDefinition) {
                $definitionPieces = explode('.', $objectTypeDefinition);
                $definitionIdString = implode('', array_map('ucfirst', $definitionPieces));
                
@@ -812,7 +812,7 @@ XML;
         * @param       bool                    $addConditionGroup
         * @since       3.2
         */
-       public function addConditionFields(IFormContainer $dataContainer, bool $addConditionObject = true, bool $addConditionGroup = true) {
+       public function addConditionFields(IFormContainer $dataContainer, $addConditionObject = true, $addConditionGroup = true) {
                $prefix = preg_replace('~Fields$~', '', $dataContainer->getId());
                
                if ($addConditionObject) {
@@ -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) {
+       public function getIntegerConditionPropertyNameField(TextFormField $classNameField, $conditionClass, $id, $languageItemPrefix, $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) {
+       public static function getObjectTypeAlikeValueValidator($languageItemPrefix, $minimumSegmentCount = 4) {
                return new FormFieldValidator('format', function(TextFormField $formField) use ($languageItemPrefix, $minimumSegmentCount) {
                        if ($formField->getValue()) {
                                $segments = explode('.', $formField->getValue());
index 1e188829acd83f6a3815f2a3a4f8349d015074c3..c5b1e98d2e6fdb4a418eaf45eec593115bf74fd5 100644 (file)
@@ -144,7 +144,7 @@ class PIPPackageInstallationPlugin extends AbstractXMLPackageInstallationPlugin
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false) {
+       protected function getElementData(\DOMElement $element, $saveData = false) {
                return [
                        'className' => $element->nodeValue,
                        'pluginName' => $element->getAttribute('name'),
index 7ed1649b54aec70a3e8df8383af873e907c5013b..916975822107983bae20bdbcdb768ab4a84d4122 100644 (file)
@@ -636,7 +636,7 @@ class PagePackageInstallationPlugin extends AbstractXMLPackageInstallationPlugin
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false) {
+       protected function getElementData(\DOMElement $element, $saveData = false) {
                $data = [
                        'identifier' => $element->getAttribute('identifier'),
                        'originIsSystem' => 1,
@@ -668,7 +668,7 @@ class PagePackageInstallationPlugin extends AbstractXMLPackageInstallationPlugin
                        }
                }
                
-               $readData = function(int $languageID, \DOMElement $content) use (&$data) {
+               $readData = function($languageID, \DOMElement $content) use (&$data) {
                        foreach (['title', 'content', 'customURL', 'metaDescription', 'metaKeywords'] as $contentElementName) {
                                $contentElement = $content->getElementsByTagName($contentElementName)->item(0);
                                if ($contentElement) {
index 2b786d45a5952c8fa6bfeebd2581253c0b82ac20..f5f2a8b99c2ad4a04ab235d64f160b045786e9b8 100644 (file)
@@ -324,7 +324,7 @@ class TemplateListenerPackageInstallationPlugin extends AbstractXMLPackageInstal
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false) {
+       protected function getElementData(\DOMElement $element, $saveData = false) {
                return [
                        'environment' => $element->getElementsByTagName('environment')->item(0)->nodeValue,
                        'eventName' => $element->getElementsByTagName('eventname')->item(0)->nodeValue,
index 53afd19c2470d7e91a65e7222da61e83c76a7afe..1529d617fe756684bdce7992449e84a3ed1eab20 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) {
+               $menuItemsSupportingLinks = array_keys(array_filter($menuItemLevels, function($menuItemLevel) {
                        return $menuItemLevel >= 2;
                }));
                
@@ -126,7 +126,7 @@ class UserMenuPackageInstallationPlugin extends AbstractMenuPackageInstallationP
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false) {
+       protected function getElementData(\DOMElement $element, $saveData = false) {
                $data = parent::getElementData($element);
                
                $className = $element->getElementsByTagName('classname')->item(0);
index 2ae02cc09055d28ca3a2c45327c94ed94dcf6ec6..07abbe635ce37827d434edbac4281c1e6c419a2b 100644 (file)
@@ -280,7 +280,7 @@ class UserNotificationEventPackageInstallationPlugin extends AbstractXMLPackageI
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false) {
+       protected function getElementData(\DOMElement $element, $saveData = false) {
                $data = [
                        'className' => $element->getElementsByTagName('classname')->item(0)->nodeValue,
                        'objectTypeID' => $this->getObjectTypeID($element->getElementsByTagName('objecttype')->item(0)->nodeValue),
index 5879b9e6881ac6c0121a251038d98589d0bc8530..dbc609a20b56399a80fe77890d861bac27582281 100644 (file)
@@ -178,7 +178,7 @@ class UserProfileMenuPackageInstallationPlugin extends AbstractXMLPackageInstall
         * @inheritDoc
         * @since       3.2
         */
-       protected function getElementData(\DOMElement $element, bool $saveData = false) {
+       protected function getElementData(\DOMElement $element, $saveData = false) {
                $data = [
                        'className' => $element->getElementsByTagName('classname')->item(0)->nodeValue,
                        'menuItem' => $element->getAttribute('name'),