Multiple PHP 8 warnings: Argument #1 ($result) must be passed by reference

Created on 15 December 2021, almost 3 years ago
Updated 6 February 2023, almost 2 years ago

Problem/Motivation

After migrating from PHP 7.4 to PHP 8.0 we've started seeing these errors:

Warning: Drupal\agreement\AgreementHandler::Drupal\agreement\{closure}(): Argument #1 ($result) must be passed by reference, value given in Drupal\agreement\AgreementHandler->getAgreementByUserAndPath() (line 154 of modules/contrib/agreement/src/AgreementHandler.php).

Steps to reproduce

Install the module with PHP 8.0 - the warnings will appear straight away.

Proposed resolution

Get rid of unneeded passing by reference in all cases.

Remaining tasks

None.

User interface changes

None.

API changes

None.

Data model changes

None.

๐Ÿ› Bug report
Status

Fixed

Version

3.0

Component

Code

Created by

๐Ÿ‡ง๐Ÿ‡พBelarus spleshka UAE

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

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024