Doesn't work with php7 and dokuwiki Release 2017-02-19b "Frusterick Manners"

Astrel / dokuwiki_plugin_logautherror

Plugin dokuwiki: when an authentification fail log in apache log_error. Use it with fail2ban !

http://www.dokuwiki.org/plugin:logAuthError

5 stars 5 forks source link

Doesn't work with php7 and dokuwiki Release 2017-02-19b "Frusterick Manners" #3

Open thorus90 opened 7 years ago

thorus90 commented 7 years ago

It throws php errors on the site.

robinvanleeuwen commented 7 years ago

Actually it trows warnings about the new registration method of plugins:

PHP Warning: Declaration of action_plugin_logautherror::register(&$controller) should be compatible with DokuWiki_Action_Plugin::register(Doku_Event_Handler $controller) in /var/www/kb/lib/plugins/logautherror/action.php on line 0, referer

The proposed pull request https://github.com/Astrel/dokuwiki_plugin_logautherror/pull/2 fixes that.... Please include these fixes!

thorus90 commented 7 years ago

Thanks, with that pull request the errors have gone away, either the update process in dokuwiki itself doesn't work for that plugin, or it that patch isn't included yet?