Changeset 11082 for trunk/tests/phpunit/testcases/core/functions.php

Timestamp:

09/10/2016 05:14:34 PM (8 years ago)

Author:

dcavins

Message:

bp_sort_by_key(): Add $preserve_keys parameter.

Add a parameter to bp_sort_by_key() so that it calls either usort()
or uasort(), depending on the whether the array keys are important or
not.

Fixes #7238.

File:

• trunk/tests/phpunit/testcases/core/functions.php (modified) (1 diff)

trunk/tests/phpunit/testcases/core/functions.php

@@ -427 +427 @@
 
         $this->assertEquals( $expected, bp_alpha_sort_by_key( $items, 'name' ) );
+    }
+
+    /**
+     * @group bp_sort_by_key
+     */
+    public function test_bp_sort_by_key_arrays_num_preserve_keys() {
+        $items = array(
+            'p' => array(
+                'foo' => 'bar',
+                'value' => 5,
+            ),
+            'q' => array(
+                'foo' => 'bar',
+                'value' => 10,
+            ),
+            'r' => array(
+                'foo' => 'bar',
+                'value' => 1,
+            ),
+        );
+
+        $expected = array(
+            'r' => array(
+                'foo' => 'bar',
+                'value' => 1,
+            ),
+            'p' => array(
+                'foo' => 'bar',
+                'value' => 5,
+            ),
+            'q' => array(
+                'foo' => 'bar',
+                'value' => 10,
+            ),
+        );
+
+        $this->assertEquals( $expected, bp_sort_by_key( $items, 'value', 'num', true ) );
+    }
+
+    /**
+     * @group bp_sort_by_key
+     */
+    public function test_bp_sort_by_key_num_should_respect_0_preserve_keys() {
+        $items = array(
+            's' => array(
+                'foo' => 'bar',
+                'value' => 2,
+            ),
+            't' => array(
+                'foo' => 'bar',
+                'value' => 0,
+            ),
+            'u' => array(
+                'foo' => 'bar',
+                'value' => 4,
+            ),
+        );
+
+        $expected = array(
+            't' => array(
+                'foo' => 'bar',
+                'value' => 0,
+            ),
+            's' => array(
+                'foo' => 'bar',
+                'value' => 2,
+            ),
+            'u' => array(
+                'foo' => 'bar',
+                'value' => 4,
+            ),
+        );
+
+        $this->assertEquals( $expected, bp_sort_by_key( $items, 'value', 'num', true ) );
     }