killserver 2 Опубликовано: 20 апреля 2017 Рассказать Опубликовано: 20 апреля 2017 заметил несколько ошибок в новой версии 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" хотя - они есть в дистрибутиве. Если найду ещё чего-то в таком духе - опубликую сюда-же ;-) Ссылка на сообщение Поделиться на других сайтах
celsoft 6242 Опубликовано: 20 апреля 2017 Рассказать Опубликовано: 20 апреля 2017 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 сказал: Если найду ещё чего-то в таком духе - опубликую сюда-же ;-) Это хорошо и даже полезно. Мы всегда рады когда в случае обнаружения каких то проблем нам сразу сообщают об этом. Ссылка на сообщение Поделиться на других сайтах
killserver 2 Опубликовано: 20 апреля 2017 Рассказать Опубликовано: 20 апреля 2017 Автор 2 часа назад, celsoft сказал: Таких файлов в дистрибутиве нет. В данном случае вы не внимательны. Это файлы от старых версий. В DLE 11.3 они уже не используются, и в дистрибутиве 11.3 отсутствуют. Потому антивирус и говорит что их нужно удалить. делал обновление движка просто перезаписью всех файлов, потому - мог просто не заметить того, что файлы отсутствуют 2 часа назад, celsoft сказал: Это роли никакой не играет. По сути идет проверка на то остановлено ли данное расширение, если оно установлено, доступны все функции. тогда не логичнее проверять на наличие установленного расширения самого mbstring?) Ссылка на сообщение Поделиться на других сайтах
celsoft 6242 Опубликовано: 20 апреля 2017 Рассказать Опубликовано: 20 апреля 2017 25 минут назад, killserver сказал: делал обновление движка просто перезаписью всех файлов, потому - мог просто не заметить того, что файлы отсутствуют Совершенно верно делали, потом запускается антивирус и он показывает что уже не актуально. 26 минут назад, killserver сказал: тогда не логичнее проверять на наличие установленного расширения самого mbstring?) Не на всех хостингах можно корректно получить список расширений. Поэтому проверка по функциям надежней и работает на любом хостинге и любых настройках. А для нас важна именно надежность работы, при максимальном количестве различных конфигураций, у нас же скрипт не под конкретный сервер пишется. Ссылка на сообщение Поделиться на других сайтах
Рекомендованные сообщения
Создайте аккаунт или войдите в него для комментирования
Вы должны быть пользователем, чтобы оставить комментарий
Создать аккаунт
Зарегистрируйтесь для получения аккаунта. Это просто!
Зарегистрировать аккаунтВойти
Уже зарегистрированы? Войдите здесь.
Войти сейчас