More lint removal from material-ui example.

Author: prescottprue

examples/complete/material/src/config.js

@@ -13,7 +13,7 @@ export const firebase = {
 export const reduxFirebase = {
   userProfile: 'users', // root that user profiles are written to
   enableLogging: false, // enable/disable Firebase Database Logging
-  updateProfileOnLogin: false, // enable/disable updating of profile on login
+  updateProfileOnLogin: false // enable/disable updating of profile on login
   // profileDecorator: (userData) => ({ email: userData.email }) // customize format of user profile
 }
 

examples/complete/material/src/containers/Navbar/Navbar.js

@@ -52,7 +52,6 @@ export default class Navbar extends Component {
   }
 
   static propTypes = {
-    auth: PropTypes.object,
     account: PropTypes.object,
     firebase: PropTypes.object.isRequired
   }

examples/complete/material/src/routes/Account/containers/AccountContainer.js

@@ -2,10 +2,8 @@ import React, { Component } from 'react'
 import PropTypes from 'prop-types'
 import Paper from 'material-ui/Paper'
 import { connect } from 'react-redux'
-import { reduxForm } from 'redux-form'
 import { firebaseConnect, pathToJS, isLoaded } from 'react-redux-firebase'
 import { reduxFirebase as rfConfig } from 'config'
-import { ACCOUNT_FORM_NAME } from 'constants'
 import { UserIsAuthenticated } from 'utils/router'
 import defaultUserImageUrl from 'static/User.png'
 import LoadingSpinner from 'components/LoadingSpinner'
@@ -17,12 +15,15 @@ import classes from './AccountContainer.scss'
 @connect( // Map redux state to props
   ({ firebase }) => ({
     auth: pathToJS(firebase, 'auth'),
-    account: pathToJS(firebase, 'profile'),
+    account: pathToJS(firebase, 'profile')
   })
 )
 export default class Account extends Component {
   static propTypes = {
     account: PropTypes.object,
+    auth: PropTypes.shape({
+      uid: PropTypes.string
+    }),
     firebase: PropTypes.shape({
       update: PropTypes.func.isRequired,
       logout: PropTypes.func.isRequired
@@ -43,7 +44,7 @@ export default class Account extends Component {
     this.props.firebase
       .update(`${rfConfig.userProfile}/${this.props.auth.uid}`, newData)
       .catch((err) => {
-        console.error('Error updating account', err)
+        console.error('Error updating account', err) // eslint-disable-line no-console
         // TODO: Display error to user
       })
 

examples/complete/material/src/routes/Home/containers/HomeContainer.js

@@ -8,7 +8,7 @@ import {
   isLoaded,
   pathToJS,
   // dataToJS, // needed for full list and once
-  orderedToJS, // needed for ordered list
+  orderedToJS // needed for ordered list
   // populatedDataToJS // needed for populated list
 } from 'react-redux-firebase'
 import CircularProgress from 'material-ui/CircularProgress'
@@ -25,17 +25,16 @@ import classes from './HomeContainer.scss'
 @firebaseConnect([
   // 'todos' // sync full list of todos
   // { path: 'todos', type: 'once' } // for loading once instead of binding
-  { path: 'todos', queryParams: ['orderByKey', 'limitToLast=8'] } // limit to first 20
+  { path: 'todos', queryParams: ['orderByKey', 'limitToLast=8'] } // 8 most recent
   // { path: 'todos', populates } // populate
-  // { path: 'todos', queryParams: ['orderByKey', 'limitToLast=5'] }, // get five most recent (array will need to be inverted)
 ])
 @connect(
   ({firebase}) => ({
     auth: pathToJS(firebase, 'auth'),
     account: pathToJS(firebase, 'profile'),
     // todos: dataToJS(firebase, 'todos')
     // todos: populatedDataToJS(firebase, '/todos', populates), // if populating
-    todos: orderedToJS(firebase, 'todos'), // if using ordering such as orderByChild
+    todos: orderedToJS(firebase, 'todos') // if using ordering such as orderByChild
   })
 )
 export default class Home extends Component {
@@ -47,7 +46,11 @@ export default class Home extends Component {
     firebase: PropTypes.shape({
       set: PropTypes.func.isRequired,
       remove: PropTypes.func.isRequired,
-      push: PropTypes.func.isRequired
+      push: PropTypes.func.isRequired,
+      database: PropTypes.oneOfType([
+        PropTypes.object,
+        PropTypes.func
+      ])
     }),
     auth: PropTypes.shape({
       uid: PropTypes.string
@@ -67,7 +70,7 @@ export default class Home extends Component {
   }
 
   deleteTodo = (id) => {
-    const { todos, auth, firebase } = this.props
+    const { auth, firebase } = this.props
     if (!auth || !auth.uid) {
       return this.setState({ error: 'You must be Logged into Delete' })
     }
@@ -100,15 +103,13 @@ export default class Home extends Component {
       <div className={classes.container} style={{ color: Theme.palette.primary2Color }}>
         {
           error
-            ?
-              <Snackbar
-                open={!!error}
-                message={error}
-                autoHideDuration={4000}
-                onRequestClose={() => this.setState({ error: null })}
+            ? <Snackbar
+              open={!!error}
+              message={error}
+              autoHideDuration={4000}
+              onRequestClose={() => this.setState({ error: null })}
               />
-            :
-              null
+            : null
         }
         <div className={classes.info}>
           <span>data loaded from</span>

examples/complete/material/src/routes/Projects/containers/ProjectsContainer.js

@@ -41,8 +41,7 @@ export default class Projects extends Component {
     projects: PropTypes.object,
     firebase: PropTypes.object,
     auth: PropTypes.object,
-    children: PropTypes.object,
-    params: PropTypes.object
+    children: PropTypes.object
   }
 
   state = {
@@ -51,7 +50,7 @@ export default class Projects extends Component {
   }
 
   newSubmit = (newProject) => {
-    const { auth, firebase: { push } } = this.props
+    const { firebase: { pushWithMeta } } = this.props
     return pushWithMeta('projects', newProject)
       .then(() => this.setState({ newProjectModal: false }))
       .catch(err => {
@@ -72,7 +71,9 @@ export default class Projects extends Component {
   }
 
   render () {
-    if (!isLoaded(this.props.projects, this.props.auth)) {
+    const { projects, auth } = this.props
+
+    if (!isLoaded(projects, auth)) {
       return <LoadingSpinner />
     }
 
@@ -82,7 +83,6 @@ export default class Projects extends Component {
       return cloneElement(this.props.children, this.props)
     }
 
-    const { projects, auth } = this.props
     const { newProjectModal } = this.state
 
     return (

examples/complete/material/src/routes/Projects/routes/Project/components/Project/Project.js

@@ -1,31 +1,25 @@
-import React, { Component } from 'react'
+import React from 'react'
 import PropTypes from 'prop-types'
-import { connect } from 'react-redux'
-import { firebaseConnect, isLoaded, dataToJS } from 'react-redux-firebase'
-import LoadingSpinner from 'components/LoadingSpinner'
 import classes from './Project.scss'
 
 export const Project = ({ projects, params: { projectname } }) => (
   <div className={classes.container}>
     {
       projects[projectname]
-        ?
-          <div>
-            <h2>Project Container</h2>
-            <pre>{JSON.stringify(projects[projectname], null, 2)}</pre>
-          </div>
-        :
-          <div className={classes.empty}>
-            <span>Project Not Found</span>
-          </div>
+        ? <div>
+          <h2>Project Container</h2>
+          <pre>{JSON.stringify(projects[projectname], null, 2)}</pre>
+        </div>
+        : <div className={classes.empty}>
+          <span>Project Not Found</span>
+        </div>
     }
   </div>
 )
 
 Project.propTypes = {
   projects: PropTypes.object,
-  params: PropTypes.object.isRequired,
-  children: PropTypes.object
+  params: PropTypes.object.isRequired
 }
 
 export default Project

examples/complete/material/src/routes/Recover/components/EmailForm/EmailForm.js

@@ -1,19 +1,15 @@
 import React from 'react'
 import PropTypes from 'prop-types'
-import { connect } from 'react-redux'
-import { Field, reduxForm, submit } from 'redux-form'
-import { firebaseConnect, pathToJS, isLoaded } from 'react-redux-firebase'
+import { Field, reduxForm } from 'redux-form'
 import RaisedButton from 'material-ui/RaisedButton'
 import TextField from 'components/TextField'
-import { List, ListItem } from 'material-ui/List'
-import AccountCircle from 'material-ui/svg-icons/action/account-circle'
 import { RECOVER_EMAIL_FORM_NAME } from 'constants'
 import classes from './EmailForm.scss'
 
 const required = value => value ? undefined : 'Required'
 const email = value =>
-  value && !/^[A-Z0-9._%+-]+@[A-Z0-9.-]+\.[A-Z]{2,4}$/i.test(value) ?
-  'Invalid email address' : undefined
+  value && !/^[A-Z0-9._%+-]+@[A-Z0-9.-]+\.[A-Z]{2,4}$/i.test(value)
+  ? 'Invalid email address' : undefined
 
 export const EmailForm = ({ account, handleSubmit, submitting, pristine, valid }) => (
   <form className={classes.container} onSubmit={handleSubmit}>
@@ -39,8 +35,10 @@ EmailForm.propTypes = {
   account: PropTypes.shape({
     providerData: PropTypes.array
   }),
-  handleSubmit: PropTypes.func.isRequired,
-  submitting: PropTypes.bool
+  pristine: PropTypes.bool, // added by redux-form
+  valid: PropTypes.bool, // added by redux-form
+  handleSubmit: PropTypes.func.isRequired, // added by redux-form
+  submitting: PropTypes.bool // added by redux-form
 }
 
 export default reduxForm({

examples/complete/material/src/routes/Recover/components/RecoverForm/RecoverForm.js

@@ -1,13 +1,9 @@
 import React from 'react'
 import PropTypes from 'prop-types'
-import { connect } from 'react-redux'
-import { Field, reduxForm, submit } from 'redux-form'
-import { firebaseConnect, pathToJS, isLoaded } from 'react-redux-firebase'
+import { Field, reduxForm } from 'redux-form'
 import RaisedButton from 'material-ui/RaisedButton'
 import Subheader from 'material-ui/Subheader'
 import TextField from 'components/TextField'
-import { List, ListItem } from 'material-ui/List'
-import AccountCircle from 'material-ui/svg-icons/action/account-circle'
 import { RECOVER_CODE_FORM_NAME } from 'constants'
 import classes from './RecoverForm.scss'
 
@@ -48,9 +44,11 @@ RecoverForm.propTypes = {
   account: PropTypes.shape({
     providerData: PropTypes.array
   }),
-  handleSubmit: PropTypes.func,
-  onRecoverClick: PropTypes.func,
-  submitting: PropTypes.bool
+  pristine: PropTypes.bool, // added by redux-form
+  valid: PropTypes.bool, // added by redux-form
+  handleSubmit: PropTypes.func, // added by redux-form
+  submitting: PropTypes.bool, // added by redux-form
+  onRecoverClick: PropTypes.func
 }
 
 export default reduxForm({

examples/complete/material/src/routes/Recover/containers/RecoverContainer.js

@@ -1,26 +1,23 @@
 import React, { Component } from 'react'
 import PropTypes from 'prop-types'
-import { reduxForm, submit } from 'redux-form'
 import { firebaseConnect } from 'react-redux-firebase'
-import IconButton from 'material-ui/IconButton'
-import Paper from 'material-ui/Paper'
-import TextField from 'material-ui/TextField'
 import Snackbar from 'material-ui/Snackbar'
-import ContentAdd from 'material-ui/svg-icons/content/add'
-import Subheader from 'material-ui/Subheader'
+import Paper from 'material-ui/Paper'
 import RecoverForm from '../components/RecoverForm'
 import EmailForm from '../components/EmailForm'
-import { RECOVER_FORM_NAME } from 'constants'
 
 @firebaseConnect() // adds this.props.firebase
 export default class RecoverContainer extends Component {
+  static propTypes = {
+    firebase: PropTypes.object
+  }
   state = {
     message: null,
     open: false
   }
 
-  sendRecoveryEmail = ({ email }) => {
-    return this.props.firebase
+  sendRecoveryEmail = ({ email }) =>
+    this.props.firebase
       .resetPassword(email)
       .then(() => {
         this.setState({
@@ -29,11 +26,10 @@ export default class RecoverContainer extends Component {
         })
       })
       .catch((err) => {
-        console.error('Error updating account', err)
+        console.error('Error updating account', err) // eslint-disable-line no-console
         this.setState({ message: err.message || 'Error' }) // show error snackbar
         return Promise.reject(err)
       })
-  }
 
   recoverAccount = ({ code, password }) => {
     const {
@@ -44,20 +40,18 @@ export default class RecoverContainer extends Component {
     return verifyPasswordResetCode(code)
       .then(() => confirmPasswordReset(code, password))
       .then((res) => {
-        this.setState({
-          message: 'Password Changed Successfully'
-        })
+        this.setState({ message: 'Password Changed Successfully' })
       })
       .catch((err) => {
-        console.error('Error updating account', err)
+        console.error('Error updating account', err) // eslint-disable-line no-console
         this.setState({ message: err.message }) // show error snackbar
         return Promise.reject(err)
       })
   }
 
-  render() {
+  render () {
     return (
-      <div className="flex-column-center">
+      <div className='flex-column-center'>
         <Paper style={{ marginTop: '3rem' }}>
           <EmailForm onSubmit={this.sendRecoveryEmail} />
         </Paper>
@@ -68,7 +62,7 @@ export default class RecoverContainer extends Component {
           open={!!this.state.message}
           message={this.state.message || 'Error'}
           autoHideDuration={4000}
-          onRequestClose={() => this.setState({ message: null})}
+          onRequestClose={() => this.setState({ message: null })}
         />
       </div>
     )

examples/complete/material/src/routes/Recover/index.js

@@ -1,7 +1,7 @@
 import { RECOVER_PATH as path } from 'constants'
 
 export default (store) => ({
-  path: 'recover',
+  path: path,
   /*  Async getComponent is only invoked when route matches   */
   getComponent (nextState, cb) {
     /*  Webpack - use 'require.ensure' to create a split point