issue 2132 fix

Author: rashmi73

src/shared/components/Settings/Tools/Devices/index.jsx

@@ -280,6 +280,17 @@ export default class Devices extends ConsentComponent {
     this.setState({ isMobileView: window.innerWidth <= screenSM });
   }
 
+  isFormValid() {
+    const { newDevice } = this.state;
+    if (!newDevice.deviceType) {
+      valid = false;
+    }
+    if (newDevice.manufacturer.trim().length === 0) {
+      valid = false;
+    }
+    return this.onCheckFormValue(newDevice);
+  }
+
   render() {
     const {
       deviceTrait, isMobileView, showConfirmation, indexNo,
@@ -288,7 +299,7 @@ export default class Devices extends ConsentComponent {
       ? deviceTrait.traits.data.slice() : [];
     const { newDevice, formInvalid, errorMessage } = this.state;
     const canModifyTrait = !this.props.traitRequestCount;
-
+    const isValidDeviceForm = this.isFormValid();
     return (
       <div styleName="devices-container">
         {

src/shared/components/Settings/Tools/ServiceProviders/index.jsx

@@ -30,7 +30,7 @@ export default class ServiceProviders extends ConsentComponent {
     this.onAddServiceProvider = this.onAddServiceProvider.bind(this);
     this.loadPersonalizationTrait = this.loadPersonalizationTrait.bind(this);
     this.updatePredicate = this.updatePredicate.bind(this);
-
+    this.isFormValid = this.isFormValid.bind(this);
     const { userTraits } = props;
     this.state = {
       formInvalid: false,
@@ -247,6 +247,14 @@ export default class ServiceProviders extends ConsentComponent {
     this.setState({ isMobileView: window.innerWidth <= screenSM });
   }
 
+  isFormValid() {
+    const { newServiceProvider } = this.state;
+    if(newServiceProvider.serviceProviderType && (newServiceProvider.name.trim().length !== 0)) {
+      return true;
+    }
+    return false;
+  }
+
   render() {
     const {
       serviceProviderTrait, isMobileView, showConfirmation, indexNo,
@@ -255,7 +263,7 @@ export default class ServiceProviders extends ConsentComponent {
       ? serviceProviderTrait.traits.data.slice() : [];
     const { newServiceProvider, formInvalid, errorMessage } = this.state;
     const canModifyTrait = !this.props.traitRequestCount;
-
+    const isValidServiceProviderForm = this.isFormValid();
     return (
       <div styleName="service-provider-container">
         {
@@ -331,7 +339,7 @@ export default class ServiceProviders extends ConsentComponent {
             <PrimaryButton
               styleName="complete"
               onClick={this.onHandleAddServiceProvider}
-              disabled={!canModifyTrait}
+              disabled={!canModifyTrait || !isValidServiceProviderForm}
             >
               Add service provider to your list
             </PrimaryButton>
@@ -380,7 +388,7 @@ export default class ServiceProviders extends ConsentComponent {
             <PrimaryButton
               styleName="complete"
               onClick={this.onHandleAddServiceProvider}
-              disabled={!canModifyTrait}
+              disabled={!canModifyTrait || !isValidServiceProviderForm}
             >
               Add Provider
             </PrimaryButton>

src/shared/components/Settings/Tools/Software/index.jsx

@@ -247,6 +247,14 @@ export default class Software extends ConsentComponent {
     this.setState({ isMobileView: window.innerWidth <= screenSM });
   }
 
+  isFormValid() {
+    const { newSoftware } = this.state;
+    if(newSoftware.softwareType && (newSoftware.name.trim().length !== 0)) {
+      return true;
+    }
+    return false;
+  }
+
   render() {
     const {
       softwareTrait, isMobileView, showConfirmation, indexNo,
@@ -255,7 +263,7 @@ export default class Software extends ConsentComponent {
       ? softwareTrait.traits.data.slice() : [];
     const { newSoftware, formInvalid, errorMessage } = this.state;
     const canModifyTrait = !this.props.traitRequestCount;
-
+    const isValidSoftwareForm = this.isFormValid();
     return (
       <div styleName="software-container">
         {
@@ -331,7 +339,7 @@ export default class Software extends ConsentComponent {
             <PrimaryButton
               styleName="complete"
               onClick={this.onHandleAddSoftware}
-              disabled={!canModifyTrait}
+              disabled={!canModifyTrait || !isValidSoftwareForm}
             >
               Add software to your list
             </PrimaryButton>
@@ -380,7 +388,7 @@ export default class Software extends ConsentComponent {
             <PrimaryButton
               styleName="complete"
               onClick={this.onHandleAddSoftware}
-              disabled={!canModifyTrait}
+              disabled={!canModifyTrait || !isValidSoftwareForm}
             >
               Add Software
             </PrimaryButton>

src/shared/components/Settings/Tools/Subscriptions/index.jsx

@@ -297,7 +297,7 @@ export default class Subscription extends ConsentComponent {
             <PrimaryButton
               theme={{ button: styles.complete }}
               onClick={this.onHandleAddSubscription}
-              disabled={!canModifyTrait}
+              disabled={!canModifyTrait || (newSubscription.name.trim().length === 0)}
             >
               Add subscription to your list
             </PrimaryButton>
@@ -328,7 +328,7 @@ export default class Subscription extends ConsentComponent {
             <PrimaryButton
               theme={{ button: styles.complete }}
               onClick={this.onHandleAddSubscription}
-              disabled={!canModifyTrait}
+              disabled={!canModifyTrait || (newSubscription.name.trim().length === 0)}
             >
               Add Subscription
             </PrimaryButton>