COVID2019 и это вот все. Друзья, вся эта история начинает плохо пахнет. Мойте руки, не ходите в люди. Отложите все плановые покупки и положите в носок заначку. Заприте ваших родителей, бабушек-дедушек на даче. Лучше перебдеть чем недобдеть. Берегите себя!

Обозначение рубля РФ (ocShop v2.1.0.1.3)


Recommended Posts

кашу маслом базу индексами не испортишь, по принципу - есть выборка, неплохо бы иметь под нее индекс

он бесполезен не из-за отсутствия смысла при выборке 3 строк (можно наполнить и сотнями одинаковых кодов десятков валют с нулевым статусом, мало ли кто как хранит их, а потом страдает), а из-за самой идеи запроса в дефолтном исполнении, там один хрен все строки перебираются

 

кстати о кэше

ModelLocalisationCurrency кэш использует

Currency - нет

 

по остальному даже и спорить не с чем, что ж, идея прекрасна

 

брошу камешек в оригинальный опенкарт


выборка в ModelLocalisationCurrency->getCurrencyByCode (это которая "SELECT DISTINCT *..." по `code`)
при отсутствии уникальности поля `code` имеет все шансы выбирать что попало

Ссылка на комментарий
Поделиться на других сайтах

Страны решили пока не трогать апро ошибки подробнее где что на что поменять

А можно сделать вариант где есть страна к примеру РФ и убраны все другие?

Ссылка на комментарий
Поделиться на других сайтах

А можно сделать вариант где есть страна к примеру РФ и убраны все другие?

Украинцы тогда скажут убрать все кроме Украины :-) Белорусы все кроме Белоруссии и так далее :-)

страны лишние вы можете удалить в админке ну минут за 5ть

Ссылка на комментарий
Поделиться на других сайтах

Украинцы тогда скажут убрать все кроме Украины :-) Белорусы все кроме Белоруссии и так далее :-)

страны лишние вы можете удалить в админке ну минут за 5ть

 

Я понял вас.

Ссылка на комментарий
Поделиться на других сайтах

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Гость
Ответить в тему...

×   Вы вставили отформатированное содержимое.   Удалить форматирование

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

  • Сейчас на странице   0 пользователей

    • Нет пользователей, просматривающих эту страницу