From 7202970bdcf65edd2354216784de01f944211d25 Mon Sep 17 00:00:00 2001
From: AlexKVal <alexkval@gmail.com>
Date: Thu, 20 Aug 2015 17:48:26 +0300
Subject: [PATCH] Impose 'brace-style' 'default-case' and 'ident' rules

part of `airbnb` style
---
 .eslintrc                       |  3 ---
 docs/examples/ButtonInput.js    |  4 +--
 docs/examples/Input.js          |  6 ++---
 docs/src/PropTable.js           | 44 +++++++++++++++---------------
 src/AffixMixin.js               |  2 +-
 src/Carousel.js                 | 24 ++++++++---------
 src/Dropdown.js                 | 33 +++++++++++------------
 src/DropdownMenu.js             | 25 +++++++++---------
 src/InputBase.js                | 47 +++++++++++++++++----------------
 src/NavItem.js                  | 20 +++++++-------
 src/Navbar.js                   |  2 +-
 src/Overlay.js                  |  4 +--
 src/SplitButton.js              |  2 +-
 src/Tabs.js                     | 20 +++++++-------
 src/Thumbnail.js                |  6 ++---
 src/utils/deprecationWarning.js |  4 +--
 test/GlyphiconSpec.js           |  8 +++---
 test/ThumbnailSpec.js           |  4 +--
 tools/build.js                  | 12 ++++-----
 tools/fs-utils.js               | 42 ++++++++++++++---------------
 20 files changed, 153 insertions(+), 159 deletions(-)

