Identifiers in function calls with an object are now processed

src/nodes/FunctionCall.js

@@ -6,18 +6,21 @@ const {
 
 const printSeparatedList = require('./print-separated-list');
 
-const printObject = (node, path, print, options) => [
-  '{',
-  printSeparatedList(
-    path
-      .map(print, 'arguments')
-      .map((arg, index) => [node.names[index], ': ', arg]),
-    {
-      firstSeparator: options.bracketSpacing ? line : softline,
-      lastSeparator: [options.bracketSpacing ? line : softline, '})']
-    }
-  )
-];
+const printObject = (path, print, options) => {
+  const identifiers = path.map(print, 'identifiers');
+  return [
+    '{',
+    printSeparatedList(
+      path
+        .map(print, 'arguments')
+        .map((arg, index) => [identifiers[index], ': ', arg]),
+      {
+        firstSeparator: options.bracketSpacing ? line : softline,
+        lastSeparator: [options.bracketSpacing ? line : softline, '})']
+      }
+    )
+  ];
+};
 
 const printArguments = (path, print) =>
   printSeparatedList(path.map(print, 'arguments'), {
@@ -31,8 +34,8 @@ const FunctionCall = {
     let argumentsDoc = ')';
 
     if (node.arguments && node.arguments.length > 0) {
-      if (node.names && node.names.length > 0) {
-        argumentsDoc = printObject(node, path, print, options);
+      if (node.identifiers && node.identifiers.length > 0) {
+        argumentsDoc = printObject(path, print, options);
       } else {
         argumentsDoc = printArguments(path, print);
       }

tests/config/format-test.js

@@ -22,7 +22,7 @@ const RANGE_END_PLACEHOLDER = "<<<PRETTIER_RANGE_END>>>";
 
 // Here we add files that will not be the same when formatting a second time.
 const unstableTests = new Map(
-  [].map((fixture) => {
+  ["Comments/Comments.sol"].map((fixture) => {
     const [file, isUnstable = () => true] = Array.isArray(fixture)
       ? fixture
       : [fixture];

tests/format/Comments/Comments.sol

@@ -102,3 +102,15 @@ contract Comments11 {
     // this should not be removed
   }
 }
+
+contract Comments12 {
+  function f() public {
+    purchaseData[0] = DomainPurchaseData({
+      /* test */prices: _rootPrices,
+      // test2
+      subdomainMintingEnabled: /* test3 */_rootPrices.short > 0,
+      allowSubdomainsToMint: true, // test4
+      wasAllowedToSubdomainMintOnCreation: true
+    });
+  }
+}

tests/format/Comments/__snapshots__/jsfmt.spec.js.snap

@@ -111,6 +111,18 @@ contract Comments11 {
   }
 }
 
+contract Comments12 {
+  function f() public {
+    purchaseData[0] = DomainPurchaseData({
+      /* test */prices: _rootPrices,
+      // test2
+      subdomainMintingEnabled: /* test3 */_rootPrices.short > 0,
+      allowSubdomainsToMint: true, // test4
+      wasAllowedToSubdomainMintOnCreation: true
+    });
+  }
+}
+
 =====================================output=====================================
 pragma solidity ^0.4.24;
 
@@ -240,5 +252,19 @@ contract Comments11 {
     }
 }
 
+contract Comments12 {
+    function f() public {
+        purchaseData[0] = DomainPurchaseData({
+            /* test */
+            prices: _rootPrices,
+            // test2
+            subdomainMintingEnabled: /* test3 */
+            _rootPrices.short > 0,
+            allowSubdomainsToMint: true, // test4
+            wasAllowedToSubdomainMintOnCreation: true
+        });
+    }
+}
+
 ================================================================================
 `;