Merge pull request #214 from TailorDev/deps

Update some dependencies
TailorDev · Jan 15, 2017 · 75330ce · 75330ce
2 parents ceb6ce5 + c1c87d5
commit 75330ce
Show file tree

Hide file tree

Showing 39 changed files with 175 additions and 98 deletions.
diff --git a/.eslintrc b/.eslintrc
@@ -9,6 +9,10 @@
     "import/no-extraneous-dependencies": [2, {
       'optionalDependencies': false,
     }],
+    "react/no-unused-prop-types": 0,
+    "react/forbid-prop-types": 1,
+    "jsx-a11y/no-static-element-interactions": 1,
+    "react/no-array-index-key": 1,
   },
   "parserOptions":{
     "ecmaFeatures": {

diff --git a/app/components/Annotation/Line.jsx b/app/components/Annotation/Line.jsx
@@ -6,7 +6,7 @@ const tickPosition = {
   y: 4,
 };
 
-const Line = (props) => (
+const Line = props => (
   <g>
     <line
       x1={props.x1}

diff --git a/app/components/Annotation/index.jsx b/app/components/Annotation/index.jsx
@@ -28,7 +28,7 @@ class Annotation extends Component {
     this.props.onClick(
       this.props.labelId,
       this.props.annotation,
-      this.props.positionFrom
+      this.props.positionFrom,
     );
   }
 
@@ -37,7 +37,7 @@ class Annotation extends Component {
       segments: props.getAnnotationSegments(
         props.labelId,
         props.annotation.positionFrom - 1,
-        props.annotation.positionTo - 1
+        props.annotation.positionTo - 1,
       ),
     });
   }
@@ -67,7 +67,7 @@ class Annotation extends Component {
             color={this.props.label.color}
             hasTick={displayTick && !isUnit && indexForTick === index}
             isReverse={isReverse}
-          />
+          />,
         )}
       </g>
     );

diff --git a/app/components/AnnotationForm/presenter.jsx b/app/components/AnnotationForm/presenter.jsx
@@ -89,7 +89,7 @@ class AnnotationForm extends Component {
           comment: this.state.comment,
           displayTick: this.state.displayTick,
         },
-        this.state.annotationId
+        this.state.annotationId,
       );
     });
 
@@ -121,10 +121,10 @@ class AnnotationForm extends Component {
     event.preventDefault();
 
     this.props.updateSelectionFrom(
-      this.state.positionTo - this.props.positionFrom
+      this.state.positionTo - this.props.positionFrom,
     );
     this.props.updateSelectionTo(
-      this.state.positionFrom - this.props.positionFrom
+      this.state.positionFrom - this.props.positionFrom,
     );
   }
 
@@ -188,11 +188,11 @@ class AnnotationForm extends Component {
             {
               this.props.labels.map((label, index) =>
                 <option
-                  key={index}
+                  key={label.name}
                   value={index}
                 >
                   {label.name}
-                </option>
+                </option>,
               )
             }
           </select>
@@ -259,4 +259,8 @@ AnnotationForm.propTypes = {
   positionFrom: number.isRequired,
 };
 
+AnnotationForm.defaultProps = {
+  current: null,
+};
+
 export default AnnotationForm;
diff --git a/app/components/Annotations/index.jsx b/app/components/Annotations/index.jsx
@@ -24,7 +24,7 @@ class Annotations extends Component {
       this.props.nucleotideWidth,
       this.props.rowHeight,
       this.props.nucleotidesRowHeight,
-      this.props.trackHeight
+      this.props.trackHeight,
     );
   }
 
@@ -46,8 +46,8 @@ class Annotations extends Component {
                 getAnnotationSegments={this.getAnnotationSegments}
                 positionFrom={this.props.positionFrom}
                 onClick={this.props.onAnnotationClick}
-              />
-            )
+              />,
+            ),
           )
         }
       </g>
@@ -68,4 +68,8 @@ Annotations.propTypes = {
   selectedAnnotation: object,
 };
 
+Annotations.defaultProps = {
+  selectedAnnotation: null,
+};
+
 export default Annotations;
diff --git a/app/components/Disclaimer/index.jsx b/app/components/Disclaimer/index.jsx
@@ -1,7 +1,7 @@
 import React from 'react';
 import Modal from 'react-modal';
 
