Fix eslint errors requiring manual intervention

src/App.js

@@ -1,3 +1,4 @@
+/* eslint-disable import/first */
 import React from 'react';
 import '../node_modules/font-awesome/css/font-awesome.css';
 import './styles/App.css';

src/components/builder/BuilderPage.js

@@ -133,7 +133,7 @@ class BuilderPage extends Component {
     if (elementIndex !== undefined) {
       // get relevant parameter
       const paramIndex = elements[elementIndex].parameters.findIndex(
-        param => state.hasOwnProperty(param.id) === true);
+        param => Object.prototype.hasOwnProperty.call(state, param.id));
 
       // edit element with new value using immutability-helper
       const editedElements = update(elements, {

src/components/builder/BuilderTarget.js

@@ -53,12 +53,13 @@ const spec = {
 
     /*
       On drop, copy the element to create a new TemplateInstance.
-      TODO: clone is required because we are setting value on the parameter. this may not be the best approach
+      TODO: clone is required because we are setting value on the parameter.
+      This may not be the best approach
     */
     const clone = JSON.parse(JSON.stringify(item));
     clone.uniqueId = _.uniqueId(clone.id);
     clone.parameters.forEach((param) => {
-      param.value = '';
+      param.value = ''; // eslint-disable-line no-param-reassign
     });
     props.updateTemplateInstances(props.templateInstances.concat(clone));
   },

src/components/builder/TemplateInstance.js

@@ -78,7 +78,7 @@ class TemplateInstance extends Component {
             updateInstance={this.updateInstance} />
         );
       default:
-        return null;
+        return undefined;
     }
   }
 

src/components/builder/parameters/StringParameter.js

@@ -16,12 +16,15 @@ class StringParameter extends Component {
   render() {
     const id = _.uniqueId('parameter-');
     const helpId = _.uniqueId('parameter-help-');
+    let validityClassName = null;
+    if (this.state.valid) {
+      validityClassName = 'form__group-valid';
+    } else if (this.state.valid === false) {
+      validityClassName = 'form__group-invalid';
+    }
 
     return (
-      <div className={`form__group ${
-        (this.state.valid) ? 'form__group-valid'
-        : (this.state.valid === false) ? 'form__group-invalid' : null
-      }`}>
+      <div className={`form__group ${validityClassName}`}>
         <label htmlFor={id}>
           {this.props.name}:
 

src/data/templates.js

@@ -48,13 +48,21 @@ module.exports = [
   //   id: 2,
   //   icon: 'stethoscope',
   //   name: 'Diagnoses',
-  //   entries: ['Myocardial Infarction', 'Cerebrovascular disease, Stroke, TIA', 'Atherosclerosis and Peripheral Arterial Disease', 'Ischemic heart disease or coronary occlusion, rupture, or thrombosis', 'Stable and Unstable Angina', 'Hypertension', 'Diabetes', 'Familial Hypercholesterolemia'],
+  //   entries: [
+  //    'Myocardial Infarction', 'Cerebrovascular disease, Stroke, TIA',
+  //    'Atherosclerosis and Peripheral Arterial Disease',
+  //    'Ischemic heart disease or coronary occlusion, rupture, or thrombosis',
+  //    'Stable and Unstable Angina', 'Hypertension', 'Diabetes',
+  //    'Familial Hypercholesterolemia'
+  //   ],
   // },
   // {
   //   id: 3,
   //   icon: 'flask',
   //   name: 'Lab Results',
-  //   entries: ['Total Cholesterol', 'HDL Cholesterol', 'Systolic Blood Pressure', 'LDL Cholesterol'],
+  //   entries: [
+  //     'Total Cholesterol', 'HDL Cholesterol', 'Systolic Blood Pressure', 'LDL Cholesterol'
+  //   ],
   // },
   // {
   //   id: 4,

src/helpers/test_fixtures.js

@@ -60,6 +60,4 @@ const droppedElements = [
   }
 ];
 
-export {
-  droppedElements
-};
+export default droppedElements;