Notice: Trying to access array offset on value of type null in bootstrap_pager() (line 95 of bootstrap/templates/system/pager.func.php).

CommentFileSizeAuthor
#9 fix-array-offset-3179792-9.patch930 bytesmaxmendez
#4 fix-array-offset-3179792-4.patch704 bytesdippers

Issue fork bootstrap-3179792

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

joelpittet created an issue. See original summary.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Category: Task » Bug report
Issue tags: +PHP 7.4

I've only seen this since moving to PHP 7.4, not sure the culprit yet.

dippers’s picture

dippers commented

See https://www.drupal.org/project/drupal/issues/3084961.

The part of the patch for theme_pager() also needs to be applied to bootstrap_pager().

dippers’s picture

dippers commented
StatusFileSize
new fix-array-offset-3179792-4.patch704 bytes

Patch attached.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Status: Active » Reviewed & tested by the community

Thanks that looks reasonable, my digging didn’t find the source week enough except fontawesome and icon had an issue that I could decipher

akhilavnair’s picture

akhilavnair
Primary language Malayalam
Location Kerala
commented

Hi, Could you please merge this patch with latest version

lucatrabx’s picture

lucatrabx commented

where i have to put it?

fenstrat’s picture

fenstrat
he/him
Primary language English
Location Australia
commented

+1 RTBC, also only saw this when updating to PHP 7.4.

maxmendez’s picture

maxmendez commented
StatusFileSize
new fix-array-offset-3179792-9.patch930 bytes

The patch 4 fixed my problem, but i've added a line to update $quantity based on theme_pager function.

Note: I got confused, sorry for the first comment.

ressa’s picture

ressa
he/him
commented
Status: Reviewed & tested by the community » Needs review

Thanks @Dippers and @MaxMendez, I am also seeing these warnings. I am changing Status to trigger testing and alert the maintainers.

ressa opened merge request !6

ressa’s picture

ressa
he/him
commented

It looks like Automated testing is not enabled for this project, so the patch in #9 doesn't get tested.

I created a patch via Drupal's Gitlab integration, which seems to get registered and checked for mergeability, but not tested.

markhalliwell made their first commit to this issue’s fork.

  • markhalliwell committed 3ed0faa on 7.x-3.x authored by ressa 
    Issue #3179792 by ressa, Dippers, MaxMendez: Notice: Trying to access...
markhalliwell’s picture

markhalliwell
Primary language English
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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