Логику проверки необходимости плагина в dlog_hero\DlogHeroPluginManager переписать на Condition Plugin

Есть менеджер плагинов: https://gitlab.com/Niklan/dlog/blob/master/web/modules/custom/dlog_hero/src/Plugin/DlogHeroPluginManager.php Можно заменить 151-162 строку на этот код:

   $is_plugin_needed = ($is_match_path && $plugin['match_type'] !== 'unlisted');

Так будет правильнее. Также, xor возвращает true если, например, $is_match_path == 0, а $plugin['match_type'] === 'unlisted' что также можно рассматривать как баг.

Edited Nov 16, 2018 by Nikita Malyshev (Niklan)
Assignee Loading
Time tracking Loading