add es6lint to build process and fix most (not really) errors and warnings

.eslintrc

@@ -9,17 +9,21 @@
         "eol-last": [0],
         "no-mixed-requires": [0],
         "no-underscore-dangle": [0],
-        "react/display-name": 1,
+        "global-strict": [2, "always"],
+        "no-trailing-spaces": [2, { skipBlankLines: true }],
+        "no-console": 0,
+        "camelcase": [2, {"properties": "never"}],
+        "react/display-name": 0,
         "react/jsx-boolean-value": 1,
         "react/jsx-no-undef": 1,
         "react/jsx-quotes": 1,
-        "react/jsx-sort-prop-types": 1,
+        "react/jsx-sort-prop-types": 0,
-        "react/jsx-sort-props": 1,
+        "react/jsx-sort-props": 0,
         "react/jsx-uses-react": 1,
         "react/jsx-uses-vars": 1,
         "react/no-did-mount-set-state": 1,
         "react/no-did-update-set-state": 1,
-        "react/no-multi-comp": 1,
+        "react/no-multi-comp": 0,
         "react/no-unknown-property": 1,
         "react/prop-types": 1,
         "react/react-in-jsx-scope": 1,
@@ -31,7 +35,20 @@
         "react"
     ],
     "ecmaFeatures": {
-        "jsx": true,
+        "jsx": 1,
-        "modules": true
+        "modules": 1,
+        "arrowFunctions",
+        "classes": 1,
+        "blockBindings": 1,
+        "defaultParams": 1,
+        "destructuring": 1,
+        "objectLiteralComputedProperties": 1,
+        "objectLiteralDuplicateProperties": 0,
+        "objectLiteralShorthandMethods": 1,
+        "objectLiteralShorthandProperties": 1,
+        "restParams": 1,
+        "spread": 1,
+        "superInFunctions": 1,
+        "templateStrings": 1
     }
 }

gulpfile.js

@@ -1,3 +1,5 @@
+'use strict';
+
 var gulp = require('gulp');
 var gulpif = require('gulp-if');
 var sourcemaps = require('gulp-sourcemaps');
@@ -22,7 +24,7 @@ gulp.task('build', function() {
     bundle(false);
 });
  
-gulp.task('serve', ['browser-sync', 'sass', 'sass:watch', 'copy', 'lint'], function() {
+gulp.task('serve', ['browser-sync', 'lint:watch', 'sass', 'sass:watch', 'copy'], function() {
     bundle(true);
 });
 
@@ -78,6 +80,10 @@ gulp.task('lint', function () {
         .pipe(eslint.failOnError());
 });
 
+gulp.task('lint:watch', function () {
+    gulp.watch('js/**/*.js', ['lint']);
+});
+
 function bundle(watch) {
     var bro;
  

js/actions/edition_actions.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import alt from '../alt';
 import EditionFetcher from '../fetchers/edition_fetcher';
 

js/actions/edition_list_actions.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import alt from '../alt';
 
 import EditionListFetcher from '../fetchers/edition_list_fetcher.js';
@@ -12,7 +14,7 @@ class EditionListActions {
     }
 
     fetchEditionList(pieceId, orderBy, orderAsc) {
-        if(!orderBy && typeof orderAsc == "undefined") {
+        if(!orderBy && typeof orderAsc == 'undefined') {
             orderBy = 'edition_number';
             orderAsc = true;
         }

js/actions/piece_actions.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import alt from '../alt';
 import PieceFetcher from '../fetchers/piece_fetcher';
 

js/actions/piece_list_actions.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import alt from '../alt';
 
 import PieceListFetcher from '../fetchers/piece_list_fetcher';
@@ -28,6 +30,6 @@ class PieceListActions {
             });
     }
 
-};
+}
 
 export default alt.createActions(PieceListActions);

js/actions/user_actions.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import alt from '../alt';
 import UserFetcher from '../fetchers/user_fetcher';
 
@@ -12,12 +14,12 @@ class UserActions {
     fetchCurrentUser() {
         UserFetcher.fetchOne()
             .then((res) => {
-                this.actions.updateCurrentUser(res['users'][0]);
+                this.actions.updateCurrentUser(res.users[0]);
             })
             .catch((err) => {
-                console.log(err);           
+                console.log(err);
             });
     }
-};
+}
 
 export default alt.createActions(UserActions);

js/alt.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import Alt from 'alt';
 
 export default new Alt();

js/app.js

@@ -6,16 +6,13 @@ import promise from 'es6-promise';
 
 promise.polyfill();
 
-import AscribeApp from './components/ascribe_app';
 import AppConstants from './constants/application_constants';
 import ApiUrls from './constants/api_urls';
 import routes from './routes';
-import alt from './alt';
 import fetch from './utils/fetch';
-import AlertDismissable from './components/ascribe_forms/alert';
 
