ExplicitVisit method for some TSqlFragment descendant classes is never called

未关闭

#145 创建于 2025年7月10日 0 条评论 0 个 reaction 已指派 0 人在 GitHub 查看

还没有人认领这个 Issue。

评估

难度: 4/5
预计耗时: 3-5 天
新手友好度: 35/100

Issue 类型: 缺陷
描述清晰度: 基本清楚
活跃度: 停滞
技术栈: csharp, sql
领域: devtools

调研方向

从 TSqlFragmentVisitor 及其 ExplicitVisit 方法开始，然后将 DataModificationStatement 的分派与 InsertStatement 等具体派生类进行比较。检查 Visit 和 TSqlConcreteFragmentVisitor 的设计意图有何不同。完成的标准是解决抽象类的分派行为，并记录应在何时使用 ExplicitVisit、Visit 和 TSqlConcreteFragmentVisitor。

由索引模型根据 Issue 内容生成。

描述

For example, TSqlFragmentVisitor has virtual method for visiting DataModificationStatement - ExplicitVisit(DataModificationStatement) which can be overridden and "used" for whatever reason in an app... The thing is - it is never called. There is no code invoking ExplicitVisit for this abstract class and many others. These methods are stillborn and it takes some time to realize that the code does not work and to discover why. For concrete descendants of DataModificationStatement (InsertStatement etc.) it does work as expected.

Consider removing such stillborn methods from TSqlFragmentVisitor or mark them sealed or start invoking them.

Also it'd be great to improve documentation: please elaborate scenarios in which ExplicitVisit method should be used instead of Visit, when TSqlConcreteFragmentVisitor should be utilized.

主要语言: GAP
星标: 277
派生: 43
平均合并: 6 天 17 小时
30 天内合并 PR: 3

贡献指南

打开贡献指南

从这里开始

先读完整个 Issue，再读项目的贡献指南。
在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
Fork 仓库，在一个分支上完成修改。
提交 Pull Request，并在描述里引用这个 Issue 编号。

microsoft/SqlScriptDOM 的其他 Issue

Add a right-aligned clause keyword ("river") value to ClauseBodyAlignment 未关闭

难度 2/5 1-3 小时新手友好度 84/100

microsoft/SqlScriptDOM#228 · 2026年8月22日创建
NotLike should not be a BooleanComparisonType 未关闭

难度 2/5 1-3 小时新手友好度 62/100

microsoft/SqlScriptDOM#183 · 2025年11月14日创建
Add a Multiline option for CASE expressions (WHEN/THEN/ELSE on their own lines) 未关闭

难度 4/5 3-5 天新手友好度 56/100

microsoft/SqlScriptDOM#226 · 1 个 reaction · 2026年8月20日创建
PreserveComments: trailing single-line comments are re-anchored to a different node, and output is not idempotent 未关闭

难度 4/5 3-5 天新手友好度 48/100

microsoft/SqlScriptDOM#225 · 2026年8月20日创建
CAST function parameter fragment is incorrectly calculated during parsing 未关闭

难度 3/5 1-2 天新手友好度 58/100

microsoft/SqlScriptDOM#224 · 2026年8月20日创建

查看 microsoft/SqlScriptDOM 的全部 Issue

相似的 Issue

ls -i / --inode not supported ("invalid option -- i"), breaks autoconf's `ls -di .` working-directory check 未关闭

难度 2/5 1-3 小时新手友好度 75/100

vercel-labs/just-bash#464 · 2026年9月24日创建
Row-action modals break when core drops the kebabCase private API 未关闭

Bug

难度 2/5 1-3 小时新手友好度 70/100

Automattic/safe-publish#594 · 2026年9月24日创建
The `.vibe` directory is listed under `allow_file` in the `mistral-vibe` profile 未关闭

bug

难度 1/5 1 小时以内新手友好度 85/100

nolabs-ai/nono#1988 · 3 条评论 · 2026年9月24日创建
🎙️ task - fix(tests-node): the shipped catch ships cause error where error is unknown 未关闭

难度 2/5 1-3 小时新手友好度 65/100

ehmpathy/declapract-typescript-ehmpathy#632 · 2026年9月24日创建
fix: EvmAbi and Eip712 property specs outrun the live test timeout 未关闭

bug feat:crypto feat:utils

难度 2/5 1-3 小时新手友好度 70/100

OpenZeppelin/compact-contracts#952 · 2026年9月24日创建

更多 DevTools Issue

精选适合新手参与的 GitHub issue 摘要。