codemaid icon indicating copy to clipboard operation
codemaid copied to clipboard

partial class XXX

Open johnsunabc opened this issue 1 year ago • 1 comments

Environment

  • Visual Studio version: [example 2022 Community]
  • CodeMaid version: [example 12.0]
  • Code language: [example C#]

Description

When a class has many partial sub-classes, such as A.cs, A.Drawing.cs, A.Tool.cs, the class like that: A.cs: public sealed partial class A A.Drawing.cs: partial class A A.Tool.cs: partial class A

Steps to recreate

  1. Class A has three files: // A.cs public sealed partial class A { // Main... }

// A.Drawing.cs partial class A { // Sub for drawing ... }

// A.Tool.cs partial class A { // Sub tool for drawing and others ... }

  1. After used CodeMaid: // A.cs public sealed partial class A { // Main... }

// A.Drawing.cs internal partial class A { // Sub for drawing ... }

// A.Tool.cs internal partial class A { // Sub tool for drawing and others ... }

Current behavior

  1. Main part is using "public sealed ...", but sub parts using "internal ".

Expected behavior

All sub parts should be: "public sealed partial class (ClassName)".

johnsunabc avatar Apr 14 '23 06:04 johnsunabc

Thanks for reporting the issue. We have logic that should(tm) prevent access modifiers from being added to partial classes (link: https://github.com/codecadwallader/codemaid/blob/dev/CodeMaidShared/Logic/Cleaning/InsertExplicitAccessModifierLogic.cs#L61) but I believe this may be another symptom of a known SDK bug (see issue #879 for details).

codecadwallader avatar Apr 29 '23 19:04 codecadwallader