Загрузка изображений со стороннего сайта

Правила раздела: faq.php?mode=okay
Модератор: Модераторы

korshunov
korshunov
Репутация: 146
Сообщения: 1854
Зарегистрирован: 03.12.2015
С нами: 8 лет 3 месяца
Skype

Сообщение #1 korshunov » 29.08.2021, 06:46

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

1. Делаются две попытки загрузки (если первая неудачна). Сначала по URL, потом по такому же URL, но с http:. Изначально идея не слишком хороша, так адреса http: и https: на стороннем сервере могут работать по разному. А если уж хочется непременно такое делать, то надо бы с настройкой, делать вторую попытку или нет...
2. Из пункта 1 сразу вытекает серьезный недостаток . Если ссылка http: не рабочая, то ДВАЖДЫ выполняется одна и та же попытка. Вторая попытка - явно лишняя работа.
3. При первой попытке делается действие Перед долгим копированием займем это имя. А при второй (для исходного https:) - нет.
4. Создана функция private function isNotHttpsSource($filename). Применяется всего один раз. Лишнее усложнение на пустом месте.
5. Сама функция выглядит так:
private function isNotHttpsSource($filename)
{
if (!preg_match("~^https://~", $filename)) {
return true;
}
return false;
}
В то время как само собой напрашивается более простое
private function isNotHttpsSource($filename)
{
return !preg_match("~^https://~", $filename)) ;
}
6. Сами попытки оформлены отдельными участками кода, которые практически дублируются. Вот где надо бы функцию отдельную создавать и вызывать ее в двух местах! И тогда бы, скорее всего, ошибки 2,3 сами исчезли.

Столько ляпов в довольно простом функционале - выглядит как малоквалифицированная работа, простительная разве что начинающему...

OkayCMS M
Администратор
Аватара
OkayCMS M
Администратор
Репутация: 216
Сообщения: 1627
Зарегистрирован: 12.11.2015
С нами: 8 лет 4 месяца
Сайт Skype

Сообщение #2 OkayCMS » 30.08.2021, 10:20

Посмотрим ваши замечания, ориентировочно на следующей неделе.

korshunov
korshunov
Репутация: 146
Сообщения: 1854
Зарегистрирован: 03.12.2015
С нами: 8 лет 3 месяца
Skype

Сообщение #3 korshunov » 06.11.2021, 10:18

Вот неделя, другая проходит...
И месяц прошел, и другой...
И появилась новая версия 4.2.0, в ней изменений по этой части не видно...
В очередной раз проявляется несерьезный стиль разработчиков Okay - пообещать и не выполнить...

Dysha-Kuzbass M
Аватара
Dysha-Kuzbass M
Возраст: 37
Репутация: 3
Сообщения: 155
Зарегистрирован: 17.05.2017
С нами: 6 лет 10 месяцев
Откуда: Кузбасс
ВКонтакте

Сообщение #4 Dysha-Kuzbass » 06.11.2021, 22:31

Виталий, ждём данную реализацию в следующем snapshot`е!
Сделайте, пожалуйста! Очень удобная функция/штука!

OkayCMS M
Администратор
Аватара
OkayCMS M
Администратор
Репутация: 216
Сообщения: 1627
Зарегистрирован: 12.11.2015
С нами: 8 лет 4 месяца
Сайт Skype

Сообщение #5 OkayCMS » 17.11.2021, 15:38

Доработка сделана, доступна на гите по ссылкеhttps://github.com/OkayCMS/OkayCMS/pull/100
Войдет в новую версию, в которой будут доработки (а не только фиксы)

dimitriy
Аватара
dimitriy
Репутация: 4
Сообщения: 92
Зарегистрирован: 26.06.2016
С нами: 7 лет 9 месяцев

Сообщение #6 dimitriy » 19.11.2021, 09:03

OkayCMS писал(а):Доработка сделана, доступна на гите по ссылкеhttps://github.com/OkayCMS/OkayCMS/pull/100
Войдет в новую версию, в которой будут доработки (а не только фиксы)
то есть в 4.2.2 ?

OkayCMS M
Администратор
Аватара
OkayCMS M
Администратор
Репутация: 216
Сообщения: 1627
Зарегистрирован: 12.11.2015
С нами: 8 лет 4 месяца
Сайт Skype

Сообщение #7 OkayCMS » 19.11.2021, 10:35

dimitriy писал(а):
OkayCMS писал(а):Доработка сделана, доступна на гите по ссылкеhttps://github.com/OkayCMS/OkayCMS/pull/100
Войдет в новую версию, в которой будут доработки (а не только фиксы)
то есть в 4.2.2 ?
Нет, в 4.3.0
Если меняется в номере версии последняя цифра, то это фиксы багов, но не новый функционал.
Но если вам это нужно ранее, вы можете с гитхаба взять пулреквест и закинуть себе.


Название раздела: Баг-репорт OkayCMS (Сообщения об ошибках)
Правила раздела: faq.php?mode=okay

Быстрый ответ


Введите код в точности так, как вы его видите. Регистр символов не имеет значения.
Код подтверждения

   

Вернуться в «Баг-репорт OkayCMS (Сообщения об ошибках)»

Кто сейчас на форуме (по активности за 5 минут)

Сейчас этот раздел просматривают: 27 гостей