I upgraded to RC4, now when I go to Orphaned Authors, I get this user warning:
You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'GROUP BY cid) LIMIT 0, 50' at line 1 query: SELECT * FROM biblio_contributor_data WHERE (n.language ='nl' OR n.language ='' OR n.language IS NULL) AND ( cid NOT IN (SELECT cid FROM biblio_contributor )GROUP BY cid) LIMIT 0, 50 in sites/all/modules/biblio/biblio.admin.inc on line 1201.

Using MySQL 5.0.51a on LAMP; and this happens on a multillingual site (English, Spanish, Dutch). Dutch is default, but the Biblio content type does not have translation enabled. All biblio nodes are language neutral.

And oh, when I visit the Orphaned Authors on on the Spanish version of the site, I get:
user warning: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'GROUP BY cid) LIMIT 0, 50' at line 1 query: SELECT * FROM biblio_contributor_data WHERE (n.language ='es' OR n.language ='' OR n.language IS NULL) AND ( cid NOT IN (SELECT cid FROM biblio_contributor )GROUP BY cid) LIMIT 0, 50 in sites/all/modules/biblio/biblio.admin.inc on line 1201.

Comments

rjerome’s picture

rjerome CreditAttribution: rjerome commented

Could you try replacing line 1201 with the following...

$result = pager_query('SELECT * FROM {biblio_contributor_data} WHERE cid NOT IN (SELECT cid FROM {biblio_contributor} GROUP BY cid)', 50);

I don't think the rewrite_sql is required on this query since it's internal to biblio.

ar-jan’s picture

ar-jan CreditAttribution: ar-jan commented

Yup, that got rid of the error, and now it shows I have two orphaned authors.

However, the orphaned authors name links to "http://mydomain.com//authors/50/edit" (with to slashes and no 'biblio' in the path').

But very cool functionality!

rjerome’s picture

rjerome CreditAttribution: rjerome commented

adding...

  $base = variable_get('biblio_base', 'biblio');

as line 1195 should fix that.

rjerome’s picture

rjerome CreditAttribution: rjerome commented

BTW, thanks goes to Robert Haschke for that feature...

ar-jan’s picture

ar-jan CreditAttribution: ar-jan commented
Status: Active » Fixed

Yup, that fixed it. Looks good, and amazing speed! ;)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.