From 313fc856d19a49714dbd692fe052893bf98f1859 Mon Sep 17 00:00:00 2001
From: Andrew Nicolaou <me@andrewnicolaou.co.uk>
Date: Tue, 6 Jun 2017 04:33:32 +0200
Subject: [PATCH] Fixes linting errors (#362)

---
 client/components/Nav.jsx                         | 15 ++++++++++++---
 client/modules/IDE/components/Console.jsx         |  6 +++++-
 client/modules/IDE/components/FileNode.jsx        |  6 +++++-
 client/modules/IDE/components/GridOutput.jsx      |  2 +-
 client/modules/IDE/components/PreviewFrame.jsx    |  6 +++++-
 client/modules/IDE/components/Toolbar.jsx         |  5 ++++-
 client/modules/IDE/pages/IDEView.jsx              | 12 ++++++++++--
 client/modules/User/components/AccountForm.jsx    | 13 +++++++++++--
 .../modules/User/components/NewPasswordForm.jsx   |  6 +++++-
 .../modules/User/components/ResetPasswordForm.jsx |  7 ++++++-
 client/modules/User/components/SignupForm.jsx     |  6 +++++-
 client/utils/reduxFormUtils.js                    |  4 +++-
 12 files changed, 72 insertions(+), 16 deletions(-)

diff --git a/client/components/Nav.jsx b/client/components/Nav.jsx
index 181c0e64..6ef4fb35 100644
--- a/client/components/Nav.jsx
+++ b/client/components/Nav.jsx
@@ -21,7 +21,10 @@ class Nav extends React.PureComponent {
             </button>
           </li>
           {(() => { // eslint-disable-line
-            if (!this.props.project.owner || (this.props.project.owner && this.props.project.owner.id === this.props.user.id)) {
+            if (
+              !this.props.project.owner ||
+              (this.props.project.owner && this.props.project.owner.id === this.props.user.id)
+            ) {
               return (
                 <li className="nav__item">
                   <button
@@ -119,7 +122,9 @@ class Nav extends React.PureComponent {
               return (
                 <li className="nav__item">
                   <p>
-                    <Link to="/login">Log in</Link> <span className="nav__item-spacer">or</span> <Link to="/signup">Sign up</Link>
+                    <Link to="/login">Log in</Link>
+                    <span className="nav__item-spacer">or</span>
+                    <Link to="/signup">Sign up</Link>
                   </p>
                 </li>
               );
@@ -153,7 +158,11 @@ class Nav extends React.PureComponent {
         </ul>
         <div className="nav__announce">
           This is a preview version of the editor, that has not yet been officially released.
-          It is in development, you can report bugs <a href="https://github.com/processing/p5.js-web-editor/issues" target="_blank" rel="noopener noreferrer">here</a>.
+          It is in development, you can report bugs <a
+            href="https://github.com/processing/p5.js-web-editor/issues"
+            target="_blank"
+            rel="noopener noreferrer"
+          >here</a>.
           Please use with caution.
         </div>
       </nav>
diff --git a/client/modules/IDE/components/Console.jsx b/client/modules/IDE/components/Console.jsx
index d3f2eabb..6fb3e143 100644
--- a/client/modules/IDE/components/Console.jsx
+++ b/client/modules/IDE/components/Console.jsx
@@ -24,7 +24,11 @@ class Console extends React.Component {
             <button className="preview-console__clear" onClick={this.props.clearConsole} aria-label="clear console">
               Clear
             </button>
-            <button className="preview-console__collapse" onClick={this.props.collapseConsole} aria-label="collapse console">
+            <button
+              className="preview-console__collapse"
+              onClick={this.props.collapseConsole}
+              aria-label="collapse console"
+            >
               <InlineSVG src={downArrowUrl} />
             </button>
             <button className="preview-console__expand" onClick={this.props.expandConsole} aria-label="expand console">
diff --git a/client/modules/IDE/components/FileNode.jsx b/client/modules/IDE/components/FileNode.jsx
index 4368d2e7..d2afc08a 100644
--- a/client/modules/IDE/components/FileNode.jsx
+++ b/client/modules/IDE/components/FileNode.jsx
@@ -45,7 +45,11 @@ export class FileNode extends React.Component {
     if (oldFileExtension && !newFileExtension) {
       this.props.updateFileName(this.props.id, this.originalFileName);
     }
-    if (oldFileExtension && newFileExtension && oldFileExtension[0].toLowerCase() !== newFileExtension[0].toLowerCase()) {
+    if (
+      oldFileExtension &&
+      newFileExtension &&
+      oldFileExtension[0].toLowerCase() !== newFileExtension[0].toLowerCase()
+    ) {
       this.props.updateFileName(this.props.id, this.originalFileName);
     }
   }
diff --git a/client/modules/IDE/components/GridOutput.jsx b/client/modules/IDE/components/GridOutput.jsx
index 17fb1213..f1e855d9 100644
--- a/client/modules/IDE/components/GridOutput.jsx
+++ b/client/modules/IDE/components/GridOutput.jsx
@@ -1,4 +1,4 @@
-import React, { PropTypes } from 'react';
+import React from 'react';
 
 class GridOutput extends React.Component {
   componentDidMount() {
diff --git a/client/modules/IDE/components/PreviewFrame.jsx b/client/modules/IDE/components/PreviewFrame.jsx
index 622e7329..12fc014e 100644
--- a/client/modules/IDE/components/PreviewFrame.jsx
+++ b/client/modules/IDE/components/PreviewFrame.jsx
@@ -8,7 +8,9 @@ import { getBlobUrl } from '../actions/files';
 import { resolvePathToFile } from '../../../../server/utils/filePath';
 
 const startTag = '@fs-';
+// eslint-disable-next-line max-len
 const MEDIA_FILE_REGEX = /^('|")(?!(http:\/\/|https:\/\/)).*\.(png|jpg|jpeg|gif|bmp|mp3|wav|aiff|ogg|json|txt|csv|svg|obj|mp4|ogg|webm|mov|otf|ttf|m4a)('|")$/i;
+// eslint-disable-next-line max-len
 const MEDIA_FILE_REGEX_NO_QUOTES = /^(?!(http:\/\/|https:\/\/)).*\.(png|jpg|jpeg|gif|bmp|mp3|wav|aiff|ogg|json|txt|csv|svg|obj|mp4|ogg|webm|mov|otf|ttf|m4a)$/i;
 const STRING_REGEX = /(['"])((\\\1|.)*?)\1/gm;
 const TEXT_FILE_REGEX = /(.+\.json$|.+\.txt$|.+\.csv$)/i;
@@ -175,7 +177,9 @@ class PreviewFrame extends React.Component {
       '/loop-protect.min.js',
       '/hijackConsole.js'
     ];
-    if (this.props.isAccessibleOutputPlaying || ((this.props.textOutput || this.props.gridOutput || this.props.soundOutput) && this.props.isPlaying)) {
+    if (
+      this.props.isAccessibleOutputPlaying ||
+      ((this.props.textOutput || this.props.gridOutput || this.props.soundOutput) && this.props.isPlaying)) {
       let interceptorScripts = [];
       interceptorScripts = [
         '/p5-interceptor/registry.js',
diff --git a/client/modules/IDE/components/Toolbar.jsx b/client/modules/IDE/components/Toolbar.jsx
index 123432c0..c23bf071 100644
--- a/client/modules/IDE/components/Toolbar.jsx
+++ b/client/modules/IDE/components/Toolbar.jsx
@@ -141,7 +141,10 @@ class Toolbar extends React.Component {
             }}
           >
             {this.props.project.name}&nbsp;
-            {this.canEditProjectName() && <InlineSVG className="toolbar__edit-name-button" src={editProjectNameUrl} alt="Edit Project Name" />}
+            {
+              this.canEditProjectName() &&
+              <InlineSVG className="toolbar__edit-name-button" src={editProjectNameUrl} alt="Edit Project Name" />
+            }
           </a>
           <input
             type="text"
diff --git a/client/modules/IDE/pages/IDEView.jsx b/client/modules/IDE/pages/IDEView.jsx
index 6487fa46..7a258193 100644
--- a/client/modules/IDE/pages/IDEView.jsx
+++ b/client/modules/IDE/pages/IDEView.jsx
@@ -97,7 +97,9 @@ class IDEView extends React.Component {
   componentDidUpdate(prevProps) {
     if (this.isUserOwner() && this.props.project.id) {
       if (this.props.preferences.autosave && this.props.ide.unsavedChanges && !this.props.ide.justOpenedProject) {
-        if (this.props.selectedFile.name === prevProps.selectedFile.name && this.props.selectedFile.content !== prevProps.selectedFile.content) {
+        if (
+          this.props.selectedFile.name === prevProps.selectedFile.name &&
+          this.props.selectedFile.content !== prevProps.selectedFile.content) {
           if (this.autosaveInterval) {
             clearTimeout(this.autosaveInterval);
           }
@@ -350,7 +352,13 @@ class IDEView extends React.Component {
                 </div>
                 <div>
                   {(() => {
-                    if (((this.props.preferences.textOutput || this.props.preferences.gridOutput || this.props.preferences.soundOutput) && this.props.ide.isPlaying) || this.props.ide.isAccessibleOutputPlaying) {
+                    if (
+                      (
+                        (this.props.preferences.textOutput ||
+                         this.props.preferences.gridOutput ||
+                         this.props.preferences.soundOutput
+                        ) && this.props.ide.isPlaying
+                      ) || this.props.ide.isAccessibleOutputPlaying) {
                       return (
                         <AccessibleOutput
                           isPlaying={this.props.ide.isPlaying}
diff --git a/client/modules/User/components/AccountForm.jsx b/client/modules/User/components/AccountForm.jsx
index b8980d9f..f8b4296e 100644
--- a/client/modules/User/components/AccountForm.jsx
+++ b/client/modules/User/components/AccountForm.jsx
@@ -43,7 +43,11 @@ function AccountForm(props) {
           id="currentPassword"
           {...domOnlyProps(currentPassword)}
         />
-        {currentPassword.touched && currentPassword.error && <span className="form-error">{currentPassword.error}</span>}
+        {
+          currentPassword.touched &&
+          currentPassword.error &&
+          <span className="form-error">{currentPassword.error}</span>
+        }
       </p>
       <p className="form__field">
         <label htmlFor="new password" className="form__label">New Password</label>
@@ -56,7 +60,12 @@ function AccountForm(props) {
         />
         {newPassword.touched && newPassword.error && <span className="form-error">{newPassword.error}</span>}
       </p>
-      <input type="submit" disabled={submitting || invalid || pristine} value="Save All Settings" aria-label="updateSettings" />
+      <input
+        type="submit"
+        disabled={submitting || invalid || pristine}
+        value="Save All Settings"
+        aria-label="updateSettings"
+      />
     </form>
   );
 }
diff --git a/client/modules/User/components/NewPasswordForm.jsx b/client/modules/User/components/NewPasswordForm.jsx
index 6e28de49..22c2839f 100644
--- a/client/modules/User/components/NewPasswordForm.jsx
+++ b/client/modules/User/components/NewPasswordForm.jsx
@@ -25,7 +25,11 @@ function NewPasswordForm(props) {
           id="confirm password"
           {...domOnlyProps(confirmPassword)}
         />
-        {confirmPassword.touched && confirmPassword.error && <span className="form-error">{confirmPassword.error}</span>}
+        {
+          confirmPassword.touched &&
+          confirmPassword.error &&
+          <span className="form-error">{confirmPassword.error}</span>
+        }
       </p>
       <input type="submit" disabled={submitting || invalid || pristine} value="Set New Password" aria-label="sign up" />
     </form>
diff --git a/client/modules/User/components/ResetPasswordForm.jsx b/client/modules/User/components/ResetPasswordForm.jsx
index 2c3dd4c9..ba59cd4a 100644
--- a/client/modules/User/components/ResetPasswordForm.jsx
+++ b/client/modules/User/components/ResetPasswordForm.jsx
@@ -15,7 +15,12 @@ function ResetPasswordForm(props) {
           {...domOnlyProps(email)}
         />
       </p>
-      <input type="submit" disabled={submitting || invalid || pristine || props.user.resetPasswordInitiate} value="Send Password Reset Email" aria-label="Send email to reset password" />
+      <input
+        type="submit"
+        disabled={submitting || invalid || pristine || props.user.resetPasswordInitiate}
+        value="Send Password Reset Email"
+        aria-label="Send email to reset password"
+      />
     </form>
   );
 }
diff --git a/client/modules/User/components/SignupForm.jsx b/client/modules/User/components/SignupForm.jsx
index 58513d1e..8f5a99b3 100644
--- a/client/modules/User/components/SignupForm.jsx
+++ b/client/modules/User/components/SignupForm.jsx
@@ -47,7 +47,11 @@ function SignupForm(props) {
           id="confirm password"
           {...domOnlyProps(confirmPassword)}
         />
-        {confirmPassword.touched && confirmPassword.error && <span className="form-error">{confirmPassword.error}</span>}
+        {
+          confirmPassword.touched &&
+          confirmPassword.error &&
+          <span className="form-error">{confirmPassword.error}</span>
+        }
       </p>
       <input type="submit" disabled={submitting || invalid || pristine} value="Sign Up" aria-label="sign up" />
     </form>
diff --git a/client/utils/reduxFormUtils.js b/client/utils/reduxFormUtils.js
index c2c68fa0..4bb74319 100644
--- a/client/utils/reduxFormUtils.js
+++ b/client/utils/reduxFormUtils.js
@@ -26,7 +26,9 @@ function validateNameEmail(formProps, errors) {
 
   if (!formProps.email) {
     errors.email = 'Please enter an email.';
-  } else if (!formProps.email.match(/^(([^<>()[\]\\.,;:\s@"]+(\.[^<>()[\]\\.,;:\s@"]+)*)|(".+"))@((\[[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}])|(([a-zA-Z\-0-9]+\.)+[a-zA-Z]{2,}))$/i)) {
+  } else if (
+    // eslint-disable-next-line max-len
+    !formProps.email.match(/^(([^<>()[\]\\.,;:\s@"]+(\.[^<>()[\]\\.,;:\s@"]+)*)|(".+"))@((\[[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}])|(([a-zA-Z\-0-9]+\.)+[a-zA-Z]{2,}))$/i)) {
     errors.email = 'Please enter a valid email address.';
   }
 }