Skip to content

Array access on int will fail on php7.4 - #18926

Merged
nickvergessen merged 2 commits into
masterfrom
fix/18689/int-array-access
Feb 6, 2020
Merged

Array access on int will fail on php7.4#18926
nickvergessen merged 2 commits into
masterfrom
fix/18689/int-array-access

Conversation

@kesselb

@kesselbkesselb commented Jan 16, 2020

Copy link
Copy Markdown
Contributor

Fix#18689

Probably possible to move writeAttribute into the else block of is_numeric. No idea how to trigger this kind of failure in the wild hence those weird test ;)

@kesselb

Copy link
Copy Markdown
ContributorAuthor

/backport to stable18

Comment threadlib/private/AppFramework/OCS/BaseResponse.php Outdated
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
@kesselb
kesselbforce-pushed the fix/18689/int-array-access branch from fc09249 to 8f32092CompareJanuary 23, 2020 09:24
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
@kesselb
kesselbforce-pushed the fix/18689/int-array-access branch from 8f32092 to 96d1640CompareJanuary 23, 2020 09:26
Comment threadlib/private/AppFramework/OCS/BaseResponse.php
@nickvergessen
nickvergessen merged commit 3a0bfbd into masterFeb 6, 2020
@nickvergessen
nickvergessen deleted the fix/18689/int-array-access branch February 6, 2020 20:31
@backportbot-nextcloud

Copy link
Copy Markdown

backport to stable18 in #19332

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

NC18rc1 - php7.4 report

4 participants

@kesselb@nickvergessen@ChristophWurst@juliusknorr