drupal-check report results

------ ----------------------------------------------------------------------- 
  Line   src/EventSubscriber/SecKitEventSubscriber.php           
 ------ ----------------------------------------------------------------------- 
  71     Parameter $event of method                                             
         Drupal\seckit\EventSubscriber\SecKitEventSubscriber::onKernelRequest(  
         ) has typehint with deprecated class                                   
         since Symfony 4.3, use RequestEvent instead                            
  86     Parameter $event of method                                             
         Drupal\seckit\EventSubscriber\SecKitEventSubscriber::onKernelResponse  
         () has typehint with deprecated class                                  
         Symfony\Component\HttpKernel\Event\FilterResponseEvent:                
         since Symfony 4.3, use ResponseEvent instead                           
 ------ ----------------------------------------------------------------------- 
 ------ ------------------------------------------------------------ 
  Line   tests/src/Functional/SecKitTestCaseTest.php  
 ------ ------------------------------------------------------------ 
  75     Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  103    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  132    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  161    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  190    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  253    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  268    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  323    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  355    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  364    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  373    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  382    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  391    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  393    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  407    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  409    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  423    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  425    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  434    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  436    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  437    Call to deprecated method assertEqual() of class            
         Drupal\Tests\BrowserTestBase:                               
         in drupal:8.0.0 and is removed from drupal:10.0.0. Use      
  446    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  455    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  464    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  474    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  484    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  512    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  525    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  535    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  547    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  556    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  568    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  577    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  591    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  601    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
  613    Call to deprecated method drupalPostForm() of class         
         Drupal\Tests\BrowserTestBase:                               
         in drupal:9.1.0 and is removed from drupal:10.0.0. Use      
 ------ ------------------------------------------------------------ 

Comments

suzymasri created an issue. See original summary.

suzymasri’s picture

Status: Needs work » Needs review
StatusFileSize
new17.9 KB

Patch attached.

Status: Needs review » Needs work

The last submitted patch, 2: 3190816-2.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

suzymasri’s picture

StatusFileSize
new19.03 KB
new2.39 KB

Updated the patch to cover the below errors:

 ------ ----------------------------------------------------------------------- 
  Line   src/EventSubscriber/SecKitEventSubscriber.php           
 ------ ----------------------------------------------------------------------- 
  71     Parameter $event of method                                             
         Drupal\seckit\EventSubscriber\SecKitEventSubscriber::onKernelRequest(  
         ) has typehint with deprecated class                                   
         since Symfony 4.3, use RequestEvent instead                            
 ------ ----------------------------------------------------------------------- 
 ------ ------------------------------------------------------------ 
  Line   tests/src/Functional/SecKitTestCaseTest.php  
 ------ ------------------------------------------------------------ 
  437    Call to deprecated method assertEqual() of class            
         Drupal\Tests\BrowserTestBase:                               
         in drupal:8.0.0 and is removed from drupal:10.0.0. Use      
 ------ ------------------------------------------------------------ 
Pooja Ganjage’s picture

StatusFileSize
new21.13 KB

Hi,

Creating a patch for raised issue in #2 comment patch and as well follow to solve deprecated code as per the below link mentioned
https://www.drupal.org/node/3168858

Please review the patch.

Thanks.

suzymasri’s picture

StatusFileSize
new19.14 KB
new709 bytes

Thanks @Pooja Ganjage, there aren't any changes between patch #4 and #5, interdiff was empty.

Fixed a missing class issue, patch and interdiff attached.

Pooja Ganjage’s picture

StatusFileSize
new22.18 KB
Pooja Ganjage’s picture

Status: Needs work » Needs review

The last submitted patch, 6: 3190816-6.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

suzymasri’s picture

StatusFileSize
new22.21 KB
new615 bytes

@Pooja Ganjage a minor text change was overridden by your patch #7, it would make it a lot easier for everyone to provide interdiff so the changes are clear, thanks!

Patch updated with text fixes.

timmillwood’s picture

Status: Needs review » Reviewed & tested by the community

Looks like the failing test above is on 9.1.0 due to #3186443: PHPUnit 9.5 Call to undefined method ::getAnnotations() which has been committed to 9.1.x and will be released in 9.1.1. The tests are passing fine in 9.1.x and 9.0.10.

Therefore, looks good to me, great to get these deprecations fixed early.

damienmckenna’s picture

Assigned: suzymasri » Unassigned
mcdruid’s picture

Issue tags: +Needs reroll

There are a handful of these wide-ranging patches in the queue which all unfortunately require a reroll after some other fixes were committed.

mcdruid’s picture

Status: Reviewed & tested by the community » Needs work
sourabhjain’s picture

Assigned: Unassigned » sourabhjain
sourabhjain’s picture

Status: Needs work » Needs review
StatusFileSize
new19.07 KB

Created a latest patch. Please review.

silvi.addweb’s picture

Status: Needs review » Reviewed & tested by the community

I've tested the #16 patch and there are no problems found.

the_g_bomb’s picture

Status: Reviewed & tested by the community » Closed (outdated)
Issue tags: -Needs reroll +Rotterdam2026

I suspect all deprecations listed here were fixed in #3289537: Automated Drupal 10 compatibility fixes. Patch #16 no longer applies. Marking "Closed (outdated)".

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.