Merge pull request #3813 from vigneshTheDev/fix-3634

Fix issue #3634 for tabs

Author: maxceem

src/projects/detail/components/SkillsQuestion/SkillsQuestion.jsx

@@ -98,16 +98,22 @@ class SkillsQuestion extends React.Component {
     const indexOfSpace = value.indexOf(' ')
     const indexOfSemiColon = value.indexOf(';')
 
-    // if user enter only ' '  or ';' we should clean it to not allow
+    // if user enter ' '  or ';' in the start of the input, we should clean it to not allow
     if (indexOfSpace === 0 || indexOfSemiColon === 0 ) {
-      return ''
+      return value.slice(1)
     }
 
     if (indexOfSemiColon >= 1 ) {
+      const newValue = value.replace(';', '').trim()
       const currentValues = getValue()
-      this.handleChange([...currentValues, { name: value.substring(0, value.length -1) }])
-      // this is return empty to nullify value post processing
-      return ''
+      if (!_.some(currentValues, v => v && v.name === newValue)) {
+        this.handleChange([...currentValues, { name:  newValue}])
+        // this is return empty to nullify value post processing
+        return ''
+      } else {
+        // don't allow semicolon for duplicate values
+        return value.replace(';', '')
+      }
     }
 
     this.setState({ customOptionValue: value })
@@ -172,7 +178,7 @@ class SkillsQuestion extends React.Component {
             placeholder="Start typing a skill then select from the list"
             value={selectGroupValues}
             getOptionLabel={(option) => option.name || ''}
-            getOptionValue={(option) => option.name || ''}
+            getOptionValue={(option) => (option.name || '').trim()}
             onInputChange={this.onSelectType}
             onChange={(val) => {
               this.handleChange(_.union(val, checkboxGroupValues))