Opened 2 years ago

Closed 5 months ago

#16767 closed enhancement (fixed)

PHP5-port - De-Reference as needed.

Reported by: hakre Owned by:
Priority: normal Milestone: 3.2
Component: General Version: 3.1
Severity: normal Keywords: has-patch
Cc:

Description

In PHP5 objects and references to objects aren't what those were in PHP4.

Attachments (3)

16767.patch​ (39.9 KB) - added by hakre 2 years ago.
handpicked (where applicable) &new -> new, &$this -> $this
16767.2.patch​ (43.1 KB) - added by hakre 2 years ago.
Patch against trunk, merged with latest 16661 patch
16767.3.patch​ (63.1 KB) - added by hakre 2 years ago.
Most callbacks and functions definitions (PHP uses copy on write, no need to optimize what needs no optimisation)

Download all attachments as: .zip

Change History (12)

hakre — 2 years ago

handpicked (where applicable) &new -> new, &$this -> $this

comment:1   scribu — 2 years ago

  • Owner set to scribu
  • Status changed from new to reviewing

It would be good to coordonate with #16661

Version 0, edited 2 years ago by scribu (next)

comment:2   scribu — 2 years ago

  • Owner scribu deleted

comment:3   nacin — 2 years ago

  • Type changed from defect (bug) to enhancement

comment:4   hakre — 2 years ago

Reviewed #16661 for the first time.

hakre — 2 years ago

Patch against trunk, merged with latest 16661 patch

hakre — 2 years ago

Most callbacks and functions definitions (PHP uses copy on write, no need to optimize what needs no optimisation)

comment:5   hakre — 2 years ago

Reference: #13319

comment:6   ryan — 2 years ago

(In [17573]) Assigning the return value of new by reference is deprecated. Props hakre. see #16767

comment:7   hakre — 2 years ago

Related: #17443

comment:8   ocean90 — 22 months ago

Related: #18237

comment:9   nacin — 5 months ago

  • Milestone changed from Awaiting Review to 3.2
  • Resolution set to fixed
  • Status changed from reviewing to closed
Note: See TracTickets for help on using tickets.