корекция в кода в add_eur_to_order_tax_totals()

Здравей и благодаря, че си пуснал този плъгин. Прегледах кода и е доста добре написан. Даже бих го включил в сайтовете хоствани на Go359.com

Имам само 2 препоръки.
1) Ако плъгинът би израстнал си струва да има собствена позиция на главно нива, иначе в Settings би било най-удачно или някъде в WooCommerce.
2) strpos в add_eur_to_order_tax_totals() функцията

функции, катоstrpos и подобни трябва да са в скоби, защото има странно приоритизране, особено [от лявно надясно], ако има няколко операции и има вероятност да не работи правилно. Преди години трудно открих един такъв бъг.

if ($formatted_amount && strpos($formatted_amount, $this->get_eur_label()) === false && $amount > 0) {

аз бих го заместил с този по-ефективен код.

if ($amount > 0 && !empty($formatted_amount) && (strpos($formatted_amount, $this->get_eur_label()) === false)) {
// ….
}

• This topic was modified 3 months, 2 weeks ago by Svetoslav Marinov.