dev/core#1742 Expose CiviCRM Database details for views configuration…

[seamuslee001]

… in Drupal 8 as well as in Drupal 7 and Backdrop

Overview

This exposes the necessary database configuration needed to be added to settings.php for views to work with CiviCRM tables when CiviCRM is in a different schema for Drupal 8 as well as Drupal 7 and Backdrop

Before

Views DB config only showing on Drupal 7 and Backdrop

After

Views DB config shows on Drupal 8/7 and Backdrop

ping @MikeyMJCO @jackrabbithanna

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[homotechsual]

(CiviCRM Review Template WORD-1.2)

• General standards
  • (r-explain) PASS
  • (r-user) PASS: This restores expected behaviour on D8/Backdrop.
  • (r-doc) PASS: Docs not required.
  • (r-run) PASS: Behaves correctly on D8 and Backdrop test sites.
• Developer standards
  • (r-tech) PASS
  • (r-code) PASS
  • (r-maint) PASS
  • (r-test) PENDING

[homotechsual]

One question/observation - is that syntax correct for Backdrop, that's the Drupal 6 style syntax for additional tables, my understanding is that Backdrop uses the Drupal 7 style (same as D8).

[seamuslee001]

@MikeyMJCO looks like it is correct https://api.backdropcms.org/database-configuration

[homotechsual]

@MikeyMJCO looks like it is correct https://api.backdropcms.org/database-configuration

Okay :-)

[eileenmcnaughton]

Code looks good - merging based on @MikeyMJCO's review