Skip to content

Fix #342 - Adding the support of altering user options#345

Merged
williamdes merged 1 commit into
phpmyadmin:QAfrom
iifawzi:supporting-user-alter
Aug 17, 2021
Merged

Fix #342 - Adding the support of altering user options#345
williamdes merged 1 commit into
phpmyadmin:QAfrom
iifawzi:supporting-user-alter

Conversation

@iifawzi
Copy link
Copy Markdown
Contributor

@iifawzi iifawzi commented Aug 17, 2021

Hi, this PR should add the support of altering user options, not all the options are included though, because I'm not aware of all of them, I've just introduced the base.

Fixes #342
Signed-off-by: Fawzi E. Abdulfattah iifawzie@gmail.com

@iifawzi iifawzi force-pushed the supporting-user-alter branch 4 times, most recently from 4e66de5 to 9e88376 Compare August 17, 2021 17:54
@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 17, 2021

Codecov Report

Merging #345 (2eff24a) into QA (f6723ea) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##                 QA     #345   +/-   ##
=========================================
  Coverage     99.80%   99.80%           
- Complexity     1915     1916    +1     
=========================================
  Files            63       63           
  Lines          4602     4604    +2     
=========================================
+ Hits           4593     4595    +2     
  Misses            9        9           
Impacted Files Coverage Δ
src/Components/AlterOperation.php 100.00% <ø> (ø)
src/Statements/AlterStatement.php 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f6723ea...2eff24a. Read the comment docs.

@iifawzi iifawzi changed the title Adding the support of altering user statements Adding the support of altering user options Aug 17, 2021
@williamdes williamdes added this to the 4.7.3 milestone Aug 17, 2021
Copy link
Copy Markdown
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.

This looks good, to better understand I will cut out and merge into QA and master your newly added tests

@williamdes williamdes self-assigned this Aug 17, 2021
williamdes pushed a commit that referenced this pull request Aug 17, 2021
Pull-request: #345

Signed-off-by: Fawzi E. Abdulfattah <iifawzie@gmail.com>
Copy link
Copy Markdown
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.

Awesome !
I really like the diff it makes on master branch out files

Would you mind rebasing please ?

Comment thread src/Components/AlterOperation.php
@iifawzi iifawzi force-pushed the supporting-user-alter branch from 9e88376 to 15d8e9b Compare August 17, 2021 20:56
Signed-off-by: Fawzi E. Abdulfattah <iifawzie@gmail.com>
@iifawzi iifawzi force-pushed the supporting-user-alter branch from 15d8e9b to 2eff24a Compare August 17, 2021 21:22
Comment thread src/Components/AlterOperation.php
Copy link
Copy Markdown
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 williamdes changed the title Adding the support of altering user options Fix #342 - Adding the support of altering user options Aug 17, 2021
@williamdes williamdes merged commit a20e660 into phpmyadmin:QA Aug 17, 2021
williamdes added a commit that referenced this pull request Aug 17, 2021
Pull-request: #345

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.

2 participants