Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Add support for set = ? for parser#499

Closed
BackEndTea wants to merge 1 commit intophpmyadmin:masterphpmyadmin/sql-parser:masterfrom
BackEndTea:fix/set-parameterBackEndTea/sql-parser:fix/set-parameterCopy head branch name to clipboard
Closed

Add support for set = ? for parser#499
BackEndTea wants to merge 1 commit intophpmyadmin:masterphpmyadmin/sql-parser:masterfrom
BackEndTea:fix/set-parameterBackEndTea/sql-parser:fix/set-parameterCopy head branch name to clipboard

Conversation

@BackEndTea
Copy link
Contributor

Fixes: #492

tests/TestCase.php Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Aug 17, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (d70e65d) 96.61% compared to head (13938da) 96.61%.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #499   +/-   ##
=========================================
  Coverage     96.61%   96.61%           
- Complexity     2173     2174    +1     
=========================================
  Files            66       66           
  Lines          5054     5058    +4     
=========================================
+ Hits           4883     4887    +4     
  Misses          171      171           
Files Changed Coverage Δ
src/Components/SetOperation.php 100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

src/Components/SetOperation.php Show resolved Hide resolved
MauricioFauth added a commit to MauricioFauth/sql-parser that referenced this pull request Sep 26, 2023
- Fixes phpmyadmin#202
- Fixes phpmyadmin#412
- Fixes phpmyadmin#492
- Closes phpmyadmin#499
- Related to phpmyadmin#172

Signed-off-by: Maurício Meneghini Fauth <mauricio@fauth.dev>
@MauricioFauth MauricioFauth self-assigned this Sep 27, 2023
@MauricioFauth
Copy link
Member

Thank you for your contribution. I fixed the issue in the Expression class when I was investigating another issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Missing support for SET = ?

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.