Hi,
I get the following error when I look at the Signup administration page.
warning: pg_query() [function.pg-query]: Query failed: ERROR: column "n.type" must appear in the GROUP BY clause or be used in an aggregate function in /var/www/html/includes/database.pgsql.inc on line 139.
user warning: query: SELECT n.nid, n.title, n.type, s.status AS signup_status, COUNT(s_l.nid) AS signup_total, s.close_signup_limit AS signup_close_signup_limit FROM node n INNER JOIN signup s ON s.nid = n.nid LEFT JOIN signup_log s_l ON s.nid = s_l.nid GROUP BY n.nid, n.title, signup_status, signup_close_signup_limit ORDER BY n.title ASC LIMIT 25 OFFSET 0 in /var/www/html/modules/signup/includes/admin.signup_administration.inc on line 76.
Also, when I create new content, I don't see an option to make the content "Signup enabled". I'm no Drupal expert so I'm assuming it has something to do with the above error.
Thanks
Comments
Comment #1
-Mania- commentedSame error, using PostgreSQL 8.1.19.
Comment #2
maikeru commentedCaused by n.type not being include in the group by clause.
More information : http://drupal.org/node/555530
Heres a patch against 6.x-1.0-rc6.
Comment #3
maikeru commentedAnother error when creating a CCK date field and linking to the sign-up content type.
Not sure if replacing backticks with quotes is correct. There was also an error relating to the date field not being included in a different group_by clause.
Attached a new patch.
Comment #4
ezra-g commentedComment #5
ezra-g commentedWorks fine on MySQL 5.1.44. Committed, thanks!
This administration page seems like it should be replaced by Views + VBO integration as a followup to #582986: Make views a required dependency and ditch non-views code.
Comment #6
ezra-g commentedComment #7
gregglesIt might be correct for postgresql, but not mysql. I get:
Attached patch switches to no table escaping and instead using a prefix on the alias name so we are sure it's never a reserved word.
Comment #8
ezra-g commentedFixed again. Thanks!
http://drupal.org/cvs?commit=477460
Comment #9
dwwThanks for fixing this! Back to the 6.x-2.x branch for joachim...
Comment #10
dwwUhh, and wtf with these tags? ;) the pgsql one makes sense, but the others? Ugh. We really need #917880: Write a handbook page about how issue tags are intended to be used ...