Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
+++ b/core/modules/simpletest/src/RandomGeneratorTrait.php
@@ -0,0 +1,120 @@
+trait RandomGeneratorTrait {
Yay! Can we get a followup to use this in BrowserTestBase please?
There we go
Proposed resolution
In \Drupal\simpletest\BrowserTestBase
use RandomGeneratorTrait;
Remaining tasks
None.
User interface changes
None.
API changes
None.
Beta phase evaluation
Issue category | Task because it's a cleanup that removes a code duplication. |
---|---|
Issue priority | Not critical because it doesn't affect in the way Drupal works. |
Disruption | No disruption. |
Comment | File | Size | Author |
---|---|---|---|
#1 | 2499199-1.patch | 3.47 KB | claudiu.cristea |
Comments
Comment #1
claudiu.cristeaIt should be a patch with only one green line, right? :)
Comment #2
dawehnerYeah this is exactly what is supposed to be needed.
Nice!
If you want you should do more research what is missing in BrowserTestBase in comparison with WebTestBase and see whether we can continuously port them over.
Comment #3
claudiu.cristeaBeta evaluation
Comment #4
webchickImpressive diffstat. :)
Committed and pushed to 8.0.x. Thanks!