Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #374 - Adding NOT to the select options #388

Merged
merged 3 commits into from
Nov 23, 2022

Conversation

iifawzi
Copy link
Contributor

@iifawzi iifawzi commented Nov 21, 2022

Hi, This PR is fixing #374
Fixes #374

Signed-off-by: iifawzi <iifawzie@gmail.com>
Signed-off-by: iifawzi <iifawzie@gmail.com>
@codecov
Copy link

codecov bot commented Nov 21, 2022

Codecov Report

Base: 95.64% // Head: 95.64% // No change to project coverage 👍

Coverage data is based on head (0d4e00c) compared to base (8fddb4b).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #388   +/-   ##
=========================================
  Coverage     95.64%   95.64%           
  Complexity     2144     2144           
=========================================
  Files            67       67           
  Lines          4548     4548           
=========================================
  Hits           4350     4350           
  Misses          198      198           
Impacted Files Coverage Δ
src/Statements/SelectStatement.php 100.00% <ø> (ø)
src/Utils/Query.php 98.94% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Member

@williamdes williamdes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔥

@williamdes
Copy link
Member

Can we add the full query of #374 or is there some more things to fix ?

@iifawzi
Copy link
Contributor Author

iifawzi commented Nov 21, 2022

Can we add the full query of #374 or is there some more things to fix ?

Yes, we can add it for sure, there are no more things to fix, the issue was only because of the NOT keyword

Signed-off-by: iifawzi <iifawzie@gmail.com>
@williamdes williamdes self-assigned this Nov 23, 2022
@williamdes williamdes added this to the 5.6.0 milestone Nov 23, 2022
@williamdes williamdes merged commit eabf67e into phpmyadmin:master Nov 23, 2022
williamdes added a commit that referenced this pull request Nov 23, 2022
Pull-request: #388

Signed-off-by: William Desportes <williamdes@wdes.fr>
williamdes added a commit that referenced this pull request Nov 25, 2022
Pull-request: #391
Ref: #388

This fixes the first implementation

Signed-off-by: William Desportes <williamdes@wdes.fr>
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.

Lots of bogus errors in editor with perfectly valid query
2 participants