Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
in:
/includes/database/mysql/database.inc
on line 18, line is
$this->transactionSupport = !empty($connection_option['transactions']);
and should be
$this->transactionSupport = !empty($connection_options['transactions']);
($connection_option -> $connection_options).
this preveents transactions from being enabled since it always returns false
Comment | File | Size | Author |
---|---|---|---|
#3 | drupal-7-dev-373514-1.patch | 595 bytes | jbomb |
Comments
Comment #1
Crell CreditAttribution: Crell commentedHm, right you are. Can you roll a quick patch?
Comment #2
karschsp CreditAttribution: karschsp commentedTagging for novice queue.
Comment #3
jbomb CreditAttribution: jbomb commentedrequested patch from #1 with jaydee18's fix
Comment #4
jbomb CreditAttribution: jbomb commentedComment #5
Crell CreditAttribution: Crell commentedLooks good to me. Thanks!
Comment #6
Dries CreditAttribution: Dries commentedIt looks good to me, but I wonder why the tests work without this patch ... should we look at fixing our tests?
Comment #7
Crell CreditAttribution: Crell commentedThe test bot only runs MyISAM right now, so it's not actually testing transaction logic. Known issue, has not yet been solved, but not something we can solve in this patch.
Comment #8
webchickCommitted to HEAD. Thanks!