Problem/Motivation

When #3264195: Move non-migration tests to Forum in preparation for deprecation is committed there will be only one use of \Drupal\Tests\dblog\Functional\DbLogTest::getContent and it can be made an inline function. At the same time \Drupal\Tests\dblog\Functional\DbLogTest::getContentUpdate can be made inline.

Steps to reproduce

Proposed resolution

TBA

Remaining tasks

Patch
Review
Commit

User interface changes

N/A

API changes

N/A

Data model changes

N/A

Release notes snippet

N/A

Issue fork drupal-3339780

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

quietone created an issue. See original summary.

Version: 10.1.x-dev » 11.x-dev

Drupal core is moving towards using a “main” branch. As an interim step, a new 11.x branch has been opened, as Drupal.org infrastructure cannot currently fully support a branch named main. New developments and disruptive changes should now be targeted for the 11.x branch, which currently accepts only minor-version allowed changes. For more information, see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

dagmar’s picture

dagmar
Primary language Spanish
Location Argentina
commented
Status: Postponed » Active

Nikolas Haliotis made their first commit to this issue’s fork.

Nikolas Haliotis opened merge request !4435

nikolas haliotis’s picture

nikolas haliotis commented
Status: Active » Needs review
nikolas haliotis’s picture

nikolas haliotis commented

I don't think the test fails are relevant to my changes.

dagmar’s picture

dagmar
Primary language Spanish
Location Argentina
commented
Status: Needs review » Reviewed & tested by the community

Thanks! @Nikolas Haliotis

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Version: 11.x-dev » 10.1.x-dev
Status: Reviewed & tested by the community » Fixed

Backported to 10.1.x as a test only change.

Committed and pushed e646c8e5d9 to 11.x and 5fd0284cee to 10.1.x. Thanks!

  • longwave committed 5fd0284c on 10.1.x 
    Issue #3339780 by Nikolas Haliotis, quietone: Move getContent and...

  • longwave committed e646c8e5 on 11.x 
    Issue #3339780 by Nikolas Haliotis, quietone: Move getContent and...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.