-/* 
+/*
-    Taken from 
+    Taken from
     http://stackoverflow.com/questions/30613447/how-to-debug-reactjss-setstate?noredirect=1#comment49301874_30613447
 
     <remove this in production>

js/components/acl_button.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 
 import ConsignForm from './ascribe_forms/form_consign';
@@ -17,31 +19,33 @@ let AclButton = React.createClass({
     },
 
     actionProperties(){
-        if (this.props.action == 'consign'){
+        if (this.props.action === 'consign'){
             return {
-                title: "Consign artwork",
+                title: 'Consign artwork',
-                tooltip: "Have someone else sell the artwork",
+                tooltip: 'Have someone else sell the artwork',
                 form: <ConsignForm />
-            }
+            };
         }
-        else if (this.props.action == 'transfer') {
+        else if (this.props.action === 'transfer') {
             return {
-                title: "Transfer artwork",
+                title: 'Transfer artwork',
-                tooltip: "Transfer the ownership of the artwork",
+                tooltip: 'Transfer the ownership of the artwork',
                 form: <TransferForm />
-            }
+            };
         }
-        else if (this.props.action == 'loan'){
+        else if (this.props.action === 'loan'){
             return {
-                title: "Loan artwork",
+                title: 'Loan artwork',
-                tooltip: "Loan your artwork for a limited period of time",
+                tooltip: 'Loan your artwork for a limited period of time',
-                form: <LoanForm />}
+                form: <LoanForm />
+            };
         }
-        else if (this.props.action == 'share'){
+        else if (this.props.action === 'share'){
             return {
-                title: "Share artwork",
+                title: 'Share artwork',
-                tooltip: "Share the artwork",
+                tooltip: 'Share the artwork',
-                form: <ShareForm />}
+                form: <ShareForm />
+            };
         }
     },
     render() {

js/components/ascribe_accordion_list/accordion_list.js

@@ -1,5 +1,6 @@
+'use strict';
+
 import React from 'react';
-import ReactAddons from 'react/addons';
 
 
 let AccordionList = React.createClass({

js/components/ascribe_accordion_list/accordion_list_item.js

@@ -1,8 +1,8 @@
+'use strict';
+
 import React from 'react';
 import Router from 'react-router';
 
-import AccordionListItemTable from './accordion_list_item_table';
-
 import { getLangText } from '../../utils/lang_utils';
 
 let Link = Router.Link;
@@ -10,7 +10,8 @@ let Link = Router.Link;
 let AccordionListItem = React.createClass({
     propTypes: {
         className: React.PropTypes.string,
-        content: React.PropTypes.object
+        content: React.PropTypes.object,
+        children: React.PropTypes.object
     },
 
     render() {

js/components/ascribe_accordion_list/accordion_list_item_table.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 
 import Table from '../ascribe_table/table';
@@ -5,8 +7,6 @@ import TableItem from '../ascribe_table/table_item';
 
 import { ColumnModel } from '../ascribe_table/models/table_models';
 
-import { getLangText } from '../../utils/lang_utils';
-
 let AccordionListItemTable = React.createClass({
     propTypes: {
         className: React.PropTypes.string,
@@ -26,7 +26,7 @@ let AccordionListItemTable = React.createClass({
                 <div className={this.props.className}>
                     <Table
                         className="ascribe-table"
-                        columnList={this.props.columnList} 
+                        columnList={this.props.columnList}
                         itemList={this.props.itemList}
                         changeOrder={this.props.changeOrder}
                         orderBy={this.props.orderBy}
@@ -35,8 +35,7 @@ let AccordionListItemTable = React.createClass({
                             return (
                                  <TableItem
                                     className="ascribe-table-item-selectable"
-                                    key={i}>
+                                    key={i} />
-                                </TableItem>
                             );
                         })}
                     </Table>

js/components/ascribe_accordion_list/accordion_list_item_table_editions.js

@@ -1,9 +1,10 @@
+'use strict';
+
 import React from 'react';
 import Router from 'react-router';
 
 import EditionListStore from '../../stores/edition_list_store';
 import EditionListActions from '../../actions/edition_list_actions';
-import PieceListStore from '../../stores/piece_list_store';
 import PieceListActions from '../../actions/piece_list_actions';
 
 import AccordionListItemTable from './accordion_list_item_table';
@@ -12,7 +13,6 @@ import AccordionListItemTableSelectAllEditionsToggle from './accordion_list_item
 
 import { ColumnModel, TransitionModel } from '../ascribe_table/models/table_models';
 
-import TableItemImg from '../ascribe_table/table_item_img';
 import TableItemText from '../ascribe_table/table_item_text';
 import TableItemCheckbox from '../ascribe_table/table_item_checkbox';
 import TableItemAclFiltered from '../ascribe_table/table_item_acl_filtered';
@@ -34,10 +34,6 @@ let AccordionListItemTableEditions = React.createClass({
         return EditionListStore.getState();
     },
 
-    onChange(state) {
-        this.setState(state);
-    },
-
     componentDidMount() {
         EditionListStore.listen(this.onChange);
     },
@@ -46,6 +42,10 @@ let AccordionListItemTableEditions = React.createClass({
         EditionListStore.unlisten(this.onChange);
     },
 
+    onChange(state) {
+        this.setState(state);
+    },
+
     selectItem(pieceId, editionId) {
         EditionListActions.selectEdition({pieceId, editionId});
     },
@@ -84,7 +84,7 @@ let AccordionListItemTableEditions = React.createClass({
         if(this.state.editionList[this.props.parentId]) {
             selectedEditionsCount = this.filterSelectedEditions().length;
             allEditionsCount = this.state.editionList[this.props.parentId].length;
-            orderBy = this.state.editionList[this.props.parentId].orderBy
+            orderBy = this.state.editionList[this.props.parentId].orderBy;
             orderAsc = this.state.editionList[this.props.parentId].orderAsc;
         }
 
@@ -98,7 +98,7 @@ let AccordionListItemTableEditions = React.createClass({
                         'pieceId': this.props.parentId,
                         'selectItem': this.selectItem,
                         'selected': item.selected
-                    }},
+                    }; },
                     '',
                     <AccordionListItemTableSelectAllEditionsToggle
                         onChange={this.selectAllItems}
@@ -109,10 +109,10 @@ let AccordionListItemTableEditions = React.createClass({
                     false
             ),
             new ColumnModel(
-                (item) => { 
+                (item) => {
                     return {
                         'content': item.edition_number
-                    }},
+                    }; },
                     'edition_number',
                     '#',
                     TableItemText,
@@ -124,7 +124,7 @@ let AccordionListItemTableEditions = React.createClass({
                 (item) => {
                     return {
                         'content': item.bitcoin_id
-                    }},
+                    }; },
                     'bitcoin_id',
                     getLangText('Bitcoin Address'),
                     TableItemText,
@@ -133,10 +133,10 @@ let AccordionListItemTableEditions = React.createClass({
                     transition
             ),
             new ColumnModel(
-                (item) => { 
+                (item) => {
                     return {
                         'content': item.acl
-                    }},
+                    }; },
                     'acl',
                     getLangText('Actions'),
                     TableItemAclFiltered,
@@ -159,7 +159,7 @@ let AccordionListItemTableEditions = React.createClass({
                     orderAsc={orderAsc}
                     changeOrder={this.changeEditionListOrder}>
                     <AccordionListItemTableToggle
-                        className="ascribe-accordion-list-table-toggle" 
+                        className="ascribe-accordion-list-table-toggle"
                         onClick={this.toggleTable}
                         show={this.props.show}
                         numOfTableItems={this.props.numOfEditions} />

js/components/ascribe_accordion_list/accordion_list_item_table_select_all_editions_toggle.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 
 
@@ -11,8 +13,8 @@ let AccordionListItemTableSelectAllEditionsToggle = React.createClass({
 
     render() {
         return (
-            <input type="checkbox" 
+            <input type="checkbox"
-                onChange={this.props.onChange} 
+                onChange={this.props.onChange}
                 checked={this.props.numOfAllEditions === this.props.numOfSelectedEditions} />
         );
     }

js/components/ascribe_accordion_list/accordion_list_item_table_toggle.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 
 let AccordionListItemTableToggle = React.createClass({
@@ -10,7 +12,7 @@ let AccordionListItemTableToggle = React.createClass({
 
     render() {
         return (
-            <span 
+            <span
                 className={this.props.className}
                 onClick={this.props.onClick}>
                 {this.props.show ? 'Hide all ' + this.props.numOfTableItems + ' Editions' : 'Show all ' + this.props.numOfTableItems + ' Editions'}

js/components/ascribe_app.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 import Router from 'react-router';
 import Header from '../components/header';
@@ -12,7 +14,7 @@ let AscribeApp = React.createClass({
             <div>
                 <Header />
                 <RouteHandler />
-            </div> 
+            </div>
         );
     }
 });

js/components/ascribe_forms/alert.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 import Alert from 'react-bootstrap/lib/Alert';
 
@@ -7,12 +9,15 @@ let AlertDismissable = React.createClass({
             alertVisible: true
         };
     },
+    
     show() {
         this.setState({alertVisible: true});
     },
+
     hide() {
         this.setState({alertVisible: false});
     },
+
     render() {
         if (this.state.alertVisible) {
             let key = this.props.error;

js/components/ascribe_forms/button_submit_close.js

@@ -1,5 +1,13 @@
+'use strict';
+
 import React from 'react';
 
+
+/*
+    Is this even used somewhere?
+    Deprecate? 5.6.15 - Tim
+
+ */
 let ButtonSubmitOrClose = React.createClass({
     render() {
         if (this.props.submitted){
@@ -14,7 +22,7 @@ let ButtonSubmitOrClose = React.createClass({
                 <button type="submit" className="btn btn-ascribe-inv">{this.props.text}</button>
                 <button className="btn btn-ascribe-inv" onClick={this.props.onClose}>CLOSE</button>
             </div>
-        )
+        );
     }
 });
 

js/components/ascribe_forms/form_consign.js

@@ -1,4 +1,4 @@
-import fetch from 'isomorphic-fetch';
+'use strict';
 
 import React from 'react';
 
@@ -12,19 +12,20 @@ let ConsignForm = React.createClass({
     mixins: [FormMixin],
 
     url() {
-        return ApiUrls.ownership_consigns
+        return ApiUrls.ownership_consigns;
     },
+    
     getFormData() {
         return {
             bitcoin_id: this.getBitcoinIds().join(),
             consignee: this.refs.consignee.state.value,
             consign_message: this.refs.consign_message.state.value,
             password: this.refs.password.state.value
-        }
+        };
     },
 
     renderForm() {
-        let title = this.getTitlesString().join("");
+        let title = this.getTitlesString().join('');
         let username = this.props.currentUser.username;
         let message =
 `Hi,

js/components/ascribe_forms/form_loan.js

@@ -1,4 +1,4 @@
-import fetch from 'isomorphic-fetch';
+'use strict';
 
 import React from 'react';
 
@@ -9,39 +9,47 @@ import InputHidden from './input_hidden';
 import InputCheckbox from './input_checkbox';
 import InputDate from './input_date';
 import InputTextArea from './input_textarea';
-import OwnershipFetcher from '../../fetchers/ownership_fetcher'
+import OwnershipFetcher from '../../fetchers/ownership_fetcher';
 import ButtonSubmitOrClose from './button_submit_close';
 
 let LoanForm = React.createClass({
+    
+    getInitialState() {
+        this.setState({
+            contract_key: null,
+            contract_url: null,
+            loaneeHasContract: false
+        });
+    },
+
     mixins: [FormMixin],
 
     url() {
-        return ApiUrls.ownership_loans
+        return ApiUrls.ownership_loans;
-    },
-    componentDidMount(){
-        this.setState({contract_key: null,
-                       contract_url: null,
-                       loaneeHasContract: false});
     },
+
     getFormData() {
         return {
             bitcoin_id: this.getBitcoinIds().join(),
             loanee: this.refs.loanee.state.value,
             gallery_name: this.refs.gallery_name.state.value,
-            startdate: this.refs.startdate.state.value.format("YYYY-MM-DD"),
+            startdate: this.refs.startdate.state.value.format('YYYY-MM-DD'),
-            enddate: this.refs.enddate.state.value.format("YYYY-MM-DD"),
+            enddate: this.refs.enddate.state.value.format('YYYY-MM-DD'),
             loan_message: this.refs.loan_message.state.value,
             password: this.refs.password.state.value,
             terms: this.refs.terms.state.value
-        }
+        };
     },
-    handleLoanEmailBlur(e){
+
+    handleLoanEmailBlur(){
         OwnershipFetcher.fetchLoanContract(this.refs.loanee.state.value)
             .then((res) => {
                 if (res && res.length > 0) {
-                    this.setState({contract_key: res[0].s3Key,
+                    this.setState({
-                                   contract_url: res[0].s3Url,
+                        contract_key: res[0].s3Key,
-                                   loaneeHasContract: true});
+                        contract_url: res[0].s3Url,
+                        loaneeHasContract: true
+                    });
                 }
                 else{
                     this.resetLoanContract();
@@ -52,14 +60,16 @@ let LoanForm = React.createClass({
                 this.resetLoanContract();
             });
     },
+
     resetLoanContract(){
         this.setState({contract_key: null,
                         contract_url: null,
                         loaneeHasContract: false
                        });
     },
+    
     renderForm() {
-        let title = this.getTitlesString().join("");
+        let title = this.getTitlesString().join('');
         let username = this.props.currentUser.username;
         let message =
 `Hi,
@@ -72,18 +82,19 @@ ${username}`;
 
         let contract = <InputHidden ref="terms" value="True"/>;
         if (this.state.loaneeHasContract){
-            let label = <div>
+            let label = (<div>
                             I agree to the&nbsp;
                             <a href={this.state.contract_url} target="_blank">
                                 terms of {this.refs.loanee.state.value}
                             </a>
-                        </div>;
+                        </div>);
-            contract = <InputCheckbox
+            contract = (<InputCheckbox
                             ref="terms"
                             required="required"
                             label={label}
-                        />
+                        />);
         }
+
         return (
             <form id="loan_modal_content" role="form" onSubmit={this.submit}>
                 <input className="invisible" type="email" name="fake_loanee"/>

js/components/ascribe_forms/form_share_email.js

@@ -1,4 +1,4 @@
-import fetch from 'isomorphic-fetch';
+'use strict';
 
 import React from 'react';
 
@@ -12,17 +12,19 @@ let ShareForm = React.createClass({
     mixins: [FormMixin],
 
     url() {
-        return ApiUrls.ownership_shares_mail
+        return ApiUrls.ownership_shares_mail;
     },
+
     getFormData() {
         return {
             bitcoin_id: this.getBitcoinIds().join(),
             share_emails: this.refs.share_emails.state.value,
             share_message: this.refs.share_message.state.value
-        }
+        };
     },
+
     renderForm() {
-        let title = this.getTitlesString().join("");
+        let title = this.getTitlesString().join('');
         let username = this.props.currentUser.username;
         let message =
 `Hi,

js/components/ascribe_forms/form_transfer.js

@@ -1,4 +1,4 @@
-import fetch from 'isomorphic-fetch';
+'use strict';
 
 import React from 'react';
 
@@ -9,23 +9,24 @@ import InputTextArea from './input_textarea';
 import ButtonSubmitOrClose from './button_submit_close';
 
 
-
 let TransferForm = React.createClass({
     mixins: [FormMixin],
 
     url() {
-        return ApiUrls.ownership_transfers
+        return ApiUrls.ownership_transfers;
     },
+    
     getFormData() {
         return {
             bitcoin_id: this.getBitcoinIds().join(),
             transferee: this.refs.transferee.state.value,
             transfer_message: this.refs.transfer_message.state.value,
             password: this.refs.password.state.value
-        }
+        };
     },
+
     renderForm() {
-        let title = this.getTitlesString().join("");
+        let title = this.getTitlesString().join('');
         let username = this.props.currentUser.username;
         let message =
 `Hi,

js/components/ascribe_forms/form_unconsign.js

@@ -1,4 +1,4 @@
-import fetch from 'isomorphic-fetch';
+'use strict';
 
 import React from 'react';
 
@@ -12,15 +12,17 @@ let UnConsignForm = React.createClass({
     mixins: [FormMixin],
 
     url() {
-        return ApiUrls.ownership_unconsigns
+        return ApiUrls.ownership_unconsigns;
     },
+
     getFormData() {
         return {
             bitcoin_id: this.props.edition.bitcoin_id,
             unconsign_message: this.refs.unconsign_message.state.value,
             password: this.refs.password.state.value
-        }
+        };
     },
+
     renderForm() {
         let title = this.props.edition.title;
         let username = this.props.currentUser.username;

js/components/ascribe_forms/form_unconsign_request.js

@@ -1,10 +1,9 @@
-import fetch from 'isomorphic-fetch';
+'use strict';
 
 import React from 'react';
 
 import ApiUrls from '../../constants/api_urls';
 import FormMixin from '../../mixins/form_mixin';
-import InputText from './input_text';
 import InputTextArea from './input_textarea';
 import ButtonSubmitOrClose from './button_submit_close';
 
@@ -12,14 +11,16 @@ let UnConsignRequestForm = React.createClass({
     mixins: [FormMixin],
 
     url() {
-        return ApiUrls.ownership_unconsigns_request
+        return ApiUrls.ownership_unconsigns_request;
     },
+
     getFormData() {
         return {
             bitcoin_id: this.props.edition.bitcoin_id,
             unconsign_request_message: this.refs.unconsign_request_message.state.value
-        }
+        };
     },
+
     renderForm() {
         let title = this.props.edition.title;
         let username = this.props.currentUser.username;

js/components/ascribe_forms/input_checkbox.js

@@ -1,19 +1,24 @@
+'use strict';
+
 import React from 'react';
 
-import AlertMixin from '../../mixins/alert_mixin'
+import AlertMixin from '../../mixins/alert_mixin';
 
 let InputCheckbox = React.createClass({
 
-    mixins : [AlertMixin],
+    mixins: [AlertMixin],
 
     getInitialState() {
-        return {value: null,
+        return {
-                alerts: null // needed in AlertMixin
+            value: null,
+            alerts: null // needed in AlertMixin
         };
     },
+    
     handleChange(event) {
         this.setState({value: event.target.value});
     },
+
     render() {
         let alerts = (this.props.submitted) ? null : this.state.alerts;
         return (

js/components/ascribe_forms/input_date.js

@@ -1,15 +1,18 @@
+'use strict';
+
 import React from 'react';
 
-import AlertMixin from '../../mixins/alert_mixin'
+import AlertMixin from '../../mixins/alert_mixin';
-import DatePicker from 'react-datepicker/dist/react-datepicker'
+import DatePicker from 'react-datepicker/dist/react-datepicker';
 
 let InputDate = React.createClass({
 
-    mixins : [AlertMixin],
+    mixins: [AlertMixin],
 
     getInitialState() {
-        return {value: null,
+        return {
-                alerts: null // needed in AlertMixin
+            value: null,
+            alerts: null // needed in AlertMixin
         };
     },
 
@@ -18,7 +21,7 @@ let InputDate = React.createClass({
     },
 
     render: function () {
-        let className = "form-control input-text-ascribe";
+        let className = 'form-control input-text-ascribe';
         let alerts = (this.props.submitted) ? null : this.state.alerts;
         return (
              <DatePicker
@@ -29,6 +32,10 @@ let InputDate = React.createClass({
                  placeholderText={this.props.placeholderText}
               />
         );
+        // CAN THIS BE REMOVED???
+        //
+        //  - Tim?
+        //
         //return (
         //    <div className="input-group date"
         //        ref={this.props.name + "_picker"}

js/components/ascribe_forms/input_hidden.js

@@ -1,10 +1,12 @@
+'use strict';
+
 import React from 'react';
 
-import AlertMixin from '../../mixins/alert_mixin'
+import AlertMixin from '../../mixins/alert_mixin';
 
 let InputHidden = React.createClass({
 
-    mixins : [AlertMixin],
+    mixins: [AlertMixin],
 
     getInitialState() {
         return {value: this.props.value,

js/components/ascribe_forms/input_text.js

@@ -1,21 +1,25 @@
+'use strict';
+
 import React from 'react';
 
-import AlertMixin from '../../mixins/alert_mixin'
+import AlertMixin from '../../mixins/alert_mixin';
 
 let InputText = React.createClass({
 
-    mixins : [AlertMixin],
+    mixins: [AlertMixin],
 
     getInitialState() {
         return {value: null,
                 alerts: null // needed in AlertMixin
         };
     },
+
     handleChange(event) {
         this.setState({value: event.target.value});
     },
+
     render() {
-        let className = "form-control input-text-ascribe";
+        let className = 'form-control input-text-ascribe';
         let alerts = (this.props.submitted) ? null : this.state.alerts;
         return (
             <div className="form-group">

js/components/ascribe_forms/input_textarea.js

@@ -1,21 +1,24 @@
+'use strict';
+
 import React from 'react';
 
-import AlertMixin from '../../mixins/alert_mixin'
+import AlertMixin from '../../mixins/alert_mixin';
 
 let InputTextArea = React.createClass({
 
-    mixins : [AlertMixin],
+    mixins: [AlertMixin],
 
     getInitialState() {
-        return {value: this.props.defaultValue,
+        return {
-                alerts: null // needed in AlertMixin
+            value: this.props.defaultValue,
+            alerts: null // needed in AlertMixin
         };
     },
     handleChange(event) {
         this.setState({value: event.target.value});
     },
     render() {
-        let className = "form-control input-text-ascribe textarea-ascribe-message";
+        let className = 'form-control input-text-ascribe textarea-ascribe-message';
 
         let alerts = (this.props.submitted) ? null : this.state.alerts;
         return (

js/components/ascribe_media/resource_viewer.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 import InjectInHeadMixin from '../../mixins/inject_in_head_mixin';
 
@@ -14,7 +16,7 @@ import InjectInHeadMixin from '../../mixins/inject_in_head_mixin';
 
 let resourceMap = {
     'image': 1
-}
+};
 
 let ResourceViewer = React.createClass({
     propTypes: {

js/components/ascribe_modal/modal_loan.js

@@ -1,11 +1,13 @@
+'use strict';
+
 import React from 'react';
 import Modal from 'react-bootstrap/lib/Modal';
 import OverlayTrigger from 'react-bootstrap/lib/OverlayTrigger';
 import ModalTrigger from 'react-bootstrap/lib/ModalTrigger';
 import Tooltip from 'react-bootstrap/lib/Tooltip';
 
-import LoanForm from '../ascribe_forms/form_loan'
+import LoanForm from '../ascribe_forms/form_loan';
-import ModalMixin from '../../mixins/modal_mixin'
+import ModalMixin from '../../mixins/modal_mixin';
 
 let LoanModalButton = React.createClass({
     render() {
@@ -19,12 +21,12 @@ let LoanModalButton = React.createClass({
                     </div>
                 </ModalTrigger>
             </OverlayTrigger>
-        )
+        );
     }
 });
 
 let LoanModal = React.createClass({
-    mixins : [ModalMixin],
+    mixins: [ModalMixin],
 
     render() {
         return (
@@ -35,7 +37,7 @@ let LoanModal = React.createClass({
                               onRequestHide={this.onRequestHide}/>
                 </div>
             </Modal>
-        )
+        );
     }
 });
 

js/components/ascribe_modal/modal_share.js

@@ -1,13 +1,13 @@
+'use strict';
+
 import React from 'react';
 import Modal from 'react-bootstrap/lib/Modal';
 import OverlayTrigger from 'react-bootstrap/lib/OverlayTrigger';
 import ModalTrigger from 'react-bootstrap/lib/ModalTrigger';
 import Tooltip from 'react-bootstrap/lib/Tooltip';
 
-import ModalMixin from '../../mixins/modal_mixin'
+import ModalMixin from '../../mixins/modal_mixin';
-
+import ShareForm from '../ascribe_forms/form_share_email';
-
-import ShareForm from '../ascribe_forms/form_share_email'
 
 let ShareModalButton = React.createClass({
     render() {
@@ -20,12 +20,12 @@ let ShareModalButton = React.createClass({
                     </div>
                 </ModalTrigger>
             </OverlayTrigger>
-        )
+        );
     }
 });
 
 let ShareModal = React.createClass({
-    mixins : [ModalMixin],
+    mixins: [ModalMixin],
 
     render() {
         return (
@@ -36,7 +36,7 @@ let ShareModal = React.createClass({
                                 onRequestHide={this.onRequestHide}/>
                 </div>
             </Modal>
-        )
+        );
     }
 });
 

js/components/ascribe_modal/modal_unconsign.js

@@ -1,11 +1,13 @@
+'use strict';
+
 import React from 'react';
 import Modal from 'react-bootstrap/lib/Modal';
 import OverlayTrigger from 'react-bootstrap/lib/OverlayTrigger';
 import ModalTrigger from 'react-bootstrap/lib/ModalTrigger';
 import Tooltip from 'react-bootstrap/lib/Tooltip';
 
-import UnConsignForm from '../ascribe_forms/form_unconsign'
+import UnConsignForm from '../ascribe_forms/form_unconsign';
-import ModalMixin from '../../mixins/modal_mixin'
+import ModalMixin from '../../mixins/modal_mixin';
 
 let UnConsignModalButton = React.createClass({
     render() {
@@ -19,12 +21,12 @@ let UnConsignModalButton = React.createClass({
                     </div>
                 </ModalTrigger>
             </OverlayTrigger>
-        )
+        );
     }
 });
 
 let UnConsignModal = React.createClass({
-    mixins : [ModalMixin],
+    mixins: [ModalMixin],
 
     render() {
         return (
@@ -35,7 +37,7 @@ let UnConsignModal = React.createClass({
                                  onRequestHide={this.onRequestHide}/>
                 </div>
             </Modal>
-        )
+        );
     }
 });
 

js/components/ascribe_modal/modal_unconsign_request.js

@@ -1,11 +1,13 @@
+'use strict';
+
 import React from 'react';
 import Modal from 'react-bootstrap/lib/Modal';
 import OverlayTrigger from 'react-bootstrap/lib/OverlayTrigger';
 import ModalTrigger from 'react-bootstrap/lib/ModalTrigger';
 import Tooltip from 'react-bootstrap/lib/Tooltip';
 
-import UnConsignRequestForm from '../ascribe_forms/form_unconsign_request'
+import UnConsignRequestForm from '../ascribe_forms/form_unconsign_request';
-import ModalMixin from '../../mixins/modal_mixin'
+import ModalMixin from '../../mixins/modal_mixin';
 
 let UnConsignRequestModalButton = React.createClass({
     render() {
@@ -19,12 +21,12 @@ let UnConsignRequestModalButton = React.createClass({
                     </div>
                 </ModalTrigger>
             </OverlayTrigger>
-        )
+        );
     }
 });
 
 let UnConsignRequestModal = React.createClass({
-    mixins : [ModalMixin],
+    mixins: [ModalMixin],
 
     render() {
         return (
@@ -35,7 +37,7 @@ let UnConsignRequestModal = React.createClass({
                                  onRequestHide={this.onRequestHide}/>
                 </div>
             </Modal>
-        )
+        );
     }
 });
 

js/components/ascribe_modal/modal_wrapper.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 import ReactAddons from 'react/addons';
 
@@ -6,7 +8,7 @@ import OverlayTrigger from 'react-bootstrap/lib/OverlayTrigger';
 import ModalTrigger from 'react-bootstrap/lib/ModalTrigger';
 import Tooltip from 'react-bootstrap/lib/Tooltip';
 
-import ModalMixin from '../../mixins/modal_mixin'
+import ModalMixin from '../../mixins/modal_mixin';
 
 let ModalWrapper = React.createClass({
 
@@ -27,20 +29,22 @@ let ModalWrapper = React.createClass({
                 {this.props.button}
                 </ModalTrigger>
             </OverlayTrigger>
-        )
+        );
     }
 });
 
 //
 let ModalBody = React.createClass({
-    mixins : [ModalMixin],
+
+    mixins: [ModalMixin],
 
     handleSuccess(){
         this.props.handleSuccess();
         this.props.onRequestHide();
     },
+
     renderChildren() {
-        return ReactAddons.Children.map(this.props.children, (child, i) => {
+        return ReactAddons.Children.map(this.props.children, (child) => {
             return ReactAddons.addons.cloneWithProps(child, {
                 editions: this.props.editions,
                 currentUser: this.props.currentUser,
@@ -49,6 +53,7 @@ let ModalBody = React.createClass({
             });
         });
     },
+
     render() {
         return (
             <Modal {...this.props} title={this.props.title}>
@@ -56,7 +61,7 @@ let ModalBody = React.createClass({
                 {this.renderChildren()}
                 </div>
             </Modal>
-        )
+        );
     }
 });
 

js/components/ascribe_pagination/pagination.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 
 import PaginationButton from './pagination_button';
@@ -7,25 +9,23 @@ let Pagination = React.createClass({
         goToPage: React.PropTypes.func.isRequired,
         currentPage: React.PropTypes.number.isRequired,
         totalPages: React.PropTypes.number.isRequired
-        //itemListCount: React.PropTypes.number.isRequired 
+        //itemListCount: React.PropTypes.number.isRequired
     },
 
     render() {
-        return(
+        return (
             <nav>
                 <ul className="pager">
-                    <PaginationButton 
+                    <PaginationButton
                         direction='previous'
                         goToPage={this.props.goToPage}
                         currentPage={this.props.currentPage}
-                        totalPages={this.props.totalPages}>
+                        totalPages={this.props.totalPages} />
-                    </PaginationButton>
+                    <PaginationButton
-                    <PaginationButton 
                         direction='next'
                         goToPage={this.props.goToPage}
                         currentPage={this.props.currentPage}
-                        totalPages={this.props.totalPages}>
+                        totalPages={this.props.totalPages} />
-                    </PaginationButton>
                 </ul>
             </nav>
         );

js/components/ascribe_pagination/pagination_button.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 import Router from 'react-router';
 

js/components/ascribe_piece_list_bulk_modal/piece_list_bulk_modal.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 
 import { mergeOptions } from '../../utils/general_utils';
@@ -108,7 +110,7 @@ let PieceListBulkModal = React.createClass({
                                     <PieceListBulkModalSelectedEditionsWidget
                                         numberOfSelectedEditions={this.fetchSelectedEditionList().length} />
                                         &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
-                                    <span 
+                                    <span
                                         className="piece-list-bulk-modal-clear-all"
                                         onClick={this.clearAllSelections}>clear all</span>
                                 </div>
@@ -148,7 +150,7 @@ let PieceListBulkModal = React.createClass({
             );
         } else {
             return null;
-        } 
+        }
         
     }
 });

js/components/ascribe_piece_list_bulk_modal/piece_list_bulk_modal_selected_editions_widget.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 
 let PieceListBulkModalSelectedEditionsWidget = React.createClass({

js/components/ascribe_piece_list_toolbar/piece_list_toolbar.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 
 import PieceListStore from '../../stores/piece_list_store';
@@ -6,18 +8,16 @@ import PieceListActions from '../../actions/piece_list_actions';
 import Input from 'react-bootstrap/lib/Input';
 import Glyphicon from 'react-bootstrap/lib/Glyphicon';
 
-import PieceListToolbarFilterWidgetFilter from './piece_list_toolbar_filter_widget';
-
 let PieceListToolbar = React.createClass({
 
+    propTypes: {
+        className: React.PropTypes.string
+    },
+
     getInitialState() {
         return PieceListStore.getState();
     },
 
-    onChange(state) {
-        this.setState(state);
-    },
-
     componentDidMount() {
         PieceListStore.listen(this.onChange);
     },
@@ -26,6 +26,10 @@ let PieceListToolbar = React.createClass({
         PieceListStore.unlisten(this.onChange);
     },
 
+    onChange(state) {
+        this.setState(state);
+    },
+
     searchFor() {
          let searchTerm = this.refs.search.getInputDOMNode().value;
          PieceListActions.fetchPieceList(this.state.page, this.pageSize, searchTerm, this.state.orderBy, this.state.orderAsc);

js/components/ascribe_piece_list_toolbar/piece_list_toolbar_filter_widget.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 
 import Glyphicon from 'react-bootstrap/lib/Glyphicon';

js/components/ascribe_table/models/table_models.js

@@ -1,3 +1,5 @@
+'use strict';
+
 export class ColumnModel {
     // ToDo: Add validation for all passed-in parameters
     constructor(transformFn, columnName, displayName, displayType, rowWidth, canBeOrdered, transition) {
@@ -12,7 +14,7 @@ export class ColumnModel {
 }
 
 /**
- * If a user opens an editionList of a piece and clicks on a specific edition to go to the 
+ * If a user opens an editionList of a piece and clicks on a specific edition to go to the
  * piece detail page, all previously opened editionLists are still saved as show = true in the
  * pieceList store.
  *

js/components/ascribe_table/table.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 import ReactAddons from 'react/addons';
 
@@ -12,7 +14,9 @@ let Table = React.createClass({
         changeOrder: React.PropTypes.func,
         orderBy: React.PropTypes.string,
         orderAsc: React.PropTypes.bool,
-        className: React.PropTypes.string
+        className: React.PropTypes.string,
+        children: React.PropTypes.object,
+        itemList: React.PropTypes.array
     },
 
     renderChildren() {
@@ -29,7 +33,7 @@ let Table = React.createClass({
     render() {
         return (
             <table className={'table ' + this.props.className}>
-                <TableHeader 
+                <TableHeader
                     columnList={this.props.columnList}
                     itemList={this.props.itemList}
                     changeOrder={this.props.changeOrder}

js/components/ascribe_table/table_header.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 
 import TableColumnMixin from '../../mixins/table_column_mixin';
@@ -7,7 +9,6 @@ import { ColumnModel } from './models/table_models';
 
 
 let TableHeader = React.createClass({
-    mixins: [TableColumnMixin],
     propTypes: {
         columnList: React.PropTypes.arrayOf(React.PropTypes.instanceOf(ColumnModel)),
         itemList: React.PropTypes.array.isRequired,
@@ -16,6 +17,8 @@ let TableHeader = React.createClass({
         orderBy: React.PropTypes.string
     },
 
+    mixins: [TableColumnMixin],
+
     render() {
         return (
             <thead>
@@ -29,20 +32,18 @@ let TableHeader = React.createClass({
                         return (
                             <TableHeaderItem
                                 key={i}
-                                columnClasses={columnClasses} 
+                                columnClasses={columnClasses}
-                                displayName={val.displayName} 
+                                displayName={val.displayName}
-                                columnName={columnName} 
+                                columnName={columnName}
-                                canBeOrdered={canBeOrdered} 
+                                canBeOrdered={canBeOrdered}
                                 orderAsc={this.props.orderAsc}
                                 orderBy={this.props.orderBy}
-                                changeOrder={this.props.changeOrder}>
+                                changeOrder={this.props.changeOrder} />
-                            </TableHeaderItem>
                         );
                     })}
                 </tr>
-            </thead> 
+            </thead>
         );
-
     }
 });
 

js/components/ascribe_table/table_header_item.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 
 import TableHeaderItemCarret from './table_header_item_carret';
@@ -25,7 +27,7 @@ let TableHeaderItem = React.createClass({
         if(this.props.canBeOrdered && this.props.changeOrder && this.props.orderAsc != null && this.props.orderBy) {
             if(this.props.columnName === this.props.orderBy) {
                 return (
-                    <th 
+                    <th
                         className={'ascribe-table-header-column'}
                         onClick={this.changeOrder}>
                         <span>{this.props.displayName} <TableHeaderItemCarret orderAsc={this.props.orderAsc} /></span>
@@ -33,7 +35,7 @@ let TableHeaderItem = React.createClass({
                 );
             } else {
                 return (
-                    <th 
+                    <th
                         className={'ascribe-table-header-column'}
                         onClick={this.changeOrder}>
                         <span>{this.props.displayName}</span>

js/components/ascribe_table/table_header_item_carret.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 
 let TableHeaderItemCarret = React.createClass({

js/components/ascribe_table/table_item.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 
 import { ColumnModel } from './models/table_models';
@@ -18,8 +20,7 @@ let TableItem = React.createClass({
             <TableItemWrapper
                 columnList={this.props.columnList}
                 columnContent={this.props.columnContent}
-                columnWidth={12}>
+                columnWidth={12} />
-            </TableItemWrapper>
         );
     }
 });

js/components/ascribe_table/table_item_acl.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 
 

js/components/ascribe_table/table_item_acl_filtered.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 
 

js/components/ascribe_table/table_item_checkbox.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 
 

js/components/ascribe_table/table_item_img.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 
 /**
@@ -5,7 +7,7 @@ import React from 'react';
  */
 let TableItemImg = React.createClass({
     propTypes: {
-        content: React.PropTypes.string.isRequired,
+        content: React.PropTypes.string.isRequired
     },
 
     render() {

js/components/ascribe_table/table_item_selectable.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 import classNames from 'classnames';
 
@@ -12,7 +14,8 @@ let TableItemSelectable = React.createClass({
         columnList: React.PropTypes.arrayOf(React.PropTypes.instanceOf(ColumnModel)),
         columnContent: React.PropTypes.object,
         parentId: React.PropTypes.number,
-        className: React.PropTypes.string
+        className: React.PropTypes.string,
+        selectItem: React.PropTypes.func
     },
 
     selectItem() {
@@ -25,12 +28,11 @@ let TableItemSelectable = React.createClass({
         });
 
         return (
-            <TableItem 
+            <TableItem
                 className={tableItemClasses + ' ' + this.props.className}
                 columnList={this.props.columnList}
                 columnContent={this.props.columnContent}
-                onClick={this.selectItem}>
+                onClick={this.selectItem} />
-            </TableItem>
         );
         
     }

js/components/ascribe_table/table_item_subtable.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 
 import { ColumnModel } from './models/table_models';
@@ -26,14 +28,18 @@ let TableItemSubtable = React.createClass({
         };
     },
 
-    onChange(state) {
-        this.setState(state);
-    },
-
     componentDidMount() {
         EditionListStore.listen(this.onChange);
     },
 
+    componentWillUnmount() {
+        EditionListStore.unlisten(this.onChange);
+    },
+
+    onChange(state) {
+        this.setState(state);
+    },
+
     loadEditionList() {
         if(this.state.open) {
             this.setState({
@@ -77,10 +83,9 @@ let TableItemSubtable = React.createClass({
                                         className="ascribe-table-item-selectable"
                                         selectItem={this.selectItem}
                                         parentId={this.props.columnContent.id}
-                                        key={i}>
+                                        key={i} />
-                                    </TableItemSelectable>
                                 );
-                            })} 
+                            })}
                         </Table>
                       </div>  
                     </div>
@@ -94,15 +99,13 @@ let TableItemSubtable = React.createClass({
                     <TableItemWrapper
                         columnList={this.props.columnList}
                         columnContent={this.props.columnContent}
-                        columnWidth={12}>
+                        columnWidth={12} />
-                    </TableItemWrapper>
                     <div className="col-xs-1 col-sm-1 col-md-1 col-lg-1 ascribe-table-item-column">
-                        <TableItemSubtableButton content="+" onClick={this.loadEditionList}>
+                        <TableItemSubtableButton content="+" onClick={this.loadEditionList} />
-                        </TableItemSubtableButton>
                     </div>
                 </div>
                 {renderEditionListTable()}
-            </div> 
+            </div>
         );
     }
 });

js/components/ascribe_table/table_item_subtable_button.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 
 let TableItemSubtableButton = React.createClass({

js/components/ascribe_table/table_item_text.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 
 

js/components/ascribe_table/table_item_wrapper.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 import Router from 'react-router';
 
@@ -7,13 +9,14 @@ import TableColumnMixin from '../../mixins/table_column_mixin';
 let Link = Router.Link;
 
 let TableItemWrapper = React.createClass({
-    mixins: [TableColumnMixin, Router.Navigation],
     propTypes: {
         columnList: React.PropTypes.arrayOf(React.PropTypes.instanceOf(ColumnModel)),
         columnContent: React.PropTypes.object,
         columnWidth: React.PropTypes.number.isRequired
     },
 
+    mixins: [TableColumnMixin, Router.Navigation],
+
     render() {
         return (
             <tr>
@@ -27,7 +30,7 @@ let TableItemWrapper = React.createClass({
                     if(!column.transition) {
                         return (
                             <td
-                                className={'ascribe-table-item-column'} 
+                                className={'ascribe-table-item-column'}
                                 key={i}>
                                 <TypeElement {...typeElementProps} />
                             </td>
@@ -42,8 +45,8 @@ let TableItemWrapper = React.createClass({
                          */
                         return (
                             <td key={i}>
-                                <Link 
+                                <Link
-                                    className={'ascribe-table-item-column'} 
+                                    className={'ascribe-table-item-column'}
                                     onClick={column.transition.callback}
                                     {...linkProps}>
                                     <TypeElement {...typeElementProps} />

js/components/edition.js

@@ -1,14 +1,21 @@
+'use strict';
+
 import React from 'react';
 
 import ResourceViewer from './ascribe_media/resource_viewer';
 
-import EditionActions from '../actions/edition_actions'
+import EditionActions from '../actions/edition_actions';
-import AclButton from './acl_button'
+import AclButton from './acl_button';
 
 /**
  * This is the component that implements display-specific functionality
  */
 let Edition = React.createClass({
+    propTypes: {
+        edition: React.PropTypes.object,
+        currentUser: React.PropTypes.object
+    },
+
     render() {
         let thumbnail = this.props.edition.thumbnail;
         let mimetype = this.props.edition.digital_work.mime;
@@ -31,13 +38,17 @@ let Edition = React.createClass({
 });
 
 let EditionHeader = React.createClass({
+    propTypes: {
+        edition: React.PropTypes.object
+    },
+
     render() {
-        var title_html = <div className="ascribe-detail-title">{this.props.edition.title}</div>;
+        var titleHtml = <div className="ascribe-detail-title">{this.props.edition.title}</div>;
         return (
             <div className="ascribe-detail-header">
-                <EditionDetailProperty label="title" value={title_html} />
+                <EditionDetailProperty label="title" value={titleHtml} />
                 <EditionDetailProperty label="by" value={this.props.edition.artist_name} />
-                <EditionDetailProperty label="date" value={ this.props.edition.date_created.slice(0,4) } />
+                <EditionDetailProperty label="date" value={ this.props.edition.date_created.slice(0, 4) } />
                 <hr/>
             </div>
         );
@@ -45,26 +56,32 @@ let EditionHeader = React.createClass({
 });
 
 let EditionDetails = React.createClass({
+    propTypes: {
+        edition: React.PropTypes.object,
+        currentUser: React.PropTypes.object
+    },
+
     handleSuccess(){
         EditionActions.fetchOne(this.props.edition.id);
     },
+
     render() {
         return (
             <div className="ascribe-detail-header">
                 <EditionDetailProperty label="edition"
-                    value={this.props.edition.edition_number + " of " + this.props.edition.num_editions} />
+                    value={this.props.edition.edition_number + ' of ' + this.props.edition.num_editions} />
                 <EditionDetailProperty label="id" value={ this.props.edition.bitcoin_id } />
                 <EditionDetailProperty label="owner" value={ this.props.edition.owner } />
                 <br/>
                 <AclButton
-                    availableAcls={["transfer"]}
+                    availableAcls={['transfer']}
                     action="transfer"
                     editions={[this.props.edition]}
                     currentUser={this.props.currentUser}
                     handleSuccess={this.handleSuccess}
                 />
                 <AclButton
-                    availableAcls={["consign"]}
+                    availableAcls={['consign']}
                     action="consign"
                     editions={[this.props.edition]}
                     currentUser={this.props.currentUser}
@@ -78,6 +95,11 @@ let EditionDetails = React.createClass({
 });
 
 let EditionDetailProperty = React.createClass({
+    propTypes: {
+        label: React.PropTypes.string,
+        value: React.PropTypes.string
+    },
+
     render() {
         return (
             <div className="row ascribe-detail-property">

js/components/edition_container.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 
 import { mergeOptions } from '../utils/general_utils';
@@ -28,6 +30,7 @@ let EditionContainer = React.createClass({
         UserActions.fetchCurrentUser();
         EditionActions.fetchOne(this.props.params.editionId);
     },
+
     componentWillUnmount() {
         EditionStore.unlisten(this.onChange);
         UserStore.unlisten(this.onChange);
@@ -36,10 +39,9 @@ let EditionContainer = React.createClass({
     render() {
         if('title' in this.state.edition) {
             return (
-                <Edition 
+                <Edition
-                    edition={this.state.edition} 
+                    edition={this.state.edition}
-                    currentUser={this.state.currentUser}>
+                    currentUser={this.state.currentUser} />
-                </Edition>
             );
         } else {
             return (

js/components/header.js

@@ -1,13 +1,13 @@
+'use strict';
+
 import React from 'react';
 import Router from 'react-router';
 
-import AltContainer from 'alt/AltContainer';
 import UserActions from '../actions/user_actions';
 import UserStore from '../stores/user_store';
 
 import Nav from 'react-bootstrap/lib/Nav';
 import Navbar from 'react-bootstrap/lib/Navbar';
-import NavItem from 'react-bootstrap/lib/NavItem';
 import DropdownButton from 'react-bootstrap/lib/DropdownButton';
 import MenuItem from 'react-bootstrap/lib/MenuItem';
 
@@ -22,7 +22,7 @@ let Header = React.createClass({
     },
 
     componentDidMount() {
-        UserStore.listen(this.onChange)
+        UserStore.listen(this.onChange);
         UserActions.fetchCurrentUser();
     },
 

js/components/piece_list.js

@@ -1,5 +1,6 @@
+'use strict';
+
 import React from 'react';
-import AltContainer from 'alt/AltContainer';
 
 import PieceListStore from '../stores/piece_list_store';
 import PieceListActions from '../actions/piece_list_actions';
@@ -15,7 +16,10 @@ import PieceListToolbar from './ascribe_piece_list_toolbar/piece_list_toolbar';
 
 
 let PieceList = React.createClass({
-    
+    propTypes: {
+        query: React.PropTypes.object
+    },
+
     getInitialState() {
         return PieceListStore.getState();
     },
@@ -35,7 +39,7 @@ let PieceList = React.createClass({
     },
 
     paginationGoToPage(page) {
-        return (e) => PieceListActions.fetchPieceList(page, this.state.pageSize,
+        return () => PieceListActions.fetchPieceList(page, this.state.pageSize,
                                                       this.state.search, this.state.orderBy,
                                                       this.state.orderAsc);
     },
@@ -47,8 +51,8 @@ let PieceList = React.createClass({
 
     render() {
         let currentPage = parseInt(this.props.query.page, 10) || 1;
-        let totalPages = Math.ceil(this.state.pieceListCount / this.state.pageSize)
+        let totalPages = Math.ceil(this.state.pieceListCount / this.state.pageSize);
-        
+
         return (
             <div>
                 <PieceListToolbar className="ascribe-piece-list-toolbar" />
@@ -64,11 +68,11 @@ let PieceList = React.createClass({
                     pageSize={this.state.pageSize}>
                     {this.state.pieceList.map((item, i) => {
                         return (
-                            <AccordionListItem 
+                            <AccordionListItem
                                 className="col-xs-12 col-sm-10 col-md-8 col-lg-8 col-sm-offset-1 col-md-offset-2 col-lg-offset-2 ascribe-accordion-list-item"
                                 content={item}
                                 key={i}>
-                                <AccordionListItemTableEditions 
+                                <AccordionListItemTableEditions
                                     className="ascribe-accordion-list-item-table col-xs-12 col-sm-8 col-md-6 col-lg-6 col-sm-offset-2 col-md-offset-3 col-lg-offset-3"
                                     parentId={item.id}
                                     show={item.show}
@@ -80,8 +84,7 @@ let PieceList = React.createClass({
                 <Pagination
                     currentPage={currentPage}
                     totalPages={totalPages}
-                    goToPage={this.paginationGoToPage}>
+                    goToPage={this.paginationGoToPage} />
-                </Pagination>
             </div>
         );
     }

js/constants/api_urls.js

@@ -1,17 +1,19 @@
+'use strict';
+
 import AppConstants from './application_constants';
 
 let apiUrls = {
-    'ownership_shares_mail' : AppConstants.baseUrl + 'ownership/shares/mail/',
+    'ownership_shares_mail': AppConstants.baseUrl + 'ownership/shares/mail/',
-    'ownership_transfers' : AppConstants.baseUrl + 'ownership/transfers/',
+    'ownership_transfers': AppConstants.baseUrl + 'ownership/transfers/',
     'user': AppConstants.baseUrl + 'users/',
     'pieces_list': AppConstants.baseUrl + 'pieces/',
     'piece': AppConstants.baseUrl + 'pieces/${piece_id}',
     'edition': AppConstants.baseUrl + 'editions/${bitcoin_id}/',
     'editions_list': AppConstants.baseUrl + 'pieces/${piece_id}/editions/',
-    'ownership_loans' : AppConstants.baseUrl + 'ownership/loans/',
+    'ownership_loans': AppConstants.baseUrl + 'ownership/loans/',
-    'ownership_consigns' : AppConstants.baseUrl + 'ownership/consigns/',
+    'ownership_consigns': AppConstants.baseUrl + 'ownership/consigns/',
-    'ownership_unconsigns' : AppConstants.baseUrl + 'ownership/unconsigns/',
+    'ownership_unconsigns': AppConstants.baseUrl + 'ownership/unconsigns/',
-    'ownership_unconsigns_request' : AppConstants.baseUrl + 'ownership/unconsigns/request/'
+    'ownership_unconsigns_request': AppConstants.baseUrl + 'ownership/unconsigns/request/'
 };
 
 export default apiUrls;

js/constants/application_constants.js

@@ -1,3 +1,5 @@
+'use strict';
+
 let constants = {
     //'baseUrl': 'http://localhost:8000/api/',
     'baseUrl': 'http://staging.ascribe.io/api/',

js/constants/languages.js

@@ -1,3 +1,5 @@
+'use strict';
+
 const languages = {
     'en-US': {
         'Bitcoin Address': 'Bitcoin Address',

js/fetchers/edition_fetcher.js

@@ -1,13 +1,11 @@
+'use strict';
+
 import fetch from '../utils/fetch';
 
-import AppConstants from '../constants/application_constants';
-
-
 let EditionFetcher = {
     /**
      * Fetch one user from the API.
      * If no arg is supplied, load the current user
-     * 
      */
     fetchOne(editionId) {
         return fetch.get('edition', {'bitcoin_id': editionId});

js/fetchers/edition_list_fetcher.js

@@ -1,9 +1,9 @@
+'use strict';
+
 import fetch from '../utils/fetch';
 
 import { generateOrderingQueryParams } from '../utils/fetch_api_utils';
 
-import AppConstants from '../constants/application_constants';
-
 
 let EditionListFetcher = {
     /**

js/fetchers/ownership_fetcher.js

@@ -1,14 +1,14 @@
+'use strict';
+
 import fetch from 'isomorphic-fetch';
 
 import AppConstants from '../constants/application_constants';
-import FetchApiUtils from '../utils/fetch_api_utils';
 
 
 let OwnershipFetcher = {
     /**
      * Fetch one user from the API.
      * If no arg is supplied, load the current user
-     * 
      */
     fetchLoanContract(email) {
         return fetch(AppConstants.baseUrl + 'ownership/loans/contract/?loanee=' + email, {

js/fetchers/piece_fetcher.js

@@ -1,15 +1,14 @@
-import fetch from '../utils/fetch';
+'use strict';
 
-import AppConstants from '../constants/application_constants';
+import fetch from '../utils/fetch';
 
 
 let PieceFetcher = {
     /**
      * Fetch one user from the API.
      * If no arg is supplied, load the current user
-     * 
      */
-    fetchOne(pieceId) {
+    fetchOne() {
         return fetch.get('piece');
     }
 };

js/fetchers/piece_list_fetcher.js

@@ -1,4 +1,5 @@
-import AppConstants from '../constants/application_constants';
+'use strict';
+
 import { generateOrderingQueryParams } from '../utils/fetch_api_utils';
 import fetch from '../utils/fetch';
 

js/fetchers/user_fetcher.js

@@ -1,13 +1,12 @@
-import fetch from '../utils/fetch';
+'use strict';
 
-import AppConstants from '../constants/application_constants';
+import fetch from '../utils/fetch';
 
 
 let UserFetcher = {
     /**
      * Fetch one user from the API.
      * If no arg is supplied, load the current user
-     * 
      */
     fetchOne() {
         return fetch.get('user');

js/mixins/alert_mixin.js

@@ -1,15 +1,17 @@
+'use strict';
+
 import React from 'react';
 import AlertDismissable from '../components/ascribe_forms/alert';
 
 let AlertMixin = {
     setAlerts(errors){
-        let alerts = errors.map(
+        let alerts = errors.map((error) => {
-                        function(error) {
+            return <AlertDismissable error={error} key={error}/>;
-                            return <AlertDismissable error={error} key={error}/>;
+        });
-                        }.bind(this)
+
-                    );
         this.setState({alerts: alerts});
     },
+    
     clearAlerts(){
         this.setState({alerts: null});
     }

js/mixins/form_mixin.js

@@ -1,8 +1,9 @@
+'use strict';
+
 import fetch from '../utils/fetch';
 import React from 'react';
 
-import AppConstants from '../constants/application_constants'
+import AlertDismissable from '../components/ascribe_forms/alert';
-import AlertDismissable from '../components/ascribe_forms/alert'
 
 export const FormMixin = {
     getInitialState() {
@@ -10,7 +11,7 @@ export const FormMixin = {
             submitted: false
             , status: null
             , errors: []
-        }
+        };
     },
 
     submit(e) {
@@ -18,7 +19,7 @@ export const FormMixin = {
         this.setState({submitted: true});
         fetch
             .post(this.url(), { body: this.getFormData() })
-            .then(response => { this.props.handleSuccess(); })
+            .then(() => { this.props.handleSuccess(); })
             .catch(this.handleError);
     },
 
@@ -40,31 +41,29 @@ export const FormMixin = {
 
     getBitcoinIds(){
         return this.props.editions.map(function(edition){
-            return edition.bitcoin_id
+            return edition.bitcoin_id;
-        })
+        });
     },
     
     getTitlesString(){
         return this.props.editions.map(function(edition){
-            return '- \"' + edition.title + ', edition ' + edition.edition_number + '\"\n'
+            return '- \"' + edition.title + ', edition ' + edition.edition_number + '\"\n';
-        })
+        });
     },
 
     render(){
         let alert = null;
         if (this.state.errors.length > 0){
-            alert = this.state.errors.map(
+            alert = this.state.errors.map((error) => {
-                    function(error) {
+                return <AlertDismissable error={error} key={error}/>;
-                        return <AlertDismissable error={error} key={error}/>;
+            });
-                    }.bind(this)
-                );
         }
         return (
             <div>
                 {alert}
                 {this.renderForm()}
             </div>
-        )
+        );
     }
 };
 

js/mixins/inject_in_head_mixin.js

@@ -1,12 +1,14 @@
+'use strict';
+
 let mapAttr = {
     link: 'href',
     source: 'src'
-}
+};
 
 let mapExt = {
     js: 'source',
     css: 'link'
-}
+};
 
 
 let InjectInHeadMixin = {
@@ -23,9 +25,9 @@ let InjectInHeadMixin = {
     },
 
     injectTag(tag, src){
-        console.log(this.foobar);
+        if (InjectInHeadMixin.isPresent(tag, src)) {
-        if (InjectInHeadMixin.isPresent(tag, src))
             return;
+        }
 
         let attr = mapAttr[tag];
         let element = document.createElement(tag);
@@ -49,6 +51,8 @@ let InjectInHeadMixin = {
         } catch (e) {
             throw new Error(`Cannot inject ${src} in the DOM, cannot guess the tag name from extension ${ext}. Valid extensions are "js" and "css".`);
         }
+        // ES6Lint says tag is not defined, pls fix
+        // - Tim
         InjectInHeadMixin.injectTag(tag, src);
     }
 

js/mixins/modal_mixin.js

@@ -1,9 +1,10 @@
-import React from 'react';
+'use strict';
 
 let ModalMixin = {
     onRequestHide(e){
-        if (e)
+        if (e) {
             e.preventDefault();
+        }
         this.props.onRequestHide();
     }
 };

js/mixins/table_column_mixin.js

@@ -1,4 +1,4 @@
-import React from 'react';
+'use strict';
 
 import { sumNumList } from '../utils/general_utils';
 
@@ -14,7 +14,7 @@ let TableColumnMixin = {
         let numOfUsedColumns = sumNumList(listOfRowValues);
 
         if(numOfUsedColumns > numOfColumns) {
-            throw new Error('This table has only ' + numOfColumns + ' columns to assign. You defined ' + numOfUsedColumns + '. Change this in the columnMap you\'re passing to the table.')
+            throw new Error('This table has only ' + numOfColumns + ' columns to assign. You defined ' + numOfUsedColumns + '. Change this in the columnMap you\'re passing to the table.');
         } else {
             return bootstrapClasses.join( listOfRowValues[i] + ' ') + listOfRowValues[i];
         }

js/routes.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 import Router from 'react-router';
 
@@ -10,11 +12,8 @@ let Route = Router.Route;
 
 let routes = (
     <Route name="app" handler={AscribeApp}>
-        <Route name="pieces" path="/" handler={PieceList}>
+        <Route name="pieces" path="/" handler={PieceList} />
-            
+        <Route name="edition" path="/editions/:editionId" handler={EditionContainer} />
-        </Route>
-        <Route name="edition" path="/editions/:editionId" handler={EditionContainer}>
-        </Route>
     </Route>
 );
 

js/stores/edition_list_store.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import React from 'react';
 
 import alt from '../alt';
@@ -15,7 +17,7 @@ class EditionListStore {
                 // This uses the index of the new editionList for determining the edition.
                 // If the list of editions can be sorted in the future, this needs to be changed!
                 editionListOfPiece[i] = React.addons.update(edition, {$merge: editionListOfPiece[i]});
-            })
+            });
         }
         this.editionList[pieceId] = editionListOfPiece;
 
@@ -26,7 +28,7 @@ class EditionListStore {
          * Default values for both are set in the editon_list-actions.
          */
         this.editionList[pieceId].orderBy = orderBy;
-        this.editionList[pieceId].orderAsc = orderAsc
+        this.editionList[pieceId].orderAsc = orderAsc;
     }
 
     onSelectEdition({pieceId, editionId}) {
@@ -55,6 +57,6 @@ class EditionListStore {
                     });
             });
     }
-};
+}
 
 export default alt.createStore(EditionListStore);

js/stores/edition_store.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import alt from '../alt';
 import EditionActions from '../actions/edition_actions';
 

js/stores/piece_list_store.js

@@ -1,4 +1,5 @@
-import React from 'react';
+'use strict';
+
 import alt from '../alt';
 
 import PieceListActions from '../actions/piece_list_actions';
@@ -7,7 +8,7 @@ import PieceListActions from '../actions/piece_list_actions';
 class PieceListStore {
     constructor() {
         /**
-         * The store manages the state that is introduced by fetching 
+         * The store manages the state that is introduced by fetching
          * the resource with certain parameters.
          *
          * This means that pieceList for example only contains pageSize-many items.
@@ -20,8 +21,8 @@ class PieceListStore {
         this.pieceListCount = 0;
         this.page = 1;
         this.pageSize = 10;
-        this.search = "";
+        this.search = '';
-        this.orderBy = "artist_name";
+        this.orderBy = 'artist_name';
         this.orderAsc = true;
         this.bindActions(PieceListActions);
     }
@@ -58,13 +59,13 @@ class PieceListStore {
          * Pagination - Known Issue:
          * #########################
          *
-         * 
+         *
          * The piece list store currently stores the open/close state of a piece list item.
          *
-         * Once a new page is requested, this.pieceList will be overwritten, which means that the 
+         * Once a new page is requested, this.pieceList will be overwritten, which means that the
          * open/close state of a specific list item will be thrown away.
          *
-         * This means that when opening an editionListTable on a piece, and continuing 
+         * This means that when opening an editionListTable on a piece, and continuing
          * clicking next or back in the pagination, the editionListTable will return to its
          * default value, which is "close".
          *
@@ -73,6 +74,6 @@ class PieceListStore {
          */
         this.pieceList = pieceList;
     }
-};
+}
 
 export default alt.createStore(PieceListStore);

js/stores/piece_store.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import alt from '../alt';
 import PieceAction from '../actions/piece_actions';
 

js/stores/user_store.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import alt from '../alt';
 import UserAction from '../actions/user_actions';
 

js/utils/fetch.js

@@ -1,11 +1,13 @@
+'use strict';
+
 import { default as _fetch } from 'isomorphic-fetch';
 
 import { argsToQueryParams } from '../utils/fetch_api_utils';
 
 
-class UrlMapError extends Error {};
+class UrlMapError extends Error {}
-class ServerError extends Error {};
+class ServerError extends Error {}
-class APIError extends Error {};
+class APIError extends Error {}
 
 
 class Fetch {
@@ -33,7 +35,7 @@ class Fetch {
     }
 
     handleAPIError(json) {
-        if (!json['success']) {
+        if (!json.success) {
             let error = new APIError();
             error.json = json;
             throw error;
@@ -58,7 +60,7 @@ class Fetch {
         let re = /\${(\w+)}/g;
 
         newUrl = newUrl.replace(re, (match, key) => {
-            let val = params[key]
+            let val = params[key];
             if (!val) {
                 throw new Error(`Cannot find param ${key}`);
             }
@@ -76,7 +78,7 @@ class Fetch {
     request(verb, url, options) {
         options = options || {};
         let merged = this._merge(this.httpOptions, options);
-        merged['method'] = verb;
+        merged.method = verb;
         return _fetch(url, merged)
                     .then(this.unpackResponse)
                     .then(JSON.parse)
@@ -95,8 +97,8 @@ class Fetch {
         let newUrl = this.prepareUrl(url, params);
         let body = null;
 
-        if (params['body']) {
+        if (params.body) {
-            body = JSON.stringify(params['body'])
+            body = JSON.stringify(params.body);
         }
         return this.request('post', url, { body });
     }

js/utils/fetch_api_utils.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import { sanitize } from './general_utils';
 
 // TODO: Create Unittests that test all functions
@@ -16,7 +18,7 @@ import { sanitize } from './general_utils';
      * ?page=1&page_size=10
      *
      * CamelCase gets converted to snake_case!
-     * 
+     *
      */
 export function argsToQueryParams(obj) {
 
@@ -38,10 +40,10 @@ export function argsToQueryParams(obj) {
                 return s + snakeCaseKey + '=' + encodeURIComponent(obj[key]);
             })
             .join('');
-};
+}
 
 /**
- * Takes a string and a boolean and generates a string query parameter for 
+ * Takes a string and a boolean and generates a string query parameter for
  * an API call.
  */
 export function generateOrderingQueryParams(orderBy, orderAsc) {
@@ -52,11 +54,11 @@ export function generateOrderingQueryParams(orderBy, orderAsc) {
     }
 
     return interpolation + orderBy;
-};
+}
 
 export function status(response) {
     if (response.status >= 200 && response.status < 300) {
-        return response
+        return response;
     }
-    throw new Error(response.json())
+    throw new Error(response.json());
-};
+}

js/utils/general_utils.js

@@ -1,3 +1,5 @@
+'use strict';
+
 // TODO: Create Unittests that test all functions
 
 export function sanitize(obj) {
@@ -12,7 +14,7 @@ export function sanitize(obj) {
         });
 
     return obj;
-};
+}
 
 /**
  * Returns the values of an object.
@@ -21,7 +23,7 @@ export function valuesOfObject(obj) {
     return Object
             .keys(obj)
             .map(key => obj[key]);
-};
+}
 
 /**
  * Sums up a list of numbers. Like a Epsilon-math-kinda-sum...
@@ -30,11 +32,14 @@ export function sumNumList(l) {
     let sum = 0;
     l.forEach((num) => sum += parseFloat(num) || 0);
     return sum;
-};
+}
 
 /*
     Taken from http://stackoverflow.com/a/4795914/1263876
     Behaves like C's format string function
+
+    REFACTOR TO ES6 (let instead of var)
+
 */
 export function formatText() {
     var args = arguments,
@@ -60,7 +65,7 @@ export function formatText() {
         }
         return val;
     });
-};
+}
 
 /**
  * Takes a list of object and merges their keys to one object.
@@ -72,11 +77,11 @@ export function mergeOptions(...l) {
     let newObj = {};
 
     for(let i = 1; i < l.length; i++) {
-        newObj = _mergeOptions(newObj, _mergeOptions(l[i-1], l[i]));
+        newObj = _mergeOptions(newObj, _mergeOptions(l[i - 1], l[i]));
     }
 
     return newObj;
-};
+}
 
 /**
  * Overwrites obj1's values with obj2's and adds obj2's if non existent in obj1
@@ -85,9 +90,9 @@ export function mergeOptions(...l) {
  * @returns obj3 a new object based on obj1 and obj2
  * Taken from: http://stackoverflow.com/a/171256/1263876
  */
-function _mergeOptions(obj1,obj2){
+function _mergeOptions(obj1, obj2){
-    var obj3 = {};
+    let obj3 = {};
-    for (var attrname in obj1) { obj3[attrname] = obj1[attrname]; }
+    for (let attrname in obj1) { obj3[attrname] = obj1[attrname]; }
-    for (var attrname in obj2) { obj3[attrname] = obj2[attrname]; }
+    for (let attrname in obj2) { obj3[attrname] = obj2[attrname]; }
     return obj3;
-};
+}

js/utils/lang_utils.js

@@ -1,3 +1,5 @@
+'use strict';
+
 import languages from '../constants/languages';
 
 import { formatText } from './general_utils';
@@ -27,4 +29,4 @@ export function getLangText(s, ...args) {
         }
         
     }
-};
+}