ExplicitVisit method for some TSqlFragment descendant classes is never called

オープン

#145 2025年7月10日に作成コメント 0 件リアクション 0 件担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度: 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時間
マージ済み PR（30日）: 3

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
リポジトリをフォークし、ブランチを切って変更します。
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 件 · 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

P4 FR: Add one-click copy-to-clipboard button to code snippets in static workshop HTML オープン

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

palladius/rails8-app-on-gcp#145 · 2026年9月23日に作成
wrapper-provision-jdk: require curl only when downloading a JDK オープン

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

elastic/gradle-plugins#156 · 2026年9月23日に作成
fix: in-app bug reports carry the user-report label — ADR 0143's P1 door never fires オープン

area:workflow bug ready-for-agent

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

fil-donadoni/tolaria#4409 · 2026年9月23日に作成
Disable auto-cloaking not working オープン

難易度 2/5 1〜3時間初心者へのやさしさ 65/100

dotenvx/dotenv-vscode#139 · 2026年9月23日に作成
Add GigaCode as a supported --tools target オープン

難易度 2/5 1〜3時間初心者へのやさしさ 70/100

Fission-AI/OpenSpec#1960 · 2026年9月23日に作成

DevTools の issue をもっと見る