{"patch":"diff --git a\/src\/utils\/__tests__\/mapChildren-test.js b\/src\/utils\/__tests__\/mapChildren-test.js\nnew file mode 100644\nindex 0000000000..2d0bfec5a9\n--- \/dev\/null\n+++ b\/src\/utils\/__tests__\/mapChildren-test.js\n@@ -0,0 +1,117 @@\n+\/**\n+ * @emails react-core\n+ * @jsx React.DOM\n+ *\/\n+\n+\"use strict\";\n+\n+describe('mapChildren', function() {\n+\n+ var React;\n+ var ReactTestUtils;\n+\n+ var mapChildren;\n+ var reactComponentExpect;\n+\n+ var Wrap;\n+\n+ beforeEach(function() {\n+ React = require('React');\n+ ReactTestUtils = require('ReactTestUtils');\n+\n+ mapChildren = require('mapChildren');\n+ reactComponentExpect = require('reactComponentExpect');\n+\n+ Wrap = React.createClass({\n+ render: function() {\n+ return (\n+

\n+ {mapChildren(this.props.children, this.props.mapFn, this)}\n+ <\/div>\n+ );\n+ }\n+ });\n+\n+ this.addMatchers({\n+ toHaveKeys: function(expected) {\n+ if (this.actual.length != expected.length) {\n+ return false;\n+ }\n+ return this.actual.every(function(component, index) {\n+ return component._key === expected[index];\n+ }, this);\n+ }\n+ });\n+ });\n+\n+\n+ it('should support identity for simple', function() {\n+ var mapFn = jasmine.createSpy().andCallFake(function (kid, key, index) {\n+ return kid;\n+ });\n+\n+ var simpleKid = ;\n+\n+ var instance = {simpleKid}<\/Wrap>;\n+ ReactTestUtils.renderIntoDocument(instance);\n+\n+ var rendered = reactComponentExpect(instance)\n+ .expectRenderedChild()\n+ .instance();\n+\n+ expect(mapFn).toHaveBeenCalledWith(simpleKid, 'simple', 0);\n+ expect(rendered.props.children[0]).toBe(simpleKid);\n+ expect(rendered.props.children).toHaveKeys(['simple']);\n+ });\n+\n+ it('should pass key to returned component', function() {\n+ var mapFn = function (kid, key, index) {\n+ return

{kid}<\/div>;\n+ };\n+\n+ var simpleKid = ;\n+\n+ var instance = {simpleKid}<\/Wrap>;\n+ ReactTestUtils.renderIntoDocument(instance);\n+\n+ var rendered = reactComponentExpect(instance)\n+ .expectRenderedChild()\n+ .instance();\n+\n+ expect(rendered.props.children[0]).not.toBe(simpleKid);\n+ expect(rendered.props.children[0].props.children[0]).toBe(simpleKid);\n+ expect(rendered.props.children).toHaveKeys(['simple']);\n+ expect(rendered.props.children[0].props.children).toHaveKeys(['simple']);\n+ });\n+\n+ it('should be called for each child', function() {\n+ var mapFn = jasmine.createSpy().andCallFake(function (kid, key, index) {\n+ return

{kid}<\/div>;\n+ });\n+\n+ var kidOne =

;\n+ var kidTwo =

;\n+ var kidThree =

;\n+\n+ var instance = ReactTestUtils.renderIntoDocument(\n+ \n+ {kidOne}\n+ {null}\n+ {kidTwo}\n+ {null}\n+ {kidThree}\n+ <\/Wrap>\n+ );\n+\n+ var rendered = reactComponentExpect(instance)\n+ .expectRenderedChild()\n+ .instance();\n+\n+ expect(mapFn.calls.length).toBe(3);\n+ expect(mapFn).toHaveBeenCalledWith(kidOne, 'one', 0);\n+ expect(mapFn).toHaveBeenCalledWith(kidTwo, 'two', 1);\n+ expect(mapFn).toHaveBeenCalledWith(kidThree, 'three', 2);\n+ expect(rendered.props.children).not.toEqual(instance.props.children);\n+ expect(rendered.props.children).toHaveKeys(['one', 'two', 'three']);\n+ });\n+});\ndiff --git a\/src\/utils\/mapChildren.js b\/src\/utils\/mapChildren.js\nnew file mode 100644\nindex 0000000000..cdba3e4d90\n--- \/dev\/null\n+++ b\/src\/utils\/mapChildren.js\n@@ -0,0 +1,35 @@\n+\/**\n+ * @providesModule mapChildren\n+ *\/\n+\n+\"use strict\";\n+\n+\/**\n+ * Maps children that are typically specified as `props.children`.\n+ *\n+ * The provided mapFunction(child, key, index) will be called for each\n+ * leaf child.\n+ *\n+ * Note: mapChildren assumes children have already been flattened.\n+ *\n+ * @param {array} children\n+ * @param {function(*, string, int)} mapFunction\n+ * @param {*} context\n+ * @return {array} mirrored array with mapped children\n+ *\/\n+function mapChildren(children, mapFunction, context) {\n+ if (children == null) {\n+ return children;\n+ }\n+ var mappedChildren = [];\n+ for (var ii = 0; ii < children.length; ii++) {\n+ var child = children[ii];\n+ var key = child._key;\n+ var mappedChild = mapFunction.call(context, child, key, ii);\n+ mappedChild._key = key;\n+ mappedChildren.push(mappedChild);\n+ }\n+ return mappedChildren;\n+}\n+\n+module.exports = mapChildren;\n","files":[{"status":"A","path":"src\/utils\/__tests__\/mapChildren-test.js"},{"status":"A","path":"src\/utils\/mapChildren.js"}],"body":"mapChilden() is similar to Array.map() and objMap() but handles deep\nnested structures and follows similar rules to flattenChildren()","parents":["4b81de93d3733ecc5d4dcaab7efad2a5eef1937d"],"commit":{"authoredAt":"2013-06-04T12:15:25-07:00","revision":"259392035daac9cf8d45d909a175e640c496f547","authorName":"CommitSyncScript","subject":"mapChildren","shortRevision":"259392035d","authorEmail":"jeffmo@fb.com"}}

dillon1000/react

Files changed