[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: make gosa packages php7 compliant



On Wed, Jun 01, 2016 at 11:51:57PM +0200, Wolfgang Schweer wrote:
> On Wed, Jun 01, 2016 at 07:27:15PM +0200, Wolfgang Schweer wrote:
> > On Wed, Jun 01, 2016 at 03:13:45PM +0200, Wolfgang Schweer wrote:
> > > On Wed, Jun 01, 2016 at 11:00:28AM +0000, Mike Gabriel wrote:
> > > > I have just uploaded gosa 2.7.4+reloaded2-11 to unstable. That new 
> > > > upload contains two new patches (1025, 1026).
> > > > 
> > > > 1025 -> fix when running under smarty 3.1.29-1 from experimental
> > > > 1026 -> replace all deprecated-style constructor methods
> > > > 
> > > > @Wolfgang: can you please confirm that gosa works with smarty 3.1.29-1 
> > > > after the fix via patch 1025? I would love to push 3.1.29-2 into 
> > > > unstable.
> > >  
> > > Will do so as soon as gosa 2.7.4+reloaded2-11 shows up on the mirrors.
> >  
> > With smarty3 from experimental and gosa 2.7.4+reloaded2-11 from unstable 
> > only a blank index page is shown after logging in (both firefox-esr and 
> > chromium).
> 
> After running 'a2enmod php7.0' the blank page showed an error message.
> 
> This fix worked for me (in /usr/share/gosa/include/)
> 
> --- a/class_acl.inc	2016-06-01 23:20:58.314147422 +0200
> +++ b/class_acl.inc	2016-06-01 23:21:21.397742892 +0200
> @@ -60,7 +60,7 @@
>      var $roleList = NULL;
>      var $aclMemberList = NULL;
>  
> -    function __construct(&$config, $parent, $dn= NULL)
> +    function construct(&$config, $parent, $dn= NULL)
>      {
>          /* Include config object */
>          plugin::plugin($config, $dn);

This wasn't even a workaround, but:

As far as I was able to find out, the general error can be seen looking 
at the last code line:

plugin::plugin($config, $dn);

All constructor names have __construct after applying the fix script, so in this case it should rather be

plugin::__construct($config, $dn);
 
Same for all other occurencies of $name:$name, I guess.

Wolfgang

Attachment: signature.asc
Description: Digital signature


Reply to: