Skip to content

fix(flow-php/postgresql): parse every condition node inside AND, OR and NOT - #2727

Merged
norberttech merged 1 commit into
1.xfrom
nested-condition-nodes
Nov 5, 2026
Merged

norberttech merged 1 commit into
1.xfrom
nested-condition-nodes

Conversation

@norberttech

Copy link
Copy Markdown
Member

Resolves: #xxx

Change Log


Added

Fixed

  • flow-php/postgresql - AND, OR and NOT conditions parse IS NULL, IN, bare booleans, operators, function calls and scalar subqueries
  • flow-php/postgresql - BETWEEN SYMMETRIC conditions parse from the AST
  • flow-php/postgresql - rebuilt BETWEEN keeps NOT and SYMMETRIC
  • flow-php/postgresql - between(not: true) produces NOT BETWEEN

Changed

  • flow-php/postgresql - unsupported condition node errors name the node kind

Removed

Deprecated

Security

…nd NOT

- AND/OR/NOT children parse through ConditionFactory (IS NULL, IN, bare booleans, operators, function calls, scalar subqueries)
- BETWEEN SYMMETRIC parses at the top level
- rebuilt BETWEEN keeps NOT and SYMMETRIC
- between(not: true) produces NOT BETWEEN
- unsupported-node errors name the node kind
@norberttech norberttech added this to the 0.48.0 milestone Nov 5, 2026
@codecov

codecov Bot commented Nov 5, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.77%. Comparing base (74b3a71) to head (05bbb8a).
⚠️ Report is 2 commits behind head on 1.x.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@             Coverage Diff              @@
##                1.x    #2727      +/-   ##
============================================
+ Coverage     90.60%   90.77%   +0.16%     
+ Complexity    31434    31356      -78     
============================================
  Files          2227     2227              
  Lines         90696    90563     -133     
============================================
+ Hits          82175    82208      +33     
+ Misses         8521     8355     -166     
Components Coverage Δ
etl 95.99% <ø> (ø)
cli 87.24% <ø> (ø)
lib-array-dot 96.52% <ø> (ø)
lib-azure-sdk 64.17% <ø> (ø)
lib-doctrine-dbal-bulk 94.25% <ø> (ø)
lib-filesystem 89.16% <ø> (ø)
lib-types 94.64% <ø> (ø)
lib-parquet 82.80% <ø> (ø)
lib-parquet-viewer 83.15% <ø> (ø)
lib-snappy 89.86% <ø> (ø)
lib-dremel 0.00% <ø> (ø)
lib-postgresql 90.15% <100.00%> (+0.71%) ⬆️
lib-telemetry 86.85% <ø> (ø)
bridge-filesystem-async-aws 96.12% <ø> (ø)
bridge-filesystem-azure 94.39% <ø> (ø)
bridge-filesystem-sftp 90.29% <ø> (ø)
bridge-monolog-http 96.84% <ø> (ø)
bridge-monolog-telemetry 92.85% <ø> (ø)
bridge-openapi-specification 91.95% <ø> (ø)
symfony-http-foundation 80.76% <ø> (ø)
bridge-psr18-telemetry 100.00% <ø> (ø)
bridge-psr3-telemetry 98.95% <ø> (ø)
bridge-psr7-telemetry 100.00% <ø> (ø)
bridge-telemetry-otlp 90.11% <ø> (ø)
bridge-symfony-http-foundation-telemetry 92.85% <ø> (ø)
bridge-symfony-filesystem-bundle 91.76% <ø> (ø)
bridge-symfony-filesystem-cache 98.18% <ø> (ø)
bridge-symfony-postgresql-bundle 93.35% <ø> (ø)
bridge-symfony-postgresql-cache 94.41% <ø> (ø)
bridge-symfony-postgresql-messenger 98.80% <ø> (ø)
bridge-symfony-postgresql-session 93.65% <ø> (ø)
bridge-symfony-telemetry-bundle 90.76% <ø> (ø)
adapter-chartjs 82.11% <ø> (ø)
adapter-csv 97.67% <ø> (ø)
adapter-doctrine 95.13% <ø> (ø)
adapter-google-sheet 97.49% <ø> (ø)
adapter-http 87.18% <ø> (ø)
adapter-json 90.59% <ø> (ø)
adapter-logger 47.05% <ø> (ø)
adapter-parquet 97.20% <ø> (ø)
adapter-text 97.68% <ø> (ø)
adapter-xml 98.01% <ø> (ø)
adapter-avro 6.25% <ø> (ø)
adapter-excel 96.77% <ø> (ø)
adapter-postgresql 97.34% <ø> (ø)
adapter-seal 89.91% <ø> (ø)
bridge-phpunit-postgresql 75.30% <ø> (ø)
bridge-phpunit-telemetry 87.36% <ø> (ø)
bridge-phpstan-types 0.00% <ø> (ø)
bridge-postgresql-valinor 100.00% <ø> (ø)
🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@norberttech
norberttech merged commit bbfa945 into 1.x Nov 5, 2026
21 checks passed
@norberttech
norberttech deleted the nested-condition-nodes branch November 5, 2026 18:13
@github-project-automation github-project-automation Bot moved this from Todo to Done in Roadmap Nov 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant