Problem/Motivation

Fixing depreciations for PHP 8.2 line 119 of handlers/views_handler_area_result.inc

Proposed resolution

Fix code to new standards

Remaining tasks

Create Patch
Tests if needed, guess existing ones cover

CommentFileSizeAuthor
#3 views-variablevariables-3458269-3.patch518 bytesdarrenwh

Issue fork views-3458269

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

darrenwh created an issue. See original summary.

darrenwh’s picture

darrenwh commented
Issue summary: View changes
darrenwh’s picture

darrenwh commented
StatusFileSize
new views-variablevariables-3458269-3.patch518 bytes
darrenwh’s picture

darrenwh commented
Assigned: darrenwh » Unassigned
Status: Active » Needs review

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

rishabjasrotia opened merge request !16

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Issue tags: +PHP 8.2

Does this break on older versions of PHP?

darrenwh’s picture

darrenwh commented

@DamienMcKenna a minimum PHP version could be added to the info file? Will need to go back and see when it changed as pretty sure the syntax has been around for a while.

darrenwh’s picture

darrenwh commented

This should not break in older versions

nagy.balint’s picture

nagy.balint commented

Not the exact same code part, but this is related I think #3453503: Previous commit broke views_handler_area_result

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Reviewed & tested by the community
Parent issue: » #3347650: Plan for Views 7.x-3.30

  • damienmckenna committed b472f2ea on 7.x-3.x 
    Issue #3458269 by rishabjasrotia, darrenwh, damienmckenna: Variable...
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Title: Variable Variables for php 8.2 » Variable variables for PHP 8.2
Status: Reviewed & tested by the community » Fixed

Committed. Thank you.

Status: Fixed » Closed (fixed)

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