[phpmyadmin/phpmyadmin] ce49be: Fix condition to activate tracker

Branch: refs/heads/QA_4_7 Home: https://github.com/phpmyadmin/phpmyadmin Commit: ce49be7e070a4e2caab33314703f6f95c3cb0492 https://github.com/phpmyadmin/phpmyadmin/commit/ce49be7e070a4e2caab33314703f... Author: Michal Čihař <michal@cihar.com> Date: 2017-03-01 (Wed, 01 Mar 2017) Changed paths: M libraries/DatabaseInterface.php Log Message: ----------- Fix condition to activate tracker Invoking empty() on the result might work in most cases as false is empty and false is result from failure, but on the other side empty can be true for valid results as well. Signed-off-by: Michal Čihař <michal@cihar.com> Commit: bb95ecf852013869e6ffb4fef3a9bbbeba0ffdeb https://github.com/phpmyadmin/phpmyadmin/commit/bb95ecf852013869e6ffb4fef3a9... Author: Michal Čihař <michal@cihar.com> Date: 2017-03-01 (Wed, 01 Mar 2017) Changed paths: M libraries/dbi/DBIDummy.php M test/libraries/database_interface_test.php Log Message: ----------- Extend Dummy driver to be able to inject expected queries results Signed-off-by: Michal Čihař <michal@cihar.com> Commit: a72a048f1e22db9eaacf99c7c96ade5b0bde5c94 https://github.com/phpmyadmin/phpmyadmin/commit/a72a048f1e22db9eaacf99c7c96a... Author: Michal Čihař <michal@cihar.com> Date: 2017-03-01 (Wed, 01 Mar 2017) Changed paths: M libraries/DatabaseInterface.php M test/classes/DatabaseInterfaceTest.php Log Message: ----------- Correctly handle case when CURRENT_USER() fails It's not likely, but the code path was obviously returning wrong value. Maybe it did happen in issue #13037 Signed-off-by: Michal Čihař <michal@cihar.com> Compare: https://github.com/phpmyadmin/phpmyadmin/compare/dee50ad8d182...a72a048f1e22
participants (1)
-
Michal Čihař