Как переписать правильно запрос SQL чтобы не выдавало ошибку?

Question

[Анна Денисова]

$sql = "SELECT  m.*,pp.*,p.id,p.originalpid,p.added,p.name,p.owner,p.favorite_cover,COUNT(pp.trackid)
            FROM ".nobil_om_playlists." p
         inner join ".nobil_om_playlist_pos." pp on pp.playlistid = p.id
         inner join ".nobil_om_songs." m on m.id = pp.trackid
         WHERE p.id=" . $this->id . "
          group by pp.trackid ORDER BY pp.position ASC, pp.id DESC LIMIT " . $pl_songs_limit;

Answer 1

[Станислав Шендаков]

Текст ошибки приведите. И итоговую строку запроса ( echo $sql ).

Comments

[Анна Денисова]

Expression #16 of SELECT list is not in GROUP BY clause and contains nonaggregated column 'druzer.pp.id' which is not functionally dependent on columns in GROUP BY clause; this is incompatible with sql_mode=only_full_group_by

[Анна Денисова]

{foreach $rows as $res}
{assign var=pl_name value={urldecode($res.name)}}
{assign var=pl_new value={strtotime($days,$res.added)}}
{assign var=pl_id value={$res.id}}
{assign var=pl_songs_count value={$res.songs_count}}
{assign var=pl_cover value={$res.favorite_cover}}

[Станислав Шендаков]

В запросе вместо pp.* укажите например pp.playlistid, или pp.trackid. И согласен с d-stream, "group by и звездочки в селекте - не есть нормально."

Answer 2

[d-stream]

Без обдумывания: group by и звездочки в селекте - не есть нормально. Хотя mysql типа прощает.
Ну и дальше нагруппировалась сотня строк по pp.track_id - то есть в итоге одна строка, у которой "вовнутрях" два десятка разных pp.position - и какую из них и почему sql должен использовать как критерий сортировки (order by)?

Comments

[Анна Денисова]

Добрый день! Вы не поможете бедной девушке с этим одним оставшимся запросом. Все как-то быстро переделала, а этот не поддаётся. Ошибка началась и-за того, что на хостинге не разрешают настойки sql mode.

[d-stream]

Вообще нормальная практика:
0. приводить текст ошибки
1. уточнять диалект
2. приводить удобный вариантик генерации таблиц - нечто типа
нескольких строк create table и несколько insert в них
+ кратенько что означают таблицы и что хочется получить

тогда есть огромная вероятность что желающим помочь останется только дописать просимое и убедиться на образчике данных что там все хорошо

[Анна Денисова]

d-stream, Expression #16 of SELECT list is not in GROUP BY clause and contains nonaggregated column 'druzer.pp.id' which is not functionally dependent on columns in GROUP BY clause; this is incompatible with sql_mode=only_full_group_by

{foreach $rows as $res}
{assign var=pl_name value={urldecode($res.name)}}
{assign var=pl_new value={strtotime($days,$res.added)}}
{assign var=pl_id value={$res.id}}
{assign var=pl_songs_count value={$res.songs_count}}
{assign var=pl_cover value={$res.favorite_cover}}

Тут происходит сортировка песен. Это музыкальный плагин.

[d-stream]

Собственно - как изначально и озвучивалось про группировку и звездочки.

Правильный и человеческий вариант при группировке - все отображаемые в select колонки должны или быть элементами группировки или агрегированы или литералами.

Про что sql и говорит. И даже внятно показывает ту самую проблемную колонку druzer.pp.id
они либо должна фигурировать в критерии группировки (group by ) или же агрегироваться в выводе (min/max/sum/avg/etc)