Fix the sample SQL when the source has no database

[b-per]

resolves #264

Description

Fix for the issue raised where the sample SQL Code is incorrect when the data source isn't linked to any database (like in databricks).

To test it, I updated the database field in my manifest.json, replacing "jaffle_shop" by "".

Behaviour before the change, when no DB is defined:

Behaviour after the change, when no DB is defined:

Behaviour after the change, when a DB is defined:

Checklist

• I have signed the CLA
• I have generated docs locally, and this change appears to resolve the stated issue
• I have updated the CHANGELOG.md and added information about my change to the "dbt next" section.

[stu-k]

LGTM! 👍

[emmyoop]

Looks good. I tested locally and it works as expected!

This is going to go in to 1.2.0. If you pull in recent changes, there's a spot in the CHANGELOG for that now. Please update your CHANGELOG entry to be under the accurate release and I'll get this merged in. Also, don't forget to add yourself under Contributors 😃

Thank you so much for the contribution @b-per!

[emmyoop]

Looks good. Thanks again!

[martinburch]

Sorry, bit late to this, and many thanks to the reviewers, but is it ok that this code will now require model.identifier whereas the old code tried to use model.identifier, model.alias, or model.name?

[b-per]

I think that it is OK, yes.
The previous code was more generic and had some logic that could apply to models as well as sources. The bit of code changed is actually only executed for sources so we don't have to worry about alias (which don't apply to sources) and so on.