Skip to content

Fix for HHVM compatibility#1530

Closed
bygiro wants to merge 2 commits intophpmyadmin:masterfrom
bygiro:patch-1
Closed

Fix for HHVM compatibility#1530
bygiro wants to merge 2 commits intophpmyadmin:masterfrom
bygiro:patch-1

Conversation

@bygiro
Copy link

@bygiro bygiro commented Mar 4, 2015

this will fix the error with HHVM:

Fatal error: Uncaught exception 'Exception' with message 'Port is not numeric'

Signed-off-by

bygiro added 2 commits March 4, 2015 16:48
this will fix the error with HHVM:

Fatal error: Uncaught exception 'Exception' with message 'Port is not numeric'
@lem9
Copy link
Contributor

lem9 commented Mar 5, 2015

Please read https://github.com/phpmyadmin/phpmyadmin/blob/master/DCO to see what a Signed-Off-By line should look like.

@bygiro
Copy link
Author

bygiro commented Mar 5, 2015

Hello,
sorry, but I do not have time to waste, and I already wasted too much time for a PR.....do whatever you want.

@bygiro bygiro closed this Mar 5, 2015
@bygiro bygiro deleted the patch-1 branch March 5, 2015 14:11
nijel added a commit that referenced this pull request Mar 5, 2015
The port always should be integer.

See PR #1530

Signed-off-by: Michal Čihař <michal@cihar.com>
@nijel
Copy link
Contributor

nijel commented Mar 5, 2015

Anyway forcing the conversion in all cases does not sound like a good idea, the code should rather fail if it gets something unexpected. In 2b23430 I've implemented handling of single value which can cause problems.

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.

4 participants