When using external database, the conf is like $databases['sso']['default'] = array(...). The current code does not use 'default' but use the connection name (['sso']['sso']). Patch attached (from @DeFr).

CommentFileSizeAuthor
#8 views_data_export_external.patch643 bytessteven jones
#4 views_data_export_external.patch567 bytessteven jones
views_data_export_external.txt567 bytesjcisio

Comments

steven jones’s picture

steven jones commented
Status: Active » Needs review

Marking for review.

steven jones’s picture

steven jones commented
Status: Needs review » Needs work
Issue tags: +Needs tests

But needs tests.

steven jones’s picture

steven jones commented
Status: Needs work » Needs review

Marking for review, I'm not sure this patch applies any more?

steven jones’s picture

steven jones commented
StatusFileSize
new views_data_export_external.patch567 bytes
steven jones’s picture

steven jones commented

Ah, I see what this is doing now!

Right, if this applies we'll commit it.

Status: Needs review » Needs work

The last submitted patch, views_data_export_external.patch, failed testing.

steven jones’s picture

steven jones commented

Patch needs a re-roll.

steven jones’s picture

steven jones commented
Status: Needs work » Needs review
StatusFileSize
new views_data_export_external.patch643 bytes

Re-rolled.

steven jones’s picture

steven jones commented
Status: Needs review » Fixed

Many thanks, committed to 7.x-3.x.

jcisio’s picture

jcisio commented

Cool ;)

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