Update to React 0.15 and fix bad names

client/source/scripts/components/forms/Checkbox.js

@@ -5,7 +5,7 @@ import Checkmark from '../icons/Checkmark';
 
 const METHODS_TO_BIND = ['handleCheckboxChange'];
 
-export default class SearchBox extends React.Component {
+export default class Checkbox extends React.Component {
   constructor() {
     super();
 
@@ -17,7 +17,8 @@ export default class SearchBox extends React.Component {
   }
 
   componentDidMount() {
-    if (this.state.checked !== this.props.checked) {
+    if (this.props.checked != null
+      && this.state.checked !== this.props.checked) {
       this.setState({checked: this.props.checked});
     }
   }
@@ -52,3 +53,7 @@ export default class SearchBox extends React.Component {
     );
   }
 }
+
+Checkbox.defaultProps = {
+  checked: false
+};

client/source/scripts/components/modals/AddTorrentsActions.js

@@ -6,7 +6,7 @@ import SettingsStore from '../../stores/SettingsStore';
 
 const METHODS_TO_BIND = ['handleStartTorrentsToggle'];
 
-export default class AddTorrents extends React.Component {
+export default class AddTorrentsActions extends React.Component {
   constructor() {
     super();
 

client/source/scripts/components/modals/AddTorrentsByFile.js

@@ -19,7 +19,7 @@ const METHODS_TO_BIND = [
   'handleStartTorrentsToggle'
 ];
 
-export default class AddTorrents extends React.Component {
+export default class AddTorrentsByFile extends React.Component {
   constructor() {
     super();
 

client/source/scripts/components/modals/AddTorrentsByURL.js

@@ -15,7 +15,7 @@ const METHODS_TO_BIND = [
   'handleUrlRemove'
 ];
 
-export default class AddTorrents extends React.Component {
+export default class AddTorrentsByURL extends React.Component {
   constructor() {
     super();
 
@@ -23,7 +23,7 @@ export default class AddTorrents extends React.Component {
       addTorrentsError: null,
       destination: null,
       isAddingTorrents: false,
-      urlTextboxes: [{value: null}],
+      urlTextboxes: [{value: ''}],
       startTorrents: true
     };
 
@@ -59,7 +59,7 @@ export default class AddTorrents extends React.Component {
 
   handleUrlAdd(index) {
     let urlTextboxes = Object.assign([], this.state.urlTextboxes);
-    urlTextboxes.splice(index + 1, 0, {value: null});
+    urlTextboxes.splice(index + 1, 0, {value: ''});
     this.setState({urlTextboxes});
   }
 

client/source/scripts/components/modals/AddTorrentsDestination.js

@@ -6,7 +6,7 @@ import SettingsStore from '../../stores/SettingsStore';
 
 const METHODS_TO_BIND = ['handleDestinationChange'];
 
-export default class AddTorrents extends React.Component {
+export default class AddTorrentsDestination extends React.Component {
   constructor() {
     super();
 

client/source/scripts/components/modals/ConfirmModal.js

@@ -2,7 +2,7 @@ import React from 'react';
 
 import Modal from './Modal';
 
-export default class AddTorrents extends React.Component {
+export default class ConfirmModal extends React.Component {
   constructor() {
     super();
   }

client/source/scripts/components/modals/Modal.js

@@ -1,5 +1,6 @@
 import _ from 'lodash';
 import classnames from 'classnames';
+import createFragment from 'react-addons-create-fragment';
 import React from 'react';
 
 import CustomScrollbars from '../ui/CustomScrollbars';
@@ -46,8 +47,8 @@ export default class Modal extends React.Component {
         'modal--tabs-in-header': !this.props.tabsInBody,
         'modal--tabs-in-body': this.props.tabsInBody
       }, this.props.classNames);
-    let modalBody = [this.props.content];
-    let modalHeader = [this.props.heading];
+    let modalBody = [createFragment({'modal-body': this.props.content})];
+    let modalHeader = [createFragment({'modal-header': this.props.heading})];
     let headerClasses = classnames('modal__header', {
       'has-tabs': this.props.tabs
     });

client/source/scripts/components/modals/MoveTorrents.js

@@ -20,7 +20,7 @@ const METHODS_TO_BIND = [
   'onMoveError'
 ];
 
-export default class AddTorrents extends React.Component {
+export default class MoveTorrents extends React.Component {
   constructor() {
     super();
 
@@ -121,13 +121,12 @@ export default class AddTorrents extends React.Component {
 
   getContent() {
     return (
-      <div className="form">
+      <div className="form modal__content">
         <AddTorrentsDestination onChange={this.handleDestinationChange}
           suggested={this.state.originalSource} />
         <div className="form__row">
           <Checkbox onChange={this.handleCheckboxChange}>Move data</Checkbox>
         </div>
-        <ModalActions actions={this.getActions()} />
       </div>
     );
   }
@@ -144,9 +143,10 @@ export default class AddTorrents extends React.Component {
 
   render() {
     return (
-      <Modal heading="Set Download Location"
+      <Modal actions={this.getActions()}
+        content={this.getContent()}
         dismiss={this.props.dismiss}
-        content={this.getContent()} />
+        heading="Set Download Location" />
     );
   }
 }

client/source/scripts/components/modals/SettingsModal.js

@@ -12,7 +12,7 @@ const METHODS_TO_BIND = [
   'handleSettingsFetchRequestSuccess'
 ];
 
-export default class AddTorrents extends React.Component {
+export default class SettingsModal extends React.Component {
   constructor() {
     super();
 

client/source/scripts/components/modals/SettingsSpeedLimit.js

@@ -2,7 +2,7 @@ import React from 'react';
 
 const METHODS_TO_BIND = ['handleDownloadTextChange', 'handleUploadTextChange'];
 
-export default class AddTorrents extends React.Component {
+export default class SettingsSpeedLimit extends React.Component {
   constructor() {
     super();
 
@@ -97,8 +97,8 @@ export default class AddTorrents extends React.Component {
   }
 
   render() {
-    let downloadValue = this.getDownloadValue();
-    let uploadValue = this.getUploadValue();
+    let downloadValue = this.getDownloadValue() || 0;
+    let uploadValue = this.getUploadValue() || 0;
 
     return (
       <div>