6.0.0-beta1
7/4/25

[#14777] compatibility with php 7.2
Summary compatibility with php 7.2
Queue Horde Framework Packages
Type Bug
State Not A Bug
Priority 1. Low
Owners
Requester mail (at) webthatworks (dot) it
Created 02/04/2018 (2707 days ago)
Due
Updated 02/24/2020 (1957 days ago)
Assigned
Resolved 02/24/2020 (1957 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch Yes

History
02/24/2020 01:29:42 AM Michael Rubinsky State ⇒ Not A Bug
Priority ⇒ 1. Low
 
01/11/2019 01:32:36 PM ssri_abo (at) u-paris2 (dot) fr Comment #4 Reply to this comment
Issue happens when using an http_login customized alternate login 
page. Moving Horde_Registry::appInit at the beginning of the 
customized login page solved it.

[Show Quoted Text - 16 lines]
01/10/2019 01:50:48 PM ssri_abo (at) u-paris2 (dot) fr Comment #3 Reply to this comment
Since 7.2 , got these errors :

HORDE PHP ERROR: ini_set(): A session is active. You cannot change the 
session module's ini settings at this time [pid 14215 on line 159 of 
"/usr/share/horde/pear/php/Horde/Session.php"]
HORDE PHP ERROR: session_set_cookie_params(): Cannot change session 
cookie parameters when session is active [pid 14215 on line 175 of 
"/usr/share/horde/pear/php/Horde/Session.php"]
HORDE PHP ERROR: session_cache_limiter(): Cannot change cache limiter 
when session is active [pid 14215 on line 177 of 
"/usr/share/horde/pear/php/Horde/Session.php"]
HORDE PHP ERROR: session_name(): Cannot change session name when 
session is active [pid 14215 on line 178 of 
"/usr/share/horde/pear/php/Horde/Session.php"]


02/04/2018 12:56:39 PM mail (at) webthatworks (dot) it Comment #2 Reply to this comment
Unfortunately count() is used in too many places expecting an array 
when it is not always the case.

A more global solution should be considered other than wrapping every 
count in an if(is_array())

Too many hidden things seems fail. And waiting errors show up in the 
log doesn't seem a good way to fix this.

I'll revert to an older version of php.
02/04/2018 12:01:38 AM mail (at) webthatworks (dot) it Comment #1
Priority ⇒ 2. Medium
Patch ⇒ Yes
Milestone ⇒
Queue ⇒ Horde Framework Packages
Summary ⇒ compatibility with php 7.2
Type ⇒ Bug
State ⇒ Unconfirmed
Reply to this comment
Hi,

there are 2 minor incompatibilities with php 7.2

1)
HORDE: PHP ERROR: ini_set(): Cannot set 'user' save handler by 
ini_set() or session_module_name() [pid 2887 on line 95 of 
"/usr/share/php/Horde/SessionHandler.php"]

line 95 can simpli be deleted

2)
HORDE: PHP ERROR: count(): Parameter must be an array or an object 
that implements Countable [pid 7442 on line 371 of 
"/usr/share/php/Horde/ActiveSync/State/Sql.php"]

         if(is_array($this->_changes)) {
           $c = count($this->_changes);
         } else {
           $c = 0;
         }
$this->_logger->meta(
             sprintf('STATE: Saving state: %s',
                 serialize(array(
                     $params['sync_key'],
                     $params['sync_data'],
                     $params['sync_devid'],
                     $params['sync_mod'],
                     $params['sync_folderid'],
                     $params['sync_user'],
                     $c,
                     time()))
                 )
             );

I hope I can publish a cleaner patch here

https://github.com/Ivan-SB/Db



Saved Queries