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

error 500 when exporting - set time limit when $cfg['ExecTimeLimit'] used a string and not an int value #15725

Closed
williamdes opened this issue Dec 31, 2019 · 1 comment
Assignees
Labels
Bug A problem or regression with an existing feature
Projects
Milestone

Comments

@williamdes
Copy link
Member

Another user report that there is an error 500 when you try to export a database:

[28-Dec-2019 23:18:27 UTC] PHP Fatal error: Uncaught TypeError: set_time_limit() expects parameter 1 to be integer, string given in /usr/local/nginx/html/25218_mysqladmin106/libraries/classes/Util.php:4842
Stack trace:
#0 /usr/local/nginx/html/25218_mysqladmin106/libraries/classes/Util.php(4842): set_time_limit('28800')
#1 /usr/local/nginx/html/25218_mysqladmin106/export.php(333): PhpMyAdmin\Util::setTimeLimit()
#2 {main}
thrown in /usr/local/nginx/html/25218_mysqladmin106/libraries/classes/Util.php on line 4842

Didn't test that myself but you may want to check it...

Thanks

Originally posted by @pamamolf in #15200 (comment)

@williamdes williamdes added the Bug A problem or regression with an existing feature label Dec 31, 2019
@williamdes williamdes added this to Needs triage in issues via automation Dec 31, 2019
@williamdes williamdes added this to the 5.0.1 milestone Dec 31, 2019
@williamdes williamdes self-assigned this Dec 31, 2019
@williamdes williamdes changed the title 500 when exporting - set time limit error 500 when exporting - set time limit when $cfg['ExecTimeLimit'] used a string and not an int valu Dec 31, 2019
@williamdes williamdes changed the title error 500 when exporting - set time limit when $cfg['ExecTimeLimit'] used a string and not an int valu error 500 when exporting - set time limit when $cfg['ExecTimeLimit'] used a string and not an int value Dec 31, 2019
williamdes added a commit that referenced this issue Dec 31, 2019
Signed-off-by: William Desportes <williamdes@wdes.fr>
issues automation moved this from Needs triage to Closed Dec 31, 2019
@williamdes
Copy link
Member Author

@pamamolf now fixed.
Please read the documentation and use int for in values or you will break more things
But I still agree that we should handle it on our side

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 31, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug A problem or regression with an existing feature
Projects
issues
  
Closed
Development

No branches or pull requests

1 participant