Pocketmine-mp: Remove remaining usages of {} for string offset access

Created on 15 Jul 2019  路  8Comments  路  Source: pmmp/PocketMine-MP

It's looking likely that {} will be deprecated in PHP 7.4, and since it's used throughout the PM core we have some work to do to avoid a mess when that happens.

https://wiki.php.net/rfc/deprecate_curly_braces_array_access

Core Fixed
Source
picture dktapps
🎉1 👍1

All 8 comments

Also note the presence of a migration script along with the RFC.

SOF3 picture SOF3 on 16 Jul 2019
👍1

https://gist.github.com/theodorejb/763b83a43522b0fc1755a537663b1863
A migration script from the curly braces, don't know if it'll help just thought I'll post it here for you.

DenielWorld picture DenielWorld on 18 Jul 2019

no idea why github didn't auto-close this ...

dktapps picture dktapps on 27 Jul 2019

FYI: GitHub doesn't close issues until commits with auto-close messages are merged into the master brunch

fuyutsuki picture fuyutsuki on 31 Jul 2019

False, they have to be merged into the default branch, which is currently stable.

dktapps picture dktapps on 31 Jul 2019
😕1

Umm, please refer this page, that's about PR.

If the commit is in a non-default branch, the issue will remain open and the issue will be referenced with a tooltip.

fuyutsuki picture fuyutsuki on 31 Jul 2019

that's exactly what i said ...

dktapps picture dktapps on 31 Jul 2019
👀1

Lol my extremely fault

fuyutsuki picture fuyutsuki on 31 Jul 2019
Was this page helpful?
0 / 5 - 0 ratings

Related issues

arfianadam picture arfianadam  路  3Comments
jasonwynn10 picture jasonwynn10  路  3Comments
MisteFr picture MisteFr  路  3Comments
Ox531 picture Ox531  路  3Comments
Hugheth picture Hugheth  路  3Comments