Textpattern CMS support forum
You are not logged in. Register | Login | Help
- Topics: Active | Unanswered
Offline
Re: Testers needed: PHP 8 compatibility
bici wrote #327535:
To be clear I downloaded and tried to install
github.com/textpattern/textpattern/archive/4.8.5.zip
I’m guessing it’s the symlink in the /sites folder (see that folder in GitHub) that’s the reason. If you’re not doing a multisite installation, you don’t need that folder.
If I recall correctly, symlinks a) don’t zip well and don’t transfer via FTP (you need SFTP or SSH). In the public releases, that folder is I believe only in the tar.gz archive.
TXP Builders – finely-crafted code, design and txp
Offline
Re: Testers needed: PHP 8 compatibility
Topic split to forum.textpattern.com/viewtopic.php?pid=327544#p327544
Last edited by gaekwad (2020-12-18 12:33:38)
Offline
Re: Testers needed: PHP 8 compatibility
Because the new 4.8.5 version seems to be imminent, here is the error messages I got after an upgrade to the dev version (https://github.com/textpattern/textpattern/tree/5b579f7aa1589b12573d542e44e30309ace8f42e) with PHP 8 on a server in production:
Fatal error: Uncaught Error: Undefined constant "MYSQLI_CLIENT_FOUND_ROWS" in /home/{hidden}/public_html/textpattern/lib/txplib_db.php:169 Stack trace: #0 /home/{hidden}/public_html/textpattern/lib/txplib_db.php(283): DB->__construct() #1 /home/{hidden}/public_html/textpattern/index.php(63): include('/home/{hidden}/...') #2 {main} thrown in /home/{hidden}/public_html/textpattern/lib/txplib_db.php on line 169Patrick.
Github | CodePen | Codier | Simplr theme | Wait Me: a maintenance theme | [\a mi.ni.ma]: a “Low Tech” simple Blog theme.
Offline
Re: Testers needed: PHP 8 compatibility
Pat64 wrote #327906:
Undefined constant “MYSQLI_CLIENT_FOUND_ROWS”
Weird. According to the docs on mysql_real_connect that constant is still valid as far as PHP 7 is concerned. No idea about PHP 8, but it’d be odd for them to axe it without at least mentioning it’s been deprecated in the PHP 7 docs.
What does your PHP info report? If you can’t get into Admin>Diagnostics>PHP Configuration then perhaps write an executable test file in the root of your server:
<?php
echo phpinfo();and run that in your browser. Post anything relevant please (PM me if you don’t want to post it publicly). I’m just looking for anything mysqli-related that may indicate what’s going on here. Thanks.
The smd plugin menagerie — for when you need one more gribble of power from Textpattern. Bleeding-edge code available on GitHub.
Txp Builders – finely-crafted code, design and Txp
Offline
Re: Testers needed: PHP 8 compatibility
Further reading: p319953
Could be a hosting-side issue based on that link above.
Offline
Re: Testers needed: PHP 8 compatibility
Nice catch thanks. I thought this smelled familiar.
The smd plugin menagerie — for when you need one more gribble of power from Textpattern. Bleeding-edge code available on GitHub.
Txp Builders – finely-crafted code, design and Txp
Offline
Re: Testers needed: PHP 8 compatibility
Thank you Stef. E-mail send.
Patrick.
Github | CodePen | Codier | Simplr theme | Wait Me: a maintenance theme | [\a mi.ni.ma]: a “Low Tech” simple Blog theme.
Offline
Re: Testers needed: PHP 8 compatibility
Follow-up. From the output of phpinfo() it appears mysqli is not installed on Pat64’s hosting instance. That would certainly knacker things up!
I wonder if this is commonplace on some hosts or if it’s the start of a general trend?
Last edited by Bloke (2020-12-31 12:48:48)
The smd plugin menagerie — for when you need one more gribble of power from Textpattern. Bleeding-edge code available on GitHub.
Txp Builders – finely-crafted code, design and Txp
Offline
Re: Testers needed: PHP 8 compatibility
Assuming Diagnostics still shows in some capacity, perhaps 1309 might be worth further investigation. An upfront “system requirements check”, either as part of the pre-flight and/or included with the diagnostic info, with current status for each thing would be a sensible move.
Offline
Re: Testers needed: PHP 8 compatibility
gaekwad wrote #327932:
Assuming Diagnostics still shows in some capacity
Sadly it doesn’t. If mysqli isn’t installed, we can’t even make the connection to the database and it bails really ungracefully. Not really sure how to detect this, bar something called really really really high up in the stack to check system requirements: like maybe where we check the PHP version which I believe is in janitor(). Would have to check.
The smd plugin menagerie — for when you need one more gribble of power from Textpattern. Bleeding-edge code available on GitHub.
Txp Builders – finely-crafted code, design and Txp
Offline
Re: Testers needed: PHP 8 compatibility
Bloke wrote #327933:
Sadly it doesn’t. If mysqli isn’t installed, we can’t even make the connection to the database.
Oh yeah, of course! Off to dunce jail for me.
Offline