alternative
1 unresolved thread
1 unresolved thread
Closes #2347867
Merge request reports
Activity
Filter activity
1 <?php 2 3 declare(strict_types=1); 4 5 namespace Drupal\Core\Database; 6 7 /** 8 * Interface for database connections. 9 */ 10 interface DatabaseConnectionInterface { I like the idea here and agree in principle that connections should be typehinted with an interface generally... the problem, though, is that the interface only declares a public API of
open
, which is rather incomplete. It seems that the path forward for moving to an interface would be adding all the public methods fromConnection
to this interface, adding a deprecation, etc. etc. - In the spirit of keeping this change narrow-ish, this is better as a follow-up?
added 5 commits
-
8d6761a4...717b02a8 - 2 commits from branch
project:11.x
- 27645673 - Merge branch '11.x' into 2347867-mkII
- a9d9047e - make interface empty
- d232d097 - a simpler way to prevent non-transactional connections being opened
Toggle commit list-
8d6761a4...717b02a8 - 2 commits from branch
Please register or sign in to reply