hello,

because of an error-message I got, I looked into the module-code and found following line in the function _get_aliasable_displays() :

$sql = "SELECT DISTINCT(vv.name) FROM drupal_views_view vv JOIN drupal_views_display vd ";

the table prefix 'drupal_' is hardcoded into the module code. This really shouldn't be so.
Some people (like me) don't use table prefixes at all, other will use different prefixes.

Would be great if you could correct this, not just for me, cause i modified the source, but for all others.

Comments

Methos76’s picture

Methos76 CreditAttribution: Methos76 commented
Title: Hardcoded Tablöe-Prefix » Hardcoded Table-Prefix
emackn’s picture

emackn CreditAttribution: emackn commented
Status: Active » Fixed

beta2 will have this fix in it. I set the release up, but it might not get rolled for a couple of hours.

Thanks for picking that up.

emackn’s picture

emackn CreditAttribution: emackn commented
Assigned: Methos76 » emackn
emackn’s picture

emackn CreditAttribution: emackn commented

assigning to me

Status: Fixed » Closed (fixed)

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

catofcheshir’s picture

catofcheshir CreditAttribution: catofcheshir commented
Version: 6.x-2.0-beta1 » 6.x-2.0-beta7
Status: Closed (fixed) » Active

It's not fixed for beta7 :( Have to remove prefix manually.

emackn’s picture

emackn CreditAttribution: emackn commented

how did this get back in there? ;) Thanks for catching it. I'll have a new release out later today.

emackn’s picture

emackn CreditAttribution: emackn commented
Status: Active » Fixed

fixed with beta8

aroq’s picture

aroq CreditAttribution: aroq commented

Seems still not fixed in beta 8 line 123:

$sql = "SELECT DISTINCT(vv.name) FROM {views_view} vv JOIN drupal_views_display vd ";

emackn’s picture

emackn CreditAttribution: emackn commented

Funny how you can look right at something and not see it. Fixed in beta9.

Status: Fixed » Closed (fixed)

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

vbilici’s picture

vbilici CreditAttribution: vbilici commented

same problem on line 175 with 6.x 2.0