Projekt

Allgemein

Profil

Aktionen

GX-Task #45652

geschlossen

Signatur von überladenen Methoden vereinheitlicht

Von Daniel Würdemann vor mehr als 8 Jahren hinzugefügt. Vor mehr als 8 Jahren aktualisiert.

Status:
Erledigt
Priorität:
Normal
Zugewiesen an:
Daniel Würdemann
Kategorie:
Change
% erledigt:

100%

Betroffene Versionen:
Unbestimmt

Beschreibung

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 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 = '')
  • 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)

Aktionen #1

Von Daniel Wu vor mehr als 8 Jahren aktualisiert

  • Status wurde von Gemeldet zu Erledigt geändert
  • % erledigt wurde von 0 zu 100 geändert
Aktionen #2

Von Daniel Wu vor mehr als 8 Jahren aktualisiert

  • Thema wurde von Signatur von überladeten Methoden vereinheitlicht zu Signatur von überladenen Methoden vereinheitlicht geändert
Aktionen #3

Von Daniel Würdemann vor mehr als 8 Jahren aktualisiert

Aktionen #4

Von Daniel Würdemann vor mehr als 8 Jahren aktualisiert

Aktionen

Auch abrufbar als: Atom PDF