Merge pull request #269 from rvsia/allowDisableSubmitPf4

fix(pf4): allow to use disableSubmit

Author: Hyperkid123

packages/pf4-component-mapper/src/form-fields/layout-components.js

@@ -4,14 +4,15 @@ import { layoutComponents } from '@data-driven-forms/react-form-renderer';
 import { Form, Button, ActionGroup, TextContent, Text, TextVariants } from '@patternfly/react-core';
 import './layout-components-styles.scss';
 
-const ButtonLayout = ({ label, bsStyle, children, ...props }) =>
-  <Button variant={ bsStyle || 'secondary' } { ...props }>
+const ButtonLayout = ({ label, bsStyle, children, disabled, ...props }) =>
+  <Button variant={ bsStyle || 'secondary' } isDisabled={ disabled } { ...props }>
     { label }{ children }
   </Button>;
 
 ButtonLayout.propTypes = {
   label: PropTypes.string.isRequired,
   bsStyle: PropTypes.string,
+  disabled: PropTypes.bool,
   children: PropTypes.oneOfType([
     PropTypes.arrayOf(PropTypes.node),
     PropTypes.node,

packages/pf4-component-mapper/src/tests/field-array/__snapshots__/field-array.test.js.snap

@@ -679,7 +679,7 @@ exports[`FieldArray should render array field correctly 1`] = `
                                     "Submit",
                                     undefined,
                                   ],
-                                  "disabled": undefined,
+                                  "isDisabled": undefined,
                                   "onClick": [Function],
                                   "type": "button",
                                   "variant": "primary",

packages/pf4-component-mapper/src/tests/layout-components.test.js

@@ -21,4 +21,11 @@ describe('Layout mapper', () => {
     expect(mount(layoutMapper[layoutComponents.TITLE]({})).find(TextContent)).toHaveLength(1);
     expect(mount(layoutMapper[layoutComponents.TITLE]({})).find(Text)).toHaveLength(1);
   });
+
+  it('should return PF4 Button disabled', () => {
+    const wrapper = mount(layoutMapper[layoutComponents.BUTTON]({ disabled: true }));
+
+    expect(wrapper.find(Button)).toHaveLength(1);
+    expect(wrapper.find(Button).props().isDisabled).toEqual(true);
+  });
 });