Page 1 of 1

kb incompatible with mod libevent installed

Posted: Sat Sep 01, 2018 16:17
by Tupsi
I realise this is not a bug per se, but just a compatibility issue, but it would be nice if you could ajust your code, so that it runs with this php mod installed.

Description:

Installing https://pecl.php.net/package/event and activating the mod in your php config breaks kb.

Steps to reproduce:

1) install libevent with 'pecl install event'
2) activate it with 'extension=event.so'
3) kb site starts throwing 500 web server errors.Logs says

Code: Select all

PHP message: PHP Fatal error:  Uncaught Error: Call to undefined method Event::register() in /var/www/killboard.xyz.net/common/xajax/xajax.php:12
4) remove the extension line from your php config, reload/restart the config and it works again

Re: kb incompatible with mod libevent installed

Posted: Sat Sep 29, 2018 16:57
by Salvoxia
Hi,

great find, thank you!
That should be addressable by adding a PHP namespace definition to the event class, so it can be addressed unambiguously.
I'll look into that.

Best Regards,
Salvoxia

Re: kb incompatible with mod libevent installed

Posted: Tue Dec 18, 2018 15:02
by Tupsi
Hello,

do you have any timeline planed getting this change in?

That other php site I am running would really love to have event enabled as php extension.

Re: kb incompatible with mod libevent installed

Posted: Fri Dec 21, 2018 18:12
by Salvoxia
Hi,

to be absolutely hones with you: I already started refactoring it for 4.4.2.0, when I realized this will be a massive incompatibility for all mods overriding killboard files.
But I hope I can take another shot at it over the Christmas holidays.

By the way: Happy holidays :)

Best Regards,
Salvoxia

Re: kb incompatible with mod libevent installed

Posted: Sat Mar 30, 2019 16:57
by Tupsi
Salvoxia wrote:
Fri Dec 21, 2018 18:12
But I hope I can take another shot at it over the Christmas holidays.
I just heard from the developer of the other php software I am using, that he will have no need of libevent in the future, so I will stop harrasing you for this change here as well :-)