Skip to content

fix(LC0086): suppress false positives on data-access method invocations#172

Merged
Arthurvdv merged 1 commit intorelease/v0.6.1from
development
Apr 1, 2026
Merged

fix(LC0086): suppress false positives on data-access method invocations#172
Arthurvdv merged 1 commit intorelease/v0.6.1from
development

Conversation

@Arthurvdv
Copy link
Copy Markdown
Member

String literals matching PageStyle enum values (e.g., 'None', 'Standard') inside method invocations on data-access types no longer trigger LC0086. This fixes false positives like Rec.SetRange(MyField, 'NONE') where the string is a data filter value, not a page style.

Suppressed types: Record, RecordRef, FieldRef, Query, Variant, DataTransfer, FilterPageBuilder, TableFilter.

Contexts that still flag (by design): if-comparisons, exit statements, assignments to text variables, custom procedure arguments, Message calls.

Added FieldRef, DataTransfer, FilterPageBuilder to EnumProvider.NavTypeKind. Added 5 new test cases (2 HasDiagnostic, 3 NoDiagnostic).

Closes #154

String literals matching PageStyle enum values (e.g., 'None', 'Standard')
inside method invocations on data-access types no longer trigger LC0086.
This fixes false positives like Rec.SetRange(MyField, 'NONE') where the
string is a data filter value, not a page style.

Suppressed types: Record, RecordRef, FieldRef, Query, Variant,
DataTransfer, FilterPageBuilder, TableFilter.

Contexts that still flag (by design): if-comparisons, exit statements,
assignments to text variables, custom procedure arguments, Message calls.

Added FieldRef, DataTransfer, FilterPageBuilder to EnumProvider.NavTypeKind.
Added 5 new test cases (2 HasDiagnostic, 3 NoDiagnostic).

Closes #154

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@Arthurvdv Arthurvdv merged commit 465e212 into release/v0.6.1 Apr 1, 2026
40 checks passed
@Arthurvdv Arthurvdv deleted the development branch April 1, 2026 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant