Skip to content
Snippets Groups Projects
Commit 32a56fef authored by catch's avatar catch
Browse files

Issue #2236983 by quietone, DenEwout: Not at all clear how/when... 

Issue #2236983 by quietone, DenEwout: Not at all clear how/when Database::addConnectionInfo should be called

(cherry picked from commit 3c075db5)
parent a30b87a3
Branches
Tags
44 merge requests!85673265330-fix-missing-hyphens: Create patch to MR and fix remaining words,!8394[warning] array_flip(): Can only flip STRING and INTEGER values, when saving a non-revisionable custom content entity,!8357Issue #2994000 by Lendude, Pasqualle, quietone, pameeela: Notice in logs when...,!7780issue 3443822: fix for 'No route found for the specified format html. Supported formats: json, xml.',!7416Simplify the HTML of field.html.twig,!7150Revert "Issue #3137119 by munish.kumar, johnwebdev, Jaypan, jungle, xjm,...,!6445Issue #3034692: Renamed the getHandler function which return the configuration of a handler instance on given display,!5013Issue #3071143: Table Render Array Example Is Incorrect,!4848Issue #1566662: Update module should send notifications on Thursdays,!4792Issue #2230689: Remove redundant "Italic" style,!4782Issue #2662898: "Links" field not displaying on custom view modes,!4488Issue #3376281: Random machine names no longer need to be wrapped in strtolower(),!4220Issue #3368223: Link field > Access to internal links is not checked on display.,!4173Issue #2123543: Add string context and location filters to the translate interface,!3884Issue #3356842,!3870Issue #3087868,!3812Draft: Issue #3339373 by alexpott, andypost, mondrake:...,!3736Issue #3294005: Refactor Claro's form--password-confirm stylesheet,!3686Issue #3219967 against 9.5.x,!3683Issue #2939397: Clearing AliasManager cache with root path raises warning,!3543Issue #3344259: Allow ajax dialog to have focus configurable,!3437Issue #3106205: Length of menu_tree.url and menu_tree.route_param_key are too short (255 characters),!3356Issue #3209129: Scrolling problems when adding a block via layout builder,!3000Issue #793660: Check for failure of hook_install,!2982Issue #3301562: Translate the default settings for this plugin (TimestampAgoFormatter),!2940Issue #3320240: Entity count query returns a string instead of int,!2921Issue #1383696: Allow a custom HTML element to be selected for a grouping field,!2920Issue #3260175: Saving media entity without an owner crashes,!2857Issue #3314541: Remove unnecessary fill from SVG icon for the "Media Library" CKEditor 5 button — enabling dark mode support in contrib,!2841Resolve #3296811 "Resourceresponsetrait needs a",!2733Issue #3293855: Update the outdated user_help text for user.admin_permissions and the description of the select box on the role settings page,!2447Issue #3293135: shouldUpdateThumbnail does not update thumbnail is source field changed,!2280Issue #3280415: Metapackage Generator Breaks Under Composer --no-dev,!2205Quote all names in the regions section.,!2050Issue #3272969: Remove UnqiueField constraint.,!1956Issue #3268872: hook_views_invalidate_cache not called when a view is deleted,!1627Issue #3082958: Add gitignore(s) to composer-ready project templates,!1459Issue #3087632: menu_name max length is too long,!1014Issue #3226806: Move filter implementations from filter.module to plugin classes,!939Issue #2971209: Allow the MediaLibraryUiBuilder service to use an alternative view display,!878Issue #3221534: throw an exception when IDs passed to loadMultiple() are badly formed,!866Issue #2845319: The highlighting of the 'Home' menu-link does not respect query strings and fragment identifiers,!204Issue #3040556: It is not possible to react to an entity being duplicated,!88Issue #3163299: Ajax exposed filters not working for multiple instances of the same Views block placed on one page
Hide whitespace changes
Inline Side-by-side
core/lib/Drupal/Core/Database/Database.php
+ 5
− 1
View file @ 32a56fef
...@@ -272,7 +272,9 @@ final public static function parseConnectionInfo(array $info) { ...@@ -272,7 +272,9 @@ final public static function parseConnectionInfo(array $info) {
* Under normal circumstances the preferred way to specify database * Under normal circumstances the preferred way to specify database
* credentials is via settings.php. However, this method allows them to be * credentials is via settings.php. However, this method allows them to be
* added at arbitrary times, such as during unit tests, when connecting to * added at arbitrary times, such as during unit tests, when connecting to
* admin-defined third party databases, etc. * admin-defined third party databases, etc. Use
* \Drupal\Core\Database\Database::setActiveConnection to select the
* connection to use.
* *
* If the given key/target pair already exists, this method will be ignored. * If the given key/target pair already exists, this method will be ignored.
* *
...@@ -284,6 +286,8 @@ final public static function parseConnectionInfo(array $info) { ...@@ -284,6 +286,8 @@ final public static function parseConnectionInfo(array $info) {
* The database connection information, as defined in settings.php. The * The database connection information, as defined in settings.php. The
* structure of this array depends on the database driver it is connecting * structure of this array depends on the database driver it is connecting
* to. * to.
*
* @see \Drupal\Core\Database\Database::setActiveConnection
*/ */
final public static function addConnectionInfo($key, $target, array $info) { final public static function addConnectionInfo($key, $target, array $info) {
if (empty(self::$databaseInfo[$key][$target])) { if (empty(self::$databaseInfo[$key][$target])) {
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment