Go to main content

Textpattern CMS support forum

You are not logged in. Register | Login | Help

#13 2017-11-06 09:46:03

CeBe
Plugin Author
From: Caen - Fr
Registered: 2010-06-25
Posts: 345
Website

Re: Feedback to: Textpattern CMS 4.6.2 released

Back to this old bug : there is the same a few lines above, checking Mysql version to decide whether to “SET NAMES” or not. Is there a reason why it has not been fixed?

Offline

#14 2017-11-06 13:26:14

Bloke
Developer
From: Leeds, UK
Registered: 2006-01-29
Posts: 11,456
Website GitHub

Re: Feedback to: Textpattern CMS 4.6.2 released

CeBe wrote #307651:

Back to this old bug : there is the same a few lines above, checking Mysql version to decide whether to “SET NAMES” or not. Is there a reason why it has not been fixed?

Seems to be fixed in 4.7.0-dev. Guess it got missed in 4.6.2, oops. You can patch it:

if ($this->charset && (version_compare($version, '5') >= 0 || preg_match('#^4\.[1-9]#', $version))) {

Sorry about that.


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

#15 2018-03-13 15:45:03

jayrope
Plugin Author
From: Berlin
Registered: 2006-07-06
Posts: 687
Website Mastodon

Re: Feedback to: Textpattern CMS 4.6.2 released

I can confirm above patch by Bloke works on TXP 4.6.2, the entry in txplib_db.php to be replaced is on line 217.


A hole turned upside down is a dome, when there’s also gravity.

Offline

Board footer

Powered by FluxBB