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

ZCode 3.14.3 に対応するオープン

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

supermomonga/zcode-acp#24 · 2026年9月23日に作成
create_predictable_zip member order differs on Windows for directory input オープン

bug

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

learningequality/ricecooker#747 · 2026年9月23日に作成
Cannot export point cloud without radius attribute オープン

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

KhronosGroup/glTF-Blender-IO#2769 · 2026年9月23日に作成
Cline agent sends legacy thinking.type "enabled" for claude-sonnet-5 (rejected) and claude-fable-5-1: bundled @clinebot/llms 0.0.38 predates cline/cline#12908 オープン

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

cline/kanban#655 · コメント 1 件 · 2026年9月22日に作成
ST04 fix deletes the space before the outer END when flattening a nested CASE オープン

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

sqlfluff/sqlfluff#8567 · 2026年9月22日に作成

DevTools の issue をもっと見る