Hello Community,
Korvell noticed problems on loggin out, and it turned out that these are caused by cmsimple/classes/Backup.php not being PHP 4 compatible. Inadvertently, I've used the "private" keyword for the instance variables, which is allowed only since PHP 5. This causes a parse error on PHP 4, and results in a blank window. The undesired use of "private" slipped through PHP_CompatInfo, because of a bug that I have reported in the meantime.
I have made a patch available that contains the necessary replacement of cmsimple/classes/Backup.php.
Christoph
PS: I have furthermore checked for uses of "private" in the other files; everything else is fine.
XH 1.6.2 requires PHP 5
XH 1.6.2 requires PHP 5
Last edited by cmb on Sun Jun 15, 2014 8:57 pm, edited 1 time in total.
Reason: added PS
Reason: added PS
Christoph M. Becker – Plugins for CMSimple_XH
Re: XH 1.6.2 requires PHP 5
Just had the same problem on the site of parken, which was php 4.4,
logout didn't work and the pagemanager also wouldn't open all pages.
Anyhow, problem solved on changing the server to php 5.4
logout didn't work and the pagemanager also wouldn't open all pages.
Anyhow, problem solved on changing the server to php 5.4
Re: XH 1.6.2 requires PHP 5
Hm, do you mean the expand/collapse buttons didn't work? I'll have a look at it later.svasti wrote:pagemanager also wouldn't open all pages.
Great! Should also bring some performance improvements.svasti wrote:Anyhow, problem solved on changing the server to php 5.4
Christoph M. Becker – Plugins for CMSimple_XH
Re: XH 1.6.2 requires PHP 5
yes. Interesting, nobody complained about it. This means to me that users who are active in updating their system also take care to use an up-to-date php version. So we shouldn't meet any opposition on raising the minimum requirement php version to 5.2, which is still pretty low.cmb wrote:Hm, do you mean the expand/collapse buttons didn't work? I'll have a look at it later.
Re: XH 1.6.2 requires PHP 5
Thanks for the fast confirmation. I had a quick look at it, and I'm suprised though, because no PHP is involved in the functionality. I suppose that the problem is related to http://cmsimpleforum.com/viewtopic.php?f=12&t=7902.svasti wrote:yes.cmb wrote:Hm, do you mean the expand/collapse buttons didn't work? I'll have a look at it later.
ACK, especially because PHP 7 seems to be on its way, and I'd rather avoid to support 3 major PHP versions.svasti wrote:Interesting, nobody complained about it. This means to me that users who are active in updating their system also take care to use an up-to-date php version. So we shouldn't meet any opposition on raising the minimum requirement php version to 5.2, which is still pretty low.
Christoph M. Becker – Plugins for CMSimple_XH