diff --git a/.eslintrc b/.eslintrc
index 7e00e12148..ae5570a5d6 100644
--- a/.eslintrc
+++ b/.eslintrc
@@ -13,15 +13,12 @@
     "babel"
   ],
   "rules": {
-    "brace-style": 0,
     "constructor-super": 2,
     "comma-dangle": 0,
     "comma-spacing": 2,
     "comma-style": [2, "last"],
-    "default-case": 0,
     "func-names": 0,
     "guard-for-in": 0,
-    "indent": 0,
     "one-var": [2, { "initialized": "never" }],
     "padded-blocks": 0,
     "prefer-const": 0,
diff --git a/docs/examples/ButtonInput.js b/docs/examples/ButtonInput.js
index e1c64bdfb2..774940998d 100644
--- a/docs/examples/ButtonInput.js
+++ b/docs/examples/ButtonInput.js
@@ -17,8 +17,8 @@ const ButtonInputExample = React.createClass({
     let length = this.refs.input.getValue().length;
     let style = 'danger';
 
-    if (length > 10) { style = 'success'; }
-    else if (length > 5) { style = 'warning'; }
+    if (length > 10) style = 'success';
+    else if (length > 5) style = 'warning';
 
     let disabled = style !== 'success';
 
diff --git a/docs/examples/Input.js b/docs/examples/Input.js
index 980413339a..c98d7911e7 100644
--- a/docs/examples/Input.js
+++ b/docs/examples/Input.js
@@ -7,9 +7,9 @@ const ExampleInput = React.createClass({
 
   validationState() {
     let length = this.state.value.length;
-    if (length > 10) { return 'success'; }
-    else if (length > 5) { return 'warning'; }
-    else if (length > 0) { return 'error'; }
+    if (length > 10) return 'success';
+    else if (length > 5) return 'warning';
+    else if (length > 0) return 'error';
   },
 
   handleChange() {
diff --git a/docs/src/PropTable.js b/docs/src/PropTable.js
index 46fb2314c8..4feaed34a4 100644
--- a/docs/src/PropTable.js
+++ b/docs/src/PropTable.js
@@ -146,28 +146,28 @@ const PropTable = React.createClass({
     let doclets = prop.doclets || {};
 
     switch (name) {
-      case 'object':
-        return name;
-      case 'union':
-        return type.value.reduce((current, val, i, list) => {
-          let item = this.getType({ type: val });
-          if (React.isValidElement(item)) {
-            item = React.cloneElement(item, {key: i});
-          }
-          current = current.concat(item);
-
-          return i === (list.length - 1) ? current : current.concat(' | ');
-        }, []);
-      case 'array':
-        let child = this.getType({ type: type.value });
-
-        return <span>{'array<'}{ child }{'>'}</span>;
-      case 'enum':
-        return this.renderEnum(type);
-      case 'custom':
-        return cleanDocletValue(doclets.type || name);
-      default:
-        return name;
+    case 'object':
+      return name;
+    case 'union':
+      return type.value.reduce((current, val, i, list) => {
+        let item = this.getType({ type: val });
+        if (React.isValidElement(item)) {
+          item = React.cloneElement(item, {key: i});
+        }
+        current = current.concat(item);
+
+        return i === (list.length - 1) ? current : current.concat(' | ');
+      }, []);
+    case 'array':
+      let child = this.getType({ type: type.value });
+
+      return <span>{'array<'}{ child }{'>'}</span>;
+    case 'enum':
+      return this.renderEnum(type);
+    case 'custom':
+      return cleanDocletValue(doclets.type || name);
+    default:
+      return name;
     }
   },
 
diff --git a/src/AffixMixin.js b/src/AffixMixin.js
index af97e131c7..8e013dd4dc 100644
--- a/src/AffixMixin.js
+++ b/src/AffixMixin.js
@@ -30,7 +30,7 @@ const AffixMixin = {
 
   checkPosition() {
     let DOMNode, scrollHeight, scrollTop, position, offsetTop, offsetBottom,
-        affix, affixType, affixPositionTop;
+      affix, affixType, affixPositionTop;
 
     // TODO: or not visible
     if (!this.isMounted()) {
diff --git a/src/Carousel.js b/src/Carousel.js
index 8801661c2d..c306b6285b 100644
--- a/src/Carousel.js
+++ b/src/Carousel.js
@@ -249,18 +249,18 @@ const Carousel = React.createClass({
             this.state.previousActiveIndex === index && this.props.slide;
 
     return cloneElement(
-        child,
-        {
-          active: isActive,
-          ref: child.ref,
-          key: child.key ? child.key : index,
-          index,
-          animateOut: isPreviousActive,
-          animateIn: isActive && this.state.previousActiveIndex != null && this.props.slide,
-          direction: this.state.direction,
-          onAnimateOutEnd: isPreviousActive ? this.handleItemAnimateOutEnd : null
-        }
-      );
+      child,
+      {
+        active: isActive,
+        ref: child.ref,
+        key: child.key ? child.key : index,
+        index,
+        animateOut: isPreviousActive,
+        animateIn: isActive && this.state.previousActiveIndex != null && this.props.slide,
+        direction: this.state.direction,
+        onAnimateOutEnd: isPreviousActive ? this.handleItemAnimateOutEnd : null
+      }
+    );
   },
 
   handleSelect(index, direction) {
diff --git a/src/Dropdown.js b/src/Dropdown.js
index a5b652af39..473e4a62f1 100644
--- a/src/Dropdown.js
+++ b/src/Dropdown.js
@@ -106,21 +106,21 @@ class Dropdown extends React.Component {
     };
 
     switch(event.keyCode) {
-      case keycode.codes.down:
-        if (!this.props.open) {
-          this.toggleOpen();
-        }
-        else {
-          focusNext();
-        }
-        event.preventDefault();
-        break;
-      case keycode.codes.esc:
-      case keycode.codes.tab:
-        if (this.props.open) {
-          this.handleClose(event);
-        }
-        break;
+    case keycode.codes.down:
+      if (!this.props.open) {
+        this.toggleOpen();
+      } else {
+        focusNext();
+      }
+      event.preventDefault();
+      break;
+    case keycode.codes.esc:
+    case keycode.codes.tab:
+      if (this.props.open) {
+        this.handleClose(event);
+      }
+      break;
+    default:
     }
   }
 
@@ -134,8 +134,7 @@ class Dropdown extends React.Component {
     // to the next focusable input
     if (event && event.keyCode === keycode.codes.tab){
       setTimeout(this.toggleOpen);
-    }
-    else {
+    } else {
       this.toggleOpen();
     }
 
diff --git a/src/DropdownMenu.js b/src/DropdownMenu.js
index 770684e26b..ed81eb21fe 100644
--- a/src/DropdownMenu.js
+++ b/src/DropdownMenu.js
@@ -20,18 +20,19 @@ class DropdownMenu extends React.Component {
   handleKeyDown(event) {
 
     switch(event.keyCode) {
-      case keycode.codes.down:
-        this.focusNext();
-        event.preventDefault();
-        break;
-      case keycode.codes.up:
-        this.focusPrevious();
-        event.preventDefault();
-        break;
-      case keycode.codes.esc:
-      case keycode.codes.tab:
-        this.props.onClose(event);
-        break;
+    case keycode.codes.down:
+      this.focusNext();
+      event.preventDefault();
+      break;
+    case keycode.codes.up:
+      this.focusPrevious();
+      event.preventDefault();
+      break;
+    case keycode.codes.esc:
+    case keycode.codes.tab:
+      this.props.onClose(event);
+      break;
+    default:
     }
   }
 
diff --git a/src/InputBase.js b/src/InputBase.js
index c5745d3867..c1e193053e 100644
--- a/src/InputBase.js
+++ b/src/InputBase.js
@@ -76,8 +76,9 @@ class InputBase extends React.Component {
 
     let inputGroupClassName;
     switch (this.props.bsSize) {
-      case 'small': inputGroupClassName = 'input-group-sm'; break;
-      case 'large': inputGroupClassName = 'input-group-lg'; break;
+    case 'small': inputGroupClassName = 'input-group-sm'; break;
+    case 'large': inputGroupClassName = 'input-group-lg'; break;
+    default:
     }
 
     return addonBefore || addonAfter || buttonBefore || buttonAfter ? (
@@ -98,10 +99,10 @@ class InputBase extends React.Component {
       }
 
       switch (this.props.bsStyle) {
-        case 'success': return <Glyphicon formControlFeedback glyph="ok" key="icon" />;
-        case 'warning': return <Glyphicon formControlFeedback glyph="warning-sign" key="icon" />;
-        case 'error': return <Glyphicon formControlFeedback glyph="remove" key="icon" />;
-        default: return <span className="form-control-feedback" key="icon" />;
+      case 'success': return <Glyphicon formControlFeedback glyph="ok" key="icon" />;
+      case 'warning': return <Glyphicon formControlFeedback glyph="warning-sign" key="icon" />;
+      case 'error': return <Glyphicon formControlFeedback glyph="remove" key="icon" />;
+      default: return <span className="form-control-feedback" key="icon" />;
       }
     } else {
       return null;
@@ -157,24 +158,24 @@ class InputBase extends React.Component {
     }
 
     switch (this.props.type) {
-      case 'select':
-        return (
-          <select {...this.props} className={classNames(this.props.className, 'form-control')} ref="input" key="input">
-            {this.props.children}
-          </select>
-        );
-      case 'textarea':
-        return <textarea {...this.props} className={classNames(this.props.className, 'form-control')} ref="input" key="input" />;
-      case 'static':
-        return (
-          <p {...this.props} className={classNames(this.props.className, 'form-control-static')} ref="input" key="input">
-            {this.props.value}
-          </p>
-        );
+    case 'select':
+      return (
+        <select {...this.props} className={classNames(this.props.className, 'form-control')} ref="input" key="input">
+          {this.props.children}
+        </select>
+      );
+    case 'textarea':
+      return <textarea {...this.props} className={classNames(this.props.className, 'form-control')} ref="input" key="input" />;
+    case 'static':
+      return (
+        <p {...this.props} className={classNames(this.props.className, 'form-control-static')} ref="input" key="input">
+          {this.props.value}
+        </p>
+      );
+    default:
+      const className = this.isCheckboxOrRadio() || this.isFile() ? '' : 'form-control';
+      return <input {...this.props} className={classNames(this.props.className, className)} ref="input" key="input" />;
     }
-
-    let className = this.isCheckboxOrRadio() || this.isFile() ? '' : 'form-control';
-    return <input {...this.props} className={classNames(this.props.className, className)} ref="input" key="input" />;
   }
 
   renderFormGroup(children) {
diff --git a/src/NavItem.js b/src/NavItem.js
index fe68c0cc1d..565c003a5c 100644
--- a/src/NavItem.js
+++ b/src/NavItem.js
@@ -39,17 +39,17 @@ const NavItem = React.createClass({
         'aria-controls': ariaControls,
         ...props } = this.props;
     let classes = {
-          active,
-          disabled
-        };
+      active,
+      disabled
+    };
     let linkProps = {
-          role,
-          href,
-          title,
-          target,
-          id: linkId,
-          onClick: this.handleClick
-        };
+      role,
+      href,
+      title,
+      target,
+      id: linkId,
+      onClick: this.handleClick
+    };
 
     if (!role && href === '#') {
       linkProps.role = 'button';
diff --git a/src/Navbar.js b/src/Navbar.js
index 787e0a4b4a..cc8943d12c 100644
--- a/src/Navbar.js
+++ b/src/Navbar.js
@@ -139,7 +139,7 @@ const Navbar = React.createClass({
         <span className="icon-bar" key={1}></span>,
         <span className="icon-bar" key={2}></span>,
         <span className="icon-bar" key={3}></span>
-    ];
+      ];
 
     return (
       <button className="navbar-toggle" type="button" onClick={this.handleToggle}>
diff --git a/src/Overlay.js b/src/Overlay.js
index e0758adb68..6566908121 100644
--- a/src/Overlay.js
+++ b/src/Overlay.js
@@ -56,8 +56,8 @@ Overlay.propTypes = {
    * Use animation
    */
   animation: React.PropTypes.oneOfType([
-      React.PropTypes.bool,
-      elementType
+    React.PropTypes.bool,
+    elementType
   ]),
 
   /**
diff --git a/src/SplitButton.js b/src/SplitButton.js
index 364577602d..46d57a1a97 100644
--- a/src/SplitButton.js
+++ b/src/SplitButton.js
@@ -17,7 +17,7 @@ class SplitButton extends React.Component {
       bsStyle, // eslint-disable-line
       ...props } = this.props;
 
-     let { disabled } = props;
+    let { disabled } = props;
 
     let button = (
       <Button
diff --git a/src/Tabs.js b/src/Tabs.js
index ebea21243e..ffcd338549 100644
--- a/src/Tabs.js
+++ b/src/Tabs.js
@@ -205,16 +205,16 @@ const Tabs = React.createClass({
     let paneIsAlreadyActive = previousActiveKey != null && child.props.eventKey === previousActiveKey;
 
     return cloneElement(
-        child,
-        {
-          active: shouldPaneBeSetActive && (thereIsNoActivePane || !this.props.animation),
-          id: paneId(this.props, child),
-          'aria-labelledby': tabId(this.props, child),
-          key: child.key ? child.key : index,
-          animation: this.props.animation,
-          onAnimateOutEnd: paneIsAlreadyActive ? this.handlePaneAnimateOutEnd : null
-        }
-      );
+      child,
+      {
+        active: shouldPaneBeSetActive && (thereIsNoActivePane || !this.props.animation),
+        id: paneId(this.props, child),
+        'aria-labelledby': tabId(this.props, child),
+        key: child.key ? child.key : index,
+        animation: this.props.animation,
+        onAnimateOutEnd: paneIsAlreadyActive ? this.handlePaneAnimateOutEnd : null
+      }
+    );
   },
 
   renderTab(child) {
diff --git a/src/Thumbnail.js b/src/Thumbnail.js
index 997d426b95..6839d5c4f7 100644
--- a/src/Thumbnail.js
+++ b/src/Thumbnail.js
@@ -27,8 +27,7 @@ const Thumbnail = React.createClass({
           <img src={this.props.src} alt={this.props.alt} />
         </SafeAnchor>
       );
-    }
-    else {
+    } else {
       if(this.props.children) {
         return (
           <div {...this.props} className={classSet(this.props.className, classes)}>
@@ -38,8 +37,7 @@ const Thumbnail = React.createClass({
             </div>
           </div>
         );
-      }
-      else {
+      } else {
         return (
           <div {...this.props} className={classSet(this.props.className, classes)}>
             <img src={this.props.src} alt={this.props.alt} />
diff --git a/src/utils/deprecationWarning.js b/src/utils/deprecationWarning.js
index 3028dc2f10..140066046d 100644
--- a/src/utils/deprecationWarning.js
+++ b/src/utils/deprecationWarning.js
@@ -7,8 +7,7 @@ function deprecationWarning(oldname, newname, link) {
 
   if (typeof oldname === 'object'){
     message = oldname.message;
-  }
-  else {
+  } else {
     message = `${oldname} is deprecated. Use ${newname} instead.`;
 
     if (link) {
@@ -38,4 +37,3 @@ deprecationWarning.wrapper = function(Component, ...args){
 };
 
 export default deprecationWarning;
-
diff --git a/test/GlyphiconSpec.js b/test/GlyphiconSpec.js
index 49d8338436..61c58b7817 100644
--- a/test/GlyphiconSpec.js
+++ b/test/GlyphiconSpec.js
@@ -12,11 +12,11 @@ describe('Glyphicon', function () {
   });
 
   it('renders without the .form-control-feedback class', function () {
-      let instance = ReactTestUtils.renderIntoDocument(
-        <Glyphicon glyph='star' />
-      );
+    let instance = ReactTestUtils.renderIntoDocument(
+      <Glyphicon glyph='star' />
+    );
 
-      assert.notOk(React.findDOMNode(instance).className.match(/\bform-control-feedback\b/));
+    assert.notOk(React.findDOMNode(instance).className.match(/\bform-control-feedback\b/));
   });
 
   context('when setting the formControlFeedback prop', function () {
diff --git a/test/ThumbnailSpec.js b/test/ThumbnailSpec.js
index f4ba5ae92b..54c1f4f1f4 100644
--- a/test/ThumbnailSpec.js
+++ b/test/ThumbnailSpec.js
@@ -26,14 +26,14 @@ describe('Thumbnail', function () {
     assert.equal(React.findDOMNode(instance).nodeName, 'DIV');
   });
 
-    it('Should have an image', function () {
+  it('Should have an image', function () {
     let instance = ReactTestUtils.renderIntoDocument(
       <Thumbnail src="#" alt="test" />
     );
     assert.ok(ReactTestUtils.findRenderedDOMComponentWithTag(instance, 'img'));
   });
 
-    it('Should have an inner div with class caption', function () {
+  it('Should have an inner div with class caption', function () {
     let instance = ReactTestUtils.renderIntoDocument(
       <Thumbnail src="#" alt="test">
         Test
diff --git a/tools/build.js b/tools/build.js
index 401b123a75..42387e7f33 100644
--- a/tools/build.js
+++ b/tools/build.js
@@ -17,10 +17,10 @@ function forkAndBuildDocs({verbose}) {
 
 export default function Build(options) {
   return Promise.all([
-      lib(),
-      bower(),
-      dist(),
-      forkAndBuildDocs(options)
-    ])
-    .then(() => copy(distRoot, bowerRoot));
+    lib(),
+    bower(),
+    dist(),
+    forkAndBuildDocs(options)
+  ])
+  .then(() => copy(distRoot, bowerRoot));
 }
diff --git a/tools/fs-utils.js b/tools/fs-utils.js
index b066408bc5..561b0cfad4 100644
--- a/tools/fs-utils.js
+++ b/tools/fs-utils.js
@@ -6,32 +6,32 @@ function copy(src, dest, options) {
   options = options || {};
 
   return Promise.all([
-      fsp.stat(src),
-      fsp.exists(dest)
-        .then(exists => {
-          if (!exists) {
-            return false;
-          }
+    fsp.stat(src),
+    fsp.exists(dest)
+      .then(exists => {
+        if (!exists) {
+          return false;
+        }
 
-          return fsp.stat(dest);
-        })
-    ])
-    .then(([srcStat, destStat]) => {
-      if (srcStat.isFile() && destStat && destStat.isDirectory()) {
-        let filename = path.basename(src);
-        dest = path.join(dest, filename);
-      }
+        return fsp.stat(dest);
+      })
+  ])
+  .then(([srcStat, destStat]) => {
+    if (srcStat.isFile() && destStat && destStat.isDirectory()) {
+      let filename = path.basename(src);
+      dest = path.join(dest, filename);
+    }
 
-      return new Promise((resolve, reject) => {
-        fse.copy(src, dest, options, err => {
-          if (err) {
-            reject(err);
-          }
+    return new Promise((resolve, reject) => {
+      fse.copy(src, dest, options, err => {
+        if (err) {
+          reject(err);
+        }
 
-          resolve();
-        });
+        resolve();
       });
     });
+  });
 }
 
 export default {