Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Problem/Motivation
Found code arrays that use the += operator to merge arrays - php does not allow this.
Proposed resolution
rewrite this to use array_merge. $x + = $y becomes $x = array_merge( $x, $y );
Remaining tasks
reviews needed, regression testing.
User interface changes
N/A
API changes
N/A
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | 1243496.patch | 2.01 KB | Kevin Baluha |
Comments
Comment #1
Kevin Baluha commentedComment #2
Kevin Baluha commentedComment #3
arcaneadam commentedA couple things.
1. PHP does use the '+' as an array union operator. (http://php.net/manual/en/language.operators.array.php) So if this is causing your site to have 500 errors then there is probably an underlying issue elsewhere (possibly).
2. The 7.x version is still very much a development version at this point. Therefore it's going to be buggy and not ready for prime time.
3. Please read the Issue queue handbook(http://drupal.org/node/317) on how the issue queues work. I'm not a a jerk about it in my issue queue but I can guarantee that as you get more involved some other project maintainers will gripe at you if you are not filing issues correctly, or worse just flat out ignore the issue you are having because you didn't file it correctly.
I appreciate your desire to help with the project and as I said the 7.x version is not ready for production yet, so take it as it is, but I am working on it.