"Insert explicit modifiers on methods" overrides "Place single line method braces on separate lines"

[TimothyMakkison]

Environment

• Visual Studio version: 2022 Community
• CodeMaid version: 12.0.300
• Code language: C#

Description

Insert explicit modifiers on methods overrides update single line methods by placing them on new lines

Steps to recreate

1. Enable Insert explicit modifiers on methods and update single line methods by placing them on new lines.
2. Cleanup code with a single line method. ie:

   public class MyClass
   {
   public void Do() { Console.WriteLine("Hello World"); }
   }

   // Becomes 
   public void Do() 
   { Console.WriteLine("Hello World"); }
   // Instead of 
   public void Do() 
   { 
   Console.WriteLine("Hello World"); 
   }

Current behavior

Method body is moved to a new line without placing braces on separate lines.

Expected behavior

Braces should be placed on separate lines.

Solution

InvokeUpdateSingleLineMethods before InsertExplicitAccessModifiersOnMethods:

...
_insertExplicitAccessModifierLogic.InsertExplicitAccessModifiersOnStructs(structs);

_updateLogic.UpdateSingleLineMethods(methods);

_insertExplicitAccessModifierLogic.InsertExplicitAccessModifiersOnMethods(methods);
...

[codecadwallader]

Thanks for reporting the issue. I believe similar to the previous issue (#983) the root cause of why it is getting split onto a separate line may be related to the VS SDK issue (see #879 for details).