aboutsummaryrefslogtreecommitdiff
path: root/crates/ra_ide/src
diff options
context:
space:
mode:
authorMatt Niemeir <[email protected]>2020-03-10 02:18:55 +0000
committerMatt Niemeir <[email protected]>2020-03-10 03:14:17 +0000
commita9b6aec8a788d33ebe9667d53d127be59d93e555 (patch)
tree37a5e634178b35561851f79207f4045831dafca4 /crates/ra_ide/src
parent530ff9f57fd27505302d082a6007b49d8b98c8af (diff)
Struct field rename renames field in constructor field shorthand
Diffstat (limited to 'crates/ra_ide/src')
-rw-r--r--crates/ra_ide/src/references/rename.rs100
1 files changed, 79 insertions, 21 deletions
diff --git a/crates/ra_ide/src/references/rename.rs b/crates/ra_ide/src/references/rename.rs
index 5b4bcf434..794a109f3 100644
--- a/crates/ra_ide/src/references/rename.rs
+++ b/crates/ra_ide/src/references/rename.rs
@@ -9,7 +9,8 @@ use ra_syntax::{
9use ra_text_edit::TextEdit; 9use ra_text_edit::TextEdit;
10 10
11use crate::{ 11use crate::{
12 FileId, FilePosition, FileSystemEdit, RangeInfo, SourceChange, SourceFileEdit, TextRange, 12 FilePosition, FileSystemEdit, RangeInfo, Reference, ReferenceKind, SourceChange,
13 SourceFileEdit, TextRange,
13}; 14};
14 15
15use super::find_all_refs; 16use super::find_all_refs;
@@ -46,12 +47,20 @@ fn find_name_and_module_at_offset(
46 Some((ast_name, ast_module)) 47 Some((ast_name, ast_module))
47} 48}
48 49
49fn source_edit_from_file_id_range( 50fn source_edit_from_reference(reference: Reference, new_name: &str) -> SourceFileEdit {
50 file_id: FileId, 51 let mut replacement_text = String::from(new_name);
51 range: TextRange, 52 let file_id = reference.file_range.file_id;
52 new_name: &str, 53 let range = match reference.kind {
53) -> SourceFileEdit { 54 ReferenceKind::StructFieldShorthand => {
54 SourceFileEdit { file_id, edit: TextEdit::replace(range, new_name.into()) } 55 replacement_text.push_str(": ");
56 TextRange::from_to(
57 reference.file_range.range.start(),
58 reference.file_range.range.start(),
59 )
60 }
61 _ => reference.file_range.range,
62 };
63 SourceFileEdit { file_id, edit: TextEdit::replace(range, replacement_text) }
55} 64}
56 65
57fn rename_mod( 66fn rename_mod(
@@ -99,13 +108,10 @@ fn rename_mod(
99 source_file_edits.push(edit); 108 source_file_edits.push(edit);
100 109
101 if let Some(RangeInfo { range: _, info: refs }) = find_all_refs(sema.db, position, None) { 110 if let Some(RangeInfo { range: _, info: refs }) = find_all_refs(sema.db, position, None) {
102 let ref_edits = refs.references.into_iter().map(|reference| { 111 let ref_edits = refs
103 source_edit_from_file_id_range( 112 .references
104 reference.file_range.file_id, 113 .into_iter()
105 reference.file_range.range, 114 .map(|reference| source_edit_from_reference(reference, new_name));
106 new_name,
107 )
108 });
109 source_file_edits.extend(ref_edits); 115 source_file_edits.extend(ref_edits);
110 } 116 }
111 117
@@ -121,13 +127,7 @@ fn rename_reference(
121 127
122 let edit = refs 128 let edit = refs
123 .into_iter() 129 .into_iter()
124 .map(|reference| { 130 .map(|reference| source_edit_from_reference(reference, new_name))
125 source_edit_from_file_id_range(
126 reference.file_range.file_id,
127 reference.file_range.range,
128 new_name,
129 )
130 })
131 .collect::<Vec<_>>(); 131 .collect::<Vec<_>>();
132 132
133 if edit.is_empty() { 133 if edit.is_empty() {
@@ -286,6 +286,64 @@ mod tests {
286 } 286 }
287 287
288 #[test] 288 #[test]
289 fn test_rename_for_struct_field() {
290 test_rename(
291 r#"
292 struct Foo {
293 i<|>: i32,
294 }
295
296 impl Foo {
297 fn new(i: i32) -> Self {
298 Self { i: i }
299 }
300 }
301 "#,
302 "j",
303 r#"
304 struct Foo {
305 j: i32,
306 }
307
308 impl Foo {
309 fn new(i: i32) -> Self {
310 Self { j: i }
311 }
312 }
313 "#,
314 );
315 }
316
317 #[test]
318 fn test_rename_for_struct_field_shorthand() {
319 test_rename(
320 r#"
321 struct Foo {
322 i<|>: i32,
323 }
324
325 impl Foo {
326 fn new(i: i32) -> Self {
327 Self { i }
328 }
329 }
330 "#,
331 "j",
332 r#"
333 struct Foo {
334 j: i32,
335 }
336
337 impl Foo {
338 fn new(i: i32) -> Self {
339 Self { j: i }
340 }
341 }
342 "#,
343 );
344 }
345
346 #[test]
289 fn test_rename_mod() { 347 fn test_rename_mod() {
290 let (analysis, position) = analysis_and_position( 348 let (analysis, position) = analysis_and_position(
291 " 349 "