Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
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'])
Comment | File | Size | Author |
---|---|---|---|
#4 | amber-amber_not_opening_database_prefix-2735053-4.patch | 469 bytes | skbly7 |
Comments
Comment #2
skbly7 CreditAttribution: skbly7 commentedPull request sent to GitHub at https://github.com/berkmancenter/amber_drupal/pull/15
Comment #3
jlicht CreditAttribution: jlicht commentedThanks 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.
Comment #4
skbly7 CreditAttribution: skbly7 as a volunteer commentedPatch file for the issue.
Comment #6
jlicht CreditAttribution: jlicht commented