René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Is `VisitBaseType` internal field used consistently?","articleBody":"While diving deep into debugging my app I accidentally have noticed this: TSqlFragmentVisitor final code contains many blocks which check of `VisitBaseType` internal field value and if it is `true` then Visit method for base types is called, but for some sql-fragment classes Visit method implementation has this check negated and the behavior is reversed:\n\n```csharp\n    // Summary:\n    //     Visitor for DeclareCursorStatement\n    public virtual void Visit(DeclareCursorStatement node)\n    {\n        if (!VisitBaseType)     \u003c\u003c\u003c--- here\n        {\n            Visit((TSqlFragment)node);\n        }\n    }\n\n    //\n    // Summary:\n    //     Explicit Visitor for DeclareCursorStatement\n    public virtual void ExplicitVisit(DeclareCursorStatement node)\n    {\n        if (VisitBaseType)\n        {\n            Visit((TSqlStatement)node);\n            Visit((TSqlFragment)node);\n        }\n\n        Visit(node);\n        node.AcceptChildren(this);\n    }\n```\n\nIs this expected behavior? If so, please clarify what was the intent, how `VisitBaseType` should be understood.","author":{"url":"https://github.com/IVNSTN","@type":"Person","name":"IVNSTN"},"datePublished":"2025-07-03T08:29:49.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/144/SqlScriptDOM/issues/144"}