GyverLibs/GyverPortal

GPfileType не определяет тип файла.

Depech101 opened this issue · 9 comments

В методе : String GPfileType(const String &uri)
Функция GPinList(ext, FPSTR(pgm_read_dword(_gp_ext + i))); возвращает всегда -1 вне зависимости от типа файла.
Соответственно всегда отправляется тип файла по умолчанию, "Text/plain"
Тогда как абсолютно такой же метод из портала версии 3.0 работает совершенно верно.
например если мы открываем index.html to GPinList первый раз возвращает -1 а второй 2 и все работает корректно.

Дополнение.
В методе GPinList обнаружилось несоответствие
В версии3.5 t = li.lastIndexOf(s, p);
В версии3.0 t = li.indexOf(s, p);
Сделал как в 3.0 все заработало.

изображение

спасибо. Осталось вспомнить, зачем был сделан lastIndexOf...

Вот и я том же, цель какая то была))

Все таки, концепцию Clean code не зря придумали, попробуйте описывать переменные более емко, этого всегда не хватает.

кажется цель была обрабатывать сжатые .gz файлы, и данный lastIndexOf должен был быть в строке 20 ultis.cpp, чтобы брать последнее расширение после последней точки 🤦‍♂️

не в названиях дело, функцией ошибся)

Да конечно) Спасибо за ваш труд.

спасибо за репорт, кто знает к чему этот косячок мог бы привести 🙈🙈🙈🙈

Главное картинки браузер открывал которые я из корня запускал а html выводил как текст