diff options
author | Mike Krüger <mkrueger@xamarin.com> | 2014-02-12 12:25:41 +0400 |
---|---|---|
committer | Mike Krüger <mkrueger@xamarin.com> | 2014-02-12 12:25:41 +0400 |
commit | 1403af7038e52d4a5ab6a3156d9b5fed363e8f80 (patch) | |
tree | fd28d2dd43ddc5fe8ed8d44d64f5d741a9a7309f /ICSharpCode.NRefactory.CSharp.Refactoring | |
parent | acca0961c1c9f1ae3b70ba8d9b0da783e28fb63a (diff) |
Added 'CS1105ExtensionMethodMustBeDeclaredStaticAction'.
Diffstat (limited to 'ICSharpCode.NRefactory.CSharp.Refactoring')
2 files changed, 58 insertions, 0 deletions
diff --git a/ICSharpCode.NRefactory.CSharp.Refactoring/CodeActions/CS1105ExtensionMethodMustBeDeclaredStaticAction.cs b/ICSharpCode.NRefactory.CSharp.Refactoring/CodeActions/CS1105ExtensionMethodMustBeDeclaredStaticAction.cs new file mode 100644 index 00000000..ad6936dc --- /dev/null +++ b/ICSharpCode.NRefactory.CSharp.Refactoring/CodeActions/CS1105ExtensionMethodMustBeDeclaredStaticAction.cs @@ -0,0 +1,57 @@ +// +// CS1105ExtensionMethodMustBeDeclaredStaticAction.cs +// +// Author: +// Mike Krüger <mkrueger@xamarin.com> +// +// Copyright (c) 2014 Xamarin Inc. (http://xamarin.com) +// +// Permission is hereby granted, free of charge, to any person obtaining a copy +// of this software and associated documentation files (the "Software"), to deal +// in the Software without restriction, including without limitation the rights +// to use, copy, modify, merge, publish, distribute, sublicense, and/or sell +// copies of the Software, and to permit persons to whom the Software is +// furnished to do so, subject to the following conditions: +// +// The above copyright notice and this permission notice shall be included in +// all copies or substantial portions of the Software. +// +// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR +// IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, +// FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE +// AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER +// LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, +// OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN +// THE SOFTWARE. +using System; +using ICSharpCode.NRefactory.TypeSystem; +using System.Threading; +using System.Collections.Generic; +using System.Linq; + +namespace ICSharpCode.NRefactory.CSharp.Refactoring +{ + [ContextAction("Extension methods must be declared static")] + public class CS1105ExtensionMethodMustBeDeclaredStaticAction : CodeActionProvider + { + public override IEnumerable<CodeAction> GetActions(RefactoringContext context) + { + var method = context.GetNode<MethodDeclaration>(); + if (method == null || !method.NameToken.Contains(context.Location)) + yield break; + + if (method.HasModifier(Modifiers.Static)) + yield break; + var param = method.Parameters.FirstOrDefault(); + if (param == null || param.ParameterModifier != ParameterModifier.This) + yield break; + yield return new CodeAction( + context.TranslateString("Convert method to static"), + script => script.ChangeModifier(method, method.Modifiers | Modifiers.Static), + method) { + Severity = ICSharpCode.NRefactory.Refactoring.Severity.Error + }; + } + } +} + diff --git a/ICSharpCode.NRefactory.CSharp.Refactoring/ICSharpCode.NRefactory.CSharp.Refactoring.csproj b/ICSharpCode.NRefactory.CSharp.Refactoring/ICSharpCode.NRefactory.CSharp.Refactoring.csproj index e6450321..97115ff6 100644 --- a/ICSharpCode.NRefactory.CSharp.Refactoring/ICSharpCode.NRefactory.CSharp.Refactoring.csproj +++ b/ICSharpCode.NRefactory.CSharp.Refactoring/ICSharpCode.NRefactory.CSharp.Refactoring.csproj @@ -427,6 +427,7 @@ <Compile Include="CodeIssues\Custom\CompilerErrors\CS0618UsageOfObsoleteMemberIssue.cs" /> <Compile Include="CodeIssues\Custom\CompilerErrors\CS0169FieldIsNeverUsedIssue.cs" /> <Compile Include="CodeIssues\Custom\StaticEventSubscriptionIssue.cs" /> + <Compile Include="CodeActions\CS1105ExtensionMethodMustBeDeclaredStaticAction.cs" /> </ItemGroup> <Import Project="$(MSBuildBinPath)\Microsoft.CSharp.targets" /> <ItemGroup> |