- Issue created by @somebodysysop
- 576aabd7 committed on 8.x-1.x
Issue #3485128: Too few arguments to function: AutobanController
- 576aabd7 committed on 8.x-1.x
- πΊπ¦Ukraine goodboy Kharkiv, Ukraine
@somebodysysop, thank you for report.
Please, check on the latest dev version. - πΊπΈUnited States somebodysysop
Yes, thank you. Installed the latest dev version and now the dblog links work as expected.
One difference between 8.x-1.10 and dev that I noticed, in case it helps:
In 8.x-1.10, whitelisted IPs in the dblog do NOT have the Advanced Ban, Advanced Ban (range), Core Ban links.
In dev version, the links DO appear next to the whitelisted IPs.
- πΊπ¦Ukraine goodboy Kharkiv, Ukraine
@somebodysysop, thanks for testing.
I can't reproduce the whitelist issue for Drupal 10.1.6 and 11 versions.The logic for adding the Autobank links is:
$ip = $dblog->hostname; if (!empty($ip) && $autobanController->canIpBan($ip)) { // Retrieve Autoban Ban Providers list. $providers = []; $banManagerList = $autobanController->getBanProvidersList(); if (!empty($banManagerList)) { $destination = $this->getDestinationArray(); foreach ($banManagerList as $id => $item) { $url_item = Url::fromRoute('autoban.direct_ban', [ 'ips' => $ip, 'provider' => $id, ], [ 'query' => [ 'destination' => $destination['destination'], ], ]); $url_link = Link::fromTextAndUrl($item['name'], $url_item); $providers[$id] = $url_link->toString(); } } }
The main checking is
autobanController->canIpBan($ip)
. It didn't change after 8.1.10.
The function calls the whitelist checking function:if ($this->whitelistIp($ip)) { return FALSE; }
Perhaps in your particular case this function is failing and needs to be fixed.