Thom LambandClaude Opus 4.7
c06ab2ea29
Rules/Rule/Groups/With.GetExpressions only called base.GetExpressions(). The comment 'do some ordering here??' indicates the override is a TODO stub. Drop the override and preserve the intent as an inline TODO on the class. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
27 lines
1.0 KiB
C#
27 lines
1.0 KiB
C#
using Strata.SqlTools.Rules.Rule.Expression;
|
|
|
|
namespace Strata.SqlTools.Rules.Rule.Groups;
|
|
|
|
/// <summary>
|
|
/// Represents a rule group with sequential rule evaluation (WITH semantics).
|
|
/// </summary>
|
|
public class With : Base
|
|
{
|
|
// TODO: revisit whether ordering should be applied here before delegating
|
|
// to the base GetExpressions(); inherit base behavior for now.
|
|
|
|
/// <summary>
|
|
/// Merges two BoolExpr expressions using WITH semantics.
|
|
/// </summary>
|
|
/// <param name="left">The left BoolExpr expression.</param>
|
|
/// <param name="right">The right BoolExpr expression.</param>
|
|
/// <returns>A WITH expression combining both expressions.</returns>
|
|
protected override BoolExpr Merge(BoolExpr left, BoolExpr right) => new Expression.With(left, right);
|
|
|
|
/// <summary>
|
|
/// Initializes a new instance of the <see cref="With"/> class.
|
|
/// </summary>
|
|
/// <param name="rules">The collection of rules to include in this WITH group.</param>
|
|
public With(IEnumerable<IRule> rules) : base(rules) { }
|
|
}
|