From 97c662b87be369c8889b0ba1e6b456973028cbdd Mon Sep 17 00:00:00 2001
From: Christopher Chedeau <vjeuxx@gmail.com>
Date: Thu, 13 Apr 2017 17:09:00 -0700
Subject: [PATCH] Do not inline member expressions as part of assignments
 (#1256)

This is a regression from #1036 and #1188.

Fixes #1241
Fixes #1236
---
 src/printer.js                                | 25 +++++++++++--
 .../__snapshots__/jsfmt.spec.js.snap          | 36 +++++++++++++++++++
 tests/method-chain/break-last-member.js       | 11 ++++++
 3 files changed, 69 insertions(+), 3 deletions(-)

diff --git a/src/printer.js b/src/printer.js
index e403e103050f..984b20417490 100644
--- a/src/printer.js
+++ b/src/printer.js
@@ -185,6 +185,7 @@ function genericPrintNoParens(path, options, print, args) {
       return concat(["(", path.call(print, "expression"), ")"]);
     case "AssignmentExpression":
       return printAssignment(
+        n.left,
         path.call(print, "left"),
         n.operator,
         n.right,
@@ -232,7 +233,22 @@ function genericPrintNoParens(path, options, print, args) {
       ]);
     case "MemberExpression": {
       const parent = path.getParentNode();
+      let firstNonMemberParent;
+      let i = 0;
+      do {
+        firstNonMemberParent = path.getParentNode(i);
+        i++;
+      } while (
+        firstNonMemberParent &&
+        firstNonMemberParent.type === "MemberExpression"
+      );
+
       const shouldInline =
+        firstNonMemberParent && (
+          (firstNonMemberParent.type === "VariableDeclarator" &&
+            firstNonMemberParent.id.type === "ObjectPattern") ||
+          (firstNonMemberParent.type === "AssignmentExpression" &&
+            firstNonMemberParent.left.type === "ObjectPattern")) ||
         n.computed ||
         (n.object.type === "Identifier" &&
           n.property.type === "Identifier" &&
@@ -964,6 +980,7 @@ function genericPrintNoParens(path, options, print, args) {
       return group(concat(parts));
     case "VariableDeclarator":
       return printAssignment(
+        n.id,
         path.call(print, "id"),
         "=",
         n.init,
@@ -3161,6 +3178,7 @@ function printBinaryishExpressions(path, print, options, isNested) {
 }
 
 function printAssignment(
+  leftNode,
   printedLeft,
   operator,
   rightNode,
@@ -3176,9 +3194,10 @@ function printAssignment(
     printed = indent(concat([hardline, printedRight]));
   } else if (
     (isBinaryish(rightNode) && !shouldInlineLogicalExpression(rightNode)) ||
-    rightNode.type === "StringLiteral" ||
-    (rightNode.type === "Literal" && typeof rightNode.value === "string") ||
-    isMemberExpressionChain(rightNode)
+    leftNode.type !== "ObjectPattern" && (
+      rightNode.type === "StringLiteral" ||
+      (rightNode.type === "Literal" && typeof rightNode.value === "string") ||
+      isMemberExpressionChain(rightNode))
   ) {
     printed = indent(concat([line, printedRight]));
   } else {
diff --git a/tests/method-chain/__snapshots__/jsfmt.spec.js.snap b/tests/method-chain/__snapshots__/jsfmt.spec.js.snap
index 2279c107050b..f37580f4b859 100644
--- a/tests/method-chain/__snapshots__/jsfmt.spec.js.snap
+++ b/tests/method-chain/__snapshots__/jsfmt.spec.js.snap
@@ -129,6 +129,17 @@ superSupersuperSupersuperSupersuperSupersuperSuperLong.exampleOfOrderOfGetterAnd
 expect(
   findDOMNode(component.instance()).getElementsByClassName(styles.inner)[0].style.paddingRight
 ).toBe('1000px');
+
+const { course, conflicts = [], index, scheduleId, studentId, something } = a.this.props;
+
+const { course, conflicts = [], index, scheduleId, studentId, something } = this.props;
+
+const {
+  updated,
+  author: { identifier: ownerId },
+  location,
+  category: categories,
+} = rawAd.entry;
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 SomeVeryLongUpperCaseConstant.someVeryLongCallExpression()
   .some_very_long_member_expression;
@@ -143,6 +154,31 @@ expect(
     .style.paddingRight
 ).toBe("1000px");
 
+const {
+  course,
+  conflicts = [],
+  index,
+  scheduleId,
+  studentId,
+  something
+} = a.this.props;
+
+const {
+  course,
+  conflicts = [],
+  index,
+  scheduleId,
+  studentId,
+  something
+} = this.props;
+
+const {
+  updated,
+  author: { identifier: ownerId },
+  location,
+  category: categories
+} = rawAd.entry;
+
 `;
 
 exports[`comment.js 1`] = `
diff --git a/tests/method-chain/break-last-member.js b/tests/method-chain/break-last-member.js
index 0fb2f7a8138f..a78b2590c61a 100644
--- a/tests/method-chain/break-last-member.js
+++ b/tests/method-chain/break-last-member.js
@@ -7,3 +7,14 @@ superSupersuperSupersuperSupersuperSupersuperSuperLong.exampleOfOrderOfGetterAnd
 expect(
   findDOMNode(component.instance()).getElementsByClassName(styles.inner)[0].style.paddingRight
 ).toBe('1000px');
+
+const { course, conflicts = [], index, scheduleId, studentId, something } = a.this.props;
+
+const { course, conflicts = [], index, scheduleId, studentId, something } = this.props;
+
+const {
+  updated,
+  author: { identifier: ownerId },
+  location,
+  category: categories,
+} = rawAd.entry;