Problem with data retrieval and string translation

Resolved devterramain
(@devterramain)

2 weeks, 3 days ago
Вітаю! Має досвід з використання вашого плагіна.

Є декілька багів котрі знаходяться в методі get_nova_poshta_warehouse класу MRKV_UA_SHIPPING_AJAX_NOVA

1. При спробі отримання відділень нової почти з поштоматами отримую 500 помилку, і

PHP Fatal error: Allowed memory size of 268435456 bytes exhausted.

Цей баг спостерігається тільки для міста Києва.

Додайте будь ласка якийсь фільтр для дефолтної кількості отриманих результатів через АПІ 1000 це забагато. Для сервера де не можна збільшити ліміт.

2. Я на сайт при використані WPML, і є проблеми з перекладом рядків, є лейбл котрий, котрий записується в transient, і потім з нього показується.

Виводить так то хто перший зайшов на сайт зробив transient тоді цей лебл потім показується для всіх мов.
- This topic was modified 2 weeks, 3 days ago by devterramain.

Viewing 8 replies - 1 through 8 (of 8 total)

Plugin Author Ihor Kit
(@bandido)

2 weeks, 3 days ago

@devterramain Доброго дня, дякую за баг репорт. Про ліміт маєте рацію, виправимо в найближчому оновленні у понеділок-вівторок.

Стосовно WPML поки не дуже розумію. Transient точно не використовується. Де саме видно неправильну мову — плейсхолдер у <select> («Оберіть відділення»), назва способу доставки, чи назви самих відділень (вони з API НП і завжди українською)?

Thread Starter devterramain
(@devterramain)

2 weeks, 3 days ago

Доброго дня. Дякую за швидку відповідь.

Стосовно WPML

Це для плейсхолдера і першого елемента списку вибору відділення.

Помилка в класі morkva-ua-shipping\classes\shipping_methods\nova-poshta\ajax\mrkv-ua-shipping-methods-ajax-nova.php

В методі get_nova_poshta_warehouse()

на 297 рядку формується зміна $label від налаштувань у метода доставки.

на 300 рядку зміна $label додається в масив $areas

на 332 рядку масив $areas зберігається в кеш вордпреса через функціюцію set_transient.

І потом повертається кеширований результат на рядку 241, якщо він існує

Тому і трапляється той баг, хто перший зайшов на сайт зробив transient тоді цей лебл потім показується для всіх мов.

Plugin Author Ihor Kit
(@bandido)

1 week, 6 days ago

@devterramain у версії 1.10.2 буде виправлено

Plugin Author Ihor Kit
(@bandido)

1 week, 3 days ago

@devterramain прошу підтвердити чи баг виправлений

Thread Starter devterramain
(@devterramain)

1 week, 3 days ago

@bandido

Дякую за підтримку. Стосовно мої кейсів, все виправлено. Але хочу повідомити, що коли перевіряв, знайшов інший баг. Не правильно працює аякс підвантаження нових відділень, коли долістуєшь до кінця першого набору відділень, то потім відправлять аякс з page 2, але в результаті показується перший набір відділень і так кожного разу.

Plugin Author Ihor Kit
(@bandido)

1 week ago

@devterramain перевірив на кількох тестових сайтах (наприклад https://demoshop.morkva.co.ua) – не відтворюється.
(Обираю місто Київ, переходжу в вибір відділення і скролю вниз. Відділення підвантажуються по 20 послідовно)
Можете надати більше деталей?

Thread Starter devterramain
(@devterramain)

1 week ago

@bandido

Доброго дня. Цей баг відтворюється при увімкненому налаштуванні

search_by_number методу оплати з описом Пошук відділення здійснюється після введення номера

Якщо вимкнути то налаштування то спочатку завантажується кеш якщо він існує (у мене 50 відділень) потім відділення по 20шт.

Як я бачу на вашому тестовому сайт то налаштування вимкнене. Тому у вас немає того бага.

Plugin Author Ihor Kit
(@bandido)

1 week ago

@devterramain дякую, виправили в 1.10.4

Viewing 8 replies - 1 through 8 (of 8 total)

You must be logged in to reply to this topic.