Generate name for StorageFile based on its mime type (#726)

[Alienjob]

Resolves #726

Synopsis

Когда в профиле загружена аватарка, при её скачивании из галереи скачивается файл без расширения. При проверке обнаружилось, что при скачивании в name передаётся checksum, и файл имеет имя checksum без расширения.

Solution

Добавить метод generateFileName(url, checksum) в PlatformUtilsImpl
При сохранении в галерею когда расширение файла не задано - генерировать новое имя
Явно генерировать имя файла при сохранении аватара
При генерации для имени файла использовать его контрольную сумму или, если она не задана, текущее время
Для определения расширения файла проверить задано ли оно в url, если не задано - определить mime тип файла и назначить верное расширение

Checklist

• Created PR:
  • In draft mode
  • Name contains issue reference
  • Has type and k:: labels applied
• Before review:
  • Documentation is updated (if required)
  • Tests are updated (if required)
  • Changes conform code style
  • CHANGELOG entry is added (if required)
  • FCM (final commit message) is posted or updated
  • Draft mode is removed
• Review is completed and changes are approved
  • FCM (final commit message) is approved
• Before merge:
  • Milestone is set
  • PR's name and description are correct and up-to-date
  • All temporary labels are removed

[Alienjob]

FCM

Implement `StorageFile.name` getter to use in `BigAvatarWidget` (#756, #726)

Additionally:
- add `prefer_relative_imports` linter rule

[Alienjob]

@SleepySquash Мы обсуждали что метод генерации имени можно расположить внутри класса StorageFile. Мне все еще кажется что это перегрузит модель лишними зависимостями от MimeResolver и CacheWorker. Как думаете, стоит его перенести внутрь класса или оставить в отдельном файле?

[SleepySquash]

@Alienjob, не вижу проблем с импортом дополнительных классов в модель. Модели бизнес-логики вполне себе могут быть большим, чем просто DTO (примеры: Attachmentы, OngoingCall). Кроме того, если имя спрятать в отдельный класс, то будет больше зависимостей у классов, которые это имя захотят получать, тоже неприятно.

[SleepySquash]

@Alienjob, имя и эмейл Вашего аккаунта в Git'е не совпадает с именем и эмейлом Вашего аккаунта на GitHub'е - посмотрите коммиты. Получается так, словно два разных человека ведут работу над PRом - GitHub будет считать Вас двумя людьми. Исправьте это, пожалуйста.

[Alienjob]

@Alienjob, имя и эмейл Вашего аккаунта в Git'е не совпадает с именем и эмейлом Вашего аккаунта на GitHub'е - посмотрите коммиты. Получается так, словно два разных человека ведут работу над PRом - GitHub будет считать Вас двумя людьми. Исправьте это, пожалуйста.

@SleepySquash Исправил настройки. В новых коммитах будет идентично гитхабу. Если я правильно понимаю, при слиянии пул реквеста ветка будет удалена вместе с некоректными автором/почтой, поэтому старые коммиты не исправляю

[Alienjob]

@SleepySquash Для веб:
Браузер берет имя из url, который оканчивается "/.jpg" и добавляет расширение ".jpeg" Получается jpg (4).jpeg.
Я не нашел способа изменить такое поведение.
whatwg/html#2722 - обсуждение проблемы без решения
https://html.spec.whatwg.org/#as-a-download - описание стандарта, который запрещает использовать чтот-то кроме ссылки

Есть тег "download" но он работает для ссылки .
Наверное нужна кнопка, или переопределить контекстное меню. Не уверен в необходимости - скайп веб просто блокирует контекстное меню на экране аватара и не дает сохранить картинку.

[SleepySquash]

@Alienjob, т.е. в вебе вообще никаким образом невозможно указать имя сохраняемого файла? А есть источники какие-нибудь, которые это подтверждают?

[SleepySquash]

@Alienjob, добавление basenameа тоже не работает?

<img src="…" basename="dog" />

[Alienjob]

@SleepySquash
Пробовал basename, download, filename

[SleepySquash]

@Alienjob, понял. Тогда пометьте, пожалуйста, в коде место, где вызывается скачивание веб файла, тудушкой по типу:

// TODO: Wait for HTML to support specifying download name:
//       https://github.com/whatwg/html/issues/2722