Перейти к публикации

Немного ошибок в версии 11.3


Рекомендованные сообщения

заметил несколько ошибок в новой версии 11.3:

function dle_strpos($str, $needle, $charset ) {

идёт проверка "mb_strrpos" вместо "mb_strpos".

Далее, антивирус ругается на файл "engine/classes/typograf.class.php", "engine/ajax/typograf.php" и "engine/classes/html5player/flashmediaelement.swf" хотя - они есть в дистрибутиве. Если найду ещё чего-то в таком духе - опубликую сюда-же ;-)

Ссылка на сообщение
Поделиться на других сайтах
1 час назад, killserver сказал:

идёт проверка "mb_strrpos" вместо "mb_strpos".

Это роли никакой не играет. По сути идет проверка на то остановлено ли данное расширение, если оно установлено, доступны все функции.

1 час назад, killserver сказал:

Далее, антивирус ругается на файл "engine/classes/typograf.class.php", "engine/ajax/typograf.php" и "engine/classes/html5player/flashmediaelement.swf" хотя - они есть в дистрибутиве.

Таких файлов в дистрибутиве нет. В данном случае вы не внимательны. Это файлы от старых версий. В DLE 11.3 они уже не используются, и в дистрибутиве 11.3 отсутствуют. Потому антивирус и говорит что их нужно удалить.

1 час назад, killserver сказал:

Если найду ещё чего-то в таком духе - опубликую сюда-же ;-)

Это хорошо и даже полезно. Мы всегда рады когда в случае обнаружения каких то проблем нам сразу сообщают об этом.

Ссылка на сообщение
Поделиться на других сайтах
2 часа назад, celsoft сказал:

Таких файлов в дистрибутиве нет. В данном случае вы не внимательны. Это файлы от старых версий. В DLE 11.3 они уже не используются, и в дистрибутиве 11.3 отсутствуют. Потому антивирус и говорит что их нужно удалить.

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

 

2 часа назад, celsoft сказал:

Это роли никакой не играет. По сути идет проверка на то остановлено ли данное расширение, если оно установлено, доступны все функции.

тогда не логичнее проверять на наличие установленного расширения самого mbstring?)

Ссылка на сообщение
Поделиться на других сайтах
25 минут назад, killserver сказал:

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

Совершенно верно делали, потом запускается антивирус и он показывает что уже не актуально. 

26 минут назад, killserver сказал:

тогда не логичнее проверять на наличие установленного расширения самого mbstring?)

Не на всех хостингах можно корректно получить список расширений. Поэтому проверка по функциям надежней и работает на любом хостинге и любых настройках. А для нас важна именно надежность работы, при максимальном количестве различных конфигураций, у нас же скрипт не под конкретный сервер пишется.

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

Присоединяйтесь к обсуждению

Вы можете опубликовать сообщение сейчас, а зарегистрироваться позже. Если у вас есть аккаунт, войдите в него для написания от своего имени.

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

×   Вставлено в виде отформатированного текста.   Вставить в виде обычного текста

  Разрешено не более 75 эмодзи.

×   Ваша ссылка была автоматически встроена.   Отобразить как ссылку

×   Ваш предыдущий контент был восстановлен.   Очистить редактор

×   Вы не можете вставить изображения напрямую. Загрузите или вставьте изображения по ссылке.

×
×
  • Создать...