Prepare a module for Drupal 9.

CommentFileSizeAuthor
#2 drupal_9_views_ajax_history_3106597.patch1.18 KBAstonVictor
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Aston Victor created an issue. See original summary.

AstonVictor’s picture

AstonVictor CreditAttribution: AstonVictor at DevBranch commented
Status: Active » Needs review
FileSize
drupal_9_views_ajax_history_3106597.patch1.18 KB
John Cook’s picture

John Cook CreditAttribution: John Cook at Creode commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +ContributionWeekend2020

The patch looks good and applies cleanly.

Before patch:

./vendor/bin/drupal-check modules/contrib/views
_ajax_history/
 1/1 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

 ------ ------------------------------------------------ 
  Line   views_ajax_history.module                       
 ------ ------------------------------------------------ 
  17     Call to deprecated function pager_find_page().  
 ------ ------------------------------------------------ 

                                                                                             
 [ERROR] Found 1 error

After patch:

./vendor/bin/drupal-check modules/contrib/views_ajax_history/
 1/1 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

                                                                                             
 [OK] No errors

Marking as RTBC

amateescu’s picture

amateescu CreditAttribution: amateescu as a volunteer commented
Status: Reviewed & tested by the community » Fixed

Committed to 8.x-1.x, thanks!

Status: Fixed » Closed (fixed)

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