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

php7 compatibility #335

Closed
dgirona opened this issue Oct 21, 2016 · 5 comments

Comments

@dgirona
Copy link

commented Oct 21, 2016

This theme cause errors on php7 execution : (

@Nikeo

This comment has been minimized.

Copy link
Collaborator

commented Oct 21, 2016

Hi @dgirona can you paste the error(s) you get in your next reply ?
We're on it. This will be pushed in the next days.
Thanks

@dgirona

This comment has been minimized.

Copy link
Author

commented Oct 21, 2016

In the last versión of the theme.

FILE: /www/wp-content/themes/hueman/functions/admin/class-admin-page.php

FOUND 1 ERROR AND 1 WARNING AFFECTING 2 LINES

307 | ERROR | Extension 'mysql_' is deprecated since PHP 5.5 and removed since PHP 7.0 - use mysqli instead.

315 | WARNING | INI directive 'safe_mode' is deprecated from PHP 5.3 and forbidden from PHP 5.4.

FILE: /www/wp-content/themes/hueman/functions.php

FOUND 1 ERROR AFFECTING 1 LINE

30 | ERROR | The use of function split is discouraged from PHP version 5.3 and forbidden from PHP version 7.0; use preg_split instead

Thanks : )

@Nikeo

This comment has been minimized.

Copy link
Collaborator

commented Oct 23, 2016

@dgirona
There's no split() function in hueman/functions.php, line 30.
Is it something that you've added ?

@Nikeo Nikeo added this to the v3.2.10 milestone Oct 23, 2016

Nikeo added a commit that referenced this issue Oct 23, 2016
@dgirona

This comment has been minimized.

Copy link
Author

commented Oct 24, 2016

Changed widt explode(). Now all works fine. Thanks Nikeo : )

@Nikeo

This comment has been minimized.

Copy link
Collaborator

commented Oct 24, 2016

ok. Thanks for the feedback.
the fix will be merged in v3.2.10

@Nikeo Nikeo added the fixed label Oct 24, 2016

@Nikeo Nikeo closed this Nov 4, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.