php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #76921 ArrayAccess only supports increment on by-ref offsetGet()
Submitted: 2018-09-23 14:11 UTC Modified: 2020-11-17 12:53 UTC
From: nikic@php.net Assigned:
Status: Open Package: Scripting Engine problem
PHP Version: Irrelevant OS:
Private report: No CVE-ID: None
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If this is not your bug, you can add a comment by following this link.
If this is your bug, but you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: nikic@php.net
New email:
PHP Version: OS:

 

 [2018-09-23 14:11 UTC] nikic@php.net
Description:
------------
Overloaded properties implement $x->y++ by calling __set('y', __get('y') + 1). 

The same does not work for ArrayAccess. ArrayAccess only supports $x[0]++ if offsetGet() returns by reference, in which case the increment is performed on the reference.

This is inconsistent both with the overloaded property behavior and with compound assignment operations on ArrayAccess. That is, $x[0] += 1 *does* work with a by-value offsetGet().

Test script:
---------------
<?php

class Test implements ArrayAccess {
    public function __get($k) {
        return 1;
    }
    public function __set($k, $v) {
        echo "__set($v)\n";
    }
    
    public function offsetGet($k) {
        return 1;
    }
    public function offsetSet($k, $v) {
        echo "offsetSet($v)\n";
    }
    public function offsetExists($k) {}
    public function offsetUnset($k) {}
}

$t = new Test;
echo "Property:\n";
$t->x++;
echo "Offset:\n";
$t[0]++;

Expected result:
----------------
Property:
__set(2)
Offset:
offsetSet(2)

Actual result:
--------------
Property:
__set(2)
Offset:
Notice: Indirect modification of overloaded element of Test has no effect

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2020-11-17 12:53 UTC] cmb@php.net
For reference: <https://3v4l.org/EFpo1>.
 
PHP Copyright © 2001-2021 The PHP Group
All rights reserved.
Last updated: Tue Jan 26 09:01:23 2021 UTC