[go: nahoru, domu]

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: PHP Fatal error: Uncaught Error: Call to a member function isAjax() on null #13157

Closed
wants to merge 2 commits into from

Conversation

Fenn-CS
Copy link
Contributor
@Fenn-CS Fenn-CS commented Apr 6, 2017

Fix to #13154

…_password.php

Signed-off-by: Fenn-CS <fenn25.fn@gmail.com>
@codecov
Copy link
codecov bot commented Apr 6, 2017

Codecov Report

Merging #13157 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master   #13157      +/-   ##
==========================================
- Coverage   54.36%   54.36%   -0.01%     
==========================================
  Files         465      465              
  Lines       69692    69693       +1     
==========================================
  Hits        37888    37888              
- Misses      31804    31805       +1

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 0a135fa...894c40e. Read the comment docs.

@@ -84,6 +84,7 @@
*/
function PMA_getChangePassMessage($change_password_message, $sql_query = '')
{
global $response;
Copy link
Contributor

Choose a reason for hiding this comment

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

It probably rather should be $response = Response::getInstance();

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@nijel did not want to do that since I found $response = Response::getInstance(); earlier in the file.
https://github.com/phpmyadmin/phpmyadmin/blob/master/user_password.php#L22
But if that's what is better let me do it that way.

Signed-off-by: Fenn-CS <fenn25.fn@gmail.com>
@nijel nijel self-assigned this Apr 7, 2017
@nijel nijel added this to the 4.7.1 milestone Apr 7, 2017
@nijel
Copy link
Contributor
nijel commented Apr 7, 2017

Thanks, merged as a661c06 into QA_4_7 branch.

@nijel nijel closed this Apr 7, 2017
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.

None yet

2 participants