Use correct-case table name in message for successful rename #12914

Merged
merged 1 commit into from Jan 23, 2017

Projects

None yet

3 participants

@MPLew-is
Contributor

Previously, the name returned was the name used in the request, rather than the name after the correcting for the server’s lower_case_table_names setting.

Following up on issue #12901 and @b2e2e77.

Signed-off-by: Mike Lewis mike@mplew.is

Before submitting pull request, please check that every commit:

  • Has proper Signed-Off-By
  • Has commit message which describes it
  • Is needed on it's own, if you have just minor fixes to previous commits, you can squash them
  • Any new functionality is covered by tests
@MPLew-is MPLew-is Use correct-case table name in message for successful rename
Previously, the name returned was the name used in the request, rather
than the name after the correcting for the server’s
`lower_case_table_names` setting.

Following up on issue #12901 and @b2e2e77.

Signed-off-by: Mike Lewis <mike@mplew.is>
52fa45e
@codecov-io
codecov-io commented Jan 21, 2017 edited

Current coverage is 54.18% (diff: 0.00%)

Merging #12914 into master will decrease coverage by <.01%

@@             master     #12914   diff @@
==========================================
  Files           466        466          
  Lines         69633      69636     +3   
  Methods        2126       2126          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits          37734      37734          
- Misses        31899      31902     +3   
  Partials          0          0          

Powered by Codecov. Last update b2e2e77...52fa45e

@nijel nijel self-assigned this Jan 23, 2017
@nijel nijel merged commit 989a34d into phpmyadmin:master Jan 23, 2017

2 of 4 checks passed

codecov/patch 0.00% of diff hit (target 54.18%)
Details
codecov/project 54.18% (-0.01%) compared to b2e2e77
Details
Scrutinizer No new issues
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@nijel
Member
nijel commented Jan 23, 2017

Merged, thanks for your contribution!

@nijel nijel added this to the 4.7.0 milestone Jan 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment