GX-Task #45652
Von Daniel Würdemann vor etwa 8 Jahren aktualisiert
Ab PHP 7 werden Warnungen ausgegeben, wenn Methoden in erbenden Klassen überschrieben werden und sich dabei die Parametersignatur ändert. Dies gilt nicht für den Konstruktor, der das Überschreiben mit unterschiedlichen Parametern erlaubt. Daher wurden folgende Methodensignaturen gemäß der geerbten Methoden vereinheitlicht: * **gmPDF:** *admin/gm/classes/gmPDF.php* * `SetProtection($permissions = array(), $user_pass = '', $owner_pass = NULL)` **=>** `SetProtection($permissions = array(), $user_pass = '', $owner_pass = NULL, $mode = 0, $pubkeys = NULL)` Ruft `_generateencryptionkey_modified()` statt `_generateencryptionkey()` auf. * `_putstream($s)` **=>** `_putstream($s, $n = 0)` Enthält veränderten Aufruf von `parent::_putstream()`. * `_textstring($s)` **=>** `_textstring($s, $n = 0)` Enthält veränderten Aufruf von `parent::_textstring()`. * `_Ovalue($user_pass, $owner_pass)` **=>** `_Ovalue_modified($user_pass, $owner_pass)` * `_generateencryptionkey($user_pass, $owner_pass, $protection)` **=>** `_generateencryptionkey_modified($user_pass, $owner_pass, $protection)` Ruft `_Ovalue_modified_modified()` statt `_Ovalue()` auf. * **PostfinderContentView:** *system/classes/external/postfinder/PostfinderContentView.inc.php* * `get_html($p_coo_message_stack)` **=>** `get_html()` Statt `$p_coo_message_stack` wird nun `$GLOBALS['messageStack']` innerhalb der Methode genutzt. * **TrustedShopsExcellenceContentView:** *system/classes/external/trusted_shops/TrustedShopsExcellenceContentView.inc.php* * `get_html($order) **=>** get_html()` `$order` wird nun dem Konstruktor übergeben. Veränderter Konstruktor: `function TrustedShopsExcellenceContentView()` **=>** `function TrustedShopsExcellenceContentView($order)` * **FeatureSetAdminContentView:** *system/classes/filter/FeatureSetAdminContentView.inc.php* * `get_html_array($p_data_array)` **=>** `get_html_array($p_data_array = array(), $p_dummy_data_array = array())` `$p_dummy_data_array` wird nicht genutzt. * **NewsletterContentControl:** *system/classes/newsletter/NewsletterContentControl.inc.php* * `proceed($p_vvcode, $p_language = null)` **=>** `proceed()` `$p_vvcode` wird nun dem Konstruktor übergeben und statt `$p_language` wird nun `$_SESSION['language']` genutzt. Veränderter Konstruktor: `__construct($p_customer_data_array = null)` **=>** `__construct($vvCode, $p_customer_data_array = null)` * **PopupImageContentView:** *system/classes/products/PopupImageContentView.inc.php* * `get_html($p_pID, $p_imgID, $p_languages_id)` **=>** `get_html()` `$p_pID`, `$p_imgID` und `$p_languages_id` werden nun dem Konstruktor übergeben. Veränderter Konstruktor: `function PopupImageContentView()` **=>** `public function __construct(IdType $productId, IdType $imageId, IdType $languageId)` * **ProductReviewsWriteContentControl:** *system/classes/reviews/ProductReviewsWriteContentControl.inc.php* * ` proceed($p_coo_product, $p_customer_id, $p_languages_id = null)` **=>** `proceed()` `$p_coo_product` und `$p_customer_id` werden nun dem Konstruktor übergeben. Statt `$p_languages_id` wird nun `$_SESSION['languages_id']` genutzt. Veränderter Konstruktor: `__construct()` **=>** `__construct($product, $customerId)` * **WithdrawalConfirmationContentView:** *system/classes/withdrawals/WithdrawalConfirmationContentView.inc.php* * `public function get_html($p_type, $p_languages_id, $p_language, $p_filename = 'withdrawal_confirmation', $p_folder = '')` **=>** `public function get_html()` `$p_type`, wird nun über `setOutputType($outputType)` oder über `__construct($templateFilename = 'withdrawal_confirmation', $templateFolder = '')` gesetzt. `$p_filename` und `$p_folder` werden über dem Konstruktor übergeben. Statt `$p_languages_id` und `$p_language` werden nun `$_SESSION['languages_id']` und `$_SESSION['languages_id']` genutzt. Veränderter Konstruktor: `__construct()` **=>** `__construct($templateFilename = 'withdrawal_confirmation', $templateFolder = '')` `__construct(FilenameStringType $templateFilename, StringType $templateFolder)` * **ShippingAndPaymentMatrixAdminContentView:** *system/classes/shop_content/ShippingAndPaymentMatrixAdminContentView.inc.php* * `build_html()` **=>** `build_html($p_content_data_array = false, $p_template_file = false)` Beide Parameter werden in dieser Klasse nicht genutzt. Methode beleibt unverändert. * **AdminOrderOverviewTableExtenderComponent:** *src/system/extender/AdminOrderOverviewTableExtenderComponent.inc.php* * ~~`proceed(IdInterface $orderId)`~~ * `public function setOrderId(IdType $orderId)` hinzugefügt` * `getRenderedContentsCells(IdInterface $orderId)` **=>** `public function getRenderedContentsCells()` Ruft `$this->renderContentCell($columnName)` statt `$this->renderContentCell($columnName, $orderId)` auf. * `renderContentCell($columnName, IdInterface $orderId)` **=>** `renderContentCell($columnName)`