Methode check() in allen Modulen verbessern

[noRiddle]

Man kann die Methode check() in allen Modulen verbessern, in sowohl System- Shipping-, Payment-, Order-Total- als auch Export-Modulen.
Wie hpzeller in diesem Post
​MODUL: Lieferung / Versand nach Postleitzahlen (PLZ) | Antwort #21
richtig erkannt hat, lässt sich eine DB-Query sparen wenn man einfach abfragt ob die gesuchte Konstante definiert ist.

function check() {
  return defined('MODULE_TYPE_WAS_WEISS_ICH_STATUS');
}

oder von mir aus so

function check() {
  if(!isset($this->_check)) {  
    $this->_check = defined('MODULE_TYPE_WAS_WEISS_ICH_STATUS');
  }
  return $this->_check;
}

Allerdings müsste man dann auch die in manchen Modulen vorkommende Abfrage

if ($this->check() > 0) {

ändern in

if ($this->check() === true) {

Bei der Anzahl an geladenen Modulen kann das durchaus einen kleinen Perfomance-Bosst geben, auch wenn man ihn vielleicht nicht Boost nennen sollte ;-).

Gruß,
noRiddle

[noRiddle]

Ich sehe gerade, sorry, daß das in den Shipping-, Payment-, Order-Total- und Export-Modulen bereits umgesetzt ist, nicht jedoch in den System-Modulen.

Gruß,
noRiddle

[Gerhard Waldemair]

In 14913:

fix #2422 - update check method to reduce sql statements

[Torsten Riemer]

In 14922:

revised r14913 (fix #2422 - update check method to reduce sql statements) - fix format for mangalister.php

[Torsten Riemer]

In 14933:

revised r14913 (fix #2422 - update check method to reduce sql statements) - use database wrapper for mangalister.php