Skip to content
This repository was archived by the owner on Aug 24, 2021. It is now read-only.

Conversation

@rogeriopradoj
Copy link

This PR proves the existence of bug #34834 (array_merge_recursive() merges arrays with objects with arrays)

https://bugs.php.net/bug.php?id=34834

User Group: PHPDF
PHPSP

Hangout presented by #PHPTestFestBrasil on 2017-11-25
https://www.youtube.com/channel/UCZDace9Yohbdbncpj9Wf_mQ

This PR proves the existence of bug #34834 (array_merge_recursive() merges arrays with objects with
arrays)
https://bugs.php.net/bug.php?id=34834

Rogerio Prado de Jesus <rogeriopradoj@gmail.com>
Ronaldo Ferreira de Oliveira <rfdeoliveira.pmsp@gmail.com>
User Group: PHPDF
            PHPSP

Hangout presented by #PHPTestFestBrasil on 2017-11-25
https://www.youtube.com/channel/UCZDace9Yohbdbncpj9Wf_mQ
$changeset = array_merge_recursive($old, $new);

var_dump($changeset);
--XFAIL--
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We won't accept XFAIL tests 😕

@petk
Copy link
Contributor

petk commented Feb 11, 2019

Hello, thank you for the pull request. The XFAIL tests for the PHP-7.4 and master branches should instead go one step further and try to fix the bug if possible. The XFAIL tests are not being merged to the php-src repository anymore. Thanks and sorry for inconvenience.

@petk petk closed this Feb 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants