Issue 23091. Fix for adding import prefix before string-interpolated identifier.

pkg/analysis_server/lib/src/services/refactoring/rename_import.dart

 // Copyright (c) 2014, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 library services.src.refactoring.rename_import;
 
 import 'dart:async';
 
 import 'package:analysis_server/src/protocol_server.dart';
 import 'package:analysis_server/src/services/correction/source_range.dart';
 import 'package:analysis_server/src/services/correction/status.dart';
 import 'package:analysis_server/src/services/refactoring/naming_conventions.dart';
 import 'package:analysis_server/src/services/refactoring/refactoring.dart';
 import 'package:analysis_server/src/services/refactoring/refactoring_internal.dart';
 import 'package:analysis_server/src/services/refactoring/rename.dart';
 import 'package:analysis_server/src/services/search/search_engine.dart';
+import 'package:analyzer/src/generated/ast.dart';
 import 'package:analyzer/src/generated/element.dart';
 import 'package:analyzer/src/generated/source.dart';
 
 /**
  * A [Refactoring] for renaming [ImportElement]s.
  */
 class RenameImportRefactoringImpl extends RenameRefactoringImpl {
   RenameImportRefactoringImpl(SearchEngine searchEngine, ImportElement element)
       : super(searchEngine, element);
 
@@ skipping 42 matching lines @@
       }
       if (edit != null) {
         doSourceChange_addElementEdit(change, element, edit);
       }
     }
     // update references
     List<SearchMatch> matches = await searchEngine.searchReferences(element);
     List<SourceReference> references = getSourceReferences(matches);
     for (SourceReference reference in references) {
       if (newName.isEmpty) {
-        reference.addEdit(change, newName);
+        reference.addEdit(change, '');
       } else {
-        reference.addEdit(change, "${newName}.");
+        SimpleIdentifier interpolationIdentifier =
+            _getInterpolationIdentifier(reference);
+        if (interpolationIdentifier != null) {
+          doSourceChange_addElementEdit(change, reference.element,
+              new SourceEdit(interpolationIdentifier.offset,
+                  interpolationIdentifier.length,
+                  '{$newName.${interpolationIdentifier.name}}'));
+        } else {
+          reference.addEdit(change, '$newName.');
+        }
       }
     }
   }
+
+  /**
+   * If the given [reference] is before an [SimpleIdentifier] in an
+   * [InterpolationElement], return it. Otherwise return `null`.
+   */
+  SimpleIdentifier _getInterpolationIdentifier(SourceReference reference) {
+    Source source = reference.element.source;
+    CompilationUnit unit = context.parseCompilationUnit(source);
+    NodeLocator nodeLocator = new NodeLocator.con1(reference.range.offset);
+    AstNode node = nodeLocator.searchWithin(unit);
+    if (node is SimpleIdentifier && node.parent is InterpolationExpression) {

[Paul Berry, 2015/04/03 21:25:27]

I don't think this is quite right.  This if-test will pass both for the case of
renaming $x and for renaming ${x}.  Which means that ${x} will get incorrectly
renamed to ${{prefix.x}}.

[scheglov, 2015/04/03 21:33:28]

Fixed.

+      return node;
+    }
+    return null;
+  }
 }