[Сайт] Запросы песен

[scorp]

Я не заметил проблем с кирилицей, вроде нормально все отправляется и работает.

Правда есть недоработка, не все может находить, так как сравнение идет в разных регистрах..

Нужно в файле request.php в строках 37-38 добавить приведение в нижний регистр

// Получаем данные из формы
$artist = isset($_POST['artist']) ? mb_strtolower(trim($_POST['artist'])) : '';
$title = isset($_POST['title']) ? mb_strtolower(trim($_POST['title'])) : '';

так же как приводятся в нижний теги из базы на строке 61-62

$found_track = false;
foreach ($xml->Track as $track) {
    $track_artist = mb_strtolower(trim((string) $track['artist']));
    $track_title = mb_strtolower(trim((string) $track['title']));

Или только для условия, чтоб в остальных местах и влоге было как введено в форму и с нее получено (строки 66-67)

if (
        (stripos($track_artist, mb_strtolower($artist)) !== false || empty($artist)) &&
        (stripos($track_title, mb_strtolower($title)) !== false || empty($title))(

 

[scorp]

С русскими символами проблема. в кеш пишет. Посмотреть вложение 6573

Я тоже это не очень люблю, сейчас нормально уже поддерживается кириллица, можно исправить чтоб писало как есть без экранирования, для этого в файле tracks.php в строке 63 надо добавить флаг JSON_UNESCAPED_UNICODE

if (!file_put_contents($cache_file, json_encode($cache_data, JSON_UNESCAPED_UNICODE | JSON_PRETTY_PRINT), LOCK_EX)) {

Но вообще конечно это все пофиг ибо никто в этот кеш смотреть не будет и нафиг он никому не сдался там гулять, кроме как скрипту, а ему пофиг что там экранирование ))

 

[mixadance]

Я тоже это не очень люблю, сейчас нормально уже поддерживается кириллица, можно исправить чтоб писало как есть без экранирования, для этого в файле tracks.php в строке 63 надо добавить флаг JSON_UNESCAPED_UNICODE

if (!file_put_contents($cache_file, json_encode($cache_data, JSON_UNESCAPED_UNICODE | JSON_PRETTY_PRINT), LOCK_EX)) {

Но вообще конечно это все пофиг ибо никто в этот кеш смотреть не будет и нафиг он никому не сдался там гулять, кроме как скрипту, а ему пофиг что там экранирование ))

Скорректировал на Гитхабе. Сам не проверял