getLogMessages() has typehint with deprecated

Created on 6 March 2024, 10 months ago
Updated 26 August 2024, 5 months ago

Return type of method
Drupal\Tests\yoast_seo\Context\Log::getLogMessages() has
typehint with deprecated class
Drupal\Core\Database\StatementWrapper. Deprecated in
drupal:10.1.0 and is removed from drupal:11.0.0. Use
Drupal\Core\Database\StatementWrapperIterator instead.

🐛 Bug report
Status

Postponed

Version

2.0

Component

Code

Created by

🇮🇳India himanshu_jhaloya Indore

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @himanshu_jhaloya
  • First commit to issue fork.
  • Pipeline finished with Failed
    9 months ago
    Total: 2877s
    #151398
  • Pipeline finished with Failed
    9 months ago
    Total: 191s
    #151457
  • Pipeline finished with Failed
    9 months ago
    #151467
  • Open on Drupal.org →
    Core: 9.5.x + Environment: PHP 7.4 & MySQL 5.7
    last update 9 months ago
    Waiting for branch to pass
  • 🇺🇸United States melnychukmy

    I updated deprecated class usage. Not sure how to confirm that it works. I didn't run Behat test locally but can give it a try. Now I'm waiting for CI result. Unfortunately I noticed that CI fails on latest 8.x-2.x branch.

  • Status changed to Postponed 5 months ago
  • 🇳🇱Netherlands kingdutch

    I've skipped this fix for now because it doesn't touch production code for the module and the tests are indeed not running in GitLab. Getting Behat to work there was unfortunately not easy. We can make this change in a way that we can test both Drupal 10 until EOL and Drupal 11 once the tests are fixed :)

    Thanks for flagging it!

Production build 0.71.5 2024