[jsx-wrap-multilines] Сompatibility with current default parametrs

Author: yepninja

docs/rules/jsx-wrap-multilines.md

@@ -6,15 +6,15 @@ Wrapping multiline JSX in parentheses can improve readability and/or convenience
 
 ## Rule Details
 
-This rule optionally takes a second parameter in the form of an object, containing places to apply the rule. By default, all the syntax listed below will be checked except the logical expressions and JSX attributes, but these can be explicitly disabled. Any syntax type missing in the object will follow the default behavior (become enabled).
+This rule optionally takes a second parameter in the form of an object, containing places to apply the rule. By default, all the syntax listed below will be checked except the conditions out of declaration or assignment, logical expressions and JSX attributes, but these can be explicitly disabled. Any syntax type missing in the object will follow the default behavior (become enabled).
 
 There are the possible syntax available:
 
 * `declaration`
 * `assignment`
 * `return`
 * `arrow`
-* `condition`
+* `condition` (not enabed by default, by default only conditions in declaraiton or assignment)
 * `logical` (not enabled by default)
 * `prop` (not enabled by default)
 

lib/rules/jsx-wrap-multilines.js

@@ -15,7 +15,7 @@ const DEFAULTS = {
   assignment: true,
   return: true,
   arrow: true,
-  condition: true,
+  condition: false,
   logical: false,
   prop: false
 };
@@ -112,13 +112,23 @@ module.exports = {
         if (!isEnabled('declaration')) {
           return;
         }
+        if (!isEnabled('condition') && node.init && node.init.type === 'ConditionalExpression') {
+          check(node.init.consequent);
+          check(node.init.alternate);
+          return;
+        }
         check(node.init);
       },
 
       AssignmentExpression: function(node) {
         if (!isEnabled('assignment')) {
           return;
         }
+        if (!isEnabled('condition') && node.right.type === 'ConditionalExpression') {
+          check(node.right.consequent);
+          check(node.right.alternate);
+          return;
+        }
         check(node.right);
       },
 

tests/lib/rules/jsx-wrap-multilines.js

@@ -217,14 +217,14 @@ ruleTester.run('jsx-wrap-multilines', rule, {
       code: DECLARATION_TERNARY_PAREN
     }, {
       code: DECLARATION_TERNARY_NO_PAREN,
-      options: [{condition: false}]
+      options: [{declaration: false}]
     }, {
       code: ASSIGNMENT_TERNARY_SINGLE_LINE
     }, {
       code: ASSIGNMENT_TERNARY_PAREN
     }, {
       code: ASSIGNMENT_TERNARY_NO_PAREN,
-      options: [{condition: false}]
+      options: [{assignment: false}]
     }, {
       code: DECLARATION_SINGLE_LINE
     }, {
@@ -252,10 +252,13 @@ ruleTester.run('jsx-wrap-multilines', rule, {
     }, {
       code: CONDITION_SINGLE_LINE
     }, {
-      code: CONDITION_NO_PAREN,
-      options: [{condition: false}]
+      code: CONDITION_SINGLE_LINE,
+      options: [{condition: true}]
+    }, {
+      code: CONDITION_NO_PAREN
     }, {
-      code: CONDITION_PAREN
+      code: CONDITION_PAREN,
+      options: [{condition: true}]
     }, {
       code: LOGICAL_SINGLE_LINE
     }, {
@@ -323,6 +326,7 @@ ruleTester.run('jsx-wrap-multilines', rule, {
     }, {
       code: CONDITION_NO_PAREN,
       output: CONDITION_PAREN,
+      options: [{condition: true}],
       errors: [{message: 'Missing parentheses around multilines JSX'}]
     }, {
       code: LOGICAL_NO_PAREN,