fix #9: allow optional (null) menu options

Author: sodik82

__tests__/MenuOptions-test.js

@@ -27,6 +27,21 @@ describe('MenuOptions', () => {
     });
   });
 
+  it('should accept optional (null) options', () => {
+    const option = false;
+    const { output } = render(
+      <MenuOptions>
+        <MenuOption />
+        {option ? <MenuOption />: null}
+        <MenuOption />
+      </MenuOptions>
+    );
+    expect(output.type).toEqual(View);
+    const children = output.props.children;
+    expect(children.length).toEqual(2);
+  });
+
+
   it("should prioritize option's onSelect handler", () => {
     const onSelect = () => 0;
     const onSelectOption = () => 1;

src/MenuOptions.js

@@ -4,10 +4,13 @@ import { View } from 'react-native';
 const MenuOptions = ({ style, children, onSelect, customStyles }) => (
   <View style={[customStyles.optionsWrapper, style]}>
     {
-      React.Children.map(children, c => React.cloneElement(c, {
-        onSelect: c.props.onSelect || onSelect,
-        customStyles: Object.keys(c.props.customStyles || {}).length ? c.props.customStyles : customStyles
-      }))
+      React.Children.map(children, c =>
+        React.isValidElement(c) ?
+          React.cloneElement(c, {
+            onSelect: c.props.onSelect || onSelect,
+            customStyles: Object.keys(c.props.customStyles || {}).length ? c.props.customStyles : customStyles
+          }) : c
+      )
     }
   </View>
 );