Обновы для плейлистов и поиска

[mrilyew]

• Отрефакторена вёрстка страницы поиска, а так же то, что от неё в шапке. Кастомные темы по идее больше не должны ломаться. Удалены лишние опции поиска. Поиск по комментам удалён из-за ненадобности. У аудио добавлен параметр "по жанру", что fixes [Аудиозаписи] Сортировка музыки по жанру #1082. Добавлено подсвечивание найденного жёлтым цветом.
  
• Добавлен поиск по плейлистам. Поддерживается order по дате создания, длине и числу прослушиваний. Плейлисты теперь могут быть unlisted.
  
• Исправлено поведение плейлистов, что они показывали аудиозаписи не в том порядке, в котором их добавляли. Теперь можно загружать аудио прямо в плейлист, минуя загрузку к себе.
• Добавлена кнопка скачивания аудиозаписи (ибо чё жлобиться то). При нажатии на кнопку "полностью удалить аудиозапись" теперь показывается предупреждение (fixes [Аудиозаписи] Подтверждение удаления аудиозаписи + фича #1035 частично).
  
• Нажатие на обложку плейлиста на странице плейлиста приведёт к открытию фото с данной обложкой. Убрано "undefined" если комменты фото не получилось загрузить.
• Удалена кнопка у аудио "добавить в группу". Чтоб добавить аудио в группу нужно нажать кнопку добавления с alt. Пофикшен баг что ползунки аудио не исчезали при включении другого аудио. Пофикшен баг, что нельзя было вводить пробелы в строку поиска на странице с плеером. Fixed Проблема с скроллингом страницы с музыкой  #1023.
  

[celestora]

Не буду всё смотреть, слишком дохуя всего, словно мердж опенвк с каким-то апстримом :D
Впрочем, я так понимаю, у ОР это очень долго варилось на локалке и он решил сразу всё слить.

Учитывая общее качество кода в этом репозитории, я если честно не думаю, что мне должно быть достаточно не похуй чтобы доёбываться до всего.

Отмечу два спорных момента:

1. Вижу несколько раз используются странные switch по типу
   конвертации чего-то из цифры в строчку или что-то вроде case "amogus": $a = "amogus" . something; break; case "jopa": $a = "jopa" . something; break;. Имхо можно заменить на более простые конструкции (в первом случае ["a", "b", "c"][$n % 3], во втором просто вначале проверить что строка в валид через in_array.
2. Сортировку во нек-рых методах поменяли со строки зачем-то в какой-то ассоциативный массив. Не знаю зачем, но очень просится сделать из этого какой-то объект и перенести в него методы для конвертации в строку для запроса.
3. Интересен вопрос быстродействия, вижу, что меняется работа с поиском, не вижу патчей с добавлением индексов. В принципе для общего поиска на индексы похуй, медленно так медленно, но если это поиск аудио, которые в плейлистах, мейби можно чёт подкрутить хз.

На будущее лучше такие жирные пул реквесты не делать, их потом невозможно предметно обсудить. Хорошо, что коммиты есть, но лучше всё-таки каждую фичу отдельным PR :)

Ну и учитывая то как мы ищем по постам встаёт вопрос не пора бы нам перелезть на какой-то solr там хз но это так.

[celestora]

[veselcraft]

Есть некоторый момент а так мне функционал понравился вкуснятина

[xRy0]

А потом ты жалуешься что в резюме не хочешь овк сувать xdd

[veselcraft]

[veselcraft]

@celestora если не сделаеш превью за 3 дня я приму это автоматом

[veselcraft]

первый день пошёл

[veselcraft]

vj[eq