Description

  • Support of prefixed Drupal installation missing
  • AmberStatus class has prefix support but prefix not being passed on it's construction

Code Reference
File amber.module line number 179 new AmberStatus(new AmberPDO($db))

Fix
Need to change with new AmberStatus(new AmberPDO($db), $db->getConnectionOptions()['prefix']['default'])

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

skbly7 created an issue. See original summary.

skbly7’s picture

Status: Active » Needs review
jlicht’s picture

Thanks for this report! We're managing code submissions through Drupal.org, so could you submit the code changes as a patch?

See https://www.drupal.org/patch/submit and https://www.drupal.org/node/707484 for details on the process.

skbly7’s picture

Patch file for the issue.

  • jlicht committed 84191c8 on 7.x-1.x authored by skbly7
    Issue #2735053 by skbly7: Amber dashboard not opening when database...
jlicht’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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