Ticket #43619: 43619.3.diff

src/wp-includes/compat.php

@@ -505 +505 @@
 if ( ! function_exists( 'spl_autoload_register' ) ) {
         require_once ABSPATH . WPINC . '/spl-autoload-compat.php';
 }
+
+if ( ! function_exists( 'is_iterable' ) ) {
+        /**
+         * Polyfill for is_iterable function added in PHP 7.1.
+         *
+         * Verify that the content of a variable is a non-empty array or an object
+         * implementing the Traversable interface.
+         *
+         * @since 4.9.6
+         *
+         * @param mixed $var Variable to check.
+         *
+         * @return bool Whether the provided variable is iterable.
+         */
+        function is_iterable( $var ) {
+                return ( is_array( $var ) || $var instanceof Traversable );
+        }
+}

tests/phpunit/tests/compat.php

@@ -186 +186 @@
                 $this->assertEquals( '["foo"]', $json->encodeUnsafe( array( 'foo' ) ) );
                 $this->assertEquals( array( 'foo' ), $json->decode( '["foo"]' ) );
         }
+
+        /**
+         * @ticket 43619
+         * @dataProvider iterable_variable_test_data
+         * @param mixed $variable
+         * @param bool $expected
+         */
+        function test_is_iterable( $variable, $expected ) {
+                $this->assertEquals( is_iterable( $variable ), $expected );
+        }
+
+        /**
+         * @ticket 43619
+         */
+        function iterable_variable_test_data() {
+                return array(
+                        array( array(), true ),
+                        array( array( 1, 2, 3 ), true ),
+                        array( new \ArrayIterator( array( 1, 2, 3 ) ), true ),
+                        array( 1, false ),
+                        array( 3.14, false ),
+                        array( new stdClass(), false ),
+                );
+        }
 }
 
 /* used in test_mb_substr_phpcore */