And rename length check

Hosch250 · Hosch250 · commit 660ebe81cfc2 · 2017-10-07T10:29:03.000-05:00
diff --git a/RetailCoder.VBE/UI/Refactorings/Rename/RenameViewModel.cs b/RetailCoder.VBE/UI/Refactorings/Rename/RenameViewModel.cs
@@ -72,7 +72,8 @@ public bool IsValidName
                 return !(NewName.Equals(Target.IdentifierName, StringComparison.InvariantCultureIgnoreCase)) &&
                        char.IsLetter(NewName.FirstOrDefault()) &&
                        !tokenValues.Contains(NewName, StringComparer.InvariantCultureIgnoreCase) &&
-                       !NewName.Any(c => !char.IsLetterOrDigit(c) && c != '_');
+                       !NewName.Any(c => !char.IsLetterOrDigit(c) && c != '_') &&
+                       NewName.Length <= (Target.DeclarationType.HasFlag(DeclarationType.Module) ? 31 : 255);
             }
         }
 

Original file line number	Diff line number	Diff line change
`@@ -72,7 +72,8 @@ public bool IsValidName`
`72`	`72`	`return !(NewName.Equals(Target.IdentifierName, StringComparison.InvariantCultureIgnoreCase)) &&`
`73`	`73`	`char.IsLetter(NewName.FirstOrDefault()) &&`
`74`	`74`	`!tokenValues.Contains(NewName, StringComparer.InvariantCultureIgnoreCase) &&`
`75`		`- !NewName.Any(c => !char.IsLetterOrDigit(c) && c != '_');`
	`75`	`+ !NewName.Any(c => !char.IsLetterOrDigit(c) && c != '_') &&`
	`76`	`+ NewName.Length <= (Target.DeclarationType.HasFlag(DeclarationType.Module) ? 31 : 255);`
`76`	`77`	`}`
`77`	`78`	`}`
`78`	`79`