-const Disclaimer = (props) => (
+const Disclaimer = props => (
   <Modal
     overlayClassName="modal-overlay"
     className="modal-content"

diff --git a/app/components/Exons/ExonEdit.jsx b/app/components/Exons/ExonEdit.jsx
@@ -2,7 +2,7 @@ import React, { PropTypes } from 'react';
 import ExonForm from './ExonForm';
 
 
-const ExonEdit = (props) => (
+const ExonEdit = props => (
   <div className="action-edit">
     <ExonForm
       exon={props.exon}

diff --git a/app/components/Exons/ExonForm.jsx b/app/components/Exons/ExonForm.jsx
@@ -168,4 +168,11 @@ ExonForm.propTypes = {
   exon: object,
 };
 
+ExonForm.defaultProps = {
+  onRemoveExon: null,
+  onCreateNewExon: null,
+  onEditExon: null,
+  exon: null,
+};
+
 export default ExonForm;
diff --git a/app/components/Exons/presenter.jsx b/app/components/Exons/presenter.jsx
@@ -45,7 +45,7 @@ class Exons extends Component {
             exon={exon}
             onEditExon={this.props.onEditExon}
             onRemoveExon={this.props.onRemoveExon}
-          />
+          />,
         )}
         {/* New exon */}
         <div className="new">
@@ -56,10 +56,10 @@ class Exons extends Component {
               onCreateNewExon={this.props.onCreateNewExon}
             />
             :
-              <button
-                className="button new-exon"
-                onClick={this.toggleNewExonForm}
-              >
+            <button
+              className="button new-exon"
+              onClick={this.toggleNewExonForm}
+            >
                 New Exon
               </button>
           }

diff --git a/app/components/Footer.jsx b/app/components/Footer.jsx
@@ -3,7 +3,7 @@ import React, { PropTypes } from 'react';
 const { string } = PropTypes;
 
 
-const Footer = (props) =>
+const Footer = props =>
   <footer className="main">
     <div className="version">
       <span className="git-ref">

diff --git a/app/components/InputNumber/InlineButton.jsx b/app/components/InputNumber/InlineButton.jsx
@@ -20,15 +20,15 @@ const InlineButton = (props) => {
           <button
             onClick={props.onClick}
             className="button"
-            title={props.titleText || null}
+            title={props.titleText}
             disabled={props.hasButtonDisabled || false}
           >
             {props.children}
           </button>
         </div>
       </div>
       <small className={`form-error${isInvalid ? ' is-visible' : ''}`}>
-        {props.errorText || 'Invalid value'}
+        {props.errorText}
       </small>
     </div>
   );
@@ -55,4 +55,16 @@ InlineButton.propTypes = {
   hasButtonDisabled: React.PropTypes.bool,
 };
 
+InlineButton.defaultProps = {
+  children: null,
+  errorText: 'Invalid value',
+  isDisabled: false,
+  placeholder: null,
+  min: null,
+  value: '',
+  className: null,
+  titleText: null,
+  hasButtonDisabled: false,
+};
+
 export default InlineButton;
diff --git a/app/components/InputNumber/RawInputNumber.jsx b/app/components/InputNumber/RawInputNumber.jsx
@@ -1,7 +1,7 @@
 import React from 'react';
 
 
-const RawInputNumber = (props) =>
+const RawInputNumber = props =>
   <input
     type="number"
     name={`input-number-${props.className}`}
@@ -31,4 +31,13 @@ RawInputNumber.propTypes = {
   className: React.PropTypes.string,
 };
 
+RawInputNumber.defaultProps = {
+  isDisabled: false,
+  isInvalid: false,
+  placeholder: null,
+  min: null,
+  value: '',
+  className: null,
+};
+
 export default RawInputNumber;
diff --git a/app/components/InputNumber/index.jsx b/app/components/InputNumber/index.jsx
@@ -14,13 +14,13 @@ const InputNumber = (props) => {
       htmlFor={`input-number-${props.className}`}
       className={isInvalid ? 'is-invalid-label' : ''}
     >
-      {props.labelText || null}
+      {props.labelText}
       <RawInputNumber
         {...props}
         isInvalid={isInvalid}
       />
       <small className={`form-error${isInvalid ? ' is-visible' : ''}`}>
-        {props.errorText || 'Invalid value'}
+        {props.errorText}
       </small>
     </label>
   );
@@ -41,4 +41,14 @@ InputNumber.propTypes = {
   className: React.PropTypes.string,
 };
 
+InputNumber.defaultProps = {
+  isDisabled: false,
+  value: '',
+  min: null,
+  labelText: null,
+  errorText: 'Invalid value',
+  className: null,
+  placeholder: '',
+};
+
 export default InputNumber;
diff --git a/app/components/Labels/LabelEdit.jsx b/app/components/Labels/LabelEdit.jsx
@@ -2,7 +2,7 @@ import React, { PropTypes } from 'react';
 import LabelForm from './LabelForm';
 
 
-const LabelEdit = (props) => (
+const LabelEdit = props => (
   <div className="action-edit">
     <LabelForm
       label={props.label}

diff --git a/app/components/Labels/LabelForm.jsx b/app/components/Labels/LabelForm.jsx
@@ -48,7 +48,7 @@ class LabelForm extends Component {
   }
 
   handleColorChange(color) {
-    this.setState((previousState) => ({
+    this.setState(previousState => ({
       label: {
         name: previousState.label.name,
         color: color.hex,
@@ -62,7 +62,7 @@ class LabelForm extends Component {
   handleNameChange(event) {
     event.persist();
 
-    this.setState((previousState) => ({
+    this.setState(previousState => ({
       label: {
         name: event.target.value,
         color: previousState.label.color,
@@ -158,4 +158,10 @@ LabelForm.propTypes = {
   label: object,
 };
 
+LabelForm.defaultProps = {
+  onCreateNewLabel: null,
+  onEditLabel: null,
+  label: null,
+};
+
 export default LabelForm;
diff --git a/app/components/Labels/LabelTools.jsx b/app/components/Labels/LabelTools.jsx
@@ -1,7 +1,7 @@
 import React, { PropTypes } from 'react';
 
 
-const LabelTools = (props) => (
+const LabelTools = props => (
   <span className="label-tools">
     <i
       className="fa fa-adjust toggle"

diff --git a/app/components/Labels/presenter.jsx b/app/components/Labels/presenter.jsx
@@ -38,7 +38,7 @@ export default class Labels extends Component {
             onToggleLabel={() => { this.props.onToggleLabel(index); }}
             onEditLabel={(editedLabel) => { this.props.onEditLabel(index, editedLabel); }}
             onRemoveLabel={() => { this.props.onRemoveLabel(index); }}
-          />
+          />,
         )}
 
         <li className="new">
@@ -48,10 +48,10 @@ export default class Labels extends Component {
               onCancel={this.toggleNewLabelForm}
             />
             :
-              <button
-                className="button new-label"
-                onClick={this.toggleNewLabelForm}
-              >
+            <button
+              className="button new-label"
+              onClick={this.toggleNewLabelForm}
+            >
                 New label
               </button>
           }

diff --git a/app/components/Loader/presenter.jsx b/app/components/Loader/presenter.jsx
@@ -1,6 +1,6 @@
 import React from 'react';
 
-const Loader = (props) => (
+const Loader = props => (
   <div className="loader" style={{ display: props.display ? 'block' : 'none' }}>
     <p className="message">
       Rendering your sequence...<br /><br />

diff --git a/app/components/Notifications/presenter.jsx b/app/components/Notifications/presenter.jsx
@@ -11,7 +11,7 @@ const Notifications = props => (
         key={index}
         message={message}
         onClose={props.onMessageBoxClose}
-      />
+      />,
     )}
   </div>
 );

diff --git a/app/components/Nucleotide/index.js b/app/components/Nucleotide/index.js
@@ -6,16 +6,16 @@ export const mapStateToProps = (state, ownProps) => {
   const exon = state.exon;
 
   const isSelected = (0 < selection.selections.filter(
-    s => (s.from === ownProps.index || s.to === ownProps.index)
+    s => (s.from === ownProps.index || s.to === ownProps.index),
   ).length);
 
   const isInSelectionRange = (0 < selection.selections.filter(s =>
     (s.from <= ownProps.index && s.to >= ownProps.index) ||
-    (s.from >= ownProps.index && s.to <= ownProps.index)
+    (s.from >= ownProps.index && s.to <= ownProps.index),
   ).length);
 
   const nbExons = exon.exons.filter(
-    e => e.positionFrom <= ownProps.position && e.positionTo >= ownProps.position
+    e => e.positionFrom <= ownProps.position && e.positionTo >= ownProps.position,
   ).size;
 
   return {

diff --git a/app/components/Nucleotide/presenter.jsx b/app/components/Nucleotide/presenter.jsx
@@ -55,7 +55,7 @@ export default class Nucleotide extends Component {
       props.visualizerMargin,
       props.nucleotidesPerRow,
       props.nucleotideWidth,
-      props.rowHeight
+      props.rowHeight,
     ));
   }
 

diff --git a/app/components/Remove/index.jsx b/app/components/Remove/index.jsx
@@ -1,7 +1,7 @@
 import React, { PropTypes } from 'react';
 
 
-const Remove = (props) => (
+const Remove = props => (
   <div className="action-remove">
     <p>
       {props.children}