Skip to content

Comments

Fix column definition form not showing default value#16366

Merged
MauricioFauth merged 1 commit intophpmyadmin:QA_5_0from
MauricioFauth:column-default-issue
Sep 20, 2020
Merged

Fix column definition form not showing default value#16366
MauricioFauth merged 1 commit intophpmyadmin:QA_5_0from
MauricioFauth:column-default-issue

Conversation

@MauricioFauth
Copy link
Member

@MauricioFauth MauricioFauth commented Sep 20, 2020

To reproduce:
Add a column that has an user defined default value to the central columns, then try to bulk edit that central column. The input field for the default value will be empty.

Introduced by 5d0fd81.

@MauricioFauth MauricioFauth added the Bug A problem or regression with an existing feature label Sep 20, 2020
@codecov
Copy link

codecov bot commented Sep 20, 2020

Codecov Report

Merging #16366 into QA_5_0 will decrease coverage by 0.00%.
The diff coverage is 36.00%.

@@             Coverage Diff              @@
##             QA_5_0   #16366      +/-   ##
============================================
- Coverage     53.19%   53.19%   -0.01%     
- Complexity    14270    14278       +8     
============================================
  Files           482      482              
  Lines         62331    62347      +16     
============================================
+ Hits          33159    33167       +8     
- Misses        29172    29180       +8     

@williamdes williamdes self-assigned this Sep 20, 2020
@williamdes williamdes added this to the 5.0.3 milestone Sep 20, 2020
@williamdes williamdes removed their assignment Sep 20, 2020
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.

Thanks

What is the scenario to see this bug exists in case someone reports it?

@MauricioFauth
Copy link
Member Author

What is the scenario to see this bug exists in case someone reports it?

Add a column that has an user defined default value to the central columns, then try to bulk edit that central column. The input field for the default value will be empty.

Introduced by 5d0fd81

Signed-off-by: Maurício Meneghini Fauth <mauricio@fauth.dev>
@MauricioFauth MauricioFauth self-assigned this Sep 20, 2020
@MauricioFauth MauricioFauth merged commit a2bb7ff into phpmyadmin:QA_5_0 Sep 20, 2020
@MauricioFauth MauricioFauth deleted the column-default-issue branch September 20, 2020 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug A problem or regression with an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants